Skip to content

Add Claude Code provider integration with provider-aware auth and rich-content support #563

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
blas0 wants to merge 2 commits into from

Merge remote-tracking branch 'origin/main' into claude-pr-sync

8f28836
Select commit
Loading
Failed to load commit list.
Closed

Add Claude Code provider integration with provider-aware auth and rich-content support #563

Merge remote-tracking branch 'origin/main' into claude-pr-sync
8f28836
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Correctness Check completed Mar 8, 2026 in 7m 22s

4 issues identified (147 code objects reviewed).

• Merge Base: c97c6b7
• Head: 8f28836

Details

✅ File Path Comments Posted
✅ apps/web/src/providerAuthGuidance.ts 0
➖ packages/contracts/src/model.test.ts
➖ README.md
➖ apps/web/src/routes/_chat.settings.test.ts
❌ apps/server/src/claudeCodeServerManager.ts 2
➖ apps/server/src/claudeCodeServerManager.test.ts
✅ apps/web/src/components/ChatView.providerOptions.ts 0
✅ apps/server/src/provider/Services/ClaudeCodeAdapter.ts 0
❌ apps/server/src/provider/Layers/ClaudeCodeAdapter.ts 2
➖ apps/web/src/components/ChatView.providerOptions.test.ts
➖ apps/server/src/provider/Layers/ClaudeCodeAdapter.test.ts
✅ packages/contracts/src/provider.ts 0
✅ apps/server/src/orchestration/decider.ts 0
✅ apps/server/src/serverLayers.ts 0
✅ packages/contracts/src/providerRuntime.ts 0
✅ apps/web/src/appSettings.ts 0
➖ packages/contracts/src/provider.test.ts
➖ apps/web/src/appSettings.test.ts
➖ packages/contracts/src/orchestration.test.ts
➖ packages/shared/src/model.test.ts
✅ packages/shared/src/model.ts 0
✅ packages/contracts/src/model.ts 0
➖ apps/server/src/provider/Layers/ProviderService.test.ts
➖ apps/server/src/provider/Layers/ProviderSessionDirectory.test.ts
➖ apps/web/src/store.test.ts
✅ packages/contracts/src/orchestration.ts 0
✅ apps/server/src/provider/Layers/ProviderAdapterRegistry.ts 0
✅ apps/server/src/provider/Layers/ProviderService.ts 0
➖ apps/server/src/provider/Layers/ProviderHealth.test.ts
✅ apps/server/src/provider/Layers/ProviderSessionDirectory.ts 0
✅ apps/web/src/composerDraftStore.ts 0
✅ apps/web/src/store.ts 0
➖ apps/web/src/session-logic.test.ts
➖ apps/server/src/orchestration/Layers/ProviderRuntimeIngestion.test.ts
✅ apps/web/src/components/ChatMarkdown.tsx 0
➖ apps/server/src/orchestration/Layers/ProviderCommandReactor.test.ts
✅ apps/web/src/components/ChatView.browser.tsx 0
➖ apps/web/src/composerDraftStore.test.ts
✅ apps/web/src/session-logic.ts 0
➖ apps/server/src/provider/Layers/ProviderAdapterRegistry.test.ts
✅ apps/server/src/orchestration/Layers/ProviderCommandReactor.ts 0
✅ apps/web/src/routes/_chat.settings.tsx 0
✅ apps/server/src/provider/Layers/ProviderHealth.ts 0
✅ apps/web/src/components/ChatView.tsx 0

Filtered Issues Details

apps/server/src/provider/Layers/ClaudeCodeAdapter.ts
  • line 175: The mapItemLifecycle function incorrectly maps the tool payload data. It assigns the raw event.payload to data, but event.payload is a wrapper object containing an item property (as evidenced by const item = asObject(payload?.item)). Canonical item types like command_execution (mapped from Bash) typically expect a flat data object with properties like command or cwd, not a nested { item: { ... } } structure. This mismatch will likely cause UI components to fail to render tool details because they will be looking for properties on the wrong object level. [ Out of scope (triage) ]
apps/web/src/session-logic.ts
  • line 21: Enabling claudeCode by setting available: true will cause a runtime crash in the ProviderModelPicker component. The type definition ProviderPickerKind (line 13) explicitly adds "claudeCode" to ProviderKind, implying claudeCode is not part of the ProviderKind union. The component's modelOptionsByProvider prop is typed as Record<ProviderKind, ...>, meaning it is not guaranteed (and likely not constructed) to contain an entry for claudeCode. When the UI iterates over available options, accessing props.modelOptionsByProvider["claudeCode"] will return undefined, and the subsequent call to .map(...) will throw an exception. [ Out of scope (triage) ]
View more details on MacroscopeApp