@jaimevalasek/aioson 1.16.0 → 1.17.2

package/src/neural-chain-telemetry.js

@@ -0,0 +1,90 @@
+'use strict';
+
+/**
+ * Neural Chain — shared chain_audit telemetry emit helper.
+ *
+ * Single emitter used by both the CLI command (`src/commands/chain-audit.js`)
+ * and the post-session hook (`src/neural-chain-agent-ingest.js`) so payload
+ * shape is identical across code paths (BR-NC-10).
+ *
+ * Spec payload schema (BR-NC-10), 8 required fields:
+ *   feature_slug         — string | null
+ *   source_files         — string[] (the files edited in this session, plural)
+ *   impacts_found        — number | null (null on query failure)
+ *   auto_fixable_count   — number (per BR-NC-02/03 classification)
+ *   noise_file           — string | null (path written, if any)
+ *   tokens_used          — number (V1 placeholder = 0; M2 hooks LLM-mediated path)
+ *   duration_ms          — number (audit query elapsed; 0 on no-op)
+ *   error                — string | null
+ *
+ * Emitters may attach extra context fields (e.g. `agent`, `autonomy_mode`,
+ * `chain_auto_threshold`, `ingest_stats`, `skipped_reason`) on top of the
+ * required schema — those are passed through verbatim.
+ *
+ * EC-NC-05 no-op event (empty artifact list) also populates `duration_ms = 0`
+ * and `error = null` so downstream aggregation never has to special-case the
+ * skip path. Hotfix v1.17.1 — bug-found-003 from @tester gap-fill audit.
+ */
+
+const REQUIRED_FIELDS = Object.freeze([
+  'feature_slug',
+  'source_files',
+  'impacts_found',
+  'auto_fixable_count',
+  'noise_file',
+  'tokens_used',
+  'duration_ms',
+  'error'
+]);
+
+function normalizeSourceFiles(value) {
+  if (Array.isArray(value)) return value.slice();
+  if (value === null || value === undefined) return [];
+  return [String(value)];
+}
+
+function buildChainAuditPayload({
+  feature_slug = null,
+  source_files = [],
+  impacts_found = 0,
+  auto_fixable_count = 0,
+  noise_file = null,
+  tokens_used = 0,
+  duration_ms = 0,
+  error = null,
+  ...extras
+} = {}) {
+  return {
+    feature_slug,
+    source_files: normalizeSourceFiles(source_files),
+    impacts_found,
+    auto_fixable_count,
+    noise_file,
+    tokens_used,
+    duration_ms,
+    error,
+    ...extras
+  };
+}
+
+function emitChainAuditEvent(db, { agent = null, message = 'chain:audit', ...payloadOverrides } = {}) {
+  if (!db || typeof db.prepare !== 'function') return false;
+  const payload = buildChainAuditPayload(payloadOverrides);
+  try {
+    db.prepare(`
+      INSERT INTO execution_events (event_type, agent_name, message, payload_json, created_at)
+      VALUES ('chain_audit', ?, ?, ?, ?)
+    `).run(agent, message, JSON.stringify(payload), new Date().toISOString());
+    return true;
+  } catch (_) {
+    // BR-NC-10 best-effort: telemetry failure must never propagate to the caller.
+    return false;
+  }
+}
+
+module.exports = {
+  buildChainAuditPayload,
+  emitChainAuditEvent,
+  normalizeSourceFiles,
+  REQUIRED_FIELDS
+};

package/src/runtime-store.js

@@ -10,6 +10,7 @@ const {
   mergeGenomeBindings
 } = require('./genomes/bindings');
 const { runMigration: runLearningLoopMigration } = require('./learning-loop-migration');
+const { runMigration: runNeuralChainMigration } = require('./neural-chain-migration');
 
 const RUNTIME_DIR = path.join('.aioson', 'runtime');
 const DB_FILE = 'aios.sqlite';
