@skillcap/gdh 0.25.3 → 0.25.4

package/INSTALL-BUNDLE.json

@@ -1,5 +1,5 @@
 {
   "product": "GDH",
-  "version": "0.25.3",
+  "version": "0.25.4",
   "installMode": "packaged_install"
 }

package/RELEASE-SPAN-UPDATE-CONTRACTS.json

@@ -2769,6 +2769,61 @@
           }
         ]
       }
+    },
+    {
+      "version": "0.25.4",
+      "releaseTag": "v0.25.4",
+      "migrationStatus": "required",
+      "summary": "v0.25.4 re-bakes managed authoring hook scripts so post-edit feedback is bounded, timeout-safe, and cleaner in Codex hook output. GDH_UPDATE_HOOK_VERSION bumps from 7 to 8.",
+      "releaseHighlights": {
+        "summary": "v0.25.4 makes managed post-edit authoring hooks faster and cleaner. Hooks now treat broker refresh as a bounded, non-fatal warmup, cap the post-edit authoring check, and only block when the compact check reports blocking diagnostics. Codex hook context no longer repeats the full diagnostic payload already present in feedback.",
+        "operatorChanges": [
+          "**Post-edit hooks are bounded.** Diagnostics broker refresh defaults to a 5s timeout and post-edit authoring check defaults to a 15s timeout instead of allowing each `npx` call to run for up to 120s.",
+          "**Timeouts do not block edit flow.** If post-edit diagnostics time out, are unconfirmed, or fail before producing diagnostics, hooks continue without blocking and remind Codex agents to run final authoring validation before claiming code-validity.",
+          "**Blocking output is easier to read.** Blocking diagnostics are separated from the hook preface by a blank line, long output is truncated with a rerun hint, and Codex `additionalContext` is a short summary instead of a duplicate copy of the diagnostics."
+        ]
+      },
+      "updateContract": {
+        "summary": "Managed targets must re-bake authoring hook scripts for v0.25.4. The update replaces long-running/noisy post-edit hook behavior with bounded opportunistic feedback.",
+        "steps": [
+          {
+            "id": "install_v0_25_4",
+            "kind": "mechanical",
+            "summary": "Install the v0.25.4 GDH package.",
+            "detail": "Run gdh self-update so the target uses the bounded authoring hook renderer.",
+            "commands": [
+              "gdh self-update --apply"
+            ],
+            "validationCommands": [
+              "gdh status"
+            ]
+          },
+          {
+            "id": "rebake_authoring_hooks",
+            "kind": "mechanical",
+            "summary": "Re-bake managed hook scripts.",
+            "detail": "Run migration/apply if the update reports pending lifecycle work. This rewrites managed Claude and Codex authoring hook scripts with hook version 8.",
+            "commands": [
+              "gdh migrate --apply"
+            ],
+            "validationCommands": [
+              "gdh adapters status"
+            ]
+          },
+          {
+            "id": "confirm_hook_version",
+            "kind": "agent_reasoning",
+            "summary": "Confirm managed hook scripts report hook version 8.",
+            "detail": "For managed Claude/Codex targets, inspect generated authoring hook headers or adapter status and confirm the hook surface has been refreshed.",
+            "commands": [
+              "gdh adapters status"
+            ],
+            "validationCommands": [
+              "gdh adapters status"
+            ]
+          }
+        ]
+      }
     }
   ]
 }

package/node_modules/@gdh/adapters/dist/templates/authoring-hook.js.tpl

@@ -10,6 +10,9 @@ const path = require('path');
 const { spawnSync } = require('child_process');
 
 const AUTHORING_EXTENSIONS = new Set(['.gd', '.tscn', '.tres']);
+const REFRESH_TIMEOUT_MS = readBoundedPositiveIntEnv('GDH_AUTHORING_HOOK_REFRESH_TIMEOUT_MS', 5000, 30000);
+const CHECK_TIMEOUT_MS = readBoundedPositiveIntEnv('GDH_AUTHORING_HOOK_CHECK_TIMEOUT_MS', 15000, 60000);
+const MAX_BLOCK_OUTPUT_CHARS = 4000;
 let CURRENT_EVENT = '';
 
 main();
