@luanpdd/kit-mcp 0.4.0 → 0.5.0

package/CHANGELOG.md

@@ -6,6 +6,30 @@ Format: [Keep a Changelog](https://keepachangelog.com/en/1.1.0/) · Versioning:
 
 ## [Unreleased]
 
+## [0.5.0] - 2026-05-03
+
+### Added
+- **Mirror-tree sync for `framework` and `hooks`.** `kit/framework/` (124 files: workflows, templates, references, libs) and `kit/hooks/` (5 files) are now projected into `.claude/framework/` and `.claude/hooks/` on every `sync install claude-code`. Without this, the bundled slash-commands like `/novo-marco` were broken-by-design — they referenced `@./.claude/framework/workflows/new-milestone.md` and similar paths that never existed in the destination project. Now they resolve correctly end-to-end.
+- New `mode: 'mirror-tree'` capability spec in `src/core/registry.js`. Each mirror-tree entry has a `source` (relative path inside `kit/`) and a `path` (destination path in the target project).
+- A `.kit-mcp-managed` marker file is written at the root of each managed tree so `kit sync remove` can recursively clean up the directory **only** when the marker is present. Trees you authored yourself (without the marker) are never touched.
+- CI smoke test asserts `.claude/framework/workflows/new-milestone.md`, `.claude/framework/templates/project.md`, and `.claude/hooks/workflow-guard.js` are projected, and that `sync remove` cleans them up.
+- New CI safety test: `sync remove` against a `.claude/framework/` directory with no marker preserves user content.
+
+### Changed
+- `statusOf` now reports `framework` and `hooks` capability paths.
+- README capability matrix gained two columns (`framework`, `hooks`) and a paragraph explaining the mirror-tree semantics.
+
+### Migration
+No action needed — `npx -y @luanpdd/kit-mcp@latest sync install claude-code --project-root .` projects the new directories automatically. If you had a manually-created `.claude/framework/` or `.claude/hooks/`, kit-mcp will overwrite individual files but won't delete user files; `sync remove` continues to leave them alone.
+
+## [0.4.1] - 2026-05-03
+
+### Fixed
+- `src/mcp-server/index.js` was importing `DEFAULT_KIT_ROOT` from `core/kit.js`, but that export was renamed to `BUNDLED_KIT_ROOT` / `resolveKitRoot` during the v0.2.0 refactor. The unused import wasn't caught by CI (which only smoke-tests CLI commands, not MCP server boot) and made the server crash on `npx -y @luanpdd/kit-mcp` for any sync/install command. Removed the dead import — server now boots cleanly.
+
+### Tests (suggestion)
+- CI should boot `node bin/mcp.js` and validate exit. Tracked in roadmap.
+
 ## [0.4.0] - 2026-05-03
 
 ### Changed
@@ -150,7 +174,9 @@ npx -y @luanpdd/kit-mcp sync install claude-code --project-root .
 - CLI mirror of all MCP tools.
 - `install` command that registers kit-mcp into an IDE's MCP config (JSON for Claude/Cursor/Gemini/Windsurf, TOML for Codex).
 
-[Unreleased]: https://github.com/luanpdd/kit-mcp/compare/v0.4.0...HEAD
+[Unreleased]: https://github.com/luanpdd/kit-mcp/compare/v0.5.0...HEAD
+[0.5.0]: https://github.com/luanpdd/kit-mcp/compare/v0.4.1...v0.5.0
+[0.4.1]: https://github.com/luanpdd/kit-mcp/compare/v0.4.0...v0.4.1
 [0.4.0]: https://github.com/luanpdd/kit-mcp/compare/v0.3.0...v0.4.0
 [0.3.0]: https://github.com/luanpdd/kit-mcp/compare/v0.2.1...v0.3.0
 [0.2.0]: https://github.com/luanpdd/kit-mcp/compare/v0.1.6...v0.2.0

package/README.md

@@ -159,19 +159,21 @@ kit sync watch --all                          # watch + auto-detect every IDE al
 
 **Per-IDE projection** — what each target receives:
 
-| IDE | rules → | agents → | commands → | skills → |
-|---|---|---|---|---|
-| Claude Code | `CLAUDE.md` | `.claude/agents/*.md` | `.claude/commands/*.md` | `.claude/skills/*/` |
-| Cursor | `.cursor/rules/*.mdc` | `.cursor/agents/*.md` | — | — |
-| Codex | `AGENTS.md` | — | — | `.codex/skills/*/` |
-| Gemini CLI | `GEMINI.md` | — | — | `.gemini/skills/*/` |
-| Copilot | `.github/copilot-instructions.md` | `.github/agents/*.agent` | — | `.github/skills/*/` |
-| Windsurf | `.windsurf/rules/*.md` | `.windsurf/agents/*.md` | — | `.windsurf/skills/*/` |
-| Antigravity | `.agents/rules/*.md` | `.agents/agents/*.md` | — | `.agents/workflows/*/` |
-| Trae | `.trae/rules/*.md` | `.trae/agents/*.md` | — | — |
+| IDE | rules → | agents → | commands → | skills → | framework → | hooks → |
+|---|---|---|---|---|---|---|
+| Claude Code | `CLAUDE.md` | `.claude/agents/*.md` | `.claude/commands/*.md` | `.claude/skills/*/` | `.claude/framework/**` | `.claude/hooks/**` |
+| Cursor | `.cursor/rules/*.mdc` | `.cursor/agents/*.md` | — | — | — | — |
+| Codex | `AGENTS.md` | — | — | `.codex/skills/*/` | — | — |
+| Gemini CLI | `GEMINI.md` | — | — | `.gemini/skills/*/` | — | — |
+| Copilot | `.github/copilot-instructions.md` | `.github/agents/*.agent` | — | `.github/skills/*/` | — | — |
+| Windsurf | `.windsurf/rules/*.md` | `.windsurf/agents/*.md` | — | `.windsurf/skills/*/` | — | — |
+| Antigravity | `.agents/rules/*.md` | `.agents/agents/*.md` | — | `.agents/workflows/*/` | — | — |
+| Trae | `.trae/rules/*.md` | `.trae/agents/*.md` | — | — | — | — |
 
 A capability marked `—` is not supported by that IDE. Adding a new IDE = one entry in [`src/core/registry.js`](src/core/registry.js).
 
+**About `framework` and `hooks`:** these are *mirror-tree* capabilities — the entire `kit/framework/` and `kit/hooks/` subtrees are copied verbatim into `.claude/framework/` and `.claude/hooks/`. They're needed by the bundled workflow because slash-commands like `/novo-marco` reference framework files via paths like `@./.claude/framework/workflows/new-milestone.md`. A `.kit-mcp-managed` marker is written at the root of each managed tree so `kit sync remove` can clean up safely without touching directories you authored yourself.
+
 ### `kit install ...` — register kit-mcp into an IDE's MCP config
 
 ```bash

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@luanpdd/kit-mcp",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "description": "Generic infrastructure to ship YOUR personal kit of agents/commands/skills as an MCP server, with cross-IDE sync (Claude Code, Cursor, Codex, Gemini, Windsurf, Antigravity, Copilot, Trae).",
   "type": "module",
   "bin": {

package/src/core/registry.js

@@ -20,6 +20,8 @@ export const TARGETS = {
     agents:   { path: '.claude/agents/',                   mode: 'multi',     extension: '.md' },
     commands: { path: '.claude/commands/',                 mode: 'multi',     extension: '.md' },
     skills:   { path: '.claude/skills/',                   mode: 'multi-dir' },
+    framework:{ path: '.claude/framework/',                mode: 'mirror-tree', source: 'framework' },
+    hooks:    { path: '.claude/hooks/',                    mode: 'mirror-tree', source: 'hooks' },
     mcpConfig:{ path: '.mcp.json',                         strategy: 'merge-mcpServers-json',
                 userPath: '~/.claude.json',                userKey: 'mcpServers' },
   },
@@ -92,11 +94,13 @@ export function listTargets() {
     id,
     label: t.label,
     capabilities: {
-      rules:    !!t.rules,
-      agents:   !!t.agents,
-      commands: !!t.commands,
-      skills:   !!t.skills,
-      mcpConfig:!!t.mcpConfig,
+      rules:     !!t.rules,
+      agents:    !!t.agents,
+      commands:  !!t.commands,
+      skills:    !!t.skills,
+      framework: !!t.framework,
+      hooks:     !!t.hooks,
+      mcpConfig: !!t.mcpConfig,
     },
   }));
 }

package/src/core/sync.js

@@ -15,6 +15,8 @@ import { getTarget } from './registry.js';
 import { listKit, resolveKitRoot } from './kit.js';
 
 const STUB_MARKER = '<!-- kit-mcp:reference -->';
+const MANAGED_MARKER_FILE = '.kit-mcp-managed';
+const MANAGED_MARKER_BODY = '# Managed by @luanpdd/kit-mcp — this directory is overwritten on every `kit sync install`.\n# Do not edit files here directly; edit the canonical source under kit/ and re-run sync.\n# Removing this file disables `kit sync remove` cleanup of this tree.\n';
 
 export async function syncTo(targetId, opts = {}) {
   const target      = getTarget(targetId);
@@ -62,21 +64,62 @@ export async function syncTo(targetId, opts = {}) {
     }
   }
 
+  // Mirror-tree capabilities (framework, hooks) — copy a whole subtree of kit/<source>
+  // into target.<cap>.path, preserving relative structure. Dropped a marker file at the
+  // root so `kit sync remove` can clean up the tree safely.
+  for (const cap of ['framework', 'hooks']) {
+    const spec = target[cap];
+    if (!spec || spec.mode !== 'mirror-tree') continue;
+    const srcRoot = path.join(kitRoot, spec.source);
+    const dstRoot = path.join(projectRoot, spec.path);
+    const files = await walkTree(srcRoot);
+    if (files.length === 0) continue;
+    ops.push({ path: path.join(dstRoot, MANAGED_MARKER_FILE), content: MANAGED_MARKER_BODY, kind: cap });
+    for (const f of files) {
+      const dst = path.join(dstRoot, f.rel);
+      ops.push({ path: dst, srcAbs: f.abs, kind: cap, treeCopy: true });
+    }
+  }
+
   if (!dryRun) {
     for (const op of ops) {
       await fs.mkdir(path.dirname(op.path), { recursive: true });
-      await fs.writeFile(op.path, op.content, 'utf8');
+      if (op.treeCopy) {
+        await fs.copyFile(op.srcAbs, op.path);
+      } else {
+        await fs.writeFile(op.path, op.content, 'utf8');
+      }
     }
   }
 
   return { target: targetId, mode, projectRoot, kitRoot, written: ops.map(o => o.path), dryRun };
 }
 
+async function walkTree(dir) {
+  const out = [];
+  async function visit(current, relPrefix) {
+    let entries;
+    try { entries = await fs.readdir(current, { withFileTypes: true }); }
+    catch { return; }
+    for (const e of entries) {
+      const abs = path.join(current, e.name);
+      const rel = relPrefix ? `${relPrefix}/${e.name}` : e.name;
+      if (e.isDirectory()) {
+        await visit(abs, rel);
+      } else if (e.isFile()) {
+        out.push({ abs, rel });
+      }
+    }
+  }
+  await visit(dir, '');
+  return out;
+}
+
 export async function statusOf(targetId, opts = {}) {
   const target      = getTarget(targetId);
   const projectRoot = path.resolve(opts.projectRoot ?? process.cwd());
   const checks = [];
-  for (const cap of ['rules', 'agents', 'commands', 'skills']) {
+  for (const cap of ['rules', 'agents', 'commands', 'skills', 'framework', 'hooks']) {
     if (!target[cap]) continue;
     const probe = path.join(projectRoot, target[cap].path);
     let exists = false;
@@ -105,6 +148,18 @@ export async function removeFrom(targetId, opts = {}) {
       }
     } catch {}
   }
+  // Mirror-tree capabilities: only remove if our marker is present (we manage the whole subtree).
+  for (const cap of ['framework', 'hooks']) {
+    const spec = target[cap];
+    if (!spec || spec.mode !== 'mirror-tree') continue;
+    const dir = path.join(projectRoot, spec.path);
+    const marker = path.join(dir, MANAGED_MARKER_FILE);
+    try {
+      await fs.access(marker);
+      await fs.rm(dir, { recursive: true, force: true });
+      removed.push(dir);
+    } catch {}
+  }
   return { target: targetId, projectRoot, removed };
 }
 

package/src/mcp-server/index.js

@@ -12,7 +12,7 @@ import { Server } from '@modelcontextprotocol/sdk/server/index.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
 import { CallToolRequestSchema, ListToolsRequestSchema } from '@modelcontextprotocol/sdk/types.js';
 
-import { listKit, searchKit, findItem, DEFAULT_KIT_ROOT } from '../core/kit.js';
+import { listKit, searchKit, findItem } from '../core/kit.js';
 import { listTargets } from '../core/registry.js';
 import { syncTo, statusOf, removeFrom } from '../core/sync.js';
 import { detectReverse, applyReverse } from '../core/reverse-sync.js';