test(app): match renamed command payload key

packages/app/e2e/regression/home-session-actions.spec.ts

@@ -72,7 +72,8 @@ test("renames, exports, and deletes a home session from its context menu", async
   expect(await container.evaluate((element) => getComputedStyle(element).outlineStyle)).toBe("none")
   await title.fill("Renamed from Home")
   const renamed = page.waitForRequest(
-    (request) => request.method() === "POST" && new URL(request.url()).pathname.endsWith("/rename"),
+    (request) =>
+      request.method() === "PATCH" && new URL(request.url()).pathname.endsWith(`/api/session/${fixture.targetID}`),
   )
   await title.press("Enter")
   expect((await renamed).postDataJSON()).toEqual({ title: "Renamed from Home" })

packages/app/src/composer/submit.test.ts

@@ -140,7 +140,7 @@ describe("Composer submission", () => {
         await committed.promise
       },
       command: async (request) => {
-        expect(request).toMatchObject({ command: "review", text: "changes", delivery: "steer" })
+        expect(request).toMatchObject({ name: "review", text: "changes", delivery: "steer" })
         expect(request).not.toHaveProperty("model")
         expect(request).not.toHaveProperty("agent")
         calls.push("command")

packages/codemode/src/stdlib/string.ts

@@ -218,12 +218,12 @@ export const stringGlobal = <R>(runner: Runner<R>) => {
           `String.matchAll requires a regular expression with the global (g) flag: write /${pattern.source}/${pattern.flags}g, or use String.match for a single match.`,
         )
       }
-      const matches: Array<unknown> = []
-      for (const match of value.matchAll(pattern)) {
-        checkArrayLength(matches.length + 1)
-        matches.push(matchToValue(protos, match))
-      }
-      return new ProgramArray(protos.Array, matches)
+      // Count natively first: converting each match is the expensive part, and an oversized result must fail before that.
+      checkArrayLength(value.match(pattern)?.length ?? 0)
+      return new ProgramArray(
+        protos.Array,
+        Array.from(value.matchAll(pattern), (match) => matchToValue(protos, match)),
+      )
     }),
     simple("search", 1, (value, args) => value.search(toHostRegex(args[0], "search"))),
     simple("repeat", 1, (value, args) => {