chore: remove redundant comments

Author: Tarquinen

lib/fetch-wrapper/formats/gemini.ts

@@ -50,8 +50,6 @@ export const geminiFormat: FormatDescriptor = {
     extractToolOutputs(data: any[], state: PluginState): ToolOutput[] {
         const outputs: ToolOutput[] = []
 
-        // We need the position mapping to correlate functionResponses to tool call IDs
-        // Find the mapping from any active session
         let positionMapping: Map<string, string> | undefined
         for (const [_sessionId, mapping] of state.googleToolCallMapping) {
             if (mapping && mapping.size > 0) {
@@ -64,7 +62,6 @@ export const geminiFormat: FormatDescriptor = {
             return outputs
         }
 
-        // Track position counters per tool name
         const toolPositionCounters = new Map<string, number>()
 
         for (const content of data) {
@@ -95,7 +92,6 @@ export const geminiFormat: FormatDescriptor = {
     },
 
     replaceToolOutput(data: any[], toolId: string, prunedMessage: string, state: PluginState): boolean {
-        // Find the position mapping
         let positionMapping: Map<string, string> | undefined
         for (const [_sessionId, mapping] of state.googleToolCallMapping) {
             if (mapping && mapping.size > 0) {

lib/fetch-wrapper/formats/openai-chat.ts

@@ -49,7 +49,6 @@ export const openaiChatFormat: FormatDescriptor = {
         const outputs: ToolOutput[] = []
 
         for (const m of data) {
-            // OpenAI Chat format: role='tool' with tool_call_id
             if (m.role === 'tool' && m.tool_call_id) {
                 const metadata = state.toolParameters.get(m.tool_call_id.toLowerCase())
                 outputs.push({
@@ -58,7 +57,6 @@ export const openaiChatFormat: FormatDescriptor = {
                 })
             }
 
-            // Anthropic format: role='user' with content[].type='tool_result'
             if (m.role === 'user' && Array.isArray(m.content)) {
                 for (const part of m.content) {
                     if (part.type === 'tool_result' && part.tool_use_id) {
@@ -82,13 +80,11 @@ export const openaiChatFormat: FormatDescriptor = {
         for (let i = 0; i < data.length; i++) {
             const m = data[i]
 
-            // OpenAI Chat format
             if (m.role === 'tool' && m.tool_call_id?.toLowerCase() === toolIdLower) {
                 data[i] = { ...m, content: prunedMessage }
                 replaced = true
             }
 
-            // Anthropic format
             if (m.role === 'user' && Array.isArray(m.content)) {
                 let messageModified = false
                 const newContent = m.content.map((part: any) => {

lib/fetch-wrapper/handler.ts

@@ -29,10 +29,8 @@ export async function handleFormat(
 
     let modified = false
 
-    // Cache tool parameters for this format
     format.cacheToolParameters(data, ctx.state, ctx.logger)
 
-    // Inject synthetic instructions if strategies are enabled
     if (ctx.config.strategies.onTool.length > 0) {
         if (format.injectSynth(data, ctx.prompts.synthInstruction, ctx.prompts.nudgeInstruction)) {
             modified = true
@@ -70,25 +68,21 @@ export async function handleFormat(
         }
     }
 
-    // Check if there are any tool outputs to potentially prune
     if (!format.hasToolOutputs(data)) {
         return { modified, body }
     }
 
-    // Get all pruned IDs across sessions
     const { allSessions, allPrunedIds } = await getAllPrunedIds(ctx.client, ctx.state, ctx.logger)
 
     if (allPrunedIds.size === 0) {
         return { modified, body }
     }
 
-    // Extract tool outputs and replace pruned ones
     const toolOutputs = format.extractToolOutputs(data, ctx.state)
     const protectedToolsLower = new Set(ctx.config.protectedTools.map(t => t.toLowerCase()))
     let replacedCount = 0
 
     for (const output of toolOutputs) {
-        // Skip protected tools
         if (output.toolName && protectedToolsLower.has(output.toolName.toLowerCase())) {
             continue
         }
@@ -106,7 +100,6 @@ export async function handleFormat(
             total: toolOutputs.length
         })
 
-        // Save context for debugging if logging is enabled
         if (ctx.logger.enabled) {
             const activeSessions = allSessions.data?.filter((s: any) => !s.parentID) || []
             let sessionMessages: any[] | undefined

lib/fetch-wrapper/index.ts

@@ -56,7 +56,7 @@ export function installFetchWrapper(
 
                 const toolIdsBefore = new Set(state.toolParameters.keys())
 
-                // Mutually exclusive format handlers to avoid double-processing
+                // Mutually exclusive format handlers
                 if (openaiResponsesFormat.detect(body)) {
                     const result = await handleFormat(body, ctx, inputUrl, openaiResponsesFormat)
                     if (result.modified) {

lib/fetch-wrapper/types.ts

@@ -99,7 +99,6 @@ export async function getAllPrunedIds(
     if (currentSession) {
         await ensureSessionRestored(state, currentSession.id, logger)
         const prunedIds = state.prunedIds.get(currentSession.id) ?? []
-        // Normalize to lowercase for case-insensitive matching
         prunedIds.forEach((id: string) => allPrunedIds.add(id.toLowerCase()))
 
         if (logger && prunedIds.length > 0) {

lib/hooks.ts

@@ -32,7 +32,6 @@ export function createEventHandler(
             if (await isSubagentSession(client, event.properties.sessionID)) return
             if (config.strategies.onIdle.length === 0) return
 
-            // Skip idle pruning if the last tool used was prune and idle strategies cover tool strategies
             if (toolTracker?.skipNextIdle) {
                 toolTracker.skipNextIdle = false
                 if (toolStrategiesCoveredByIdle(config.strategies.onIdle, config.strategies.onTool)) {
@@ -43,7 +42,6 @@ export function createEventHandler(
             try {
                 const result = await runOnIdle(janitorCtx, event.properties.sessionID, config.strategies.onIdle)
 
-                // Reset nudge counter if idle pruning succeeded and covers tool strategies
                 if (result && result.prunedCount > 0 && toolTracker && config.nudge_freq > 0) {
                     if (toolStrategiesCoveredByIdle(config.strategies.onIdle, config.strategies.onTool)) {
                         resetToolTrackerCount(toolTracker)
@@ -91,8 +89,7 @@ export function createChatParamsHandler(
             }
         }
 
-        // Build Google/Gemini tool call mapping for position-based correlation
-        // This is needed because Google's native format loses tool call IDs
+        // Build position-based mapping for Gemini (which loses tool call IDs in native format)
         if (providerID === 'google' || providerID === 'google-vertex') {
             try {
                 const messagesResponse = await client.session.messages({
@@ -116,8 +113,6 @@ export function createChatParamsHandler(
                                     }
                                     toolCallsByName.get(toolName)!.push(callId)
 
-                                    // Also populate toolParameters for Gemini
-                                    // This is needed for buildPrunableToolsList to work
                                     if (!state.toolParameters.has(callId)) {
                                         state.toolParameters.set(callId, {
                                             tool: part.tool,

lib/pruning-tool.ts

@@ -86,7 +86,6 @@ export function createPruningTool(
 
             const toolMetadata = new Map<string, { tool: string, parameters?: any }>()
             for (const id of prunedIds) {
-                // Look up with lowercase since all IDs are stored lowercase
                 const meta = state.toolParameters.get(id.toLowerCase())
                 if (meta) {
                     toolMetadata.set(id.toLowerCase(), meta)
@@ -144,7 +143,6 @@ async function calculateTokensSaved(
         })
         const messages = messagesResponse.data || messagesResponse
 
-        // Build map of tool call ID -> output content
         const toolOutputs = new Map<string, string>()
         for (const msg of messages) {
             if (msg.role === 'tool' && msg.tool_call_id) {
@@ -153,7 +151,6 @@ async function calculateTokensSaved(
                     : JSON.stringify(msg.content)
                 toolOutputs.set(msg.tool_call_id.toLowerCase(), content)
             }
-            // Handle Anthropic format
             if (msg.role === 'user' && Array.isArray(msg.content)) {
                 for (const part of msg.content) {
                     if (part.type === 'tool_result' && part.tool_use_id) {
@@ -166,7 +163,6 @@ async function calculateTokensSaved(
             }
         }
 
-        // Collect content for pruned outputs
         const contents: string[] = []
         for (const id of prunedIds) {
             const content = toolOutputs.get(id.toLowerCase())
@@ -176,14 +172,12 @@ async function calculateTokensSaved(
         }
 
         if (contents.length === 0) {
-            return prunedIds.length * 500 // fallback estimate
+            return prunedIds.length * 500
         }
 
-        // Estimate tokens
         const tokenCounts = await estimateTokensBatch(contents)
         return tokenCounts.reduce((sum, count) => sum + count, 0)
     } catch (error: any) {
-        // If we can't calculate, estimate based on average
         return prunedIds.length * 500
     }
 }

lib/state/tool-cache.ts

@@ -38,7 +38,6 @@ export function cacheToolParametersFromMessages(
                 })
                     openaiCached++
                 } catch (error) {
-                    // Silently ignore parse errors
                 }
             }
         }
@@ -93,7 +92,6 @@ export function cacheToolParametersFromInput(
             })
             cached++
         } catch (error) {
-            // Silently ignore parse errors
         }
     }