@adhisang/minecraft-modding-mcp 6.3.0 → 7.0.0-rc.1

package/dist/entry-tools/manage-cache-service.d.ts

@@ -1,123 +1,113 @@
 import { z } from "zod";
 import { type CacheRegistry } from "../cache-registry.js";
 export declare const manageCacheShape: {
-    action: z.ZodEnum<["summary", "list", "inspect", "delete", "prune", "rebuild", "verify"]>;
-    cacheKinds: z.ZodOptional<z.ZodArray<z.ZodEnum<["artifact-index", "downloads", "mapping", "registry", "decompiled-source", "mod-remap", "binary-remap", "nested-jar", "workspace"]>, "many">>;
+    action: z.ZodEnum<{
+        summary: "summary";
+        list: "list";
+        inspect: "inspect";
+        delete: "delete";
+        prune: "prune";
+        rebuild: "rebuild";
+        verify: "verify";
+    }>;
+    cacheKinds: z.ZodOptional<z.ZodArray<z.ZodEnum<{
+        workspace: "workspace";
+        "artifact-index": "artifact-index";
+        downloads: "downloads";
+        mapping: "mapping";
+        registry: "registry";
+        "decompiled-source": "decompiled-source";
+        "mod-remap": "mod-remap";
+        "binary-remap": "binary-remap";
+        "nested-jar": "nested-jar";
+    }>>>;
     selector: z.ZodOptional<z.ZodObject<{
         artifactId: z.ZodOptional<z.ZodString>;
         version: z.ZodOptional<z.ZodString>;
         jarPath: z.ZodOptional<z.ZodString>;
         entryId: z.ZodOptional<z.ZodString>;
-        status: z.ZodOptional<z.ZodEnum<["healthy", "partial", "stale", "orphaned", "corrupt", "in_use"]>>;
+        status: z.ZodOptional<z.ZodEnum<{
+            stale: "stale";
+            healthy: "healthy";
+            partial: "partial";
+            orphaned: "orphaned";
+            corrupt: "corrupt";
+            in_use: "in_use";
+        }>>;
         olderThan: z.ZodOptional<z.ZodString>;
         mapping: z.ZodOptional<z.ZodString>;
         scope: z.ZodOptional<z.ZodString>;
         projectPath: z.ZodOptional<z.ZodString>;
-    }, "strip", z.ZodTypeAny, {
-        version?: string | undefined;
-        projectPath?: string | undefined;
-        mapping?: string | undefined;
-        jarPath?: string | undefined;
-        entryId?: string | undefined;
-        artifactId?: string | undefined;
-        status?: "healthy" | "partial" | "stale" | "orphaned" | "corrupt" | "in_use" | undefined;
-        scope?: string | undefined;
-        olderThan?: string | undefined;
-    }, {
-        version?: string | undefined;
-        projectPath?: string | undefined;
-        mapping?: string | undefined;
-        jarPath?: string | undefined;
-        entryId?: string | undefined;
-        artifactId?: string | undefined;
-        status?: "healthy" | "partial" | "stale" | "orphaned" | "corrupt" | "in_use" | undefined;
-        scope?: string | undefined;
-        olderThan?: string | undefined;
+    }, z.core.$strip>>;
+    detail: z.ZodOptional<z.ZodEnum<{
+        summary: "summary";
+        standard: "standard";
+        full: "full";
     }>>;
-    detail: z.ZodOptional<z.ZodEnum<["summary", "standard", "full"]>>;
-    include: z.ZodOptional<z.ZodArray<z.ZodEnum<[string, ...string[]]>, "many">>;
+    include: z.ZodOptional<z.ZodArray<z.ZodEnum<{
+        [x: string]: string;
+    }>>>;
     limit: z.ZodDefault<z.ZodNumber>;
     cursor: z.ZodOptional<z.ZodString>;
-    executionMode: z.ZodDefault<z.ZodEnum<["preview", "apply"]>>;
+    executionMode: z.ZodDefault<z.ZodEnum<{
+        preview: "preview";
+        apply: "apply";
+    }>>;
 };
 export declare const manageCacheSchema: z.ZodObject<{
-    action: z.ZodEnum<["summary", "list", "inspect", "delete", "prune", "rebuild", "verify"]>;
-    cacheKinds: z.ZodOptional<z.ZodArray<z.ZodEnum<["artifact-index", "downloads", "mapping", "registry", "decompiled-source", "mod-remap", "binary-remap", "nested-jar", "workspace"]>, "many">>;
+    action: z.ZodEnum<{
+        summary: "summary";
+        list: "list";
+        inspect: "inspect";
+        delete: "delete";
+        prune: "prune";
+        rebuild: "rebuild";
+        verify: "verify";
+    }>;
+    cacheKinds: z.ZodOptional<z.ZodArray<z.ZodEnum<{
+        workspace: "workspace";
+        "artifact-index": "artifact-index";
+        downloads: "downloads";
+        mapping: "mapping";
+        registry: "registry";
+        "decompiled-source": "decompiled-source";
+        "mod-remap": "mod-remap";
+        "binary-remap": "binary-remap";
+        "nested-jar": "nested-jar";
+    }>>>;
     selector: z.ZodOptional<z.ZodObject<{
         artifactId: z.ZodOptional<z.ZodString>;
         version: z.ZodOptional<z.ZodString>;
         jarPath: z.ZodOptional<z.ZodString>;
         entryId: z.ZodOptional<z.ZodString>;
-        status: z.ZodOptional<z.ZodEnum<["healthy", "partial", "stale", "orphaned", "corrupt", "in_use"]>>;
+        status: z.ZodOptional<z.ZodEnum<{
+            stale: "stale";
+            healthy: "healthy";
+            partial: "partial";
+            orphaned: "orphaned";
+            corrupt: "corrupt";
+            in_use: "in_use";
+        }>>;
         olderThan: z.ZodOptional<z.ZodString>;
         mapping: z.ZodOptional<z.ZodString>;
         scope: z.ZodOptional<z.ZodString>;
         projectPath: z.ZodOptional<z.ZodString>;
-    }, "strip", z.ZodTypeAny, {
-        version?: string | undefined;
-        projectPath?: string | undefined;
-        mapping?: string | undefined;
-        jarPath?: string | undefined;
-        entryId?: string | undefined;
-        artifactId?: string | undefined;
-        status?: "healthy" | "partial" | "stale" | "orphaned" | "corrupt" | "in_use" | undefined;
-        scope?: string | undefined;
-        olderThan?: string | undefined;
-    }, {
-        version?: string | undefined;
-        projectPath?: string | undefined;
-        mapping?: string | undefined;
-        jarPath?: string | undefined;
-        entryId?: string | undefined;
-        artifactId?: string | undefined;
-        status?: "healthy" | "partial" | "stale" | "orphaned" | "corrupt" | "in_use" | undefined;
-        scope?: string | undefined;
-        olderThan?: string | undefined;
+    }, z.core.$strip>>;
+    detail: z.ZodOptional<z.ZodEnum<{
+        summary: "summary";
+        standard: "standard";
+        full: "full";
     }>>;
-    detail: z.ZodOptional<z.ZodEnum<["summary", "standard", "full"]>>;
-    include: z.ZodOptional<z.ZodArray<z.ZodEnum<[string, ...string[]]>, "many">>;
+    include: z.ZodOptional<z.ZodArray<z.ZodEnum<{
+        [x: string]: string;
+    }>>>;
     limit: z.ZodDefault<z.ZodNumber>;
     cursor: z.ZodOptional<z.ZodString>;
-    executionMode: z.ZodDefault<z.ZodEnum<["preview", "apply"]>>;
-}, "strip", z.ZodTypeAny, {
-    limit: number;
-    executionMode: "preview" | "apply";
-    action: "summary" | "list" | "inspect" | "verify" | "delete" | "prune" | "rebuild";
-    cursor?: string | undefined;
-    detail?: "summary" | "standard" | "full" | undefined;
-    include?: string[] | undefined;
-    cacheKinds?: ("workspace" | "artifact-index" | "downloads" | "mapping" | "registry" | "decompiled-source" | "mod-remap" | "binary-remap" | "nested-jar")[] | undefined;
-    selector?: {
-        version?: string | undefined;
-        projectPath?: string | undefined;
-        mapping?: string | undefined;
-        jarPath?: string | undefined;
-        entryId?: string | undefined;
-        artifactId?: string | undefined;
-        status?: "healthy" | "partial" | "stale" | "orphaned" | "corrupt" | "in_use" | undefined;
-        scope?: string | undefined;
-        olderThan?: string | undefined;
-    } | undefined;
-}, {
-    action: "summary" | "list" | "inspect" | "verify" | "delete" | "prune" | "rebuild";
-    limit?: number | undefined;
-    cursor?: string | undefined;
-    detail?: "summary" | "standard" | "full" | undefined;
-    include?: string[] | undefined;
-    executionMode?: "preview" | "apply" | undefined;
-    cacheKinds?: ("workspace" | "artifact-index" | "downloads" | "mapping" | "registry" | "decompiled-source" | "mod-remap" | "binary-remap" | "nested-jar")[] | undefined;
-    selector?: {
-        version?: string | undefined;
-        projectPath?: string | undefined;
-        mapping?: string | undefined;
-        jarPath?: string | undefined;
-        entryId?: string | undefined;
-        artifactId?: string | undefined;
-        status?: "healthy" | "partial" | "stale" | "orphaned" | "corrupt" | "in_use" | undefined;
-        scope?: string | undefined;
-        olderThan?: string | undefined;
-    } | undefined;
-}>;
+    executionMode: z.ZodDefault<z.ZodEnum<{
+        preview: "preview";
+        apply: "apply";
+    }>>;
+}, z.core.$strip>;
 export type ManageCacheInput = z.infer<typeof manageCacheSchema>;
 export declare class ManageCacheService {
     private readonly deps;

package/dist/entry-tools/validate-project/cases/mixin.js

@@ -21,14 +21,34 @@ export async function handleMixin(deps, input, detail, include) {
             details: {
                 task: "mixin",
                 failedStage: "input-validation",
-                nextAction: "Pass version explicitly (e.g. \"1.21.10\"). task=\"project-summary\" supports preferProjectVersion for auto-detection from gradle.properties, but direct task=\"mixin\" requires an explicit version.",
+                nextAction: "Pass version explicitly: the Minecraft version this project targets. task=\"project-summary\" supports preferProjectVersion for auto-detection from gradle.properties, but direct task=\"mixin\" requires an explicit version. Call the suggested list-versions to see what is available, then replay the exampleCalls template with that version substituted.",
+                // No concrete version can be derived here -- the caller omitted it and this
+                // task does not auto-detect one. A `suggestedCall` is a payload the caller
+                // may replay verbatim, so filling the hole with a made-up version produced
+                // a call that RUNS and validates the mixin against the wrong Minecraft
+                // version, which is worse than no suggestion.
+                //
+                // So the two roles are split, following what the sibling "a version is
+                // required but none was resolved" site already does in
+                // src/source/class-source.ts: `suggestedCall` is a REAL next step that
+                // needs nothing the caller does not have (list-versions takes no
+                // arguments and answers exactly the question blocking them), while the
+                // task="mixin" retry shape travels as an `exampleCalls` template whose
+                // placeholder makes the substitution the caller must perform obvious.
+                ...buildSuggestedCall({ tool: "list-versions", params: {} }),
                 ...buildSuggestedCall({
                     tool: "validate-project",
-                    params: {
-                        task: "mixin",
-                        subject: input.subject,
-                        version: "1.21.10"
-                    }
+                    params: undefined,
+                    examples: [
+                        {
+                            params: {
+                                task: "mixin",
+                                subject: input.subject,
+                                version: "<your-mc-version>"
+                            },
+                            reason: "Replace <your-mc-version> with the Minecraft version this project targets (gradle.properties, or task=\"project-summary\" with preferProjectVersion=true reports it)."
+                        }
+                    ]
                 })
             }
         });

