test(codegen-embed): match require() wiring not bare specifier (fix false positives)

The two E2E tests `--vscode-extension-adapter overrides the require specifier`
and `--vscode-extension-no-lc skips the language client` were checking for
the *absence* of bare substrings `@hyperpolymath/affine-vscode` and
`vscode-languageclient/node`.

Codegen-embed now inlines the adapter source (packages/affine-vscode/mod.js),
which carries documentation comments showing example usage of the default
specifier and a `vscode-languageclient/node` section delimiter. Those bare
substrings legitimately appear inside the embedded source — but they are
not require() wiring.

The off-by-default test at line 2994 already uses the precise pattern
`require("@hyperpolymath/affine-vscode")` for exactly this reason. Bringing
the override + no-lc tests to the same precision restores their semantic
intent: assert which require() fires, not which strings appear anywhere.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: hyperpolymath

test/test_e2e.ml

@@ -3014,14 +3014,24 @@ let test_vscode_extension_adapter_override () =
       ~vscode_extension_adapter:"../local/adapter.cjs" activate_src in
   Alcotest.(check bool) "uses the overridden adapter specifier"
     true (contains cjs {|require("../local/adapter.cjs")|});
-  Alcotest.(check bool) "does not fall back to the default adapter"
-    false (contains cjs "@hyperpolymath/affine-vscode")
+  (* Match the require wiring, not the bare specifier — the embedded
+     adapter source (packages/affine-vscode/mod.js) carries documentation
+     comments that legitimately mention the default specifier as an
+     example usage; the override semantics is about which require()
+     fires, not which strings appear anywhere in the file. *)
+  Alcotest.(check bool) "does not fall back to the default adapter require"
+    false (contains cjs {|require("@hyperpolymath/affine-vscode")|})
 
 let test_vscode_extension_no_lc () =
   let cjs = cjs_of ~vscode_extension:true ~vscode_extension_no_lc:true
       activate_src in
+  (* Match the require wiring, not the bare specifier — the embedded
+     adapter source includes a `vscode-languageclient/node` section
+     delimiter comment that is harmless because no actual require fires
+     unless the language-client argument is wired in (which `no_lc` skips
+     by passing `null`). *)
   Alcotest.(check bool) "skips the language-client require"
-    false (contains cjs "vscode-languageclient/node");
+    false (contains cjs {|require("vscode-languageclient/node")|});
   Alcotest.(check bool) "passes null in its place"
     true (contains cjs "    null,\n");
   Alcotest.(check bool) "still requires vscode + adapter"