@codeyam/codeyam-cli 0.1.25 → 0.1.27

package/analyzer-template/.build-info.json

@@ -1,10 +1,10 @@
 {
-  "buildTimestamp": "2026-03-27T13:51:55.725Z",
-  "buildTime": 1774619515725,
-  "gitCommit": "cefcc7c0aa462f7267e8912e66de352da511a931",
+  "buildTimestamp": "2026-03-28T18:03:51.730Z",
+  "buildTime": 1774721031730,
+  "gitCommit": "3d7e1f0d118b714119562b24234d8e7ae6832002",
   "nodeVersion": "v20.20.1",
   "contentHash": "c92230c027acb71cab56d2a696876a6a52206bfadd59fbc31a512b00a7ee8826",
-  "buildNumber": 1256,
-  "semanticVersion": "0.1.1256",
-  "version": "0.1.1256 (2026-03-27T13:51+c92230c)"
+  "buildNumber": 1271,
+  "semanticVersion": "0.1.1271",
+  "version": "0.1.1271 (2026-03-28T18:03+c92230c)"
 }

package/analyzer-template/log.txt

@@ -1,7 +1,7 @@
 
-[3/27/2026, 1:51:55 PM] > codeyam-combo@1.0.0 mergeDependencies
-[3/27/2026, 1:51:55 PM] > node ./scripts/mergePackageJsonFiles.cjs
+[3/28/2026, 6:03:51 PM] > codeyam-combo@1.0.0 mergeDependencies
+[3/28/2026, 6:03:51 PM] > node ./scripts/mergePackageJsonFiles.cjs
 
 
-[3/27/2026, 1:51:55 PM] Merged dependencies into root package.json
+[3/28/2026, 6:03:51 PM] Merged dependencies into root package.json
 

package/codeyam-cli/src/commands/__tests__/editor.statePersistence.test.js

@@ -0,0 +1,55 @@
+import fs from 'fs';
+import path from 'path';
+/**
+ * Structural test: every writeState() call in editor.ts must preserve
+ * `techStackId` and `appFormats` from the previous state.
+ *
+ * Bug: Steps 4-18 dropped techStackId from writeState calls, causing
+ * getTechStackContext() to fall back to Next.js defaults. This broke
+ * `codeyam editor isolate` for Expo projects — it created Next.js-style
+ * subdirectories instead of Expo flat files.
+ */
+describe('editor state persistence', () => {
+    const editorSource = fs.readFileSync(path.join(__dirname, '..', 'editor.ts'), 'utf8');
+    // Find all writeState blocks in the source.
+    // Each writeState call spans multiple lines like:
+    //   writeState(root, {
+    //     feature,
+    //     step: N,
+    //     ...
+    //   });
+    const writeStateBlocks = [];
+    const lines = editorSource.split('\n');
+    for (let i = 0; i < lines.length; i++) {
+        if (lines[i].includes('writeState(root,')) {
+            // Collect lines until we find the closing `});`
+            let block = '';
+            for (let j = i; j < lines.length; j++) {
+                block += lines[j] + '\n';
+                if (lines[j].match(/^\s*\}\);/))
+                    break;
+            }
+            writeStateBlocks.push({ lineNumber: i + 1, block });
+        }
+    }
+    it('should find writeState calls in editor.ts', () => {
+        expect(writeStateBlocks.length).toBeGreaterThan(10);
+    });
+    // Filter to only writeState blocks that set a step number (printStep functions).
+    // Exclude the handleTemplate/migration-survey writeState that sets step: 0.
+    const stepWriteStates = writeStateBlocks.filter(({ block }) => block.match(/step:\s*\d+/) && !block.match(/step:\s*0/));
+    it('should find step writeState calls', () => {
+        expect(stepWriteStates.length).toBeGreaterThanOrEqual(18);
+    });
+    for (const { lineNumber, block } of stepWriteStates) {
+        const stepMatch = block.match(/step:\s*(\d+)/);
+        const stepNum = stepMatch ? stepMatch[1] : '?';
+        it(`step ${stepNum} writeState (line ${lineNumber}) should preserve techStackId`, () => {
+            expect(block).toContain('techStackId');
+        });
+        it(`step ${stepNum} writeState (line ${lineNumber}) should preserve appFormats`, () => {
+            expect(block).toContain('appFormats');
+        });
+    }
+});
+//# sourceMappingURL=editor.statePersistence.test.js.map

