Skip to content

feat(claude): show SDK prompt suggestions as ghost text in the composer - #8524

Closed
getbrikly wants to merge 1 commit into
pingdotgg:mainfrom
getbrikly:feat/claude-prompt-suggestions
Closed

getbrikly wants to merge 1 commit into
pingdotgg:mainfrom
getbrikly:feat/claude-prompt-suggestions

feat(claude): show SDK prompt suggestions as ghost text in the composer

1894b8e
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Effect Service Conventions succeeded Aug 28, 2026 in 41s

All clear

All clear

Details

Note

Your check run agent prompt is: .macroscope/check-run-agents/effect-service-conventions.md
More information about how Check Run Agents work can be found in our Help Center.


🔍 Agent activity — 5 tool calls

💭 I'm looking at the diff to check for Effect service convention violations, scanning through the changed files across mobile, server orchestration, provider adapters, and web components. This appears to be a feature addition for prompt suggestions rather than an Effect service refactor, so the conventions may not strictly apply here—but I still want to verify how Effect, Stream, Fiber, and Random are imported in the ClaudeAdapter test to make sure they're consolidated from "effect" properly.

1. git_tools — ^import",

2. browse_code — apps/web/src/lib/promptSuggestion.ts

💭 That helper looks fine on its own. Now I'm checking the imports in the touched Effect service files like ClaudeAdapter.ts and ProviderRuntimeIngestion.ts to see if the diff needs to add any there, and verifying the test file consolidates its imports from "effect".

3. browse_code — apps/server/src/provider/Layers/ClaudeAdapter.test.ts

4. browse_code — packages/contracts/src/providerRuntime.ts

💭 The ClaudeAdapter.ts changes look consistent with existing patterns, just adding a mutable session field and an Effect.fn handler without introducing new services or errors. I'm now checking whether ChatComposer.tsx and ComposerPromptEditor.tsx follow the repo's import conventions, since they're plain React rather than Effect services. The mutable field on the context isn't wrapped in an Effect Ref, so I want to check ClaudeSessionContext for other mutable fields to see if that's the established pattern here.

5. browse_code — apps/server/src/provider/Layers/ClaudeAdapter.ts

💭 The other changes look fine too - the ingestion function addition is a straightforward pure function case, and the React components don't introduce any Effect runtime concerns. I don't see any runPromise or Layer.succeed additions in the diff, so this review comes back clean.


Agent Credits: 38 credits

Token usage: 8 uncached input · 104,068 cache-read input · 40,274 cache-write input · 1,995 output
Agent Credits may also include non-token charges from external tools such as web research.