hypomnema 1.4.0 → 1.4.2

package/skills/crystallize/SKILL.md

@@ -61,7 +61,8 @@ When uncertain, surface the question rather than skip it. None of the four block
 3. **hot.md (root)** — update `<wiki-root>/hot.md` active-projects pointer table: set the `Last Session` date for this project to today.
 4. **session-log** — append a session entry to `projects/<name>/session-log/YYYY-MM-DD.md` (daily shard; the apply path creates today's file with seeded frontmatter if it does not exist yet).
 5. **open-questions** — only if `pages/open-questions.md` exists and questions were raised or resolved this session: move resolved ones out; add newly raised ones. Skip if unchanged.
-6. **log.md** — append a `session` entry to `<wiki-root>/log.md`.
+
+> Do **not** hand-write the root `log.md` session entry, and do **not** put a `log` field in the apply payload. The `## [date] session | <project>` entry is a derivable artifact: `--apply-session-close` calls `deriveRootLogEntries` after writing the session-log, reconstructing the canonical line from the session-log heading. (Supply `log` only for a deliberately custom line; it is now an optional payload field.)
 
 After completing the checklist, verify it before reporting:
 
@@ -76,7 +77,34 @@ reports and re-run until it prints **"Compact-ready"** — that is the signal th
 session is closed. A close-files-only pass is not enough; the real `/compact`
 also blocks on those other checks.
 
-Once it passes, report each item with ✓ and ask: "Session closed. Would you like to also run knowledge synthesis now, or stop here?"
+Optional `--project=<slug>` narrows the check to ONE project (close status plus
+lint scope). It is a project-scoped **diagnostic**, not the compact-ready
+signal. A green `--project` result (JSON `scope: "project"`) means only that
+project is close-complete; another today-active project can still block
+`/compact`. Use the plain `--check-session-close` (no `--project`) for the
+go/no-go close signal.
+
+**When using `--apply-session-close --session-id=<id>`** (the payload-driven
+path), the `--session-id` must be the main conversation's session id. Do NOT
+extract it from a background task or Agent output path (e.g., a UUID from
+`/tmp/.../<uuid>/tasks/...`). Such a UUID is a background task id, not the
+main conversation id. Passing the wrong id causes `markerWritten: false` with
+`markerSkipReason: "transcript-unresolved"` or `"no-user-close-signal"`: the
+5 mandatory files are written (ok: true) but the Stop-chain marker is withheld,
+so the session is not actually closed and the Stop hook re-prompts. The correct
+id comes from the `[WIKI_AUTOCLOSE]` block reason or `$CLAUDE_SESSION_ID`.
+
+Once it passes, report each item with ✓:
+- ✓ session-state.md
+- ✓ hot.md (project + root)
+- ✓ session-log entry
+- ✓ open-questions (or skipped if unchanged)
+- ✓ log.md entry
+- **marker written?** (required, if `--apply-session-close --session-id` was used): check `markerWritten` in the JSON output. If `true`, report "session-close marker written." If `false`, do NOT declare the session closed or complete. Instead report: "Files applied and verified (ok: true), but the session-close marker was not written (reason: `<markerSkipReason>`). The Stop-chain is still active. Re-run with the correct main-conversation `--session-id`."
+
+If `markerWritten: true` (or `--session-id` was not passed), ask: "Session closed. Would you like to also run knowledge synthesis now, or stop here?"
+
+If `markerWritten: false`, do NOT say "session closed." Surface the skip reason and instruct a re-run with the correct main-conversation `--session-id` (see the bullet above) before treating the session as closed.
 
 ---
 

package/templates/SCHEMA.md

@@ -154,9 +154,11 @@ Edit the feedback page only — never hand-edit the generated
 
 ## 4. Tag Vocabulary
 
-Use lowercase, hyphenated tags. Vocabulary is locked — `lint` blocks unknown tags
-and forbidden patterns (PascalCase, plurals, whitespace, generic words).
-Extend this list (and `~/hypomnema/SCHEMA.md`) before introducing a new tag.
+Use lowercase, hyphenated tags. `lint` hard-blocks forbidden patterns (PascalCase,
+plurals, whitespace, generic words); an unknown but well-formed tag is a warning,
+and a session close auto-registers it into the `### Pending` section below so the
+next lint accepts it. Promote a pending tag into a category here (and in
+`~/hypomnema/SCHEMA.md`) once it has settled.
 
 **Meta**: `wiki`, `index`, `pages`, `home`, `overview`, `guide`, `operations`, `schema`, `reference`, `hypo`, `commands`, `hot-cache`, `migration`
 **Workflow**: `automation`, `hooks`, `observability`, `autonomy`, `wiki-health`, `weekly`
@@ -165,6 +167,12 @@ Extend this list (and `~/hypomnema/SCHEMA.md`) before introducing a new tag.
 **Status**: `active`, `completed`, `archived`, `draft`, `stable`, `deprecated`, `needs-review`, `proposed`, `superseded`
 **Content classification**: `learning`, `tip`, `feedback`, `gotcha`, `concept`, `pattern`
 
+### Pending (auto-registered)
+
+Tags auto-registered by a crystallize close because they were not yet in the
+vocabulary above. Review periodically: promote each into a category and delete it
+here, or drop the tag from the page.
+
 ### Forbidden patterns
 
 | Pattern | Reason | Use instead |

package/templates/hypo-config.md

@@ -1,7 +1,7 @@
 ---
 title: Hypomnema Config
 type: config
-version: "1.4.0"
+version: "1.4.2"
 created: YYYY-MM-DD
 ---
 

package/templates/hypo-guide.md

@@ -95,7 +95,10 @@ Ask: *"이 작업이 마무리되었나요? 세션을 정리(crystallize)할까
    guarantee: the live gate can still differ on a context-≥70% prompt,
    `HYPO_SKIP_GATE`, or a transcript-scoped lint error — pass `--transcript-path`
    to include the last.) Pass `--session-id=<id>` to also see `marker_present`
