Skip to content

fix: register FarcasterActionProvider in main.ts#104

Open
mehmetkr-31 wants to merge 1 commit intobase:masterfrom
mehmetkr-31:fix/register-farcaster-action-provider
Open

fix: register FarcasterActionProvider in main.ts#104
mehmetkr-31 wants to merge 1 commit intobase:masterfrom
mehmetkr-31:fix/register-farcaster-action-provider

Conversation

@mehmetkr-31
Copy link
Copy Markdown

Summary

  • Add missing factory export to FarcasterActionProvider
  • Register farcasterActionProvider in main.ts action providers array

Problem

The farcaster_username tool was added in #57 and refactored to the ActionProvider pattern in #75, but two things were dropped during the refactor:

  1. No factory export in src/tools/farcaster/index.ts (every other provider has one)
  2. Not imported or registered in src/main.ts actionProviders array

This means the farcaster_username tool has been dead code since v1.0.13 — it doesn't appear in the MCP tools list regardless of NEYNAR_API_KEY configuration.

Changes

src/tools/farcaster/index.ts — Add factory export following the existing pattern:

export const farcasterActionProvider = () =>
  new FarcasterActionProvider();

src/main.ts — Import and register:

import { farcasterActionProvider } from './tools/farcaster/index.js';
// ...
actionProviders: [
  // ...existing providers...
  farcasterActionProvider(),
],

Verification

  • tsc --noEmit passes (zero errors)
  • eslint passes (zero warnings)
  • prettier passes (no changes)

Fixes #103

The farcaster_username tool was added in PR base#57 and refactored to the
ActionProvider pattern in PR base#75, but was never registered in main.ts
after the refactor. This made the tool completely inaccessible to MCP
clients.

- Add factory export to src/tools/farcaster/index.ts
- Import and register farcasterActionProvider in src/main.ts

Fixes base#103
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: FarcasterActionProvider not registered in main.ts after ActionProvider refactor

1 participant