@tekyzinc/gsd-t 5.11.27 → 5.11.28

package/CHANGELOG.md

@@ -2,6 +2,41 @@
 
 All notable changes to GSD-T are documented here. Updated with each release.
 
+## [5.11.28] - 2026-08-11
+
+### Fixed — the graph stored the alias edges, then could not find them (two bugs)
+
+`who-imports src/lib/db.ts` returned 5 importers on hilo-figma-atos.
+`grep -rl '@/lib/db' src` returned 793. Found by David, who halted the scan
+rather than running it on a graph that was answering wrongly.
+
+v5.11.26 taught the INDEXER to expand `@/lib/db` into `src/lib/db`, and it does
+— the edges are in the database, correctly. It never met the QUERY side, which
+is what turns `src/lib/db` into the real file id `src/lib/db.ts`. Two
+independent bugs there, and fixing either alone still returns nothing:
+
+1. The resolver opened with `if (!dst.startsWith(".")) return dst` — an expanded
+   alias has no leading dot, so it was classed as an external package like
+   "react" and returned untouched. The extension was never appended.
+2. The set of known files was built from FUNCTION nodes, so a module exporting
+   only constants, types, or re-exports was not in it. Atos's `src/lib/db.ts`
+   exports a constant. The indexer already records every file it walked in the
+   `files` table — the authoritative list was stored all along.
+
+- `bin/gsd-t-graph-query-cli.cjs`: resolve non-relative targets against the file
+  set (packages still pass through untouched, and an expanded alias is NOT
+  re-joined to the importer's directory); read the file set from the `files`
+  table, announcing the degradation on an older graph that lacks it.
+- `test/m112-alias-query-resolution.test.js`: 7 regressions, all going through
+  the QUERY rather than the stored edge.
+
+Every v5.11.26 test asserted the edge was STORED correctly. Not one asked
+whether it could then be FOUND — the feature was tested at the write and called
+done. These tests close that gap.
+
+Requires `gsd-t graph index` only if the graph predates the `files` table;
+otherwise the fix applies to existing indexes immediately.
+
 ## [5.11.27] - 2026-08-11
 
 ### Fixed — the scan's volume probe returned a stand-in, and the whole run was built on it

package/README.md

@@ -1,6 +1,6 @@
 # GSD-T: Contract-Driven Development for Claude Code
 
-**v5.11.27** - A methodology for reliable, parallelizable development using Claude Code with optional Agent Teams support.
+**v5.11.28** - A methodology for reliable, parallelizable development using Claude Code with optional Agent Teams support.
 
 **Eliminates context rot** — task-level fresh dispatch (one subagent per task, ~10-20% context each) means compaction never triggers.
 **Compaction-proof debug loops** — `gsd-t headless --debug-loop` runs test-fix-retest cycles as separate `claude -p` sessions. A JSONL debug ledger persists all hypothesis/fix/learning history across fresh sessions. Anti-repetition preamble injection prevents retrying failed hypotheses. Escalation tiers (sonnet → opus → human) and a hard iteration ceiling enforced externally.

package/bin/gsd-t-graph-query-cli.cjs

@@ -437,10 +437,68 @@ function loadSqliteStore(dbPath) {
     // TS/JS routinely omit the extension; the graph keys on real file ids.
     // Every indexed file appears as the `file` of its nodes (the indexer does not
     // emit dedicated FILE nodes), so collect the distinct file set from there.
-    const fileIds = new Set(nodes.map((n) => norm(n.file)).filter(Boolean));
+    // [RULE] query-file-set-comes-from-files-table-not-inferred-from-functions
+    //
+    // hilo-figma-atos, 2026-08-11, the second half of the same miss. This set
+    // decides whether an extensionless import target can be matched to a real
+    // file. Built from node rows, it contains only files that declare a
+    // FUNCTION — so `src/lib/db.ts`, which exports a constant, was not in it,
+    // and no query could ever resolve an import of it. A file of constants,
+    // types, or re-exports is exactly the kind of shared module a whole codebase
+    // imports, and it was invisible.
+    //
+    // The indexer records every file it walked in the `files` table. That is the
+    // authoritative list; inferring one from functions was always an
+    // approximation of a fact already stored.
+    let fileIds;
+    const hasFilesTable = db
+      .prepare("SELECT name FROM sqlite_master WHERE type='table' AND name='files'")
+      .get();
+    if (hasFilesTable) {
+      fileIds = new Set(
+        db.prepare("SELECT file FROM files").all().map((r) => norm(r.file)).filter(Boolean)
+      );
+    } else {
+      // A graph built before the files table existed. Node-derived is what this
+      // has always done — announced, because it under-reports imports of any
+      // file that declares no function.
+      fileIds = new Set(nodes.map((n) => norm(n.file)).filter(Boolean));
+      process.stderr.write(
+        "[graph] this index predates the files table — imports of constant-only/type-only files may be missed; re-run `gsd-t graph index`\n"
+      );
+    }
     const EXTS = ["", ".ts", ".tsx", ".js", ".jsx", ".mjs", ".cjs", ".py", "/index.ts", "/index.tsx", "/index.js"];
+    // [RULE] query-resolves-expanded-alias-not-only-relative
+    //
+    // hilo-figma-atos, 2026-08-11: `who-imports src/lib/db.ts` returned 5 of 793
+    // real importers. The 788 missing ones all wrote `@/lib/db`.
+    //
+    // The two halves of the alias fix did not meet. The indexer (v5.11.26)
+    // expands `@/lib/db` to `src/lib/db` — correct, and stored. But this
+    // resolver, which is what appends the file extension, skipped it: the gate
+    // below returned early for anything not starting with ".", treating an
+    // expanded project path exactly like the package import `react`. So the edge
+    // sat in the database as `src/lib/db` while the query asked for
+    // `src/lib/db.ts`, and the two never met.
+    //
+    // A relative specifier is resolved against its source file's directory; an
+    // expanded alias is ALREADY project-relative and must not be — joining it to
+    // the importer's directory would produce `src/app/src/lib/db`. So the two
+    // take different routes to the same place: add the extension either way.
     const resolveDst = (srcFile, dst) => {
-      if (typeof dst !== "string" || !dst.startsWith(".")) return dst; // package/external
+      if (typeof dst !== "string" || !dst) return dst;
+
+      if (!dst.startsWith(".")) {
+        // Not relative. Either a package ("react", "next/navigation") or an
+        // alias the indexer already expanded to a project path. The file set is
+        // what tells them apart — a package matches nothing in it.
+        const base = norm(dst);
+        for (const ext of EXTS) {
+          if (fileIds.has(base + ext)) return base + ext;
+        }
+        return dst; // genuinely external — keep it exactly as written
+      }
+
       const base = path.posix.normalize(path.posix.join(path.posix.dirname(norm(srcFile)), norm(dst)));
       // Prefer an actual indexed file id (try the bare path, then common extensions).
       for (const ext of EXTS) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tekyzinc/gsd-t",
-  "version": "5.11.27",
+  "version": "5.11.28",
   "description": "GSD-T: Contract-Driven Development for Claude Code — 54 slash commands with headless-by-default workflow spawning, unattended supervisor relay with event stream, graph-powered code analysis, real-time agent dashboard, task telemetry, doc-ripple enforcement, backlog management, impact analysis, test sync, milestone archival, and PRD generation",
   "author": "Tekyz, Inc.",
   "license": "MIT",