package/dist/entry-tools/validate-project/cases/project-summary.d.ts

@@ -9,7 +9,7 @@ export declare function handleProjectSummary(deps: ValidateProjectDeps, input: V
     warnings: string[];
     tasks?: {
         "workspace.detected": {
-            status: "error" | "ok" | "skipped" | "missing";
+            status: "ok" | "error" | "missing" | "skipped";
             durationMs?: number;
             error?: {
                 code: string;
@@ -20,7 +20,7 @@ export declare function handleProjectSummary(deps: ValidateProjectDeps, input: V
             evidence?: string[];
         };
         "gradle.readable": {
-            status: "error" | "ok" | "skipped" | "missing";
+            status: "ok" | "error" | "missing" | "skipped";
             durationMs?: number;
             error?: {
                 code: string;
@@ -32,7 +32,7 @@ export declare function handleProjectSummary(deps: ValidateProjectDeps, input: V
             buildScripts?: string[];
         };
         "loom.cache.found": {
-            status: "error" | "ok" | "skipped" | "missing";
+            status: "ok" | "error" | "missing" | "skipped";
             durationMs?: number;
             error?: {
                 code: string;
@@ -43,7 +43,7 @@ export declare function handleProjectSummary(deps: ValidateProjectDeps, input: V
             cachePath?: string;
         };
         "minecraft.artifact.resolved": {
-            status: "error" | "ok" | "skipped" | "missing";
+            status: "ok" | "error" | "missing" | "skipped";
             durationMs?: number;
             error?: {
                 code: string;
@@ -55,7 +55,7 @@ export declare function handleProjectSummary(deps: ValidateProjectDeps, input: V
             mapping?: import("../../../types.js").SourceMapping;
         };
         "mixins.validated": {
-            status: "error" | "ok" | "skipped" | "missing";
+            status: "ok" | "error" | "missing" | "skipped";
             durationMs?: number;
             error?: {
                 code: string;
@@ -70,7 +70,7 @@ export declare function handleProjectSummary(deps: ValidateProjectDeps, input: V
             };
         };
         "accessWideners.validated": {
-            status: "error" | "ok" | "skipped" | "missing";
+            status: "ok" | "error" | "missing" | "skipped";
             durationMs?: number;
             error?: {
                 code: string;
@@ -84,7 +84,7 @@ export declare function handleProjectSummary(deps: ValidateProjectDeps, input: V
             };
         };
         "accessTransformers.validated": {
-            status: "error" | "ok" | "skipped" | "missing";
+            status: "ok" | "error" | "missing" | "skipped";
             durationMs?: number;
             error?: {
                 code: string;