From fed6582d3e22a9ba94bc4ead0a5331357aecd510 Mon Sep 17 00:00:00 2001
From: Devvyn <22340871+opticon454@users.noreply.github.com>
Date: Tue, 15 Sep 2026 14:30:53 +0800
Subject: [PATCH] fix(test): strip the custom-model Run-menu picker's injected
 script too

CI on PR #430 failed test/server-index-title.test.ts's byte-identity
check: renderIndexHtml now injects a second unconditional <script> before
</head> (window.__codemanCustomModelClis, added alongside the existing
__codemanCliAvailable one), and the test only knew to strip the older one
before comparing the rendered HTML against the raw template.

Strip both. Unlike __codemanCliAvailable (an object, historically injected
only where something resolved), the new one is a plain array injected
unconditionally, possibly empty, so it needs stripping on every machine,
not just one with CLIs installed.

Verified the two replace() calls compose correctly against the exact
strings server.ts actually produces (simulated in isolation; this box has
no tmux, so the real WebServer-backed test file cannot run here at all --
same environment gap noted throughout this PR's review).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RqZeHrRS6DYcGcGX2p9EwG
---
 test/server-index-title.test.ts | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/test/server-index-title.test.ts b/test/server-index-title.test.ts
index 506e19bb..e2c7b979 100644
--- a/test/server-index-title.test.ts
+++ b/test/server-index-title.test.ts
@@ -96,16 +96,20 @@ describe('WebServer index.html <title> templating (#82)', () => {
 
   it('only substitutes the <title> tag — the rest of the template is identical (modulo asset cache-busting)', async () => {
     // renderIndexHtml also appends ?v=<mtime> cache-bust params to same-origin
-    // .js/.css refs, and injects the CLI-availability flags before </head>; strip
-    // both so the title remains the only other change.
+    // .js/.css refs, and injects the CLI-availability flags plus the custom-model
+    // Run-menu picker's CLI list before </head>; strip all so the title remains
+    // the only other change.
     //
-    // The flag strip is what keeps this test environment-independent. It used to
-    // pass here by luck: the availability script was injected only where a CLI
-    // resolved, so the assertion held on a machine with none installed and would
-    // have failed on a developer's box that had them.
+    // The flag strips are what keep this test environment-independent. The
+    // CLI-availability one used to pass here by luck: that script was injected
+    // only where a CLI resolved, so the assertion held on a machine with none
+    // installed and would have failed on a developer's box that had them. The
+    // custom-model list is injected unconditionally (a plain array, possibly
+    // empty), so it needs stripping on every machine, not just where non-empty.
     const html = (await render('laptop'))
       .replace(/(\.(?:js|css))\?v=[^"]*/g, '$1')
-      .replace(/<script>window\.__codemanCliAvailable=\{.*?\};<\/script>\n/, '');
+      .replace(/<script>window\.__codemanCliAvailable=\{.*?\};<\/script>\n/, '')
+      .replace(/<script>window\.__codemanCustomModelClis=\[.*?\];<\/script>\n/, '');
     const beforeTitle = rawTemplate.split('<title>Codeman</title>')[0];
     const afterTitle = rawTemplate.split('<title>Codeman</title>')[1];
     expect(html.startsWith(beforeTitle)).toBe(true);