hypomnema 1.4.0 → 1.4.1

package/scripts/smoke-pack.mjs

@@ -1,261 +0,0 @@
-#!/usr/bin/env node
-/**
- * scripts/smoke-pack.mjs — pre-publish smoke test
- *
- * Runs `npm pack`, installs the resulting tarball into an isolated temp
- * directory, and exercises the installed CLI to verify the published
- * artifact actually works. Catches mistakes that local tests miss:
- *   - `files:` / `.npmignore` excluding required assets
- *   - `bin` entry pointing to a missing file
- *   - runtime requires on paths not shipped in the tarball
- *
- * Usage:
- *   node scripts/smoke-pack.mjs [--keep]
- *
- * --keep   Leave the temp directory in place for inspection on success.
- */
-
-import { spawnSync } from 'node:child_process';
-import {
-  mkdtempSync,
-  rmSync,
-  readFileSync,
-  writeFileSync,
-  existsSync,
-  readdirSync,
-  renameSync,
-} from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-import { fileURLToPath } from 'node:url';
-
-const REPO = join(fileURLToPath(new URL('.', import.meta.url)), '..');
-const KEEP = process.argv.includes('--keep');
-
-// When this script runs inside `npm publish --dry-run` (the release workflow's
-// publish-credential pre-check), npm exports `npm_config_dry_run=true` into the
-// lifecycle environment. spawnSync inherits process.env, so the nested
-// `npm pack` below would ALSO run in dry-run mode — reporting a tarball
-// filename/size it never actually writes to disk — and the subsequent
-// `npm install <tarball>` then fails with ENOENT (npm maps errno -2 → exit 254).
-// Strip the flag so the nested npm commands always perform real writes,
-// matching the real tag-push publish job (which has no outer `--dry-run`). The
-// outer workflow stays dry-run and still skips the registry PUT.
-// (This was the precheck-254 root cause; the earlier "missing NODE_AUTH_TOKEN"
-// theory was wrong — it is file absence, not auth.)
-const npmEnv = { ...process.env };
-for (const key of Object.keys(npmEnv)) {
-  if (key.toLowerCase().replaceAll('-', '_') === 'npm_config_dry_run') {
-    delete npmEnv[key];
-  }
-}
-
-const PKG = JSON.parse(readFileSync(join(REPO, 'package.json'), 'utf-8'));
-const PKG_NAME = PKG.name;
-
-function run(cmd, args, opts = {}) {
-  const res = spawnSync(cmd, args, { stdio: 'pipe', encoding: 'utf-8', ...opts });
-  if (res.status !== 0) {
-    process.stderr.write(res.stdout || '');
-    process.stderr.write(res.stderr || '');
-    throw new Error(`${cmd} ${args.join(' ')} exited ${res.status}`);
-  }
-  return res;
-}
-
-function step(msg) {
-  console.log(`\n▸ ${msg}`);
-}
-
-const work = mkdtempSync(join(tmpdir(), 'hypo-smoke-'));
-const sandboxHome = join(work, 'home');
-const installRoot = join(work, 'install');
-const wikiDir = join(work, 'wiki');
-let cleanupOk = false;
-
-try {
-  // Capture pre-commit hook contents (if any) BEFORE pack so we can prove
-  // `npm pack` didn't mutate it. The `prepare` lifecycle script runs during
-  // `npm pack` and could theoretically touch .git/hooks/pre-commit; the
-  // installer's CI/lifecycle guards must prevent that.
-  const preCommitPath = join(REPO, '.git', 'hooks', 'pre-commit');
-  const preCommitBefore = existsSync(preCommitPath) ? readFileSync(preCommitPath, 'utf-8') : null;
-
-  step('npm pack');
-  const pack = run('npm', ['pack', '--json'], { cwd: REPO, env: npmEnv });
-  const meta = JSON.parse(pack.stdout)[0];
-  const tarball = join(REPO, meta.filename);
-  console.log(`  → ${meta.filename} (${meta.size} bytes, ${meta.entryCount} entries)`);
-
-  step(`install into ${installRoot}`);
-  run('mkdir', ['-p', installRoot]);
-  writeFileSync(
-    join(installRoot, 'package.json'),
-    JSON.stringify({
-      name: 'hypo-smoke-host',
-      version: '0.0.0',
-      private: true,
-    }) + '\n',
-  );
-  // No `--silent`: run() only prints npm's stdout/stderr on a non-zero exit, so
-  // a real nested-install failure must not be muted (the precheck-254 error was
-  // masked by `--silent` for two release cycles).
-  run('npm', ['install', '--no-audit', '--no-fund', tarball], { cwd: installRoot, env: npmEnv });
-
-  // Move tarball into the work dir so it's not left in the repo.
-  renameSync(tarball, join(work, meta.filename));
-
-  const cliBin = join(installRoot, 'node_modules', '.bin', 'hypomnema');
-  if (!existsSync(cliBin)) {
-    throw new Error(`bin not installed at ${cliBin}`);
-  }
-
-  step('hypomnema --help (installed)');
-  const help = run(cliBin, ['--help']);
-  if (!help.stdout.includes('Usage: hypomnema')) {
-    throw new Error('--help output did not match expected banner');
-  }
-
-  step('hypomnema init --dry-run (installed)');
-  const dryEnv = { ...process.env, HOME: sandboxHome, HYPO_DIR: wikiDir };
-  const dry = run(
-    cliBin,
-    [
-      'init',
-      '--dry-run',
-      `--hypo-dir=${wikiDir}`,
-      '--no-hooks',
-      '--no-commands',
-      '--no-git-init',
-      '--no-shell',
-    ],
-    { env: dryEnv },
-  );
-  if (!/dry[\s-]?run/i.test(dry.stdout + dry.stderr)) {
-    console.log(dry.stdout);
-    throw new Error('init --dry-run did not announce dry-run mode');
-  }
-
-  step('verify shipped assets are present');
-  const required = [
-    'scripts/init.mjs',
-    'scripts/upgrade.mjs',
-    'hooks',
-    'commands',
-    'templates',
-    'README.md',
-  ];
-  const pkgRoot = join(installRoot, 'node_modules', PKG_NAME);
-  for (const rel of required) {
-    if (!existsSync(join(pkgRoot, rel))) {
-      throw new Error(`shipped tarball missing: ${rel}`);
-    }
-  }
-
-  step('hypomnema feedback-sync (installed) — check/write idempotency + bootstrap');
-  // Seed a fixture wiki + claude-home and exercise the feedback-sync surface
-  // through the installed CLI (proves the subcommand is routed + shipped and the
-  // Phase D bootstrap helper runs end-to-end). --check exits 1 on a fresh
-  // (un-written) projection, so use spawnSync (run() throws on non-zero).
-  const fbWiki = join(work, 'fb-wiki');
-  const fbHome = join(work, 'fb-home');
-  const fbProject = 'smoke';
-  run('mkdir', ['-p', join(fbWiki, 'pages', 'feedback')]);
-  run('mkdir', ['-p', join(fbHome, 'projects', fbProject, 'memory')]);
-  writeFileSync(
-    join(fbWiki, 'hypo-config.md'),
-    '---\ntitle: config\ntype: reference\n---\n# config\n',
-  );
-  writeFileSync(
-    join(fbWiki, 'pages', 'feedback', 'smoke-rule.md'),
-    [
-      '---',
-      'title: Smoke Rule',
-      'type: feedback',
-      'status: active',
-      'scope: global',
-      'tier: L1',
-      'targets: [project-memory, claude-learned]',
-      'sensitivity: public',
-      'priority: 4',
-      'memory_summary: smoke rule for the packed CLI',
-      'global_summary: always smoke-test the packed CLI',
-      'promote_to_global: true',
-      'reason: catch packaging regressions',
-      'source: session:2026-05-21',
-      'updated: 2026-05-21',
-      '---',
-      'body',
-      '',
-    ].join('\n'),
-  );
-  writeFileSync(
-    join(fbHome, 'CLAUDE.md'),
-    '# Global\n<learned_behaviors>\n- [2026-05-20] legacy hand rule — 이유: bootstrap source\n</learned_behaviors>\n',
-  );
-  writeFileSync(join(fbHome, 'projects', fbProject, 'memory', 'MEMORY.md'), '# Memory Index\n');
-
-  const fbArgs = [`--hypo-dir=${fbWiki}`, `--claude-home=${fbHome}`, `--project-id=${fbProject}`];
-  const fbRun = (extra) =>
-    spawnSync(cliBin, ['feedback-sync', ...extra, ...fbArgs], { encoding: 'utf-8' });
-
-  const fbCheck1 = fbRun(['--check']);
-  if (fbCheck1.status !== 1) {
-    throw new Error(
-      `feedback-sync --check on fresh projection expected exit 1, got ${fbCheck1.status}`,
-    );
-  }
-  const fbWrite = fbRun(['--write']);
-  if (fbWrite.status !== 0) {
-    throw new Error(
-      `feedback-sync --write expected exit 0, got ${fbWrite.status}: ${fbWrite.stderr}`,
-    );
-  }
-  if (
-    !readFileSync(join(fbHome, 'CLAUDE.md'), 'utf-8').includes(
-      'HYPO:FEEDBACK-SYNC:START source=smoke-rule',
-    )
-  ) {
-    throw new Error('feedback-sync --write did not project the managed block into CLAUDE.md');
-  }
-  const fbCheck2 = fbRun(['--check']);
-  if (fbCheck2.status !== 0) {
-    throw new Error(
-      `feedback-sync --check after --write expected clean exit 0, got ${fbCheck2.status}`,
-    );
-  }
-  const fbBootstrap = fbRun(['--bootstrap', '--dry-run']);
-  if (fbBootstrap.status !== 0 || !/would create draft/.test(fbBootstrap.stderr)) {
-    throw new Error(
-      `feedback-sync --bootstrap --dry-run did not announce drafts (exit ${fbBootstrap.status})`,
-    );
-  }
-
-  step('verify pre-commit hook was not mutated by npm pack');
-  const preCommitAfter = existsSync(preCommitPath) ? readFileSync(preCommitPath, 'utf-8') : null;
-  if (preCommitBefore !== preCommitAfter) {
-    throw new Error(
-      'npm pack mutated .git/hooks/pre-commit — the prepare lifecycle ' +
-        'guard (npm_command=pack) is not firing.',
-    );
-  }
-
-  console.log('\n✓ smoke-pack passed');
-  cleanupOk = true;
-} catch (err) {
-  console.error(`\n✗ smoke-pack failed: ${err.message}`);
-  console.error(`  work dir preserved: ${work}`);
-  process.exitCode = 1;
-} finally {
-  if (cleanupOk && !KEEP) {
-    rmSync(work, { recursive: true, force: true });
-  } else if (KEEP) {
-    console.log(`\nwork dir kept: ${work}`);
-  }
-  // Sweep any stray tarballs left in the repo root from a crashed run.
-  for (const f of readdirSync(REPO)) {
-    if (f.startsWith(`${PKG_NAME}-`) && f.endsWith('.tgz')) {
-      rmSync(join(REPO, f), { force: true });
-    }
-  }
-}

