kodingo-cli 1.3.0 → 1.3.2

package/dist/__tests__/csharpParser.test.js

@@ -0,0 +1,187 @@
+"use strict";
+/**
+ * Real tests for C#/.NET support in tree-sitter-parser.ts / ast-parser.ts —
+ * build brief "Add real C#/.NET support to the CLI's parser" (2026-09-20).
+ *
+ * The real bar per the brief: not just "C# symbols get extracted", but
+ * genuine, stable deduplication — the same, unchanged function produces a
+ * byte-for-byte identical fingerprint across repeated runs, and a genuinely
+ * changed function's fingerprint correctly changes.
+ *
+ * Uses Node's built-in test runner (node:test) — this repo has no existing
+ * test framework to inherit (confirmed: package.json's own "test" script
+ * was a stub), and node:test needs zero new dependencies. Compiled and run
+ * via `npm test` (tsc then `node --test dist/__tests__`).
+ */
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const node_test_1 = require("node:test");
+const strict_1 = __importDefault(require("node:assert/strict"));
+const tree_sitter_parser_1 = require("../utils/tree-sitter-parser");
+const ast_parser_1 = require("../utils/ast-parser");
+const REAL_CSHARP_SOURCE = `
+using System;
+using System.Threading.Tasks;
+
+namespace Acme.Orders
+{
+    public class OrderService : IOrderService
+    {
+        private readonly IRepository _repo;
+
+        public OrderService(IRepository repo)
+        {
+            _repo = repo;
+        }
+
+        public async Task<Order> GetOrderAsync(int id)
+        {
+            var order = await _repo.FindAsync(id);
+            LogAccess(id);
+            return order;
+        }
+
+        private void LogAccess(int id)
+        {
+            Console.WriteLine($"Accessed order {id}");
+        }
+
+        public static int Add(int a, int b)
+        {
+            if (a > 0)
+            {
+                return a + b;
+            }
+            return b;
+        }
+    }
+
+    public interface IOrderService
+    {
+        Task<Order> GetOrderAsync(int id);
+    }
+
+    public struct Point
+    {
+        public int Sum(int x, int y) => x + y;
+    }
+
+    public record OrderDto(int Id, string Name)
+    {
+        public string Describe() => $"{Id}: {Name}";
+    }
+
+    public class Order { }
+}
+`;
+// A real, structural change to GetOrderAsync — a distinct code path added,
+// not a cosmetic rename/whitespace/comment tweak (those are what
+// computeFingerprint is specifically designed to SURVIVE).
+const CHANGED_CSHARP_SOURCE = REAL_CSHARP_SOURCE.replace(`public async Task<Order> GetOrderAsync(int id)
+        {
+            var order = await _repo.FindAsync(id);
+            LogAccess(id);
+            return order;
+        }`, `public async Task<Order> GetOrderAsync(int id)
+        {
+            var order = await _repo.FindAsync(id);
+            LogAccess(id);
+            if (order == null) throw new InvalidOperationException("not found");
+            return order;
+        }`);
+(0, node_test_1.describe)("parseWithTreeSitter — real C# extraction", () => {
+    (0, node_test_1.it)("extracts every real method, constructor, and local function with class-qualified names", async () => {
+        const symbols = await (0, tree_sitter_parser_1.parseWithTreeSitter)("OrderService.cs", REAL_CSHARP_SOURCE);
+        const names = symbols.map(s => s.name).sort();
+        strict_1.default.deepEqual(names, [
+            "IOrderService.GetOrderAsync",
+            "OrderDto.Describe",
+            "OrderService.Add",
+            "OrderService.GetOrderAsync",
+            "OrderService.LogAccess",
+            "OrderService.OrderService",
+            "Point.Sum",
+        ].sort());
+    });
+    (0, node_test_1.it)("real, correct method vs constructor typing", async () => {
+        const symbols = await (0, tree_sitter_parser_1.parseWithTreeSitter)("OrderService.cs", REAL_CSHARP_SOURCE);
+        const ctor = symbols.find(s => s.name === "OrderService.OrderService");
+        const method = symbols.find(s => s.name === "OrderService.GetOrderAsync");
+        strict_1.default.equal(ctor?.type, "method");
+        strict_1.default.equal(method?.type, "method");
+    });
+    (0, node_test_1.it)("real call extraction trims member-access callees to their trailing identifier", async () => {
+        const symbols = await (0, tree_sitter_parser_1.parseWithTreeSitter)("OrderService.cs", REAL_CSHARP_SOURCE);
+        const method = symbols.find(s => s.name === "OrderService.GetOrderAsync");
+        strict_1.default.ok(method);
+        strict_1.default.ok(method.calls.includes("FindAsync"), `expected FindAsync in ${JSON.stringify(method.calls)}`);
+        strict_1.default.ok(method.calls.includes("LogAccess"), `expected LogAccess in ${JSON.stringify(method.calls)}`);
+    });
+    (0, node_test_1.it)("real using-directive extraction", async () => {
+        const symbols = await (0, tree_sitter_parser_1.parseWithTreeSitter)("OrderService.cs", REAL_CSHARP_SOURCE);
+        strict_1.default.ok(symbols[0].imports.some(i => i.source.includes("using System;")));
+        strict_1.default.ok(symbols[0].imports.some(i => i.source.includes("using System.Threading.Tasks;")));
+    });
+    (0, node_test_1.it)("returns [] for an unrecognized extension rather than throwing", async () => {
+        const symbols = await (0, tree_sitter_parser_1.parseWithTreeSitter)("readme.md", "# not code");
+        strict_1.default.deepEqual(symbols, []);
+    });
+});
+(0, node_test_1.describe)("real deduplication — the actual bar this brief is judged against", () => {
+    (0, node_test_1.it)("the SAME, unchanged C# function produces a byte-for-byte identical fingerprint across repeated runs", async () => {
+        const run1 = await (0, tree_sitter_parser_1.parseWithTreeSitter)("OrderService.cs", REAL_CSHARP_SOURCE);
+        const run2 = await (0, tree_sitter_parser_1.parseWithTreeSitter)("OrderService.cs", REAL_CSHARP_SOURCE);
+        const run3 = await (0, tree_sitter_parser_1.parseWithTreeSitter)("OrderService.cs", REAL_CSHARP_SOURCE);
+        const fp1 = run1.find(s => s.name === "OrderService.GetOrderAsync")?.fingerprint;
+        const fp2 = run2.find(s => s.name === "OrderService.GetOrderAsync")?.fingerprint;
+        const fp3 = run3.find(s => s.name === "OrderService.GetOrderAsync")?.fingerprint;
+        strict_1.default.ok(fp1);
+        strict_1.default.equal(fp1, fp2);
+        strict_1.default.equal(fp2, fp3);
+        // Every single real symbol in the file, not just one — genuine, stable
+        // dedup across the whole file, not a cherry-picked case.
+        for (let i = 0; i < run1.length; i++) {
+            strict_1.default.equal(run1[i].fingerprint, run2[i].fingerprint, `symbol ${run1[i].name} fingerprint drifted across runs`);
+        }
+    });
+    (0, node_test_1.it)("via the real, full parseFile dispatch (the actual path scan-repo.ts calls) — same stability", async () => {
+        const run1 = await (0, ast_parser_1.parseFile)("OrderService.cs", REAL_CSHARP_SOURCE);
+        const run2 = await (0, ast_parser_1.parseFile)("OrderService.cs", REAL_CSHARP_SOURCE);
+        strict_1.default.ok(run1.length > 0);
+        strict_1.default.deepEqual(run1.map(s => s.fingerprint), run2.map(s => s.fingerprint));
+    });
+    (0, node_test_1.it)("cosmetic-only changes (comments, whitespace) do NOT change the fingerprint — by design", async () => {
+        const cosmetic = REAL_CSHARP_SOURCE.replace(`public static int Add(int a, int b)`, `// adds two numbers\n        public static int Add(int a, int b)`);
+        const before = await (0, tree_sitter_parser_1.parseWithTreeSitter)("OrderService.cs", REAL_CSHARP_SOURCE);
+        const after = await (0, tree_sitter_parser_1.parseWithTreeSitter)("OrderService.cs", cosmetic);
+        const fpBefore = before.find(s => s.name === "OrderService.Add")?.fingerprint;
+        const fpAfter = after.find(s => s.name === "OrderService.Add")?.fingerprint;
+        strict_1.default.equal(fpBefore, fpAfter);
+    });
+    (0, node_test_1.it)("a genuinely CHANGED function's fingerprint correctly changes — an edit is not invisible", async () => {
+        const before = await (0, tree_sitter_parser_1.parseWithTreeSitter)("OrderService.cs", REAL_CSHARP_SOURCE);
+        const after = await (0, tree_sitter_parser_1.parseWithTreeSitter)("OrderService.cs", CHANGED_CSHARP_SOURCE);
+        const fpBefore = before.find(s => s.name === "OrderService.GetOrderAsync")?.fingerprint;
+        const fpAfter = after.find(s => s.name === "OrderService.GetOrderAsync")?.fingerprint;
+        strict_1.default.ok(fpBefore);
+        strict_1.default.ok(fpAfter);
+        strict_1.default.notEqual(fpBefore, fpAfter);
+        // Every OTHER real symbol in the file is untouched by the edit and must
+        // keep its own stable fingerprint — proving the change is properly
+        // scoped to the actual edited function, not a global reshuffle.
+        for (const name of ["OrderService.Add", "OrderService.LogAccess", "OrderService.OrderService", "Point.Sum"]) {
+            const b = before.find(s => s.name === name)?.fingerprint;
+            const a = after.find(s => s.name === name)?.fingerprint;
+            strict_1.default.equal(b, a, `unrelated symbol ${name} should not have changed`);
+        }
+    });
+});
+(0, node_test_1.describe)("computeFingerprint — sanity on real C# body text", () => {
+    (0, node_test_1.it)("normalizes C# string interpolation/comments deterministically", () => {
+        const a = (0, ast_parser_1.computeFingerprint)(`Console.WriteLine($"Accessed order {id}"); // log it`);
+        const b = (0, ast_parser_1.computeFingerprint)(`Console.WriteLine($"Accessed order {id}");`);
+        strict_1.default.equal(a, b);
+    });
+});

