@okfit/mcp 0.5.1 → 0.5.3

package/README.md

@@ -41,7 +41,7 @@ or `.config/okfit.toml`.
 | `get_concept` | One concept by id: its whole decoded frontmatter, the file's raw markdown text, its bundle-relative path, and every outgoing link. |
 | `concept_neighbors` | A concept's graph neighbours — everything it links to and everything that links to it — each with its node kind and, for a concept target, its full summary. |
 | `stale_report` | Every concept whose `stale_after` instant has passed, each with its summary and how many whole days past it, as of now or an explicit instant. |
-| `validate_bundle` | The same conformance and lint report `okfit validate --format json` produces, unchanged. Its `engine_version` and `okf_version` match the CLI's over one bundle; `okfit_version` is this package's own version and `producer` is `@okfit/mcp`. |
+| `validate_bundle` | The same conformance and lint report `okfit validate --format json` produces, unchanged. Its `engine_version` and `okf_version` match the CLI's over one bundle; `okfit_version` is this package's own version and `producer` is `@okfit/mcp`. Optional `documents: [{ path, text }]` validates unsaved text (bundle-relative paths, a new file only under an existing directory) without writing it. |
 
 ## Resources
 

package/errors.js

@@ -16,7 +16,7 @@ const Remediation = Schema.Struct({
 * `{ isError: true, content: [{ type: "text", text: error.message }] }`
 * and never surfaces `structuredContent` for it
 * (`.repos/effect/packages/effect/src/unstable/ai/McpServer.ts:1774-1778,1842-1846`
-* at effect@4.0.0-rc.116: `toolErrorResult` and `declaredFailureResult`'s
+* at `effect@4.0.0-rc.116`: `toolErrorResult` and `declaredFailureResult`'s
 * `error instanceof Error` branch; confirmed empirically in Task B1's own
 * build). Since rc.116 that declared branch is rendered WITHOUT a log line
 * — only an internal failure goes through `Effect.logError` — so the

package/index.d.ts

@@ -22,7 +22,7 @@ export type Remediation = typeof Remediation.Type;
  * `{ isError: true, content: [{ type: "text", text: error.message }] }`
  * and never surfaces `structuredContent` for it
  * (`.repos/effect/packages/effect/src/unstable/ai/McpServer.ts:1774-1778,1842-1846`
- * at effect@4.0.0-rc.116: `toolErrorResult` and `declaredFailureResult`'s
+ * at `effect@4.0.0-rc.116`: `toolErrorResult` and `declaredFailureResult`'s
  * `error instanceof Error` branch; confirmed empirically in Task B1's own
  * build). Since rc.116 that declared branch is rendered WITHOUT a log line
  * — only an internal failure goes through `Effect.logError` — so the
@@ -356,9 +356,13 @@ export declare const StaleReportSuccess: Schema.Struct<{
 }>;
 /** @public */
 export type StaleReportSuccess = typeof StaleReportSuccess.Type;
-/** `validate_bundle`'s arguments: the same shape and decode path as stale_report. @public */
+/** `validate_bundle`'s arguments: stale_report's `now`, plus optional unsaved `documents` (spec 4.5). @public */
 export declare const ValidateBundleParams: Schema.Struct<{
   readonly now: Schema.optionalKey<Schema.String>;
+  readonly documents: Schema.optionalKey<Schema.$Array<Schema.Struct<{
+    readonly path: Schema.String;
+    readonly text: Schema.String;
+  }>>>;
 }>;
 /** @public */
 export type ValidateBundleParams = typeof ValidateBundleParams.Type;
@@ -584,6 +588,10 @@ export declare const OkfitToolkit: Toolkit.Toolkit<{
   readonly validate_bundle: import("effect/unstable/ai/Tool").Tool<"validate_bundle", {
     readonly parameters: import("effect/Schema").Struct<{
       readonly now: import("effect/Schema").optionalKey<import("effect/Schema").String>;
+      readonly documents: import("effect/Schema").optionalKey<import("effect/Schema").$Array<import("effect/Schema").Struct<{
+        readonly path: import("effect/Schema").String;
+        readonly text: import("effect/Schema").String;
+      }>>>;
     }>;
     readonly success: import("effect/Schema").Struct<{
       readonly schema: import("effect/Schema").Literal<1>;
@@ -770,6 +778,10 @@ export declare const ToolsLayer: (projectRoot: string, distribution?: Distributi
   readonly validate_bundle: import("effect/unstable/ai/Tool").Tool<"validate_bundle", {
     readonly parameters: import("effect/Schema").Struct<{
       readonly now: import("effect/Schema").optionalKey<import("effect/Schema").String>;
+      readonly documents: import("effect/Schema").optionalKey<import("effect/Schema").$Array<import("effect/Schema").Struct<{
+        readonly path: import("effect/Schema").String;
+        readonly text: import("effect/Schema").String;
+      }>>>;
     }>;
     readonly success: import("effect/Schema").Struct<{
       readonly schema: import("effect/Schema").Literal<1>;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@okfit/mcp",
-	"version": "0.5.1",
+	"version": "0.5.3",
 	"private": false,
 	"description": "Model Context Protocol server for okfit: query and understand Open Knowledge Format (OKF) bundles from an agent.",
 	"keywords": [
@@ -44,13 +44,13 @@
 		"okfit-mcp": "bin/okfit-mcp.js"
 	},
 	"dependencies": {
-		"@effect/platform-node": "4.0.0-rc.116",
-		"@effected/git": "^0.16.0",
-		"@effected/xdg": "^0.6.1",
-		"@okfit/core": "0.7.3",
-		"@okfit/engine": "0.7.4",
-		"@okfit/profiles": "0.7.4",
-		"effect": "4.0.0-rc.116"
+		"@effect/platform-node": "4.0.0-rc.117",
+		"@effected/git": "^0.17.0",
+		"@effected/xdg": "^0.7.0",
+		"@okfit/core": "0.8.0",
+		"@okfit/engine": "0.8.0",
+		"@okfit/profiles": "0.8.0",
+		"effect": "4.0.0-rc.117"
 	},
 	"engines": {
 		"node": ">=24.11.0"

package/schema/tools.js

@@ -71,8 +71,16 @@ const StaleReportSuccess = Schema.Struct({
 		days_past: Schema.Int
 	}))
 });
-/** `validate_bundle`'s arguments: the same shape and decode path as stale_report. @public */
-const ValidateBundleParams = Schema.Struct({ now: Schema.optionalKey(Schema.String) });
+/** One unsaved document for `validate_bundle`: bundle-relative posix `path`, full `text`. @public */
+const ValidateBundleDocument = Schema.Struct({
+	path: Schema.String,
+	text: Schema.String
+});
+/** `validate_bundle`'s arguments: stale_report's `now`, plus optional unsaved `documents` (spec 4.5). @public */
+const ValidateBundleParams = Schema.Struct({
+	now: Schema.optionalKey(Schema.String),
+	documents: Schema.optionalKey(Schema.Array(ValidateBundleDocument))
+});
 
 //#endregion
-export { ConceptNeighborsSuccess, DescribeVocabularySuccess, GetConceptSuccess, ListConceptsParams, ListConceptsSuccess, Neighbor, StaleReportParams, StaleReportSuccess, ValidateBundleParams };
+export { ConceptNeighborsSuccess, DescribeVocabularySuccess, GetConceptSuccess, ListConceptsParams, ListConceptsSuccess, Neighbor, StaleReportParams, StaleReportSuccess, ValidateBundleDocument, ValidateBundleParams };

package/tools/validateBundle.js

@@ -1,4 +1,4 @@
-import { BundleNotFound, McpToolError, composeRemediatedMessage } from "../errors.js";
+import { BundleNotFound, InvalidArgument, McpToolError, composeRemediatedMessage } from "../errors.js";
 import { resolveConfigOnly } from "../internal/toolContext.js";
 import { ValidateBundleParams } from "../schema/tools.js";
 import { resolveNow } from "../internal/resolveNow.js";
@@ -6,13 +6,13 @@ import { MCP_VERSION } from "../version.js";
 import { Crypto, Effect, FileSystem, Option, Path } from "effect";
 import { OKF_SPEC_VERSION } from "@okfit/core";
 import { Tool } from "effect/unstable/ai";
-import { JsonEnvelope, collect, forDiagnostics, json, run } from "@okfit/engine";
+import { DocumentPathError, JsonEnvelope, collect, forDiagnostics, json, provideDocuments, run } from "@okfit/engine";
 import { Git } from "@effected/git";
 import { GitHistory } from "@okfit/profiles";
 import { AppDirs, Xdg } from "@effected/xdg";
 
 //#region src/tools/validateBundle.ts
-const DESCRIPTION = "Runs the same conformance and lint checks as `okfit validate --format json` and returns that report unchanged: every diagnostic's file, code, severity, message and range, plus the summary counts and the exit code the CLI would use. Call it after writing or editing any concept file, before moving on to the next one.";
+const DESCRIPTION = "Returns the report `okfit validate --format json` prints, unchanged: each diagnostic's file, code, severity, message and range, the summary counts and the CLI's exit code. Call it after editing any concept file. Optional documents: [{ path, text }] (bundle-relative, e.g. metrics/churn.md) stand in for disk, a file not yet written under an existing directory included; nothing is written.";
 /**
 * `dependencies` mirrors the other tools' (Task B1's Deviation 1):
 * `resolveConfigOnly` and `run()` need the same platform services, and
@@ -75,7 +75,15 @@ const handleValidateBundle = (projectRoot, params, distribution) => Effect.gen(f
 		config: resolved.config,
 		profile: resolved.profile,
 		now
-	}).pipe(Effect.mapError((cause) => {
+	}).pipe(provideDocuments(resolved.bundleRoot, params.documents ?? []), Effect.mapError((cause) => {
+		if (cause instanceof DocumentPathError) {
+			const remediation = { hint: "Give each document path relative to the bundle root, in posix form, naming a .md file once (for example metrics/churn.md)." };
+			return new InvalidArgument({
+				argument: "documents",
+				message: composeRemediatedMessage(cause.message, remediation),
+				remediation
+			});
+		}
 		const remediation = { hint: `The bundle root "${resolved.bundleRoot}" does not exist or could not be read; check the config's [bundle].path, or run \`okfit init\`.` };
 		return new BundleNotFound({
 			root: resolved.bundleRoot,

package/version.js

@@ -5,7 +5,7 @@
 *
 * @public
 */
-const MCP_VERSION = "0.5.1";
+const MCP_VERSION = "0.5.3";
 
 //#endregion
 export { MCP_VERSION };