fix: filter function variables

Author: dkattan

.gitignore

@@ -6,3 +6,4 @@ dist
 score_tally.txt
 coverage
 .nyc_output
+Issues.md

CHANGELOG.md

@@ -82,7 +82,7 @@ serverReady: {
 
 ## [0.0.16] - 2025-11-20
 
-**Documentation:** Clarified that `variableFilter` values are exact, case-sensitive variable names (no regex support). Removed prior regex-style examples (`^(user|session)$`, `^order_`) from README and replaced with explicit name lists. Added note that `resume_debug_session` breakpoints may omit `variableFilter` (optional) while `start_debugger_with_breakpoints` requires it per breakpoint to keep responses compact.
+**Documentation:** Clarified that `variableFilter` values are exact, case-sensitive variable names (no regex support). Removed prior regex-style examples (`^(user|session)$`, `^order_`) from README and replaced with explicit name lists. Added note that `resumeDebugSession` breakpoints may omit `variableFilter` (optional) while `startDebugSessionWithBreakpoints` requires it per breakpoint to keep responses compact.
 
 **Added:** Expanded README action guidance (difference between `break`, `capture`, `stopDebugging`), release workflow steps, and improved examples for capture interpolation.
 

Issues.md

@@ -6,12 +6,12 @@ Use GitHub Copilot (or any LM-enabled workflow in VS Code) to start, inspect, an
 
 The extension contributes Language Model Tools that Copilot can invoke:
 
-1. **Start Debugger** (`start_debugger_with_breakpoints`) – Launch a configured debug session and wait for the first breakpoint. You must supply at least one breakpoint, but `variableFilter` is optional: include it to narrow captured variables, omit it for a pure pause (`break` / `stopDebugging`) or automatic capture-all (`capture` action).
-2. **Resume Debug Session** (`resume_debug_session`) – Continue execution of an existing paused session and optionally wait for the next stop (new breakpoints added during resume may omit `variableFilter` if you only need a pause, but include it for scoped variable output or interpolation).
-3. **Get Variables** (`get_variables`) – Retrieve all variables in the current top stack frame scopes.
-4. **Expand Variable** (`expand_variable`) – Drill into a single variable to inspect its immediate children.
-5. **Evaluate Expression** (`evaluate_expression`) – Run an arbitrary expression (like the Debug Console) in the paused stack frame.
-6. **Stop Debug Session** (`stop_debug_session`) – Terminate matching debug sessions when you’re done.
+1. **Start Debugger** (`startDebugSessionWithBreakpoints`) – Launch a configured debug session and wait for the first breakpoint. You must supply at least one breakpoint, but `variableFilter` is optional: include it to narrow captured variables, omit it for a pure pause (`break` / `stopDebugging`) or automatic capture-all (`capture` action).
+2. **Resume Debug Session** (`resumeDebugSession`) – Continue execution of an existing paused session and optionally wait for the next stop (new breakpoints added during resume may omit `variableFilter` if you only need a pause, but include it for scoped variable output or interpolation).
+3. **Get Variables** (`getVariables`) – Retrieve all variables in the current top stack frame scopes.
+4. **Expand Variable** (`expandVariable`) – Drill into a single variable to inspect its immediate children.
+5. **Evaluate Expression** (`evaluateExpression`) – Run an arbitrary expression (like the Debug Console) in the paused stack frame.
+6. **Stop Debug Session** (`stopDebugSession`) – Terminate matching debug sessions when you’re done.
 
 All tools return structured data that Copilot can reason over (JSON-like text parts containing call stacks, variables, and metadata).
 
@@ -110,7 +110,7 @@ Default        : `false`
 
 The repo already ships with a workspace-level Run on Save configuration (see `.vscode/settings.json`) that fires `npm run update` any time `package.json` is saved. Just install the [Run on Save](https://marketplace.visualstudio.com/items?itemName=emeraldwalk.RunOnSave) extension when VS Code recommends it and the tables will stay in sync automatically.
 
-> **Important (updated):** `start_debugger_with_breakpoints` requires at least one breakpoint. `variableFilter` is **only required** when you want a _subset_ of variables for a `capture` action. If you set `action: "capture"` and omit `variableFilter`, the tool auto-captures the first `captureMaxVariables` locals (case‑sensitive exact names) to reduce friction. For `break` or `stopDebugging` actions, omit `variableFilter` for a pure pause without variable output.
+> **Important (updated):** `startDebugSessionWithBreakpoints` requires at least one breakpoint. `variableFilter` is **only required** when you want a _subset_ of variables for a `capture` action. If you set `action: "capture"` and omit `variableFilter`, the tool auto-captures the first `captureMaxVariables` locals (case‑sensitive exact names) to reduce friction. For `break` or `stopDebugging` actions, omit `variableFilter` for a pure pause without variable output.
 
 ### Entry Timeout Diagnostics
 

README.md

@@ -14,7 +14,7 @@
   "bugs": {
     "url": "https://github.com/dkattan/Copilot-Breakpoint-Debugger/issues"
   },
-        "when": "debugState == 'running'",
+  "when": "debugState == 'running'",
   "keywords": [
     "copilot",
     "debug",
@@ -118,8 +118,8 @@
       }
     },
     "languageModelTools": [
-        {
-          "name": "startDebugSessionWithBreakpoints",
+      {
+        "name": "startDebugSessionWithBreakpoints",
         "displayName": "Start Debugger",
         "toolReferenceName": "startDebugSessionWithBreakpoints",
         "canBeReferencedInPrompt": true,
@@ -328,8 +328,8 @@
           ]
         }
       },
-        {
-          "name": "resumeDebugSession",
+      {
+        "name": "resumeDebugSession",
         "displayName": "Resume Debug Session",
         "toolReferenceName": "resumeDebugSession",
         "canBeReferencedInPrompt": true,
@@ -404,8 +404,8 @@
           ]
         }
       },
