@adhisang/minecraft-modding-mcp 7.0.0 → 7.1.1

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

@@ -1,4 +1,5 @@
 import { z } from "zod";
+import { diffLibraries } from "../version-diff-service.js";
 import { createError, ERROR_CODES } from "../errors.js";
 import { buildIncludeSchema, detailSchema, positiveIntSchema } from "./entry-tool-schema.js";
 import { buildEntryToolMeta, buildEntryToolResult, createNextAction, createSummarySubject, createTruncationMeta } from "./response-contract.js";
@@ -42,10 +43,37 @@ export const compareMinecraftSchema = z.object(compareMinecraftShape);
 function compareStatusFromCounts(changedCount) {
     return changedCount > 0 ? "changed" : "unchanged";
 }
+/** Default bound on migration-overview's library enrichment (see `libraryDiffDeadlineMs`). */
+const DEFAULT_LIBRARY_DIFF_DEADLINE_MS = 5000;
+class LibraryDiffTimeoutError extends Error {
+}
+/**
+ * Races `promise` against a `ms` timer. The timer is always cleared (on
+ * either settlement) and unref'd so it can never keep the process alive.
+ */
+function withDeadline(promise, ms) {
+    return new Promise((resolve, reject) => {
+        const timer = setTimeout(() => {
+            reject(new LibraryDiffTimeoutError(`timed out after ${ms}ms`));
+        }, ms);
+        if (typeof timer.unref === "function") {
+            timer.unref();
+        }
+        promise.then((value) => {
+            clearTimeout(timer);
+            resolve(value);
+        }, (error) => {
+            clearTimeout(timer);
+            reject(error);
+        });
+    });
+}
 export class CompareMinecraftService {
     deps;
-    constructor(deps) {
+    libraryDiffDeadlineMs;
+    constructor(deps, options) {
         this.deps = deps;
+        this.libraryDiffDeadlineMs = options?.libraryDiffDeadlineMs ?? DEFAULT_LIBRARY_DIFF_DEADLINE_MS;
     }
     async execute(input) {
         const task = input.task && input.task !== "auto"
@@ -362,6 +390,32 @@ export class CompareMinecraftService {
                 const registrySignals = compare.registry?.summary.registriesChanged ?? 0;
                 const status = compareStatusFromCounts(classSignals + registrySignals);
                 const representativeClassName = compare.classes?.added[0] ?? compare.classes?.removed[0];
+                // Library changes (e.g. LWJGL GLFW replaced by SDL) never show up in the
+                // class/registry diff above, so fetch and diff them separately. This must
+                // never fail the whole migration-overview: a missing dependency wiring, a
+                // fetch failure, or a deadline expiry (e.g. offline after a restart, with
+                // only cached jars) just omits the block and adds a warning. On a
+                // timeout, the in-flight fetch is left running — it may still warm the
+                // version-detail cache for a later call — we simply stop waiting on it.
+                let librariesResult;
+                const libraryWarnings = [];
+                if (this.deps.getVersionLibraries) {
+                    try {
+                        const [fromLibraries, toLibraries] = await withDeadline(Promise.all([
+                            this.deps.getVersionLibraries({ version: subject.fromVersion }),
+                            this.deps.getVersionLibraries({ version: subject.toVersion })
+                        ]), this.libraryDiffDeadlineMs);
+                        librariesResult = diffLibraries(fromLibraries, toLibraries);
+                    }
+                    catch (error) {
+                        if (error instanceof LibraryDiffTimeoutError) {
+                            libraryWarnings.push(`Library comparison between ${subject.fromVersion} and ${subject.toVersion} timed out after ${this.libraryDiffDeadlineMs}ms.`);
+                        }
+                        else {
+                            libraryWarnings.push(`Could not compare libraries between ${subject.fromVersion} and ${subject.toVersion}: ${error instanceof Error ? error.message : String(error)}`);
+                        }
+                    }
+                }
                 const nextActions = representativeClassName
                     ? [
                         createNextAction("compare-minecraft", {
@@ -399,7 +453,13 @@ export class CompareMinecraftService {
                             }),
                             counts: {
                                 classSignals,
-                                registrySignals
+                                registrySignals,
+                                ...(librariesResult
+                                    ? {
+                                        librariesAdded: librariesResult.addedCount,
+                                        librariesRemoved: librariesResult.removedCount
+                                    }
+                                    : {})
                             },
                             nextActions
                         },
@@ -413,11 +473,12 @@ export class CompareMinecraftService {
                                         : registrySignals > 0
                                             ? "registry"
                                             : "minimal",
-                                nextActions
+                                nextActions,
+                                ...(librariesResult ? { libraries: librariesResult } : {})
                             }
                         }
                     }),
-                    warnings: compare.warnings
+                    warnings: [...compare.warnings, ...libraryWarnings]
                 };
             }
             default:

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