-   (step 7).
+   (step 7). `--project=<slug>` narrows the check to one project (a scoped
+   diagnostic, JSON `scope: "project"`): green there means only that slug is
+   close-complete, **not** that `/compact` is globally unblocked. Use the plain
+   check for the go/no-go signal.
 7. Record the session-closed marker (ADR 0047). The Stop hook blocks until this
    session's per-session marker exists, and a hand-edit close (writing the files
    directly + committing) never writes it; the marker is written only by the

package/scripts/bump-version.mjs

@@ -1,77 +0,0 @@
-#!/usr/bin/env node
-// Sync version across package.json, .claude-plugin/plugin.json, and templates/hypo-config.md.
-// Usage: node scripts/bump-version.mjs <new-version>
-
-import { readFileSync, writeFileSync } from 'node:fs';
-import { resolve, dirname } from 'node:path';
-import { fileURLToPath } from 'node:url';
-
-const root = resolve(dirname(fileURLToPath(import.meta.url)), '..');
-const next = process.argv[2];
-
-if (!next || !/^\d+\.\d+\.\d+(-[\w.]+)?$/.test(next)) {
-  console.error('Usage: node scripts/bump-version.mjs <semver>');
-  console.error('Example: node scripts/bump-version.mjs 1.0.0');
-  process.exit(1);
-}
-
-const targets = [
-  {
-    path: 'package.json',
-    pattern: /("version"\s*:\s*")([^"]+)(")/,
-  },
-  {
-    path: '.claude-plugin/plugin.json',
-    pattern: /("version"\s*:\s*")([^"]+)(")/,
-  },
-  {
-    path: '.claude-plugin/marketplace.json',
-    pattern: /("version"\s*:\s*")([^"]+)(")/,
-  },
-  {
-    path: 'templates/hypo-config.md',
-    pattern: /(^version:\s*")([^"]+)(")/m,
-  },
-];
-
-for (const { path, pattern } of targets) {
-  const abs = resolve(root, path);
-  const before = readFileSync(abs, 'utf-8');
-  const match = before.match(pattern);
-  if (!match) {
-    console.error(`✗ ${path}: version pattern not found`);
-    process.exit(1);
-  }
-  const current = match[2];
-  if (current === next) {
-    console.log(`= ${path} already ${next}`);
-    continue;
-  }
-  writeFileSync(abs, before.replace(pattern, `$1${next}$3`));
-  console.log(`✓ ${path}: ${current} → ${next}`);
-}
-
-console.log(`\nThis bumps the 4 manifest files only. The full release checklist lives in`);
-console.log(`docs/CONTRIBUTING.md "Cutting a release" — follow it, or these steps:`);
-console.log(`\nNext steps:`);
-console.log(`  1. Sync package-lock (bump-version does NOT touch it; the lock carries the`);
-console.log(`     version twice and a stale lock fails check:versions + breaks npm ci):`);
-console.log(`       npm install --package-lock-only`);
-console.log(`  2. Edit CHANGELOG.md — ensure the "## [${next}]" section has a`);
-console.log(`     "### 한글 요약" sub-section (release.yml check-bilingual gate).`);
-console.log(`  3. Reconcile BOTH READMEs — add a v${next} sentence to README.md AND`);
-console.log(`     README.ko.md, and update the first-viewport "current release" pointer.`);
-console.log(`     (This step was dropped 3x; check:readme is now the floor gate.)`);
-console.log(`  4. Local pre-checks — the full gate set CI + prepublishOnly run:`);
-console.log(`       npm test && npm run lint && npm run check:versions \\`);
-console.log(`         && npm run check:bilingual && npm run check:readme \\`);
-console.log(`         && npm run smoke:plugin && npm run smoke-pack && npm run check:tracker-ids`);
-console.log(`  5. git add package.json package-lock.json .claude-plugin/ \\`);
-console.log(`            templates/hypo-config.md CHANGELOG.md README.md README.ko.md`);
-console.log(`     git commit -m "chore(release): v${next}"`);
-console.log(`  6. Create an ANNOTATED tag (lightweight tags are rejected by CI):`);
-console.log(`       git tag -a v${next} -m "<English body>\\n\\n---\\n\\n<한글 요약>"`);
-console.log(`     See docs/CONTRIBUTING.md "Cutting a release" for the full template.`);
-console.log(`  7. node scripts/check-bilingual.mjs --tag v${next}   # local pre-check`);
-console.log(`  8. git push origin <branch>   # then push the tag ALONE to trigger release:`);
-console.log(`       git push origin v${next}   # NOT --tags (avoid triggering stale tags)`);

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();