@adhisang/minecraft-modding-mcp 3.0.0 → 3.1.1

package/dist/entry-tools/compare-minecraft-service.js

@@ -1,7 +1,7 @@
 import { z } from "zod";
 import { createError, ERROR_CODES } from "../errors.js";
 import { buildIncludeSchema, detailSchema, positiveIntSchema } from "./entry-tool-schema.js";
-import { buildEntryToolMeta, buildEntryToolResult, createNextAction, createTruncationMeta } from "./response-contract.js";
+import { buildEntryToolMeta, buildEntryToolResult, createNextAction, createSummarySubject, createTruncationMeta } from "./response-contract.js";
 import { capArray, resolveDetail, resolveInclude } from "./request-normalizers.js";
 const nonEmptyString = z.string().trim().min(1);
 const INCLUDE_GROUPS = ["warnings", "classes", "registry", "diff", "samples", "timings"];
@@ -33,9 +33,9 @@ export const compareMinecraftShape = {
     detail: detailSchema.optional(),
     include: buildIncludeSchema(INCLUDE_GROUPS),
     limit: positiveIntSchema.optional(),
-    maxClassResults: positiveIntSchema.optional(),
+    maxClassResults: positiveIntSchema.default(500),
     maxEntriesPerRegistry: positiveIntSchema.optional(),
-    includeFullDiff: z.boolean().optional()
+    includeFullDiff: z.boolean().default(true)
 };
 export const compareMinecraftSchema = z.object(compareMinecraftShape);
 function compareStatusFromCounts(changedCount) {
@@ -93,6 +93,14 @@ export class CompareMinecraftService {
                         summary: {
                             status: compareStatusFromCounts(changedCount),
                             headline: `Compared ${output.fromVersion} to ${output.toVersion}.`,
+                            subject: createSummarySubject({
+                                task: "versions",
+                                kind: input.subject.kind,
+                                fromVersion: input.subject.fromVersion,
+                                toVersion: input.subject.toVersion,
+                                packageFilter: "packageFilter" in input.subject ? input.subject.packageFilter : undefined,
+                                registry: input.subject.kind === "registry" ? input.subject.registry : undefined
+                            }),
                             counts: {
                                 addedClasses: output.classes?.addedCount ?? 0,
                                 removedClasses: output.classes?.removedCount ?? 0,
@@ -180,6 +188,15 @@ export class CompareMinecraftService {
                         summary: {
                             status: compareStatusFromCounts(changedCount),
                             headline: `Compared ${output.query.className} between ${output.range.fromVersion} and ${output.range.toVersion}.`,
+                            subject: createSummarySubject({
+                                task: "class-diff",
+                                kind: "class",
+                                className: input.subject.className,
+                                fromVersion: input.subject.fromVersion,
+                                toVersion: input.subject.toVersion,
+                                mapping: input.subject.mapping,
+                                sourcePriority: input.subject.sourcePriority
+                            }),
                             counts: output.summary.total
                         },
                         blocks: {
@@ -264,6 +281,13 @@ export class CompareMinecraftService {
                     headline: partialDetail
                         ? `Compared registry changes between ${subject.fromVersion} and ${subject.toVersion} with partial detail.`
                         : `Compared registry changes between ${subject.fromVersion} and ${subject.toVersion}.`,
+                    subject: createSummarySubject({
+                        task: "registry-diff",
+                        kind: "registry",
+                        fromVersion: subject.fromVersion,
+                        toVersion: subject.toVersion,
+                        registry: subject.registry
+                    }),
                     counts: {
                         changedRegistries: registrySummary.registriesChanged,
                         addedEntries: registrySummary.totalAdded,
@@ -335,6 +359,28 @@ export class CompareMinecraftService {
                 const classSignals = (compare.classes?.addedCount ?? 0) + (compare.classes?.removedCount ?? 0);
                 const registrySignals = compare.registry?.summary.registriesChanged ?? 0;
                 const status = compareStatusFromCounts(classSignals + registrySignals);
+                const representativeClassName = compare.classes?.added[0] ?? compare.classes?.removed[0];
+                const nextActions = representativeClassName
+                    ? [
+                        createNextAction("compare-minecraft", {
+                            task: "class-diff",
+                            subject: {
+                                kind: "class",
+                                className: representativeClassName,
+                                fromVersion: subject.fromVersion,
+                                toVersion: subject.toVersion
+                            }
+                        })
+                    ]
+                    : [
+                        createNextAction("inspect-minecraft", {
+                            task: "artifact",
+                            subject: {
+                                kind: "version",
+                                version: subject.toVersion
+                            }
+                        })
+                    ];
                 return {
                     ...buildEntryToolResult({
                         task: "migration-overview",
@@ -343,10 +389,17 @@ export class CompareMinecraftService {
                         summary: {
                             status,
                             headline: `Summarized migration impact from ${subject.fromVersion} to ${subject.toVersion}.`,
+                            subject: createSummarySubject({
+                                task: "migration-overview",
+                                kind: "version-pair",
+                                fromVersion: subject.fromVersion,
+                                toVersion: subject.toVersion
+                            }),
                             counts: {
                                 classSignals,
                                 registrySignals
-                            }
+                            },
+                            nextActions
                         },
                         blocks: {
                             comparison: subject,
@@ -358,28 +411,7 @@ export class CompareMinecraftService {
                                         : registrySignals > 0
                                             ? "registry"
                                             : "minimal",
-                                nextActions: [
-                                    {
-                                        tool: classSignals > 0 ? "compare-minecraft" : "inspect-minecraft",
-                                        params: classSignals > 0
-                                            ? {
-                                                task: "class-diff",
-                                                subject: {
-                                                    kind: "class",
-                                                    className: "net.minecraft.world.item.ItemStack",
-                                                    fromVersion: subject.fromVersion,
-                                                    toVersion: subject.toVersion
-                                                }
-                                            }
-                                            : {
-                                                task: "artifact",
-                                                subject: {
-                                                    kind: "version",
-                                                    version: subject.toVersion
-                                                }
-                                            }
-                                    }
-                                ]
+                                nextActions
                             }
                         }
                     }),

package/dist/entry-tools/inspect-minecraft-service.d.ts

@@ -1,5 +1,5 @@
 import { z } from "zod";
-import type { FindClassOutput, GetArtifactFileOutput, GetClassMembersOutput, GetClassSourceOutput, ListArtifactFilesOutput, ResolveArtifactOutput, SearchClassSourceOutput } from "../source-service.js";
+import type { CheckSymbolExistsOutput, FindClassOutput, GetArtifactFileOutput, GetClassMembersOutput, GetClassSourceOutput, ListArtifactFilesOutput, ResolveArtifactOutput, SearchClassSourceOutput } from "../source-service.js";
 import type { ListVersionsOutput } from "../version-service.js";
 export declare const inspectMinecraftShape: {
     task: z.ZodOptional<z.ZodEnum<["auto", "versions", "artifact", "class-overview", "class-source", "class-members", "search", "file", "list-files"]>>;
@@ -285,10 +285,11 @@ export declare const inspectMinecraftShape: {
         symbolKind: z.ZodOptional<z.ZodEnum<["class", "interface", "enum", "record", "method", "field"]>>;
         packagePrefix: z.ZodOptional<z.ZodString>;
         fileGlob: z.ZodOptional<z.ZodString>;
-        queryMode: z.ZodOptional<z.ZodEnum<["auto", "token", "literal"]>>;
+        queryMode: z.ZodDefault<z.ZodEnum<["auto", "token", "literal"]>>;
     }, "strip", z.ZodTypeAny, {
         kind: "search";
         query: string;
+        queryMode: "auto" | "token" | "literal";
         match?: "exact" | "prefix" | "contains" | "regex" | undefined;
         symbolKind?: "class" | "method" | "field" | "record" | "interface" | "enum" | undefined;
         intent?: "symbol" | "path" | "text" | undefined;
@@ -304,7 +305,6 @@ export declare const inspectMinecraftShape: {
             };
         } | undefined;
         packagePrefix?: string | undefined;
-        queryMode?: "auto" | "token" | "literal" | undefined;
     }, {
         kind: "search";
         query: string;
@@ -499,10 +499,11 @@ export declare const inspectMinecraftShape: {
             symbolKind: z.ZodOptional<z.ZodEnum<["class", "interface", "enum", "record", "method", "field"]>>;
             packagePrefix: z.ZodOptional<z.ZodString>;
             fileGlob: z.ZodOptional<z.ZodString>;
-            queryMode: z.ZodOptional<z.ZodEnum<["auto", "token", "literal"]>>;
+            queryMode: z.ZodDefault<z.ZodEnum<["auto", "token", "literal"]>>;
         }, "strip", z.ZodTypeAny, {
             kind: "search";
             query: string;
+            queryMode: "auto" | "token" | "literal";
             match?: "exact" | "prefix" | "contains" | "regex" | undefined;
             symbolKind?: "class" | "method" | "field" | "record" | "interface" | "enum" | undefined;
             intent?: "symbol" | "path" | "text" | undefined;
@@ -518,7 +519,6 @@ export declare const inspectMinecraftShape: {
                 };
             } | undefined;
             packagePrefix?: string | undefined;
-            queryMode?: "auto" | "token" | "literal" | undefined;
         }, {
             kind: "search";
             query: string;
@@ -575,6 +575,7 @@ export declare const inspectMinecraftShape: {
         } | {
             kind: "search";
             query: string;
+            queryMode: "auto" | "token" | "literal";
             match?: "exact" | "prefix" | "contains" | "regex" | undefined;
             symbolKind?: "class" | "method" | "field" | "record" | "interface" | "enum" | undefined;
             intent?: "symbol" | "path" | "text" | undefined;
@@ -590,7 +591,6 @@ export declare const inspectMinecraftShape: {
                 };
             } | undefined;
             packagePrefix?: string | undefined;
-            queryMode?: "auto" | "token" | "literal" | undefined;
         } | undefined;
     }, {
         kind: "workspace";
@@ -646,7 +646,7 @@ export declare const inspectMinecraftShape: {
             queryMode?: "auto" | "token" | "literal" | undefined;
         } | undefined;
     }>]>>;
-    includeSnapshots: z.ZodOptional<z.ZodBoolean>;
+    includeSnapshots: z.ZodDefault<z.ZodBoolean>;
     detail: z.ZodOptional<z.ZodEnum<["summary", "standard", "full"]>>;
     include: z.ZodOptional<z.ZodArray<z.ZodEnum<[string, ...string[]]>, "many">>;
     limit: z.ZodOptional<z.ZodNumber>;
@@ -936,10 +936,11 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
         symbolKind: z.ZodOptional<z.ZodEnum<["class", "interface", "enum", "record", "method", "field"]>>;
         packagePrefix: z.ZodOptional<z.ZodString>;
         fileGlob: z.ZodOptional<z.ZodString>;
-        queryMode: z.ZodOptional<z.ZodEnum<["auto", "token", "literal"]>>;
+        queryMode: z.ZodDefault<z.ZodEnum<["auto", "token", "literal"]>>;
     }, "strip", z.ZodTypeAny, {
         kind: "search";
         query: string;
+        queryMode: "auto" | "token" | "literal";
         match?: "exact" | "prefix" | "contains" | "regex" | undefined;
         symbolKind?: "class" | "method" | "field" | "record" | "interface" | "enum" | undefined;
         intent?: "symbol" | "path" | "text" | undefined;
@@ -955,7 +956,6 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
             };
         } | undefined;
         packagePrefix?: string | undefined;
-        queryMode?: "auto" | "token" | "literal" | undefined;
     }, {
         kind: "search";
         query: string;
@@ -1150,10 +1150,11 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
             symbolKind: z.ZodOptional<z.ZodEnum<["class", "interface", "enum", "record", "method", "field"]>>;
             packagePrefix: z.ZodOptional<z.ZodString>;
             fileGlob: z.ZodOptional<z.ZodString>;
-            queryMode: z.ZodOptional<z.ZodEnum<["auto", "token", "literal"]>>;
+            queryMode: z.ZodDefault<z.ZodEnum<["auto", "token", "literal"]>>;
         }, "strip", z.ZodTypeAny, {
             kind: "search";
             query: string;
+            queryMode: "auto" | "token" | "literal";
             match?: "exact" | "prefix" | "contains" | "regex" | undefined;
             symbolKind?: "class" | "method" | "field" | "record" | "interface" | "enum" | undefined;
             intent?: "symbol" | "path" | "text" | undefined;
@@ -1169,7 +1170,6 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
                 };
             } | undefined;
             packagePrefix?: string | undefined;
-            queryMode?: "auto" | "token" | "literal" | undefined;
         }, {
             kind: "search";
             query: string;
@@ -1226,6 +1226,7 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
         } | {
             kind: "search";
             query: string;
+            queryMode: "auto" | "token" | "literal";
             match?: "exact" | "prefix" | "contains" | "regex" | undefined;
             symbolKind?: "class" | "method" | "field" | "record" | "interface" | "enum" | undefined;
             intent?: "symbol" | "path" | "text" | undefined;
@@ -1241,7 +1242,6 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
                 };
             } | undefined;
             packagePrefix?: string | undefined;
-            queryMode?: "auto" | "token" | "literal" | undefined;
         } | undefined;
     }, {
         kind: "workspace";
@@ -1297,15 +1297,15 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
             queryMode?: "auto" | "token" | "literal" | undefined;
         } | undefined;
     }>]>>;
