Skip to content

fix(client-runtime): refresh edited pull request comments - #8094

Merged
juliusmarminge merged 2 commits into
pingdotgg:mainfrom
Bil0000:t3code/fix-comment-update-stale-text
Aug 29, 2026
Merged

juliusmarminge merged 2 commits into
pingdotgg:mainfrom
Bil0000:t3code/fix-comment-update-stale-text

Merge branch 'main' into t3code/fix-comment-update-stale-text

9b0b424
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Effect Service Conventions succeeded Aug 29, 2026 in 31s

Effect Service Conventions: 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 — 3 tool calls

💭 I'm examining the diff, which adds a test file and modifies pullRequests.ts, checking whether the import from "effect/unstable/reactivity" counts as a consolidated named import subject to the same convention flag as imports from "effect" directly. The new test verifies a genuine behavior change—refreshing activity after a comment update—so it's a legitimate focused test rather than one added just to validate a mechanical refactor. I'll check how the rest of the repo handles this.

1. browse_code — packages/client-runtime/src/state/pullRequests.ts

2. git_tools — effect/unstable/reactivity

💭 The test's import style matches repo conventions, and since this is a behavior change rather than a mechanical refactor, adding a test is justified. I want to double-check createEnvironmentRpcCommand's onSuccess signature and confirm executeAtomQuery exists in runtime.ts before finalizing.

3. git_tools — onSuccess|executeAtomQuery

💭 Everything checks out against existing conventions—the test uses proper namespace imports, Layer.succeed for test seams, and dependency injection patterns that match the codebase style. With no findings, this should resolve as success. I need to fit the response into the required JSON schema while still conveying that everything is clear in the title and summary fields.


Agent Credits: 34 credits

Token usage: 6 uncached input · 61,573 cache-read input · 40,808 cache-write input · 1,296 output
Agent Credits may also include non-token charges from external tools such as web research.