refactor: rename nudgeGap to nudgeFrequency and fix config inconsistencies

Author: Tarquinen

README.md

@@ -120,6 +120,8 @@ DCP uses its own config file:
 >             //     "openai/gpt-5": 120000,
 >             //     "anthropic/claude-3-7-sonnet": "80%"
 >             // },
+>             // How often the context-limit nudge fires (1 = every fetch, 5 = every 5th)
+>             "nudgeFrequency": 5,
 >             // Additional tools to protect from pruning
 >             "protectedTools": [],
 >         },

dcp.schema.json

@@ -112,9 +112,9 @@
                     "description": "General tool settings",
                     "additionalProperties": false,
                     "properties": {
-                        "nudgeGap": {
+                        "nudgeFrequency": {
                             "type": "number",
-                            "default": 1,
+                            "default": 5,
                             "minimum": 1,
                             "description": "How often the context-limit nudge fires when above contextLimit (1 = every fetch, 5 = every 5th fetch)"
                         },

lib/config.ts

@@ -17,7 +17,7 @@ export interface CompressTool {
 }
 
 export interface ToolSettings {
-    nudgeGap: number
+    nudgeFrequency: number
     protectedTools: string[]
     contextLimit: number | `${number}%`
     modelLimits?: Record<string, number | `${number}%`>
@@ -101,7 +101,7 @@ export const VALID_CONFIG_KEYS = new Set([
     "manualMode.automaticStrategies",
     "tools",
     "tools.settings",
-    "tools.settings.nudgeGap",
+    "tools.settings.nudgeFrequency",
     "tools.settings.protectedTools",
     "tools.settings.contextLimit",
     "tools.settings.modelLimits",
@@ -289,26 +289,16 @@ export function validateConfigTypes(config: Record<string, any>): ValidationErro
     if (tools) {
         if (tools.settings) {
             if (
-                tools.settings.nudgeGap !== undefined &&
-                typeof tools.settings.nudgeGap !== "number"
+                tools.settings.nudgeFrequency !== undefined &&
+                typeof tools.settings.nudgeFrequency !== "number"
             ) {
                 errors.push({
-                    key: "tools.settings.nudgeGap",
+                    key: "tools.settings.nudgeFrequency",
                     expected: "number",
-                    actual: typeof tools.settings.nudgeGap,
+                    actual: typeof tools.settings.nudgeFrequency,
                 })
             }
 
-            if (
-                typeof tools.settings.nudgeFrequency === "number" &&
-                tools.settings.nudgeFrequency < 1
-            ) {
-                errors.push({
-                    key: "tools.settings.nudgeFrequency",
-                    expected: "positive number (>= 1)",
-                    actual: `${tools.settings.nudgeFrequency} (will be clamped to 1)`,
-                })
-            }
             if (
                 typeof tools.settings.nudgeFrequency === "number" &&
                 tools.settings.nudgeFrequency < 1
@@ -552,7 +542,7 @@ const defaultConfig: PluginConfig = {
     protectedFilePatterns: [],
     tools: {
         settings: {
-            nudgeGap: 5,
+            nudgeFrequency: 5,
             protectedTools: [...DEFAULT_PROTECTED_TOOLS],
             contextLimit: 100000,
         },
@@ -716,7 +706,7 @@ function mergeTools(base: PluginConfig["tools"], override?: ToolOverride): Plugi
 
     return {
         settings: {
-            nudgeGap: override.settings?.nudgeGap ?? base.settings.nudgeGap,
+            nudgeFrequency: override.settings?.nudgeFrequency ?? base.settings.nudgeFrequency,
             protectedTools: [
                 ...new Set([
                     ...base.settings.protectedTools,

lib/messages/inject/inject.ts

@@ -7,7 +7,7 @@ import {
     addAnchor,
     applyAnchoredNudge,
     findLastNonIgnoredMessage,
-    getNudgeGap,
+    getNudgeFrequency,
     getModelInfo,
     isContextOverLimit,
     messageHasCompress,
@@ -67,7 +67,7 @@ export const insertCompressToolContext = (
     if (isContextOverLimit(config, state, providerId, modelId, messages)) {
         const lastNonIgnoredMessage = findLastNonIgnoredMessage(messages)
         if (lastNonIgnoredMessage) {
-            const interval = getNudgeGap(config)
+            const interval = getNudgeFrequency(config)
             const added = addAnchor(
                 state.contextLimitAnchors,
                 lastNonIgnoredMessage.message.info.id,

lib/messages/inject/utils.ts

@@ -17,8 +17,8 @@ export interface LastNonIgnoredMessage {
     index: number
 }
 
-export function getNudgeGap(config: PluginConfig): number {
-    return Math.max(1, Math.floor(config.tools.settings.nudgeGap || 1))
+export function getNudgeFrequency(config: PluginConfig): number {
+    return Math.max(1, Math.floor(config.tools.settings.nudgeFrequency || 1))
 }
 
 export function persistAnchors(state: SessionState, logger: Logger): void {