trackfw 7.2.0 → 7.4.0

package/src/serve/api_chain.js

@@ -2,6 +2,8 @@
 
 const fs = require('fs')
 const path = require('path')
+const { resolveAgentNamespaces } = require('../validator')
+const { normalizeRefSeparator } = require('../lib/pathfmt')
 
 const ROADMAP_STATES = ['wip', 'backlog', 'blocked', 'done', 'abandoned']
 
@@ -70,7 +72,15 @@ function collectNodes(dir, type, state) {
     try { content = fs.readFileSync(path.join(dir, file), 'utf8') } catch (_) {}
     const fm = parseFrontmatter(content)
     const title = extractTitle(content, file)
-    const id = path.join(dir, file)
+    // node ID do grafo — IDENTIFICADOR emitido em JSON (ADR-2026-09-04, D1 categoria 2),
+    // e tambem o alvo de toda aresta: resolveRef() abaixo so devolve valores vindos de
+    // fileIndex/titleIndex, isto e, sempre um node id — nunca o valor cru do frontmatter.
+    // Normalizar o id aqui, portanto, normaliza `edge.to` por construcao.
+    //
+    // 🔴 O path.join da linha 71 (fs.readFileSync) e uma EXPRESSAO INDEPENDENTE sobre os
+    // mesmos operandos: normalizar esta nao pode afeta-la. O caminho de travessia
+    // permanece nativo (ADR D2). Espelha internal/serve/api_chain.go:111.
+    const id = normalizeRefSeparator(path.join(dir, file))
     nodes.push({ id, type, title, state, fm })
   }
   return nodes
@@ -102,14 +112,7 @@ function handleChain(cfg, req, res) {
 
   // REQs
   if (namespacing === 'by_agent') {
-    let agents = cfg.agents || []
-    if (!agents.length) {
-      try {
-        agents = fs.readdirSync(reqDir).filter(f => {
-          try { return fs.statSync(path.join(reqDir, f)).isDirectory() } catch (_) { return false }
-        })
-      } catch (_) {}
-    }
+    const agents = resolveAgentNamespaces(cfg, reqDir)
     for (const agent of agents) {
       for (const state of ROADMAP_STATES) {
         const dir = path.join(reqDir, agent, state)
@@ -122,14 +125,7 @@ function handleChain(cfg, req, res) {
 
   // Roadmaps
   if (namespacing === 'by_agent') {
-    let agents = cfg.agents || []
-    if (!agents.length) {
-      try {
-        agents = fs.readdirSync(roadmapDir).filter(f => {
-          try { return fs.statSync(path.join(roadmapDir, f)).isDirectory() } catch (_) { return false }
-        })
-      } catch (_) {}
-    }
+    const agents = resolveAgentNamespaces(cfg, roadmapDir)
     for (const agent of agents) {
       for (const state of ROADMAP_STATES) {
         const dir = path.join(roadmapDir, agent, state)

package/src/serve/api_metrics.js

@@ -3,6 +3,7 @@
 const fs = require('fs')
 const path = require('path')
 const { parseLog, calculate } = require('../commands/metrics')
+const { resolveAgentNamespaces } = require('../validator')
 
 const STATES = ['wip', 'backlog', 'blocked', 'done', 'abandoned']
 const MS_PER_DAY = 24 * 60 * 60 * 1000
@@ -34,14 +35,7 @@ function buildStateDistribution(cfg) {
   const namespacing = cfg.roadmapNamespacing || 'flat'
 
   if (namespacing === 'by_agent') {
-    let agents = cfg.agents || []
-    if (!agents.length) {
-      try {
-        agents = fs.readdirSync(roadmapDir).filter(f => {
-          try { return fs.statSync(path.join(roadmapDir, f)).isDirectory() } catch (_) { return false }
-        })
-      } catch (_) { agents = [] }
-    }
+    const agents = resolveAgentNamespaces(cfg, roadmapDir)
     for (const agent of agents) {
       for (const state of STATES) {
         dist[state] = (dist[state] || 0) + countFilesInState(path.join(roadmapDir, agent, state))