@@ -775,6 +776,7 @@ function ensureLegacyColumns(db) {
   `);
 
   runLearningLoopMigration(db);
+  runNeuralChainMigration(db);
 }
 
 function insertEvent(db, record) {

package/template/.aioson/agents/neo.md

@@ -84,6 +84,26 @@ Check these in order. Stop at the first failure:
 | Readiness | `.aioson/context/readiness.md` | If exists, read status |
 | Implementation plan | `.aioson/context/implementation-plan.md` | Note presence and status |
 | Skeleton system | `.aioson/context/skeleton-system.md` | Note presence |
+| Neural Chain noises | `.aioson/context/noises/*.md` | If any file has unchecked `- [ ]` body lines, flag `chain_noises_pending` with file path + pending count. Treated as BLOCKER in Step 1.5. |
+
+### Step 1.5 — Neural Chain noise check (BLOCKER, takes precedence over routing)
+
+Glob `.aioson/context/noises/*.md`. For each file, count body lines matching `^- \[ \]` (unchecked) versus `^- \[x\]` (checked). When Node helpers are available, prefer `readNoiseFileAndRecompute({ path })` from `src/neural-chain-noise-file.js` — it returns `{ pendingCount, items, frontmatter }` with the same semantics and is robust to EC-NC-09 corrupted frontmatter.
+
+**If any noise file has `pendingCount > 0`:**
+- This is a BLOCKER, not info — routing to any other agent (`/dev`, `/deyvin`, `/qa`, etc.) is paused.
+- Surface in the dashboard under the ⛔ section, one block per file:
+  - Path (relative to project root)
+  - `{pendingCount}/{totalCount}` resolved
+  - Each pending item: `target_path — {motivo}` (the `motivo` already includes `edge_type` and `confidence` from BR-NC-06)
+- Recommended next action becomes: "Resolve the noise items above (mark `- [x]` once verified or fixed), OR explicitly say *skip noises* and re-activate `/neo` to confirm intent. Routing stays paused until one of those happens."
+- Set `confidence: low` and `clarification` in the routing block; do NOT recommend a downstream agent until the user resolves or explicitly skips.
+
+**If `pendingCount === 0` across every noise file:** noise files are stale — the next `runChainHookOnAgentDone` invocation (or `chain:audit` call) will `maybeDeleteNoiseFile` them automatically (EC-NC-10 idempotent). Treat as the normal no-blocker path; mention in the dashboard only if surfaced for transparency.
+
+**If the user explicitly skips:** include `reason: skipped <N> noise file(s)` in the routing block and proceed with normal routing. The noise files persist until resolved.
+
+Background: noise files are produced by the Neural Chain audit hook (`runChainHookOnAgentDone` in `src/neural-chain-agent-ingest.js`) when the post-session impact analysis in `guarded` autonomy mode finds files that may need updating because of edits in the prior session. See `.aioson/context/spec-neural-chain.md` § BR-NC-06 for the format and lifecycle.
 
 ### Step 2 — Git state snapshot
 
@@ -99,6 +119,7 @@ Based on Step 1 results, classify the project into one of these stages:
 
 | Stage | Condition | Primary agent |
 |---|---|---|
+| **Chain audit pending** | `chain_noises_pending` flagged in Step 1.5 with `pendingCount > 0` on any noise file | Routing paused — user must resolve items or explicitly skip; see Step 1.5 |
 | **Not initialized** | config.md missing | Manual: user needs to run `aioson init` |
 | **Needs setup** | `needs_setup` or `needs_setup_repair` | `/setup` |
 | **Needs product definition** | Context valid, no PRD | `/product` |
@@ -128,6 +149,7 @@ Memory: bootstrap {N}/4 | brains {count} indexed | last distillation {when or "
 {if features in progress: "Active feature: {slug} — stage: {feature_stage} | dossier: {yes/no} | harness: {progress.status or "—"}"}
 {if blockers in readiness.md: "⚠ Blockers: {summary}"}
 {if harness pending gate or circuit_open: "⛔ Harness: {circuit reason or pending gate id}"}
+{if chain_noises_pending: "⛔ Chain: {N} noise file(s) with pending items — resolve before routing (see list below)"}
 
 → Recommended next: /agent — {one-line reason}
 {if alternative paths exist: "Also possible: /agent2 — {reason}"}