opencode-context-indicator 1.8.2 → 1.8.3

package/CHANGELOG.md

@@ -4,6 +4,33 @@ Notable user-facing changes per release. Dates are commit days (UTC).
 The package also ships a dual V1/V2 plugin — see the README for the surface
 matrix; everything below concerns the V2 path unless stated otherwise.
 
+## 1.8.3 — 2026-11-03
+
+- **A deleted session disappears immediately.** `session.deleted` now removes the
+  session's `state.json` entry - and every entry under it in the `parentID` chain -
+  instead of leaving it to the 7-day tombstone. The row used to stay in
+  `/context-breakdown`, the Desktop tab and the `/cx` picker for up to a week, and
+  a phantom row has no step event left for the phantom filter to catch. Removed
+  IDs are remembered for the lifetime of the instance (the host never reuses an
+  ID), so a late write cannot recreate the row: without that barrier a child that
+  was still running when its parent was deleted came back as a phantom ROOT,
+  because deleting the parent had already wiped its attribution cache. The same
+  barrier covers a child that has no persisted entry yet. A state lock that is
+  busy at delete time queues the removal for the next write instead of dropping
+  it. The write path follows the same lock/atomic-rename contract as the state
+  writer; a missing entry is a no-op and an unsafe key is rejected.
+- **The Desktop subagent list is heaviest-first.** A family can hold a dozen
+  children and the RPC order is the `parentID` walk, which buried the session
+  actually eating the window; the list is now sorted by `ctx` (the polled array is
+  copied first, never reordered in place).
+- **`.gitattributes` normalises line endings.** Text files are stored with LF and
+  the image assets are marked binary, so a checkout on Windows and on the Linux CI
+  produces the same bytes instead of a CRLF/LF diff on every edit.
+- Unit harness extended to T43 (deleted-session purge: subtree removal, unrelated
+  entries kept, missing/unsafe keys are no-ops, late writes for a pruned or
+  not-yet-persisted child are dropped, a busy lock queues the whole subtree for
+  the next write), 116 checks in total.
+
 ## 1.8.2 — 2026-11-03
 
 - **Model-rendered commands no longer echo the instruction.** `/context`,

package/README.md

@@ -358,12 +358,16 @@ notice if needed.
   unambiguous payload boundary instead of a leading "reproduce exactly"
   phrasing. Residual risk: a model may still add light commentary or include the
   tags themselves.
-* **Deleted sessions age out over 7 days.** A `state.json` entry is pruned on
-  the next write once it has been untouched for 7 days (tombstone prune), so
-  deleted sessions no longer linger until the `MAX_TRACKED_SESSIONS` LRU cap.
-  Until the prune fires, a deleted session that had already served a step (has
-  tokens) is not filtered as a phantom and may momentarily appear in the
-  table.
+* **Deleted sessions are dropped immediately.** A `session.deleted` event removes
+  the session's `state.json` entry - and every entry under it in the `parentID`
+  chain - instead of waiting for the 7-day tombstone prune, so the row leaves
+  `/context-breakdown`, the `/cx` picker and the Desktop tab at once. The removed
+  IDs are remembered for the lifetime of the instance (the host never reuses an
+  ID), so a late idle/step write, or a child that was still running when its
+  parent was deleted, cannot recreate the row - without that barrier it came back
+  as a phantom ROOT session. A state lock that is busy at delete time queues the
+  removal for the next write instead of losing it. The 7-day tombstone remains as
+  the backstop for sessions that vanish without an event.
 
 ### TUI session panel (V2, terminal only)
 
