@savvy-web/mcp 1.6.7 → 1.7.1

package/index.d.ts

@@ -1,6 +1,7 @@
 import { Changesets, SilkWorkspaceAnalyzer, Turbo } from "@savvy-web/silk-effects";
 import { Layer, ManagedRuntime } from "effect";
 import { WorkspaceDiscoveryError, WorkspaceRoot } from "workspaces-effect";
+import "@modelcontextprotocol/sdk/server/mcp.js";
 //#region src/context.d.ts
 /** The long-lived runtime and the project working directory. */
 interface McpContext {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@savvy-web/mcp",
-	"version": "1.6.7",
+	"version": "1.7.1",
 	"private": false,
 	"description": "The savvy MCP server — Silk Suite tooling and library knowledge for coding agents",
 	"homepage": "https://github.com/savvy-web/systems/tree/main/packages/mcp",
@@ -38,7 +38,7 @@
 		"@effect/sql": "^0.51.1",
 		"@effect/workflow": "^0.18.2",
 		"@modelcontextprotocol/sdk": "^1.29.0",
-		"@savvy-web/silk-effects": "3.0.3",
+		"@savvy-web/silk-effects": "3.2.0",
 		"effect": "^3.21.4",
 		"workspaces-effect": "^2.0.2",
 		"zod": "^4.4.3"

package/server.js

@@ -100,6 +100,8 @@ function buildServer(ctx) {
 		inputSchema: {
 			base: z.optional(z.string()).describe("Override the base branch used to compute the merge-base."),
 			package: z.optional(z.string()).describe("Restrict output to a single workspace package."),
+			packages: z.optional(z.array(z.string())).describe("Restrict output to these workspace packages (unioned with package)."),
+			exclude: z.optional(z.array(z.string())).describe("Drop these packages from the output entirely."),
 			cwd: z.optional(z.string()).describe("Directory to resolve the workspace root from.")
 		},
 		outputSchema: effectToZodSchema(ChangesetDepsDetectResult),
@@ -124,6 +126,8 @@ function buildServer(ctx) {
 		inputSchema: {
 			base: z.optional(z.string()).describe("Override the base branch used to compute the merge-base."),
 			package: z.optional(z.string()).describe("Restrict regeneration to a single workspace package."),
+			packages: z.optional(z.array(z.string())).describe("Restrict regeneration to these workspace packages (unioned with package)."),
+			exclude: z.optional(z.array(z.string())).describe("Skip these packages entirely: nothing written, existing changesets untouched."),
 			dryRun: z.optional(z.boolean()).describe("Compute the plan without writing or deleting any file."),
 			cwd: z.optional(z.string()).describe("Directory to resolve the workspace root from.")
 		},
@@ -138,12 +142,13 @@ function buildServer(ctx) {
 		return structuredResult(text, data);
 	});
 	server.registerTool("biome_check", {
-		description: "Run Biome over a path and get structured diagnostics back. mode=check (default; lint + format + organize-imports) or mode=lint. Set write=true to apply safe fixes (--write), unsafe=true for unsafe fixes (--write --unsafe). Prefer this over shelling out to biome; the LSP already covers files you've edited. Returns markdown in content[] and a typed object in structuredContent. NOTE: with write/unsafe this tool MUTATES files (git-reversible).",
+		description: "Run Biome over a path and get structured diagnostics back. mode=check (default; lint + format + organize-imports) or mode=lint. Set write=true to apply safe fixes (--write), unsafe=true for unsafe fixes (--write --unsafe). Severities match the project's Biome config (what `biome check` reports); set strict=true to surface project warnings as errors, each marked with its originalSeverity. Prefer this over shelling out to biome; the LSP already covers files you've edited. Returns markdown in content[] and a typed object in structuredContent. NOTE: with write/unsafe this tool MUTATES files (git-reversible).",
 		inputSchema: {
 			paths: z.optional(z.array(z.string())).describe("Paths to check. Defaults to the whole workspace."),
 			mode: z.optional(z.enum(["check", "lint"])).describe("check = lint+format+imports (default); lint = lint only."),
 			write: z.optional(z.boolean()).describe("Apply safe fixes (--write)."),
 			unsafe: z.optional(z.boolean()).describe("Apply unsafe fixes (--write --unsafe); implies write."),
+			strict: z.optional(z.boolean()).describe("Report project warnings as errors (marked with originalSeverity). Default: honor project config."),
 			cwd: z.optional(z.string()).describe("Directory to resolve the workspace root from.")
 		},
 		outputSchema: effectToZodSchema(BiomeCheckResult)

package/tools/biome-check.js

@@ -21,13 +21,17 @@ const BiomeDiagnostic = Schema.Struct({
 	line: Schema.Number,
 	severity: BiomeSeverity,
 	rule: Schema.String,
-	message: Schema.String
+	message: Schema.String,
+	/** Present only when `strict` upgraded this diagnostic; holds the project-configured severity. */
+	originalSeverity: Schema.optional(BiomeSeverity)
 }).annotations({ identifier: "BiomeDiagnostic" });
 /** The `biome_check` tool result. */
 const BiomeCheckResult = Schema.Struct({
 	summary: Schema.Struct({
 		errors: Schema.Number,
-		warnings: Schema.Number
+		warnings: Schema.Number,
+		/** Count of project-level warnings reported as errors because `strict` was set. */
+		upgradedWarnings: Schema.optional(Schema.Number)
 	}),
 	diagnostics: Schema.Array(BiomeDiagnostic),
 	wrote: Schema.Boolean,
@@ -38,7 +42,11 @@ const BiomeCheckResult = Schema.Struct({
 	description: "Structured Biome diagnostics, with a flag for whether a --write pass ran."
 });
 /** Guardrail shown to the agent so it fixes code rather than silencing rules. */
-const GUIDANCE = "Fix the actual code. Do NOT disable rules or add config overrides to silence these.";
+const GUIDANCE_ERRORS = "Fix the actual code. Do NOT disable rules or add config overrides to silence these.";
+/** Guidance when only project-tolerated warnings remain (default, non-strict run). */
+const GUIDANCE_WARNINGS = "These are warnings under the project's Biome config — the project's own lint passes and they do not block commits or CI. Fix them when they sit in code you are already changing; do not churn unrelated files to silence them, and do NOT disable rules.";
+/** Suffix appended when strict mode upgraded project warnings to errors. */
+const GUIDANCE_STRICT_NOTE = "Diagnostics marked with originalSeverity are project-level warnings surfaced strictly by this run — they are not CI blockers.";
 /** Shape of a single diagnostic in Biome's `--reporter=gitlab` output. */
 const GitlabDiagnostic = Schema.Struct({
 	description: Schema.String,
@@ -82,22 +90,40 @@ const parseBiomeGitlab = (stdout) => {
 };
 /** Assemble the structured result from normalized diagnostics + the write flag. */
 const buildBiomeResult = (params) => {
+	const diagnostics = params.strict ? params.diagnostics.map((d) => d.severity === "warning" ? {
+		...d,
+		severity: "error",
+		originalSeverity: "warning"
+	} : d) : [...params.diagnostics];
+	const upgradedWarnings = diagnostics.filter((d) => d.originalSeverity === "warning").length;
+	const errors = diagnostics.filter((d) => d.severity === "error").length;
+	const warnings = diagnostics.filter((d) => d.severity === "warning").length;
+	const guidance = errors - upgradedWarnings > 0 ? params.strict && upgradedWarnings > 0 ? `${GUIDANCE_ERRORS} ${GUIDANCE_STRICT_NOTE}` : GUIDANCE_ERRORS : upgradedWarnings > 0 ? `${GUIDANCE_WARNINGS} ${GUIDANCE_STRICT_NOTE}` : GUIDANCE_WARNINGS;
 	return {
-		summary: {
-			errors: params.diagnostics.filter((d) => d.severity === "error").length,
-			warnings: params.diagnostics.filter((d) => d.severity === "warning").length
+		summary: params.strict ? {
+			errors,
+			warnings,
+			upgradedWarnings
+		} : {
+			errors,
+			warnings
 		},
-		diagnostics: [...params.diagnostics],
+		diagnostics,
 		wrote: params.wrote,
-		guidance: GUIDANCE
+		guidance: diagnostics.length === 0 ? GUIDANCE_ERRORS : guidance
 	};
 };
 /** Render the structured result as markdown. */
 const renderMarkdown = (data) => {
 	if (data.diagnostics.length === 0) return `# biome — clean\n\n✅ No remaining diagnostics.${data.wrote ? " A --write pass ran; check `git diff` for what changed." : ""}`;
-	const lines = [`# biome — ${data.summary.errors} error(s), ${data.summary.warnings} warning(s)`, ``];
+	const upgraded = data.summary.upgradedWarnings ?? 0;
+	const upgradedNote = upgraded > 0 ? ` (${upgraded} strict-upgraded from project warnings)` : "";
+	const lines = [`# biome — ${data.summary.errors} error(s)${upgradedNote}, ${data.summary.warnings} warning(s)`, ``];
 	if (data.wrote) lines.push(`A --write pass ran; the diagnostics below remain unfixed.`, ``);
-	for (const d of data.diagnostics) lines.push(`- \`${d.file}:${d.line}\` **${d.severity}** ${d.rule} — ${d.message}`);
+	for (const d of data.diagnostics) {
+		const severity = d.originalSeverity ? `${d.severity} (project ${d.originalSeverity}, strict)` : d.severity;
+		lines.push(`- \`${d.file}:${d.line}\` **${severity}** ${d.rule} — ${d.message}`);
+	}
 	lines.push(``, `---`, data.guidance);
 	return lines.join("\n");
 };
@@ -166,7 +192,6 @@ const runBiomeCheck = async (args, fallbackCwd) => {
 		...prefix,
 		mode,
 		"--reporter=gitlab",
-		"--error-on-warnings",
 		"--no-errors-on-unmatched",
 		...paths
 	], {
@@ -180,7 +205,8 @@ const runBiomeCheck = async (args, fallbackCwd) => {
 	if ((read.status ?? 0) > 1) throw new Error(`Biome failed (exit ${read.status}): ${(read.stderr ?? "").trim() || "unknown error"}`);
 	return buildBiomeResult({
 		diagnostics: parseBiomeGitlab(read.stdout ?? ""),
-		wrote
+		wrote,
+		...args.strict !== void 0 ? { strict: args.strict } : {}
 	});
 };
 

package/tools/changeset-deps-detect.js

@@ -75,7 +75,9 @@ const changesetDepsDetect = (args, fallbackCwd) => Effect.gen(function* () {
 			cwd: root,
 			includeDevDeps: true,
 			...args.base ? { base: args.base } : {},
-			...args.package ? { package: args.package } : {}
+			...args.package ? { package: args.package } : {},
+			...args.packages && args.packages.length > 0 ? { packages: args.packages } : {},
+			...args.exclude && args.exclude.length > 0 ? { exclude: args.exclude } : {}
 		})).toWrite.map((entry) => ({
 			package: entry.diff.package,
 			relativePath: entry.diff.relativePath,

package/tools/changeset-deps-regen.js

@@ -77,7 +77,9 @@ const changesetDepsRegen = (args, fallbackCwd) => Effect.gen(function* () {
 	const plan = yield* service.plan({
 		cwd: root,
 		...args.base ? { base: args.base } : {},
-		...args.package ? { package: args.package } : {}
+		...args.package ? { package: args.package } : {},
+		...args.packages && args.packages.length > 0 ? { packages: args.packages } : {},
+		...args.exclude && args.exclude.length > 0 ? { exclude: args.exclude } : {}
 	});
 	if (args.dryRun === true) return {
 		root,