package/codeyam-cli/src/commands/__tests__/editor.statePersistence.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"editor.statePersistence.test.js","sourceRoot":"","sources":["../../../../../src/commands/__tests__/editor.statePersistence.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,IAAI,CAAC;AACpB,OAAO,IAAI,MAAM,MAAM,CAAC;AAExB;;;;;;;;GAQG;AACH,QAAQ,CAAC,0BAA0B,EAAE,GAAG,EAAE;IACxC,MAAM,YAAY,GAAG,EAAE,CAAC,YAAY,CAClC,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,WAAW,CAAC,EACvC,MAAM,CACP,CAAC;IAEF,4CAA4C;IAC5C,kDAAkD;IAClD,uBAAuB;IACvB,eAAe;IACf,eAAe;IACf,UAAU;IACV,QAAQ;IACR,MAAM,gBAAgB,GAA4C,EAAE,CAAC;IACrE,MAAM,KAAK,GAAG,YAAY,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IACvC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,kBAAkB,CAAC,EAAE,CAAC;YAC1C,gDAAgD;YAChD,IAAI,KAAK,GAAG,EAAE,CAAC;YACf,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;gBACtC,KAAK,IAAI,KAAK,CAAC,CAAC,CAAC,GAAG,IAAI,CAAC;gBACzB,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,WAAW,CAAC;oBAAE,MAAM;YACzC,CAAC;YACD,gBAAgB,CAAC,IAAI,CAAC,EAAE,UAAU,EAAE,CAAC,GAAG,CAAC,EAAE,KAAK,EAAE,CAAC,CAAC;QACtD,CAAC;IACH,CAAC;IAED,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,MAAM,CAAC,gBAAgB,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,EAAE,CAAC,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,iFAAiF;IACjF,4EAA4E;IAC5E,MAAM,eAAe,GAAG,gBAAgB,CAAC,MAAM,CAC7C,CAAC,EAAE,KAAK,EAAE,EAAE,EAAE,CAAC,KAAK,CAAC,KAAK,CAAC,aAAa,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,WAAW,CAAC,CACvE,CAAC;IAEF,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC,sBAAsB,CAAC,EAAE,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;IAEH,KAAK,MAAM,EAAE,UAAU,EAAE,KAAK,EAAE,IAAI,eAAe,EAAE,CAAC;QACpD,MAAM,SAAS,GAAG,KAAK,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC;QAC/C,MAAM,OAAO,GAAG,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC;QAE/C,EAAE,CAAC,QAAQ,OAAO,qBAAqB,UAAU,+BAA+B,EAAE,GAAG,EAAE;YACrF,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,aAAa,CAAC,CAAC;QACzC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,QAAQ,OAAO,qBAAqB,UAAU,8BAA8B,EAAE,GAAG,EAAE;YACpF,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;QACxC,CAAC,CAAC,CAAC;IACL,CAAC;AACH,CAAC,CAAC,CAAC"}

package/codeyam-cli/src/commands/editor.js

@@ -305,6 +305,10 @@ function printAppScenarioInstructions(pageName, route) {
         console.log(chalk.dim('    Example: "Page - Full Data", "Page - Empty State", "Page - Error State"'));
     }
     console.log();
+    checkbox('Ensure every page file used in app scenarios has a glossary entry with its filePath:');
+    console.log(chalk.dim('    The register command validates pageFilePath against the glossary — add missing pages now.'));
+    console.log(chalk.dim('    Example: {"name":"CounterScreen","filePath":"app/(tabs)/index.tsx","type":"page","description":"Main counter page"}'));
+    console.log();
     checkbox('Register each scenario with type "application", the real page URL, and pageFilePath:');
     console.log(chalk.dim(`    codeyam editor register '{"name":"${pageName || 'Page'} - Full Data",`));
     console.log(chalk.dim(`      "type":"application","url":"${route || '/'}","pageFilePath":"src/path/to/Page.tsx",...}'`));
