@haposoft/cafekit 0.18.2 → 0.18.4

package/bin/__tests__/codex-specs-projection.test.js

@@ -1,76 +0,0 @@
-'use strict';
-
-const assert = require('node:assert/strict');
-const fs = require('node:fs');
-const path = require('node:path');
-const { test } = require('node:test');
-const { normalizeCodexBody, CODEX_SPECS_SCOPE_MARKER, CODEX_SPECS_SCOPE_NOTE,
-  CODEX_RESEARCH_DEPTH_MARKER } = require('../lib/codex-install');
-const { splitFrontmatter } = require('../lib/codex-frontmatter');
-
-const SKILLS = path.resolve(__dirname, '../../src/claude/skills');
-const specsPath = path.join(SKILLS, 'specs/SKILL.md');
-const specs = fs.readFileSync(specsPath, 'utf8');
-
-function skillFiles(directory) {
-  return fs.readdirSync(directory, { withFileTypes: true }).flatMap(entry => {
-    const file = path.join(directory, entry.name);
-    return entry.isDirectory() ? skillFiles(file) : entry.name === 'SKILL.md' ? [file] : [];
-  });
-}
-
-function assertMarker() {
-  assert.equal(typeof CODEX_SPECS_SCOPE_MARKER, 'string');
-  assert.ok(CODEX_SPECS_SCOPE_MARKER.length > 0);
-}
-
-test('codex specs projection carries the scope note immediately after unchanged frontmatter', () => {
-  assertMarker();
-  assert.equal(typeof CODEX_SPECS_SCOPE_NOTE, 'string');
-  const plain = normalizeCodexBody(specs);
-  const projected = normalizeCodexBody(specs, specsPath);
-  assert.match(projected, /^---\r?\n/);
-  const before = splitFrontmatter(plain), after = splitFrontmatter(projected);
-  assert.equal(after.frontmatter.name, 'cf-specs');
-  assert.deepEqual(after.frontmatter, before.frontmatter);
-  assert.equal(projected.slice(0, projected.length - after.body.length), plain.slice(0, plain.length - before.body.length));
-  assert.equal(after.body, `\n${CODEX_SPECS_SCOPE_NOTE}\n\n${before.body}`);
-  assert.equal(projected.split(CODEX_SPECS_SCOPE_MARKER).length - 1, 1);
-  assert.equal(normalizeCodexBody(projected, specsPath), projected);
-  assert.ok(CODEX_SPECS_SCOPE_NOTE.includes(CODEX_SPECS_SCOPE_MARKER));
-  assert.match(CODEX_SPECS_SCOPE_MARKER, /GATE-SCOPE.*final answer/);
-  assert.match(CODEX_SPECS_SCOPE_NOTE, /not in commentary or through `request_user_input`/);
-  assert.match(CODEX_SPECS_SCOPE_NOTE, /Do not send a summary after/);
-  assert.match(CODEX_SPECS_SCOPE_NOTE, /\[NEEDS CLARIFICATION: <question>\]/);
-  assert.match(CODEX_SPECS_SCOPE_NOTE, /real question.*not a placeholder/);
-  assert.match(CODEX_SPECS_SCOPE_NOTE, /EXPAND, KEEP, and CUT together/);
-  assert.match(CODEX_SPECS_SCOPE_NOTE, /Preserve.*NEEDS CLARIFICATION.*EXPAND.*KEEP.*CUT.*do not translate/);
-  assert.doesNotMatch(CODEX_SPECS_SCOPE_NOTE, /\b(?:Write|Edit|Read|Bash|WebSearch|WebFetch|AskUserQuestion)\b/);
-  assert.doesNotMatch(CODEX_SPECS_SCOPE_NOTE, /customer|sales|notif|khách|bán hàng|thông báo/i);
-});
-
-test('specs marker is absent from every other skill and research retains exactly one depth marker', () => {
-  assertMarker();
-  const others = skillFiles(SKILLS).filter(file => file !== specsPath);
-  for (const name of ['ask', 'research']) assert.ok(others.includes(path.join(SKILLS, name, 'SKILL.md')));
-  for (const file of others) {
-    const source = fs.readFileSync(file, 'utf8');
-    const projected = normalizeCodexBody(source, file);
-    assert.ok(!projected.includes(CODEX_SPECS_SCOPE_MARKER), file);
-    if (file === path.join(SKILLS, 'research/SKILL.md')) {
-      assert.equal(projected.split(CODEX_RESEARCH_DEPTH_MARKER).length - 1, 1);
-    } else if (file !== path.join(SKILLS, 'ask/SKILL.md')) {
-      assert.equal(projected, normalizeCodexBody(source), file);
-    }
-  }
-});
-
-test('specs note is scoped to the exact skill source path and supports Windows separators', () => {
-  assertMarker();
-  for (const sourcePath of ['', 'skills/specs/references/templates.md', 'other-skills/specs/SKILL.md']) {
-    const projected = normalizeCodexBody(specs, sourcePath);
-    assert.ok(!projected.includes(CODEX_SPECS_SCOPE_MARKER), sourcePath);
-    assert.equal(projected, normalizeCodexBody(specs), sourcePath);
-  }
-  assert.ok(normalizeCodexBody(specs, 'C:\\project\\skills\\specs\\SKILL.md').includes(CODEX_SPECS_SCOPE_MARKER));
-});

package/bin/__tests__/specs-v2-execution-closeout.test.js

