Skip to content

perf(server): stop retaining unused OpenCode tool history - #9684

Merged
t3dotgg merged 1 commit into
mainfrom
t3code/perf-opencode-history-retention
Sep 4, 2026
Merged

t3dotgg merged 1 commit into
mainfrom
t3code/perf-opencode-history-retention

perf(server): stop retaining unused OpenCode tool history

ecc5669
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Effect Service Conventions succeeded Sep 4, 2026 in 4m 52s

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 — 1 tool call

💭 This diff strips out turn snapshot tracking entirely -- removing the turns field, resolveTurnSnapshot, appendTurnItem, and its call site. It's purely a removal, so I should check whether OpenCodeTurnSnapshot becomes unused or if other code still references context.turns before assuming this violates any service pattern.

1. browse_code — In apps/server/src/provider/Layers/OpenCodeAdapter.ts, where is OpenCodeTurnSnapshot type declared and used? Also show all usages of context.turns, turns, resolveTurnSnapshot, appendTurnItem in the repo. Show surrounding code for lines ~330-360 and ~470-500 and ~2330-2350.

💭 The removal looks clean — the helpers and field are fully gone, this is just dead in-memory retention being removed, and I don't see any Effect service convention violations. Since there's nothing else changed, there are no findings, so I'll report "All clear" as both the title and summary.


Agent Credits: 104 credits

Token usage: 22 uncached input · 299,926 cache-read input · 95,222 cache-write input · 9,554 output
Agent Credits may also include non-token charges from external tools such as web research.