@@ -402,9 +406,15 @@ function printExtractionPlanInstructions() {
 function printComponentCaptureInstructions(root) {
     const ctx = root ? getTechStackContext(root) : undefined;
     const isExpo = ctx?.isExpo ?? false;
-    checkbox('Create isolation route dirs: `codeyam editor isolate ComponentA ComponentB ...`');
-    console.log(chalk.dim(`    This creates app/isolated-components/layout.tsx (with production ${isExpo ? '__DEV__' : 'notFound()'} guard) and`));
-    console.log(chalk.dim('    a directory per component. List ALL components that need isolation routes.'));
+    checkbox('Set up isolation routes: `codeyam editor isolate ComponentA ComponentB ...`');
+    if (isExpo) {
+        console.log(chalk.dim('    This creates app/isolated-components/_layout.tsx (with __DEV__ guard).'));
+        console.log(chalk.dim('    You then create a flat .tsx file per component (NOT subdirectories — Expo Router uses flat files).'));
+    }
+    else {
+        console.log(chalk.dim(`    This creates app/isolated-components/layout.tsx (with notFound() guard) and`));
+        console.log(chalk.dim('    a directory per component. List ALL components that need isolation routes.'));
+    }
     checkbox('For each visual component:');
     console.log(chalk.dim('    1. Read the source AND find where it is used in the app to understand:'));
     console.log(chalk.dim('       — Props/interface'));
@@ -805,8 +815,8 @@ function printSetup(root) {
     console.log(chalk.bold('Tech Stack Selection:'));
     console.log(chalk.dim('  Based on the selected formats, present ONLY the matching tech stacks.'));
     console.log(chalk.dim('  A stack matches if ANY of the user\'s selected formats appears in its "Supports" list.'));
-    console.log(chalk.dim('  Show ALL matching stacks even if there is only one. Mark the recommended option.'));
-    console.log(chalk.dim('  Use AskUserQuestion to let the user pick one.'));
+    console.log(chalk.dim('  If only ONE stack matches, confirm it directly (e.g. "Expo + React Native is the only stack for mobile apps — using that.").'));
+    console.log(chalk.dim('  If MULTIPLE stacks match, use AskUserQuestion to let the user pick one.'));
     console.log();
     console.log(chalk.bold('  Available Tech Stacks:'));
     for (const stack of TECH_STACKS) {
@@ -1249,6 +1259,8 @@ function printStep4(root, feature) {
         label: STEP_LABELS[4],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 4, label: 'Verify Prototype', feature });
     stepHeader(4, 'Verify Prototype', feature);
@@ -1305,6 +1317,8 @@ function printStep5(root, feature) {
         label: STEP_LABELS[5],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 5, label: 'Confirm', feature });
     stepHeader(5, 'Confirm', feature);
@@ -1364,6 +1378,8 @@ function printStep6(root, feature) {
         label: STEP_LABELS[6],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 6, label: 'Deconstruct', feature });
     stepHeader(6, 'Deconstruct', feature);
@@ -1389,6 +1405,8 @@ function printStep7(root, feature) {
         label: STEP_LABELS[7],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 7, label: 'Extract', feature });
     stepHeader(7, 'Extract', feature);
@@ -1409,6 +1427,7 @@ function printStep7(root, feature) {
     console.log(chalk.dim('    Cover: typical inputs, edge cases, empty/null inputs, error conditions'));
     console.log(chalk.dim('    Aim for 3-8 test cases per function depending on complexity'));
     console.log(chalk.dim('    Hooks count as functions — useDrinks, useAuth, etc. all need test files'));
+    console.log(chalk.dim('    Wrap all tests in a describe("FunctionName", ...) block — the audit matches on this name'));
     if (ctx.isExpo) {
         checkbox('Place test files next to source but OUTSIDE `app/` (Expo Router treats all files in app/ as routes): `lib/storage.ts` → `lib/storage.test.ts`, `app/hooks/useCounter.ts` → `__tests__/hooks/useCounter.test.ts`');
     }
@@ -1446,6 +1465,8 @@ function printStep8(root, feature) {
         label: STEP_LABELS[8],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 8, label: 'Glossary', feature });
     stepHeader(8, 'Glossary', feature);
@@ -1472,6 +1493,8 @@ function printStep9(root, feature) {
         label: STEP_LABELS[9],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 9, label: 'Analyze', feature });
     stepHeader(9, 'Analyze and Verify', feature);
@@ -1522,6 +1545,8 @@ function printStep10(root, feature) {
         label: STEP_LABELS[10],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 10, label: 'App Scenarios', feature });
     stepHeader(10, 'App Scenarios', feature);
