@razdolbai/merls 1.2.2 → 1.3.0

package/AGENTS.md

@@ -1,64 +1,64 @@
-# Repository Guidelines
-
-## Project Structure & Module Organization
-
-This repository has an established baseline. The main document is `README.md` for scope. The runtime is Node.js with TypeScript and `vscode-languageserver`.
-
-When the workspace is scaffolded, keep code under `src/`, tests under `test/`, and editor integration examples under `examples/` such as `examples/coc-settings.json`. Store parser fixtures in a dedicated test-fixture area and separate valid 6502 cases from invalid 65816 cases.
-
-The current bootstrap server entrypoint is `src/server.ts`, and the packaged CLI entrypoint is `src/cli.ts`, which compiles to `dist/src/cli.js`.
-The repository now includes `examples/coc-settings.json` as the baseline coc.nvim launch example, targeting `dist/src/cli.js --stdio` with `.git` and `package.json` root markers.
-The initial positive fixture corpus now lives under `test/fixtures/valid/` and is transcribed from upstream Merlin32 sources.
-The initial negative fixture corpus now lives under `test/fixtures/invalid/` and captures unsupported 65816-only syntax.
-Shared 6502 opcode and Merlin directive metadata now lives under `src/asm/metadata.ts`.
-Shared token-kind and line-shape metadata now lives under `src/asm/syntax.ts`.
-The current lexer implementation now lives under `src/asm/lexer.ts` and is covered by fixture-driven unit tests.
-The current expression parser now lives under `src/asm/expression.ts` and is covered by unit tests for numeric forms, modifiers, arithmetic, and indexed operands.
-The current line parser now lives under `src/asm/parser.ts` and classifies equates, instructions, directives, data lines, and malformed input.
-The current document model now lives under `src/asm/document.ts` and preserves line-by-line structure while collecting malformed-line errors.
-The current symbol collector now lives under `src/asm/symbols.ts` and indexes labels, equates, and named storage/data definitions.
-The current local-label resolver now lives under `src/asm/local-labels.ts` and resolves Merlin `]local` and `:local` labels within the nearest global-label scope.
-The current workspace indexer now lives under `src/asm/workspace.ts` and follows `asm`/`put`/`use` directives across the local fixture corpus.
-The current diagnostics pass now lives under `src/asm/diagnostics.ts` and reports duplicate symbols, unresolved references, malformed lines, and unsupported 65816-only syntax.
-The current `textDocument/documentSymbol` provider is wired through `src/server.ts` and `src/lsp/document-symbols.ts`.
-The current `workspace/symbol` provider is wired through `src/server.ts` and `src/lsp/workspace-symbols.ts` over the open-document symbol set.
-The current `textDocument/definition` and `textDocument/references` handlers are wired through `src/server.ts` and `src/lsp/symbol-navigation.ts`.
-The current `textDocument/hover` handler is wired through `src/server.ts` and `src/lsp/hover.ts`.
-The current `textDocument/completion` handler is wired through `src/server.ts` and `src/lsp/completion.ts`.
-The current `textDocument/semanticTokens` handler is wired through `src/server.ts` and `src/lsp/semantic-tokens.ts`.
-The current `textDocument/publishDiagnostics` path is wired through `src/server.ts` and `src/lsp/diagnostics.ts`, with full-document sync on open/change so editor clients receive live parser and resolver diagnostics.
-The current packaged CLI entrypoint lives under `src/cli.ts` and is covered by a compiled stdio launch-contract integration test.
-The coc.nvim smoke test now lives under `test/smoke/` with a headless Vim runner (`run-smoke.ps1`) and a minimal vimrc that isolates the test from the user's real Vim configuration.
-
-## Build, Test, and Development Commands
-
-The repository now includes the initial Node/TypeScript workspace scaffold:
-
-- `npm install`: install project dependencies.
-- `npm run build`: compile TypeScript sources to `dist/`.
-- `npm test`: build the project and run the current test suite.
-- `npm run dev`: run the TypeScript compiler in watch mode during bootstrap work.
-
-The supported stdio launch contract is now `merls --stdio`, with `node dist/src/cli.js --stdio` as the equivalent local-development invocation.
-
-## Coding Style & Naming Conventions
-
-Use TypeScript throughout the implementation. Prefer small modules with explicit types and single-purpose exports. Use `camelCase` for variables and functions, `PascalCase` for types and classes, and kebab-case for example/config file names.
-
-Keep parser, symbol, and LSP layers separate. Name tests and fixtures after the behavior they cover, for example `parser.labels.test.ts` or `fixtures/invalid/65816-long-a.asm`.
-
-## Testing Guidelines
-
-TDD is mandatory in this repository: add or extend a failing test or fixture before implementation. Positive fixtures must cover supported Merlin-style 6502 syntax. Negative fixtures must explicitly cover unsupported 65816 syntax and expected diagnostics.
-
-Prefer focused unit tests for lexer/parser behavior and integration tests for LSP requests such as `initialize`, hover, and definition.
-
-The current bootstrap test suite already includes compiled-stdio integration checks for the packaged CLI contract, `initialize`, and diagnostics publication.
-
-## Commit & Pull Request Guidelines
-
-This workspace does not currently include accessible git history, so no local commit convention can be inferred from prior commits. Use short, imperative commit subjects such as `Add lexer token fixtures`.
-
-Task completion includes documentation maintenance. When a change affects repository behavior, structure, workflow, or contributor expectations, update both `README.md` and `AGENTS.md` in the same task.
-
-PRs should describe the user-visible behavior change, list the tests added or updated, and include sample diagnostics or editor screenshots when LSP behavior changes. Keep PRs scoped to one phase task or one coherent feature.
+# Repository Guidelines
+
+## Project Structure & Module Organization
+
+This repository has an established baseline. The main document is `README.md` for scope. The runtime is Node.js with TypeScript and `vscode-languageserver`.
+
+When the workspace is scaffolded, keep code under `src/`, tests under `test/`, and editor integration examples under `examples/` such as `examples/coc-settings.json`. Store parser fixtures in a dedicated test-fixture area and separate valid 6502 cases from invalid 65816 cases.
+
+The current bootstrap server entrypoint is `src/server.ts`, and the packaged CLI entrypoint is `src/cli.ts`, which compiles to `dist/src/cli.js`.
+The repository now includes `examples/coc-settings.json` as the baseline coc.nvim launch example, targeting `dist/src/cli.js --stdio` with `.git` and `package.json` root markers.
+The initial positive fixture corpus now lives under `test/fixtures/valid/` and is transcribed from upstream Merlin32 sources.
+The initial negative fixture corpus now lives under `test/fixtures/invalid/` and captures unsupported 65816-only syntax.
+Shared 6502 opcode and Merlin directive metadata now lives under `src/asm/metadata.ts`.
+Shared token-kind and line-shape metadata now lives under `src/asm/syntax.ts`.
+The current lexer implementation now lives under `src/asm/lexer.ts` and is covered by fixture-driven unit tests.
+The current expression parser now lives under `src/asm/expression.ts` and is covered by unit tests for numeric forms, modifiers, arithmetic, and indexed operands.
+The current line parser now lives under `src/asm/parser.ts` and classifies equates, instructions, directives, data lines, and malformed input.
+The current document model now lives under `src/asm/document.ts` and preserves line-by-line structure while collecting malformed-line errors.
+The current symbol collector now lives under `src/asm/symbols.ts` and indexes labels, equates, and named storage/data definitions.
+The current local-label resolver now lives under `src/asm/local-labels.ts` and resolves Merlin `]local` and `:local` labels within the nearest global-label scope.
+The current workspace indexer now lives under `src/asm/workspace.ts` and follows `asm`/`put`/`use` directives across the local fixture corpus.
+The current diagnostics pass now lives under `src/asm/diagnostics.ts` and reports duplicate symbols, unresolved references, malformed lines, and unsupported 65816-only syntax.
+The current `textDocument/documentSymbol` provider is wired through `src/server.ts` and `src/lsp/document-symbols.ts`.
+The current `workspace/symbol` provider is wired through `src/server.ts` and `src/lsp/workspace-symbols.ts` over the open-document symbol set.
+The current `textDocument/definition` and `textDocument/references` handlers are wired through `src/server.ts` and `src/lsp/symbol-navigation.ts`.
+The current `textDocument/hover` handler is wired through `src/server.ts` and `src/lsp/hover.ts`.
+The current `textDocument/completion` handler is wired through `src/server.ts` and `src/lsp/completion.ts`.
+The current `textDocument/semanticTokens` handler is wired through `src/server.ts` and `src/lsp/semantic-tokens.ts`.
+The current `textDocument/publishDiagnostics` path is wired through `src/server.ts` and `src/lsp/diagnostics.ts`, with full-document sync on open/change so editor clients receive live parser and resolver diagnostics.
+The current packaged CLI entrypoint lives under `src/cli.ts` and is covered by a compiled stdio launch-contract integration test.
+The coc.nvim smoke test now lives under `test/smoke/` with a headless Vim runner (`run-smoke.ps1`) and a minimal vimrc that isolates the test from the user's real Vim configuration.
+
+## Build, Test, and Development Commands
+
+The repository now includes the initial Node/TypeScript workspace scaffold:
+
+- `npm install`: install project dependencies.
+- `npm run build`: compile TypeScript sources to `dist/`.
+- `npm test`: build the project and run the current test suite.
+- `npm run dev`: run the TypeScript compiler in watch mode during bootstrap work.
+
+The supported stdio launch contract is now `merls --stdio`, with `node dist/src/cli.js --stdio` as the equivalent local-development invocation.
+
+## Coding Style & Naming Conventions
+
+Use TypeScript throughout the implementation. Prefer small modules with explicit types and single-purpose exports. Use `camelCase` for variables and functions, `PascalCase` for types and classes, and kebab-case for example/config file names.
+
+Keep parser, symbol, and LSP layers separate. Name tests and fixtures after the behavior they cover, for example `parser.labels.test.ts` or `fixtures/invalid/65816-long-a.S`.
+
+## Testing Guidelines
+
+TDD is mandatory in this repository: add or extend a failing test or fixture before implementation. Positive fixtures must cover supported Merlin-style 6502 syntax. Negative fixtures must explicitly cover unsupported 65816 syntax and expected diagnostics.
+
+Prefer focused unit tests for lexer/parser behavior and integration tests for LSP requests such as `initialize`, hover, and definition.
+
+The current bootstrap test suite already includes compiled-stdio integration checks for the packaged CLI contract, `initialize`, and diagnostics publication.
+
+## Commit & Pull Request Guidelines
+
+This workspace does not currently include accessible git history, so no local commit convention can be inferred from prior commits. Use short, imperative commit subjects such as `Add lexer token fixtures`.
+
+Task completion includes documentation maintenance. When a change affects repository behavior, structure, workflow, or contributor expectations, update both `README.md` and `AGENTS.md` in the same task.
+
+PRs should describe the user-visible behavior change, list the tests added or updated, and include sample diagnostics or editor screenshots when LSP behavior changes. Keep PRs scoped to one phase task or one coherent feature.