@@ -1,8 +1,8 @@
 import { z } from "zod";
 export declare const detailSchema: z.ZodEnum<{
+    full: "full";
     summary: "summary";
     standard: "standard";
-    full: "full";
 }>;
 export declare const includeGroupSchema: z.ZodEnum<{
     workspace: "workspace";
@@ -17,6 +17,7 @@ export declare const includeGroupSchema: z.ZodEnum<{
     samples: "samples";
     timings: "timings";
     matrix: "matrix";
+    artifact: "artifact";
     provenance: "provenance";
     descriptors: "descriptors";
     diff: "diff";
@@ -27,7 +28,6 @@ export declare const includeGroupSchema: z.ZodEnum<{
     paths: "paths";
     owners: "owners";
     cacheEntries: "cacheEntries";
-    artifact: "artifact";
     classes: "classes";
 }>;
 export declare const executionModeSchema: z.ZodEnum<{

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

@@ -5,6 +5,10 @@ export async function handleVersions(deps, input, detail, include) {
         includeSnapshots: input.includeSnapshots,
         limit: input.limit
     });
+    // versions.latest.release and releases[0] can both be absent for an effectively
+    // empty version manifest; omit the nextAction rather than suggest an unusable
+    // follow-up call with no version.
+    const resolvedNextVersion = versions.latest.release ?? versions.releases[0]?.id;
     const summary = {
         status: "ok",
         headline: `Found ${versions.totalAvailable} Minecraft versions.`,
@@ -18,15 +22,17 @@ export async function handleVersions(deps, input, detail, include) {
             releases: versions.releases.length,
             snapshots: versions.snapshots?.length ?? 0
         },
-        nextActions: nextActionsOrUndefined([
-            createNextAction("inspect-minecraft", {
-                task: "artifact",
-                subject: {
-                    kind: "version",
-                    version: versions.latest.release ?? versions.releases[0]?.id
-                }
-            })
-        ])
+        nextActions: nextActionsOrUndefined(resolvedNextVersion !== undefined
+            ? [
+                createNextAction("inspect-minecraft", {
+                    task: "artifact",
+                    subject: {
+                        kind: "version",
+                        version: resolvedNextVersion
+                    }
+                })
+            ]
+            : [])
     };
     return {
         ...buildEntryToolResult({

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

@@ -251,9 +251,9 @@ export declare const inspectMinecraftShape: {
     }, z.core.$strip>], "kind">>;
     includeSnapshots: z.ZodDefault<z.ZodBoolean>;
     detail: z.ZodOptional<z.ZodEnum<{
+        full: "full";
         summary: "summary";
         standard: "standard";
-        full: "full";
     }>>;
     include: z.ZodOptional<z.ZodArray<z.ZodEnum<{
         [x: string]: string;
@@ -512,9 +512,9 @@ export declare const inspectMinecraftSchema: z.ZodObject<{
     }, z.core.$strip>], "kind">>;
     includeSnapshots: z.ZodDefault<z.ZodBoolean>;
     detail: z.ZodOptional<z.ZodEnum<{
+        full: "full";
         summary: "summary";
         standard: "standard";
-        full: "full";
     }>>;
     include: z.ZodOptional<z.ZodArray<z.ZodEnum<{
         [x: string]: string;

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

@@ -40,9 +40,9 @@ export declare const manageCacheShape: {
         projectPath: z.ZodOptional<z.ZodString>;
     }, z.core.$strip>>;
     detail: z.ZodOptional<z.ZodEnum<{
+        full: "full";
         summary: "summary";
         standard: "standard";
-        full: "full";
     }>>;
     include: z.ZodOptional<z.ZodArray<z.ZodEnum<{
         [x: string]: string;
@@ -94,9 +94,9 @@ export declare const manageCacheSchema: z.ZodObject<{
         projectPath: z.ZodOptional<z.ZodString>;
     }, z.core.$strip>>;
     detail: z.ZodOptional<z.ZodEnum<{
+        full: "full";
         summary: "summary";
         standard: "standard";
-        full: "full";
     }>>;
     include: z.ZodOptional<z.ZodArray<z.ZodEnum<{
         [x: string]: string;

package/dist/entry-tools/manage-cache-service.js

@@ -225,23 +225,19 @@ export class ManageCacheService {
                         include,
                         summary: {
                             status: output.rebuiltEntries > 0 && executionMode === "apply" ? "changed" : "unchanged",
-                            headline: `${executionMode === "apply" ? "Applied" : "Previewed"} rebuild for ${cacheKinds.length} cache kind(s).`,
+                            // Neither headline may read as work done: rebuild is not
+                            // implemented for any cache kind, so apply changes nothing and
+                            // preview has nothing to promise.
+                            headline: executionMode === "apply"
+                                ? `Rebuild is not implemented for ${cacheKinds.length} cache kind(s); nothing changed.`
+                                : `Previewed rebuild for ${cacheKinds.length} cache kind(s): nothing would change.`,
                             subject: summarySubject,
                             counts: {
                                 rebuiltEntries: output.rebuiltEntries
-                            },
-                            ...(executionMode === "preview"
-                                ? {
-                                    nextActions: [
-                                        createNextAction("manage-cache", {
-                                            action: "rebuild",
-                                            cacheKinds,
-                                            executionMode: "apply",
-                                            selector: input.selector
-                                        })
-                                    ]
-                                }
-                                : {})
+                            }
+                            // No apply follow-up: rebuild is not implemented for any cache
+                            // kind, so advertising one would promise a call that changes
+                            // nothing. The registry's warning names what to do instead.
                         },
                         blocks: {
                             operation: {

package/dist/entry-tools/validate-project/cases/access-transformer.js

@@ -34,9 +34,28 @@ export async function handleAccessTransformer(deps, input, detail, include) {
             }
         });
     }
-    const content = input.subject.input.mode === "inline"
-        ? input.subject.input.content
-        : await readFile(input.subject.input.path, "utf8");
+    let content;
+    if (input.subject.input.mode === "inline") {
+        content = input.subject.input.content;
+    }
+    else {
+        const atPath = input.subject.input.path;
+        try {
+            content = await readFile(atPath, "utf8");
+        }
+        catch (err) {
+            const osCode = err?.code ?? "UNKNOWN";
+            throw createError({
+                code: ERROR_CODES.INVALID_INPUT,
+                message: `Could not read access transformer file "${atPath}": ${osCode} - ${err instanceof Error ? err.message : String(err)}`,
+                details: {
+                    task: "access-transformer",
+                    failedStage: "input-validation",
+                    nextAction: `Verify the file exists and is readable at "${atPath}".`
+                }
+            });
+        }
+    }
     if (!deps.validateAccessTransformer) {
         throw createError({
             code: ERROR_CODES.CONTEXT_UNRESOLVED,

package/dist/entry-tools/validate-project/cases/access-widener.js

@@ -34,9 +34,28 @@ export async function handleAccessWidener(deps, input, detail, include) {
             }
         });
     }
-    const content = input.subject.input.mode === "inline"
-        ? input.subject.input.content
-        : await readFile(input.subject.input.path, "utf8");
+    let content;
+    if (input.subject.input.mode === "inline") {
+        content = input.subject.input.content;
+    }
+    else {
+        const awPath = input.subject.input.path;
+        try {
+            content = await readFile(awPath, "utf8");
+        }
+        catch (err) {
+            const osCode = err?.code ?? "UNKNOWN";
+            throw createError({
+                code: ERROR_CODES.INVALID_INPUT,
+                message: `Could not read access widener file "${awPath}": ${osCode} - ${err instanceof Error ? err.message : String(err)}`,
+                details: {
+                    task: "access-widener",
+                    failedStage: "input-validation",
+                    nextAction: `Verify the file exists and is readable at "${awPath}".`
+                }
+            });
+        }
+    }
     const output = await deps.validateAccessWidener({
         content,
         version: input.version,
@@ -45,6 +64,14 @@ export async function handleAccessWidener(deps, input, detail, include) {
         scope: input.scope,
         preferProjectVersion: input.preferProjectVersion
     });
+    const issueEntries = Array.isArray(output.entries)
+        ? output.entries.filter((entry) => {
+            if (!entry || typeof entry !== "object" || !("valid" in entry)) {
+                return true;
+            }
+            return entry.valid !== true;
+        })
+        : undefined;
     return {
         ...buildEntryToolResult({
             task: "access-widener",
@@ -76,7 +103,7 @@ export async function handleAccessWidener(deps, input, detail, include) {
                         invalid: output.valid ? 0 : 1
                     }
                 },
-                issues: include.includes("issues") || detail !== "summary" ? output.issues : undefined
+                issues: include.includes("issues") || detail !== "summary" ? issueEntries : undefined
             },
             alwaysBlocks: ["project"]
         }),

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

@@ -74,14 +74,21 @@ export async function handleMixin(deps, input, detail, include) {
     const summary = output.summary;
     const invalidCount = summary?.invalid ?? 0;
     const partialCount = summary?.partial ?? 0;
+    const processingErrorsCount = summary?.processingErrors ?? 0;
     return {
         ...buildEntryToolResult({
             task: "mixin",
             detail,
             include,
             summary: {
-                status: invalidCount > 0 ? "invalid" : partialCount > 0 ? "partial" : "ok",
-                headline: `Validated ${summary?.total ?? 0} mixin input(s).`,
+                status: invalidCount > 0
+                    ? "invalid"
+                    : partialCount > 0 || processingErrorsCount > 0
+                        ? "partial"
+                        : "ok",
+                headline: processingErrorsCount > 0
+                    ? `Validated ${summary?.total ?? 0} mixin input(s); ${processingErrorsCount} could not be processed.`
+                    : `Validated ${summary?.total ?? 0} mixin input(s).`,
                 subject: createSummarySubject({
                     task: "mixin",
                     kind: input.subject.kind,
@@ -94,7 +101,8 @@ export async function handleMixin(deps, input, detail, include) {
                 counts: {
                     valid: summary?.valid ?? 0,
                     partial: partialCount,
-                    invalid: invalidCount
+                    invalid: invalidCount,
+                    processingErrors: processingErrorsCount
                 }
             },
             blocks: {

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

@@ -1,7 +1,7 @@
 import { readFile } from "node:fs/promises";
 import { mapWithConcurrencyLimit } from "../../../concurrency.js";
 import { buildEntryToolResult, createSummarySubject } from "../../response-contract.js";
-import { ERROR_CODES, createError } from "../../../errors.js";
+import { ERROR_CODES, createError, isAppError } from "../../../errors.js";
 import { buildEarlyTasksForBlocked, buildFullTaskStatusReport } from "../internal.js";
 // Telemetry failures must not change validation outcomes; swallow rejections
 // so a broken emitter does not abort the summary or count as a validation error.
@@ -15,6 +15,40 @@ async function safeEmit(emitter, stage, payload) {
         // swallow telemetry failure
     }
 }
+const DISCOVER_KIND_LABELS = [
+    ["mixins", "mixin configs"],
+    ["access-wideners", "access wideners"],
+    ["access-transformers", "access transformers"]
+];
+function joinWithOr(items) {
+    if (items.length <= 2) {
+        return items.join(" or ");
+    }
+    return `${items.slice(0, -1).join(", ")}, or ${items[items.length - 1]}`;
+}
+// A run that discovered no files keeps status "ok" (nothing failed), so the
+// headline and a warning must say that nothing was checked; otherwise the
+// result reads as a pass. Only the kinds subject.discover searched are named.
+function describeNothingToValidate(discover, projectPath) {
+    const searched = DISCOVER_KIND_LABELS
+        .filter(([kind]) => discover.includes(kind))
+        .map(([, label]) => label);
+    const notOk = "so status \"ok\" does not mean any file passed validation.";
+    const accessTransformerHint = discover.includes("access-transformers")
+        ? ""
+        : " Access Transformer files are searched only when subject.discover includes \"access-transformers\".";
+    if (searched.length === 0) {
+        return {
+            headline: "Nothing to validate: subject.discover selected no file kinds to search.",
+            warning: `Nothing was validated: subject.discover selected no file kinds to search, ${notOk}${accessTransformerHint}`
+        };
+    }
+    const list = joinWithOr(searched);
+    return {
+        headline: `Nothing to validate: no ${list} were found.`,
+        warning: `Nothing was validated: no ${list} were found under ${projectPath}, ${notOk}${accessTransformerHint}`
+    };
+}
 export async function handleProjectSummary(deps, input, detail, include, options = {}) {
     // Forwarded emitter for nested validators and probes; same swallow contract
     // as safeEmit so a rejecting raw emitter cannot leak into their outcomes.
@@ -27,7 +61,9 @@ export async function handleProjectSummary(deps, input, detail, include, options
             message: "task=project-summary requires subject.kind=workspace."
         });
     }
-    if (!input.version && !input.preferProjectVersion) {
+    // An omitted version is inferred from the project below; only an explicit
+    // preferProjectVersion=false opts out of that inference.
+    if (!input.version && input.preferProjectVersion === false) {
         const baseResult = buildEntryToolResult({
             task: "project-summary",
             detail,
@@ -52,7 +88,7 @@ export async function handleProjectSummary(deps, input, detail, include, options
                     }
                 ],
                 notes: [
-                    "Pass version explicitly, or retry with preferProjectVersion=true when gradle.properties declares the Minecraft version. The suggested retry sets preferProjectVersion=true for you."
+                    "Version inference was suppressed by preferProjectVersion=false. Pass version explicitly, or retry without preferProjectVersion=false to infer it from gradle.properties. The suggested retry sets preferProjectVersion=true for you."
                 ]
             },
             blocks: {
@@ -79,10 +115,21 @@ export async function handleProjectSummary(deps, input, detail, include, options
         projectPath,
         discover
     });
-    const detectedProjectVersion = input.preferProjectVersion
+    // preferProjectVersion=true overrides an explicit version; an omitted version is
+    // inferred (the explicit-false opt-out returned above).
+    const detectedProjectVersion = input.preferProjectVersion === true || !input.version
         ? await deps.detectProjectMinecraftVersion?.(projectPath)
         : undefined;
     const resolvedVersion = detectedProjectVersion ?? input.version;
+    const versionInferred = !input.version && Boolean(detectedProjectVersion);
+    // An inferred version is a project-version resolution: sub-validators and the
+    // artifact probe see exactly what an explicit preferProjectVersion=true run sees.
+    const effectivePreferProjectVersion = versionInferred ? true : input.preferProjectVersion;
+    const versionInferenceWarnings = versionInferred
+        ? [
+            `version was inferred from the workspace: ${detectedProjectVersion} (source: projectPath:gradle.properties (${projectPath})).`
+        ]
+        : [];
     const [mixinConfigs, accessWideners, accessTransformers] = await Promise.all([
         discover.includes("mixins")
             ? deps.discoverMixins(projectPath, input.configPaths)
@@ -111,17 +158,20 @@ export async function handleProjectSummary(deps, input, detail, include, options
                     sourcePriority: input.sourcePriority,
                     scope: input.scope
                 }),
+                // Retrying without a version would infer, fail, and block again, so the
+                // recovery asks for an explicit version instead.
                 nextActions: [
                     {
                         tool: "validate-project",
                         params: {
                             task: "project-summary",
-                            subject: input.subject
+                            subject: input.subject,
+                            version: "<minecraft-version>"
                         }
                     }
                 ],
                 notes: [
-                    "Pass version explicitly, or make sure gradle.properties declares the Minecraft version before using preferProjectVersion=true."
+                    "The Minecraft version could not be inferred from gradle.properties (minecraft_version, mc_version, or minecraftVersion). Retry with version set explicitly: replace \"<minecraft-version>\" in the suggested call with the project's Minecraft version, or declare it in gradle.properties."
                 ]
             },
             blocks: {
@@ -150,14 +200,16 @@ export async function handleProjectSummary(deps, input, detail, include, options
             ]
         };
     }
+    // Reached only when discovery found nothing (the branch above handles found files).
     if (!resolvedVersion) {
+        const nothingToValidate = describeNothingToValidate(discover, projectPath);
         const baseResult = buildEntryToolResult({
             task: "project-summary",
             detail,
             include,
             summary: {
                 status: "ok",
-                headline: `Validated ${mixinConfigs.length} mixin config(s), ${accessWideners.length} access widener(s), and ${accessTransformers.length} access transformer(s).`,
+                headline: nothingToValidate.headline,
                 subject: createSummarySubject({
                     task: "project-summary",
                     kind: input.subject.kind,
@@ -187,15 +239,16 @@ export async function handleProjectSummary(deps, input, detail, include, options
         return {
             ...baseResult,
             ...(tasks ? { tasks } : {}),
-            warnings: []
+            warnings: [nothingToValidate.warning]
         };
     }
     const validationVersion = resolvedVersion;
-    const warnings = [];
+    const warnings = [...versionInferenceWarnings];
     const mixinDurationStart = Date.now();
     let validMixins = 0;
     let partialMixins = 0;
     let invalidMixins = 0;
+    let mixinProcessingErrors = 0;
     let mixinCaughtErrors = 0;
     await safeEmit(options.stageEmitter, "validate-project:mixin-validation", {
         targetTotal: mixinConfigs.length
@@ -244,6 +297,8 @@ export async function handleProjectSummary(deps, input, detail, include, options
                 valid: summary?.valid ?? 0,
                 partial: summary?.partial ?? 0,
                 invalid: summary?.invalid ?? 0,
+                // Mixin sources that failed to load count in none of valid/partial/invalid.
+                processingErrors: summary?.processingErrors ?? 0,
                 caught: 0,
                 warnings: Array.isArray(mixinResult.warnings) ? [...mixinResult.warnings] : []
             };
@@ -253,6 +308,7 @@ export async function handleProjectSummary(deps, input, detail, include, options
                 valid: 0,
                 partial: 0,
                 invalid: 1,
+                processingErrors: 0,
                 caught: 1,
                 warnings: error instanceof Error ? [`${configPath}: ${error.message}`] : []
             };
@@ -262,6 +318,7 @@ export async function handleProjectSummary(deps, input, detail, include, options
         validMixins += result.valid;
         partialMixins += result.partial;
         invalidMixins += result.invalid;
+        mixinProcessingErrors += result.processingErrors;
         mixinCaughtErrors += result.caught;
         warnings.push(...result.warnings);
     }
@@ -270,6 +327,7 @@ export async function handleProjectSummary(deps, input, detail, include, options
     let validAw = 0;
     let invalidAw = 0;
     let awCaughtErrors = 0;
+    let awProcessingErrors = 0;
     await safeEmit(options.stageEmitter, "validate-project:access-widener-validation", {
         targetTotal: accessWideners.length
     });
@@ -288,7 +346,7 @@ export async function handleProjectSummary(deps, input, detail, include, options
                 projectPath,
                 gradleUserHome,
                 scope: input.scope,
-                preferProjectVersion: input.preferProjectVersion
+                preferProjectVersion: effectivePreferProjectVersion
             });
             if (output.valid) {
                 validAw += 1;
@@ -301,6 +359,12 @@ export async function handleProjectSummary(deps, input, detail, include, options
             }
         }
         catch (error) {
+            if (isAppError(error) && error.details?.unsupportedFormat === "classTweaker") {
+                // A format this server does not read is not a verdict on the file.
+                awProcessingErrors += 1;
+                warnings.push(`${awPath}: ${error.message}`);
+                continue;
+            }
             invalidAw += 1;
             awCaughtErrors += 1;
             if (error instanceof Error) {
@@ -337,7 +401,7 @@ export async function handleProjectSummary(deps, input, detail, include, options
                 projectPath,
                 gradleUserHome,
                 scope: input.scope,
-                preferProjectVersion: input.preferProjectVersion
+                preferProjectVersion: effectivePreferProjectVersion
             });
             if (output.valid) {
                 validAt += 1;
@@ -360,14 +424,26 @@ export async function handleProjectSummary(deps, input, detail, include, options
     const atDurationMs = Date.now() - atDurationStart;
     const invalidCount = invalidMixins + invalidAw + invalidAt;
     const partialCount = partialMixins;
-    const status = invalidCount > 0 ? "invalid" : partialCount > 0 ? "partial" : "ok";
+    const processingErrors = mixinProcessingErrors + awProcessingErrors;
+    const status = invalidCount > 0
+        ? "invalid"
+        : partialCount > 0 || processingErrors > 0
+            ? "partial"
+            : "ok";
+    const nothingToValidate = mixinConfigs.length === 0 && accessWideners.length === 0 && accessTransformers.length === 0
+        ? describeNothingToValidate(discover, projectPath)
+        : undefined;
+    if (nothingToValidate) {
+        warnings.push(nothingToValidate.warning);
+    }
     const baseResult = buildEntryToolResult({
         task: "project-summary",
         detail,
         include,
         summary: {
             status,
-            headline: `Validated ${mixinConfigs.length} mixin config(s), ${accessWideners.length} access widener(s), and ${accessTransformers.length} access transformer(s).`,
+            headline: nothingToValidate?.headline ??
+                `Validated ${mixinConfigs.length} mixin config(s), ${accessWideners.length} access widener(s), and ${accessTransformers.length} access transformer(s)${mixinProcessingErrors > 0 ? `; ${mixinProcessingErrors} mixin input(s) could not be processed` : ""}${awProcessingErrors > 0 ? `; ${awProcessingErrors} access widener(s) could not be processed` : ""}.`,
             subject: createSummarySubject({
                 task: "project-summary",
                 kind: input.subject.kind,
@@ -381,7 +457,8 @@ export async function handleProjectSummary(deps, input, detail, include, options
             counts: {
                 valid: validMixins + validAw + validAt,
                 partial: partialCount,
-                invalid: invalidCount
+                invalid: invalidCount,
+                processingErrors
             }
         },
         blocks: {
@@ -389,7 +466,8 @@ export async function handleProjectSummary(deps, input, detail, include, options
                 summary: {
                     valid: validMixins + validAw + validAt,
                     partial: partialCount,
-                    invalid: invalidCount
+                    invalid: invalidCount,
+                    processingErrors
                 }
             },
             workspace: {
@@ -416,7 +494,7 @@ export async function handleProjectSummary(deps, input, detail, include, options
         sourcePriority: input.sourcePriority,
         gradleUserHome,
         scope: input.scope,
-        preferProjectVersion: input.preferProjectVersion,
+        preferProjectVersion: effectivePreferProjectVersion,
         mixinDiscoveryCount: mixinConfigs.length,
         mixinCaughtErrors,
         mixinCounts: { ok: validMixins, partial: partialMixins, invalid: invalidMixins },