refactor: remove tools.settings.protectedTools

Author: Tarquinen

README.md

@@ -122,8 +122,6 @@ DCP uses its own config file:
 >             // },
 >             // How often the context-limit nudge fires (1 = every fetch, 5 = every 5th)
 >             "nudgeFrequency": 5,
->             // Additional tools to protect from pruning
->             "protectedTools": [],
 >         },
 >         // Unified context compression tool
 >         "compress": {
@@ -176,7 +174,7 @@ DCP provides a `/dcp` slash command:
 By default, these tools are always protected from pruning:
 `task`, `todowrite`, `todoread`, `compress`, `batch`, `plan_enter`, `plan_exit`
 
-The `protectedTools` arrays in each section add to this default list.
+The `protectedTools` arrays in `commands` and `strategies` add to this default list.
 
 ### Config Precedence
 

dcp.schema.json

@@ -118,14 +118,6 @@
                             "minimum": 1,
                             "description": "How often the context-limit nudge fires when above contextLimit (1 = every fetch, 5 = every 5th fetch)"
                         },
-                        "protectedTools": {
-                            "type": "array",
-                            "items": {
-                                "type": "string"
-                            },
-                            "default": [],
-                            "description": "Tool names that should be protected from automatic pruning"
-                        },
                         "contextLimit": {
                             "description": "When session tokens exceed this limit, a compress nudge is injected (\"X%\" uses percentage of the model's context window)",
                             "default": 100000,

lib/config.ts

@@ -18,7 +18,6 @@ export interface CompressTool {
 
 export interface ToolSettings {
     nudgeFrequency: number
-    protectedTools: string[]
     contextLimit: number | `${number}%`
     modelLimits?: Record<string, number | `${number}%`>
 }
@@ -102,7 +101,6 @@ export const VALID_CONFIG_KEYS = new Set([
     "tools",
     "tools.settings",
     "tools.settings.nudgeFrequency",
-    "tools.settings.protectedTools",
     "tools.settings.contextLimit",
     "tools.settings.modelLimits",
     "tools.compress",
@@ -309,17 +307,6 @@ export function validateConfigTypes(config: Record<string, any>): ValidationErro
                     actual: `${tools.settings.nudgeFrequency} (will be clamped to 1)`,
                 })
             }
-            if (
-                tools.settings.protectedTools !== undefined &&
-                !Array.isArray(tools.settings.protectedTools)
-            ) {
-                errors.push({
-                    key: "tools.settings.protectedTools",
-                    expected: "string[]",
-                    actual: typeof tools.settings.protectedTools,
-                })
-            }
-
             if (tools.settings.contextLimit !== undefined) {
                 const isValidNumber = typeof tools.settings.contextLimit === "number"
                 const isPercentString =
@@ -543,7 +530,6 @@ const defaultConfig: PluginConfig = {
     tools: {
         settings: {
             nudgeFrequency: 5,
-            protectedTools: [...DEFAULT_PROTECTED_TOOLS],
             contextLimit: 100000,
         },
         compress: {
@@ -707,12 +693,6 @@ function mergeTools(base: PluginConfig["tools"], override?: ToolOverride): Plugi
     return {
         settings: {
             nudgeFrequency: override.settings?.nudgeFrequency ?? base.settings.nudgeFrequency,
-            protectedTools: [
-                ...new Set([
-                    ...base.settings.protectedTools,
-                    ...(override.settings?.protectedTools ?? []),
-                ]),
-            ],
             contextLimit: override.settings?.contextLimit ?? base.settings.contextLimit,
             modelLimits: override.settings?.modelLimits ?? base.settings.modelLimits,
         },
@@ -765,7 +745,6 @@ function deepCloneConfig(config: PluginConfig): PluginConfig {
         tools: {
             settings: {
                 ...config.tools.settings,
-                protectedTools: [...config.tools.settings.protectedTools],
                 modelLimits: { ...config.tools.settings.modelLimits },
             },
             compress: { ...config.tools.compress },

lib/state/tool-cache.ts

@@ -51,9 +51,7 @@ export function syncToolCache(
                     continue
                 }
 
-                const allProtectedTools = config.tools.settings.protectedTools
-                const isProtectedTool = allProtectedTools.includes(part.tool)
-                const tokenCount = isProtectedTool ? undefined : countToolTokens(part)
+                const tokenCount = countToolTokens(part)
 
                 state.toolParameters.set(part.callID, {
                     tool: part.tool,