Skip to content

fix: hide the upstream Claude Agent provider - #467

Merged
shuv1337 merged 3 commits into
integration-v2from
cursor/hide-claude-agent-e884
Oct 30, 2026
Merged

shuv1337 merged 3 commits into
integration-v2from
cursor/hide-claude-agent-e884

Conversation

@shuv1337

Copy link
Copy Markdown
Owner

Issue for this PR

No linked issue. Hide the upstream Claude Agent SDK provider so it cannot appear in shuvcode.

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

The upstream Claude Agent SDK tree stays on disk (packages/core/src/plugin/provider/claude-agent/** and its tests) so a later sync does not have to resurrect a deleted tree. It is not a product provider. Anthropic stays @shuvcode/claude-plugin (ClaudePlugin).

  • ProviderPlugins no longer imports or lists ClaudeAgentPlugin, and drops plugin id opencode.provider.claude-agent if a merge puts it back.
  • The exported ClaudeAgentPlugin is a no-op. The on-disk tests still call make.
  • The app connect setup dialog, popular lists, settings note, and Claude Agent copy are gone. Catalog helpers drop id claude-agent from integrations, providers, and voice model lists.
  • The TUI connect menu and model picker, and CLI auth login/list, drop the same id.
  • AGENTS.md and PLAN-fresh-v2-rewrite.md tell future merges not to register it again.

The optional @anthropic-ai/claude-agent-sdk peer and #claude-agent-sdk alias stay so the dormant source still typechecks. They are not wired to a selectable provider.

How did you verify your code works?

  • bun typecheck in packages/core, packages/app, packages/tui, and packages/cli. The pre-push hook typechecked the workspace: 40 tasks successful.
  • packages/core/test/plugin/bundled-subscriptions.test.ts: ClaudePlugin is in ProviderPlugins and still registers Anthropic claude-pro-max. Plugin id opencode.provider.claude-agent is absent. Running the product ClaudeAgentPlugin export does not register an integration or a provider.
  • On-disk packages/core/test/plugin/provider/claude-agent/plugin.test.ts still passes (19 tests) through make.
  • App order.test.ts, TUI integration-options.test.ts, CLI auth-login-choices.test.ts, and voice/tools.test.ts pass.
  • Did not click through a running app. Connect, settings, and the model picker render the filtered catalogs those tests cover.

Visual

<svg xmlns="http://www.w3.org/2000/svg" width="240" height="72" viewBox="0 0 240 72" fill="none">
  <g fill="currentColor">
    <rect x="20" y="14" width="200" height="10" rx="2" opacity="0.9"/>
    <rect x="20" y="48" width="168" height="10" rx="2" opacity="0.9"/>
    <rect x="20" y="31" width="140" height="10" rx="2" opacity="0">
      <animate attributeName="opacity" values="0;0;0.4;0;0" keyTimes="0;0.2;0.45;0.7;1" dur="3.2s" repeatCount="indefinite"/>
    </rect>
  </g>
</svg>

Screenshots / recordings

No live recording. The menus are the filtered lists covered above.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR
Open in Web Open in Cursor 

Keep the SDK tree on disk for sync, but do not register it or show it in
connect, catalog, settings, TUI, or CLI menus. Anthropic stays on ClaudePlugin.
Skip claude-agent when config builds providers, and share isHiddenProvider
across app, TUI, and CLI lists so an old server cannot show it.
@shuv1337
shuv1337 merged commit d375986 into integration-v2 Oct 30, 2026
5 checks passed
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.

2 participants