@adhisang/minecraft-modding-mcp 4.2.1 → 6.0.0

package/dist/entry-tools/batch-class-members-service.d.ts

@@ -1,3 +1,4 @@
+import { type ResponseDetailLevel } from "../response-utils.js";
 import type { GetClassMembersInput, GetClassMembersOutput, ResolveArtifactInput, ResolveArtifactOutput } from "../source-service.js";
 import type { ArtifactScope, MappingSourcePriority, ResolveArtifactTargetInput, SourceMapping } from "../types.js";
 import { type BatchOutput } from "./batch-runner.js";
@@ -25,7 +26,8 @@ export type BatchClassMembersInput = {
     strictVersion?: boolean;
     concurrency?: number;
     failFast?: boolean;
-    compact?: boolean;
+    detail?: ResponseDetailLevel;
+    include?: readonly string[];
     entries: readonly BatchClassMembersEntry[];
 };
 export declare class BatchClassMembersService {

package/dist/entry-tools/batch-class-members-service.js

@@ -1,5 +1,5 @@
 import { buildSuggestedCall } from "../build-suggested-call.js";
-import { compactResponse, compactMembersResponse, TOOL_PRESERVE_PAYLOAD_KEYS } from "../response-utils.js";
+import { projectByDetail } from "../response-utils.js";
 import { runBatch, splitEntryWarnings } from "./batch-runner.js";
 export class BatchClassMembersService {
     deps;
@@ -9,7 +9,8 @@ export class BatchClassMembersService {
     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,
@@ -53,6 +54,7 @@ export class BatchClassMembersService {
                     includeInherited: entry.includeInherited,
                     memberPattern: entry.memberPattern,
                     maxMembers: entry.maxMembers,
+                    includeDescriptors: include.has("descriptors"),
                     mapping: input.mapping,
                     sourcePriority: input.sourcePriority,
                     allowDecompile: input.allowDecompile,
@@ -63,16 +65,14 @@ export class BatchClassMembersService {
                     strictVersion: input.strictVersion
                 }));
                 const { result, warnings } = splitEntryWarnings(raw);
-                const projected = compact
-                    ? compactResponse(compactMembersResponse(result), TOOL_PRESERVE_PAYLOAD_KEYS["get-class-members"])
-                    : result;
+                const projected = projectByDetail("get-class-members", result, detail, include);
                 return { result: projected, warnings };
             },
             buildErrorSuggestedCall: (entry, sharedArtifact) => {
                 if (!sharedArtifact)
                     return undefined;
                 const params = {
-                    target: { type: "artifact", artifactId: sharedArtifact.artifactId },
+                    target: { kind: "artifact", artifactId: sharedArtifact.artifactId },
                     className: entry.className
                 };
                 if (entry.access !== undefined)

package/dist/entry-tools/batch-class-source-service.d.ts

@@ -1,3 +1,4 @@
+import { type ResponseDetailLevel } from "../response-utils.js";
 import type { GetClassSourceInput, GetClassSourceOutput, ResolveArtifactInput, ResolveArtifactOutput } from "../source-service.js";
 import type { ArtifactScope, MappingSourcePriority, ResolveArtifactTargetInput, SourceMapping } from "../types.js";
 import { type BatchOutput } from "./batch-runner.js";
@@ -23,7 +24,8 @@ export type BatchClassSourceInput = {
     strictVersion?: boolean;
     concurrency?: number;
     failFast?: boolean;
-    compact?: boolean;
+    detail?: ResponseDetailLevel;
+    include?: readonly string[];
     entries: readonly BatchClassSourceEntry[];
 };
 export type BatchClassSourceDeps = {

package/dist/entry-tools/batch-class-source-service.js

@@ -1,5 +1,5 @@
 import { buildSuggestedCall } from "../build-suggested-call.js";
-import { compactResponse, compactSourceResponse, TOOL_PRESERVE_PAYLOAD_KEYS } from "../response-utils.js";
+import { projectByDetail } from "../response-utils.js";
 import { runBatch, splitEntryWarnings } from "./batch-runner.js";
 export class BatchClassSourceService {
     deps;
@@ -9,7 +9,8 @@ export class BatchClassSourceService {
     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,
@@ -64,16 +65,14 @@ export class BatchClassSourceService {
                     strictVersion: input.strictVersion
                 }));
                 const { result, warnings } = splitEntryWarnings(raw);
-                const projected = compact
-                    ? compactResponse(compactSourceResponse(result), TOOL_PRESERVE_PAYLOAD_KEYS["get-class-source"])
-                    : result;
+                const projected = projectByDetail("get-class-source", result, detail, include);
                 return { result: projected, warnings };
             },
             buildErrorSuggestedCall: (entry, sharedArtifact) => {
                 if (!sharedArtifact)
                     return undefined;
                 const params = {
-                    target: { type: "artifact", artifactId: sharedArtifact.artifactId },
+                    target: { kind: "artifact", artifactId: sharedArtifact.artifactId },
                     className: entry.className
                 };
                 if (entry.mode !== undefined)

package/dist/entry-tools/batch-mappings-service.d.ts

@@ -1,3 +1,4 @@
+import { type ResponseDetailLevel } from "../response-utils.js";
 import type { FindMappingInput, FindMappingOutput } from "../source-service.js";
 import type { MappingSourcePriority, SourceMapping } from "../types.js";
 export type BatchMappingsDeps = {
@@ -26,7 +27,8 @@ export type BatchMappingsInput = {
     gradleUserHome?: string;
     concurrency?: number;
     failFast?: boolean;
-    compact?: boolean;
+    detail?: ResponseDetailLevel;
+    include?: readonly string[];
     entries: readonly BatchMappingsEntry[];
 };
 export declare class BatchMappingsService {

package/dist/entry-tools/batch-mappings-service.js

@@ -1,5 +1,5 @@
 import { buildSuggestedCall } from "../build-suggested-call.js";
-import { compactResponse, compactMappingResponse } from "../response-utils.js";
+import { projectByDetail } from "../response-utils.js";
 import { runBatch, splitEntryWarnings } from "./batch-runner.js";
 export class BatchMappingsService {
     deps;
@@ -9,7 +9,8 @@ export class BatchMappingsService {
     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,
@@ -32,9 +33,7 @@ export class BatchMappingsService {
                     maxCandidates: entry.maxCandidates
                 }));
                 const { result, warnings } = splitEntryWarnings(raw);
-                const projected = compact
-                    ? compactResponse(compactMappingResponse(result))
-                    : result;
+                const projected = projectByDetail("find-mapping", result, detail, include);
                 return { result: projected, warnings };
             },
             buildErrorSuggestedCall: (entry) => {

package/dist/entry-tools/batch-symbol-exists-service.d.ts

@@ -1,3 +1,4 @@
+import { type ResponseDetailLevel } from "../response-utils.js";
 import type { CheckSymbolExistsInput, CheckSymbolExistsOutput, ResolveArtifactInput, ResolveArtifactOutput } from "../source-service.js";
 import type { ArtifactScope, MappingSourcePriority, SourceMapping, WorkspaceTargetInput, SourceTargetInput } from "../types.js";
 export type BatchSymbolExistsDeps = {
@@ -36,7 +37,8 @@ export type BatchSymbolExistsInput = {
     strictVersion?: boolean;
     concurrency?: number;
     failFast?: boolean;
-    compact?: boolean;
+    detail?: ResponseDetailLevel;
+    include?: readonly string[];
     entries: readonly BatchSymbolExistsEntry[];
 };
 export declare class BatchSymbolExistsService {

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/compare-minecraft-service.js

@@ -64,7 +64,7 @@ export class CompareMinecraftService {
                     : {
                         fromVersion: input.subject.fromVersion,
                         toVersion: input.subject.toVersion,
-                        packageFilter: input.subject.kind === "class" ? undefined : input.subject.registry
+                        packageFilter: undefined
                     };
                 const output = await this.deps.compareVersions({
                     fromVersion: subject.fromVersion,

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
         }),
@@ -63,12 +63,13 @@ export async function handleClassMembers(deps, subject, detail, include, limit)
                     include,
                     warnings: [...artifact.warnings, ...members.warnings],
                     truncated: createTruncationMeta({
-                        omittedGroups: ["members"],
+                        omittedGroups: detail === "summary" ? ["members"] : [],
                         nextActions: [
                             createNextAction("inspect-minecraft", {
                                 task: "class-members",
                                 detail: "full",
                                 include: ["members"],
+                                limit: Math.min(members.counts.total, 5000),
                                 subject
                             })
                         ]

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

@@ -8,6 +8,31 @@ export async function handleFile(deps, subject, detail, include) {
     const artifact = subject.kind === "file"
         ? await resolveArtifactReference(deps, subject, "file")
         : await resolveWorkspaceArtifactReference(deps, subject, fileSubject.artifact);
+    if (!artifact.artifactId) {
+        const summary = {
+            status: "blocked",
+            headline: `Could not resolve artifact context for ${fileSubject.filePath}.`,
+            subject: createSummarySubject({
+                task: "file",
+                filePath: fileSubject.filePath
+            })
+        };
+        return {
+            ...buildEntryToolResult({
+                task: "file",
+                summary,
+                detail,
+                include,
+                blocks: {
+                    subject: {
+                        requested: subject
+                    }
+                },
+                alwaysBlocks: ["subject"]
+            }),
+            warnings: artifact.warnings
+        };
+    }
     const file = await deps.getArtifactFile({
         artifactId: artifact.artifactId,
         filePath: fileSubject.filePath
@@ -17,7 +42,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
         }),
@@ -46,7 +70,7 @@ export async function handleFile(deps, subject, detail, include) {
                     content: include.includes("source") || detail !== "summary" ? file.content : undefined
                 }
             },
-            alwaysBlocks: ["subject"]
+            alwaysBlocks: ["subject", "file"]
         }),
         warnings: [...artifact.warnings]
     };

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

@@ -41,9 +41,10 @@ export async function handleVersions(deps, input, detail, include) {
                     snapshots: input.includeSnapshots ? versions.snapshots : undefined,
                     cached: versions.cached
                 }
-            }
+            },
+            alwaysBlocks: ["versions"]
         }),
-        warnings: []
+        warnings: versions.warnings ?? []
     };
 }
 //# sourceMappingURL=versions.js.map