package/scripts/smoke-plugin.mjs

@@ -1,194 +0,0 @@
-#!/usr/bin/env node
-// smoke-plugin.mjs — a cheap, LOCAL structural check that the Claude Code plugin
-// would load, with no network and no Claude CLI dependency (so it runs in CI). It
-// validates the three surfaces Claude Code actually reads:
-//   1. .claude-plugin/plugin.json  — manifest (name, version, commands/skills paths)
-//   2. hooks/hooks.json            — every `${CLAUDE_PLUGIN_ROOT}/...` target must exist
-//   3. component files             — commands/*.md and skills/*/SKILL.md must be REAL
-//      (a .gitkeep alone is not a surface — a plugin with zero commands/skills must fail)
-// plus marketplace↔plugin name parity and that the marketplace `source` resolves to a
-// dir holding .claude-plugin/plugin.json.
-//
-// For a deep, real load check on a dev machine: `claude --plugin-dir . plugin list`.
-// That needs Claude Code installed, so it is NOT used here; this is the CI floor.
-//
-// Usage:
-//   node scripts/smoke-plugin.mjs               # smoke the repo's plugin
-//   node scripts/smoke-plugin.mjs --root <dir>  # point at a fixture (tests)
-//
-// Exit 0 = plugin surfaces are structurally valid. Exit 1 = a load-blocking problem.
-
-import { readFileSync, existsSync, statSync, readdirSync } from 'fs';
-import { join, dirname } from 'path';
-import { fileURLToPath } from 'url';
-
-function parseArgs(argv) {
-  const args = { root: null };
-  for (let i = 0; i < argv.length; i++) {
-    const a = argv[i];
-    if (a.startsWith('--root=')) args.root = a.slice(7);
-    else if (a === '--root') args.root = argv[++i];
-  }
-  return args;
-}
-
-const REPO_ROOT = join(dirname(fileURLToPath(import.meta.url)), '..');
-
-function isDir(p) {
-  try {
-    return statSync(p).isDirectory();
-  } catch {
-    return false;
-  }
-}
-
-function isFile(p) {
-  try {
-    return statSync(p).isFile();
-  } catch {
-    return false;
-  }
-}
-
-function smoke(root) {
-  const errors = [];
-  const notes = [];
-  const fail = (msg) => errors.push(msg);
-
-  // 1. plugin.json manifest.
-  let plugin = null;
-  const pluginPath = join(root, '.claude-plugin', 'plugin.json');
-  try {
-    plugin = JSON.parse(readFileSync(pluginPath, 'utf-8'));
-  } catch (err) {
-    fail(`.claude-plugin/plugin.json: ${err?.message ?? err}`);
-  }
-  if (plugin) {
-    if (typeof plugin.name !== 'string' || !plugin.name) fail('plugin.json: missing "name"');
-    if (typeof plugin.version !== 'string' || !plugin.version)
-      fail('plugin.json: missing "version"');
-    // commands/skills are declared as relative dir paths; if declared they must resolve.
-    for (const key of ['commands', 'skills']) {
-      if (plugin[key] != null) {
-        const rel = String(plugin[key]).replace(/^\.\//, '');
-        if (!isDir(join(root, rel)))
-          fail(`plugin.json: "${key}" path "${plugin[key]}" is not a directory`);
-      }
-    }
-  }
-
-  // 2. Component files must be REAL regular files, not just a .gitkeep placeholder
-  // (or a directory that happens to be named like a component — isFile, not exists).
-  const commandsMd = isDir(join(root, 'commands'))
-    ? readdirSync(join(root, 'commands')).filter(
-        (f) => f.endsWith('.md') && isFile(join(root, 'commands', f)),
-      )
-    : [];
-  if (commandsMd.length === 0) fail('commands/: no *.md command files (only a placeholder?)');
-  else notes.push(`commands: ${commandsMd.length}`);
-
-  let skillCount = 0;
-  if (isDir(join(root, 'skills'))) {
-    for (const entry of readdirSync(join(root, 'skills'))) {
-      if (isFile(join(root, 'skills', entry, 'SKILL.md'))) skillCount++;
-    }
-  }
-  if (skillCount === 0) fail('skills/: no */SKILL.md skills (only a placeholder?)');
-  else notes.push(`skills: ${skillCount}`);
-
-  // 3. hooks/hooks.json — every command target AND every `shared` file must be a
-  // real regular file on disk. hooks.json is the hook source of truth, so a missing
-  // hypo-shared.mjs / version-check.mjs would pass a manifest-only check but break
-  // hook imports at runtime.
-  const hooksPath = join(root, 'hooks', 'hooks.json');
-  if (!existsSync(hooksPath)) {
-    fail('hooks/hooks.json: missing');
-  } else {
-    let hooksJson = null;
-    try {
-      hooksJson = JSON.parse(readFileSync(hooksPath, 'utf-8'));
-    } catch (err) {
-      fail(`hooks/hooks.json: ${err?.message ?? err}`);
-    }
-    if (hooksJson) {
-      if (!hooksJson.hooks || typeof hooksJson.hooks !== 'object') {
-        fail('hooks/hooks.json: missing top-level "hooks" object');
-      } else {
-        const targets = new Set();
-        for (const groups of Object.values(hooksJson.hooks)) {
-          for (const group of groups || []) {
-            for (const hk of group?.hooks || []) {
-              if (hk?.command) {
-                // command looks like `node ${CLAUDE_PLUGIN_ROOT}/hooks/foo.mjs [args]`
-                const m = String(hk.command).match(/\$\{CLAUDE_PLUGIN_ROOT\}\/(\S+)/);
-                if (m) targets.add(m[1]);
-              }
-            }
-          }
-        }
-        if (targets.size === 0)
-          fail('hooks/hooks.json: no ${CLAUDE_PLUGIN_ROOT} command targets found');
-        for (const rel of targets) {
-          if (!isFile(join(root, rel))) fail(`hooks/hooks.json: target "${rel}" is not a file`);
-        }
-        notes.push(`hook targets: ${targets.size}`);
-      }
-      // `shared` lists hook-relative support files that the targets import.
-      if (Array.isArray(hooksJson.shared)) {
-        for (const shared of hooksJson.shared) {
-          if (!isFile(join(root, 'hooks', shared)))
-            fail(`hooks/hooks.json: shared file "hooks/${shared}" is not a file`);
-        }
-        notes.push(`shared: ${hooksJson.shared.length}`);
-      }
-    }
-  }
-
-  // 4. marketplace.json — name parity + source resolves.
-  const mpPath = join(root, '.claude-plugin', 'marketplace.json');
-  let mp = null;
-  try {
-    mp = JSON.parse(readFileSync(mpPath, 'utf-8'));
-  } catch (err) {
-    fail(`.claude-plugin/marketplace.json: ${err?.message ?? err}`);
-  }
-  if (mp) {
-    const plugins = Array.isArray(mp.plugins) ? mp.plugins : [];
-    if (plugins.length === 0) fail('marketplace.json: "plugins" is empty');
-    const name = plugin?.name;
-    if (name) {
-      const matches = plugins.filter((p) => p && p.name === name);
-      if (matches.length !== 1) {
-        fail(
-          `marketplace.json: expected exactly one entry named "${name}", found ${matches.length}`,
-        );
-      } else {
-        const src = matches[0].source ?? './';
-        const srcDir = join(root, String(src).replace(/^\.\//, ''));
-        if (!existsSync(join(srcDir, '.claude-plugin', 'plugin.json'))) {
-          fail(
-            `marketplace.json: source "${src}" does not resolve to a dir containing .claude-plugin/plugin.json`,
-          );
-        }
-      }
-    }
-  }
-
-  return { errors, notes };
-}
-
-function main() {
-  const args = parseArgs(process.argv.slice(2));
-  const root = args.root || REPO_ROOT;
-  const { errors, notes } = smoke(root);
-
-  if (errors.length) {
-    for (const e of errors) console.error(`  ✗ ${e}`);
-    console.error(`\n✗ plugin smoke failed (${errors.length} problem(s)).`);
-    process.exit(1);
-  }
-  console.log(`✓ plugin surfaces valid (${notes.join(', ')}).`);
-  process.exit(0);
-}
-
-main();