-        {
-          "name": "getVariables",
+      {
+        "name": "getVariables",
         "displayName": "Get Variables",
         "toolReferenceName": "getVariables",
         "canBeReferencedInPrompt": true,
@@ -416,8 +416,8 @@
           "properties": {}
         }
       },
-        {
-          "name": "expandVariable",
+      {
+        "name": "expandVariable",
         "displayName": "Expand Variable",
         "toolReferenceName": "expandVariable",
         "canBeReferencedInPrompt": true,
@@ -436,8 +436,8 @@
           ]
         }
       },
-        {
-          "name": "evaluateExpression",
+      {
+        "name": "evaluateExpression",
         "displayName": "Evaluate Expression",
         "toolReferenceName": "evaluateExpression",
         "canBeReferencedInPrompt": true,
@@ -460,8 +460,8 @@
           ]
         }
       },
-        {
-          "name": "stopDebugSession",
+      {
+        "name": "stopDebugSession",
         "displayName": "Stop Debug Session",
         "toolReferenceName": "stopDebugSession",
         "canBeReferencedInPrompt": true,

package.json

@@ -115,7 +115,18 @@ export class DAPHelpers {
     if (!variablesResponse?.variables) {
       return [];
     }
-    return variablesResponse.variables.map((v: Variable) => ({
+    const filtered = variablesResponse.variables.filter((v: Variable) => {
+      const type = v.type?.toLowerCase();
+      if (type === "function") {
+        return false;
+      }
+      if (!type && typeof v.value === "string") {
+        // Some adapters omit type but include a "function ..." value string.
+        return !v.value.startsWith("function");
+      }
+      return true;
+    });
+    return filtered.map((v: Variable) => ({
       name: v.evaluateName || v.name,
       value: v.value,
       type: v.type,

src/debugUtils.ts

@@ -19,19 +19,19 @@ export const activate = extension.activate;
 export const deactivate = extension.deactivate;
 
 function registerTools(context: vscode.ExtensionContext) {
-  context.subscriptions.push(
-    vscode.lm.registerTool(
-      "start_debugger_with_breakpoints",
-      new StartDebuggerTool()
-    ),
-    vscode.lm.registerTool(
-      "resume_debug_session",
-      new ResumeDebugSessionTool()
-    ),
-    vscode.lm.registerTool("get_variables", new GetVariablesTool()),
-    vscode.lm.registerTool("expand_variable", new ExpandVariableTool()),
-    vscode.lm.registerTool("evaluate_expression", new EvaluateExpressionTool()),
-    vscode.lm.registerTool("stop_debug_session", new StopDebugSessionTool()),
+    context.subscriptions.push(
+      vscode.lm.registerTool(
+        "startDebugSessionWithBreakpoints",
+        new StartDebuggerTool()
+      ),
+      vscode.lm.registerTool(
+        "resumeDebugSession",
+        new ResumeDebugSessionTool()
+      ),
+      vscode.lm.registerTool("getVariables", new GetVariablesTool()),
+      vscode.lm.registerTool("expandVariable", new ExpandVariableTool()),
+      vscode.lm.registerTool("evaluateExpression", new EvaluateExpressionTool()),
+      vscode.lm.registerTool("stopDebugSession", new StopDebugSessionTool()),
 
     vscode.commands.registerCommand(
       "copilotBreakpointDebugger.startAndWaitManual",

src/extension.ts

@@ -232,6 +232,40 @@ describe("debugUtils - DAPHelpers", () => {
     assert.strictEqual(found, null, "Should not find non-existent variable");
   });
 
+  it("filters function-typed variables from capture", async () => {
+    const functionBlockLine = 28; // console.log after numberVar/fnVar definitions
+    const context = await startDebuggingAndWaitForStop(
+      Object.assign({}, baseParams, {
+        sessionName: "",
+        breakpointConfig: {
+          breakpoints: [
+            {
+              path: scriptPath,
+              line: functionBlockLine,
+              action: "break" as const,
+              variableFilter: ["numberVar", "fnVar"],
+            },
+          ],
+        },
+      })
+    );
+
+    const activeSession = vscode.debug.activeDebugSession;
+    assert.ok(activeSession, "No active debug session");
+
+    const capturedNames = context.scopeVariables.flatMap((scope) =>
+      scope.variables.map((v) => v.name)
+    );
+    assert.ok(
+      capturedNames.includes("numberVar"),
+      "Expected numberVar to be captured"
+    );
+    assert.ok(
+      !capturedNames.includes("fnVar"),
+      "Function-typed variable fnVar should be filtered out"
+    );
+  });
+
   it("getDebugContext works in active session", async () => {
     await startDebuggingAndWaitForStop(
       Object.assign({}, baseParams, {

src/test/debugUtils.test.ts

@@ -21,3 +21,8 @@ console.log('Completed loop');
 setTimeout(() => {
   console.log('Exiting after idle wait.');
 }, 3000);
+
+// Function-type variable for filtering behavior tests
+const numberVar = 42;
+const fnVar = () => numberVar;
+console.log('Function var test', numberVar, typeof fnVar);