@skillcap/gdh 0.25.5 → 0.26.1

package/node_modules/@gdh/authoring/dist/lsp.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"lsp.d.ts","sourceRoot":"","sources":["../src/lsp.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EACV,uBAAuB,EAEvB,0BAA0B,EAC1B,wBAAwB,EAExB,kBAAkB,EAKlB,yBAAyB,EACzB,gBAAgB,EAChB,mBAAmB,EACpB,MAAM,WAAW,CAAC;AAsInB,KAAK,6BAA6B,GAAG,OAAO,GAAG,MAAM,GAAG,SAAS,GAAG,OAAO,GAAG,QAAQ,CAAC;AACvF,KAAK,mCAAmC,GACpC,OAAO,GACP,UAAU,GACV,aAAa,GACb,SAAS,GACT,QAAQ,GACR,WAAW,CAAC;AAEhB,MAAM,WAAW,mCAAmC;IAClD,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,UAAU,EAAE,eAAe,CAAC;IACrC,QAAQ,CAAC,OAAO,EAAE,6BAA6B,CAAC;IAChD,QAAQ,CAAC,MAAM,EAAE,mCAAmC,CAAC;IACrD,QAAQ,CAAC,OAAO,EAAE,MAAM,CAAC;IACzB,QAAQ,CAAC,OAAO,EAAE,SAAS,MAAM,EAAE,CAAC;IACpC,QAAQ,CAAC,QAAQ,EAAE,mBAAmB,CAAC;IACvC,QAAQ,CAAC,SAAS,EAAE;QAClB,QAAQ,CAAC,MAAM,EAAE,mCAAmC,CAAC;QACrD,QAAQ,CAAC,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;QACnC,QAAQ,CAAC,QAAQ,EAAE,kBAAkB,GAAG,IAAI,CAAC;QAC7C,QAAQ,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;QAC5B,QAAQ,CAAC,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;KAC9B,CAAC;IACF,QAAQ,CAAC,SAAS,EAAE;QAClB,QAAQ,CAAC,MAAM,EAAE,SAAS,GAAG,WAAW,GAAG,SAAS,CAAC;QACrD,QAAQ,CAAC,OAAO,EAAE,SAAS,MAAM,EAAE,CAAC;QACpC,QAAQ,CAAC,UAAU,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,GAAG,IAAI,CAAC;KAC/D,CAAC;IACF,QAAQ,CAAC,WAAW,EAAE;QACpB,QAAQ,CAAC,MAAM,EAAE,eAAe,CAAC;QACjC,QAAQ,CAAC,OAAO,EAAE,MAAM,CAAC;KAC1B,CAAC;IACF,QAAQ,CAAC,OAAO,EAAE;QAChB,QAAQ,CAAC,MAAM,EAAE,MAAM,GAAG,aAAa,GAAG,SAAS,GAAG,QAAQ,GAAG,YAAY,CAAC;QAC9E,QAAQ,CAAC,MAAM,EAAE,MAAM,GAAG,IAAI,CAAC;QAC/B,QAAQ,CAAC,SAAS,EAAE,OAAO,CAAC;KAC7B,CAAC;IACF,QAAQ,CAAC,eAAe,EAAE,SAAS,MAAM,EAAE,CAAC;IAC5C,QAAQ,CAAC,YAAY,EAAE,yBAAyB,GAAG,IAAI,CAAC;CACzD;AAuGD,wBAAsB,mBAAmB,CAAC,KAAK,EAAE;IAC/C,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;IAChD,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IACxB,QAAQ,CAAC,YAAY,CAAC,EAAE,kBAAkB,GAAG,gBAAgB,CAAC;CAC/D,GAAG,OAAO,CAAC,yBAAyB,CAAC,CAuNrC;AAED,wBAAsB,eAAe,CAAC,KAAK,EAAE;IAC3C,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;CACjD,GAAG,OAAO,CAAC,mCAAmC,CAAC,CA6G/C;AAED,wBAAsB,cAAc,CAAC,KAAK,EAAE;IAC1C,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;CACjD,GAAG,OAAO,CAAC,mCAAmC,CAAC,CAyD/C;AAED,wBAAsB,iBAAiB,CAAC,KAAK,EAAE;IAC7C,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;IAChD,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;CACzB,GAAG,OAAO,CAAC,mCAAmC,CAAC,CAgC/C;AAED,wBAAsB,eAAe,CAAC,KAAK,EAAE;IAC3C,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;CACjD,GAAG,OAAO,CAAC,mCAAmC,CAAC,CA0H/C;AAED,wBAAsB,gBAAgB,CAAC,KAAK,EAAE;IAC5C,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;CACjD,GAAG,OAAO,CAAC,mCAAmC,CAAC,CAmB/C;AAED,wBAAsB,wBAAwB,CAAC,KAAK,EAAE;IACpD,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;IAChD,QAAQ,CAAC,SAAS,EAAE,yBAAyB,CAAC;IAC9C,QAAQ,CAAC,IAAI,CAAC,EAAE,OAAO,GAAG,WAAW,CAAC;IACtC,QAAQ,CAAC,YAAY,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC1C,QAAQ,CAAC,cAAc,CAAC,EAAE,0BAA0B,CAAC;CACtD,GAAG,OAAO,CAAC,uBAAuB,CAAC,CAwNnC;AA2SD,wBAAsB,yBAAyB,CAC7C,WAAW,EAAE,MAAM,EACnB,UAAU,EAAE,MAAM,EAClB,cAAc,EAAE,SAAS,MAAM,EAAE,EACjC,UAAU,EAAE,SAAS,MAAM,EAAE,GAC5B,OAAO,CAAC,SAAS,MAAM,EAAE,CAAC,CA6B5B;AA6ID,wBAAsB,wBAAwB,IAAI,OAAO,CAAC,IAAI,CAAC,CAE9D"}
+{"version":3,"file":"lsp.d.ts","sourceRoot":"","sources":["../src/lsp.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EACV,uBAAuB,EAEvB,2BAA2B,EAC3B,0BAA0B,EAC1B,wBAAwB,EAExB,kBAAkB,EAKlB,yBAAyB,EACzB,gBAAgB,EAEhB,mBAAmB,EACpB,MAAM,WAAW,CAAC;AA0InB,KAAK,6BAA6B,GAAG,OAAO,GAAG,MAAM,GAAG,SAAS,GAAG,OAAO,GAAG,QAAQ,CAAC;AACvF,KAAK,mCAAmC,GACpC,OAAO,GACP,UAAU,GACV,aAAa,GACb,SAAS,GACT,QAAQ,GACR,WAAW,CAAC;AAEhB,MAAM,WAAW,mCAAmC;IAClD,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,UAAU,EAAE,eAAe,CAAC;IACrC,QAAQ,CAAC,OAAO,EAAE,6BAA6B,CAAC;IAChD,QAAQ,CAAC,MAAM,EAAE,mCAAmC,CAAC;IACrD,QAAQ,CAAC,OAAO,EAAE,MAAM,CAAC;IACzB,QAAQ,CAAC,OAAO,EAAE,SAAS,MAAM,EAAE,CAAC;IACpC,QAAQ,CAAC,QAAQ,EAAE,mBAAmB,CAAC;IACvC,QAAQ,CAAC,SAAS,EAAE;QAClB,QAAQ,CAAC,MAAM,EAAE,mCAAmC,CAAC;QACrD,QAAQ,CAAC,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;QACnC,QAAQ,CAAC,QAAQ,EAAE,kBAAkB,GAAG,IAAI,CAAC;QAC7C,QAAQ,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;QAC5B,QAAQ,CAAC,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;KAC9B,CAAC;IACF,QAAQ,CAAC,SAAS,EAAE;QAClB,QAAQ,CAAC,MAAM,EAAE,SAAS,GAAG,WAAW,GAAG,SAAS,CAAC;QACrD,QAAQ,CAAC,OAAO,EAAE,SAAS,MAAM,EAAE,CAAC;QACpC,QAAQ,CAAC,UAAU,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,GAAG,IAAI,CAAC;KAC/D,CAAC;IACF,QAAQ,CAAC,WAAW,EAAE;QACpB,QAAQ,CAAC,MAAM,EAAE,eAAe,CAAC;QACjC,QAAQ,CAAC,OAAO,EAAE,MAAM,CAAC;KAC1B,CAAC;IACF,QAAQ,CAAC,OAAO,EAAE;QAChB,QAAQ,CAAC,MAAM,EAAE,MAAM,GAAG,aAAa,GAAG,SAAS,GAAG,QAAQ,GAAG,YAAY,CAAC;QAC9E,QAAQ,CAAC,MAAM,EAAE,MAAM,GAAG,IAAI,CAAC;QAC/B,QAAQ,CAAC,SAAS,EAAE,OAAO,CAAC;KAC7B,CAAC;IACF,QAAQ,CAAC,eAAe,EAAE,SAAS,MAAM,EAAE,CAAC;IAC5C,QAAQ,CAAC,YAAY,EAAE,yBAAyB,GAAG,IAAI,CAAC;CACzD;AAuGD,wBAAsB,mBAAmB,CAAC,KAAK,EAAE;IAC/C,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;IAChD,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IACxB,QAAQ,CAAC,YAAY,CAAC,EAAE,kBAAkB,GAAG,gBAAgB,CAAC;CAC/D,GAAG,OAAO,CAAC,yBAAyB,CAAC,CAkQrC;AA6BD,wBAAsB,gBAAgB,CAAC,KAAK,EAAE;IAC5C,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;IAChD,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;CACxB,GAAG,OAAO,CAAC,2BAA2B,CAAC,CAGvC;AAED,wBAAsB,eAAe,CAAC,KAAK,EAAE;IAC3C,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;CACjD,GAAG,OAAO,CAAC,mCAAmC,CAAC,CA6G/C;AAED,wBAAsB,cAAc,CAAC,KAAK,EAAE;IAC1C,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;CACjD,GAAG,OAAO,CAAC,mCAAmC,CAAC,CAuE/C;AAED,wBAAsB,iBAAiB,CAAC,KAAK,EAAE;IAC7C,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;IAChD,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;CACzB,GAAG,OAAO,CAAC,mCAAmC,CAAC,CAgC/C;AAED,wBAAsB,eAAe,CAAC,KAAK,EAAE;IAC3C,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;CACjD,GAAG,OAAO,CAAC,mCAAmC,CAAC,CA0H/C;AAED,wBAAsB,gBAAgB,CAAC,KAAK,EAAE;IAC5C,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;CACjD,GAAG,OAAO,CAAC,mCAAmC,CAAC,CAmB/C;AAED,wBAAsB,wBAAwB,CAAC,KAAK,EAAE;IACpD,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,QAAQ,CAAC,MAAM,EAAE,wBAAwB,CAAC;IAC1C,QAAQ,CAAC,aAAa,EAAE,gBAAgB,GAAG,IAAI,CAAC;IAChD,QAAQ,CAAC,SAAS,EAAE,yBAAyB,CAAC;IAC9C,QAAQ,CAAC,IAAI,CAAC,EAAE,OAAO,GAAG,WAAW,CAAC;IACtC,QAAQ,CAAC,YAAY,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC1C,QAAQ,CAAC,cAAc,CAAC,EAAE,0BAA0B,CAAC;CACtD,GAAG,OAAO,CAAC,uBAAuB,CAAC,CAwNnC;AA2SD,wBAAsB,yBAAyB,CAC7C,WAAW,EAAE,MAAM,EACnB,UAAU,EAAE,MAAM,EAClB,cAAc,EAAE,SAAS,MAAM,EAAE,EACjC,UAAU,EAAE,SAAS,MAAM,EAAE,GAC5B,OAAO,CAAC,SAAS,MAAM,EAAE,CAAC,CA6B5B;AA6ID,wBAAsB,wBAAwB,IAAI,OAAO,CAAC,IAAI,CAAC,CAE9D"}

package/node_modules/@gdh/authoring/dist/lsp.js

@@ -5,7 +5,7 @@ import fs from "node:fs/promises";
 import net from "node:net";
 import path from "node:path";
 import { fileURLToPath, pathToFileURL } from "node:url";
-import { GDH_MANAGED_LSP_SURFACE_VERSION, resolveConfiguredGodotEditorBin } from "@gdh/core";
+import { GDH_MANAGED_LSP_MIN_GODOT_VERSION, GDH_MANAGED_LSP_SURFACE_VERSION, resolveConfiguredGodotEditorBin, } from "@gdh/core";
 import { connectGodotLspClient, GdhLspClientError, } from "./lsp-client.js";
 import { summarizeBlockingDiagnostics } from "./diagnostics-summary.js";
 import { classifyDiagnosticsFreshness, DIAGNOSTICS_BROKER_DEFAULT_FRESHNESS_STALE_AFTER_MS, resolveDiagnosticsBrokerPaths, } from "./diagnostics-broker-contract.js";
@@ -122,149 +122,190 @@ export async function getManagedLspStatus(input) {
         return unavailable;
     }
     try {
-        return await withManagedLspStateLock(worktree, input.owner ?? "gdh lsp status", async (stateLock) => await withLease(worktree, input.owner ?? "gdh lsp status", async (lease, activeLeaseCount) => {
-            let cleanedStaleInstance = false;
-            let cleanupReasons = [];
+        return await withManagedLspStateLock(worktree, input.owner ?? "gdh lsp status", async (stateLock) => {
             const projectPath = path.resolve(input.targetPath, input.status.primaryProjectPath ?? ".");
             const expectedIdentity = await resolveCurrentManagedLspIdentity(worktree, projectPath);
-            const existing = await readPersistedInstance(worktree);
-            if (existing !== null) {
-                const validation = await validatePersistedInstance(existing.instance, expectedIdentity);
-                if (validation.reusable) {
-                    const reusedInstance = {
-                        ...existing.instance,
-                        lastValidatedAt: new Date().toISOString(),
-                        trust: validation.trust,
-                    };
-                    await persistInstance(worktree, {
-                        ...existing,
-                        instance: reusedInstance,
-                        rawPayload: validation.rawPayload ?? existing.rawPayload,
+            // Phase 81 / LSP-08 / D-17: refuse-to-launch when the configured
+            // Godot version is below the managed-LSP floor. Short-circuits
+            // BEFORE lsp-leases.json acquisition (withLease) and BEFORE
+            // persisted-instance reads — but executes inside the lsp.lock
+            // (stateLock) so identity resolution is serialized with other
+            // lifecycle mutations. Misconfigured 4.5.0/4.5.1 setups never
+            // acquire a lease or block other waiters.
+            if (expectedIdentity.editorVersionReason === "godot_editor_version_unsupported_for_lsp") {
+                return createUnavailableStatus({
+                    targetPath: input.targetPath,
+                    readiness: input.status.readiness,
+                    worktree,
+                    inventoryObservedAt: input.status.inventoryObservedAt,
+                    projectConfigPresent: input.projectConfig !== null,
+                    availability: "unavailable",
+                    reasons: dedupe([
+                        ...stateLock.recoveredReasons,
+                        "godot_editor_version_unsupported_for_lsp",
+                    ]),
+                    summary: "The configured Godot editor version does not meet the minimum version required for the managed authoring.lsp. Upgrade to Godot 4.5.2 or later.",
+                    versionFloorAdvisory: {
+                        detectedVersion: expectedIdentity.editorVersion,
+                        minimumVersion: GDH_MANAGED_LSP_MIN_GODOT_VERSION,
+                        recommendedVersions: ["4.5.2", "4.6.x"],
+                    },
+                });
+            }
+            return await withLease(worktree, input.owner ?? "gdh lsp status", async (lease, activeLeaseCount) => {
+                let cleanedStaleInstance = false;
+                let cleanupReasons = [];
+                const existing = await readPersistedInstance(worktree);
+                if (existing !== null) {
+                    const validation = await validatePersistedInstance(existing.instance, expectedIdentity);
+                    if (validation.reusable) {
+                        const reusedInstance = {
+                            ...existing.instance,
+                            lastValidatedAt: new Date().toISOString(),
+                            trust: validation.trust,
+                        };
+                        await persistInstance(worktree, {
+                            ...existing,
+                            instance: reusedInstance,
+                            rawPayload: validation.rawPayload ?? existing.rawPayload,
+                        });
+                        return createStatusResult({
+                            targetPath: input.targetPath,
+                            readiness: input.status.readiness,
+                            availability: "available",
+                            status: validation.trust === "trusted" ? "ready" : "degraded",
+                            validationMode: existing.validationMode,
+                            summary: validation.trust === "trusted"
+                                ? "Reused the managed authoring.lsp instance for this worktree."
+                                : "Reused the managed authoring.lsp instance, but trust is degraded and the instance should be treated cautiously.",
+                            reasons: dedupe([
+                                ...stateLock.recoveredReasons,
+                                ...(validation.trust === "trusted" ? [] : ["lsp_instance_degraded"]),
+                            ]),
+                            worktree,
+                            lease,
+                            activeLeaseCount,
+                            reusedInstance: true,
+                            cleanedStaleInstance,
+                            instance: reusedInstance,
+                            inventoryObservedAt: input.status.inventoryObservedAt,
+                            projectConfigPresent: input.projectConfig !== null,
+                            rawPayload: validation.rawPayload ?? existing.rawPayload,
+                        });
+                    }
+                    cleanedStaleInstance = true;
+                    cleanupReasons = validation.reasons;
+                    await cleanupPersistedInstance(worktree, existing.instance, validation.reason, {
+                        expectedIdentity,
                     });
-                    return createStatusResult({
+                    // Phase 81 / LSP-04 / D-11 / Pitfall 7: when the LSP instance
+                    // identity has changed, the broker snapshot referenced the
+                    // now-defunct instance. Prune so the next read returns
+                    // broker_metadata_missing instead of lsp_instance_identity_mismatch.
+                    {
+                        const { pruneAuthoringDiagnostics } = await import("./diagnostics-broker.js");
+                        await pruneAuthoringDiagnostics({
+                            targetPath: input.targetPath,
+                            status: input.status,
+                            projectConfig: input.projectConfig,
+                        });
+                    }
+                }
+                if (input.launchPolicy === "reuse_existing") {
+                    return createUnavailableStatus({
                         targetPath: input.targetPath,
                         readiness: input.status.readiness,
-                        availability: "available",
-                        status: validation.trust === "trusted" ? "ready" : "degraded",
-                        validationMode: existing.validationMode,
-                        summary: validation.trust === "trusted"
-                            ? "Reused the managed authoring.lsp instance for this worktree."
-                            : "Reused the managed authoring.lsp instance, but trust is degraded and the instance should be treated cautiously.",
+                        worktree,
+                        inventoryObservedAt: input.status.inventoryObservedAt,
+                        projectConfigPresent: input.projectConfig !== null,
+                        availability: lspCapability?.availability ?? "available",
                         reasons: dedupe([
                             ...stateLock.recoveredReasons,
-                            ...(validation.trust === "trusted" ? [] : ["lsp_instance_degraded"]),
+                            ...(cleanedStaleInstance ? ["stale_lsp_instance_cleaned"] : []),
+                            ...cleanupReasons,
+                            "lsp_instance_not_running",
+                            "post_edit_fast_path_did_not_launch_lsp",
                         ]),
-                        worktree,
+                        summary: "No reusable managed authoring.lsp instance is currently running; post-edit fast check did not launch Godot.",
                         lease,
                         activeLeaseCount,
-                        reusedInstance: true,
                         cleanedStaleInstance,
-                        instance: reusedInstance,
+                        rawPayload: {
+                            launchPolicy: "reuse_existing",
+                            projectPath,
+                            cleanedStaleInstance,
+                            cleanupReasons,
+                        },
+                    });
+                }
+                const launchResult = await launchManagedInstance(worktree, projectPath);
+                if ("unavailable" in launchResult) {
+                    return createUnavailableStatus({
+                        targetPath: input.targetPath,
+                        readiness: input.status.readiness,
+                        worktree,
                         inventoryObservedAt: input.status.inventoryObservedAt,
                         projectConfigPresent: input.projectConfig !== null,
-                        rawPayload: validation.rawPayload ?? existing.rawPayload,
+                        availability: "unavailable",
+                        reasons: [
+                            ...stateLock.recoveredReasons,
+                            ...(cleanedStaleInstance ? ["stale_lsp_instance_cleaned"] : []),
+                            ...cleanupReasons,
+                            ...launchResult.unavailable.reasons,
+                        ],
+                        summary: launchResult.unavailable.summary,
+                        lease,
+                        activeLeaseCount,
+                        cleanedStaleInstance,
+                        rawPayload: launchResult.unavailable.rawPayload,
                     });
                 }
-                cleanedStaleInstance = true;
-                cleanupReasons = validation.reasons;
-                await cleanupPersistedInstance(worktree, existing.instance, validation.reason, {
-                    expectedIdentity,
+                const launchedInstance = {
+                    instanceId: launchResult.instanceId,
+                    launcher: launchResult.launcher,
+                    port: launchResult.port,
+                    pid: launchResult.pid,
+                    projectPath,
+                    command: launchResult.command,
+                    trust: launchResult.trust,
+                    launchedAt: new Date().toISOString(),
+                    lastValidatedAt: new Date().toISOString(),
+                    identity: launchResult.identity,
+                };
+                await persistInstance(worktree, {
+                    schemaVersion: LSP_INSTANCE_SCHEMA_VERSION,
+                    instance: launchedInstance,
+                    validationMode: launchResult.validationMode,
+                    diagnostics: launchResult.diagnostics,
+                    rawPayload: launchResult.rawPayload,
                 });
-            }
-            if (input.launchPolicy === "reuse_existing") {
-                return createUnavailableStatus({
+                return createStatusResult({
                     targetPath: input.targetPath,
                     readiness: input.status.readiness,
-                    worktree,
-                    inventoryObservedAt: input.status.inventoryObservedAt,
-                    projectConfigPresent: input.projectConfig !== null,
-                    availability: lspCapability?.availability ?? "available",
+                    availability: "available",
+                    status: launchResult.trust === "trusted" ? "ready" : "degraded",
+                    validationMode: launchResult.validationMode,
+                    summary: launchResult.trust === "trusted"
+                        ? "Started a managed authoring.lsp instance for this worktree."
+                        : "Started a managed authoring.lsp instance, but trust is degraded and validation should be treated cautiously.",
                     reasons: dedupe([
                         ...stateLock.recoveredReasons,
                         ...(cleanedStaleInstance ? ["stale_lsp_instance_cleaned"] : []),
                         ...cleanupReasons,
-                        "lsp_instance_not_running",
-                        "post_edit_fast_path_did_not_launch_lsp",
+                        ...(launchResult.trust === "trusted" ? [] : ["lsp_instance_degraded"]),
                     ]),
-                    summary: "No reusable managed authoring.lsp instance is currently running; post-edit fast check did not launch Godot.",
+                    worktree,
                     lease,
                     activeLeaseCount,
+                    reusedInstance: false,
                     cleanedStaleInstance,
-                    rawPayload: {
-                        launchPolicy: "reuse_existing",
-                        projectPath,
-                        cleanedStaleInstance,
-                        cleanupReasons,
-                    },
-                });
-            }
-            const launchResult = await launchManagedInstance(worktree, projectPath);
-            if ("unavailable" in launchResult) {
-                return createUnavailableStatus({
-                    targetPath: input.targetPath,
-                    readiness: input.status.readiness,
-                    worktree,
+                    instance: launchedInstance,
                     inventoryObservedAt: input.status.inventoryObservedAt,
                     projectConfigPresent: input.projectConfig !== null,
-                    availability: "unavailable",
-                    reasons: [
-                        ...stateLock.recoveredReasons,
-                        ...(cleanedStaleInstance ? ["stale_lsp_instance_cleaned"] : []),
-                        ...cleanupReasons,
-                        ...launchResult.unavailable.reasons,
-                    ],
-                    summary: launchResult.unavailable.summary,
-                    lease,
-                    activeLeaseCount,
-                    cleanedStaleInstance,
-                    rawPayload: launchResult.unavailable.rawPayload,
+                    rawPayload: launchResult.rawPayload,
                 });
-            }
-            const launchedInstance = {
-                instanceId: launchResult.instanceId,
-                launcher: launchResult.launcher,
-                port: launchResult.port,
-                pid: launchResult.pid,
-                projectPath,
-                command: launchResult.command,
-                trust: launchResult.trust,
-                launchedAt: new Date().toISOString(),
-                lastValidatedAt: new Date().toISOString(),
-                identity: launchResult.identity,
-            };
-            await persistInstance(worktree, {
-                schemaVersion: LSP_INSTANCE_SCHEMA_VERSION,
-                instance: launchedInstance,
-                validationMode: launchResult.validationMode,
-                diagnostics: launchResult.diagnostics,
-                rawPayload: launchResult.rawPayload,
             });
-            return createStatusResult({
-                targetPath: input.targetPath,
-                readiness: input.status.readiness,
-                availability: "available",
-                status: launchResult.trust === "trusted" ? "ready" : "degraded",
-                validationMode: launchResult.validationMode,
-                summary: launchResult.trust === "trusted"
-                    ? "Started a managed authoring.lsp instance for this worktree."
-                    : "Started a managed authoring.lsp instance, but trust is degraded and validation should be treated cautiously.",
-                reasons: dedupe([
-                    ...stateLock.recoveredReasons,
-                    ...(cleanedStaleInstance ? ["stale_lsp_instance_cleaned"] : []),
-                    ...cleanupReasons,
-                    ...(launchResult.trust === "trusted" ? [] : ["lsp_instance_degraded"]),
-                ]),
-                worktree,
-                lease,
-                activeLeaseCount,
-                reusedInstance: false,
-                cleanedStaleInstance,
-                instance: launchedInstance,
-                inventoryObservedAt: input.status.inventoryObservedAt,
-                projectConfigPresent: input.projectConfig !== null,
-                rawPayload: launchResult.rawPayload,
-            });
-        }));
+        });
     }
     catch (error) {
         if (error instanceof GdhManagedLspStateLockError) {
@@ -282,6 +323,36 @@ export async function getManagedLspStatus(input) {
         throw error;
     }
 }
+// ─────────────────────────────────────────────────────────────────────────────
+// Phase 82 / LSP-02 / LSP-07 — managed-LSP warmup surface.
+//
+// `warmupManagedLsp` is the in-process entry point that the post-edit hook
+// detach-spawns (via `gdh lsp warmup`) and the MCP `authoring.warmup` tool
+// invokes. It is idempotent: concurrent callers fan in via a single-attempt
+// `<targetPath>/.gdh-state/lsp.lock` acquire so the underlying Godot launch
+// happens at most once.
+//
+// The implementation lives in `./lsp-warmup.js`. That sibling module imports
+// `getManagedLspStatus` from `./lsp.js`, which is the documented vitest
+// workaround for the "vi.mock cannot intercept same-module internal calls"
+// pitfall (https://vitest.dev/guide/mocking/modules — Internal Method Calls).
+//
+// IMPORTANT: this is a thin wrapper that DEFERS to the impl via dynamic
+// `await import("./lsp-warmup.js")`. The deferral guarantees `lsp-warmup.ts`
+// is NOT a static dependency of `lsp.ts`. Were it static (a re-export), it
+// would be evaluated as part of `./lsp.js`'s module graph, with its
+// `import { getManagedLspStatus } from "./lsp.js"` resolving against the
+// partially-evaluated original module record — so the unit-test mock
+// (`vi.mock("./lsp.js", ...)`) would never reach the warmup impl. By
+// keeping it deferred, the test's `await import("./lsp-warmup.js")` is
+// the FIRST point at which lsp-warmup.ts loads, and at that time the mock
+// is already registered, so the `./lsp.js` import resolves to the mocked
+// module. See `./lsp-warmup.ts` and `./lsp-warmup.test.ts` for context.
+// ─────────────────────────────────────────────────────────────────────────────
+export async function warmupManagedLsp(input) {
+    const { warmupManagedLsp: impl } = await import("./lsp-warmup.js");
+    return impl(input);
+}
 export async function checkManagedLsp(input) {
     const worktree = resolveWorktreeIdentity(input.targetPath, input.status, input.projectConfig);
     return await withManagedLspLifecycleLock(worktree, "check", "gdh lsp check", async (stateLock) => {
@@ -404,6 +475,19 @@ export async function stopManagedLsp(input) {
             killProcess: true,
             expectedIdentity,
         });
+        // Phase 81 / LSP-04 / D-11: prune the broker snapshot inside the lifecycle
+        // lock so concurrent readers cannot observe a stale snapshot pointing at
+        // the just-stopped LSP instance. The .primed marker is intentionally
+        // preserved (D-10) — broker absence remains "broker_metadata_missing"
+        // until the next refresh, distinct from "broker_not_yet_primed".
+        {
+            const { pruneAuthoringDiagnostics } = await import("./diagnostics-broker.js");
+            await pruneAuthoringDiagnostics({
+                targetPath: input.targetPath,
+                status: input.status,
+                projectConfig: input.projectConfig,
+            });
+        }
         return createLifecycleCommandResult({
             targetPath: input.targetPath,
             command: "stop",
@@ -1152,13 +1236,20 @@ function resolveWorktreeIdentity(targetPath, status, projectConfig) {
 async function resolveCurrentManagedLspIdentity(worktree, projectPath) {
     const testMode = process.env["GDH_TEST_LSP_MODE"];
     if (testMode) {
+        const syntheticVersion = normalizeTestMode(testMode);
+        // Phase 81 / LSP-08 / D-15: honor the version floor for synthetic test versions
+        // so unit tests can exercise the floor-blocked path via GDH_TEST_LSP_MODE.
+        const syntheticFloorViolation = typeof syntheticVersion === "string" &&
+            (syntheticVersion.startsWith("4.5.0") || syntheticVersion.startsWith("4.5.1"));
         return await createManagedLspInstanceIdentity({
             worktree,
             projectPath,
             launcher: "test_fake",
-            editorBinPath: `test_fake:${normalizeTestMode(testMode)}`,
-            editorVersion: normalizeTestMode(testMode),
-            editorVersionReason: null,
+            editorBinPath: `test_fake:${syntheticVersion}`,
+            editorVersion: syntheticVersion,
+            editorVersionReason: syntheticFloorViolation
+                ? "godot_editor_version_unsupported_for_lsp"
+                : null,
         });
     }
     const godotBin = await resolveConfiguredGodotEditorBin({
@@ -1166,15 +1257,25 @@ async function resolveCurrentManagedLspIdentity(worktree, projectPath) {
         environment: process.env,
     });
     const version = godotBin === null ? null : await probeGodotEditorVersion(godotBin);
+    const versionStr = version?.version ?? null;
+    // Phase 81 / LSP-08 / D-15 / Pitfall 4: Godot reports versions like
+    // "4.5.0.stable" or "4.5.1.stable.official"; use a startsWith prefix match
+    // on the 3-part version prefix to handle every suffix form. No semver
+    // dependency is added — the floor check is intentionally narrow.
+    const isFloorViolation = versionStr !== null &&
+        (versionStr.startsWith("4.5.0") || versionStr.startsWith("4.5.1"));
+    const editorVersionReason = godotBin === null
+        ? "godot_editor_not_configured"
+        : isFloorViolation
+            ? "godot_editor_version_unsupported_for_lsp"
+            : (version?.reason ?? null);
     return await createManagedLspInstanceIdentity({
         worktree,
         projectPath,
         launcher: "godot_editor",
         editorBinPath: godotBin,
-        editorVersion: version?.version ?? null,
-        editorVersionReason: godotBin === null
-            ? "godot_editor_not_configured"
-            : (version?.reason ?? "godot_editor_version_unavailable"),
+        editorVersion: versionStr,
+        editorVersionReason,
     });
 }
 async function createManagedLspInstanceIdentity(input) {
@@ -2026,6 +2127,7 @@ function createUnavailableStatus(input) {
             stateRootPath: input.worktree.stateRootPath,
             rawPayload: input.rawPayload ?? null,
         },
+        versionFloorAdvisory: input.versionFloorAdvisory ?? null,
     };
 }
 function isPidAlive(pid) {