-    includeSnapshots: z.ZodOptional<z.ZodBoolean>;
+    includeSnapshots: z.ZodDefault<z.ZodBoolean>;
     detail: z.ZodOptional<z.ZodEnum<["summary", "standard", "full"]>>;
     include: z.ZodOptional<z.ZodArray<z.ZodEnum<[string, ...string[]]>, "many">>;
     limit: z.ZodOptional<z.ZodNumber>;
     cursor: z.ZodOptional<z.ZodString>;
 }, "strip", z.ZodTypeAny, {
+    includeSnapshots: boolean;
     limit?: number | undefined;
     cursor?: string | undefined;
-    includeSnapshots?: boolean | undefined;
     task?: "search" | "auto" | "versions" | "class-source" | "class-members" | "artifact" | "class-overview" | "file" | "list-files" | undefined;
     subject?: {
         kind: "version";
@@ -1366,6 +1366,7 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
     } | {
         kind: "search";
         query: string;
+        queryMode: "auto" | "token" | "literal";
         match?: "exact" | "prefix" | "contains" | "regex" | undefined;
         symbolKind?: "class" | "method" | "field" | "record" | "interface" | "enum" | undefined;
         intent?: "symbol" | "path" | "text" | undefined;
@@ -1381,7 +1382,6 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
             };
         } | undefined;
         packagePrefix?: string | undefined;