@@ -31,25 +34,47 @@ function handlePostEdit(input, targetRoot) {
   const changedArgs = authoring.flatMap((file) => ['--changed', file]);
   // Attempt to refresh the diagnostics broker for changed files.
   // Broker refresh failure is non-fatal: the authoring check falls back to direct LSP.
-  runGdh(targetRoot, ['authoring', 'diagnostics', 'refresh', '--target', targetRoot, ...changedArgs]);
+  runGdh(targetRoot, ['authoring', 'diagnostics', 'refresh', '--target', targetRoot, ...changedArgs], {
+    timeoutMs: REFRESH_TIMEOUT_MS,
+  });
   // Run compact authoring check. When broker was refreshed, this uses the fresh snapshot.
   // When broker is unavailable, this falls back to direct LSP post-edit collection.
-  const result = runGdh(targetRoot, ['authoring', 'check', '--target', targetRoot, '--mode', 'post-edit', '--format', 'compact', ...changedArgs]);
-  if (!result.ok || !/Completion allowed\./.test(result.output)) {
-    return block(`GDH post-edit authoring check needs attention.\n${result.output || result.error || 'No check output.'}`);
+  const result = runGdh(targetRoot, ['authoring', 'check', '--target', targetRoot, '--mode', 'post-edit', '--format', 'compact', ...changedArgs], {
+    timeoutMs: CHECK_TIMEOUT_MS,
+  });
+  if (result.timedOut) {
+    return context(`GDH post-edit authoring check timed out after ${CHECK_TIMEOUT_MS}ms; continuing without blocking. Run \`gdh authoring check --mode final\` before claiming code-validity.`);
+  }
+  if (/Completion blocked\./.test(result.output)) {
+    return block(formatBlockingReason(result.output));
+  }
+  if (/Completion allowed\./.test(result.output)) {
+    return allow();
+  }
+  if (/Completion unconfirmed\./.test(result.output)) {
+    return context('GDH post-edit authoring check was unconfirmed; continuing without blocking. Run `gdh authoring check --mode final` before claiming code-validity.');
+  }
+  if (!result.ok) {
+    return context(`GDH post-edit authoring check failed before producing diagnostics; continuing without blocking. ${compactOneLine(result.error || result.output || 'Run final authoring validation before claiming code-validity.')}`);
   }
   return allow();
 }
 
-function runGdh(targetRoot, args) {
+function runGdh(targetRoot, args, options = {}) {
   const result = spawnSync('npx', ['-y', `@skillcap/gdh@${PINNED_VERSION}`, ...args], {
     cwd: targetRoot,
     encoding: 'utf8',
     windowsHide: true,
-    timeout: 120000,
+    timeout: options.timeoutMs || CHECK_TIMEOUT_MS,
   });
   const output = `${result.stdout || ''}${result.stderr || ''}`.trim();
-  return { ok: result.status === 0, output, error: result.error ? String(result.error.message || result.error) : '' };
+  const error = result.error ? String(result.error.message || result.error) : '';
+  return {
+    ok: result.status === 0 && !result.error,
+    output,
+    error,
+    timedOut: Boolean(result.error && result.error.code === 'ETIMEDOUT'),
+  };
 }
 
 function collectChangedFiles(input, targetRoot) {
@@ -118,11 +143,43 @@ function resolveTargetRoot(input) {
 function unique(values) { return [...new Set(values.filter(Boolean))].sort(); }
 function readHookInput() { try { return JSON.parse(fs.readFileSync(0, 'utf8') || '{}'); } catch { return {}; } }
 
+function readBoundedPositiveIntEnv(name, fallback, max) {
+  const raw = process.env[name];
+  if (!raw) return fallback;
+  const parsed = Number(raw);
+  if (!Number.isFinite(parsed) || parsed <= 0) return fallback;
+  return Math.min(Math.floor(parsed), max);
+}
+
+function formatBlockingReason(output) {
+  return `GDH post-edit authoring check found blocking diagnostics.\n\n${truncate(output || 'No check output.', MAX_BLOCK_OUTPUT_CHARS)}`;
+}
+
+function truncate(value, maxChars) {
+  const text = String(value || '').trim();
+  if (text.length <= maxChars) return text;
+  return `${text.slice(0, maxChars - 80).trimEnd()}\n... GDH hook output truncated; run \`gdh authoring check --mode post-edit\` for full diagnostics.`;
+}
+
+function compactOneLine(value) {
+  return String(value || '').replace(/\s+/g, ' ').trim();
+}
+
 function allow() { process.exit(0); }
+function context(additionalContext) {
+  if (AGENT !== 'codex') return allow();
+  const payload = { hookSpecificOutput: { hookEventName: CURRENT_EVENT, additionalContext } };
+  process.stdout.write(`${JSON.stringify(payload)}\n`);
+  process.exit(0);
+}
 function block(reason) {
   const payload = { decision: 'block', reason };
-  if (AGENT === 'codex') payload.hookSpecificOutput = { hookEventName: CURRENT_EVENT, additionalContext: reason };
+  if (AGENT === 'codex') {
+    payload.hookSpecificOutput = {
+      hookEventName: CURRENT_EVENT,
+      additionalContext: 'GDH found blocking authoring diagnostics after this edit. Fix the diagnostics in feedback before continuing.',
+    };
+  }
   process.stdout.write(`${JSON.stringify(payload)}\n`);
   process.exit(0);
 }
-

package/node_modules/@gdh/adapters/package.json

@@ -11,13 +11,13 @@
     }
   },
   "dependencies": {
-    "@gdh/authoring": "0.25.3",
-    "@gdh/core": "0.25.3",
-    "@gdh/docs": "0.25.3",
-    "@gdh/observability": "0.25.3",
-    "@gdh/runtime": "0.25.3",
-    "@gdh/scan": "0.25.3",
-    "@gdh/verify": "0.25.3"
+    "@gdh/authoring": "0.25.4",
+    "@gdh/core": "0.25.4",
+    "@gdh/docs": "0.25.4",
+    "@gdh/observability": "0.25.4",
+    "@gdh/runtime": "0.25.4",
+    "@gdh/scan": "0.25.4",
+    "@gdh/verify": "0.25.4"
   },
-  "version": "0.25.3"
+  "version": "0.25.4"
 }

package/node_modules/@gdh/authoring/package.json

@@ -14,7 +14,7 @@
     "test": "vitest run"
   },
   "dependencies": {
-    "@gdh/core": "0.25.3"
+    "@gdh/core": "0.25.4"
   },
