Skip to content

fix(server): preserve merged PR badges after branch deletion - #6216

Merged
t3dotgg merged 2 commits into
pingdotgg:mainfrom
tris203:t3code/preserve-merged-pr-badge
Aug 24, 2026
Merged

t3dotgg merged 2 commits into
pingdotgg:mainfrom
tris203:t3code/preserve-merged-pr-badge

test(server): include PR update timestamp in fixture

73f6c59
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Effect Service Conventions succeeded Aug 23, 2026 in 32s

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 want to check the changes made to GitManager.ts, so I'll look at that section of the file.

1. browse_code — apps/server/src/git/GitManager.ts

2. git_tools — readConfigValue

💭 I notice the whole check is wrapped so any failure in reading the config value defaults to false, which conservatively keeps the lookup — same safe fallback as before. I need to verify whether readConfigValue actually fails when the key is missing or just returns null, so I should check GitV's implementation.

3. browse_code — apps/server/src/vcs/GitVcsDriverCore.ts

💭 The allowNonZeroExit flag means a missing key just returns null rather than failing, which seems correct. The Effect.gen and orElseSucceed usage follows existing conventions without adding new imports or services, and the test file properly uses it.effect with a justified behavior-change test. I want to double check the imports in that test file next. Since I don't see any actual issues, I'm leaning toward reporting everything as clear, but I need to reconcile that with the required JSON output format — I'll structure the response as JSON indicating a clean result with a summary noting no findings.


Agent Credits: 25 credits