@shrkcrft/mcp-server 0.1.0-alpha.7 → 0.1.0-alpha.8

package/dist/server/create-mcp-server.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"create-mcp-server.d.ts","sourceRoot":"","sources":["../../src/server/create-mcp-server.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,MAAM,EAAE,MAAM,2CAA2C,CAAC;AAQnE,OAAO,EAAqB,KAAK,qBAAqB,EAAE,MAAM,qBAAqB,CAAC;AACpF,OAAO,EAGL,KAAK,gBAAgB,EACtB,MAAM,wBAAwB,CAAC;AAChC,OAAO,KAAK,EACV,eAAe,EAEf,eAAe,EAChB,MAAM,sBAAsB,CAAC;AAK9B,MAAM,WAAW,qBAAqB;IACpC,0DAA0D;IAC1D,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,0DAA0D;IAC1D,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,qEAAqE;IACrE,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB;;;;OAIG;IACH,SAAS,CAAC,EAAE,OAAO,GAAG,MAAM,CAAC;IAC7B,yCAAyC;IACzC,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,yCAAyC;IACzC,IAAI,CAAC,EAAE,MAAM,CAAC;IACd;;;;;OAKG;IACH,YAAY,CAAC,EAAE,eAAe,CAAC;CAChC;AAED,MAAM,WAAW,6BAA8B,SAAQ,gBAAgB;IACrE,0DAA0D;IAC1D,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,sDAAsD;IACtD,YAAY,CAAC,EAAE,eAAe,CAAC;CAChC;AAED,UAAU,WAAW;IACnB,MAAM,EAAE,gBAAgB,CAAC;IACzB,UAAU,EAAE,qBAAqB,GAAG,IAAI,CAAC;IACzC,WAAW,EAAE,GAAG,CAAC,MAAM,EAAE,eAAe,CAAC,CAAC;IAC1C,mCAAmC;IACnC,YAAY,CAAC,EAAE,eAAe,CAAC;CAChC;AAED;;;GAGG;AACH,wBAAgB,iBAAiB,CAC/B,SAAS,EAAE,MAAM,GAAG,SAAS,EAC7B,iBAAiB,EAAE,MAAM,GAAG,SAAS,EACrC,GAAG,GAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,SAAS,CAAe,GACpD,MAAM,CAIR;AAqBD,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,6BAA6B,GAAG;IAC7E,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,WAAW,CAAC;CACpB,CAqHA;AAED,wBAAsB,cAAc,CAAC,OAAO,GAAE,qBAA0B,GAAG,OAAO,CAAC,IAAI,CAAC,CAmEvF"}
+{"version":3,"file":"create-mcp-server.d.ts","sourceRoot":"","sources":["../../src/server/create-mcp-server.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,MAAM,EAAE,MAAM,2CAA2C,CAAC;AAQnE,OAAO,EAAqB,KAAK,qBAAqB,EAAE,MAAM,qBAAqB,CAAC;AACpF,OAAO,EAGL,KAAK,gBAAgB,EACtB,MAAM,wBAAwB,CAAC;AAChC,OAAO,KAAK,EACV,eAAe,EAEf,eAAe,EAChB,MAAM,sBAAsB,CAAC;AAM9B,MAAM,WAAW,qBAAqB;IACpC,0DAA0D;IAC1D,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,0DAA0D;IAC1D,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,qEAAqE;IACrE,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB;;;;OAIG;IACH,SAAS,CAAC,EAAE,OAAO,GAAG,MAAM,CAAC;IAC7B,yCAAyC;IACzC,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,yCAAyC;IACzC,IAAI,CAAC,EAAE,MAAM,CAAC;IACd;;;;;OAKG;IACH,YAAY,CAAC,EAAE,eAAe,CAAC;CAChC;AAED,MAAM,WAAW,6BAA8B,SAAQ,gBAAgB;IACrE,0DAA0D;IAC1D,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,sDAAsD;IACtD,YAAY,CAAC,EAAE,eAAe,CAAC;CAChC;AAED,UAAU,WAAW;IACnB,MAAM,EAAE,gBAAgB,CAAC;IACzB,UAAU,EAAE,qBAAqB,GAAG,IAAI,CAAC;IACzC,WAAW,EAAE,GAAG,CAAC,MAAM,EAAE,eAAe,CAAC,CAAC;IAC1C,mCAAmC;IACnC,YAAY,CAAC,EAAE,eAAe,CAAC;CAChC;AAED;;;GAGG;AACH,wBAAgB,iBAAiB,CAC/B,SAAS,EAAE,MAAM,GAAG,SAAS,EAC7B,iBAAiB,EAAE,MAAM,GAAG,SAAS,EACrC,GAAG,GAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,SAAS,CAAe,GACpD,MAAM,CAIR;AAqBD,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,6BAA6B,GAAG;IAC7E,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,WAAW,CAAC;CACpB,CAgIA;AAED,wBAAsB,cAAc,CAAC,OAAO,GAAE,qBAA0B,GAAG,OAAO,CAAC,IAAI,CAAC,CAmEvF"}

package/dist/server/create-mcp-server.js

@@ -5,6 +5,7 @@ import { CallToolRequestSchema, ListResourcesRequestSchema, ListToolsRequestSche
 import { inspectSharkcraft } from '@shrkcrft/inspector';
 import { MCP_SERVER_NAME, MCP_SERVER_VERSION, } from "./mcp-server-config.js";
 import { ALL_TOOLS } from "../tools/index.js";
+import { PRIMARY_MCP_TOOLS, shouldAdvertiseFullToolset } from "../tools/primary-tools.js";
 import { validateToolInput } from "./tool-input-validators.js";
 import { buildResourceList, readResource } from "../resources/index.js";
 /**
@@ -52,13 +53,24 @@ export function createSharkcraftServer(config) {
             resources: { listChanged: true },
         },
     });
-    server.setRequestHandler(ListToolsRequestSchema, async () => ({
-        tools: ALL_TOOLS.map((t) => ({
-            name: t.name,
-            description: t.description,
-            inputSchema: t.inputSchema,
-        })),
-    }));
+    server.setRequestHandler(ListToolsRequestSchema, async () => {
+        // Default `tools/list` advertises only the ~30 primary tools to
+        // keep the agent's tool-selection surface focused. The other ~200
+        // tools stay CALLABLE via `tools/call` for agents that already
+        // know the name (e.g. via `shrk export claude-skill` referencing
+        // them). Set SHRK_MCP_FULL_TOOLS=1 to advertise everything.
+        const advertiseFull = shouldAdvertiseFullToolset();
+        const advertised = advertiseFull
+            ? ALL_TOOLS
+            : ALL_TOOLS.filter((t) => PRIMARY_MCP_TOOLS.has(t.name));
+        return {
+            tools: advertised.map((t) => ({
+                name: t.name,
+                description: t.description,
+                inputSchema: t.inputSchema,
+            })),
+        };
+    });
     server.setRequestHandler(CallToolRequestSchema, async (request) => {
         const name = request.params.name;
         const args = (request.params.arguments ?? {});

package/dist/tools/all-tools.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"all-tools.d.ts","sourceRoot":"","sources":["../../src/tools/all-tools.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAmTpE,eAAO,MAAM,SAAS,EAAE,SAAS,eAAe,EA6P9C,CAAC"}
+{"version":3,"file":"all-tools.d.ts","sourceRoot":"","sources":["../../src/tools/all-tools.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAiTpE,eAAO,MAAM,SAAS,EAAE,SAAS,eAAe,EA0P9C,CAAC"}

package/dist/tools/all-tools.js

@@ -116,7 +116,8 @@ import { getContractTemplateTool, getLanguageCommandsTool, getLanguageProfilesTo
 import { getTaskContextTool, understandTaskTool, validateChangeContextTool, } from "./r26-task-context.tool.js";
 import { getLanguageCacheStatusTool, getLanguageProfilesLiveTool, getLanguageRunPlanTool, getPolyglotBoundaryReportTool, } from "./r27-polyglot.tool.js";
 import { getChangedBoundaryReportTool } from "./r28-changed-boundary.tool.js";
-import { previewPluginRemoveTool, previewPluginRenameTool } from "./r28-plugin-lifecycle.tool.js";
+import { getDiffCheckReportTool } from "./diff-check.tool.js";
+import { getFileAdviceTool } from "./file-advice.tool.js";
 import { getHelperTool, listHelpersTool, previewHelperPlanTool } from "./r28-helpers.tool.js";
 import { getPackDevStatusTool, previewPackTestsTool } from "./r28-pack-author.tool.js";
 import { getRegistryLifecycleReportTool } from "./r28-registry-lifecycle.tool.js";
@@ -135,7 +136,7 @@ import { suggestCommandsTool, searchCommandsTool, explainCommandTool, } from "./
 import { previewFixTool, listFixKindsTool, } from "./r31-fix-preview.tool.js";
 import { getScaffoldCoverageReportTool } from "./r31-scaffold-coverage.tool.js";
 import { getChangesSummaryTool, getPrSummaryPreviewTool, getCiIntegrityReportTool, } from "./r31-changes-pr-ci.tool.js";
-import { listProfilesTool, getProfileTool, getProfilesDoctorTool, listPluginLifecycleProfilesTool, getPluginLifecycleProfileTool, getPluginLifecycleProfileDoctorTool, } from "./r32-profiles.tool.js";
+import { listProfilesTool, getProfileTool, getProfilesDoctorTool, } from "./r32-profiles.tool.js";
 import { getProjectCouplingReportTool } from "./r32-project-coupling.tool.js";
 import { getPackContributionsTool, getPackConflictsTool, } from "./r33-pack-contributions.tool.js";
 import { listConventionsTool, getConventionTool, getConventionsDoctorTool, } from "./r33-conventions.tool.js";
@@ -334,8 +335,8 @@ export const ALL_TOOLS = Object.freeze([
     getLanguageCacheStatusTool,
     getLanguageProfilesLiveTool,
     getChangedBoundaryReportTool,
-    previewPluginRenameTool,
-    previewPluginRemoveTool,
+    getDiffCheckReportTool,
+    getFileAdviceTool,
     listHelpersTool,
     getHelperTool,
     previewHelperPlanTool,
@@ -370,9 +371,6 @@ export const ALL_TOOLS = Object.freeze([
     listProfilesTool,
     getProfileTool,
     getProfilesDoctorTool,
-    listPluginLifecycleProfilesTool,
-    getPluginLifecycleProfileTool,
-    getPluginLifecycleProfileDoctorTool,
     getProjectCouplingReportTool,
     getPackContributionsTool,
     getPackConflictsTool,

package/dist/tools/diff-check.tool.d.ts

@@ -0,0 +1,15 @@
+/**
+ * Read-only MCP tool: get_diff_check_report.
+ *
+ * The MCP-side mirror of `shrk diff-check`. Same envelope, same
+ * verdict logic — the agent gets a single structured answer to "did
+ * my edits pass this project's boundary + import-hygiene gates?".
+ *
+ * Still read-only: this tool DOES NOT fix anything, even when it
+ * could trivially suggest the fix. The agent reads the envelope, then
+ * the human (or the agent, via a separate write-path tool) runs the
+ * fix on the CLI. Keeps the safety contract intact.
+ */
+import type { IToolDefinition } from '../server/tool-definition.js';
+export declare const getDiffCheckReportTool: IToolDefinition;
+//# sourceMappingURL=diff-check.tool.d.ts.map

package/dist/tools/diff-check.tool.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"diff-check.tool.d.ts","sourceRoot":"","sources":["../../src/tools/diff-check.tool.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAaH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAyBpE,eAAO,MAAM,sBAAsB,EAAE,eAyHpC,CAAC"}

package/dist/tools/diff-check.tool.js

@@ -0,0 +1,157 @@
+/**
+ * Read-only MCP tool: get_diff_check_report.
+ *
+ * The MCP-side mirror of `shrk diff-check`. Same envelope, same
+ * verdict logic — the agent gets a single structured answer to "did
+ * my edits pass this project's boundary + import-hygiene gates?".
+ *
+ * Still read-only: this tool DOES NOT fix anything, even when it
+ * could trivially suggest the fix. The agent reads the envelope, then
+ * the human (or the agent, via a separate write-path tool) runs the
+ * fix on the CLI. Keeps the safety contract intact.
+ */
+import { buildImportHygieneReport, filterViolationsToChangedScope, resolveChangedFiles, } from '@shrkcrft/inspector';
+import { evaluateBoundaries, loadTsconfigPaths, scanImports, } from '@shrkcrft/boundaries';
+const SCHEMA = 'sharkcraft.diff-check/v1';
+function resolveScopeFromInput(input, cwd) {
+    const staged = input.staged === true;
+    const since = typeof input.since === 'string' ? input.since : undefined;
+    const files = Array.isArray(input.files)
+        ? input.files.filter((f) => typeof f === 'string')
+        : [];
+    if (files.length > 0) {
+        return { mode: 'files', options: { projectRoot: cwd, files } };
+    }
+    if (staged) {
+        return { mode: 'staged', options: { projectRoot: cwd, staged: true } };
+    }
+    if (since) {
+        return { mode: 'since', options: { projectRoot: cwd, since } };
+    }
+    return { mode: 'worktree', options: { projectRoot: cwd, includeWorktree: true } };
+}
+export const getDiffCheckReportTool = {
+    name: 'get_diff_check_report',
+    description: 'Self-check the current git diff against this project\'s boundary + import-hygiene rules. Single-call composite of the boundary-check and import-hygiene engines, scoped to the changed files, with one verdict (ok | warnings | errors) and one nextAction line. Use after editing code so you can validate before declaring done. Read-only.',
+    inputSchema: {
+        type: 'object',
+        additionalProperties: false,
+        properties: {
+            staged: { type: 'boolean', description: 'Scope to staged changes only.' },
+            since: { type: 'string', description: 'Compare against ref (HEAD, origin/main, SHA).' },
+            files: {
+                type: 'array',
+                items: { type: 'string' },
+                description: 'Explicit file list (overrides --staged / --since).',
+            },
+        },
+    },
+    async handler(input, ctx) {
+        const cwd = ctx.cwd;
+        const { mode, options: scopeOptions } = resolveScopeFromInput(input, cwd);
+        const changed = resolveChangedFiles(scopeOptions);
+        const changedFiles = changed.files;
+        const rules = ctx.inspection.boundaryRegistry.list();
+        let boundaryBlock = {
+            ran: false,
+            rulesEvaluated: 0,
+            counts: { error: 0, warning: 0, info: 0 },
+            violations: [],
+        };
+        if (rules.length > 0 && changedFiles.length > 0) {
+            const scan = scanImports({ projectRoot: cwd });
+            const tsconfigPaths = loadTsconfigPaths(cwd);
+            const evalResult = evaluateBoundaries(scan, rules, {
+                ...(tsconfigPaths.aliases.size > 0 ? { tsconfigPaths } : {}),
+            });
+            const filtered = filterViolationsToChangedScope(evalResult.violations, scopeOptions);
+            boundaryBlock = {
+                ran: true,
+                rulesEvaluated: evalResult.rulesEvaluated,
+                counts: {
+                    error: filtered.includedViolations.filter((v) => v.severity === 'error').length,
+                    warning: filtered.includedViolations.filter((v) => v.severity === 'warning').length,
+                    info: filtered.includedViolations.filter((v) => v.severity === 'info').length,
+                },
+                violations: filtered.includedViolations,
+            };
+        }
+        else if (rules.length > 0) {
+            boundaryBlock = { ...boundaryBlock, ran: true, rulesEvaluated: rules.length };
+        }
+        let importsBlock = {
+            ran: false,
+            verdict: 'skipped',
+            counts: {},
+            findings: [],
+        };
+        if (changedFiles.length > 0) {
+            const report = buildImportHygieneReport(cwd, { files: changedFiles });
+            importsBlock = {
+                ran: true,
+                verdict: report.verdict,
+                counts: report.counts ?? {},
+                findings: report.findings,
+            };
+        }
+        // Derive verdict — same logic as the CLI command, duplicated
+        // intentionally to keep the MCP tool self-contained (no CLI
+        // import — preserves the package dependency direction).
+        const bErr = boundaryBlock.counts.error;
+        const bWarn = boundaryBlock.counts.warning;
+        const iErr = importsBlock.verdict === 'errors' ? (importsBlock.counts.error ?? importsBlock.findings.length) : 0;
+        const iWarn = importsBlock.verdict === 'warnings' ? (importsBlock.counts.warning ?? importsBlock.findings.length) : 0;
+        let verdict;
+        let summary;
+        let nextAction;
+        if (changedFiles.length === 0) {
+            verdict = 'ok';
+            summary = 'No files changed in the current diff scope.';
+            nextAction =
+                'Nothing to check. If you expected changes, verify the `staged` / `since` argument or save edits first.';
+        }
+        else if (bErr > 0 || iErr > 0) {
+            verdict = 'errors';
+            const parts = [];
+            if (bErr > 0)
+                parts.push(`${bErr} boundary violation${bErr === 1 ? '' : 's'}`);
+            if (iErr > 0)
+                parts.push(`${iErr} import-hygiene error${iErr === 1 ? '' : 's'}`);
+            summary = `Diff fails the gate: ${parts.join(', ')}.`;
+            nextAction =
+                'Fix every error in `boundaries.violations` and `imports.findings` (each entry\'s `suggestedFix` shows the fix), then re-run.';
+        }
+        else if (bWarn > 0 || iWarn > 0) {
+            verdict = 'warnings';
+            const parts = [];
+            if (bWarn > 0)
+                parts.push(`${bWarn} boundary warning${bWarn === 1 ? '' : 's'}`);
+            if (iWarn > 0)
+                parts.push(`${iWarn} import-hygiene warning${iWarn === 1 ? '' : 's'}`);
+            summary = `Diff passes the gate with ${parts.join(', ')}.`;
+            nextAction = 'Safe to declare done. Review warnings if the diff touches a sensitive area.';
+        }
+        else {
+            verdict = 'ok';
+            summary = `Diff passes the gate (${changedFiles.length} file${changedFiles.length === 1 ? '' : 's'}, 0 violations).`;
+            nextAction = 'Safe to declare done.';
+        }
+        return {
+            text: `verdict=${verdict}. ${summary} ${nextAction}`,
+            data: {
+                schema: SCHEMA,
+                generatedAt: new Date().toISOString(),
+                scope: {
+                    mode,
+                    files: changedFiles,
+                    fileCount: changedFiles.length,
+                },
+                boundaries: boundaryBlock,
+                imports: importsBlock,
+                verdict,
+                summary,
+                nextAction,
+            },
+        };
+    },
+};

package/dist/tools/file-advice.tool.d.ts

@@ -0,0 +1,22 @@
+/**
+ * Read-only MCP tool: get_file_advice.
+ *
+ * For a given file path, returns the rules, path conventions, boundary
+ * rules, and knowledge entries that apply to it. The agent equivalent
+ * of `shrk why <file>` — same engine (`buildWhyReport`), shaped for
+ * MCP consumption.
+ *
+ * Why this exists alongside `inspect_workspace` and the various
+ * `list_*` tools: agents tend to over-explore when given browsable
+ * catalogs. Asking "give me everything for this file" in one call
+ * keeps the prompt window tight and the answer focused — the agent
+ * doesn't have to discover which rule matches the file's path glob,
+ * which boundary rule constrains its imports, or which knowledge
+ * entry is the right one to read first.
+ *
+ * Use after a `shrk diff-check` flags a violation, or before editing
+ * an unfamiliar file. Read-only.
+ */
+import type { IToolDefinition } from '../server/tool-definition.js';
+export declare const getFileAdviceTool: IToolDefinition;
+//# sourceMappingURL=file-advice.tool.d.ts.map

package/dist/tools/file-advice.tool.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"file-advice.tool.d.ts","sourceRoot":"","sources":["../../src/tools/file-advice.tool.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AAGH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAEpE,eAAO,MAAM,iBAAiB,EAAE,eAyC/B,CAAC"}

package/dist/tools/file-advice.tool.js

@@ -0,0 +1,88 @@
+/**
+ * Read-only MCP tool: get_file_advice.
+ *
+ * For a given file path, returns the rules, path conventions, boundary
+ * rules, and knowledge entries that apply to it. The agent equivalent
+ * of `shrk why <file>` — same engine (`buildWhyReport`), shaped for
+ * MCP consumption.
+ *
+ * Why this exists alongside `inspect_workspace` and the various
+ * `list_*` tools: agents tend to over-explore when given browsable
+ * catalogs. Asking "give me everything for this file" in one call
+ * keeps the prompt window tight and the answer focused — the agent
+ * doesn't have to discover which rule matches the file's path glob,
+ * which boundary rule constrains its imports, or which knowledge
+ * entry is the right one to read first.
+ *
+ * Use after a `shrk diff-check` flags a violation, or before editing
+ * an unfamiliar file. Read-only.
+ */
+import { buildWhyReport } from '@shrkcrft/inspector';
+export const getFileAdviceTool = {
+    name: 'get_file_advice',
+    description: 'For a given file path, return the rules, path conventions, boundary rules, and knowledge entries that apply to it. Single-call replacement for browsing the registry to figure out what constrains one file. Read-only.',
+    inputSchema: {
+        type: 'object',
+        additionalProperties: false,
+        required: ['file'],
+        properties: {
+            file: {
+                type: 'string',
+                description: 'Path to the file (absolute, or relative to the project root). The file does not need to exist — path-string matching still works.',
+            },
+            limit: {
+                type: 'number',
+                description: 'Cap rules and knowledge entries returned (default 10).',
+            },
+        },
+    },
+    async handler(input, ctx) {
+        const file = typeof input.file === 'string' ? input.file : '';
+        if (!file) {
+            return {
+                text: 'Error: `file` argument is required.',
+                data: { error: 'missing-argument', argument: 'file' },
+            };
+        }
+        const limit = typeof input.limit === 'number' && input.limit > 0 ? Math.floor(input.limit) : 10;
+        const report = buildWhyReport({
+            inspection: ctx.inspection,
+            projectRoot: ctx.cwd,
+            target: file,
+            limit,
+        });
+        const summary = buildSummary(report);
+        return {
+            text: summary,
+            data: report,
+        };
+    },
+};
+function buildSummary(report) {
+    const lines = [];
+    lines.push(`File: ${report.target.relativePath} (${report.target.kind})`);
+    if (report.inferredPackage)
+        lines.push(`Package: ${report.inferredPackage}`);
+    if (report.inferredLayer)
+        lines.push(`Layer: ${report.inferredLayer}`);
+    const counts = {
+        rules: report.rules.length,
+        boundaries: report.boundaries.length,
+        paths: report.pathConventions.length,
+        knowledge: report.knowledge.length,
+    };
+    lines.push(`Matches: ${counts.rules} rule${counts.rules === 1 ? '' : 's'}, ` +
+        `${counts.boundaries} boundary rule${counts.boundaries === 1 ? '' : 's'}, ` +
+        `${counts.paths} path convention${counts.paths === 1 ? '' : 's'}, ` +
+        `${counts.knowledge} knowledge entr${counts.knowledge === 1 ? 'y' : 'ies'}.`);
+    if (counts.rules === 0 &&
+        counts.boundaries === 0 &&
+        counts.paths === 0 &&
+        counts.knowledge === 0) {
+        lines.push('No registry entries matched. The file may be outside the conventions, or the workspace may not have rules / paths defined yet.');
+    }
+    else {
+        lines.push('See `data` for the full per-category list.');
+    }
+    return lines.join('\n');
+}

package/dist/tools/primary-tools.d.ts

@@ -0,0 +1,24 @@
+/**
+ * Primary MCP tools — the ~30 tools advertised to a connected agent
+ * by default. Every tool in {@link ALL_TOOLS} stays callable (so an
+ * agent that already knows the name can use it), but `tools/list`
+ * only advertises the primary set. Smaller surface = better
+ * tool-selection accuracy for the agent.
+ *
+ * Picked to match the CLI's allowlist semantics: anything an agent
+ * could realistically reach for during a normal task — discovery,
+ * context, planning, validation — is in. Internal introspection
+ * (catalog dumps, fix-preview internals, drift baselines) is out;
+ * still callable, just not in the default tool list.
+ *
+ * Escape hatch: set `SHRK_MCP_FULL_TOOLS=1` to advertise the full
+ * catalog (useful when debugging an agent's tool selection).
+ */
+export declare const PRIMARY_MCP_TOOLS: ReadonlySet<string>;
+/**
+ * Should `tools/list` advertise the full catalog instead of the
+ * primary set? Driven by the env var so an agent operator can flip it
+ * without rebuilding.
+ */
+export declare function shouldAdvertiseFullToolset(): boolean;
+//# sourceMappingURL=primary-tools.d.ts.map

package/dist/tools/primary-tools.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"primary-tools.d.ts","sourceRoot":"","sources":["../../src/tools/primary-tools.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AACH,eAAO,MAAM,iBAAiB,EAAE,WAAW,CAAC,MAAM,CA4ChD,CAAC;AAEH;;;;GAIG;AACH,wBAAgB,0BAA0B,IAAI,OAAO,CAGpD"}

package/dist/tools/primary-tools.js

@@ -0,0 +1,70 @@
+/**
+ * Primary MCP tools — the ~30 tools advertised to a connected agent
+ * by default. Every tool in {@link ALL_TOOLS} stays callable (so an
+ * agent that already knows the name can use it), but `tools/list`
+ * only advertises the primary set. Smaller surface = better
+ * tool-selection accuracy for the agent.
+ *
+ * Picked to match the CLI's allowlist semantics: anything an agent
+ * could realistically reach for during a normal task — discovery,
+ * context, planning, validation — is in. Internal introspection
+ * (catalog dumps, fix-preview internals, drift baselines) is out;
+ * still callable, just not in the default tool list.
+ *
+ * Escape hatch: set `SHRK_MCP_FULL_TOOLS=1` to advertise the full
+ * catalog (useful when debugging an agent's tool selection).
+ */
+export const PRIMARY_MCP_TOOLS = new Set([
+    // Project orientation
+    'inspect_workspace',
+    'get_project_overview',
+    'get_agent_instructions',
+    'get_start_here',
+    // Context / task routing
+    'get_relevant_context',
+    'get_task_packet',
+    'get_action_hints',
+    'create_agent_brief',
+    'get_relevant_rules',
+    'explain_command',
+    // Browse the registries
+    'list_knowledge',
+    'get_knowledge',
+    'list_rules',
+    'get_rule',
+    'list_path_conventions',
+    'list_templates',
+    'get_template',
+    'list_pipelines',
+    'get_pipeline',
+    'list_presets',
+    'get_preset',
+    'list_packs',
+    'get_pack',
+    // Safe code generation (plan-first)
+    'create_generation_plan',
+    'render_template_preview',
+    'explain_generation_target',
+    // Validation gates (read-only)
+    'check_boundaries',
+    'get_diff_check_report',
+    'get_file_advice',
+    'get_architecture_constraints',
+    'get_architecture_violations',
+    // Doctor / readiness
+    'get_ai_readiness_report',
+    'doctor_packs',
+    // Search
+    'search_all',
+    'search_knowledge',
+    'search_commands',
+]);
+/**
+ * Should `tools/list` advertise the full catalog instead of the
+ * primary set? Driven by the env var so an agent operator can flip it
+ * without rebuilding.
+ */
+export function shouldAdvertiseFullToolset() {
+    const v = process.env.SHRK_MCP_FULL_TOOLS;
+    return v === '1' || v === 'true' || v === 'yes';
+}

package/dist/tools/r19-extras.tool.js

@@ -93,7 +93,7 @@ export const getCommandTaxonomyTool = {
     inputSchema: { type: 'object', properties: {}, additionalProperties: false },
     async handler() {
         // The CLI carries the canonical catalog; here we describe the surface
-        // without importing it (no inspector → CLI dependency). Adopters can
+        // without importing it (no inspector → CLI dependency). Consumers can
         // run `shrk commands taxonomy` for the full grouped output.
         return {
             data: {

package/dist/tools/r32-profiles.tool.d.ts

@@ -2,7 +2,4 @@ import type { IToolDefinition } from '../server/tool-definition.js';
 export declare const listProfilesTool: IToolDefinition;
 export declare const getProfileTool: IToolDefinition;
 export declare const getProfilesDoctorTool: IToolDefinition;
-export declare const listPluginLifecycleProfilesTool: IToolDefinition;
-export declare const getPluginLifecycleProfileTool: IToolDefinition;
-export declare const getPluginLifecycleProfileDoctorTool: IToolDefinition;
 //# sourceMappingURL=r32-profiles.tool.d.ts.map

package/dist/tools/r32-profiles.tool.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"r32-profiles.tool.d.ts","sourceRoot":"","sources":["../../src/tools/r32-profiles.tool.ts"],"names":[],"mappings":"AAeA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAEpE,eAAO,MAAM,gBAAgB,EAAE,eAe9B,CAAC;AAEF,eAAO,MAAM,cAAc,EAAE,eAsB5B,CAAC;AAEF,eAAO,MAAM,qBAAqB,EAAE,eAOnC,CAAC;AAEF,eAAO,MAAM,+BAA+B,EAAE,eAO7C,CAAC;AAEF,eAAO,MAAM,6BAA6B,EAAE,eAmB3C,CAAC;AAEF,eAAO,MAAM,mCAAmC,EAAE,eAqBjD,CAAC"}
+{"version":3,"file":"r32-profiles.tool.d.ts","sourceRoot":"","sources":["../../src/tools/r32-profiles.tool.ts"],"names":[],"mappings":"AAaA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAEpE,eAAO,MAAM,gBAAgB,EAAE,eAe9B,CAAC;AAEF,eAAO,MAAM,cAAc,EAAE,eAsB5B,CAAC;AAEF,eAAO,MAAM,qBAAqB,EAAE,eAOnC,CAAC"}

package/dist/tools/r32-profiles.tool.js

@@ -1,11 +1,11 @@
 /**
- * Read-only MCP tools for profiles + lifecycle profiles.
+ * Read-only MCP tools for profiles.
  *
  * Engine never writes from MCP. These tools surface pack-contributed and
  * locally configured profiles so an agent can decide which one to pass to
- * `shrk plugin rename|remove --profile <id>` (the human runs the CLI).
+ * a profile-aware command (the human runs the CLI).
  */
-import { checkPluginLifecycleProfileHealth, findProfile, listProfileIssues, listProfiles, listPluginLifecycleProfiles, ProfileKind, } from '@shrkcrft/inspector';
+import { findProfile, listProfileIssues, listProfiles, ProfileKind, } from '@shrkcrft/inspector';
 export const listProfilesTool = {
     name: 'list_profiles',
     description: 'List all pack-contributed and locally configured profiles. Read-only.',
@@ -54,54 +54,3 @@ export const getProfilesDoctorTool = {
         return { data: { issues: await listProfileIssues(ctx.inspection) } };
     },
 };
-export const listPluginLifecycleProfilesTool = {
-    name: 'list_plugin_lifecycle_profiles',
-    description: 'List registered plugin lifecycle profiles. Read-only.',
-    inputSchema: { type: 'object', additionalProperties: false, properties: {} },
-    async handler(_input, ctx) {
-        return { data: await listPluginLifecycleProfiles(ctx.inspection) };
-    },
-};
-export const getPluginLifecycleProfileTool = {
-    name: 'get_plugin_lifecycle_profile',
-    description: 'Get one plugin lifecycle profile by id. Read-only.',
-    inputSchema: {
-        type: 'object',
-        additionalProperties: false,
-        required: ['id'],
-        properties: { id: { type: 'string' } },
-    },
-    async handler(input, ctx) {
-        const id = typeof input.id === 'string' ? input.id : '';
-        if (!id)
-            return { isError: true, error: { code: 'invalid-input', message: 'id is required.' } };
-        const entries = await listPluginLifecycleProfiles(ctx.inspection);
-        const entry = entries.find((e) => e.profile.id === id);
-        if (!entry) {
-            return { isError: true, error: { code: 'not-found', message: `Unknown lifecycle profile "${id}".` } };
-        }
-        return { data: entry };
-    },
-};
-export const getPluginLifecycleProfileDoctorTool = {
-    name: 'get_plugin_lifecycle_profile_doctor',
-    description: 'Health check for plugin lifecycle profiles. Read-only.',
-    inputSchema: {
-        type: 'object',
-        additionalProperties: false,
-        properties: { id: { type: 'string' } },
-    },
-    async handler(input, ctx) {
-        const id = typeof input.id === 'string' ? input.id : undefined;
-        const entries = await listPluginLifecycleProfiles(ctx.inspection);
-        const targets = id ? entries.filter((e) => e.profile.id === id) : entries;
-        if (id && targets.length === 0) {
-            return { isError: true, error: { code: 'not-found', message: `Unknown lifecycle profile "${id}".` } };
-        }
-        const health = {};
-        for (const e of targets) {
-            health[e.profile.id] = checkPluginLifecycleProfileHealth(ctx.cwd, e.profile);
-        }
-        return { data: { health } };
-    },
-};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shrkcrft/mcp-server",
-  "version": "0.1.0-alpha.7",
+  "version": "0.1.0-alpha.8",
   "description": "SharkCraft MCP server: 25 tools over @modelcontextprotocol/sdk's stdio transport.",
   "license": "MIT",
   "author": "SharkCraft contributors",
@@ -44,20 +44,20 @@
     "typecheck": "tsc --noEmit -p tsconfig.json"
   },
   "dependencies": {
-    "@shrkcrft/core": "^0.1.0-alpha.7",
-    "@shrkcrft/config": "^0.1.0-alpha.7",
-    "@shrkcrft/workspace": "^0.1.0-alpha.7",
-    "@shrkcrft/knowledge": "^0.1.0-alpha.7",
-    "@shrkcrft/context": "^0.1.0-alpha.7",
-    "@shrkcrft/rules": "^0.1.0-alpha.7",
-    "@shrkcrft/paths": "^0.1.0-alpha.7",
-    "@shrkcrft/templates": "^0.1.0-alpha.7",
-    "@shrkcrft/pipelines": "^0.1.0-alpha.7",
-    "@shrkcrft/presets": "^0.1.0-alpha.7",
-    "@shrkcrft/boundaries": "^0.1.0-alpha.7",
-    "@shrkcrft/packs": "^0.1.0-alpha.7",
-    "@shrkcrft/generator": "^0.1.0-alpha.7",
-    "@shrkcrft/inspector": "^0.1.0-alpha.7",
+    "@shrkcrft/core": "^0.1.0-alpha.8",
+    "@shrkcrft/config": "^0.1.0-alpha.8",
+    "@shrkcrft/workspace": "^0.1.0-alpha.8",
+    "@shrkcrft/knowledge": "^0.1.0-alpha.8",
+    "@shrkcrft/context": "^0.1.0-alpha.8",
+    "@shrkcrft/rules": "^0.1.0-alpha.8",
+    "@shrkcrft/paths": "^0.1.0-alpha.8",
+    "@shrkcrft/templates": "^0.1.0-alpha.8",
+    "@shrkcrft/pipelines": "^0.1.0-alpha.8",
+    "@shrkcrft/presets": "^0.1.0-alpha.8",
+    "@shrkcrft/boundaries": "^0.1.0-alpha.8",
+    "@shrkcrft/packs": "^0.1.0-alpha.8",
+    "@shrkcrft/generator": "^0.1.0-alpha.8",
+    "@shrkcrft/inspector": "^0.1.0-alpha.8",
     "@modelcontextprotocol/sdk": "^1.0.0",
     "zod": "^3.25.0 || ^4.0.0"
   },

package/dist/tools/r28-plugin-lifecycle.tool.d.ts

@@ -1,4 +0,0 @@
-import type { IToolDefinition } from '../server/tool-definition.js';
-export declare const previewPluginRenameTool: IToolDefinition;
-export declare const previewPluginRemoveTool: IToolDefinition;
-//# sourceMappingURL=r28-plugin-lifecycle.tool.d.ts.map

package/dist/tools/r28-plugin-lifecycle.tool.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"r28-plugin-lifecycle.tool.d.ts","sourceRoot":"","sources":["../../src/tools/r28-plugin-lifecycle.tool.ts"],"names":[],"mappings":"AAkBA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAMpE,eAAO,MAAM,uBAAuB,EAAE,eA0CrC,CAAC;AAEF,eAAO,MAAM,uBAAuB,EAAE,eAuCrC,CAAC"}

package/dist/tools/r28-plugin-lifecycle.tool.js

@@ -1,94 +0,0 @@
-/**
- * Plan-only plugin lifecycle previews (profile-driven).
- *
- *  preview_plugin_rename — returns the rename plan (replace ops + manual steps).
- *  preview_plugin_remove — returns the destructive remove plan.
- *
- *  Both tools never write source — the human runs `shrk plugin rename|remove`
- *  to produce the saved plan and applies it via `shrk apply --verify-signature`.
- *
- *  A `profile` input is required. If exactly one lifecycle profile is
- *  registered, the tool implicitly uses it; otherwise the caller must supply
- *  `profile`.
- */
-import { buildPluginRemovePlan, buildPluginRenamePlan, resolvePluginLifecycleProfile, } from '@shrkcrft/inspector';
-function nextHint(cmd) {
-    return `Next: \`${cmd}\` (CLI is the only write path).`;
-}
-export const previewPluginRenameTool = {
-    name: 'preview_plugin_rename',
-    description: 'Preview a plugin rename plan. Read-only — returns the structured plan only. Requires a registered plugin lifecycle profile (pass `profile` if more than one is registered).',
-    inputSchema: {
-        type: 'object',
-        additionalProperties: false,
-        required: ['oldName', 'newName'],
-        properties: {
-            oldName: { type: 'string' },
-            newName: { type: 'string' },
-            profile: { type: 'string' },
-        },
-    },
-    async handler(input, ctx) {
-        const oldName = String(input.oldName ?? '');
-        const newName = String(input.newName ?? '');
-        const profileId = typeof input.profile === 'string' ? input.profile : undefined;
-        const resolved = await resolvePluginLifecycleProfile(ctx.inspection, {
-            profileId,
-            allowSingleDefault: true,
-        });
-        if (!resolved.entry) {
-            return {
-                isError: true,
-                error: { code: 'profile-required', message: resolved.error ?? 'Profile resolution failed' },
-                data: { availableProfiles: resolved.availableIds },
-            };
-        }
-        const plan = buildPluginRenamePlan({
-            projectRoot: ctx.cwd,
-            profile: resolved.entry.profile,
-            oldName,
-            newName,
-        });
-        return {
-            text: nextHint(`shrk plugin rename ${oldName} ${newName} --profile ${resolved.entry.profile.id} --output /tmp/plan.json`),
-            data: plan,
-        };
-    },
-};
-export const previewPluginRemoveTool = {
-    name: 'preview_plugin_remove',
-    description: 'Preview a destructive plugin remove plan. Read-only — returns the structured plan only. Requires a registered plugin lifecycle profile (pass `profile` if more than one is registered).',
-    inputSchema: {
-        type: 'object',
-        additionalProperties: false,
-        required: ['name'],
-        properties: {
-            name: { type: 'string' },
-            profile: { type: 'string' },
-        },
-    },
-    async handler(input, ctx) {
-        const name = String(input.name ?? '');
-        const profileId = typeof input.profile === 'string' ? input.profile : undefined;
-        const resolved = await resolvePluginLifecycleProfile(ctx.inspection, {
-            profileId,
-            allowSingleDefault: true,
-        });
-        if (!resolved.entry) {
-            return {
-                isError: true,
-                error: { code: 'profile-required', message: resolved.error ?? 'Profile resolution failed' },
-                data: { availableProfiles: resolved.availableIds },
-            };
-        }
-        const plan = buildPluginRemovePlan({
-            projectRoot: ctx.cwd,
-            profile: resolved.entry.profile,
-            oldName: name,
-        });
-        return {
-            text: nextHint(`shrk plugin remove ${name} --profile ${resolved.entry.profile.id} --output /tmp/plan.json`),
-            data: plan,
-        };
-    },
-};