@aikdna/kdna-cli 0.17.0 → 0.18.0

package/src/cmds/studio.js

@@ -10,7 +10,7 @@
 
 const fs = require('fs');
 const path = require('path');
-const { error, readJson, writeJson, EXIT } = require('./_common');
+const { error, readJson, writeJson, EXIT, isYesNoSelfCheck } = require('./_common');
 
 // ─── Scaffold ─────────────────────────────────────────────────────────
 
@@ -240,8 +240,8 @@ function cmdCardsValidate(projectPath, args = []) {
           const label = sc.id || '?';
           if (!sc.question || sc.question.includes('[TODO]')) {
             warn(`self_check ${label}: question is placeholder`);
-          } else if (!sc.question.trim().endsWith('?')) {
-            fail(`self_check ${label}: question should end with "?"`);
+          } else if (!isYesNoSelfCheck(sc.question)) {
+            fail(`self_check ${label}: question should be answerable with yes/no`);
           } else {
             ok(`self_check ${label}: question OK`);
           }
@@ -398,7 +398,6 @@ function cmdStudioCompile(projectPath, args = []) {
   // Compile axioms → KDNA_Core.json
   const axioms = loadCards(project, path.dirname(abs), 'axioms');
   const ontology = loadCards(project, path.dirname(abs), 'ontology');
-  const _boundaries = loadCards(project, path.dirname(abs), 'boundaries');
 
   const core = {
     meta: {
@@ -495,7 +494,7 @@ function cmdStudioCompile(projectPath, args = []) {
 
   console.log('');
   console.log('Next:');
-  console.log(`  kdna validate ${outDir}`);
+  console.log(`  kdna dev validate ${outDir}`);
 }
 
 function loadCards(project, projectDir, cardType) {

package/src/cmds/test.js

@@ -12,9 +12,9 @@ const fs = require('fs');
 const path = require('path');
 const { error, readJson, writeJson, EXIT } = require('./_common');
 const { parseName } = require('../registry');
+const { getInstalled } = require('../package-store');
 
 const USER_KDNA_DIR = path.join(process.env.HOME || process.env.USERPROFILE || '.', '.kdna');
-const INSTALL_DIR = path.join(USER_KDNA_DIR, 'domains');
 const RUNS_DIR = path.join(USER_KDNA_DIR, 'runs');
 
 function cmdTestRun(args = []) {
@@ -38,8 +38,8 @@ function cmdTestRun(args = []) {
 
   const parsed = parseName(domain);
   if (!parsed) error(`Invalid name "${domain}".`, EXIT.INPUT_ERROR);
-  const destDir = path.join(INSTALL_DIR, parsed.scope, parsed.ident);
-  if (!fs.existsSync(destDir)) {
+  const installed = getInstalled(parsed.full);
+  if (!installed) {
     error(`${parsed.full} not installed. Run: kdna install ${domain}`, EXIT.INPUT_ERROR);
   }
 
@@ -64,7 +64,7 @@ function cmdTestRun(args = []) {
   const result = {
     test_id: testCase.id || `test_${Date.now()}`,
     domain: parsed.full,
-    domain_path: destDir,
+    domain_asset: installed.asset_path,
     input: typeof testCase.input === 'string' ? testCase.input : JSON.stringify(testCase.input),
     run_at: new Date().toISOString(),
     expected: {

package/src/cmds/trace.js

@@ -1,10 +1,9 @@
 const fs = require('fs');
 const path = require('path');
-const crypto = require('crypto');
-const { EXIT, error, readJson } = require('./_common');
+const { EXIT } = require('./_common');
+const PATHS = require('../paths');
 
-const USER_KDNA_DIR = path.join(process.env.HOME || process.env.USERPROFILE || '.', '.kdna');
-const TRACES_DIR = path.join(USER_KDNA_DIR, 'traces');
+const TRACES_DIR = PATHS.traces;
 
 function ensureTracesDir() {
   fs.mkdirSync(TRACES_DIR, { recursive: true });
@@ -60,9 +59,14 @@ function readAllTraces(opts = {}) {
 }
 
 function recordTrace(entry) {
-  ensureTracesDir();
-  const line = JSON.stringify(entry) + '\n';
-  fs.appendFileSync(todayFile(), line);
+  try {
+    ensureTracesDir();
+    const line = JSON.stringify(entry) + '\n';
+    fs.appendFileSync(todayFile(), line);
+  } catch {
+    // Traces are observability data. Loading and comparing KDNA assets must not
+    // fail just because the local trace directory is unavailable or read-only.
+  }
 }
 
 function parseSinceFlag(args) {

package/src/compare.js

@@ -1,5 +1,5 @@
 /**
- * kdna compare <name> --input "<text>" — Reasoning trajectory diff.
+ * kdna compare <name|file.kdna> --input "<text>" — Reasoning trajectory diff.
  *
  * Runs the same prompt twice on a real LLM:
  *   1. Without KDNA loaded (baseline)
@@ -24,7 +24,7 @@ const path = require('path');
 const https = require('https');
 
 const USER_KDNA_DIR = path.join(process.env.HOME || process.env.USERPROFILE || '.', '.kdna');
-const INSTALL_DIR = path.join(USER_KDNA_DIR, 'domains');
+const { readContainer, resolveAsset } = require('./package-store');
 const CONFIG_FILE = path.join(USER_KDNA_DIR, 'config.json');
 
 const { parseName } = require('./registry');
@@ -179,10 +179,10 @@ async function callLlm(cfg, systemPrompt, userMessage) {
 
 // ─── KDNA → system prompt ─────────────────────────────────────────────
 
-function buildKdnaPrompt(destDir) {
-  const core = readJson(path.join(destDir, 'KDNA_Core.json'));
-  const pat = readJson(path.join(destDir, 'KDNA_Patterns.json'));
-  const manifest = readJson(path.join(destDir, 'kdna.json'));
+function buildKdnaPrompt(container) {
+  const core = container.core;
+  const pat = container.patterns;
+  const manifest = container.manifest;
 
   if (!core || !pat) return '';
 
@@ -302,7 +302,6 @@ function emitMarkdownReport(parsed, manifest, core, pat, responseA, responseB, d
   const bannedTerms = (pat.terminology?.banned_terms || []).map((t) =>
     typeof t === 'string' ? t : t.term,
   );
-  const misunderstandings = pat.misunderstandings || [];
 
   const lines = [];
   lines.push('# KDNA Judgment Comparison Report');
@@ -487,27 +486,26 @@ async function cmdCompare(input, args = []) {
   const idxInput = args.indexOf('--input');
   if (idxInput < 0 || !args[idxInput + 1]) {
     error(
-      'Usage: kdna compare <name> --input "<text>" [--report-md|--report-json] [--output <file>]',
+      'Usage: kdna compare <name|file.kdna> --input "<text>" [--report-md|--report-json] [--output <file>]',
       EXIT.INPUT_ERROR,
     );
   }
   const userInput = args[idxInput + 1];
 
-  const parsed = parseName(input);
-  if (!parsed) error(`Invalid name "${input}".`, EXIT.INPUT_ERROR);
-  const destDir = path.join(INSTALL_DIR, parsed.scope, parsed.ident);
-  if (!fs.existsSync(destDir)) {
-    error(`${parsed.full} not installed. Run: kdna install ${input}`, EXIT.INPUT_ERROR);
-  }
+  const asset = resolveAsset(input);
+  if (!asset) error(`KDNA asset not found: ${input}. Use an installed name or a .kdna file.`, EXIT.INPUT_ERROR);
+  const parsed = asset.parsed || parseName(asset.name || '');
+  const label = parsed?.full || asset.name || input;
 
   const llm = loadLlmConfig();
-  const manifest = readJson(path.join(destDir, 'kdna.json')) || {};
-  const core = readJson(path.join(destDir, 'KDNA_Core.json')) || {};
-  const pat = readJson(path.join(destDir, 'KDNA_Patterns.json')) || {};
+  const container = readContainer(asset.asset_path);
+  const manifest = container.manifest || {};
+  const core = container.core || {};
+  const pat = container.patterns || {};
 
   if (!jsonMode && !reportMd && !reportJson) {
     console.log('═'.repeat(64));
-    console.log(`  kdna compare  ${parsed.full}`);
+    console.log(`  kdna compare  ${label}`);
     console.log(`  provider:     ${llm.provider} / ${llm.model}`);
     console.log(`  input length: ${userInput.length} chars`);
     console.log('═'.repeat(64));
@@ -516,7 +514,7 @@ async function cmdCompare(input, args = []) {
 
   const BASELINE_SYSTEM =
     'You are a helpful assistant. Respond to the user request concisely and specifically.';
-  const kdnaPrompt = buildKdnaPrompt(destDir);
+  const kdnaPrompt = buildKdnaPrompt(container);
   if (!kdnaPrompt) error('Could not build KDNA prompt — missing KDNA_Core or KDNA_Patterns.');
   const TREATMENT_SYSTEM =
     'You are a helpful assistant. The following domain judgment is loaded and you MUST apply it when relevant.\n\n' +
@@ -540,13 +538,21 @@ async function cmdCompare(input, args = []) {
   recordTrace({
     timestamp: new Date().toISOString(),
     agent: 'cli',
-    domain: parsed.full,
+    domain: label,
     type: 'compare',
+    asset: {
+      asset_path: asset.asset_path,
+      asset_digest: asset.asset_digest || null,
+      content_digest: asset.content_digest || null,
+      version: manifest.version || asset.version || null,
+      judgment_version: manifest.judgment_version || asset.judgment_version || null,
+      access: manifest.access || asset.access || null,
+    },
     compare: { model: llm.model, input_length: userInput.length },
   });
 
   if (reportMd) {
-    const report = emitMarkdownReport(parsed, manifest, core, pat, responseA, responseB, diff, llm);
+    const report = emitMarkdownReport(parsed || { full: label }, manifest, core, pat, responseA, responseB, diff, llm);
     if (outputFile) {
       fs.writeFileSync(outputFile, report);
       console.log(`Report saved to ${outputFile}`);
@@ -558,7 +564,7 @@ async function cmdCompare(input, args = []) {
 
   if (reportJson) {
     const report = emitJsonReport(
-      parsed,
+      parsed || { full: label },
       manifest,
       core,
       pat,

package/src/diff.js

@@ -1,7 +1,7 @@
 /**
  * kdna diff <name>@<ver1> <name>@<ver2> — Judgment-level diff between versions.
  *
- * Downloads two .kdna packages from the registry, extracts to temp dirs,
+ * Downloads two .kdna dev packages from the registry, extracts to temp dirs,
  * compares axioms / misunderstandings / banned_terms / stances / boundary,
  * and surfaces what would change for an agent loading the new version.
  *
@@ -19,11 +19,10 @@
 const fs = require('fs');
 const path = require('path');
 const { execSync, execFileSync } = require('child_process');
-const { RegistryResolver, parseName } = require('./registry');
+const { RegistryResolver } = require('./registry');
 const { EXIT } = require('./cmds/_common');
+const { getInstalled, readContainer } = require('./package-store');
 
-const USER_KDNA_DIR = path.join(process.env.HOME || process.env.USERPROFILE || '.', '.kdna');
-const INSTALL_DIR = path.join(USER_KDNA_DIR, 'domains');
 const TMP_DIR = '/tmp';
 
 function error(msg, code = EXIT.VALIDATION_FAILED) {
@@ -58,14 +57,15 @@ function parseNameVersion(input) {
 // ─── Download specific version ────────────────────────────────────────
 
 function downloadVersion(entry, version, destDir) {
-  // entry.kdna_url is for the registry-current version. For older versions
+  const assetUrl = entry.asset_url;
+  // assetUrl is for the registry-current version. For older versions
   // we infer the URL pattern from the registry-current URL.
   if (entry.version === version) {
-    return downloadAndExtract(entry.kdna_url, destDir);
+    return downloadAndExtract(assetUrl, destDir);
   }
 
   // Infer pattern: replace v<current> in the URL with v<requested>
-  const inferredUrl = entry.kdna_url
+  const inferredUrl = assetUrl
     .replace(`/v${entry.version}/`, `/v${version}/`)
     .replace(`-${entry.version}.kdna`, `-${version}.kdna`);
 
@@ -232,12 +232,11 @@ async function cmdDiff(a, b, args = []) {
     newEntry = entryB;
   } else {
     // single-arg form: installed vs registry-current
-    const parsed = parseName(aParsed.full);
-    const localDir = path.join(INSTALL_DIR, parsed.scope, parsed.ident);
-    if (!fs.existsSync(localDir)) {
+    const installed = getInstalled(aParsed.full);
+    if (!installed) {
       error(`${aParsed.full} not installed. Run: kdna install ${aParsed.full}`, EXIT.INPUT_ERROR);
     }
-    const localManifest = readJson(path.join(localDir, 'kdna.json'));
+    const localManifest = readContainer(installed.asset_path).manifest || {};
     oldVersion = localManifest?.version || '?';
     newVersion = entryA.version;
     oldEntry = entryA;
@@ -285,7 +284,7 @@ async function cmdDiff(a, b, args = []) {
   }
 
   const axiomsDiff = diffMaps('axioms', oldJ.axioms, newJ.axioms, (a) => a.one_sentence || a.id, jsonMode);
-  const ontologyDiff = diffMaps('ontology', oldJ.ontology, newJ.ontology, (o) => o.one_sentence || o.id, jsonMode);
+  diffMaps('ontology', oldJ.ontology, newJ.ontology, (o) => o.one_sentence || o.id, jsonMode);
   const misunderstandingsDiff = diffMaps(
     'misunderstandings',
     oldJ.misunderstandings,
@@ -345,7 +344,6 @@ async function cmdDiff(a, b, args = []) {
     }));
 
   // Determine recommended version bump
-  const axiomDrift = Object.keys(newJ.axioms).length - Object.keys(oldJ.axioms).length;
   const hasRemoved = axiomsDiff.removed.length > 0 || misunderstandingsDiff.removed.length > 0;
   const hasAdded = axiomsDiff.added.length > 0 || misunderstandingsDiff.added.length > 0;
   const hasChanged = axiomsDiff.changed.length > 0 || bannedDiff.changed.length > 0;

package/src/init.js

@@ -91,7 +91,7 @@ function cmdInit(name) {
     `  2. Edit ${targetDir}/KDNA_Patterns.json — replace terminology and misunderstandings`,
   );
   console.log(`  3. Edit ${targetDir}/kdna.json — set author, description, repo`);
-  console.log(`  4. Run: kdna validate ${name}               (structural check)`);
+  console.log(`  4. Run: kdna dev validate ${name}           (structural check)`);
   console.log(`  5. Run: kdna publish --check ${name}         (content quality gate)`);
   console.log(`  6. Run: kdna verify ${name}                  (full judgment scoring)`);
 }
@@ -162,7 +162,7 @@ function cmdClusterInit(name) {
   console.log(
     `  3. Add more sub-domains: cp -r ${targetDir}/domain_one ${targetDir}/your_new_domain`,
   );
-  console.log(`  4. Run: kdna validate ${name}              (check all sub-domains)`);
+  console.log(`  4. Run: kdna dev validate ${name}          (check all sub-domains)`);
 }
 
 module.exports = { cmdInit, cmdClusterInit };