@codename_inc/spectre 3.7.0 → 4.0.0

package/src/pack.test.js

@@ -0,0 +1,87 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import fs from 'fs';
+import os from 'os';
+import path from 'path';
+import { execFileSync } from 'child_process';
+
+function makeTempDir(prefix) {
+  return fs.mkdtempSync(path.join(os.tmpdir(), prefix));
+}
+
+function exec(command, args, options = {}) {
+  return execFileSync(command, args, {
+    encoding: 'utf8',
+    stdio: ['ignore', 'pipe', 'pipe'],
+    ...options
+  });
+}
+
+function collectFiles(root) {
+  const files = [];
+  if (!fs.existsSync(root)) return files;
+
+  function walk(dir) {
+    for (const entry of fs.readdirSync(dir, { withFileTypes: true })) {
+      const fullPath = path.join(dir, entry.name);
+      if (entry.isDirectory()) {
+        walk(fullPath);
+      } else if (entry.isFile()) {
+        files.push(fullPath);
+      }
+    }
+  }
+
+  walk(root);
+  return files;
+}
+
+test('packed npm artifact installs Codex assets from generated tree', { concurrency: false }, () => {
+  const repoRoot = path.resolve('.');
+  const packDir = makeTempDir('spectre-pack-');
+  const projectDir = makeTempDir('spectre-pack-install-');
+
+  try {
+    const packOutput = exec('npm', ['pack', '--pack-destination', packDir], { cwd: repoRoot }).trim();
+    const tarball = path.join(packDir, packOutput.split('\n').at(-1));
+
+    exec('npm', ['init', '-y'], { cwd: projectDir });
+    exec('npm', ['install', '--ignore-scripts', '--no-audit', '--no-fund', tarball], { cwd: projectDir });
+    exec('git', ['init', '-b', 'main'], { cwd: projectDir });
+
+    const env = { ...process.env };
+    delete env.CODEX_HOME;
+
+    const unscopedHelp = exec('npx', ['spectre', 'help'], {
+      cwd: projectDir,
+      env
+    });
+    assert.match(unscopedHelp, /spectre install codex/);
+
+    exec('npx', ['@codename_inc/spectre', 'install', 'codex', '--scope', 'project', '--project-dir', projectDir], {
+      cwd: projectDir,
+      env
+    });
+
+    const codexHome = path.join(projectDir, '.codex');
+    assert.ok(fs.existsSync(path.join(codexHome, 'skills', 'plan', 'SKILL.md')));
+    assert.ok(fs.existsSync(path.join(codexHome, 'spectre', 'agents', 'dev.toml')));
+    assert.ok(fs.existsSync(path.join(codexHome, 'spectre', 'hooks', 'scripts', 'load-knowledge.mjs')));
+    assert.ok(!fs.existsSync(path.join(codexHome, 'spectre', 'hooks', 'session-start.mjs')));
+
+    const hooksConfig = JSON.parse(fs.readFileSync(path.join(codexHome, 'hooks.json'), 'utf8'));
+    assert.ok(hooksConfig.hooks.SessionStart.some(group =>
+      Array.isArray(group.hooks) && group.hooks.some(hook => hook.command.includes('spectre/hooks/scripts/load-knowledge.mjs'))
+    ));
+
+    const runtimeFiles = collectFiles(path.join(codexHome, 'spectre'));
+    for (const filePath of runtimeFiles) {
+      const content = fs.readFileSync(filePath, 'utf8');
+      assert.doesNotMatch(content, /file:\/\//, `${filePath} should not contain package-cache file:// imports`);
+      assert.doesNotMatch(content, new RegExp(repoRoot.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')), `${filePath} should not reference the repo checkout`);
+    }
+  } finally {
+    fs.rmSync(packDir, { recursive: true, force: true });
+    fs.rmSync(projectDir, { recursive: true, force: true });
+  }
+});

package/plugins/spectre/commands/learn.md

@@ -1,15 +0,0 @@
----
-description: "\ud83d\udc7b | Capture knowledge for future sessions"
----
-
-# /learn - Capture Project Knowledge
-
-Load the `spectre-learn` skill via `Skill(spectre-learn)` and **execute its workflow step-by-step**.
-
-<CRITICAL>
-This skill supersedes auto-memory. Do NOT write to MEMORY.md or any auto-memory directory. The skill defines the exclusive storage location, format, and registration workflow for captured knowledge.
-
-Treat the skill output as a binding directive, not informational context.
-</CRITICAL>
-
-**Topic**: $ARGUMENTS

package/plugins/spectre/commands/recall.md

@@ -1,5 +0,0 @@
-# /recall - Search Project Knowledge
-
-Load the `spectre-recall` skill and follow its instructions.
-
-**Search query**: $ARGUMENTS

package/plugins/spectre/hooks/scripts/load-knowledge.cjs

@@ -1,120 +0,0 @@
-#!/usr/bin/env node
-'use strict';
-
-/**
- * load-knowledge.cjs
- *
- * SessionStart hook that injects the apply skill content with embedded registry
- * directly into Claude's context.
- *
- * Reads:
- * - Apply skill from plugin: skills/spectre-apply/SKILL.md
- * - Registry from project: .claude/skills/spectre-recall/references/registry.toon
- *
- * Combines them by replacing the Registry Location section with actual registry content.
- */
-
-const fs = require('fs');
-const path = require('path');
-
-function countRegistryEntries(lines) {
-  let count = 0;
-  for (const line of lines) {
-    if (line.trim() && line.includes('|') && !line.startsWith('#')) {
-      count++;
-    }
-  }
-  return count;
-}
-
-function stripFrontmatter(content) {
-  if (content.startsWith('---')) {
-    const end = content.indexOf('---', 3);
-    if (end !== -1) {
-      return content.slice(end + 3).trim();
-    }
-  }
-  return content;
-}
-
-function main() {
-  const projectDir = process.env.CLAUDE_PROJECT_DIR || process.cwd();
-  const pluginRoot = process.env.CLAUDE_PLUGIN_ROOT || '';
-
-  const applySkillPath = path.join(pluginRoot, 'skills', 'spectre-apply', 'SKILL.md');
-
-  if (!fs.existsSync(applySkillPath)) {
-    process.exit(0);
-  }
-
-  // Paths - check new name first, fall back to old names for migration
-  let registryPath = path.join(projectDir, '.claude', 'skills', 'spectre-recall', 'references', 'registry.toon');
-  const oldRegistryPath = path.join(projectDir, '.claude', 'skills', 'spectre-find', 'references', 'registry.toon');
-
-  // Support old "spectre-find" path for projects that haven't migrated
-  if (!fs.existsSync(registryPath) && fs.existsSync(oldRegistryPath)) {
-    registryPath = oldRegistryPath;
-  }
-
-  // Read registry if it exists
-  let registryContent = '';
-  let entryCount = 0;
-  if (fs.existsSync(registryPath)) {
-    registryContent = fs.readFileSync(registryPath, 'utf8').trim();
-    const lines = registryContent ? registryContent.split('\n') : [];
-    entryCount = countRegistryEntries(lines);
-  }
-
-  // Read apply skill and strip frontmatter
-  let applyContent = fs.readFileSync(applySkillPath, 'utf8');
-  applyContent = stripFrontmatter(applyContent);
-
-  // Replace the Registry Location section with embedded registry or empty notice
-  let registrySection;
-  if (entryCount > 0) {
-    registrySection =
-      '## Registry\n\n' +
-      '**Format**: `skill-name|category|triggers|description`\n\n' +
-      '```\n' +
-      registryContent + '\n' +
-      '```\n\n' +
-      'Each entry corresponds to a skill that can be loaded via `Skill({skill-name})`\n\n' +
-      '**Categories:** feature, gotchas, patterns, decisions, procedures, integration, performance, testing, ux, strategy';
-  } else {
-    registrySection =
-      '## Registry\n\n' +
-      'No knowledge has been captured for this project yet. The behavioral rules in this document still apply.\n\n' +
-      'To capture knowledge from this session, use `/spectre:learn` after completing significant work.\n\n' +
-      '**Categories:** feature, gotchas, patterns, decisions, procedures, integration, performance, testing, ux, strategy';
-  }
-
-  // Replace the Registry Location section
-  applyContent = applyContent.replace(
-    /## Registry Location[\s\S]*?(?=## Workflow)/,
-    registrySection + '\n\n'
-  );
-
-  // Build final context
-  const context = `<spectre-knowledge>\n${applyContent}\n</spectre-knowledge>`;
-
-  // Visible notice
-  let visibleNotice;
-  if (entryCount > 0) {
-    visibleNotice = `\ud83d\udc7b spectre: ${entryCount} knowledge skills available`;
-  } else {
-    visibleNotice = '\ud83d\udc7b spectre: ready \u2014 capture knowledge with /spectre:learn';
-  }
-
-  const output = {
-    systemMessage: visibleNotice,
-    hookSpecificOutput: {
-      hookEventName: 'SessionStart',
-      additionalContext: context
-    }
-  };
-
-  process.stdout.write(JSON.stringify(output) + '\n');
-  process.exit(0);
-}
-
-main();

package/plugins/spectre/hooks/scripts/precompact-warning.cjs

@@ -1,19 +0,0 @@
-#!/usr/bin/env node
-'use strict';
-
-/**
- * precompact-warning.cjs
- *
- * PreCompact hook that suggests using /spectre:handoff + /clear
- * instead of auto-compact for better context continuity.
- */
-
-const output = {
-  systemMessage:
-    '\u26a0\ufe0f Auto-compact can cause context loss. ' +
-    'For full continuity: /spectre:handoff \u2192 /clear \u2192 new session. ' +
-    'Consider disabling auto-compact in /config.'
-};
-
-process.stdout.write(JSON.stringify(output) + '\n');
-process.exit(0);

package/plugins/spectre/hooks/scripts/register_learning.cjs

@@ -1,144 +0,0 @@
-#!/usr/bin/env node
-'use strict';
-
-/**
- * register_learning.cjs
- *
- * Registers a spectre learning and manages the project-level recall skill.
- *
- * Responsibilities:
- * 1. Create/update registry at .claude/skills/spectre-recall/references/registry.toon
- * 2. Read recall-template.md from plugin
- * 3. Generate .claude/skills/spectre-recall/SKILL.md with embedded registry
- *
- * Usage:
- *   node register_learning.cjs \
- *     --project-root "/path/to/project" \
- *     --skill-name "feature-my-feature" \
- *     --category "feature" \
- *     --triggers "keyword1, keyword2" \
- *     --description "Use when doing X or Y"
- */
-
-const fs = require('fs');
-const path = require('path');
-
-function getRegistryHeader() {
-  return [
-    '# SPECTRE Knowledge Registry',
-    '# Format: skill-name|category|triggers|description',
-    ''
-  ];
-}
-
-function updateRegistry(registryPath, entry, skillName) {
-  const entryPrefix = skillName + '|';
-  let lines;
-
-  if (fs.existsSync(registryPath)) {
-    const content = fs.readFileSync(registryPath, 'utf8').trim();
-    lines = content ? content.split('\n') : [];
-  } else {
-    lines = getRegistryHeader();
-  }
-
-  let entryExists = false;
-  const updatedLines = [];
-
-  for (const line of lines) {
-    if (line.startsWith(entryPrefix)) {
-      updatedLines.push(entry);
-      entryExists = true;
-    } else {
-      updatedLines.push(line);
-    }
-  }
-
-  if (!entryExists) {
-    updatedLines.push(entry);
-  }
-
-  let content = updatedLines.join('\n');
-  if (!content.endsWith('\n')) {
-    content += '\n';
-  }
-
-  fs.writeFileSync(registryPath, content);
-  return content;
-}
-
-function generateFindSkill(findSkillPath, templatePath, registryContent) {
-  if (!fs.existsSync(templatePath)) {
-    process.stderr.write(`Warning: Template not found at ${templatePath}\n`);
-    return;
-  }
-
-  const template = fs.readFileSync(templatePath, 'utf8');
-  const skillContent = template.replace('{{REGISTRY}}', registryContent.trim());
-
-  fs.mkdirSync(path.dirname(findSkillPath), { recursive: true });
-  fs.writeFileSync(findSkillPath, skillContent);
-}
-
-function parseArgs(argv) {
-  const args = {};
-  const flags = ['--project-root', '--skill-name', '--category', '--triggers', '--description'];
-
-  for (let i = 0; i < argv.length; i++) {
-    if (flags.includes(argv[i]) && i + 1 < argv.length) {
-      // Convert --project-root to projectRoot
-      const key = argv[i].slice(2).replace(/-([a-z])/g, (_, c) => c.toUpperCase());
-      args[key] = argv[i + 1];
-      i++;
-    }
-  }
-
-  return args;
-}
-
-function main() {
-  const args = parseArgs(process.argv.slice(2));
-
-  const required = ['projectRoot', 'skillName', 'category', 'triggers', 'description'];
-  for (const key of required) {
-    if (!args[key]) {
-      process.stderr.write(`Error: missing required argument --${key.replace(/[A-Z]/g, c => '-' + c.toLowerCase())}\n`);
-      process.exit(1);
-    }
-  }
-
-  const projectRoot = args.projectRoot;
-
-  // New paths: registry lives inside spectre-recall skill
-  const recallDir = path.join(projectRoot, '.claude', 'skills', 'spectre-recall');
-  const registryDir = path.join(recallDir, 'references');
-  const registryPath = path.join(registryDir, 'registry.toon');
-  const recallSkillPath = path.join(recallDir, 'SKILL.md');
-
-  // Template is in the plugin — resolve via env var (hooks) or __filename (manual invocation)
-  let pluginRoot;
-  if (process.env.CLAUDE_PLUGIN_ROOT) {
-    pluginRoot = process.env.CLAUDE_PLUGIN_ROOT;
-  } else {
-    // Fallback: resolve relative to this script
-    // Script is at: <plugin_root>/hooks/scripts/register_learning.cjs
-    pluginRoot = path.resolve(__dirname, '..', '..');
-  }
-  const templatePath = path.join(pluginRoot, 'skills', 'spectre-learn', 'references', 'recall-template.md');
-
-  // Ensure directories exist
-  fs.mkdirSync(registryDir, { recursive: true });
-
-  // Build the registry entry
-  const entry = `${args.skillName}|${args.category}|${args.triggers}|${args.description}`;
-
-  // Update registry and get full content
-  const registryContent = updateRegistry(registryPath, entry, args.skillName);
-
-  // Generate recall skill with embedded registry
-  generateFindSkill(recallSkillPath, templatePath, registryContent);
-
-  process.stdout.write(`Registered: ${entry}\n`);
-}
-
-main();

package/plugins/spectre/hooks/scripts/test_register-learning.cjs

@@ -1,146 +0,0 @@
-#!/usr/bin/env node
-'use strict';
-
-/**
- * Tests for register_learning.cjs
- *
- * Run with: node --test plugins/spectre/hooks/scripts/test_register-learning.cjs
- */
-
-const { describe, it } = require('node:test');
-const assert = require('node:assert/strict');
-const fs = require('fs');
-const path = require('path');
-const os = require('os');
-const { execFileSync } = require('child_process');
-
-const SCRIPT_PATH = path.join(__dirname, 'register_learning.cjs');
-
-function createTmpDir() {
-  return fs.mkdtempSync(path.join(os.tmpdir(), 'spectre-rl-'));
-}
-
-function cleanup(dir) {
-  fs.rmSync(dir, { recursive: true, force: true });
-}
-
-function runScript(args, opts) {
-  opts = opts || {};
-  const env = Object.assign({}, process.env);
-  if (opts.pluginRoot) {
-    env.CLAUDE_PLUGIN_ROOT = opts.pluginRoot;
-  } else {
-    delete env.CLAUDE_PLUGIN_ROOT;
-  }
-
-  try {
-    const stdout = execFileSync(process.execPath, [SCRIPT_PATH, ...args], {
-      env,
-      timeout: 10000,
-      encoding: 'utf8'
-    });
-    return { stdout, exitCode: 0 };
-  } catch (err) {
-    return { stdout: err.stdout || '', stderr: err.stderr || '', exitCode: err.status };
-  }
-}
-
-describe('register_learning', () => {
-  it('creates new registry with entry', () => {
-    const tmp = createTmpDir();
-    try {
-      const result = runScript([
-        '--project-root', tmp,
-        '--skill-name', 'feature-auth',
-        '--category', 'feature',
-        '--triggers', 'auth, login',
-        '--description', 'Use when working on authentication'
-      ]);
-
-      assert.equal(result.exitCode, 0);
-      assert.ok(result.stdout.includes('Registered:'));
-
-      const registryPath = path.join(tmp, '.claude', 'skills', 'spectre-recall', 'references', 'registry.toon');
-      assert.ok(fs.existsSync(registryPath));
-
-      const content = fs.readFileSync(registryPath, 'utf8');
-      assert.ok(content.includes('# SPECTRE Knowledge Registry'));
-      assert.ok(content.includes('feature-auth|feature|auth, login|Use when working on authentication'));
-    } finally {
-      cleanup(tmp);
-    }
-  });
-
-  it('updates existing entry by skill name', () => {
-    const tmp = createTmpDir();
-    try {
-      // First registration
-      runScript([
-        '--project-root', tmp,
-        '--skill-name', 'feature-auth',
-        '--category', 'feature',
-        '--triggers', 'auth',
-        '--description', 'Old description'
-      ]);
-
-      // Second registration with same skill name
-      runScript([
-        '--project-root', tmp,
-        '--skill-name', 'feature-auth',
-        '--category', 'feature',
-        '--triggers', 'auth, login, oauth',
-        '--description', 'Updated description'
-      ]);
-
-      const registryPath = path.join(tmp, '.claude', 'skills', 'spectre-recall', 'references', 'registry.toon');
-      const content = fs.readFileSync(registryPath, 'utf8');
-
-      // Should have the updated entry, not the old one
-      assert.ok(content.includes('Updated description'));
-      assert.ok(!content.includes('Old description'));
-
-      // Should only have one entry for feature-auth
-      const entries = content.split('\n').filter(l => l.startsWith('feature-auth|'));
-      assert.equal(entries.length, 1);
-    } finally {
-      cleanup(tmp);
-    }
-  });
-
-  it('generates recall skill with template', () => {
-    const tmp = createTmpDir();
-    const pluginRoot = path.join(tmp, 'plugin');
-
-    // Create template
-    const templateDir = path.join(pluginRoot, 'skills', 'spectre-learn', 'references');
-    fs.mkdirSync(templateDir, { recursive: true });
-    fs.writeFileSync(
-      path.join(templateDir, 'recall-template.md'),
-      '# Recall Skill\n\nRegistry:\n{{REGISTRY}}\n\nEnd.\n'
-    );
-
-    try {
-      runScript([
-        '--project-root', tmp,
-        '--skill-name', 'feature-test',
-        '--category', 'feature',
-        '--triggers', 'test',
-        '--description', 'Test skill'
-      ], { pluginRoot });
-
-      const skillPath = path.join(tmp, '.claude', 'skills', 'spectre-recall', 'SKILL.md');
-      assert.ok(fs.existsSync(skillPath));
-
-      const content = fs.readFileSync(skillPath, 'utf8');
-      assert.ok(content.includes('# Recall Skill'));
-      assert.ok(content.includes('feature-test|feature|test|Test skill'));
-    } finally {
-      cleanup(tmp);
-    }
-  });
-
-  it('fails with missing required arguments', () => {
-    const result = runScript(['--project-root', '/tmp/fake']);
-    assert.notEqual(result.exitCode, 0);
-  });
-});