@zigrivers/scaffold 2.1.0

package/dist/wizard/wizard.test.js

@@ -0,0 +1,167 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import fs from 'node:fs';
+import path from 'node:path';
+import os from 'node:os';
+import yaml from 'js-yaml';
+// ---------------------------------------------------------------------------
+// Hoisted mocks
+// ---------------------------------------------------------------------------
+vi.mock('../project/detector.js', () => ({
+    detectProjectMode: vi.fn(() => ({
+        mode: 'greenfield',
+        signals: [],
+        methodologySuggestion: 'deep',
+    })),
+}));
+vi.mock('../core/assembly/meta-prompt-loader.js', () => ({
+    discoverMetaPrompts: vi.fn(() => new Map()),
+}));
+// ---------------------------------------------------------------------------
+// Imports (after mocks)
+// ---------------------------------------------------------------------------
+import { detectProjectMode } from '../project/detector.js';
+import { runWizard } from './wizard.js';
+// ---------------------------------------------------------------------------
+// Helpers
+// ---------------------------------------------------------------------------
+function makeTempDir() {
+    return fs.mkdtempSync(path.join(os.tmpdir(), 'scaffold-wizard-test-'));
+}
+function makeOutputContext() {
+    return {
+        success: vi.fn(),
+        info: vi.fn(),
+        warn: vi.fn(),
+        error: vi.fn(),
+        result: vi.fn(),
+        prompt: vi.fn().mockResolvedValue(''),
+        confirm: vi.fn().mockResolvedValue(false),
+        startSpinner: vi.fn(),
+        stopSpinner: vi.fn(),
+        startProgress: vi.fn(),
+        updateProgress: vi.fn(),
+        stopProgress: vi.fn(),
+    };
+}
+// ---------------------------------------------------------------------------
+// Tests
+// ---------------------------------------------------------------------------
+describe('runWizard', () => {
+    const mockDetectProjectMode = vi.mocked(detectProjectMode);
+    let tmpDir;
+    beforeEach(() => {
+        tmpDir = makeTempDir();
+        mockDetectProjectMode.mockReturnValue({
+            mode: 'greenfield',
+            signals: [],
+            methodologySuggestion: 'deep',
+        });
+    });
+    afterEach(() => {
+        fs.rmSync(tmpDir, { recursive: true, force: true });
+        vi.restoreAllMocks();
+    });
+    // Test 1: Creates .scaffold/ directory
+    it('creates .scaffold/ directory', async () => {
+        const output = makeOutputContext();
+        await runWizard({ projectRoot: tmpDir, auto: true, force: false, output });
+        expect(fs.existsSync(path.join(tmpDir, '.scaffold'))).toBe(true);
+    });
+    // Test 2: Writes valid config.yml with YAML content
+    it('writes valid config.yml with YAML content', async () => {
+        const output = makeOutputContext();
+        await runWizard({ projectRoot: tmpDir, auto: true, force: false, output });
+        const configPath = path.join(tmpDir, '.scaffold', 'config.yml');
+        expect(fs.existsSync(configPath)).toBe(true);
+        const content = fs.readFileSync(configPath, 'utf8');
+        const parsed = yaml.load(content);
+        expect(parsed).toMatchObject({ version: 2, methodology: expect.any(String) });
+        expect(Array.isArray(parsed['platforms'])).toBe(true);
+        expect(parsed['platforms'].length).toBeGreaterThan(0);
+    });
+    // Test 3: Creates state.json
+    it('creates state.json', async () => {
+        const output = makeOutputContext();
+        await runWizard({ projectRoot: tmpDir, auto: true, force: false, output });
+        const statePath = path.join(tmpDir, '.scaffold', 'state.json');
+        expect(fs.existsSync(statePath)).toBe(true);
+        const parsed = JSON.parse(fs.readFileSync(statePath, 'utf8'));
+        expect(parsed['schema-version']).toBe(1);
+    });
+    // Test 4: Creates empty decisions.jsonl
+    it('creates empty decisions.jsonl', async () => {
+        const output = makeOutputContext();
+        await runWizard({ projectRoot: tmpDir, auto: true, force: false, output });
+        const decisionsPath = path.join(tmpDir, '.scaffold', 'decisions.jsonl');
+        expect(fs.existsSync(decisionsPath)).toBe(true);
+        expect(fs.readFileSync(decisionsPath, 'utf8')).toBe('');
+    });
+    // Test 5: Creates .scaffold/instructions/ directory
+    it('creates .scaffold/instructions/ directory', async () => {
+        const output = makeOutputContext();
+        await runWizard({ projectRoot: tmpDir, auto: true, force: false, output });
+        expect(fs.existsSync(path.join(tmpDir, '.scaffold', 'instructions'))).toBe(true);
+        expect(fs.statSync(path.join(tmpDir, '.scaffold', 'instructions')).isDirectory()).toBe(true);
+    });
+    // Test 6: Returns INIT_SCAFFOLD_EXISTS error when .scaffold/ exists without --force
+    it('returns INIT_SCAFFOLD_EXISTS error when .scaffold/ exists and force=false', async () => {
+        fs.mkdirSync(path.join(tmpDir, '.scaffold'));
+        const output = makeOutputContext();
+        const result = await runWizard({ projectRoot: tmpDir, auto: true, force: false, output });
+        expect(result.success).toBe(false);
+        expect(result.errors.length).toBeGreaterThan(0);
+        expect(result.errors[0].code).toBe('INIT_SCAFFOLD_EXISTS');
+    });
+    // Test 7: With --force, backs up existing .scaffold/ and reinitializes
+    it('backs up existing .scaffold/ when force=true and reinitializes', async () => {
+        const scaffoldDir = path.join(tmpDir, '.scaffold');
+        fs.mkdirSync(scaffoldDir);
+        fs.writeFileSync(path.join(scaffoldDir, 'marker.txt'), 'original');
+        const output = makeOutputContext();
+        const result = await runWizard({ projectRoot: tmpDir, auto: true, force: true, output });
+        expect(result.success).toBe(true);
+        // Original .scaffold/ backed up — a .scaffold.backup dir should exist
+        const backupExists = fs.existsSync(path.join(tmpDir, '.scaffold.backup'));
+        expect(backupExists).toBe(true);
+        // New .scaffold/ should exist and have config.yml, not marker.txt
+        expect(fs.existsSync(path.join(scaffoldDir, 'config.yml'))).toBe(true);
+        expect(fs.existsSync(path.join(scaffoldDir, 'marker.txt'))).toBe(false);
+    });
+    // Test 8: --auto mode uses suggestion as methodology without prompting
+    it('uses suggestion methodology in auto mode without calling prompt', async () => {
+        const output = makeOutputContext();
+        await runWizard({ projectRoot: tmpDir, auto: true, force: false, output });
+        expect(output.prompt).not.toHaveBeenCalled();
+        const configPath = path.join(tmpDir, '.scaffold', 'config.yml');
+        const parsed = yaml.load(fs.readFileSync(configPath, 'utf8'));
+        // Default suggestion from greenfield is 'deep'
+        expect(parsed['methodology']).toBe('deep');
+    });
+    // Test 9: --methodology 'mvp' skips methodology question and uses it
+    it('uses pre-set --methodology flag without prompting', async () => {
+        const output = makeOutputContext();
+        const result = await runWizard({
+            projectRoot: tmpDir,
+            auto: true,
+            force: false,
+            methodology: 'mvp',
+            output,
+        });
+        expect(result.success).toBe(true);
+        expect(result.methodology).toBe('mvp');
+        const configPath = path.join(tmpDir, '.scaffold', 'config.yml');
+        const parsed = yaml.load(fs.readFileSync(configPath, 'utf8'));
+        expect(parsed['methodology']).toBe('mvp');
+        expect(output.prompt).not.toHaveBeenCalled();
+    });
+    // Test 10: Result has success: true on successful init
+    it('returns success: true on successful init', async () => {
+        const output = makeOutputContext();
+        const result = await runWizard({ projectRoot: tmpDir, auto: true, force: false, output });
+        expect(result.success).toBe(true);
+        expect(result.errors).toHaveLength(0);
+        expect(result.projectRoot).toBe(tmpDir);
+        expect(result.configPath).toBe(path.join(tmpDir, '.scaffold', 'config.yml'));
+    });
+});
+//# sourceMappingURL=wizard.test.js.map