-  "version": "0.25.3"
+  "version": "0.25.4"
 }

package/node_modules/@gdh/cli/package.json

@@ -15,16 +15,16 @@
   },
   "dependencies": {
     "@clack/prompts": "^1.2.0",
-    "@gdh/adapters": "0.25.3",
-    "@gdh/authoring": "0.25.3",
-    "@gdh/core": "0.25.3",
-    "@gdh/docs": "0.25.3",
-    "@gdh/mcp": "0.25.3",
-    "@gdh/observability": "0.25.3",
-    "@gdh/runtime": "0.25.3",
-    "@gdh/scan": "0.25.3",
-    "@gdh/verify": "0.25.3",
+    "@gdh/adapters": "0.25.4",
+    "@gdh/authoring": "0.25.4",
+    "@gdh/core": "0.25.4",
+    "@gdh/docs": "0.25.4",
+    "@gdh/mcp": "0.25.4",
+    "@gdh/observability": "0.25.4",
+    "@gdh/runtime": "0.25.4",
+    "@gdh/scan": "0.25.4",
+    "@gdh/verify": "0.25.4",
     "picocolors": "^1.1.1"
   },
-  "version": "0.25.3"
+  "version": "0.25.4"
 }

package/node_modules/@gdh/core/dist/index.d.ts

@@ -46,7 +46,7 @@ export declare const GDH_AUTHORING_DOGFOOD_VERSION = 1;
 export declare const GDH_AUTHORING_SLICE_REPORT_VERSION = 1;
 export declare const GDH_MCP_MANIFEST_VERSION = 1;
 export declare const GDH_CURSOR_RULE_VERSION = 4;
-export declare const GDH_UPDATE_HOOK_VERSION = 7;
+export declare const GDH_UPDATE_HOOK_VERSION = 8;
 export declare const GDH_RUNTIME_RECIPE_RUN_VERSION = 1;
 export declare const GDH_RUNTIME_RUN_BUNDLE_VERSION = 1;
 export declare const GDH_RUNTIME_CORPUS_ARTIFACT_VERSION = 1;

package/node_modules/@gdh/core/dist/index.js

@@ -38,7 +38,7 @@ export const GDH_AUTHORING_DOGFOOD_VERSION = 1;
 export const GDH_AUTHORING_SLICE_REPORT_VERSION = 1;
 export const GDH_MCP_MANIFEST_VERSION = 1;
 export const GDH_CURSOR_RULE_VERSION = 4;
-export const GDH_UPDATE_HOOK_VERSION = 7;
+export const GDH_UPDATE_HOOK_VERSION = 8;
 export const GDH_RUNTIME_RECIPE_RUN_VERSION = 1;
 export const GDH_RUNTIME_RUN_BUNDLE_VERSION = 1;
 export const GDH_RUNTIME_CORPUS_ARTIFACT_VERSION = 1;

package/node_modules/@gdh/core/package.json

@@ -10,5 +10,5 @@
       "import": "./dist/index.js"
     }
   },
-  "version": "0.25.3"
+  "version": "0.25.4"
 }

