clckernel 1.2.6 → 1.2.8

package/test/integration/cli-flow.test.js

@@ -1,270 +0,0 @@
-const { describe, it } = require('node:test');
-const assert = require('node:assert');
-const path = require('path');
-const fs = require('fs');
-const { execSync } = require('child_process');
-const { createTmpProject, teardown } = require('../helpers');
-
-const CLI_PATH = path.join(__dirname, '..', '..', 'bin', 'cli.js');
-
-describe('Integration: Full CLI Flow', () => {
-  describe('runCli() doctor command', () => {
-    it('runs doctor and exits 0 on healthy project', () => {
-      const dir = createTmpProject({
-        'AGENTS.md': '# AGENTS\n',
-        'sdds/.gitkeep': '',
-        'docs/Journal/001-adr.md': '# ADR\n',
-        'tools/audit.js': 'module.exports = {};\n',
-        'tools/check_scan.js': 'module.exports = function() {};\n',
-        '.husky/pre-commit': '#!/usr/bin/env sh\necho test\n',
-      });
-      try {
-        const result = execSync(`node "${CLI_PATH}" doctor "${dir}"`, {
-          encoding: 'utf-8',
-          timeout: 15000,
-        });
-        assert.ok(result.includes('HEALTHY'), 'doctor should report HEALTHY');
-        assert.ok(result.includes('CLC KERNEL DOCTOR') || result.includes('CLC FORGE DOCTOR'), 'should show doctor header');
-        assert.ok(result.includes('[PASS]'), 'should show PASS checks');
-      } finally {
-        teardown(dir);
-      }
-    });
-
-    it('runs doctor and reports issues on unhealthy project', () => {
-      const dir = createTmpProject({
-        'package.json': '{}',
-      });
-      try {
-        const result = execSync(`node "${CLI_PATH}" doctor "${dir}"`, {
-          encoding: 'utf-8',
-          timeout: 15000,
-        });
-        assert.ok(result.includes('[FAIL]'), 'doctor should report FAIL checks');
-        assert.ok(!result.includes('HEALTHY'), 'should not be HEALTHY');
-        assert.ok(result.includes('issue(s) detected'), 'should report issue count');
-      } finally {
-        teardown(dir);
-      }
-    });
-
-    it('resolves target directory from second argument', () => {
-      const dir = createTmpProject({
-        'AGENTS.md': '# AGENTS\n',
-        'sdds/.gitkeep': '',
-        'docs/Journal/001-adr.md': '# ADR\n',
-        'tools/check_scan.js': 'module.exports = function() {};\n',
-        '.husky/pre-commit': '#!/usr/bin/env sh\necho test\n',
-      });
-      try {
-        const result = execSync(`node "${CLI_PATH}" doctor "${dir}"`, {
-          encoding: 'utf-8',
-          timeout: 15000,
-        });
-        // The output should show the target directory path
-        assert.ok(result.includes(dir), 'output should show target directory');
-      } finally {
-        teardown(dir);
-      }
-    });
-
-    it('uses cwd when no directory argument is provided', () => {
-      // Create a healthy project in a temp dir and run from there
-      const dir = createTmpProject({
-        'AGENTS.md': '# AGENTS\n',
-        'sdds/.gitkeep': '',
-        'docs/Journal/001-adr.md': '# ADR\n',
-        'tools/check_scan.js': 'module.exports = function() {};\n',
-        '.husky/pre-commit': '#!/usr/bin/env sh\necho test\n',
-      });
-      try {
-        const result = execSync(`node "${CLI_PATH}" doctor`, {
-          encoding: 'utf-8',
-          timeout: 15000,
-          cwd: dir,
-        });
-        assert.ok(result.includes('HEALTHY'), 'doctor should use cwd');
-        assert.ok(result.includes(dir), 'should show cwd path');
-      } finally {
-        teardown(dir);
-      }
-    });
-  });
-
-  describe('runCli() default flow (non-doctor)', () => {
-    it('resolves target directory from first argument', () => {
-      const dir = createTmpProject({
-        'package.json': JSON.stringify({ dependencies: { next: '14.0.0' } }),
-      });
-      try {
-        // The default flow will try to printBanner → autoDetectStack → promptOptions
-        // promptOptions reads stdin so it will hang — use timeout and check output
-        try {
-          execSync(`node "${CLI_PATH}" "${dir}"`, {
-            encoding: 'utf-8',
-            timeout: 3000,
-            input: '1\n',
-          });
-        } catch (e) {
-          // May exit non-zero due to prompt timeout, but we check the output
-          const output = e.stdout || e.stderr || '';
-          // Should at least detect the stack (Next.js)
-          assert.ok(output.length > 0, 'should produce output');
-        }
-      } finally {
-        teardown(dir);
-      }
-    });
-  });
-
-  describe('runCli() with --doctor flag', () => {
-    it('accepts --doctor flag as alternative to doctor command', () => {
-      const dir = createTmpProject({
-        'AGENTS.md': '# AGENTS\n',
-        'sdds/.gitkeep': '',
-        'docs/Journal/001-adr.md': '# ADR\n',
-        'tools/check_scan.js': 'module.exports = function() {};\n',
-        '.husky/pre-commit': '#!/usr/bin/env sh\necho test\n',
-      });
-      try {
-        const result = execSync(`node "${CLI_PATH}" --doctor "${dir}"`, {
-          encoding: 'utf-8',
-          timeout: 15000,
-        });
-        assert.ok(result.includes('HEALTHY'), '--doctor flag should run doctor');
-        assert.ok(result.includes('CLC KERNEL DOCTOR') || result.includes('CLC FORGE DOCTOR'), 'should show doctor header');
-      } finally {
-        teardown(dir);
-      }
-    });
-  });
-
-  describe('doctor detects all hook types', () => {
-    it('detects .githooks/pre-commit via CLI', () => {
-      const dir = createTmpProject({
-        'AGENTS.md': '# AGENTS\n',
-        'sdds/.gitkeep': '',
-        'docs/Journal/001-adr.md': '# ADR\n',
-        'tools/check_lint.py': 'def check():\n    return True\n',
-        '.githooks/pre-commit': '#!/usr/bin/env bash\necho test\n',
-      });
-      try {
-        const result = execSync(`node "${CLI_PATH}" doctor "${dir}"`, {
-          encoding: 'utf-8',
-          timeout: 15000,
-        });
-        // .githooks should satisfy the pre-commit check
-        assert.ok(result.includes('[PASS]') && result.includes('pre-commit git hooks'), '.githooks should PASS');
-      } finally {
-        teardown(dir);
-      }
-    });
-
-    it('validates Python tool parsing via CLI', () => {
-      const dir = createTmpProject({
-        'AGENTS.md': '# AGENTS\n',
-        'sdds/.gitkeep': '',
-        'docs/Journal/001-adr.md': '# ADR\n',
-        'tools/check_syntax.py': 'x = 1\n',
-        '.husky/pre-commit': '#!/usr/bin/env sh\necho test\n',
-      });
-      try {
-        const result = execSync(`node "${CLI_PATH}" doctor "${dir}"`, {
-          encoding: 'utf-8',
-          timeout: 15000,
-        });
-        assert.ok(result.includes('tool/check_syntax.py parses'), 'should validate Python tool');
-        assert.ok(result.includes('[PASS]') && result.includes('check_syntax.py parses'), 'valid Python should PASS');
-      } finally {
-        teardown(dir);
-      }
-    });
-  });
-});
-
-describe('Integration: Component Composition', () => {
-  it('autoDetectStack + generateHarness creates valid project structure', () => {
-    const { autoDetectStack } = require('../../src/detector');
-    const { generateHarness } = require('../../src/generator');
-
-    const dir = createTmpProject({
-      'package.json': JSON.stringify({ dependencies: { next: '14.0.0' } }),
-    });
-    try {
-      const config = autoDetectStack(dir);
-      assert.strictEqual(config.framework, 'Next.js (App Router)', 'should detect Next.js');
-
-      generateHarness(dir, config);
-
-      // Verify all expected artifacts
-      assert.ok(fs.existsSync(path.join(dir, 'AGENTS.md')), 'AGENTS.md should exist');
-      assert.ok(fs.existsSync(path.join(dir, 'sdds')), 'sdds/ should exist');
-      assert.ok(fs.existsSync(path.join(dir, 'docs', 'Journal')), 'docs/Journal/ should exist');
-      assert.ok(fs.existsSync(path.join(dir, 'tools')), 'tools/ should exist');
-      assert.ok(fs.existsSync(path.join(dir, '.husky', 'pre-commit')), '.husky/pre-commit should exist');
-
-      // Verify AGENTS.md content
-      const agents = fs.readFileSync(path.join(dir, 'AGENTS.md'), 'utf-8');
-      assert.ok(agents.includes('Next.js'), 'AGENTS.md should mention Next.js');
-      assert.ok(agents.includes('FRONTEND'), 'AGENTS.md should mention FRONTEND');
-    } finally {
-      teardown(dir);
-    }
-  });
-
-  it('autoDetectStack fallback + generateHarness creates backend structure', () => {
-    const { autoDetectStack } = require('../../src/detector');
-    const { generateHarness } = require('../../src/generator');
-
-    const dir = createTmpProject({
-      'go.mod': 'module example.com\n\ngo 1.21\n',
-    });
-    try {
-      const config = autoDetectStack(dir);
-      assert.strictEqual(config.framework, 'Go (Golang)', 'should detect Go');
-
-      generateHarness(dir, config);
-
-      assert.ok(fs.existsSync(path.join(dir, 'AGENTS.md')), 'AGENTS.md should exist');
-      assert.ok(fs.existsSync(path.join(dir, 'sdds')), 'sdds/ should exist');
-      assert.ok(fs.existsSync(path.join(dir, '.githooks', 'pre-commit')), '.githooks/pre-commit should exist');
-
-      const agents = fs.readFileSync(path.join(dir, 'AGENTS.md'), 'utf-8');
-      assert.ok(agents.includes('Go'), 'AGENTS.md should mention Go');
-      assert.ok(agents.includes('BACKEND'), 'AGENTS.md should mention BACKEND');
-    } finally {
-      teardown(dir);
-    }
-  });
-
-  it('detect → generate → doctor reports HEALTHY', () => {
-    const { autoDetectStack } = require('../../src/detector');
-    const { generateHarness } = require('../../src/generator');
-    const { runDoctor } = require('../../src/doctor');
-
-    const dir = createTmpProject({
-      'package.json': JSON.stringify({ dependencies: { next: '14.0.0' } }),
-    });
-    try {
-      // Step 1: Detect
-      const config = autoDetectStack(dir);
-      assert.strictEqual(config.framework, 'Next.js (App Router)');
-
-      // Step 2: Generate
-      generateHarness(dir, config);
-
-      // Step 3: Doctor should report HEALTHY
-      let capturedOutput = '';
-      const originalLog = console.log;
-      console.log = (...args) => { capturedOutput += args.join(' ') + '\n'; };
-      try {
-        runDoctor(dir);
-        assert.ok(capturedOutput.includes('HEALTHY'), 'after detect+generate, doctor should be HEALTHY');
-      } finally {
-        console.log = originalLog;
-      }
-    } finally {
-      teardown(dir);
-    }
-  });
-});

