From 36e712c1241952a791b6bb9a77fed4e84e290bd4 Mon Sep 17 00:00:00 2001
From: JF <john.franklin@gmail.com>
Date: Sun, 30 Aug 2026 18:47:56 -0400
Subject: [PATCH 1/4] fix: keep JavaScript globals out of locals (#595)

---
 changelog.d/595.fixed.md                      |  3 ++
 docs/tool-reference.md                        |  2 +-
 .../src/interfaces/adapter-policy-js.ts       | 27 +++++++---------
 .../shared/src/interfaces/adapter-policy.ts   |  6 ++--
 src/server/tool-schemas.ts                    |  2 +-
 src/session/session-manager-data.ts           |  8 +++++
 .../unit/session/session-manager-dap.test.ts  | 32 +++++++++++++++++++
 tests/unit/shared/adapter-policy-js.test.ts   | 13 ++++++++
 8 files changed, 73 insertions(+), 20 deletions(-)
 create mode 100644 changelog.d/595.fixed.md

diff --git a/changelog.d/595.fixed.md b/changelog.d/595.fixed.md
new file mode 100644
index 00000000..cbe7b9e3
--- /dev/null
+++ b/changelog.d/595.fixed.md
@@ -0,0 +1,3 @@
+`get_local_variables` no longer returns JavaScript Global bindings as locals
+when a frame exposes no Local or block scope. It returns an empty list with an
+`anchorNote` directing callers to `get_scopes` and `get_variables` instead.
diff --git a/docs/tool-reference.md b/docs/tool-reference.md
index 8d165bfc..b282269d 100644
--- a/docs/tool-reference.md
+++ b/docs/tool-reference.md
@@ -788,7 +788,7 @@ If the originally stopped thread has no frames, this tool uses the same user-fra
 
 **Language-Specific Behavior:**
 - **Python**: Looks for "Locals" scope, filters out `__builtins__`, special variables, and internal debugger variables
-- **JavaScript**: Merges every block scope on the frame — js-debug names them `Block`, `Catch Block`, `With Block` (or the legacy `Block:<label>`) — ahead of the frame's `Local` scope, innermost block first, so a `let` declared in a `for` body or a `catch (e)` binding is returned alongside the function's own locals and shadows them in the order JavaScript does (#558). `scopeName` stays `Local` with no `note` whenever the frame has a Local scope and any block or local scope produced variables, even if `Local` itself contributed nothing. On an ESM top-level frame (blocks but no `Local`), the frame's `Module`/`Script` scope joins the merge as its base and `scopeName` is the block. When nothing local-like has anything to show, the response falls through to a `Closure` scope, then `Module`/`Script`, on the same frame, and `note` says which scope was used. `Global` is consulted only for frames that expose no local or block scope at all — never as a fall-through, so Node's globals are not reported as locals. Filters out `this`, `__proto__`, and V8 internals
+- **JavaScript**: Merges every block scope on the frame — js-debug names them `Block`, `Catch Block`, `With Block` (or the legacy `Block:<label>`) — ahead of the frame's `Local` scope, innermost block first, so a `let` declared in a `for` body or a `catch (e)` binding is returned alongside the function's own locals and shadows them in the order JavaScript does (#558). `scopeName` stays `Local` with no `note` whenever the frame has a Local scope and any block or local scope produced variables, even if `Local` itself contributed nothing. On an ESM top-level frame (blocks but no `Local`), the frame's `Module`/`Script` scope joins the merge as its base and `scopeName` is the block. An empty Local/block frame may fall through to Closure or Module on that same frame. A frame with no Local or block scope returns empty variables and an `anchorNote` directing you to `get_scopes`/`get_variables`; Global is never mislabeled as locals. Filters out `this`, `__proto__`, and V8 internals.
 - **Ruby**: Reads rdbg's "Local variables" scope, hiding the `%self` pseudo-variable unless `includeSpecial: true`; a frame whose only local is `%self` (a native `[C]` frame) counts as empty
 - **Other Languages**: Falls back to generic behavior (first non-global scope)
 
diff --git a/packages/shared/src/interfaces/adapter-policy-js.ts b/packages/shared/src/interfaces/adapter-policy-js.ts
index bb540f06..3e3135a0 100644
--- a/packages/shared/src/interfaces/adapter-policy-js.ts
+++ b/packages/shared/src/interfaces/adapter-policy-js.ts
@@ -33,8 +33,7 @@ export const JS_SCOPE_KINDS = {
   local: ['Local', 'Locals', 'Local:'],
   block: ['Block', 'Catch Block', 'With Block', 'Block:'],
   closure: ['Closure', 'Closure:'],
-  module: ['Script', 'Module', 'module'],
-  global: ['Global']
+  module: ['Script', 'Module', 'module']
 } as const;
 
 /** Exact match, or prefix match for a name written with a trailing ':'. */
@@ -65,8 +64,8 @@ const isClosureScope = (scope: DebugProtocol.Scope): boolean =>
   matchesScopeNames(scope, JS_SCOPE_KINDS.closure) || scope.name.startsWith('Closure ');
 const isModuleScope = (scope: DebugProtocol.Scope): boolean =>
   matchesScopeNames(scope, JS_SCOPE_KINDS.module) || scope.name.toLowerCase() === 'module';
-const isGlobalScope = (scope: DebugProtocol.Scope): boolean =>
-  scope.name.toLowerCase().includes('global');
+const JS_NO_LOCAL_SCOPE_NOTE =
+  'This JavaScript frame exposes no Local or block scope, so get_local_variables intentionally returned no variables; use get_scopes with this frame ID, then get_variables for an explicit scope (for example Global).';
 
 /**
  * JavaScript-specific adapter state
@@ -204,6 +203,13 @@ export const JsDebugAdapterPolicy: AdapterPolicy = {
     if (!frameScopes || frameScopes.length === 0) {
       return emptyLocalVariableExtraction();
     }
+
+    // Script/Module/Global-only frames have no local-variable contract.
+    // Returning Global can dump hundreds of runtime bindings while labeling
+    // them as locals; leave scope selection explicit instead (#595).
+    if (!frameScopes.some(isLocalLikeScope)) {
+      return emptyLocalVariableExtraction(JS_NO_LOCAL_SCOPE_NOTE);
+    }
     
     const variablesForScope = (scope: DebugProtocol.Scope): Variable[] => {
       let scopeVariables = variables[scope.variablesReference] || [];
@@ -316,16 +322,6 @@ export const JsDebugAdapterPolicy: AdapterPolicy = {
       isClosureScope,
       isModuleScope
     ];
-    // Global is a last resort only for frames that expose no Local or block
-    // scope at all (top-level script frames). It must never be a fall-through
-    // past an empty Local: js-debug marks Global expensive but the session layer
-    // still fetches it, so Node's ~140 globals would be reported as locals
-    // and the non-empty result would keep the issue #468 walk-down to the
-    // caller frame from ever running.
-    if (!frameScopes.some(isLocalLikeScope)) {
-      fallbackGroups.push(isGlobalScope);
-    }
-
     for (const matches of fallbackGroups) {
       for (const scope of frameScopes.filter(matches)) {
         const scopeVariables = variablesForScope(scope);
@@ -354,8 +350,7 @@ export const JsDebugAdapterPolicy: AdapterPolicy = {
       ...JS_SCOPE_KINDS.local,
       ...JS_SCOPE_KINDS.block,
       ...JS_SCOPE_KINDS.closure,
-      ...JS_SCOPE_KINDS.module,
-      ...JS_SCOPE_KINDS.global
+      ...JS_SCOPE_KINDS.module
     ];
   },
   
diff --git a/packages/shared/src/interfaces/adapter-policy.ts b/packages/shared/src/interfaces/adapter-policy.ts
index c25d7da3..c9d128bf 100644
--- a/packages/shared/src/interfaces/adapter-policy.ts
+++ b/packages/shared/src/interfaces/adapter-policy.ts
@@ -90,11 +90,13 @@ export interface LocalVariableExtraction {
    * variable, in the same order. Empty when `variables` is empty.
    */
   scopeRefs: number[];
+  /** Optional explanation for an intentionally empty policy result. */
+  note?: string;
 }
 
 /** The "no locals here" result: no variables, and therefore no scopes. */
-export function emptyLocalVariableExtraction(): LocalVariableExtraction {
-  return { variables: [], scopeRefs: [] };
+export function emptyLocalVariableExtraction(note?: string): LocalVariableExtraction {
+  return { variables: [], scopeRefs: [], ...(note ? { note } : {}) };
 }
 
 /**
diff --git a/src/server/tool-schemas.ts b/src/server/tool-schemas.ts
index e801fe8f..7ac62e44 100644
--- a/src/server/tool-schemas.ts
+++ b/src/server/tool-schemas.ts
@@ -201,7 +201,7 @@ export function buildToolDefinitions(options: BuildToolDefinitionsOptions): Tool
     { name: 'pause_execution', description: 'Pause a running program. Waits briefly for the stop; if the program cannot stop within ~5s (e.g. blocked in native code, or an idle server waiting for input), returns success with pending:true and the session reports "paused" the next time the program runs code. Fails with an actionable error if the session has no debuggable target to pause (e.g. a js attach whose target session was never adopted or has ended)', inputSchema: { type: 'object', properties: { sessionId: { type: 'string' }, threadId: { type: 'number', description: 'Thread ID to pause. If omitted or 0, pauses all threads.' } }, required: ['sessionId'] } },
     { name: 'list_threads', description: 'List all threads in the debugged process', inputSchema: { type: 'object', properties: { sessionId: { type: 'string' } }, required: ['sessionId'] } },
     { name: 'get_variables', description: 'Get variables (scope is variablesReference: number). Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full', inputSchema: { type: 'object', properties: { sessionId: { type: 'string' }, scope: { type: 'number', description: "The variablesReference number from a StackFrame or Variable" }, names: namesProp }, required: getVariablesRequired } },
-    { name: 'get_local_variables', description: 'Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full', inputSchema: { type: 'object', properties: { sessionId: { type: 'string' }, includeSpecial: { type: 'boolean', description: 'Include special/internal variables like this, __proto__, __builtins__, etc. Default: false' }, names: namesProp }, required: getLocalVariablesRequired } },
+    { name: 'get_local_variables', description: 'Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full', inputSchema: { type: 'object', properties: { sessionId: { type: 'string' }, includeSpecial: { type: 'boolean', description: 'Include special/internal variables like this, __proto__, __builtins__, etc. Default: false' }, names: namesProp }, required: getLocalVariablesRequired } },
     { name: 'get_stack_trace', description: 'Get stack trace. The response includes stopReason — why the session is paused. For an implicit frameless stopped thread, a sibling with policy-recognized user frames is preferred and adopted, with the switch disclosed in note. Internal/runtime frames are filtered by default; when any are hidden the response carries hiddenFrames and a note', inputSchema: { type: 'object', properties: { sessionId: { type: 'string' }, includeInternals: { type: 'boolean', description: 'Include internal/framework frames (e.g., Node.js internals). Default: false for cleaner output.' }, threadId: { type: 'number', description: 'Inspect this exact thread (ids from list_threads). Explicit selection is authoritative and is never silently switched' } }, required: ['sessionId'] } },
     { name: 'get_scopes', description: 'Get scopes for a stack frame', inputSchema: { type: 'object', properties: { sessionId: { type: 'string' }, frameId: { type: 'number', description: "The ID of the stack frame from a stackTrace response" } }, required: ['sessionId', 'frameId'] } },
     { name: 'evaluate_expression', description: 'Evaluate expression in the current debug context. Without frameId, uses the same adopted top frame as stack and locals and discloses an automatic thread switch in anchorNote. Expressions can read and modify program state. Waits up to 30s by default', inputSchema: { type: 'object', properties: { sessionId: { type: 'string' }, expression: { type: 'string' }, frameId: { type: 'number', description: 'Optional authoritative stack frame ID from get_stack_trace. When provided, no automatic frame/thread selection occurs' }, timeout: { type: 'number', description: 'Max time (ms) to wait for the evaluation to complete (default: 30000, max: 600000). On expiry the request fails but the expression may keep executing in the debuggee. Note: your MCP client may enforce its own overall request timeout' } }, required: ['sessionId', 'expression'] } },
diff --git a/src/session/session-manager-data.ts b/src/session/session-manager-data.ts
index 6c61e88a..3a30be97 100644
--- a/src/session/session-manager-data.ts
+++ b/src/session/session-manager-data.ts
@@ -372,6 +372,14 @@ export abstract class SessionManagerData extends SessionManagerCore {
         const anchor = frames[0];
         if (policy.extractLocalVariables) {
           const extraction = policy.extractLocalVariables(frames, scopesMap, variablesMap, includeSpecial);
+          if (extraction.variables.length === 0 && extraction.note) {
+            return {
+              localVars: [],
+              scopeRefs: [],
+              scopeName: null,
+              scopeNote: extraction.note
+            };
+          }
 
           // Report the ACTUAL scope name the adapter returned, not the policy's
           // canonical name — adapters may annotate it (e.g. Delve's "Locals
diff --git a/tests/core/unit/session/session-manager-dap.test.ts b/tests/core/unit/session/session-manager-dap.test.ts
index a7a19bfc..c658c159 100644
--- a/tests/core/unit/session/session-manager-dap.test.ts
+++ b/tests/core/unit/session/session-manager-dap.test.ts
@@ -1272,6 +1272,38 @@ describe('SessionManager - DAP Operations', () => {
       expect(result.anchorNote).toMatch(/Module/);
     });
 
+    it('returns empty JS locals with guidance for a Global-only frame (#595)', async () => {
+      const session = await createPausedSession(sessionManager, dependencies);
+      (sessionManager as unknown as { selectPolicy: () => unknown }).selectPolicy = () => JsDebugAdapterPolicy;
+      dependencies.mockProxyManager.sendDapRequest = vi.fn().mockImplementation(
+        async (command: string) => {
+          if (command === 'stackTrace') {
+            return { success: true, body: { stackFrames: [
+              { id: 1, name: '<module>', source: { path: '/workspace/app.js' }, line: 1, column: 1 }
+            ] } };
+          }
+          if (command === 'scopes') {
+            return { success: true, body: { scopes: [
+              { name: 'Global', variablesReference: 400, expensive: true }
+            ] } };
+          }
+          if (command === 'variables') {
+            return { success: true, body: { variables: [
+              { name: 'process', value: 'Process', type: 'object', variablesReference: 2 }
+            ] } };
+          }
+          return { success: true, body: {} };
+        }
+      );
+
+      const result = await sessionManager.getLocalVariables(session.id);
+
+      expect(result.variables).toEqual([]);
+      expect(result.scopeName).toBeNull();
+      expect(result.anchorNote).toMatch(/get_scopes/);
+      expect(result.anchorNote).toMatch(/get_variables/);
+    });
+
     it('returns a JS for-body block binding alongside the function locals (issue #558)', async () => {
       const session = await createPausedSession(sessionManager, dependencies);
       (sessionManager as unknown as { selectPolicy: () => unknown }).selectPolicy = () => JsDebugAdapterPolicy;
diff --git a/tests/unit/shared/adapter-policy-js.test.ts b/tests/unit/shared/adapter-policy-js.test.ts
index fdef7e8e..d6b617e2 100644
--- a/tests/unit/shared/adapter-policy-js.test.ts
+++ b/tests/unit/shared/adapter-policy-js.test.ts
@@ -105,6 +105,19 @@ describe('JsDebugAdapterPolicy', () => {
     expect(withSpecial.variables.map(variable => variable.name)).toContain('this');
   });
 
+  it('does not relabel Global as locals when no Local or block scope exists (#595)', () => {
+    const result = JsDebugAdapterPolicy.extractLocalVariables!(
+      [{ id: 9 }] as any,
+      { 9: [{ name: 'Global', variablesReference: 90, expensive: true }] } as any,
+      { 90: [{ name: 'process', value: 'Process', variablesReference: 1 }] } as any
+    );
+
+    expect(result.variables).toEqual([]);
+    expect(result.scopeRefs).toEqual([]);
+    expect(result.note).toMatch(/get_scopes/);
+    expect(result.note).toMatch(/get_variables/);
+  });
+
   it('determines command queueing based on initialization state', () => {
     const state = JsDebugAdapterPolicy.createInitialState() as any;
 

From 242f2cc36764c2deeb2935ffc3a000cb8a93d12f Mon Sep 17 00:00:00 2001
From: JF <john.franklin@gmail.com>
Date: Sun, 30 Aug 2026 21:43:19 -0400
Subject: [PATCH 2/4] test: ratchet JavaScript locals contract

---
 .../tool-list/bp-assert.va-explicit.container-true.json         | 2 +-
 .../tool-list/bp-assert.va-explicit.container-unset.json        | 2 +-
 .../tool-list/bp-assert.va-open.container-true.json             | 2 +-
 .../tool-list/bp-assert.va-open.container-unset.json            | 2 +-
 .../tool-list/bp-assert.va-unset.container-true.json            | 2 +-
 .../tool-list/bp-assert.va-unset.container-unset.json           | 2 +-
 .../tool-list/bp-content.va-explicit.container-true.json        | 2 +-
 .../tool-list/bp-content.va-explicit.container-unset.json       | 2 +-
 .../tool-list/bp-content.va-open.container-true.json            | 2 +-
 .../tool-list/bp-content.va-open.container-unset.json           | 2 +-
 .../tool-list/bp-content.va-unset.container-true.json           | 2 +-
 .../tool-list/bp-content.va-unset.container-unset.json          | 2 +-
 .../tool-list/bp-line.va-explicit.container-true.json           | 2 +-
 .../tool-list/bp-line.va-explicit.container-unset.json          | 2 +-
 .../__snapshots__/tool-list/bp-line.va-open.container-true.json | 2 +-
 .../tool-list/bp-line.va-open.container-unset.json              | 2 +-
 .../tool-list/bp-line.va-unset.container-true.json              | 2 +-
 .../tool-list/bp-line.va-unset.container-unset.json             | 2 +-
 .../tool-list/bp-unset.va-explicit.container-true.json          | 2 +-
 .../tool-list/bp-unset.va-explicit.container-unset.json         | 2 +-
 .../tool-list/bp-unset.va-open.container-true.json              | 2 +-
 .../tool-list/bp-unset.va-open.container-unset.json             | 2 +-
 .../tool-list/bp-unset.va-unset.container-true.json             | 2 +-
 .../tool-list/bp-unset.va-unset.container-unset.json            | 2 +-
 24 files changed, 24 insertions(+), 24 deletions(-)

diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-explicit.container-true.json b/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-explicit.container-true.json
index 7c9b14e2..ebe43655 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-explicit.container-true.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-explicit.container-true.json
@@ -507,7 +507,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-explicit.container-unset.json b/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-explicit.container-unset.json
index f3c3ce8b..1175ff32 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-explicit.container-unset.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-explicit.container-unset.json
@@ -507,7 +507,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-open.container-true.json b/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-open.container-true.json
index e1b76281..deca9b21 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-open.container-true.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-open.container-true.json
@@ -506,7 +506,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-open.container-unset.json b/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-open.container-unset.json
index 86c8ad82..70959ce3 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-open.container-unset.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-open.container-unset.json
@@ -506,7 +506,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-unset.container-true.json b/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-unset.container-true.json
index e1b76281..deca9b21 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-unset.container-true.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-unset.container-true.json
@@ -506,7 +506,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-unset.container-unset.json b/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-unset.container-unset.json
index 86c8ad82..70959ce3 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-unset.container-unset.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-assert.va-unset.container-unset.json
@@ -506,7 +506,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-explicit.container-true.json b/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-explicit.container-true.json
index 562c3442..994db3b8 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-explicit.container-true.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-explicit.container-true.json
@@ -517,7 +517,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-explicit.container-unset.json b/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-explicit.container-unset.json
index 0795abf6..b9b7cc64 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-explicit.container-unset.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-explicit.container-unset.json
@@ -517,7 +517,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-open.container-true.json b/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-open.container-true.json
index e3a6b1e9..e2394ed9 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-open.container-true.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-open.container-true.json
@@ -516,7 +516,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-open.container-unset.json b/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-open.container-unset.json
index c27da61f..2345546e 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-open.container-unset.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-open.container-unset.json
@@ -516,7 +516,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-unset.container-true.json b/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-unset.container-true.json
index e3a6b1e9..e2394ed9 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-unset.container-true.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-unset.container-true.json
@@ -516,7 +516,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-unset.container-unset.json b/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-unset.container-unset.json
index c27da61f..2345546e 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-unset.container-unset.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-content.va-unset.container-unset.json
@@ -516,7 +516,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-explicit.container-true.json b/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-explicit.container-true.json
index 913263c3..15e308af 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-explicit.container-true.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-explicit.container-true.json
@@ -503,7 +503,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-explicit.container-unset.json b/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-explicit.container-unset.json
index 1783a6f4..bcf18f2c 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-explicit.container-unset.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-explicit.container-unset.json
@@ -503,7 +503,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-open.container-true.json b/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-open.container-true.json
index e762286a..1e466d69 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-open.container-true.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-open.container-true.json
@@ -502,7 +502,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-open.container-unset.json b/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-open.container-unset.json
index 666c4498..c77e2b42 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-open.container-unset.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-open.container-unset.json
@@ -502,7 +502,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-unset.container-true.json b/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-unset.container-true.json
index e762286a..1e466d69 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-unset.container-true.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-unset.container-true.json
@@ -502,7 +502,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-unset.container-unset.json b/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-unset.container-unset.json
index 666c4498..c77e2b42 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-unset.container-unset.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-line.va-unset.container-unset.json
@@ -502,7 +502,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-explicit.container-true.json b/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-explicit.container-true.json
index 562c3442..994db3b8 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-explicit.container-true.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-explicit.container-true.json
@@ -517,7 +517,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-explicit.container-unset.json b/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-explicit.container-unset.json
index 0795abf6..b9b7cc64 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-explicit.container-unset.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-explicit.container-unset.json
@@ -517,7 +517,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-open.container-true.json b/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-open.container-true.json
index e3a6b1e9..e2394ed9 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-open.container-true.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-open.container-true.json
@@ -516,7 +516,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-open.container-unset.json b/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-open.container-unset.json
index c27da61f..2345546e 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-open.container-unset.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-open.container-unset.json
@@ -516,7 +516,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-unset.container-true.json b/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-unset.container-true.json
index e3a6b1e9..e2394ed9 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-unset.container-true.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-unset.container-true.json
@@ -516,7 +516,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {
diff --git a/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-unset.container-unset.json b/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-unset.container-unset.json
index c27da61f..2345546e 100644
--- a/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-unset.container-unset.json
+++ b/tests/core/unit/server/__snapshots__/tool-list/bp-unset.va-unset.container-unset.json
@@ -516,7 +516,7 @@
       },
       {
         "name": "get_local_variables",
-        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. Responses are size-guarded: oversized values are cut (truncated:true) and very large scopes return a capped list with a truncation notice — pass names:[...] to fetch specific variables in full",
+        "description": "Get local variables for the shared inspection anchor. If the stopped thread is frameless, a sibling with policy-recognized user frames is preferred, adopted, and disclosed in anchorNote. A JavaScript frame with no Local/block scope returns empty with guidance instead of mislabeling Global bindings. Responses are size-guarded; pass names:[...] to fetch specific variables in full",
         "inputSchema": {
           "type": "object",
           "properties": {

From afca1dd9466282cca4196351d564ad92afddba22 Mon Sep 17 00:00:00 2001
From: JF <john.franklin@gmail.com>
Date: Sun, 30 Aug 2026 21:44:45 -0400
Subject: [PATCH 3/4] test: keep JS scope-name contract closed

---
 packages/shared/tests/unit/adapter-policy-js.spec.ts | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/packages/shared/tests/unit/adapter-policy-js.spec.ts b/packages/shared/tests/unit/adapter-policy-js.spec.ts
index a171b41b..ef6ea930 100644
--- a/packages/shared/tests/unit/adapter-policy-js.spec.ts
+++ b/packages/shared/tests/unit/adapter-policy-js.spec.ts
@@ -531,8 +531,7 @@ describe('JsDebugAdapterPolicy', () => {
         ...JS_SCOPE_KINDS.local,
         ...JS_SCOPE_KINDS.block,
         ...JS_SCOPE_KINDS.closure,
-        ...JS_SCOPE_KINDS.module,
-        ...JS_SCOPE_KINDS.global
+        ...JS_SCOPE_KINDS.module
       ]);
     });
 

From 6a502075cbed424235db3502c551acf1e53ea126 Mon Sep 17 00:00:00 2001
From: JF <john.franklin@gmail.com>
Date: Sun, 30 Aug 2026 21:51:16 -0400
Subject: [PATCH 4/4] test: reject JavaScript globals as locals

---
 packages/shared/tests/unit/adapter-policy-js.spec.ts | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/packages/shared/tests/unit/adapter-policy-js.spec.ts b/packages/shared/tests/unit/adapter-policy-js.spec.ts
index ef6ea930..2648ed99 100644
--- a/packages/shared/tests/unit/adapter-policy-js.spec.ts
+++ b/packages/shared/tests/unit/adapter-policy-js.spec.ts
@@ -221,7 +221,7 @@ describe('JsDebugAdapterPolicy', () => {
       expect(JsDebugAdapterPolicy.extractLocalVariables!([frame], scopes, vars)).toEqual({ variables: [], scopeRefs: [] });
     });
 
-    it('uses Global only for a frame that exposes no Local scope at all', () => {
+    it('never substitutes Global for a frame that exposes no Local scope', () => {
       const scopes: Record<number, DebugProtocol.Scope[]> = {
         1: [
           { name: 'Script', variablesReference: 100, expensive: false },
@@ -235,7 +235,8 @@ describe('JsDebugAdapterPolicy', () => {
 
       const result = JsDebugAdapterPolicy.extractLocalVariables!([frame], scopes, vars);
 
-      expect(result.variables.map(variable => variable.name)).toEqual(['globalValue']);
+      expect(result).toMatchObject({ variables: [], scopeRefs: [] });
+      expect(result.note).toMatch(/get_scopes.*get_variables/);
     });
 
     it('merges a for-body Block scope ahead of the function locals (issue #558)', () => {