Skip to content

fix(chat): show revert after network_error - #7983

Draft
ImBIOS wants to merge 3 commits into
pingdotgg:mainfrom
ImBIOS:fix-chat-show-revert-after-network-error
Draft

fix(chat): show revert after network_error#7983
ImBIOS wants to merge 3 commits into
pingdotgg:mainfrom
ImBIOS:fix-chat-show-revert-after-network-error

fix(chat): show revert after network_error finish_reason

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

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 — 2 tool calls

💭 I'm reviewing the diff and see it adds a shared stripTerminalEscapes helper module along with sanitization at various call sites and a ChatView.tsx change. Nothing here creates or relocates an Effect service, so I'm checking whether any Effect-related conventions were actually violated across the touched files. The added imports are named imports from a shared package for pure helper functions, which is fine per convention. Looking at OpenCodeProvider's checkOpenCodeProviderStatus using Effect.fn, that also checks out. This isn't an Effect service change, so I don't need to dig deeper — just confirming the touched lines don't introduce any consolidated effect imports or other convention issues. Nothing stands out as concerning, so I'm settling on a clean result and will note that everything checks out.

1. git_tools — from "effect"

2. git_tools — stripTerminalEscapes|sanitizeTerminalValue

💭 No Effect service convention issues here—this looks clean.


Agent Credits: 24 credits