@@ -1586,6 +1611,8 @@ function printStep11(root, feature) {
         label: STEP_LABELS[11],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 11, label: 'User Scenarios', feature });
     stepHeader(11, 'User Scenarios', feature);
@@ -1632,6 +1659,8 @@ function printStep12(root, feature) {
         label: STEP_LABELS[12],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 12, label: 'Verify', feature });
     stepHeader(12, 'Verify', feature);
@@ -1678,6 +1707,8 @@ function printStep13(root, feature) {
         label: STEP_LABELS[13],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 13, label: 'Journal', feature });
     stepHeader(13, 'Journal', feature);
@@ -1711,6 +1742,8 @@ function printStep14(root, feature) {
         label: STEP_LABELS[14],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 14, label: 'Review', feature });
     stepHeader(14, 'Review', feature);
@@ -1744,6 +1777,8 @@ function printStep15(root, feature) {
         label: STEP_LABELS[15],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 15, label: 'Present', feature });
     stepHeader(15, 'Present', feature);
@@ -2393,6 +2428,8 @@ function printStep16(root, feature) {
         label: STEP_LABELS[16],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 16, label: 'Commit', feature });
     stepHeader(16, 'Commit', feature);
@@ -2419,6 +2456,8 @@ function printStep17(root, feature) {
         label: STEP_LABELS[17],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 17, label: 'Finalize', feature });
     stepHeader(17, 'Finalize', feature);
@@ -2444,6 +2483,8 @@ function printStep18(root, feature) {
         label: STEP_LABELS[18],
         startedAt: isResuming ? prevState.startedAt : now,
         featureStartedAt: prevState?.featureStartedAt || now,
+        appFormats: prevState?.appFormats,
+        techStackId: prevState?.techStackId,
     });
     logEvent(root, 'step', { step: 18, label: 'Push', feature });
     stepHeader(18, 'Push', feature);
@@ -2884,8 +2925,13 @@ function handleIsolate(componentNames) {
         process.exit(1);
     }
     const isolateDir = path.join(root, 'app', 'isolated-components');
-    // Create layout.tsx with production guard if missing
+    // Create the framework-appropriate layout guard if missing.
+    // Clean up wrong-framework layout from a previous CLI version.
     const layoutPath = path.join(isolateDir, ctx.isExpo ? '_layout.tsx' : 'layout.tsx');
+    const wrongLayoutPath = path.join(isolateDir, ctx.isExpo ? 'layout.tsx' : '_layout.tsx');
+    if (fs.existsSync(wrongLayoutPath)) {
+        fs.unlinkSync(wrongLayoutPath);
+    }
     if (!fs.existsSync(layoutPath)) {
         fs.mkdirSync(isolateDir, { recursive: true });
         if (ctx.isExpo) {
@@ -2938,7 +2984,13 @@ function handleIsolate(componentNames) {
         }
     }
     if (created.length > 0) {
-        console.log(chalk.green(`Created ${created.length} isolation route(s): ${created.join(', ')}`));
+        if (ctx.isExpo) {
+            console.log(chalk.green(`Ready for ${created.length} isolation route(s): ${created.map((n) => `app/isolated-components/${n}.tsx`).join(', ')}`));
+            console.log(chalk.dim('    Create each file with useLocalSearchParams, a scenarios map, and nativeID="codeyam-capture"'));
+        }
+        else {
+            console.log(chalk.green(`Created ${created.length} isolation route dir(s): ${created.join(', ')}`));
+        }
     }
     if (existed.length > 0) {
         console.log(chalk.dim(`Already existed: ${existed.join(', ')}`));
@@ -3954,7 +4006,8 @@ async function handleAudit(options) {
                     if (f.errorMessage) {
                         detail += `\n      ${chalk.red(f.errorMessage)}`;
                         detail += `\n      ${chalk.yellow('Note: This is NOT a test failure — the test runner itself could not execute.')}`;
-                        detail += `\n      ${chalk.yellow('Try running the test manually: npx vitest run ' + f.testFile)}`;
+                        const ctx = getTechStackContext(process.cwd());
+                        detail += `\n      ${chalk.yellow('Try running the test manually: ' + ctx.testRunCommand + ' ' + f.testFile)}`;
                     }
                     break;
                 case 'failing':