@adhisang/minecraft-modding-mcp 4.2.0 → 5.0.0

package/dist/entry-tools/batch-symbol-exists-service.js

@@ -1,6 +1,6 @@
 import { buildSuggestedCall } from "../build-suggested-call.js";
 import { createError, ERROR_CODES } from "../errors.js";
-import { compactResponse, compactMappingResponse } from "../response-utils.js";
+import { projectByDetail } from "../response-utils.js";
 import { runBatch, splitEntryWarnings } from "./batch-runner.js";
 function deriveMinecraftVersion(resolvedVersion, workspaceVersion) {
     const candidate = resolvedVersion?.trim() || workspaceVersion?.trim();
@@ -23,7 +23,8 @@ export class BatchSymbolExistsService {
     async execute(input) {
         const concurrency = input.concurrency ?? 4;
         const failFast = input.failFast ?? false;
-        const compact = input.compact ?? true;
+        const detail = input.detail ?? "summary";
+        const include = new Set(input.include ?? []);
         return runBatch({
             entries: input.entries,
             concurrency,
@@ -79,9 +80,7 @@ export class BatchSymbolExistsService {
                     projectPath: input.projectPath
                 }));
                 const { result, warnings } = splitEntryWarnings(raw);
-                const projected = compact
-                    ? compactResponse(compactMappingResponse(result))
-                    : result;
+                const projected = projectByDetail("check-symbol-exists", result, detail, include);
                 return { result: projected, warnings };
             },
             buildErrorSuggestedCall: (entry, sharedArtifact) => {

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

@@ -33,16 +33,16 @@ export declare const compareMinecraftShape: {
         fromVersion: string;
         toVersion: string;
         mapping?: "obfuscated" | "mojang" | "intermediary" | "yarn" | undefined;
-        gradleUserHome?: string | undefined;
         sourcePriority?: "loom-first" | "maven-first" | undefined;
+        gradleUserHome?: string | undefined;
     }, {
         kind: "class";
         className: string;
         fromVersion: string;
         toVersion: string;
         mapping?: "obfuscated" | "mojang" | "intermediary" | "yarn" | undefined;
-        gradleUserHome?: string | undefined;
         sourcePriority?: "loom-first" | "maven-first" | undefined;
+        gradleUserHome?: string | undefined;
     }>, z.ZodObject<{
         kind: z.ZodLiteral<"registry">;
         fromVersion: z.ZodString;
@@ -97,16 +97,16 @@ export declare const compareMinecraftSchema: z.ZodObject<{
         fromVersion: string;
         toVersion: string;
         mapping?: "obfuscated" | "mojang" | "intermediary" | "yarn" | undefined;
-        gradleUserHome?: string | undefined;
         sourcePriority?: "loom-first" | "maven-first" | undefined;
+        gradleUserHome?: string | undefined;
     }, {
         kind: "class";
         className: string;
         fromVersion: string;
         toVersion: string;
         mapping?: "obfuscated" | "mojang" | "intermediary" | "yarn" | undefined;
-        gradleUserHome?: string | undefined;
         sourcePriority?: "loom-first" | "maven-first" | undefined;
+        gradleUserHome?: string | undefined;
     }>, z.ZodObject<{
         kind: z.ZodLiteral<"registry">;
         fromVersion: z.ZodString;
@@ -130,7 +130,6 @@ export declare const compareMinecraftSchema: z.ZodObject<{
     maxEntriesPerRegistry: z.ZodOptional<z.ZodNumber>;
     includeFullDiff: z.ZodDefault<z.ZodBoolean>;
 }, "strip", z.ZodTypeAny, {
-    maxClassResults: number;
     subject: {
         kind: "version-pair";
         fromVersion: string;
@@ -142,18 +141,19 @@ export declare const compareMinecraftSchema: z.ZodObject<{
         fromVersion: string;
         toVersion: string;
         mapping?: "obfuscated" | "mojang" | "intermediary" | "yarn" | undefined;
-        gradleUserHome?: string | undefined;
         sourcePriority?: "loom-first" | "maven-first" | undefined;
+        gradleUserHome?: string | undefined;
     } | {
         kind: "registry";
         fromVersion: string;
         toVersion: string;
         registry?: string | undefined;
     };
+    maxClassResults: number;
     includeFullDiff: boolean;
     limit?: number | undefined;
     task?: "auto" | "versions" | "class-diff" | "registry-diff" | "migration-overview" | undefined;
-    detail?: "full" | "summary" | "standard" | undefined;
+    detail?: "summary" | "standard" | "full" | undefined;
     include?: string[] | undefined;
     maxEntriesPerRegistry?: number | undefined;
 }, {
@@ -168,8 +168,8 @@ export declare const compareMinecraftSchema: z.ZodObject<{
         fromVersion: string;
         toVersion: string;
         mapping?: "obfuscated" | "mojang" | "intermediary" | "yarn" | undefined;
-        gradleUserHome?: string | undefined;
         sourcePriority?: "loom-first" | "maven-first" | undefined;
+        gradleUserHome?: string | undefined;
     } | {
         kind: "registry";
         fromVersion: string;
@@ -177,10 +177,10 @@ export declare const compareMinecraftSchema: z.ZodObject<{
         registry?: string | undefined;
     };
     limit?: number | undefined;
-    maxClassResults?: number | undefined;
     task?: "auto" | "versions" | "class-diff" | "registry-diff" | "migration-overview" | undefined;
-    detail?: "full" | "summary" | "standard" | undefined;
+    detail?: "summary" | "standard" | "full" | undefined;
     include?: string[] | undefined;
+    maxClassResults?: number | undefined;
     maxEntriesPerRegistry?: number | undefined;
     includeFullDiff?: boolean | undefined;
 }>;

package/dist/entry-tools/entry-tool-schema.d.ts

@@ -1,6 +1,6 @@
 import { z } from "zod";
 export declare const detailSchema: z.ZodEnum<["summary", "standard", "full"]>;
-export declare const includeGroupSchema: z.ZodEnum<["warnings", "provenance", "candidates", "members", "source", "files", "samples", "diff", "issues", "timeline", "matrix", "entries", "workspace", "health", "recovery", "paths", "owners", "preview", "cacheEntries", "timings", "artifact", "classes", "registry"]>;
+export declare const includeGroupSchema: z.ZodEnum<["warnings", "provenance", "candidates", "members", "descriptors", "source", "files", "samples", "diff", "issues", "timeline", "matrix", "entries", "workspace", "health", "recovery", "paths", "owners", "preview", "cacheEntries", "timings", "artifact", "classes", "registry"]>;
 export declare const executionModeSchema: z.ZodEnum<["preview", "apply"]>;
 export declare const positiveIntSchema: z.ZodNumber;
 export declare function buildIncludeSchema(allowed: readonly string[]): z.ZodOptional<z.ZodArray<z.ZodEnum<[string, ...string[]]>, "many">>;

package/dist/entry-tools/inspect-minecraft/handlers/artifact.js

@@ -8,8 +8,7 @@ export async function handleArtifact(deps, subject, detail, include) {
             status: "blocked",
             headline: "Could not resolve an artifact without a Minecraft version.",
             subject: createSummarySubject({
-                task: "artifact",
-                requested: subject
+                task: "artifact"
             }),
             nextActions: nextActionsOrUndefined([
                 createNextAction("inspect-minecraft", {
@@ -42,7 +41,6 @@ export async function handleArtifact(deps, subject, detail, include) {
         headline: `Resolved artifact ${resolved.artifactId}.`,
         subject: createSummarySubject({
             task: "artifact",
-            requested: subject,
             artifactId: resolved.artifactId,
             version: resolved.version
         }),

package/dist/entry-tools/inspect-minecraft/handlers/class-members.js

@@ -15,14 +15,14 @@ export async function handleClassMembers(deps, subject, detail, include, limit)
         gradleUserHome: classSubject.gradleUserHome,
         preferProjectVersion: classSubject.preferProjectVersion,
         strictVersion: classSubject.strictVersion,
-        maxMembers: limit
+        maxMembers: limit,
+        includeDescriptors: include.includes("descriptors")
     });
     const summary = {
         status: members.truncated ? "partial" : "ok",
         headline: `Collected ${members.counts.total} members for ${members.className}.`,
         subject: createSummarySubject({
             task: "class-members",
-            requested: subject,
             className: members.className,
             artifactId: members.artifactId
         }),

package/dist/entry-tools/inspect-minecraft/handlers/class-overview.js

@@ -14,7 +14,6 @@ export async function handleClassOverview(deps, subject, detail, include) {
             headline: `Could not resolve artifact context for ${className}.`,
             subject: createSummarySubject({
                 task: "class-overview",
-                requested: subject,
                 className
             })
         };
@@ -64,7 +63,6 @@ export async function handleClassOverview(deps, subject, detail, include) {
                 headline: `Resolved class overview for ${partialSourceFallback.className}.`,
                 subject: createSummarySubject({
                     task: "class-overview",
-                    requested: subject,
                     className: partialSourceFallback.className,
                     artifactId: metadata.artifactId
                 }),
@@ -110,7 +108,6 @@ export async function handleClassOverview(deps, subject, detail, include) {
             headline: `No class match was found for ${className}.`,
             subject: createSummarySubject({
                 task: "class-overview",
-                requested: subject,
                 className,
                 artifactId: artifact.artifactId
             })
@@ -152,7 +149,6 @@ export async function handleClassOverview(deps, subject, detail, include) {
             headline: `Found ${matches.total} class matches for ${className}.`,
             subject: createSummarySubject({
                 task: "class-overview",
-                requested: subject,
                 className,
                 artifactId: artifact.artifactId
             }),
@@ -192,7 +188,6 @@ export async function handleClassOverview(deps, subject, detail, include) {
         headline: `Resolved class overview for ${match.qualifiedName}.`,
         subject: createSummarySubject({
             task: "class-overview",
-            requested: subject,
             className: match.qualifiedName,
             artifactId: artifact.artifactId
         }),

package/dist/entry-tools/inspect-minecraft/handlers/class-source.js

@@ -23,7 +23,6 @@ export async function handleClassSource(deps, subject, detail, include) {
         headline: `Resolved source for ${source.className}.`,
         subject: createSummarySubject({
             task: "class-source",
-            requested: subject,
             className: source.className,
             artifactId: source.artifactId
         }),

package/dist/entry-tools/inspect-minecraft/handlers/file.js

@@ -17,7 +17,6 @@ export async function handleFile(deps, subject, detail, include) {
         headline: `Read ${file.filePath}.`,
         subject: createSummarySubject({
             task: "file",
-            requested: subject,
             filePath: file.filePath,
             artifactId: artifact.artifactId
         }),

package/dist/entry-tools/inspect-minecraft/handlers/list-files.js

@@ -52,7 +52,6 @@ export async function handleListFiles(deps, subject, detail, include, limit, cur
         headline: `Listed ${files.items.length} files for ${artifact.artifactId}.`,
         subject: createSummarySubject({
             task: "list-files",
-            requested: subject,
             artifactId: artifact.artifactId
         }),
         counts: {

package/dist/entry-tools/inspect-minecraft/handlers/search.d.ts

@@ -2,4 +2,12 @@ import { type DetailLevel } from "../../response-contract.js";
 import { type Subject, type InspectMinecraftDeps } from "../internal.js";
 export declare function handleSearch(deps: InspectMinecraftDeps, subject: Subject, detail: DetailLevel, include: string[], limit: number | undefined, cursor: string | undefined): Promise<{
     warnings: string[];
+    meta: {
+        pagination: {
+            cursorIgnored?: boolean | undefined;
+            nextCursor?: string | undefined;
+            returnedCount: number;
+            hasMore: boolean;
+        };
+    };
 }>;

package/dist/entry-tools/inspect-minecraft/handlers/search.js

@@ -82,7 +82,6 @@ export async function handleSearch(deps, subject, detail, include, limit, cursor
             : `No source hits were found for ${searchSubject.query}.`,
         subject: createSummarySubject({
             task: "search",
-            requested: requestedSubject,
             query: searchSubject.query,
             artifactId: artifact.artifactId
         }),
@@ -129,6 +128,7 @@ export async function handleSearch(deps, subject, detail, include, limit, cursor
                 }
                 : {})
     };
+    const returnedCount = (detail === "summary" ? sampledHits.items : effectiveHits).length;
     return {
         ...buildEntryToolResult({
             task: "search",
@@ -150,7 +150,18 @@ export async function handleSearch(deps, subject, detail, include, limit, cursor
             },
             alwaysBlocks: ["subject"]
         }),
-        warnings: [...artifact.warnings, ...(binaryBackedClassHit?.warnings ?? [])]
+        warnings: [...artifact.warnings, ...(binaryBackedClassHit?.warnings ?? [])],
+        // Always surface continuation state in meta, even in summary mode where the
+        // search block (and its nextCursor) is omitted, so callers never stop early
+        // believing they have all results.
+        meta: {
+            pagination: {
+                returnedCount,
+                hasMore: search.nextCursor != null,
+                ...(search.nextCursor != null ? { nextCursor: search.nextCursor } : {}),
+                ...(search.cursorIgnored ? { cursorIgnored: true } : {})
+            }
+        }
     };
 }
 //# sourceMappingURL=search.js.map

package/dist/entry-tools/inspect-minecraft/handlers/versions.d.ts

@@ -2,5 +2,5 @@ import { type DetailLevel } from "../../response-contract.js";
 import { type InspectMinecraftDeps } from "../internal.js";
 import { type InspectMinecraftInput } from "../../inspect-minecraft-service.js";
 export declare function handleVersions(deps: InspectMinecraftDeps, input: InspectMinecraftInput, detail: DetailLevel, include: string[]): Promise<{
-    warnings: never[];
+    warnings: string[];
 }>;

package/dist/entry-tools/inspect-minecraft/handlers/versions.js

@@ -43,7 +43,7 @@ export async function handleVersions(deps, input, detail, include) {
                 }
             }
         }),
-        warnings: []
+        warnings: versions.warnings ?? []
     };
 }
 //# sourceMappingURL=versions.js.map