package/node_modules/@gdh/docs/package.json

@@ -11,8 +11,8 @@
     }
   },
   "dependencies": {
-    "@gdh/core": "0.25.3",
+    "@gdh/core": "0.25.4",
     "yaml": "^2.8.3"
   },
-  "version": "0.25.3"
+  "version": "0.25.4"
 }

package/node_modules/@gdh/mcp/package.json

@@ -11,14 +11,14 @@
     }
   },
   "dependencies": {
-    "@gdh/adapters": "0.25.3",
-    "@gdh/authoring": "0.25.3",
-    "@gdh/core": "0.25.3",
-    "@gdh/docs": "0.25.3",
-    "@gdh/observability": "0.25.3",
-    "@gdh/scan": "0.25.3",
-    "@gdh/verify": "0.25.3",
+    "@gdh/adapters": "0.25.4",
+    "@gdh/authoring": "0.25.4",
+    "@gdh/core": "0.25.4",
+    "@gdh/docs": "0.25.4",
+    "@gdh/observability": "0.25.4",
+    "@gdh/scan": "0.25.4",
+    "@gdh/verify": "0.25.4",
     "@modelcontextprotocol/sdk": "1.29.0"
   },
-  "version": "0.25.3"
+  "version": "0.25.4"
 }

package/node_modules/@gdh/observability/package.json

@@ -11,7 +11,7 @@
     }
   },
   "dependencies": {
-    "@gdh/core": "0.25.3"
+    "@gdh/core": "0.25.4"
   },
-  "version": "0.25.3"
+  "version": "0.25.4"
 }

package/node_modules/@gdh/runtime/package.json

@@ -11,9 +11,9 @@
     }
   },
   "dependencies": {
-    "@gdh/core": "0.25.3",
+    "@gdh/core": "0.25.4",
     "ws": "^8.18.3",
     "yaml": "^2.8.3"
   },
-  "version": "0.25.3"
+  "version": "0.25.4"
 }

package/node_modules/@gdh/scan/package.json

@@ -11,8 +11,8 @@
     }
   },
   "dependencies": {
-    "@gdh/core": "0.25.3",
-    "@gdh/docs": "0.25.3"
+    "@gdh/core": "0.25.4",
+    "@gdh/docs": "0.25.4"
   },
-  "version": "0.25.3"
+  "version": "0.25.4"
 }

package/node_modules/@gdh/verify/package.json

@@ -11,13 +11,13 @@
     }
   },
   "dependencies": {
-    "@gdh/authoring": "0.25.3",
-    "@gdh/core": "0.25.3",
-    "@gdh/docs": "0.25.3",
-    "@gdh/observability": "0.25.3",
-    "@gdh/runtime": "0.25.3",
-    "@gdh/scan": "0.25.3",
+    "@gdh/authoring": "0.25.4",
+    "@gdh/core": "0.25.4",
+    "@gdh/docs": "0.25.4",
+    "@gdh/observability": "0.25.4",
+    "@gdh/runtime": "0.25.4",
+    "@gdh/scan": "0.25.4",
     "yaml": "^2.8.3"
   },
-  "version": "0.25.3"
+  "version": "0.25.4"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skillcap/gdh",
-  "version": "0.25.3",
+  "version": "0.25.4",
   "description": "Godot-specific authoring and validation harness for agentic development.",
   "license": "MIT",
   "type": "module",
@@ -31,17 +31,17 @@
     "releaseStage": "broader_internal_release"
   },
   "dependencies": {
-    "@gdh/cli": "0.25.3",
+    "@gdh/cli": "0.25.4",
     "@clack/prompts": "^1.2.0",
-    "@gdh/adapters": "0.25.3",
-    "@gdh/authoring": "0.25.3",
-    "@gdh/core": "0.25.3",
-    "@gdh/docs": "0.25.3",
-    "@gdh/mcp": "0.25.3",
-    "@gdh/observability": "0.25.3",
-    "@gdh/runtime": "0.25.3",
-    "@gdh/scan": "0.25.3",
-    "@gdh/verify": "0.25.3",
+    "@gdh/adapters": "0.25.4",
+    "@gdh/authoring": "0.25.4",
+    "@gdh/core": "0.25.4",
+    "@gdh/docs": "0.25.4",
+    "@gdh/mcp": "0.25.4",
+    "@gdh/observability": "0.25.4",
+    "@gdh/runtime": "0.25.4",
+    "@gdh/scan": "0.25.4",
+    "@gdh/verify": "0.25.4",
     "picocolors": "^1.1.1"
   },
   "bundledDependencies": [