@@ -766,7 +770,7 @@ caller.
 
 ```bash
 npm run build:gui             # build dist/gui-experimental.cjs (required by the gate)
-npm run test:prepublish       # fast gate: tarball + JSX-free TUI + resolve + T1-T41 unit checks + GUI bundle + GUI runtime smoke
+npm run test:prepublish       # fast gate: tarball + JSX-free TUI + resolve + T1-T43 unit checks + GUI bundle + GUI runtime smoke
 npm run test:types            # tsc for the TUI entry AND the Desktop GUI part
 npm run test:e2e              # full E2E: plugin load + one "Say OK" round-trip + npm-layout liveness
 npm run test:e2e:quick        # E2E load-only, no LLM call
@@ -790,7 +794,7 @@ STEP 80 s + PROBE 45 s, ~4 min total.
 | (a) tarball composition | `package.json` `files[]` expands to the base files plus every `lib/*` module (count asserted dynamically); `docs/` may contain images only; `dist/` may contain only the built GUI bundle | no test/node_modules/tsconfig in the tarball, and no published export pointing at nothing |
 | (b) JSX-free TUI | no `.tsx`/`.jsx` in the TUI surface (root + `lib/`), every TUI source parses with esbuild `loader:"ts"` (JSX syntax fails), no `@jsx*` pragma, imports stay inside the host-bridgeable allowlist, relative imports carry a file extension | OpenCode skips its Solid transform under `node_modules` (#33884): one JSX file freezes the npm-installed sidebar |
 | (c) resolve chain | walks the real TUI import graph (`resolve-bridge.cjs`): every bare specifier is host-bridged, every relative one resolves to a shipped file, and **no host-bridged module is a production dependency** (a second Solid/OpenTUI instance freezes the surfaces) | an accidental runtime dependency, or a bad local import, silently breaks the sidebar |
-| (d) unit harness T1-T42 | limit/agent/state fix assertions plus measureContributors, redacted-export, shared category-helper, sanitizer/unicode, prompt-cache, waste-detector, compaction-reserve, top-consumer, diff, subagent-rollup and Desktop-RPC checks (111 total; T30 schema-vs-builder drift, T31 registration + handler, T32-T36 fallback-vs-hook category priority across the report, the shared state file and the live log (T36: a counter-less hook entry borrows the snapshot or prints `?`), T37 the opencode 2.2.5 `min(input, context)` window, T38 the configured `compaction.buffer` (JSONC strip, project/global precedence, percent/token conversion, env/argument priority, per-project isolation), T39 the TUI composer capsule thresholds, T40 the data-first command payload + locale-free clock + labelled contributors, T41 model-scoped breakdowns after an in-session model switch, T42 the panel row width budget (the column sum never exceeds the panel; the stale-marker columns come out of the row, not on top of it), T25c/T25d compaction-hook arming, T34 cross-instance summary merge) | regressions in modelLimits, writeStateFile, hydrate, contributors, export, subagent lifetime, and a GUI/server wire mismatch |
+| (d) unit harness T1-T43 | limit/agent/state fix assertions plus measureContributors, redacted-export, shared category-helper, sanitizer/unicode, prompt-cache, waste-detector, compaction-reserve, top-consumer, diff, subagent-rollup and Desktop-RPC checks (116 total; T30 schema-vs-builder drift, T31 registration + handler, T32-T36 fallback-vs-hook category priority across the report, the shared state file and the live log (T36: a counter-less hook entry borrows the snapshot or prints `?`), T37 the opencode 2.2.5 `min(input, context)` window, T38 the configured `compaction.buffer` (JSONC strip, project/global precedence, percent/token conversion, env/argument priority, per-project isolation), T39 the TUI composer capsule thresholds, T40 the data-first command payload + locale-free clock + labelled contributors, T41 model-scoped breakdowns after an in-session model switch, T42 the panel row width budget (the column sum never exceeds the panel; the stale-marker columns come out of the row, not on top of it), T43 deleted-session purge of `state.json` (subtree removed, peers kept, unsafe keys rejected, late writes for a pruned or not-yet-persisted child dropped), T25c/T25d compaction-hook arming, T34 cross-instance summary merge) | regressions in modelLimits, writeStateFile, hydrate, contributors, export, subagent lifetime, and a GUI/server wire mismatch |
 | (e) corporate-identifier scan | no forbidden identifiers in the published surface (`index.js`, `tui.ts`, `lib/*`, `gui/*`, `README.md`, `CHANGELOG.md`, `package.json`, the GUI build config) or the test tree | compliance |
 | (f) non-English scan | no non-ASCII letters (`\p{L}` above U+007F) in the same published surface and test tree (`docs/` is image-only and is not text-scanned) | English-only source rule |
 | (g) Desktop GUI bundle | `dist/gui-experimental.cjs` exists and is a non-trivial single-file CJS bundle: no ESM `import`, no dynamic `import()`, no Node built-in, and every app-provided module (`solid-js`, `solid-js/web`, `@opencode/plugin/gui`, `@opencode/plugin/rpc`) still a `require()` - no bundled second copy, no React/OpenTUI/bare-`opencode` import; the artifact must also be newer than every source file it inlines | a stale or hand-edited bundle, or a build that inlines the app instance, breaks the Desktop panel at load time |

package/dist/gui-experimental.cjs

@@ -1 +1 @@
-Object.defineProperties(exports,{__esModule:{value:!0},[Symbol.toStringTag]:{value:`Module`}});let e=require("solid-js/web"),t=require("@opencode/plugin/gui"),n=require("solid-js"),r=require("@opencode/plugin/rpc");var i={sessionID:{type:`string`},parentID:{type:`string`},agent:{type:`string`},modelID:{type:`string`},providerID:{type:`string`},ctx:{type:`number`},input:{type:`number`},usable:{type:`number`},reserve:{type:`number`},user:{type:`number`},assistant:{type:`number`},reasoning:{type:`number`},toolArgs:{type:`number`},toolOut:{type:`number`},system:{type:`number`},toolSchemas:{type:`number`},other:{type:`number`},source:{type:`string`},updatedAt:{type:`string`},stale:{type:`boolean`}},a=Object.keys(i),o=r.Rpc.define({id:`context-indicator`,methods:{family:{input:{type:`object`,properties:{sessionID:{type:`string`}},required:[`sessionID`],additionalProperties:!1},output:{type:`object`,properties:{rows:{type:`array`,items:{type:`object`,properties:i,required:a,additionalProperties:!1}}},required:[`rows`],additionalProperties:!1}}},events:{}});function s(e){return Number.isFinite(e)?e>=1e3?`${(e/1e3).toFixed(1)}k`:String(Math.round(e)):`0`}var c=(0,e.template)(`<div>Server part unavailable: <!> (retrying)`),l=(0,e.template)(`<div>`),u=(0,e.template)(`<div>Loading context...`),d=(0,e.template)(`<div>No measurement yet. Send a prompt in this session and the numbers appear here.`),f=(0,e.template)(`<div><span></span><span>`),p=(0,e.template)(`<span>stale after compaction`),m=(0,e.template)(`<span>reserve `),h=(0,e.template)(`<div>Subagents (<!>)`),g=(0,e.template)(`<div><span></span><span></span><span><span style=display:block></span></span><span>`),_=1500,v=[[`system`,`System`],[`toolSchemas`,`Tool schemas`],[`user`,`User`],[`assistant`,`Assistant`],[`reasoning`,`Reasoning`],[`toolArgs`,`Tool args`],[`toolOut`,`Tool output`],[`other`,`Other`]];function y(e){if(!e)return`?`;try{let t=new Date(e);if(Number.isNaN(t.getTime()))return`?`;let n=e=>String(e).padStart(2,`0`);return`${n(t.getHours())}:${n(t.getMinutes())}:${n(t.getSeconds())}`}catch{return`?`}}function b(e,t){return!t||t<=0?``:`${(e/t*100).toFixed(0)}%`}var x={panel:{display:`flex`,"flex-direction":`column`,gap:`10px`,padding:`12px`,"font-size":`12px`,"line-height":`1.45`},hint:{opacity:`0.65`,"font-size":`12px`},header:{display:`flex`,"align-items":`baseline`,"justify-content":`space-between`,gap:`8px`},model:{"font-weight":`600`,overflow:`hidden`,"text-overflow":`ellipsis`,"white-space":`nowrap`},total:{"font-variant-numeric":`tabular-nums`,"white-space":`nowrap`},section:{display:`flex`,"flex-direction":`column`,gap:`6px`},row:{display:`grid`,"grid-template-columns":`1fr auto 64px auto`,"align-items":`center`,gap:`6px`},label:{opacity:`0.8`,overflow:`hidden`,"text-overflow":`ellipsis`,"white-space":`nowrap`},value:{"font-variant-numeric":`tabular-nums`},track:{height:`4px`,"border-radius":`2px`,background:`currentColor`,opacity:`0.15`,overflow:`hidden`},fill:{height:`4px`,"border-radius":`2px`,background:`currentColor`,opacity:`0.9`},share:{opacity:`0.65`,"text-align":`right`,"font-variant-numeric":`tabular-nums`},meta:{display:`flex`,gap:`8px`,opacity:`0.6`,"font-size":`11px`},sub:{display:`flex`,"justify-content":`space-between`,gap:`8px`,"font-size":`11px`},subName:{overflow:`hidden`,"text-overflow":`ellipsis`,"white-space":`nowrap`,opacity:`0.85`},title:{"font-weight":`600`,opacity:`0.85`}};function S(r){let i=(0,t.usePlugin)(),[a,S]=(0,n.createSignal)([]),[C,w]=(0,n.createSignal)(``),[T,E]=(0,n.createSignal)(!1);(0,n.createEffect)(()=>{let e=r.session.id,t=!1,a=!1;E(!1),S([]),w(``);let s=async()=>{if(!(a||t)){t=!0;try{let t=await i.client.rpc(o).family({sessionID:e});if(a)return;S(Array.isArray(t?.rows)?t.rows:[]),w(``)}catch(e){if(a)return;w(e instanceof Error?e.message:String(e))}finally{t=!1,a||E(!0)}}};s();let c=setInterval(()=>void s(),_);(0,n.onCleanup)(()=>{a=!0,clearInterval(c)})});let D=()=>a()[0],O=()=>a().slice(1),k=e=>{let t=D()?.ctx||0;return t<=0||e<=0?`0%`:`${Math.min(100,Math.max(2,Math.round(e/t*100)))}%`};return(()=>{var t=l();return(0,e.insert)(t,(0,e.createComponent)(n.Show,{get when(){return T()},get fallback(){return(()=>{var t=u();return(0,e.effect)(n=>(0,e.style)(t,x.hint,n)),t})()},get children(){return[(0,e.createComponent)(n.Show,{get when(){return C()},get children(){var t=c(),n=t.firstChild.nextSibling;return n.nextSibling,(0,e.insert)(t,C,n),(0,e.effect)(n=>(0,e.style)(t,x.hint,n)),t}}),(0,e.createComponent)(n.Show,{get when(){return D()},get fallback(){return(0,e.createComponent)(n.Show,{get when(){return!C()},get children(){var t=d();return(0,e.effect)(n=>(0,e.style)(t,x.hint,n)),t}})},children:t=>[(()=>{var n=f(),r=n.firstChild,i=r.nextSibling;return(0,e.insert)(r,()=>t().modelID||`unknown model`),(0,e.insert)(i,()=>s(t().ctx),null),(0,e.insert)(i,(()=>{var n=(0,e.memo)(()=>t().usable>0);return()=>n()?` / ${s(t().usable)} (${b(t().ctx,t().usable)})`:``})(),null),(0,e.effect)(a=>{var o=x.header,s=x.model,c=t().modelID,l=x.total;return a.e=(0,e.style)(n,o,a.e),a.t=(0,e.style)(r,s,a.t),c!==a.a&&(0,e.setAttribute)(r,`title`,a.a=c),a.o=(0,e.style)(i,l,a.o),a},{e:void 0,t:void 0,a:void 0,o:void 0}),n})(),(()=>{var r=l();return(0,e.insert)(r,(0,e.createComponent)(n.For,{each:v,children:n=>(()=>{var r=g(),i=r.firstChild,a=i.nextSibling,o=a.nextSibling,c=o.firstChild,l=o.nextSibling;return(0,e.insert)(i,()=>n[1]),(0,e.insert)(a,()=>s(t()[n[0]])),(0,e.insert)(l,()=>b(t()[n[0]],t().ctx)),(0,e.effect)(s=>{var u=x.row,d=x.label,f=x.value,p=x.track,m={...x.fill,width:k(t()[n[0]])},h=x.share;return s.e=(0,e.style)(r,u,s.e),s.t=(0,e.style)(i,d,s.t),s.a=(0,e.style)(a,f,s.a),s.o=(0,e.style)(o,p,s.o),s.i=(0,e.style)(c,m,s.i),s.n=(0,e.style)(l,h,s.n),s},{e:void 0,t:void 0,a:void 0,o:void 0,i:void 0,n:void 0}),r})()})),(0,e.effect)(t=>(0,e.style)(r,x.section,t)),r})(),(()=>{var r=f(),i=r.firstChild,a=i.nextSibling;return(0,e.insert)(i,()=>y(t().updatedAt)),(0,e.insert)(a,()=>t().source||`none`),(0,e.insert)(r,(0,e.createComponent)(n.Show,{get when(){return t().stale},get children(){return p()}}),null),(0,e.insert)(r,(0,e.createComponent)(n.Show,{get when(){return t().reserve>0},get children(){var n=m();return n.firstChild,(0,e.insert)(n,()=>s(t().reserve),null),n}}),null),(0,e.effect)(t=>(0,e.style)(r,x.meta,t)),r})(),(0,e.createComponent)(n.Show,{get when(){return O().length>0},get children(){return[(()=>{var t=h(),n=t.firstChild.nextSibling;return n.nextSibling,(0,e.insert)(t,()=>O().length,n),(0,e.effect)(n=>(0,e.style)(t,x.title,n)),t})(),(()=>{var t=l();return(0,e.insert)(t,(0,e.createComponent)(n.For,{get each(){return O()},children:t=>(()=>{var n=f(),r=n.firstChild,i=r.nextSibling;return(0,e.insert)(r,()=>t.agent||`sub`),(0,e.insert)(i,()=>s(t.ctx),null),(0,e.insert)(i,(()=>{var n=(0,e.memo)(()=>t.usable>0);return()=>n()?` (${b(t.ctx,t.usable)})`:``})(),null),(0,e.effect)(a=>{var o=x.sub,s=x.subName,c=t.modelID,l=x.value;return a.e=(0,e.style)(n,o,a.e),a.t=(0,e.style)(r,s,a.t),c!==a.a&&(0,e.setAttribute)(r,`title`,a.a=c),a.o=(0,e.style)(i,l,a.o),a},{e:void 0,t:void 0,a:void 0,o:void 0}),n})()})),(0,e.effect)(n=>(0,e.style)(t,x.section,n)),t})()]}})]})]}})),(0,e.effect)(n=>(0,e.style)(t,x.panel,n)),t})()}var C=`context`,w=`window-analytics`,T=`Context`,E=5e3,D=65,O=85,k={id:C,title:T},A=t.Plugin.define({setup(r){let i=`${r.id}:${C}`,a=e=>{let t=e??r.sessions.current();t&&r.layout.open(i,t,{tab:`select`})};r.add(t.Panel,()=>({id:C,region:`side`,wide:!0,list:e=>e.open.includes(C)?[k]:[],render:t=>(0,e.createComponent)(S,{get session(){return t.session}}),mobile:{title:T,order:1,kind:`tab`,icon:w}})),r.add(t.MenuItem,()=>({id:C,menu:`session.panel`,title:T,icon:w,description:`Context-window usage`,run:()=>a()})),r.add(t.Command,()=>({id:`open`,title:`Open context panel`,description:`Context-window usage for this session`,keywords:`context tokens window panel`,section:`session`,slash:{name:`cx`},run:()=>a()}));let[s,c]=(0,n.createSignal)(void 0),l=async()=>{let e=r.sessions.current();if(!e)c(void 0);else try{let t=(await r.client.rpc(o).family({sessionID:e.id}))?.rows?.[0];if(!t||t.usable<=0){c(void 0);return}let n=t.ctx/t.usable*100;c({sessionID:e.id,label:`${n.toFixed(0)}% ctx`,tone:n>=O?`danger`:n>=D?`warning`:void 0})}catch{c(void 0)}};l();let u=setInterval(()=>void l(),E);(0,n.onCleanup)(()=>clearInterval(u)),r.add(t.ComposerPill,()=>({id:`usage`,show:({session:e})=>{let t=s();if(t&&t.sessionID===e.id)return{label:t.label,icon:w,tone:t.tone,title:`Context window usage`,run:()=>a(e)}}}))}});exports.default=A;
+Object.defineProperties(exports,{__esModule:{value:!0},[Symbol.toStringTag]:{value:`Module`}});let e=require("solid-js/web"),t=require("@opencode/plugin/gui"),n=require("solid-js"),r=require("@opencode/plugin/rpc");var i={sessionID:{type:`string`},parentID:{type:`string`},agent:{type:`string`},modelID:{type:`string`},providerID:{type:`string`},ctx:{type:`number`},input:{type:`number`},usable:{type:`number`},reserve:{type:`number`},user:{type:`number`},assistant:{type:`number`},reasoning:{type:`number`},toolArgs:{type:`number`},toolOut:{type:`number`},system:{type:`number`},toolSchemas:{type:`number`},other:{type:`number`},source:{type:`string`},updatedAt:{type:`string`},stale:{type:`boolean`}},a=Object.keys(i),o=r.Rpc.define({id:`context-indicator`,methods:{family:{input:{type:`object`,properties:{sessionID:{type:`string`}},required:[`sessionID`],additionalProperties:!1},output:{type:`object`,properties:{rows:{type:`array`,items:{type:`object`,properties:i,required:a,additionalProperties:!1}}},required:[`rows`],additionalProperties:!1}}},events:{}});function s(e){return Number.isFinite(e)?e>=1e3?`${(e/1e3).toFixed(1)}k`:String(Math.round(e)):`0`}var c=(0,e.template)(`<div>Server part unavailable: <!> (retrying)`),l=(0,e.template)(`<div>`),u=(0,e.template)(`<div>Loading context...`),d=(0,e.template)(`<div>No measurement yet. Send a prompt in this session and the numbers appear here.`),f=(0,e.template)(`<div><span></span><span>`),p=(0,e.template)(`<span>stale after compaction`),m=(0,e.template)(`<span>reserve `),h=(0,e.template)(`<div>Subagents (<!>)`),g=(0,e.template)(`<div><span></span><span></span><span><span style=display:block></span></span><span>`),_=1500,v=[[`system`,`System`],[`toolSchemas`,`Tool schemas`],[`user`,`User`],[`assistant`,`Assistant`],[`reasoning`,`Reasoning`],[`toolArgs`,`Tool args`],[`toolOut`,`Tool output`],[`other`,`Other`]];function y(e){if(!e)return`?`;try{let t=new Date(e);if(Number.isNaN(t.getTime()))return`?`;let n=e=>String(e).padStart(2,`0`);return`${n(t.getHours())}:${n(t.getMinutes())}:${n(t.getSeconds())}`}catch{return`?`}}function b(e,t){return!t||t<=0?``:`${(e/t*100).toFixed(0)}%`}var x={panel:{display:`flex`,"flex-direction":`column`,gap:`10px`,padding:`12px`,"font-size":`12px`,"line-height":`1.45`},hint:{opacity:`0.65`,"font-size":`12px`},header:{display:`flex`,"align-items":`baseline`,"justify-content":`space-between`,gap:`8px`},model:{"font-weight":`600`,overflow:`hidden`,"text-overflow":`ellipsis`,"white-space":`nowrap`},total:{"font-variant-numeric":`tabular-nums`,"white-space":`nowrap`},section:{display:`flex`,"flex-direction":`column`,gap:`6px`},row:{display:`grid`,"grid-template-columns":`1fr auto 64px auto`,"align-items":`center`,gap:`6px`},label:{opacity:`0.8`,overflow:`hidden`,"text-overflow":`ellipsis`,"white-space":`nowrap`},value:{"font-variant-numeric":`tabular-nums`},track:{height:`4px`,"border-radius":`2px`,background:`currentColor`,opacity:`0.15`,overflow:`hidden`},fill:{height:`4px`,"border-radius":`2px`,background:`currentColor`,opacity:`0.9`},share:{opacity:`0.65`,"text-align":`right`,"font-variant-numeric":`tabular-nums`},meta:{display:`flex`,gap:`8px`,opacity:`0.6`,"font-size":`11px`},sub:{display:`flex`,"justify-content":`space-between`,gap:`8px`,"font-size":`11px`},subName:{overflow:`hidden`,"text-overflow":`ellipsis`,"white-space":`nowrap`,opacity:`0.85`},title:{"font-weight":`600`,opacity:`0.85`}};function S(r){let i=(0,t.usePlugin)(),[a,S]=(0,n.createSignal)([]),[C,w]=(0,n.createSignal)(``),[T,E]=(0,n.createSignal)(!1);(0,n.createEffect)(()=>{let e=r.session.id,t=!1,a=!1;E(!1),S([]),w(``);let s=async()=>{if(!(a||t)){t=!0;try{let t=await i.client.rpc(o).family({sessionID:e});if(a)return;S(Array.isArray(t?.rows)?t.rows:[]),w(``)}catch(e){if(a)return;w(e instanceof Error?e.message:String(e))}finally{t=!1,a||E(!0)}}};s();let c=setInterval(()=>void s(),_);(0,n.onCleanup)(()=>{a=!0,clearInterval(c)})});let D=()=>a()[0],O=()=>a().slice(1).sort((e,t)=>(t.ctx||0)-(e.ctx||0)),k=e=>{let t=D()?.ctx||0;return t<=0||e<=0?`0%`:`${Math.min(100,Math.max(2,Math.round(e/t*100)))}%`};return(()=>{var t=l();return(0,e.insert)(t,(0,e.createComponent)(n.Show,{get when(){return T()},get fallback(){return(()=>{var t=u();return(0,e.effect)(n=>(0,e.style)(t,x.hint,n)),t})()},get children(){return[(0,e.createComponent)(n.Show,{get when(){return C()},get children(){var t=c(),n=t.firstChild.nextSibling;return n.nextSibling,(0,e.insert)(t,C,n),(0,e.effect)(n=>(0,e.style)(t,x.hint,n)),t}}),(0,e.createComponent)(n.Show,{get when(){return D()},get fallback(){return(0,e.createComponent)(n.Show,{get when(){return!C()},get children(){var t=d();return(0,e.effect)(n=>(0,e.style)(t,x.hint,n)),t}})},children:t=>[(()=>{var n=f(),r=n.firstChild,i=r.nextSibling;return(0,e.insert)(r,()=>t().modelID||`unknown model`),(0,e.insert)(i,()=>s(t().ctx),null),(0,e.insert)(i,(()=>{var n=(0,e.memo)(()=>t().usable>0);return()=>n()?` / ${s(t().usable)} (${b(t().ctx,t().usable)})`:``})(),null),(0,e.effect)(a=>{var o=x.header,s=x.model,c=t().modelID,l=x.total;return a.e=(0,e.style)(n,o,a.e),a.t=(0,e.style)(r,s,a.t),c!==a.a&&(0,e.setAttribute)(r,`title`,a.a=c),a.o=(0,e.style)(i,l,a.o),a},{e:void 0,t:void 0,a:void 0,o:void 0}),n})(),(()=>{var r=l();return(0,e.insert)(r,(0,e.createComponent)(n.For,{each:v,children:n=>(()=>{var r=g(),i=r.firstChild,a=i.nextSibling,o=a.nextSibling,c=o.firstChild,l=o.nextSibling;return(0,e.insert)(i,()=>n[1]),(0,e.insert)(a,()=>s(t()[n[0]])),(0,e.insert)(l,()=>b(t()[n[0]],t().ctx)),(0,e.effect)(s=>{var u=x.row,d=x.label,f=x.value,p=x.track,m={...x.fill,width:k(t()[n[0]])},h=x.share;return s.e=(0,e.style)(r,u,s.e),s.t=(0,e.style)(i,d,s.t),s.a=(0,e.style)(a,f,s.a),s.o=(0,e.style)(o,p,s.o),s.i=(0,e.style)(c,m,s.i),s.n=(0,e.style)(l,h,s.n),s},{e:void 0,t:void 0,a:void 0,o:void 0,i:void 0,n:void 0}),r})()})),(0,e.effect)(t=>(0,e.style)(r,x.section,t)),r})(),(()=>{var r=f(),i=r.firstChild,a=i.nextSibling;return(0,e.insert)(i,()=>y(t().updatedAt)),(0,e.insert)(a,()=>t().source||`none`),(0,e.insert)(r,(0,e.createComponent)(n.Show,{get when(){return t().stale},get children(){return p()}}),null),(0,e.insert)(r,(0,e.createComponent)(n.Show,{get when(){return t().reserve>0},get children(){var n=m();return n.firstChild,(0,e.insert)(n,()=>s(t().reserve),null),n}}),null),(0,e.effect)(t=>(0,e.style)(r,x.meta,t)),r})(),(0,e.createComponent)(n.Show,{get when(){return O().length>0},get children(){return[(()=>{var t=h(),n=t.firstChild.nextSibling;return n.nextSibling,(0,e.insert)(t,()=>O().length,n),(0,e.effect)(n=>(0,e.style)(t,x.title,n)),t})(),(()=>{var t=l();return(0,e.insert)(t,(0,e.createComponent)(n.For,{get each(){return O()},children:t=>(()=>{var n=f(),r=n.firstChild,i=r.nextSibling;return(0,e.insert)(r,()=>t.agent||`sub`),(0,e.insert)(i,()=>s(t.ctx),null),(0,e.insert)(i,(()=>{var n=(0,e.memo)(()=>t.usable>0);return()=>n()?` (${b(t.ctx,t.usable)})`:``})(),null),(0,e.effect)(a=>{var o=x.sub,s=x.subName,c=t.modelID,l=x.value;return a.e=(0,e.style)(n,o,a.e),a.t=(0,e.style)(r,s,a.t),c!==a.a&&(0,e.setAttribute)(r,`title`,a.a=c),a.o=(0,e.style)(i,l,a.o),a},{e:void 0,t:void 0,a:void 0,o:void 0}),n})()})),(0,e.effect)(n=>(0,e.style)(t,x.section,n)),t})()]}})]})]}})),(0,e.effect)(n=>(0,e.style)(t,x.panel,n)),t})()}var C=`context`,w=`window-analytics`,T=`Context`,E=5e3,D=65,O=85,k={id:C,title:T},A=t.Plugin.define({setup(r){let i=`${r.id}:${C}`,a=e=>{let t=e??r.sessions.current();t&&r.layout.open(i,t,{tab:`select`})};r.add(t.Panel,()=>({id:C,region:`side`,wide:!0,list:e=>e.open.includes(C)?[k]:[],render:t=>(0,e.createComponent)(S,{get session(){return t.session}}),mobile:{title:T,order:1,kind:`tab`,icon:w}})),r.add(t.MenuItem,()=>({id:C,menu:`session.panel`,title:T,icon:w,description:`Context-window usage`,run:()=>a()})),r.add(t.Command,()=>({id:`open`,title:`Open context panel`,description:`Context-window usage for this session`,keywords:`context tokens window panel`,section:`session`,slash:{name:`cx`},run:()=>a()}));let[s,c]=(0,n.createSignal)(void 0),l=async()=>{let e=r.sessions.current();if(!e)c(void 0);else try{let t=(await r.client.rpc(o).family({sessionID:e.id}))?.rows?.[0];if(!t||t.usable<=0){c(void 0);return}let n=t.ctx/t.usable*100;c({sessionID:e.id,label:`${n.toFixed(0)}% ctx`,tone:n>=O?`danger`:n>=D?`warning`:void 0})}catch{c(void 0)}};l();let u=setInterval(()=>void l(),E);(0,n.onCleanup)(()=>clearInterval(u)),r.add(t.ComposerPill,()=>({id:`usage`,show:({session:e})=>{let t=s();if(t&&t.sessionID===e.id)return{label:t.label,icon:w,tone:t.tone,title:`Context window usage`,run:()=>a(e)}}}))}});exports.default=A;

package/index.js

@@ -184,6 +184,7 @@ import { breakdownCache, breakdownForModel, lastKnownCacheRead, lastKnownCacheWr
 import { registerDesktopRpc, toDesktopRow } from "./lib/desktop.js";
 import {
   logErr,
+  pruneSessionState,
   readStateSnapshot,
   writeStateFile,
 } from "./lib/state.js";
@@ -201,6 +202,7 @@ export {
   sessionAgentCache,
   sessionParentCache,
   writeStateFile,
+  pruneSessionState,
   hydrateModelLimitsFromState,
   getUsableContext,
   getModelLimit,

package/lib/state.js

@@ -429,6 +429,103 @@ function renameWithRetry(from, to, attempts = 3, delayMs = 50) {
 // 7 days: a tombstone only matters while a sibling/parent poll can still see it.
 const PRUNE_AFTER_MS = 7 * 24 * 60 * 60 * 1000;
 
+// Session IDs whose persisted entry was removed by `session.deleted` (or is
+// queued for removal because the state lock was busy). The host never reuses an
+// ID, so once a session is gone every later write for it must be DROPPED - a
+// write would recreate the entry, and after `clearChildSubagentTracking()` there
+// is no parent cache left either, so the child came back as a phantom ROOT row
+// that lived up to the 7-day tombstone (worse than the old phantom child).
+// Bounded FIFO like the other per-instance maps.
+const PRUNED_MAX = 512;
+const prunedSessions = new Set();
+// DELETE REQUESTS waiting for a state-file write: the lock was busy when the
+// event arrived. Stored as the deleted ROOT id, not as a flat list of entries -
+// the subtree is unknown without the file, and the request is replayed under the
+// next write's lock (where the snapshot is available) so a busy lock cannot leave
+// descendants alive. Bounded FIFO as well.
+const pendingPruneRoots = new Set();
+// Diagnostics: one log line per session that a late write was blocked for. A
+// silent drop is impossible to debug in the field, but a line per idle/step
+// would flood the log.
+const barrierLogged = new Set();
+
+function rememberPruned(id) {
+  if (typeof id !== "string" || !id) return;
+  prunedSessions.add(id);
+  if (prunedSessions.size > PRUNED_MAX) {
+    const oldest = prunedSessions.values().next().value;
+    if (oldest !== undefined) prunedSessions.delete(oldest);
+  }
+}
+
+function queuePrune(rootID) {
+  if (typeof rootID !== "string" || !rootID) return;
+  pendingPruneRoots.add(rootID);
+  if (pendingPruneRoots.size > PRUNED_MAX) {
+    const oldest = pendingPruneRoots.values().next().value;
+    if (oldest !== undefined) pendingPruneRoots.delete(oldest);
+  }
+}
+
+function logBarrierOnce(id) {
+  if (barrierLogged.has(id)) return;
+  barrierLogged.add(id);
+  if (barrierLogged.size > PRUNED_MAX) {
+    const oldest = barrierLogged.values().next().value;
+    if (oldest !== undefined) barrierLogged.delete(oldest);
+  }
+  logErr(`writeStateFile dropped for deleted session=${oneLine(id)}`);
+}
+
+// `rootID` plus every persisted entry below it in the parentID chain. The index
+// is built once (O(n)) instead of rescanning the whole snapshot per visited node:
+// the file is user-writable and unbounded on read, and this runs on the event
+// loop. `doomed` doubles as the cycle guard.
+function subtreeIDs(all, rootID) {
+  const childrenByParent = new Map();
+  for (const [id, entry] of Object.entries(all)) {
+    const parent = entry?.parentID;
+    if (typeof parent !== "string") continue;
+    const kids = childrenByParent.get(parent);
+    if (kids) kids.push(id);
+    else childrenByParent.set(parent, [id]);
+  }
+  const doomed = new Set([rootID]);
+  const stack = [rootID];
+  while (stack.length > 0) {
+    const current = stack.pop();
+    for (const child of childrenByParent.get(current) || []) {
+      if (doomed.has(child)) continue;
+      doomed.add(child);
+      stack.push(child);
+    }
+  }
+  return doomed;
+}
+
+// Atomic commit of a state snapshot: random temp suffix + exclusive create
+// (CWE-377), rename over a regular file only, temp cleanup on failure. Shared by
+// the writer and the deleter so the two can never drift apart.
+function commitState(all) {
+  const tmp = `${STATE_FILE}.${randomBytes(6).toString("hex")}.tmp`;
+  writeFileSync(tmp, JSON.stringify(all), { encoding: "utf8", flag: "wx" });
+  let renamed = false;
+  try {
+    ensureRegularFile(STATE_FILE); // never rename over a planted symlink
+    renamed = renameWithRetry(tmp, STATE_FILE);
+  } finally {
+    // A rename that exhausted its retries leaves the temp file behind; drop it
+    // so repeated failures cannot litter %TEMP% (best-effort: on a SUCCESSFUL
+    // rename the temp is already gone, existsSync filters that).
+    try {
+      if (existsSync(tmp)) unlinkSync(tmp);
+    } catch {
+      /* best-effort cleanup */
+    }
+  }
+  return renamed;
+}
+
 function writeStateFile(
   sessionID,
   modelID,
@@ -447,6 +544,16 @@ function writeStateFile(
       logErr(`writeStateFile skipped unsafe session key ${oneLine(sessionID)}`);
       return false;
     }
+    // The session was deleted: never recreate its entry. Without this barrier a
+    // late idle/step write (or a live child whose parent was just deleted)
+    // re-adds the row, and with the parent cache already cleared it lands with
+    // `parentID: null` - a phantom ROOT session in the table, the picker and the
+    // Desktop tab until the tombstone. Logged once per ID: the check runs on
+    // every write for that session and a silent drop would be undebuggable.
+    if (prunedSessions.has(sessionID)) {
+      logBarrierOnce(sessionID);
+      return false;
+    }
     const lock = acquireLock(STATE_LOCK_FILE);
     // {ok:false} => busy: skip (value stays in memory, lands next cycle), as
     // before. null => unexpected error: fail-open, write WITHOUT the lock and do
@@ -817,24 +924,23 @@ function writeStateFile(
           .slice(MAX_TRACKED_SESSIONS)
           .forEach((k) => delete all[k]);
       }
-      // Random temp suffix + exclusive create (CWE-377), then atomic rename.
-      const tmp = `${STATE_FILE}.${randomBytes(6).toString("hex")}.tmp`;
-      writeFileSync(tmp, JSON.stringify(all), { encoding: "utf8", flag: "wx" });
-      let renamed = false;
-      try {
-        ensureRegularFile(STATE_FILE); // never rename over a planted symlink
-        renamed = renameWithRetry(tmp, STATE_FILE);
-      } finally {
-        // A rename that exhausted its retries leaves the temp file behind; drop
-        // it so repeated failures cannot litter %TEMP% (best-effort: on a
-        // SUCCESSFUL rename the temp is already gone, existsSync filters that).
-        try {
-          if (existsSync(tmp)) unlinkSync(tmp);
-        } catch {
-          /* best-effort cleanup */
+      // Replay queued deletes inside THIS write's lock: the snapshot is only
+      // readable under it, so a busy lock at delete time cannot leave descendants
+      // alive. The requests are kept until the commit SUCCEEDS - clearing the
+      // queue before a rename that never happened would drop the delete silently
+      // and leave the row to the 7-day tombstone.
+      const replaying = pendingPruneRoots.size > 0;
+      if (replaying) {
+        for (const root of pendingPruneRoots) {
+          for (const id of subtreeIDs(all, root)) {
+            delete all[id];
+            rememberPruned(id);
+          }
         }
       }
-      return renamed;
+      const committed = commitState(all);
+      if (committed && replaying) pendingPruneRoots.clear();
+      return committed;
     } finally {
       // Release ONLY a lock we actually own (lock === null => fail-open, skip).
       if (lock && lock.ok) releaseLock(STATE_LOCK_FILE, lock.token);
@@ -931,10 +1037,84 @@ function pushFinalSummary(claimKey, summaryLines) {
   }
 }
 
+// Delete one session's entry AND its whole subtree from the shared state file.
+//
+// `session.deleted` is authoritative, so waiting for the tombstone (7 days) or
+// the LRU cap keeps a dead session visible in `/context-breakdown`, the Desktop
+// tab and the `/cx` picker - and a phantom row has no step event left to be
+// filtered out by. The walk follows the persisted parentID chain (the same
+// source discovery uses), so a deleted parent takes its descendants with it and
+// a deleted child takes its own sub-subagents.
+//
+// Every removed ID also enters `prunedSessions`: the host never reuses an ID, so
+// a later write for it is dropped instead of recreating the row (a live child
+// whose parent was deleted would otherwise come back as a phantom ROOT once
+// clearChildSubagentTracking() had dropped its parent cache).
+//
+// Same locking/atomic-write contract as writeStateFile, except busy: the delete
+// is QUEUED for the next write (which already holds the lock) rather than lost -
+// the tombstone would not remove it for 7 days.
+function pruneSessionState(sessionID) {
+  try {
+    if (!sessionID || !isSafeSessionKey(sessionID)) return false;
+    // In-memory descendants first: a child that has not written its first entry
+    // yet is invisible to the disk walk below, but it WILL write later - and by
+    // then `session.deleted` has already wiped its parent attribution, so it
+    // would land as a phantom root. Remember them regardless of the lock.
+    const kidsByParent = new Map();
+    for (const [childID, parentID] of sessionParentCache) {
+      if (typeof parentID !== "string") continue;
+      const kids = kidsByParent.get(parentID);
+      if (kids) kids.push(childID);
+      else kidsByParent.set(parentID, [childID]);
+    }
+    const inMemory = new Set([sessionID]);
+    const pending = [sessionID];
+    while (pending.length > 0) {
+      const current = pending.pop();
+      for (const child of kidsByParent.get(current) || []) {
+        if (inMemory.has(child)) continue;
+        inMemory.add(child);
+        pending.push(child);
+      }
+    }
+    for (const id of inMemory) rememberPruned(id);
+    const lock = acquireLock(STATE_LOCK_FILE);
+    if (lock && !lock.ok) {
+      logErr(`pruneSessionState queued (state lock busy) session=${oneLine(sessionID)}`);
+      queuePrune(sessionID); // replayed under the next write's lock, with the subtree
+      return false;
+    }
+    try {
+      const all = readStateSnapshot();
+      const doomed = subtreeIDs(all, sessionID);
+      let removed = 0;
+      for (const id of doomed) {
+        // Remembered even when nothing was persisted: the write barrier has to
+        // cover a session whose entry was already gone (another instance pruned
+        // it) and a child that has not written its first entry yet.
+        rememberPruned(id);
+        if (Object.prototype.hasOwnProperty.call(all, id)) {
+          delete all[id];
+          removed++;
+        }
+      }
+      if (removed === 0) return false; // nothing persisted (already gone)
+      return commitState(all);
+    } finally {
+      if (lock && lock.ok) releaseLock(STATE_LOCK_FILE, lock.token);
+    }
+  } catch (err) {
+    logErr("pruneSessionState failed", err);
+    return false;
+  }
+}
+
 export {
   logErr,
   writeLog,
   writeStateFile,
+  pruneSessionState,
   debugEvent,
   readStateSnapshot,
   pushFinalSummary,

package/lib/v1.js

@@ -671,6 +671,11 @@ function makeV1(client) {
       ) {
         // session.idle / session.compacted -> properties.sessionID;
         // session.deleted -> properties.info.id
+        //
+        // V1 is feature-frozen: unlike the V2 handler, a V1 delete event does NOT
+        // purge the state.json entry (it ages out via the 7-day tombstone), so a
+        // V1 host can still show a phantom row. Kept intentionally small; V2 is
+        // the supported path.
         const sid =
           event.properties?.sessionID ?? event.properties?.info?.id;
         if (!sid) return;

package/lib/v2.js

@@ -66,6 +66,7 @@ import { locationDirectory } from "./config.js";
 import {
   debugEvent,
   logErr,
+  pruneSessionState,
   pushFinalSummary,
   readStateSnapshot,
   writeLog,
@@ -995,10 +996,11 @@ async function handleV2Event(ctx, event) {
   if (event.type === "session.deleted") {
     const sid = event.data?.sessionID;
     if (!sid) return;
-    // Discovery reads the persisted parentID chain, so a deleted session drops
-    // out of the table as soon as its state.json entry is gone (or, while the
-    // stale entry lingers, via the phantom filter — a deleted session has no
-    // step). No dedicated in-memory index to clean up.
+    // Discovery reads the persisted parentID chain, so the entry has to go too:
+    // clearing the in-memory caches alone leaves the row in state.json until the
+    // tombstone (7 days) or the LRU cap drops it, and there is no step event left
+    // to filter it out as a phantom.
+    pruneSessionState(sid);
     clearSessionTracking(sid);
     // Whatever this session was, its own descendants must go too: a deleted
     // parent takes the whole subtree, and a deleted child takes its own

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-context-indicator",
-  "version": "1.8.2",
+  "version": "1.8.3",
   "description": "Real-time context-window usage for OpenCode: per-category token breakdown, /context-diff, a zero-LLM /cx panel with per-session cost and a searchable family-session picker, a TUI sidebar, and an experimental Desktop panel. Dual V1 + V2 plugin entry.",
   "type": "module",
   "license": "MIT",