@adhisang/minecraft-modding-mcp 3.1.0 → 3.2.0

package/dist/concurrency.js

@@ -0,0 +1,24 @@
+export async function mapWithConcurrencyLimit(items, limit, mapper) {
+    if (!Number.isInteger(limit) || limit < 1) {
+        throw new Error("limit must be a positive integer");
+    }
+    if (items.length === 0) {
+        return [];
+    }
+    const results = new Array(items.length);
+    let nextIndex = 0;
+    const workerCount = Math.min(limit, items.length);
+    const worker = async () => {
+        while (true) {
+            const currentIndex = nextIndex;
+            nextIndex += 1;
+            if (currentIndex >= items.length) {
+                return;
+            }
+            results[currentIndex] = await mapper(items[currentIndex], currentIndex);
+        }
+    };
+    await Promise.all(Array.from({ length: workerCount }, async () => worker()));
+    return results;
+}
+//# sourceMappingURL=concurrency.js.map

package/dist/config.js

@@ -52,6 +52,17 @@ function expandHome(pathValue) {
     const withoutTilde = pathValue.startsWith("~/") ? pathValue.slice(2) : pathValue.slice(1);
     return resolve(homedir(), withoutTilde);
 }
+function normalizeOptionalPathEnvValue(pathValue) {
+    if (!pathValue) {
+        return undefined;
+    }
+    const trimmed = pathValue.trim();
+    if (!trimmed) {
+        return undefined;
+    }
+    const normalized = trimmed.toLowerCase();
+    return normalized === "undefined" || normalized === "null" ? undefined : trimmed;
+}
 function normalizePath(pathValue, field) {
     const expanded = expandHome(pathValue.trim());
     const normalizedForHost = normalizePathForHost(expanded, undefined, field);
@@ -120,14 +131,11 @@ function parseMappingSourcePriority(envValue) {
     return DEFAULTS.mappingSourcePriority;
 }
 function parseOptionalJarPath(envValue, field) {
-    if (!envValue) {
-        return undefined;
-    }
-    const trimmed = envValue.trim();
-    if (!trimmed) {
-        return undefined;
-    }
-    return normalizePath(trimmed, field);
+    const trimmed = normalizeOptionalPathEnvValue(envValue);
+    return trimmed ? normalizePath(trimmed, field) : undefined;
+}
+function parseRequiredPath(envValue, fallback, field) {
+    return normalizePath(normalizeOptionalPathEnvValue(envValue) ?? fallback, field);
 }
 function parseVineflowerPath(envValue) {
     return parseOptionalJarPath(envValue, "MCP_VINEFLOWER_JAR_PATH");
@@ -137,12 +145,12 @@ function buildArtifactsDirectory(cacheDir) {
     return normalizePath(input, "MCP_SQLITE_PATH");
 }
 export function loadConfig() {
-    const cacheDir = normalizePath(process.env.MCP_CACHE_DIR ?? DEFAULTS.cacheDir, "MCP_CACHE_DIR");
-    const localM2Path = normalizePath(process.env.MCP_LOCAL_M2 ?? DEFAULTS.localM2Path, "MCP_LOCAL_M2");
+    const cacheDir = parseRequiredPath(process.env.MCP_CACHE_DIR, DEFAULTS.cacheDir, "MCP_CACHE_DIR");
+    const localM2Path = parseRequiredPath(process.env.MCP_LOCAL_M2, DEFAULTS.localM2Path, "MCP_LOCAL_M2");
     const sourceRepos = parseRepos(process.env.MCP_SOURCE_REPOS);
     return {
         cacheDir,
-        sqlitePath: normalizePath(process.env.MCP_SQLITE_PATH ?? buildArtifactsDirectory(cacheDir), "MCP_SQLITE_PATH"),
+        sqlitePath: parseRequiredPath(process.env.MCP_SQLITE_PATH, buildArtifactsDirectory(cacheDir), "MCP_SQLITE_PATH"),
         sourceRepos,
         localM2Path,
         vineflowerJarPath: parseVineflowerPath(process.env.MCP_VINEFLOWER_JAR_PATH),

package/dist/decompiler/vineflower.js

@@ -1,11 +1,13 @@
-import { access, constants } from "node:fs/promises";
-import { mkdirSync, readdirSync, rmSync, statSync } from "node:fs";
+import { access, constants, mkdir, readFile, readdir, stat } from "node:fs/promises";
+import { mkdirSync, rmSync } from "node:fs";
 import { createHash } from "node:crypto";
 import { basename, join, relative, sep } from "node:path";
+import { mapWithConcurrencyLimit } from "../concurrency.js";
 import { createError, ERROR_CODES, isAppError } from "../errors.js";
 import { assertJavaAvailable, runJavaProcess } from "../java-process.js";
 import { log } from "../logger.js";
 const DEFAULT_TIMEOUT_MS = 120_000;
+const DECOMPILED_JAVA_READ_CONCURRENCY = 8;
 const VINEFLOWER_FLAG_PROFILES = [
     { label: "default", flags: ["-din=1", "-rbr=1", "-dgs=1"] },
     { label: "relaxed", flags: ["-din=1", "-rbr=0", "-dgs=0"] },
@@ -47,14 +49,14 @@ async function assertVineflowerAvailable(vineflowerJarPath) {
         });
     }
 }
-function collectJavaFilesSync(baseDir, currentDir = "") {
+async function collectJavaFilesRecursive(baseDir, currentDir = "") {
     const absoluteBase = currentDir ? join(baseDir, currentDir) : baseDir;
-    const entries = readdirSync(absoluteBase, { withFileTypes: true });
+    const entries = await readdir(absoluteBase, { withFileTypes: true });
     const result = [];
     for (const entry of entries) {
         const next = currentDir ? join(currentDir, entry.name) : entry.name;
         if (entry.isDirectory()) {
-            result.push(...collectJavaFilesSync(baseDir, next));
+            result.push(...await collectJavaFilesRecursive(baseDir, next));
             continue;
         }
         if (entry.isFile() && entry.name.endsWith(".java")) {
@@ -66,23 +68,21 @@ function collectJavaFilesSync(baseDir, currentDir = "") {
 async function collectJavaFiles(baseDir) {
     try {
         const fastGlobModule = (await import("fast-glob"));
-        const sync = fastGlobModule.default?.sync;
-        if (typeof sync === "function") {
-            return sync("**/*.java", { cwd: baseDir, onlyFiles: true });
+        const glob = fastGlobModule.default?.glob;
+        if (typeof glob === "function") {
+            return (await glob("**/*.java", { cwd: baseDir, onlyFiles: true }))
+                .sort((left, right) => left.localeCompare(right));
         }
     }
     catch {
         // optional dependency: fallback to recursive traversal
     }
-    return collectJavaFilesSync(baseDir).map((candidate) => candidate.split(sep).join("/"));
+    return (await collectJavaFilesRecursive(baseDir))
+        .map((candidate) => candidate.split(sep).join("/"))
+        .sort((left, right) => left.localeCompare(right));
 }
 function readFileTreeText(filePath) {
-    return new Promise((resolve, reject) => {
-        import("node:fs/promises")
-            .then((fs) => fs.readFile(filePath, "utf8"))
-            .then(resolve)
-            .catch(reject);
-    });
+    return readFile(filePath, "utf8");
 }
 function decompileOutputDir(cacheDir, binaryJarPath, signature) {
     const digest = createHash("sha256").update(binaryJarPath).update(signature).digest("hex");
@@ -130,17 +130,18 @@ export async function decompileBinaryJar(binaryJarPath, cacheDir, options) {
     const signature = options.signature ?? basename(normalizedBinaryJarPath);
     const outputDir = decompileOutputDir(cacheDir, normalizedBinaryJarPath, signature).replace(/[/\\]$/, "");
     try {
-        mkdirSync(outputDir, { recursive: true });
-        if (statSync(outputDir, { throwIfNoEntry: false })) {
+        await mkdir(outputDir, { recursive: true });
+        const outputDirStats = await stat(outputDir).catch(() => undefined);
+        if (outputDirStats) {
             const existingJavaFiles = await collectJavaFiles(outputDir);
             if (existingJavaFiles.length > 0) {
-                const results = await Promise.all(existingJavaFiles.map(async (candidate) => {
+                const results = await mapWithConcurrencyLimit(existingJavaFiles, DECOMPILED_JAVA_READ_CONCURRENCY, async (candidate) => {
                     const abs = join(outputDir, candidate);
                     return {
                         filePath: normalizeOutputPath(outputDir, abs),
                         content: await readFileTreeText(abs)
                     };
-                }));
+                });
                 emitDecompileLog("decompile.done", {
                     durationMs: Date.now() - startedAt,
                     artifactIdCandidate: options.artifactIdCandidate,
@@ -181,13 +182,13 @@ export async function decompileBinaryJar(binaryJarPath, cacheDir, options) {
                         }
                     });
                 }
-                const javaFiles = await Promise.all(javaFileNames.map(async (candidate) => {
+                const javaFiles = await mapWithConcurrencyLimit(javaFileNames, DECOMPILED_JAVA_READ_CONCURRENCY, async (candidate) => {
                     const abs = join(outputDir, candidate);
                     return {
                         filePath: normalizeOutputPath(outputDir, abs),
                         content: await readFileTreeText(abs)
                     };
-                }));
+                });
                 emitDecompileLog("decompile.done", {
                     durationMs: Date.now() - startedAt,
                     artifactIdCandidate: options.artifactIdCandidate,

package/dist/entry-tools/analyze-mod-service.d.ts

@@ -75,7 +75,7 @@ export declare const analyzeModSchema: z.ZodEffects<z.ZodObject<{
     include: z.ZodOptional<z.ZodArray<z.ZodEnum<[string, ...string[]]>, "many">>;
 }, "strip", z.ZodTypeAny, {
     limit: number;
-    searchType: "class" | "method" | "field" | "all" | "content";
+    searchType: "class" | "field" | "method" | "all" | "content";
     task: "search" | "remap" | "summary" | "class-source" | "decompile";
     subject: {
         kind: "jar";
@@ -111,7 +111,7 @@ export declare const analyzeModSchema: z.ZodEffects<z.ZodObject<{
     targetMapping?: "mojang" | "yarn" | undefined;
     outputJar?: string | undefined;
     query?: string | undefined;
-    searchType?: "class" | "method" | "field" | "all" | "content" | undefined;
+    searchType?: "class" | "field" | "method" | "all" | "content" | undefined;
     detail?: "full" | "summary" | "standard" | undefined;
     include?: string[] | undefined;
     includeFiles?: boolean | undefined;
@@ -119,7 +119,7 @@ export declare const analyzeModSchema: z.ZodEffects<z.ZodObject<{
     executionMode?: "preview" | "apply" | undefined;
 }>, {
     limit: number;
-    searchType: "class" | "method" | "field" | "all" | "content";
+    searchType: "class" | "field" | "method" | "all" | "content";
     task: "search" | "remap" | "summary" | "class-source" | "decompile";
     subject: {
         kind: "jar";
@@ -155,7 +155,7 @@ export declare const analyzeModSchema: z.ZodEffects<z.ZodObject<{
     targetMapping?: "mojang" | "yarn" | undefined;
     outputJar?: string | undefined;
     query?: string | undefined;
-    searchType?: "class" | "method" | "field" | "all" | "content" | undefined;
+    searchType?: "class" | "field" | "method" | "all" | "content" | undefined;
     detail?: "full" | "summary" | "standard" | undefined;
     include?: string[] | undefined;
     includeFiles?: boolean | undefined;

package/dist/entry-tools/analyze-symbol-service.d.ts

@@ -8,15 +8,15 @@ export declare const analyzeSymbolShape: {
         owner: z.ZodOptional<z.ZodString>;
         descriptor: z.ZodOptional<z.ZodString>;
     }, "strip", z.ZodTypeAny, {
-        kind: "symbol" | "class" | "method" | "field";
         name: string;
-        owner?: string | undefined;
+        kind: "symbol" | "class" | "field" | "method";
         descriptor?: string | undefined;
+        owner?: string | undefined;
     }, {
-        kind: "symbol" | "class" | "method" | "field";
         name: string;
-        owner?: string | undefined;
+        kind: "symbol" | "class" | "field" | "method";
         descriptor?: string | undefined;
+        owner?: string | undefined;
     }>;
     version: z.ZodOptional<z.ZodString>;
     sourceMapping: z.ZodOptional<z.ZodEnum<["obfuscated", "mojang", "intermediary", "yarn"]>>;
@@ -39,15 +39,15 @@ export declare const analyzeSymbolSchema: z.ZodEffects<z.ZodObject<{
         owner: z.ZodOptional<z.ZodString>;
         descriptor: z.ZodOptional<z.ZodString>;
     }, "strip", z.ZodTypeAny, {
-        kind: "symbol" | "class" | "method" | "field";
         name: string;
-        owner?: string | undefined;
+        kind: "symbol" | "class" | "field" | "method";
         descriptor?: string | undefined;
+        owner?: string | undefined;
     }, {
-        kind: "symbol" | "class" | "method" | "field";
         name: string;
-        owner?: string | undefined;
+        kind: "symbol" | "class" | "field" | "method";
         descriptor?: string | undefined;
+        owner?: string | undefined;
     }>;
     version: z.ZodOptional<z.ZodString>;
     sourceMapping: z.ZodOptional<z.ZodEnum<["obfuscated", "mojang", "intermediary", "yarn"]>>;
@@ -65,10 +65,10 @@ export declare const analyzeSymbolSchema: z.ZodEffects<z.ZodObject<{
     nameMode: "auto" | "fqcn";
     task: "map" | "workspace" | "exists" | "exact-map" | "lifecycle" | "api-overview";
     subject: {
-        kind: "symbol" | "class" | "method" | "field";
         name: string;
-        owner?: string | undefined;
+        kind: "symbol" | "class" | "field" | "method";
         descriptor?: string | undefined;
+        owner?: string | undefined;
     };
     signatureMode: "exact" | "name-only";
     maxCandidates: number;
@@ -79,15 +79,15 @@ export declare const analyzeSymbolSchema: z.ZodEffects<z.ZodObject<{
     classNameMapping?: "intermediary" | "mojang" | "yarn" | "obfuscated" | undefined;
     detail?: "full" | "summary" | "standard" | undefined;
     include?: string[] | undefined;
-    includeKinds?: ("class" | "method" | "field")[] | undefined;
+    includeKinds?: ("class" | "field" | "method")[] | undefined;
     maxRows?: number | undefined;
 }, {
     task: "map" | "workspace" | "exists" | "exact-map" | "lifecycle" | "api-overview";
     subject: {
-        kind: "symbol" | "class" | "method" | "field";
         name: string;
-        owner?: string | undefined;
+        kind: "symbol" | "class" | "field" | "method";
         descriptor?: string | undefined;
+        owner?: string | undefined;
     };
     projectPath?: string | undefined;
     version?: string | undefined;
@@ -98,17 +98,17 @@ export declare const analyzeSymbolSchema: z.ZodEffects<z.ZodObject<{
     detail?: "full" | "summary" | "standard" | undefined;
     include?: string[] | undefined;
     signatureMode?: "exact" | "name-only" | undefined;
-    includeKinds?: ("class" | "method" | "field")[] | undefined;
+    includeKinds?: ("class" | "field" | "method")[] | undefined;
     maxRows?: number | undefined;
     maxCandidates?: number | undefined;
 }>, {
     nameMode: "auto" | "fqcn";
     task: "map" | "workspace" | "exists" | "exact-map" | "lifecycle" | "api-overview";
     subject: {
-        kind: "symbol" | "class" | "method" | "field";
         name: string;
-        owner?: string | undefined;
+        kind: "symbol" | "class" | "field" | "method";
         descriptor?: string | undefined;
+        owner?: string | undefined;
     };
     signatureMode: "exact" | "name-only";
     maxCandidates: number;
@@ -119,15 +119,15 @@ export declare const analyzeSymbolSchema: z.ZodEffects<z.ZodObject<{
     classNameMapping?: "intermediary" | "mojang" | "yarn" | "obfuscated" | undefined;
     detail?: "full" | "summary" | "standard" | undefined;
     include?: string[] | undefined;
-    includeKinds?: ("class" | "method" | "field")[] | undefined;
+    includeKinds?: ("class" | "field" | "method")[] | undefined;
     maxRows?: number | undefined;
 }, {
     task: "map" | "workspace" | "exists" | "exact-map" | "lifecycle" | "api-overview";
     subject: {
-        kind: "symbol" | "class" | "method" | "field";
         name: string;
-        owner?: string | undefined;
+        kind: "symbol" | "class" | "field" | "method";
         descriptor?: string | undefined;
+        owner?: string | undefined;
     };
     projectPath?: string | undefined;
     version?: string | undefined;
@@ -138,7 +138,7 @@ export declare const analyzeSymbolSchema: z.ZodEffects<z.ZodObject<{
     detail?: "full" | "summary" | "standard" | undefined;
     include?: string[] | undefined;
     signatureMode?: "exact" | "name-only" | undefined;
-    includeKinds?: ("class" | "method" | "field")[] | undefined;
+    includeKinds?: ("class" | "field" | "method")[] | undefined;
     maxRows?: number | undefined;
     maxCandidates?: number | undefined;
 }>;
@@ -180,6 +180,8 @@ type AnalyzeSymbolDeps = {
         symbol: string;
         descriptor?: string;
         mapping?: "obfuscated" | "mojang" | "intermediary" | "yarn";
+        toVersion?: string;
+        maxVersions?: number;
     }) => Promise<TraceSymbolLifecycleOutput>;
     resolveWorkspaceSymbol: (input: {
         projectPath: string;

package/dist/entry-tools/analyze-symbol-service.js

@@ -231,7 +231,9 @@ export class AnalyzeSymbolService {
                         ? `${input.subject.owner}.${input.subject.name}`
                         : input.subject.name,
                     descriptor: input.subject.descriptor,
-                    mapping: input.sourceMapping
+                    mapping: input.sourceMapping,
+                    toVersion: input.version,
+                    maxVersions: 5
                 });
                 return {
                     ...buildEntryToolResult({
@@ -309,10 +311,11 @@ export class AnalyzeSymbolService {
                 };
             }
             case "api-overview": {
+                const classNameMapping = input.classNameMapping ?? input.sourceMapping ?? "obfuscated";
                 const output = await this.deps.getClassApiMatrix({
                     version: input.version,
                     className: input.subject.name,
-                    classNameMapping: input.classNameMapping ?? "obfuscated",
+                    classNameMapping,
                     includeKinds: input.includeKinds,
                     maxRows: input.maxRows
                 });
@@ -329,7 +332,7 @@ export class AnalyzeSymbolService {
                                 kind: input.subject.kind,
                                 name: input.subject.name,
                                 version: input.version,
-                                classNameMapping: input.classNameMapping ?? "obfuscated"
+                                classNameMapping
                             }),
                             counts: {
                                 rows: output.rowCount,