package/dist/adapters/cloud-adapter/cloud-persistence.js

@@ -119,6 +119,8 @@ function mapApiRecord(r) {
         record.fingerprint = r.fingerprint;
     if (r.deniedAt)
         record.deniedAt = new Date(r.deniedAt);
+    if (r.contentAsOf)
+        record.contentAsOf = new Date(r.contentAsOf);
     return record;
 }
 // ── Public API (mirrors memory-persistence.ts) ────────────────────────────────
@@ -161,6 +163,8 @@ async function saveMemory(input) {
         body.correctsId = input.correctsId;
     if (input.correctedById)
         body.correctedById = input.correctedById;
+    if (input.contentAsOf)
+        body.contentAsOf = input.contentAsOf.toISOString();
     // Attribution
     try {
         const config = (0, persistence_config_1.readConfig)();

package/dist/adapters/db-adapter/memory-persistence.js

@@ -61,6 +61,11 @@ async function initDb() {
       END IF;
     END $$;
 
+    -- Real, best-known age of the underlying decision/code, separate from
+    -- created_at (mirrors kodingo-api's schema.ts). NULL when genuinely
+    -- unknown — never defaulted to created_at.
+    ALTER TABLE kodingo_memory ADD COLUMN IF NOT EXISTS content_as_of timestamptz NULL;
+
     -- Indexes
     CREATE INDEX IF NOT EXISTS kodingo_memory_created_at_idx
       ON kodingo_memory (created_at);
@@ -128,7 +133,7 @@ async function saveMemory(input) {
     WITH inserted AS (
       INSERT INTO kodingo_memory (
         id, type, title, content, tags, repo_path,
-        status, confidence, symbol, external_id, evidence_kind,
+        status, confidence, symbol, external_id, evidence_kind, content_as_of,
         corrects_id, corrected_by_id
       )
       VALUES (
@@ -143,12 +148,13 @@ async function saveMemory(input) {
         ${sqlNullableLiteral(input.symbol)},
         ${sqlNullableLiteral(externalId)},
         ${sqlLiteral(input.evidenceKind)},
+        ${sqlNullableTimestampLiteral(input.contentAsOf)},
         ${sqlNullableUuidLiteral(input.correctsId)},
         ${sqlNullableUuidLiteral(input.correctedById)}
       )
       RETURNING
         id, type, title, content, tags, repo_path,
-        status, confidence, symbol, external_id, evidence_kind,
+        status, confidence, symbol, external_id, evidence_kind, content_as_of,
         corrects_id, corrected_by_id,
         created_at
     )
@@ -165,7 +171,7 @@ async function getMemoryById(id) {
     SELECT row_to_json(result) FROM (
       SELECT
         id, type, title, content, tags, repo_path,
-        status, confidence, symbol, external_id, evidence_kind,
+        status, confidence, symbol, external_id, evidence_kind, content_as_of,
         corrects_id, corrected_by_id,
         created_at
       FROM kodingo_memory
@@ -193,7 +199,7 @@ async function updateMemoryLifecycle(params) {
       WHERE id = ${sqlLiteral(params.id)}
       RETURNING
         id, type, title, content, tags, repo_path,
-        status, confidence, symbol, external_id, evidence_kind,
+        status, confidence, symbol, external_id, evidence_kind, content_as_of,
         corrects_id, corrected_by_id,
         created_at
     )
@@ -246,7 +252,7 @@ async function getMemoryByExternalId(externalId, repoPath) {
     SELECT row_to_json(result) FROM (
       SELECT
         id, type, title, content, tags, repo_path,
-        status, confidence, symbol, external_id, evidence_kind,
+        status, confidence, symbol, external_id, evidence_kind, content_as_of,
         corrects_id, corrected_by_id,
         created_at
       FROM kodingo_memory
@@ -271,7 +277,7 @@ async function getProposedOrIgnoredRecordBySymbol(params) {
     SELECT row_to_json(result) FROM (
       SELECT
         id, type, title, content, tags, repo_path,
-        status, confidence, symbol, external_id, evidence_kind,
+        status, confidence, symbol, external_id, evidence_kind, content_as_of,
         corrects_id, corrected_by_id,
         created_at
       FROM kodingo_memory
@@ -303,7 +309,7 @@ async function updateMemoryConfidence(params) {
       WHERE id = ${sqlLiteral(params.id)}
       RETURNING
         id, type, title, content, tags, repo_path,
-        status, confidence, symbol, external_id, evidence_kind,
+        status, confidence, symbol, external_id, evidence_kind, content_as_of,
         corrects_id, corrected_by_id,
         created_at
     )
@@ -346,7 +352,7 @@ async function queryMemory(text, limit = 10, repoPath, opts = {}) {
       SELECT row_to_json(result) FROM (
         SELECT DISTINCT ON (coalesce(symbol, id::text))
           id, type, title, content, tags, repo_path,
-          status, confidence, symbol, external_id, evidence_kind,
+          status, confidence, symbol, external_id, evidence_kind, content_as_of,
           corrects_id, corrected_by_id,
           created_at
         FROM kodingo_memory
@@ -377,7 +383,7 @@ async function queryMemory(text, limit = 10, repoPath, opts = {}) {
     SELECT row_to_json(result) FROM (
       SELECT
         id, type, title, content, tags, repo_path,
-        status, confidence, symbol, external_id, evidence_kind,
+        status, confidence, symbol, external_id, evidence_kind, content_as_of,
         corrects_id, corrected_by_id,
         created_at,
         CASE
@@ -441,7 +447,7 @@ async function queryMemoryBySymbol(params) {
     SELECT row_to_json(result) FROM (
       SELECT
         id, type, title, content, tags, repo_path,
-        status, confidence, symbol, external_id, evidence_kind,
+        status, confidence, symbol, external_id, evidence_kind, content_as_of,
         corrects_id, corrected_by_id,
         created_at
       FROM kodingo_memory
@@ -473,6 +479,7 @@ function mapRow(row) {
         symbol: row.symbol ?? null,
         externalId: row.external_id ?? null,
         evidenceKind: row.evidence_kind ?? "sourced",
+        contentAsOf: row.content_as_of ? new Date(row.content_as_of) : null,
         correctsId: row.corrects_id ?? null,
         correctedById: row.corrected_by_id ?? null,
         createdAt: new Date(row.created_at),
@@ -538,6 +545,11 @@ function sqlNullableUuidLiteral(value) {
         return "NULL";
     return `${sqlLiteral(value)}::uuid`;
 }
+function sqlNullableTimestampLiteral(value) {
+    if (!value)
+        return "NULL";
+    return `${sqlLiteral(value.toISOString())}::timestamptz`;
+}
 function sqlArrayLiteral(values) {
     if (!values || values.length === 0)
         return "NULL";

package/dist/commands/scan-repo.js

@@ -61,7 +61,7 @@ const API_BASE = "https://kodingo-api.onrender.com";
 const SUPPORTED_EXTENSIONS = [
     ".ts", ".tsx", ".js", ".jsx",
     ".py", ".go", ".rs", ".php",
-    ".java", ".kt", ".rb", ".c", ".cpp",
+    ".java", ".kt", ".rb", ".c", ".cpp", ".cs",
 ];
 const IGNORED_DIRS = [
     "node_modules", ".git", ".kortex", "dist", "build",
@@ -112,6 +112,8 @@ async function extractSymbols(filePath) {
             code: n.body.slice(0, 2000), // cap to avoid huge payloads
             file: filePath,
             fingerprint: n.fingerprint,
+            startLine: n.startLine,
+            endLine: n.endLine,
         }));
     }
     catch {
@@ -201,6 +203,14 @@ async function inferAndCapture(token, symbol, repo, graph, repoRoot) {
                 // — Kortex's own conclusion, not traceable to one real external
                 // event, same class as kodingo-api's githubScanner.ts.
                 evidenceKind: "derived",
+                // Real line range + repo-relative file path (Problem B) — lets
+                // kodingo-api resolve this symbol's real content age via git blame
+                // asynchronously after capture (it has the GitHub App token this
+                // CLI doesn't). Without these, content_as_of stays null — never
+                // guessed from scan timing.
+                startLine: symbol.startLine ?? undefined,
+                endLine: symbol.endLine ?? undefined,
+                filePath: path_1.default.relative(repoRoot, symbol.file).split(path_1.default.sep).join("/"),
             }),
         });
         return saveRes.ok;

package/dist/ports/cloud-event-port.js

@@ -104,6 +104,7 @@ class CloudEventPort {
             evidenceKind: "sourced",
             externalId,
             tags: ["signal:git", "kind:decision"],
+            contentAsOf: decision.proposedAt,
         };
         if (title)
             input.title = title;

package/dist/ports/db-event-port.js

@@ -77,7 +77,10 @@ class DbEventPort {
         }
         // No existing proposed/ignored record — insert new record as usual.
         // Git-signal-derived captures are genuinely sourced — externalId here is
-        // a real git:<commitHash> reference (see the caller).
+        // a real git:<commitHash> reference (see the caller). contentAsOf is the
+        // real commit's own author date (decision.proposedAt) — trivially in
+        // hand here, unlike bulk/autonomous scans which have no equivalent
+        // single real event to anchor to (see contentAge.ts, kodingo-api).
         const input = {
             type: "decision",
             content,
@@ -86,6 +89,7 @@ class DbEventPort {
             confidence: 0.3,
             evidenceKind: "sourced",
             externalId,
+            contentAsOf: decision.proposedAt,
         };
         if (title)
             input.title = title;

package/dist/utils/ast-parser.js

@@ -417,7 +417,7 @@ const KOTLIN_PATTERNS = [
     /^(?:fun|override\s+fun|suspend\s+fun|private\s+fun|public\s+fun)\s+(\w+)/,
     /^(?:class|data class|object|interface)\s+(\w+)/,
 ];
-const TREE_SITTER_EXTENSIONS = new Set([".py", ".go", ".rs", ".java", ".kt", ".rb", ".php", ".c", ".cpp"]);
+const TREE_SITTER_EXTENSIONS = new Set([".py", ".go", ".rs", ".java", ".kt", ".rb", ".php", ".c", ".cpp", ".cs"]);
 async function parseFile(filePath, source) {
     const ext = path.extname(filePath).toLowerCase();
     switch (ext) {

package/dist/utils/graph-builder.js

@@ -64,7 +64,7 @@ const ast_parser_1 = require("./ast-parser");
 const SUPPORTED_EXTENSIONS = [
     ".ts", ".tsx", ".js", ".jsx",
     ".py", ".go", ".rs", ".php",
-    ".java", ".kt", ".rb", ".c", ".cpp",
+    ".java", ".kt", ".rb", ".c", ".cpp", ".cs",
 ];
 const TS_JS_EXTENSIONS = new Set([".ts", ".tsx", ".js", ".jsx"]);
 const IGNORED_DIRS = [

package/dist/utils/tree-sitter-parser.js

@@ -104,6 +104,31 @@ const LANGUAGE_CONFIGS = {
         nameFieldCandidates: [], // handled via declarator fallback
         importTypes: ["preproc_include"],
     },
+    // C# — build brief "Add real C#/.NET support to the CLI's parser"
+    // (2026-09-20). Real node types confirmed directly against the bundled
+    // tree-sitter-c_sharp.wasm grammar (tree-sitter-wasms package, already a
+    // real dependency here — no new native package needed, see this brief's
+    // own final report for why that matters). method_declaration,
+    // constructor_declaration, and local_function_statement all carry a real
+    // `name` field; invocation_expression carries a real `function` field
+    // whose text (e.g. "_repo.FindAsync", "Console.WriteLine") is trimmed to
+    // its trailing identifier by the same extractTrailingIdentifier() already
+    // proven for PHP's "$this->service->method" shape — no C#-specific call
+    // handling needed. Deliberately excludes operator_declaration,
+    // destructor_declaration, and property accessors — real, distinct node
+    // shapes with no `name` field (or none at all) that no other language
+    // here gets special-cased treatment for either; kept proportionate to the
+    // existing five languages' own level of coverage rather than gold-plating
+    // C# beyond them.
+    csharp: {
+        wasmFile: "tree-sitter-c_sharp.wasm",
+        functionTypes: ["method_declaration", "constructor_declaration", "local_function_statement"],
+        classTypes: ["class_declaration", "interface_declaration", "struct_declaration", "record_declaration"],
+        callTypes: ["invocation_expression"],
+        calleeFieldCandidates: ["function"],
+        nameFieldCandidates: ["name"],
+        importTypes: ["using_directive"],
+    },
 };
 const EXT_TO_LANGUAGE = {
     ".py": "python",
@@ -115,6 +140,7 @@ const EXT_TO_LANGUAGE = {
     ".php": "php",
     ".c": "c",
     ".cpp": "cpp",
+    ".cs": "csharp",
 };
 let initialised = false;
 const languageCache = new Map();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodingo-cli",
-  "version": "1.3.0",
+  "version": "1.3.2",
   "description": "Kodingo CLI",
   "license": "MIT",
   "private": false,
@@ -16,7 +16,7 @@
     "build": "tsc -p tsconfig.json",
     "dev": "tsc -p tsconfig.json --watch",
     "start": "node dist/cli.js",
-    "test": "echo \"Error: no test specified\" && exit 1",
+    "test": "npm run build && node --test dist/__tests__/*.js",
     "prepare": "npm run build"
   },
   "dependencies": {
@@ -26,7 +26,7 @@
     "@typescript-eslint/typescript-estree": "^8.61.1",
     "commander": "^12.1.0",
     "kodingo-agent": "^0.2.0",
-    "kodingo-core": "^0.3.0",
+    "kodingo-core": "^0.3.1",
     "open": "^11.0.0",
     "simple-git": "^3.30.0",
     "tree-sitter-wasms": "^0.1.13",