package/dist/src/lsp/completion.js

@@ -11,22 +11,25 @@ function buildCompletionItems(openDocuments, uri, line) {
         return [];
     }
     const document = (0, document_1.parseDocument)(source);
-    const text = document.lines[line]?.node.text.trimStart().toLowerCase() ?? "";
-    if (text === "ld" || text === "ld\n" || text.startsWith("ld")) {
-        return metadata_1.opcodeDefinitions.map((opcode) => ({
+    const completions = [];
+    for (const opcode of metadata_1.opcodeDefinitions) {
+        completions.push({
             label: opcode.mnemonic,
             kind: node_1.CompletionItemKind.Keyword
-        }));
+        });
     }
-    if (text === "du" || text.startsWith("du")) {
-        return metadata_1.directiveDefinitions.map((directive) => ({
+    for (const directive of metadata_1.directiveDefinitions) {
+        completions.push({
             label: directive.name,
             kind: node_1.CompletionItemKind.Function
-        }));
+        });
     }
     const symbols = (0, symbols_1.collectSymbols)(document);
-    return [...symbols.values()].map((symbol) => ({
-        label: symbol.name,
-        kind: node_1.CompletionItemKind.Variable
-    }));
+    for (const symbol of symbols.values()) {
+        completions.push({
+            label: symbol.name,
+            kind: node_1.CompletionItemKind.Variable
+        });
+    }
+    return completions;
 }

package/dist/test/completion.test.js

@@ -46,7 +46,7 @@ function positionOf(text, needle) {
 }
 async function runCompletionTest() {
     const serverPath = node_path_1.default.resolve(__dirname, "../src/server.js");
-    const mainPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.asm");
+    const mainPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.S");
     const mainUri = `file://${mainPath.replace(/\\/g, "/")}`;
     const text = `${node_fs_1.default.readFileSync(mainPath, "utf8")}\n        ld\n        du\n        bpl G`;
     const child = (0, node_child_process_1.spawn)(process.execPath, [serverPath], {

package/dist/test/definition-references.test.js

@@ -46,7 +46,7 @@ function positionOf(text, needle) {
 }
 async function runDefinitionReferencesTest() {
     const serverPath = node_path_1.default.resolve(__dirname, "../src/server.js");
-    const mainPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.asm");
+    const mainPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.S");
     const mainUri = `file://${mainPath.replace(/\\/g, "/")}`;
     const text = node_fs_1.default.readFileSync(mainPath, "utf8");
     const child = (0, node_child_process_1.spawn)(process.execPath, [serverPath], {

package/dist/test/diagnostics.test.js

@@ -20,8 +20,8 @@ function runDiagnosticsTest() {
         "        typ BLAH",
         "        end BLAH"
     ].join("\n");
-    const bankOpsPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/invalid/65816-bank-ops.asm");
-    const longPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/invalid/65816-long-addressing.asm");
+    const bankOpsPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/invalid/65816-bank-ops.S");
+    const longPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/invalid/65816-long-addressing.S");
     const diagnostics = (0, diagnostics_1.collectWorkspaceDiagnostics)([
         {
             filePath: "<memory>",

package/dist/test/document-symbol.test.js

@@ -36,7 +36,7 @@ function decodeMessages(streamBuffer) {
 }
 async function runDocumentSymbolTest() {
     const serverPath = node_path_1.default.resolve(__dirname, "../src/server.js");
-    const fixturePath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.asm");
+    const fixturePath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.S");
     const uri = `file://${fixturePath.replace(/\\/g, "/")}`;
     const text = node_fs_1.default.readFileSync(fixturePath, "utf8");
     const child = (0, node_child_process_1.spawn)(process.execPath, [serverPath], {

package/dist/test/fixture-corpus.test.js

@@ -8,12 +8,12 @@ const strict_1 = __importDefault(require("node:assert/strict"));
 const node_fs_1 = __importDefault(require("node:fs"));
 const node_path_1 = __importDefault(require("node:path"));
 const validFixturePaths = [
-    "test/fixtures/valid/merlin32-linkscript.asm",
-    "test/fixtures/valid/merlin32-main-6502.asm"
+    "test/fixtures/valid/merlin32-linkscript.S",
+    "test/fixtures/valid/merlin32-main-6502.S"
 ];
 const invalidFixturePaths = [
-    "test/fixtures/invalid/65816-bank-ops.asm",
-    "test/fixtures/invalid/65816-long-addressing.asm"
+    "test/fixtures/invalid/65816-bank-ops.S",
+    "test/fixtures/invalid/65816-long-addressing.S"
 ];
 function runFixtureCorpusTest() {
     for (const fixturePath of validFixturePaths) {

package/dist/test/hover.test.js

@@ -53,7 +53,7 @@ function positionOfInMatch(text, needle, offset) {
 }
 async function runHoverTest() {
     const serverPath = node_path_1.default.resolve(__dirname, "../src/server.js");
-    const mainPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.asm");
+    const mainPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.S");
     const mainUri = `file://${mainPath.replace(/\\/g, "/")}`;
     const text = node_fs_1.default.readFileSync(mainPath, "utf8");
     const child = (0, node_child_process_1.spawn)(process.execPath, [serverPath], {

package/dist/test/lexer.test.js

@@ -17,8 +17,8 @@ function findLine(lines, source) {
     return line;
 }
 function runLexerTest() {
-    const mainFixturePath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.asm");
-    const linkFixturePath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-linkscript.asm");
+    const mainFixturePath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.S");
+    const linkFixturePath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-linkscript.S");
     const mainFixture = node_fs_1.default.readFileSync(mainFixturePath, "utf8");
     const linkFixture = node_fs_1.default.readFileSync(linkFixturePath, "utf8");
     const mainLines = (0, lexer_1.lexSource)(mainFixture).lines;
@@ -46,8 +46,8 @@ function runLexerTest() {
         ["mnemonic", "ldx"],
         ["numericLiteral", "$C000"]
     ]);
-    strict_1.default.deepEqual(summarizeTokens(findLine(linkLines, "    asm \"merlin32-main-6502.asm\"").tokens), [
+    strict_1.default.deepEqual(summarizeTokens(findLine(linkLines, "    asm \"merlin32-main-6502.S\"").tokens), [
         ["directive", "asm"],
-        ["string", "\"merlin32-main-6502.asm\""]
+        ["string", "\"merlin32-main-6502.S\""]
     ]);
 }

package/dist/test/local-labels.test.js

@@ -10,7 +10,7 @@ const node_path_1 = __importDefault(require("node:path"));
 const document_1 = require("../src/asm/document");
 const local_labels_1 = require("../src/asm/local-labels");
 function runLocalLabelScopeTest() {
-    const fixturePath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.asm");
+    const fixturePath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.S");
     const source = node_fs_1.default.readFileSync(fixturePath, "utf8");
     const document = (0, document_1.parseDocument)(source);
     const scope = (0, local_labels_1.resolveLocalLabels)(document);

package/dist/test/publish-diagnostics.test.js

@@ -35,7 +35,7 @@ function decodeMessages(streamBuffer) {
 }
 async function runPublishDiagnosticsTest() {
     const serverPath = node_path_1.default.resolve(__dirname, "../src/server.js");
-    const documentPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/invalid/publish-diagnostics.asm");
+    const documentPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/invalid/publish-diagnostics.S");
     const documentUri = `file://${documentPath.replace(/\\/g, "/")}`;
     const brokenText = ["dup     equ 1", "        lda missing", "dup     equ 2", "        adc ("].join("\n");
     const fixedText = ["dup     equ 1", "        lda dup", "        adc #1"].join("\n");

package/dist/test/semantic-tokens.test.js

@@ -36,7 +36,7 @@ function decodeMessages(streamBuffer) {
 }
 async function runSemanticTokensTest() {
     const serverPath = node_path_1.default.resolve(__dirname, "../src/server.js");
-    const mainPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.asm");
+    const mainPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.S");
     const mainUri = `file://${mainPath.replace(/\\/g, "/")}`;
     const text = node_fs_1.default.readFileSync(mainPath, "utf8");
     const child = (0, node_child_process_1.spawn)(process.execPath, [serverPath], {

package/dist/test/symbols.test.js

@@ -10,7 +10,7 @@ const node_path_1 = __importDefault(require("node:path"));
 const document_1 = require("../src/asm/document");
 const symbols_1 = require("../src/asm/symbols");
 function runSymbolsTest() {
-    const fixturePath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.asm");
+    const fixturePath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.S");
     const source = node_fs_1.default.readFileSync(fixturePath, "utf8");
     const document = (0, document_1.parseDocument)(source);
     const symbols = (0, symbols_1.collectSymbols)(document);

package/dist/test/workspace-symbol.test.js

@@ -36,8 +36,8 @@ function decodeMessages(streamBuffer) {
 }
 async function runWorkspaceSymbolTest() {
     const serverPath = node_path_1.default.resolve(__dirname, "../src/server.js");
-    const linkPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-linkscript.asm");
-    const mainPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.asm");
+    const linkPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-linkscript.S");
+    const mainPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.S");
     const linkUri = `file://${linkPath.replace(/\\/g, "/")}`;
     const mainUri = `file://${mainPath.replace(/\\/g, "/")}`;
     const child = (0, node_child_process_1.spawn)(process.execPath, [serverPath], {

package/dist/test/workspace.test.js

@@ -8,8 +8,8 @@ const strict_1 = __importDefault(require("node:assert/strict"));
 const node_path_1 = __importDefault(require("node:path"));
 const workspace_1 = require("../src/asm/workspace");
 function runWorkspaceGraphTest() {
-    const entryPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-linkscript.asm");
-    const mainPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.asm");
+    const entryPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-linkscript.S");
+    const mainPath = node_path_1.default.resolve(process.cwd(), "test/fixtures/valid/merlin32-main-6502.S");
     const workspace = (0, workspace_1.indexWorkspace)(entryPath);
     strict_1.default.deepEqual(workspace.loadOrder, [entryPath, mainPath]);
     strict_1.default.deepEqual(workspace.dependencies.get(entryPath), [mainPath]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@razdolbai/merls",
-  "version": "1.2.2",
+  "version": "1.3.0",
   "description": "Language server for Merlin-style 6502 assembly.",
   "license": "MIT",
   "type": "commonjs",

package/src/lsp/completion.ts

@@ -18,25 +18,29 @@ export function buildCompletionItems(
   }
 
   const document = parseDocument(source);
-  const text = document.lines[line]?.node.text.trimStart().toLowerCase() ?? "";
+  const completions: CompletionItem[] = [];
 
-  if (text === "ld" || text === "ld\n" || text.startsWith("ld")) {
-    return opcodeDefinitions.map((opcode) => ({
+  for (const opcode of opcodeDefinitions) {
+    completions.push({
       label: opcode.mnemonic,
       kind: CompletionItemKind.Keyword
-    }));
+    });
   }
 
-  if (text === "du" || text.startsWith("du")) {
-    return directiveDefinitions.map((directive) => ({
+  for (const directive of directiveDefinitions) {
+    completions.push({
       label: directive.name,
       kind: CompletionItemKind.Function
-    }));
+    });
   }
 
   const symbols = collectSymbols(document);
-  return [...symbols.values()].map((symbol) => ({
-    label: symbol.name,
-    kind: CompletionItemKind.Variable
-  }));
+  for (const symbol of symbols.values()) {
+    completions.push({
+      label: symbol.name,
+      kind: CompletionItemKind.Variable
+    });
+  }
+
+  return completions;
 }