Merge pull request #115 from Tarquinen/fix/case-insensitive-tool-id-matching

Tarquinen · web-flow · commit 42f3116361c0 · 2025-12-11T11:46:48.000-05:00
Fix case-sensitive tool ID comparison in pruning logic
diff --git a/lib/core/janitor.ts b/lib/core/janitor.ts
@@ -116,7 +116,9 @@ async function runWithStrategies(
         const { toolCallIds, toolOutputs, toolMetadata } = parseMessages(messages, state.toolParameters)
 
         const alreadyPrunedIds = state.prunedIds.get(sessionID) ?? []
-        const unprunedToolCallIds = toolCallIds.filter(id => !alreadyPrunedIds.includes(id))
+        // Normalized set for filtering to avoid re-processing already pruned tools with different casing
+        const alreadyPrunedLower = new Set(alreadyPrunedIds.map(id => id.toLowerCase()))
+        const unprunedToolCallIds = toolCallIds.filter(id => !alreadyPrunedLower.has(id))
 
         const gcPending = state.gcPending.get(sessionID) ?? null
 
@@ -145,7 +147,7 @@ async function runWithStrategies(
             )
         }
 
-        const finalNewlyPrunedIds = llmPrunedIds.filter(id => !alreadyPrunedIds.includes(id))
+        const finalNewlyPrunedIds = llmPrunedIds.filter(id => !alreadyPrunedLower.has(id.toLowerCase()))
 
         if (finalNewlyPrunedIds.length === 0 && !gcPending) {
             return null
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
   "name": "@tarquinen/opencode-dcp",
-  "version": "0.4.16",
+  "version": "0.4.17",
   "type": "module",
   "description": "OpenCode plugin that optimizes token usage by pruning obsolete tool outputs from conversation context",
   "main": "./dist/index.js",

Original file line number	Diff line number	Diff line change
`@@ -116,7 +116,9 @@ async function runWithStrategies(`
`116`	`116`	`const { toolCallIds, toolOutputs, toolMetadata } = parseMessages(messages, state.toolParameters)`
`117`	`117`
`118`	`118`	`const alreadyPrunedIds = state.prunedIds.get(sessionID) ?? []`
`119`		`- const unprunedToolCallIds = toolCallIds.filter(id => !alreadyPrunedIds.includes(id))`
	`119`	`+ // Normalized set for filtering to avoid re-processing already pruned tools with different casing`
	`120`	`+ const alreadyPrunedLower = new Set(alreadyPrunedIds.map(id => id.toLowerCase()))`
	`121`	`+ const unprunedToolCallIds = toolCallIds.filter(id => !alreadyPrunedLower.has(id))`
`120`	`122`
`121`	`123`	`const gcPending = state.gcPending.get(sessionID) ?? null`
`122`	`124`
`@@ -145,7 +147,7 @@ async function runWithStrategies(`
`145`	`147`	`)`
`146`	`148`	`}`
`147`	`149`
`148`		`- const finalNewlyPrunedIds = llmPrunedIds.filter(id => !alreadyPrunedIds.includes(id))`
	`150`	`+ const finalNewlyPrunedIds = llmPrunedIds.filter(id => !alreadyPrunedLower.has(id.toLowerCase()))`
`149`	`151`
`150`	`152`	`if (finalNewlyPrunedIds.length === 0 && !gcPending) {`
`151`	`153`	`return null`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"$schema": "https://json.schemastore.org/package.json",`
`3`	`3`	`"name": "@tarquinen/opencode-dcp",`
`4`		`- "version": "0.4.16",`
	`4`	`+ "version": "0.4.17",`
`5`	`5`	`"type": "module",`
`6`	`6`	`"description": "OpenCode plugin that optimizes token usage by pruning obsolete tool outputs from conversation context",`
`7`	`7`	`"main": "./dist/index.js",`