package/dist/wizard/wizard.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"wizard.test.js","sourceRoot":"","sources":["../../src/wizard/wizard.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAA;AACxE,OAAO,EAAE,MAAM,SAAS,CAAA;AACxB,OAAO,IAAI,MAAM,WAAW,CAAA;AAC5B,OAAO,EAAE,MAAM,SAAS,CAAA;AACxB,OAAO,IAAI,MAAM,SAAS,CAAA;AAE1B,8EAA8E;AAC9E,gBAAgB;AAChB,8EAA8E;AAE9E,EAAE,CAAC,IAAI,CAAC,wBAAwB,EAAE,GAAG,EAAE,CAAC,CAAC;IACvC,iBAAiB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,CAAC;QAC9B,IAAI,EAAE,YAAY;QAClB,OAAO,EAAE,EAAE;QACX,qBAAqB,EAAE,MAAM;KAC9B,CAAC,CAAC;CACJ,CAAC,CAAC,CAAA;AAEH,EAAE,CAAC,IAAI,CAAC,wCAAwC,EAAE,GAAG,EAAE,CAAC,CAAC;IACvD,mBAAmB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,IAAI,GAAG,EAAE,CAAC;CAC5C,CAAC,CAAC,CAAA;AAEH,8EAA8E;AAC9E,wBAAwB;AACxB,8EAA8E;AAE9E,OAAO,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAA;AAC1D,OAAO,EAAE,SAAS,EAAE,MAAM,aAAa,CAAA;AAEvC,8EAA8E;AAC9E,UAAU;AACV,8EAA8E;AAE9E,SAAS,WAAW;IAClB,OAAO,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,uBAAuB,CAAC,CAAC,CAAA;AACxE,CAAC;AAED,SAAS,iBAAiB;IACxB,OAAO;QACL,OAAO,EAAE,EAAE,CAAC,EAAE,EAAE;QAChB,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE;QACb,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE;QACb,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE;QACd,MAAM,EAAE,EAAE,CAAC,EAAE,EAAE;QACf,MAAM,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,CAAC;QACrC,OAAO,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,KAAK,CAAC;QACzC,YAAY,EAAE,EAAE,CAAC,EAAE,EAAE;QACrB,WAAW,EAAE,EAAE,CAAC,EAAE,EAAE;QACpB,aAAa,EAAE,EAAE,CAAC,EAAE,EAAE;QACtB,cAAc,EAAE,EAAE,CAAC,EAAE,EAAE;QACvB,YAAY,EAAE,EAAE,CAAC,EAAE,EAAE;KACtB,CAAA;AACH,CAAC;AAED,8EAA8E;AAC9E,QAAQ;AACR,8EAA8E;AAE9E,QAAQ,CAAC,WAAW,EAAE,GAAG,EAAE;IACzB,MAAM,qBAAqB,GAAG,EAAE,CAAC,MAAM,CAAC,iBAAiB,CAAC,CAAA;IAC1D,IAAI,MAAc,CAAA;IAElB,UAAU,CAAC,GAAG,EAAE;QACd,MAAM,GAAG,WAAW,EAAE,CAAA;QACtB,qBAAqB,CAAC,eAAe,CAAC;YACpC,IAAI,EAAE,YAAY;YAClB,OAAO,EAAE,EAAE;YACX,qBAAqB,EAAE,MAAM;SAC9B,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,SAAS,CAAC,GAAG,EAAE;QACb,EAAE,CAAC,MAAM,CAAC,MAAM,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAA;QACnD,EAAE,CAAC,eAAe,EAAE,CAAA;IACtB,CAAC,CAAC,CAAA;IAEF,uCAAuC;IACvC,EAAE,CAAC,8BAA8B,EAAE,KAAK,IAAI,EAAE;QAC5C,MAAM,MAAM,GAAG,iBAAiB,EAAE,CAAA;QAClC,MAAM,SAAS,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAA;QAC1E,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IAClE,CAAC,CAAC,CAAA;IAEF,oDAAoD;IACpD,EAAE,CAAC,2CAA2C,EAAE,KAAK,IAAI,EAAE;QACzD,MAAM,MAAM,GAAG,iBAAiB,EAAE,CAAA;QAClC,MAAM,SAAS,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAA;QAC1E,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,WAAW,EAAE,YAAY,CAAC,CAAA;QAC/D,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAC5C,MAAM,OAAO,GAAG,EAAE,CAAC,YAAY,CAAC,UAAU,EAAE,MAAM,CAAC,CAAA;QACnD,MAAM,MAAM,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,CAA4B,CAAA;QAC5D,MAAM,CAAC,MAAM,CAAC,CAAC,aAAa,CAAC,EAAE,OAAO,EAAE,CAAC,EAAE,WAAW,EAAE,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,CAAA;QAC7E,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACrD,MAAM,CAAE,MAAM,CAAC,WAAW,CAAc,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAA;IACrE,CAAC,CAAC,CAAA;IAEF,6BAA6B;IAC7B,EAAE,CAAC,oBAAoB,EAAE,KAAK,IAAI,EAAE;QAClC,MAAM,MAAM,GAAG,iBAAiB,EAAE,CAAA;QAClC,MAAM,SAAS,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAA;QAC1E,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,WAAW,EAAE,YAAY,CAAC,CAAA;QAC9D,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAC3C,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC,YAAY,CAAC,SAAS,EAAE,MAAM,CAAC,CAA4B,CAAA;QACxF,MAAM,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IAC1C,CAAC,CAAC,CAAA;IAEF,wCAAwC;IACxC,EAAE,CAAC,+BAA+B,EAAE,KAAK,IAAI,EAAE;QAC7C,MAAM,MAAM,GAAG,iBAAiB,EAAE,CAAA;QAClC,MAAM,SAAS,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAA;QAC1E,MAAM,aAAa,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,WAAW,EAAE,iBAAiB,CAAC,CAAA;QACvE,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAC/C,MAAM,CAAC,EAAE,CAAC,YAAY,CAAC,aAAa,EAAE,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IACzD,CAAC,CAAC,CAAA;IAEF,oDAAoD;IACpD,EAAE,CAAC,2CAA2C,EAAE,KAAK,IAAI,EAAE;QACzD,MAAM,MAAM,GAAG,iBAAiB,EAAE,CAAA;QAClC,MAAM,SAAS,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAA;QAC1E,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,WAAW,EAAE,cAAc,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAChF,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,WAAW,EAAE,cAAc,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IAC9F,CAAC,CAAC,CAAA;IAEF,oFAAoF;IACpF,EAAE,CAAC,2EAA2E,EAAE,KAAK,IAAI,EAAE;QACzF,EAAE,CAAC,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,WAAW,CAAC,CAAC,CAAA;QAC5C,MAAM,MAAM,GAAG,iBAAiB,EAAE,CAAA;QAClC,MAAM,MAAM,GAAG,MAAM,SAAS,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAA;QACzF,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAA;QAC/C,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAE,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,sBAAsB,CAAC,CAAA;IAC7D,CAAC,CAAC,CAAA;IAEF,uEAAuE;IACvE,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;QAC9E,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,WAAW,CAAC,CAAA;QAClD,EAAE,CAAC,SAAS,CAAC,WAAW,CAAC,CAAA;QACzB,EAAE,CAAC,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,YAAY,CAAC,EAAE,UAAU,CAAC,CAAA;QAClE,MAAM,MAAM,GAAG,iBAAiB,EAAE,CAAA;QAClC,MAAM,MAAM,GAAG,MAAM,SAAS,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC,CAAA;QACxF,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACjC,sEAAsE;QACtE,MAAM,YAAY,GAAG,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,kBAAkB,CAAC,CAAC,CAAA;QACzE,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAC/B,kEAAkE;QAClE,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,YAAY,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACtE,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,YAAY,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACzE,CAAC,CAAC,CAAA;IAEF,uEAAuE;IACvE,EAAE,CAAC,iEAAiE,EAAE,KAAK,IAAI,EAAE;QAC/E,MAAM,MAAM,GAAG,iBAAiB,EAAE,CAAA;QAClC,MAAM,SAAS,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAA;QAC1E,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAA;QAC5C,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,WAAW,EAAE,YAAY,CAAC,CAAA;QAC/D,MAAM,MAAM,GAAG,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,YAAY,CAAC,UAAU,EAAE,MAAM,CAAC,CAA4B,CAAA;QACxF,+CAA+C;QAC/C,MAAM,CAAC,MAAM,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;IAC5C,CAAC,CAAC,CAAA;IAEF,qEAAqE;IACrE,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;QACjE,MAAM,MAAM,GAAG,iBAAiB,EAAE,CAAA;QAClC,MAAM,MAAM,GAAG,MAAM,SAAS,CAAC;YAC7B,WAAW,EAAE,MAAM;YACnB,IAAI,EAAE,IAAI;YACV,KAAK,EAAE,KAAK;YACZ,WAAW,EAAE,KAAK;YAClB,MAAM;SACP,CAAC,CAAA;QACF,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACjC,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACtC,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,WAAW,EAAE,YAAY,CAAC,CAAA;QAC/D,MAAM,MAAM,GAAG,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,YAAY,CAAC,UAAU,EAAE,MAAM,CAAC,CAA4B,CAAA;QACxF,MAAM,CAAC,MAAM,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACzC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAA;IAC9C,CAAC,CAAC,CAAA;IAEF,uDAAuD;IACvD,EAAE,CAAC,0CAA0C,EAAE,KAAK,IAAI,EAAE;QACxD,MAAM,MAAM,GAAG,iBAAiB,EAAE,CAAA;QAClC,MAAM,MAAM,GAAG,MAAM,SAAS,CAAC,EAAE,WAAW,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAA;QACzF,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACjC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACrC,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;QACvC,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,WAAW,EAAE,YAAY,CAAC,CAAC,CAAA;IAC9E,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/knowledge/core/adr-craft.md

@@ -0,0 +1,281 @@
+---
+name: adr-craft
+description: Writing effective architecture decision records including technology selection
+topics: [adr, architecture-decisions, tech-stack, decision-documentation, technology-selection]
+---
+
+## What Warrants an ADR
+
+An Architecture Decision Record captures a decision that is architecturally significant — one that affects the system's structure, non-functional characteristics, dependencies, interfaces, or construction techniques. Not every decision needs an ADR. The litmus test: would a new team member need to know why this choice was made to work effectively?
+
+### Categories That Always Warrant ADRs
+
+**Technology selection decisions:**
+
+- Programming language and runtime (Node.js vs. Python, JVM version)
+- Framework selection (Next.js vs. Remix, FastAPI vs. Django, Express vs. Fastify)
+- Database engine (PostgreSQL vs. SQLite vs. MongoDB, and why)
+- ORM or data access layer (Prisma vs. Drizzle vs. raw SQL)
+- Infrastructure and hosting (Vercel vs. AWS vs. self-hosted)
+- Authentication provider (Auth0 vs. Clerk vs. custom)
+- CI/CD platform (GitHub Actions vs. CircleCI)
+- Dev tooling choices (linter, formatter, test runner, build tool)
+
+Each technology choice should compare the top 2-3 realistic options, state the selection criteria weighted for the project, and document the winner with honest trade-offs.
+
+**Architectural pattern decisions:**
+
+- Monolith vs. microservices vs. modular monolith
+- Layered architecture vs. hexagonal vs. event-driven
+- Server-side rendering vs. client-side rendering vs. hybrid
+- Synchronous communication vs. event-driven vs. mixed
+- Caching strategy (when, where, what eviction policy)
+
+**Data architecture decisions:**
+
+- Normalization level and deliberate denormalizations
+- Event sourcing vs. state-based persistence
+- Read/write model separation (CQRS)
+- Data migration strategy (rolling vs. downtime)
+- Multi-tenancy approach (database per tenant, schema per tenant, row-level)
+
+**Integration decisions:**
+
+- API style (REST vs. GraphQL vs. gRPC)
+- Authentication mechanism (JWT vs. session vs. API key)
+- Third-party service selections (payment processor, email provider, CDN)
+- Message broker selection (when async is chosen)
+
+**Development process decisions:**
+
+- Branching strategy (trunk-based, GitFlow, simplified flow)
+- Testing strategy (what gets unit tests vs. integration vs. e2e)
+- Deployment strategy (blue-green, canary, rolling)
+- Monitoring approach (what to instrument, which platform)
+
+### What Does NOT Warrant an ADR
+
+- Code style preferences (tabs vs. spaces, semicolons) — use linter configuration
+- Library utility choices (lodash vs. ramda for a single function) — too granular
+- Implementation details within a bounded context (which sorting algorithm) — unless performance-critical
+- Temporary decisions (experiment with X for a spike) — not architectural
+- Decisions that have no realistic alternatives — if there's only one viable option, just use it
+
+## ADR Structure
+
+### Title
+
+Format: `ADR-NNN: <decision statement>`
+
+The title should be a concise decision statement, not a question. It states what was decided.
+
+- Good: `ADR-003: Use PostgreSQL for persistent storage`
+- Good: `ADR-007: Adopt hexagonal architecture for core services`
+- Bad: `ADR-003: Database selection` (not a decision)
+- Bad: `ADR-007: Architecture discussion` (no decision recorded)
+
+### Status
+
+Every ADR has exactly one status:
+
+- **Proposed** — Under consideration, not yet accepted. Include in the document but flag that implementation should not proceed based on this decision.
+- **Accepted** — Active and binding. The team follows this decision.
+- **Deprecated** — No longer relevant due to changing requirements. The decision was correct when made but circumstances changed. Link to what replaced it.
+- **Superseded** — Replaced by a newer ADR. Always link to the superseding ADR: "Superseded by ADR-015."
+
+### Context
+
+Describe the forces at play. What problem are we solving? What constraints exist? What options are available?
+
+The context section answers: why is this decision necessary right now?
+
+**Good context includes:**
+
+- The specific requirement or constraint driving the decision (link to PRD section or user story)
+- Technical constraints (performance needs, team expertise, existing infrastructure)
+- Business constraints (budget, timeline, compliance requirements)
+- Assumptions being made (and their risk of being wrong)
+
+**Anti-pattern: missing context.** An ADR that says "We chose React" without explaining what drove the decision is useless. When requirements change, nobody can evaluate whether this ADR should be revisited because nobody knows what forces shaped it.
+
+### Decision
+
+State the decision clearly and directly. "We will use PostgreSQL for all persistent storage in this application."
+
+The decision should be:
+- Unambiguous — one reasonable interpretation
+- Actionable — someone can implement based on this statement
+- Scoped — clear what this decision applies to and what it doesn't
+
+### Consequences
+
+Every decision has trade-offs. Document both the benefits and the costs honestly.
+
+**Benefits:**
+
+- What does this decision enable?
+- What risks does it mitigate?
+- What becomes simpler?
+
+**Costs and risks:**
+
+- What becomes harder or more complex?
+- What options does this decision foreclose?
+- What new risks does it introduce?
+- What operational burden does it add?
+
+**Anti-pattern: consequence-free decisions.** If an ADR lists only benefits with no costs, the analysis is incomplete. Every technology and pattern choice involves trade-offs. Honest cost documentation builds trust and helps future decision-makers.
+
+### Alternatives Considered
+
+For each rejected alternative, document:
+
+- What the alternative was
+- Why it was considered (what made it a realistic option)
+- Why it was rejected (specific reasons, not "we prefer X")
+
+**Anti-pattern: straw-man alternatives.** Listing alternatives that were never seriously considered (or that are obviously wrong) undermines the ADR's credibility. Only include alternatives that were genuinely viable.
+
+## Evaluation Framework
+
+### For Technology Decisions
+
+When evaluating technology options (language, framework, database, infrastructure), assess against these criteria:
+
+**Team and AI expertise** — For AI-built projects, prioritize technologies with extensive representation in training data. Well-documented, widely-adopted tools produce fewer hallucinations and bugs. Niche or bleeding-edge tools increase error rates.
+
+**Fit with requirements** — Does the technology naturally support the features described in the PRD? A real-time collaboration app needs WebSocket support. A data-heavy analytics dashboard needs efficient query capabilities. Don't force a technology into a use case it wasn't designed for.
+
+**Community and ecosystem** — Active maintenance, frequent releases, responsive issue tracking, rich plugin/extension ecosystem. Check: when was the last release? How many open issues? Is there a single maintainer (bus factor risk)?
+
+**Integration complexity** — How well does this technology compose with the other choices in the stack? Technologies that require elaborate glue code or adapter layers add maintenance burden.
+
+**Operational complexity** — What does this technology require to run in production? PostgreSQL needs backups, monitoring, and tuning. SQLite needs none of that. Match operational complexity to team capacity.
+
+**License compatibility** — Verify the license permits your intended use (commercial, SaaS, redistribution). Watch for "source available" licenses masquerading as open source (SSPL, BSL).
+
+**Long-term maintenance** — Is this technology likely to be maintained in 3-5 years? Check: backing company/foundation, contributor diversity, adoption trajectory (growing, stable, declining).
+
+### Evaluation Matrix
+
+For significant decisions, use a weighted evaluation matrix:
+
+| Criterion | Weight | Option A | Option B | Option C |
+|-----------|--------|----------|----------|----------|
+| AI familiarity | 25% | 9 | 7 | 4 |
+| Fit with requirements | 25% | 8 | 9 | 7 |
+| Community/ecosystem | 15% | 9 | 8 | 6 |
+| Integration complexity | 15% | 7 | 8 | 5 |
+| Operational complexity | 10% | 6 | 8 | 9 |
+| Long-term maintenance | 10% | 8 | 7 | 5 |
+| **Weighted total** | | **8.05** | **7.90** | **5.80** |
+
+Weights should reflect project priorities. An AI-built project weights AI familiarity higher. A startup weights speed-to-market higher. An enterprise weights long-term maintenance higher.
+
+### For Architectural Decisions
+
+Evaluate architectural patterns against:
+
+- **Complexity vs. value** — Does the added complexity deliver proportional value? Microservices for a CRUD app with one developer is over-engineering. A monolith for a system with clearly independent scaling needs is under-engineering.
+- **Reversibility** — How expensive is it to change this decision later? Prefer reversible decisions (can switch ORMs in a week) over irreversible ones (entire system depends on event sourcing). Invest more analysis time in irreversible decisions.
+- **Alignment with domain model** — Does the architecture naturally express the domain? A modular monolith with modules matching bounded contexts aligns well. A layered architecture that spreads domain concepts across layers does not.
+- **Operational readiness** — Can the team (or AI agents) actually operate this in production? A Kubernetes-based microservices deployment requires expertise that a single-developer team may lack.
+
+## ADR Lifecycle
+
+### When to Create
+
+- Create ADRs before implementation, not after. The ADR guides the implementation, not the other way around.
+- For technology decisions: create during the tech stack selection phase, before any code is written.
+- For architectural decisions: create during the architecture phase, after domain modeling and before implementation planning.
+- For emergent decisions: when implementation reveals that an assumption was wrong, create an ADR documenting the pivot before changing course.
+
+### When to Supersede
+
+- When a decision is reversed (different technology chosen, different pattern adopted), create a new ADR and mark the old one as superseded.
+- The superseding ADR must reference the original and explain what changed to invalidate the original decision.
+- Never delete a superseded ADR. The history of decisions is valuable context for understanding why things are the way they are.
+
+### When to Deprecate
+
+- When a decision becomes irrelevant (the feature it relates to was removed, the technology it selected was swapped out as part of a broader change), mark it as deprecated.
+- Link to the change that made it irrelevant.
+
+### Cross-Referencing
+
+- Related ADRs should reference each other. "See also ADR-005 for the database migration strategy that supports this caching decision."
+- ADRs should reference the PRD sections or requirements that motivated them.
+- Implementation tasks should reference the ADRs that guide them. "Implements ADR-003: PostgreSQL persistent storage."
+
+### Recording Implicit Decisions
+
+Many architectural decisions are made implicitly — someone starts using a library, a pattern emerges from code review, a deployment approach is chosen during a sprint. These are still architectural decisions.
+
+Periodically audit the codebase for implicit decisions:
+- Dependencies added to package.json/requirements.txt without documented rationale
+- Architectural patterns used consistently without a recorded decision
+- Infrastructure configurations that embody decisions (environment variables, Docker configurations)
+- Test patterns that imply testing strategy decisions
+
+Create retroactive ADRs for implicit decisions that meet the significance threshold.
+
+## Technology Selection as ADRs
+
+Technology selection is one of the most impactful categories of architectural decisions. The v1 scaffold pipeline treated tech stack as a separate document; in the current architecture, each technology choice is an ADR.
+
+### Structure for Tech Stack ADRs
+
+Each technology ADR should cover:
+
+**What the document must capture:**
+- The specific library/framework and version
+- Rationale tied to project requirements (not generic praise for the technology)
+- Why alternatives were rejected (specific technical reasons)
+- AI compatibility assessment: how well-represented in training data, known AI pitfalls
+
+**Guiding principles for AI-built projects:**
+- Convention over configuration — opinionated frameworks with clear patterns reduce AI errors
+- Minimal dependency surface — fewer dependencies mean fewer version conflicts and security risks
+- Strong typing and validation — static types catch AI mistakes at build time
+- Mature ecosystem — stable, well-documented libraries with active maintenance
+
+### Grouping Related Decisions
+
+Technology decisions often cluster. Rather than 20 individual ADRs for each package, group related decisions:
+
+- "Backend technology stack" (language + framework + ORM + auth library)
+- "Frontend technology stack" (framework + state management + styling + build tool)
+- "Database and data access" (engine + migration tool + caching layer)
+- "Infrastructure and deployment" (hosting + CI/CD + monitoring)
+- "Developer tooling" (linter + formatter + test runner + pre-commit hooks)
+
+Each group ADR can cover multiple decisions if they are tightly coupled and were evaluated together.
+
+## Common Pitfalls
+
+**Recording decisions without alternatives.** An ADR that says "We'll use React" without mentioning Vue or Svelte provides no insight into the decision process. When a new team member asks "why not Vue?", there's no answer. Fix: always document at least one alternative and why it was rejected.
+
+**Missing rationale.** "We chose PostgreSQL" is a fact, not a decision record. The rationale — "because we need JSONB for flexible metadata, strong ACID compliance for financial data, and it's the most AI-familiar relational database" — is the valuable part. Fix: the context and consequences sections should be longer than the decision section.
+
+**Contradicting other ADRs without acknowledgment.** ADR-003 says "minimize dependencies" but ADR-008 adds a heavy utility library. This isn't necessarily wrong — trade-offs are legitimate — but the contradiction must be acknowledged in ADR-008. Fix: cross-reference related ADRs and explicitly address conflicts.
+
+**Failing to record implicit decisions.** The codebase uses a specific error handling pattern everywhere, but no ADR documents why. When a new agent builds a feature, it invents a different error handling pattern. Fix: audit for implicit decisions periodically.
+
+**Technology hype bias.** Choosing a technology because it's trending rather than because it fits the requirements. "Let's use Rust for the API because it's fast" when the bottleneck is database queries, not CPU-bound computation. Fix: evaluate against project-specific criteria, not general popularity.
+
+**Premature decisions.** Recording a decision before gathering sufficient context. Choosing a database engine before understanding the data model leads to decisions that constrain the design unnecessarily. Fix: defer decisions until the last responsible moment — when further delay would reduce options.
+
+**Stale ADRs.** Accepted ADRs that no longer reflect reality because the codebase has evolved. The ADR says "modular monolith" but the system has been split into services. Fix: review ADRs periodically and update status (supersede, deprecate) as needed.
+
+## Quality Indicators
+
+An ADR set is likely complete when:
+
+- **Every technology in the dependency manifest has a corresponding ADR.** No mystery dependencies.
+- **Every ADR is traceable to a requirement or constraint.** No decisions exist in a vacuum.
+- **Alternatives are genuinely considered.** Rejected alternatives have specific, defensible rejection reasons — not "we prefer X."
+- **Consequences are honest.** Every ADR lists at least one downside or risk. A consequence-free ADR is incomplete.
+- **No contradictions between ADRs.** Or, if contradictions exist, they are acknowledged and explained.
+- **The ADR set covers all layers.** Backend, frontend (if applicable), database, infrastructure, developer tooling, deployment, testing, and security all have decisions recorded.
+- **Status is current.** No accepted ADRs that actually describe how things used to work. Superseded and deprecated ADRs are marked.
+- **Cross-references are complete.** Related ADRs link to each other. ADRs link to the requirements they address.