@@ -1,58 +0,0 @@
-'use strict';
-
-const assert = require('node:assert/strict');
-const crypto = require('node:crypto');
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-const test = require('node:test');
-
-const TEMP_RUN_ID = String(process.env.CAFEKIT_CLOSEOUT_TEMP_RUN_ID || `${process.pid}-${Date.now()}`)
-  .replace(/[^a-zA-Z0-9_-]/g, '-');
-const TEMP_PREFIX = `cafekit-v2-closeout-owned-${TEMP_RUN_ID}`;
-
-function withTempResources(callback) {
-  const ownedPaths = [];
-  const originalHome = Object.hasOwn(process.env, 'HOME')
-    ? { present: true, value: process.env.HOME }
-    : { present: false };
-  function ownedDirectory(label) {
-    const directory = fs.mkdtempSync(path.join(os.tmpdir(), `${TEMP_PREFIX}-${label}-`));
-    ownedPaths.push(directory);
-    return directory;
-  }
-  const resources = {
-    directory: ownedDirectory,
-    useHome(label) {
-      const directory = ownedDirectory(label);
-      process.env.HOME = directory;
-      return directory;
-    },
-  };
-  try {
-    return callback(resources);
-  } finally {
-    if (originalHome.present) process.env.HOME = originalHome.value;
-    else delete process.env.HOME;
-    for (const ownedPath of ownedPaths.reverse()) {
-      fs.rmSync(ownedPath, { recursive: true, force: true });
-    }
-  }
-}
-
-test('temp resource scope cleans exact owned paths and restores HOME after failure', () => {
-  const originalHome = Object.hasOwn(process.env, 'HOME')
-    ? { present: true, value: process.env.HOME }
-    : { present: false };
-  let root;
-  let home;
-  assert.throws(() => withTempResources((resources) => {
-    root = resources.directory('failure-root');
-    home = resources.useHome('failure-home');
-    throw new Error('intentional resource-scope failure');
-  }), /intentional resource-scope failure/);
-  assert.equal(fs.existsSync(root), false);
-  assert.equal(fs.existsSync(home), false);
-  assert.equal(Object.hasOwn(process.env, 'HOME'), originalHome.present);
-  if (originalHome.present) assert.equal(process.env.HOME, originalHome.value);
-});

package/src/claude/skills/chrome-devtools/scripts/__tests__/error-handling.test.js

@@ -1,102 +0,0 @@
-/**
- * Tests for error handling in chrome-devtools scripts
- * Verifies scripts exit with code 1 on errors
- * Run with: node --test __tests__/error-handling.test.js
- *
- * Note: These tests verify exit code behavior. When puppeteer is not installed,
- * scripts still exit with code 1 (module not found), which validates the error path.
- * When puppeteer IS installed, missing --url triggers application-level error with code 1.
- */
-import { describe, it } from 'node:test';
-import assert from 'node:assert';
-import { spawn } from 'node:child_process';
-import { fileURLToPath } from 'node:url';
-import path from 'node:path';
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const scriptsDir = path.join(__dirname, '..');
-
-function runScript(script, args = [], timeout = 10000) {
-  return new Promise((resolve) => {
-    const proc = spawn('node', [path.join(scriptsDir, script), ...args], {
-      timeout,
-      stdio: ['pipe', 'pipe', 'pipe']
-    });
-
-    let stdout = '';
-    let stderr = '';
-
-    proc.stdout.on('data', (data) => { stdout += data; });
-    proc.stderr.on('data', (data) => { stderr += data; });
-
-    proc.on('close', (code) => {
-      resolve({ code, stdout, stderr, combined: stdout + stderr });
-    });
-
-    proc.on('error', (err) => {
-      resolve({ code: 1, stdout, stderr: err.message, combined: err.message });
-    });
-
-    setTimeout(() => {
-      proc.kill('SIGTERM');
-      resolve({ code: null, stdout, stderr, timedOut: true, combined: stdout + stderr });
-    }, timeout);
-  });
-}
-
-describe('chrome-devtools error handling', () => {
-  describe('console.js', () => {
-    it('should exit with code 1 when --url is missing or on error', async () => {
-      const result = await runScript('console.js', []);
-      assert.strictEqual(result.code, 1, 'Expected exit code 1');
-    });
-
-    it('should output error information', async () => {
-      const result = await runScript('console.js', []);
-      assert.strictEqual(result.code, 1);
-      // Either app-level error (--url required) or module error (puppeteer not found)
-      const hasError = result.combined.toLowerCase().includes('error') ||
-                       result.combined.includes('--url');
-      assert.ok(hasError, 'Expected error in output');
-    });
-  });
-
-  describe('evaluate.js', () => {
-    it('should exit with code 1 when --url is missing or on error', async () => {
-      const result = await runScript('evaluate.js', []);
-      assert.strictEqual(result.code, 1, 'Expected exit code 1');
-    });
-  });
-
-  describe('navigate.js', () => {
-    it('should exit with code 1 when --url is missing or on error', async () => {
-      const result = await runScript('navigate.js', []);
-      assert.strictEqual(result.code, 1, 'Expected exit code 1');
-    });
-  });
-
-  describe('network.js', () => {
-    it('should exit with code 1 when --url is missing or on error', async () => {
-      const result = await runScript('network.js', []);
-      assert.strictEqual(result.code, 1, 'Expected exit code 1');
-    });
-  });
-
-  describe('performance.js', () => {
-    it('should exit with code 1 when --url is missing or on error', async () => {
-      const result = await runScript('performance.js', []);
-      assert.strictEqual(result.code, 1, 'Expected exit code 1');
-    });
-  });
-
-  describe('all scripts exit code consistency', () => {
-    const scripts = ['console.js', 'evaluate.js', 'navigate.js', 'network.js', 'performance.js'];
-
-    for (const script of scripts) {
-      it(`${script} should exit 1 on invalid input or error`, async () => {
-        const result = await runScript(script, []);
-        assert.strictEqual(result.code, 1, `${script} should exit with code 1`);
-      });
-    }
-  });
-});