@adhisang/minecraft-modding-mcp 6.2.0 → 7.0.0-rc.0

package/dist/tool-schemas.js

@@ -29,7 +29,30 @@ export const DECODE_COMPRESSIONS = ["none", "gzip", "auto"];
 export const ENCODE_COMPRESSIONS = ["none", "gzip"];
 export const nonEmptyString = z.string().trim().min(1);
 export const optionalNonEmptyString = z.string().trim().min(1).optional();
-export const optionalPositiveInt = z.number().int().positive().optional();
+/**
+ * zod3-parity integer check. zod3's `.int()` accepted every `Number.isInteger`
+ * value — including unsafe integers beyond 2^53, which downstream code clamps
+ * (e.g. src/version-service.ts) — while zod4's `.int()` rejects unsafe
+ * integers with a `too_big` issue, silently narrowing the frozen public
+ * acceptance contract. This check restores the exact zod3 acceptance domain
+ * and pushes a stock-zod4-shaped `invalid_type` issue for non-integer numbers
+ * so the ProblemDetails parity layer (src/tool-guidance.ts) maps it to zod3's
+ * exact "Expected integer, received float" bytes. `continue: true` mirrors
+ * zod3's non-aborting number checks (later min/max checks still report).
+ * Public zod API only: no zod internals, no z.config globals.
+ */
+export const zod3ParityIntCheck = (payload) => {
+    if (!Number.isInteger(payload.value)) {
+        payload.issues.push({
+            code: "invalid_type",
+            expected: "int",
+            input: payload.value,
+            message: "Invalid input: expected int, received number",
+            continue: true
+        });
+    }
+};
+export const optionalPositiveInt = z.number().check(zod3ParityIntCheck).positive().optional();
 export const gradleUserHomeSchema = optionalNonEmptyString.describe("Gradle user home for Loom cache lookups (overrides GRADLE_USER_HOME).");
 // Optional descriptor: "" and whitespace-only strings are normalized to undefined so that
 // tools with signatureMode="name-only" can accept "caller omitted descriptor" inputs whether