-        queryMode?: "auto" | "token" | "literal" | undefined;
     } | {
         kind: "workspace";
         projectPath: string;
@@ -1418,6 +1418,7 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
         } | {
             kind: "search";
             query: string;
+            queryMode: "auto" | "token" | "literal";
             match?: "exact" | "prefix" | "contains" | "regex" | undefined;
             symbolKind?: "class" | "method" | "field" | "record" | "interface" | "enum" | undefined;
             intent?: "symbol" | "path" | "text" | undefined;
@@ -1433,7 +1434,6 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
                 };
             } | undefined;
             packagePrefix?: string | undefined;
-            queryMode?: "auto" | "token" | "literal" | undefined;
         } | undefined;
     } | undefined;
     detail?: "full" | "summary" | "standard" | undefined;
@@ -1575,9 +1575,9 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
     detail?: "full" | "summary" | "standard" | undefined;
     include?: string[] | undefined;
 }>, {
+    includeSnapshots: boolean;
     limit?: number | undefined;
     cursor?: string | undefined;
-    includeSnapshots?: boolean | undefined;
     task?: "search" | "auto" | "versions" | "class-source" | "class-members" | "artifact" | "class-overview" | "file" | "list-files" | undefined;
     subject?: {
         kind: "version";
@@ -1638,6 +1638,7 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
     } | {
         kind: "search";
         query: string;
+        queryMode: "auto" | "token" | "literal";
         match?: "exact" | "prefix" | "contains" | "regex" | undefined;
         symbolKind?: "class" | "method" | "field" | "record" | "interface" | "enum" | undefined;
         intent?: "symbol" | "path" | "text" | undefined;
@@ -1653,7 +1654,6 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
             };
         } | undefined;
         packagePrefix?: string | undefined;