package/test/technologies/celery_guard.test.js

@@ -1,185 +0,0 @@
-/**
- * CLC Forge — Celery guard integration tests.
- * Tests celery_guard.py via subprocess against fixture projects.
- */
-
-const { describe, it } = require('node:test');
-const assert = require('node:assert');
-const { spawn, spawnSync } = require('child_process');
-const path = require('path');
-const fs = require('fs');
-const os = require('os');
-const { createTmpProject, teardown } = require('../helpers');
-
-/**
- * Run celery_guard.py against a target directory.
- * Returns a promise resolving to the parsed JSON result.
- */
-function runCeleryGuard(targetDir) {
-  return new Promise((resolve, reject) => {
-    const script = path.join(__dirname, '../../tools/celery_guard.py');
-    const proc = spawn('python3', [script, targetDir], { cwd: targetDir });
-    let stdout = '';
-    let stderr = '';
-    proc.stdout.on('data', (d) => (stdout += d));
-    proc.stderr.on('data', (d) => (stderr += d));
-    proc.on('close', (code) => {
-      try {
-        const result = JSON.parse(stdout);
-        resolve(result);
-      } catch {
-        reject(new Error(`Failed to parse guard output: ${stderr || stdout}`));
-      }
-    });
-    proc.on('error', reject);
-  });
-}
-
-/**
- * Create a git repo in dir and commit an initial empty file.
- */
-function initGitRepo(dir) {
-    fs.writeFileSync(path.join(dir, '.gitkeep'), '');
-    spawnSync('git', ['init'], { cwd: dir });
-    spawnSync('git', ['config', 'user.email', 'test@test.com'], { cwd: dir });
-    spawnSync('git', ['config', 'user.name', 'Test'], { cwd: dir });
-    spawnSync('git', ['add', '.'], { cwd: dir });
-    spawnSync('git', ['commit', '-m', 'initial'], { cwd: dir });
-}
-
-describe('celery_guard', () => {
-  it('skips when no git diff is available (no git repo)', async () => {
-    const dir = createTmpProject({});
-    try {
-      const result = await runCeleryGuard(dir);
-      assert.strictEqual(result.skipped, true, 'should be skipped');
-      assert.strictEqual(result.name, 'celery_guard');
-    } finally {
-      teardown(dir);
-    }
-  });
-
-  it('finds violations: hardcoded broker_url in Celery()', async () => {
-    const dir = createTmpProject({});
-    initGitRepo(dir);
-
-    // Create a new file with a violation
-    fs.writeFileSync(
-      path.join(dir, 'tasks.py'),
-      `from celery import Celery\napp = Celery("myapp", broker_url="redis://localhost")\n`
-    );
-    spawnSync('git', ['add', 'tasks.py'], { cwd: dir });
-
-    try {
-      const result = await runCeleryGuard(dir);
-      assert.strictEqual(result.exit_code, 1, 'should exit with error code');
-      assert.strictEqual(result.skipped, false);
-      assert.ok(result.data.findings.length > 0, 'should have findings');
-      const issues = result.data.findings.map((f) => f.issue);
-      assert.ok(issues.includes('hardcoded_broker_url'), 'should detect hardcoded broker_url');
-    } finally {
-      teardown(dir);
-    }
-  });
-
-  it('finds violations: hardcoded result_backend', async () => {
-    const dir = createTmpProject({});
-    initGitRepo(dir);
-
-    fs.writeFileSync(
-      path.join(dir, 'tasks.py'),
-      `from celery import Celery\napp = Celery(broker_url="redis://localhost", result_backend="redis://localhost")\n`
-    );
-    spawnSync('git', ['add', 'tasks.py'], { cwd: dir });
-
-    try {
-      const result = await runCeleryGuard(dir);
-      assert.strictEqual(result.exit_code, 1);
-      const issues = result.data.findings.map((f) => f.issue);
-      assert.ok(issues.includes('hardcoded_result_backend'), 'should detect hardcoded result_backend');
-    } finally {
-      teardown(dir);
-    }
-  });
-
-  it('finds violations: Celery with both broker_url and result_backend', async () => {
-    const dir = createTmpProject({});
-    initGitRepo(dir);
-
-    fs.writeFileSync(
-      path.join(dir, 'tasks.py'),
-      `from celery import Celery\napp = Celery("myapp", broker_url="redis://localhost", result_backend="redis://localhost")\n`
-    );
-    spawnSync('git', ['add', 'tasks.py'], { cwd: dir });
-
-    try {
-      const result = await runCeleryGuard(dir);
-      assert.strictEqual(result.exit_code, 1);
-      const issues = result.data.findings.map((f) => f.issue);
-      assert.ok(issues.includes('hardcoded_broker_url'), 'should detect hardcoded broker_url');
-      assert.ok(issues.includes('hardcoded_result_backend'), 'should detect hardcoded result_backend');
-      assert.strictEqual(result.data.findings.length, 2, 'should find both violations');
-    } finally {
-      teardown(dir);
-    }
-  });
-
-  it('no violations: properly configured @shared_task with ignore_result=True', async () => {
-    const dir = createTmpProject({});
-    initGitRepo(dir);
-
-    fs.writeFileSync(
-      path.join(dir, 'tasks.py'),
-      `from celery import shared_task\n@shared_task(ignore_result=True)\ndef add(x, y):\n    return x + y\n`
-    );
-    spawnSync('git', ['add', 'tasks.py'], { cwd: dir });
-
-    try {
-      const result = await runCeleryGuard(dir);
-      assert.strictEqual(result.exit_code, 0, 'should exit cleanly');
-      assert.strictEqual(result.skipped, false);
-      assert.strictEqual(result.data.findings.length, 0, 'should have no findings');
-    } finally {
-      teardown(dir);
-    }
-  });
-
-  it('no violations: properly configured Celery without hardcoded URLs', async () => {
-    const dir = createTmpProject({});
-    initGitRepo(dir);
-
-    fs.writeFileSync(
-      path.join(dir, 'tasks.py'),
-      `import os\nfrom celery import Celery\napp = Celery("myapp")\napp.conf.broker_url = os.environ.get("BROKER_URL")\napp.conf.result_backend = os.environ.get("RESULT_BACKEND")\n`
-    );
-    spawnSync('git', ['add', 'tasks.py'], { cwd: dir });
-
-    try {
-      const result = await runCeleryGuard(dir);
-      assert.strictEqual(result.exit_code, 0, 'should exit cleanly');
-      assert.strictEqual(result.data.findings.length, 0, 'should have no findings');
-    } finally {
-      teardown(dir);
-    }
-  });
-
-  it('no violations: no celery in project (empty findings)', async () => {
-    const dir = createTmpProject({});
-    initGitRepo(dir);
-
-    fs.writeFileSync(
-      path.join(dir, 'main.py'),
-      `def hello():\n    return "world"\n`
-    );
-    spawnSync('git', ['add', 'main.py'], { cwd: dir });
-
-    try {
-      const result = await runCeleryGuard(dir);
-      assert.strictEqual(result.exit_code, 0, 'should exit cleanly');
-      assert.strictEqual(result.skipped, false);
-      assert.strictEqual(result.data.findings.length, 0, 'should have no findings');
-    } finally {
-      teardown(dir);
-    }
-  });
-});