@@ -224,7 +247,7 @@ export const batchClassSourceShape = {
     scope: artifactScopeSchema.optional(),
     preferProjectVersion: z.boolean().optional(),
     strictVersion: z.boolean().optional(),
-    concurrency: z.number().int().min(1).max(8).optional().describe("1..8, default 4"),
+    concurrency: z.number().check(zod3ParityIntCheck).min(1).max(8).optional().describe("1..8, default 4"),
     failFast: z.boolean().optional().describe("default false"),
     detail: detailParam("summary"),
     include: responseIncludeParam,
@@ -280,7 +303,7 @@ export const batchClassMembersShape = {
     scope: artifactScopeSchema.optional(),
     preferProjectVersion: z.boolean().optional(),
     strictVersion: z.boolean().optional(),
-    concurrency: z.number().int().min(1).max(8).optional(),
+    concurrency: z.number().check(zod3ParityIntCheck).min(1).max(8).optional(),
     failFast: z.boolean().optional(),
     detail: detailParam("summary"),
     projection: memberProjectionSchema.optional().describe(MEMBER_PROJECTION_DESCRIPTION),
@@ -311,7 +334,7 @@ export const batchSymbolExistsShape = {
     scope: artifactScopeSchema.optional(),
     preferProjectVersion: z.boolean().optional(),
     strictVersion: z.boolean().optional(),
-    concurrency: z.number().int().min(1).max(8).optional(),
+    concurrency: z.number().check(zod3ParityIntCheck).min(1).max(8).optional(),
     failFast: z.boolean().optional(),
     detail: detailParam("summary"),
     include: responseIncludeParam,
@@ -342,7 +365,7 @@ export const batchMappingsShape = {
     sourcePriority: mappingSourcePrioritySchema.optional(),
     projectPath: optionalNonEmptyString,
     gradleUserHome: gradleUserHomeSchema,
-    concurrency: z.number().int().min(1).max(8).optional(),
+    concurrency: z.number().check(zod3ParityIntCheck).min(1).max(8).optional(),
     failFast: z.boolean().optional(),
     detail: detailParam("summary"),
     include: responseIncludeParam,
@@ -371,6 +394,7 @@ export const findClassShape = {
     className: nonEmptyString.describe("Simple name (e.g. Blocks) or fully-qualified name (e.g. net.minecraft.world.level.block.Blocks)"),
     artifactId: optionalNonEmptyString,
     target: sourceLookupTargetSchema.optional().describe(SOURCE_LOOKUP_TARGET_DESCRIPTION),
+    projectPath: optionalNonEmptyString.describe("Workspace root for dependency or workspace target resolution."),
     limit: optionalPositiveInt.describe("default 20, max 200")
 };
 export const findClassSchema = z.object(findClassShape).superRefine(requireExactlyOneArtifactRef);
@@ -739,12 +763,15 @@ const nbtPatchOperationSchema = z
 })
     .passthrough();
 export const nbtApplyJsonPatchShape = {
-    typedJson: z.unknown(),
+    // zod4 makes bare z.unknown() members presence-required; zod3 accepted a
+    // missing key (handler saw undefined). .optional() restores that contract.
+    typedJson: z.unknown().optional(),
     patch: z.array(nbtPatchOperationSchema).describe("RFC6902 operation array (add/remove/replace/test)")
 };
 export const nbtApplyJsonPatchSchema = z.object(nbtApplyJsonPatchShape);
 export const jsonToNbtShape = {
-    typedJson: z.unknown(),
+    // Same zod4 presence-required restoration as nbtApplyJsonPatchShape above.
+    typedJson: z.unknown().optional(),
     compression: encodeCompressionSchema.default("none")
 };
 export const jsonToNbtSchema = z.object(jsonToNbtShape);

package/dist/types.d.ts

@@ -99,10 +99,42 @@ export interface ArtifactProvenance {
      * lost to single-jar selection.
      */
     companionSourceJars?: string[];
+    /**
+     * Set when the resolver had to serve a jar built for a DIFFERENT Minecraft
+     * version than the caller asked for.
+     *
+     * Provenance used to echo the requested version here while the served jar was
+     * another version entirely, so nothing in the response could distinguish an
+     * exact hit from a fallback except a quality flag.
+     */
+    versionApproximation?: {
+        requestedVersion: string;
+        servedVersion?: string;
+        sourceJarPath: string;
+    };
 }
+/** Mod loader a runtime jar belongs to, inferred from its cache path. */
+export type RuntimeLoader = "fabric" | "forge" | "neoforge" | "unknown";
 export interface RuntimeValidationProvenance<TMapping extends RuntimeValidationNamespace = RuntimeValidationNamespace> {
+    /**
+     * The Minecraft version the served jar ACTUALLY carries.
+     *
+     * This used to echo the requested version even when a fallback served a
+     * different one, so a Fabric 1.21.11 access widener could be validated
+     * against a NeoForge 1.21.10 jar while provenance still claimed 1.21.11.
+     */
     version: string;
     jarPath: string;
+    /** Set only when the served jar is a DIFFERENT version than the caller asked for. */
+    requestedVersion?: string;
+    /** True whenever `requestedVersion` is present. */
+    versionApproximated?: boolean;
+    /** Loader the served jar belongs to, read from its path. */
+    servedLoader?: RuntimeLoader;
+    /** Loader the caller's workspace (or the file format) implies. */
+    expectedLoader?: RuntimeLoader;
+    /** True when `servedLoader` contradicts a KNOWN `expectedLoader`. */
+    loaderMismatch?: boolean;
     requestedScope?: ArtifactScope;
     appliedScope?: ArtifactScope;
     requestedMapping: TMapping;
@@ -143,9 +175,12 @@ export interface Config {
     maxNbtInputBytes: number;
     maxNbtInflatedBytes: number;
     maxNbtResponseBytes: number;
+    sqliteCacheKb: number;
+    sqliteMmapSize: number;
     tinyRemapperJarPath: string | undefined;
     remapTimeoutMs: number;
     remapMaxMemoryMb: number;
+    decompileMaxMemoryMb: number;
 }
 export interface ArtifactSignature {
     sourcePath: string;

package/dist/v1-parity-schemas.d.ts

@@ -0,0 +1,7 @@
+/**
+ * Pre-migration (MCP SDK v1) advertised inputSchema per tool, keyed by tool
+ * name. Served verbatim through the registration adapter's jsonSchema
+ * Converter so tools/list advertisement stays byte-identical across the
+ * v1 -> v2 migration.
+ */
+export declare const V1_PARITY_SCHEMAS: Readonly<Record<string, Record<string, unknown>>>;