-        queryMode?: "auto" | "token" | "literal" | undefined;
     } | {
         kind: "workspace";
         projectPath: string;
@@ -1690,6 +1690,7 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
         } | {
             kind: "search";
             query: string;
+            queryMode: "auto" | "token" | "literal";
             match?: "exact" | "prefix" | "contains" | "regex" | undefined;
             symbolKind?: "class" | "method" | "field" | "record" | "interface" | "enum" | undefined;
             intent?: "symbol" | "path" | "text" | undefined;
@@ -1705,7 +1706,6 @@ export declare const inspectMinecraftSchema: z.ZodEffects<z.ZodObject<{
                 };
             } | undefined;
             packagePrefix?: string | undefined;
-            queryMode?: "auto" | "token" | "literal" | undefined;
         } | undefined;
     } | undefined;
     detail?: "full" | "summary" | "standard" | undefined;
@@ -1869,6 +1869,18 @@ type InspectMinecraftDeps = {
         artifactId: string;
         limit?: number;
     }) => Promise<FindClassOutput>;
+    checkSymbolExists?: (input: {
+        version: string;
+        kind: "class" | "field" | "method";
+        name: string;
+        owner?: string;
+        descriptor?: string;
+        sourceMapping: "obfuscated" | "mojang" | "intermediary" | "yarn";
+        sourcePriority?: "loom-first" | "maven-first";
+        nameMode?: "fqcn" | "auto";
+        signatureMode?: "exact" | "name-only";
+        maxCandidates?: number;
+    }) => Promise<CheckSymbolExistsOutput>;
     getClassSource: (input: {
         className: string;
         artifactId?: string;
@@ -1939,6 +1951,12 @@ export declare class InspectMinecraftService {
     private resolveClassArtifactReference;
     private resolveWorkspaceArtifactReference;
     private resolveTask;
+    private summarizeRequestedSubject;
+    private exampleVersionForSubject;
+    private buildArtifactContextSuggestedCall;
+    private taskForSubject;
+    private invalidTaskSubjectError;
+    private resolveBinaryBackedClass;
     private resolveArtifactReference;
     private resolveArtifactRef;
     private handleVersions;