fix(gitagent-adapter): normalize the MERGED model in harden() too

The earlier normalizer (commit 9d9b249) only ran on the loader-side
model preference from agent.yaml. When a caller passes
options.model="claude-haiku-4-5-20251001" (bare) on the request body,
mergeEngineOptions overrides the prefixed value, and gitclaw's
parseModelString throws "Invalid model format". The throw is swallowed
inside gitclaw's query() setup — the iterator returns zero messages,
engine.turn.end fires in ~20ms with no yields, and the harness has
nothing to forward, so the client sees session_started → silence.

Surfaced on the live test page: posting a PDF to api.clawagent.sh with
runtime=bwrap + harness=gitagent + a bare model name hung for >1 min
with no error. EC2 journalctl showed engine.turn.end durationMs=18
and no LLM call ever happened. Same failure mode as issue #6 (silent
hang), this time at the model-normalization layer.

Fix: harden() now re-normalizes merged.model after merge, so bare
names get the "provider:" prefix regardless of where they came from.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: kapaleshreyas

packages/identity-gitagentprotocol/src/adapters/gitagent.ts

@@ -73,10 +73,18 @@ export async function gapToGitagentOptions(
   return {
     options: opts,
     harden: (merged) => {
-      if (requireHumanReview) {
-        return { ...merged, requireHumanReview: true };
+      // Re-normalize the model AFTER merge so caller-supplied bare names
+      // (e.g. options.model="claude-haiku-4-5-20251001") get the gitclaw
+      // "provider:modelId" prefix. Without this, gitclaw's parseModelString
+      // throws "Invalid model format" inside query() and the error is
+      // swallowed — query() returns 0 messages, the engine looks healthy
+      // but actually died at init. See issue #6.
+      const out: GitagentOptions = { ...merged };
+      if (typeof out.model === "string") {
+        out.model = normalizeGitclawModel(out.model);
       }
-      return merged;
+      if (requireHumanReview) out.requireHumanReview = true;
+      return out;
     },
   };
 }

packages/runtime-e2b/assets/harness-bundle.mjs

@@ -139594,10 +139594,13 @@ async function gapToGitagentOptions(manifest, workdir) {
   return {
     options: opts,
     harden: (merged) => {
-      if (requireHumanReview) {
-        return { ...merged, requireHumanReview: true };
+      const out = { ...merged };
+      if (typeof out.model === "string") {
+        out.model = normalizeGitclawModel(out.model);
       }
-      return merged;
+      if (requireHumanReview)
+        out.requireHumanReview = true;
+      return out;
     }
   };
 }

packages/runtime-local/assets/harness-bundle.mjs

@@ -180713,10 +180713,13 @@ async function gapToGitagentOptions(manifest, workdir) {
   return {
     options: opts,
     harden: (merged) => {
-      if (requireHumanReview) {
-        return { ...merged, requireHumanReview: true };
+      const out = { ...merged };
+      if (typeof out.model === "string") {
+        out.model = normalizeGitclawModel(out.model);
       }
-      return merged;
+      if (requireHumanReview)
+        out.requireHumanReview = true;
+      return out;
     }
   };
 }

packages/runtime-vzvm/assets/harness-bundle.mjs

@@ -139594,10 +139594,13 @@ async function gapToGitagentOptions(manifest, workdir) {
   return {
     options: opts,
     harden: (merged) => {
-      if (requireHumanReview) {
-        return { ...merged, requireHumanReview: true };
+      const out = { ...merged };
+      if (typeof out.model === "string") {
+        out.model = normalizeGitclawModel(out.model);
       }
-      return merged;
+      if (requireHumanReview)
+        out.requireHumanReview = true;
+      return out;
     }
   };
 }