test: guard core helper assertions

src/agents/pi-hooks/compaction-safeguard.test.ts

@@ -113,7 +113,6 @@ const createCompactionHandler = () => {
     }),
   } as unknown as ExtensionAPI;
   compactionSafeguardExtension(mockApi);
-  expect(compactionHandler).toBeDefined();
   if (!compactionHandler) {
     throw new Error("Expected compaction safeguard to register a handler.");
   }

src/agents/tool-search.test.ts

@@ -866,8 +866,10 @@ describe("Tool Search", () => {
         code: `return await openclaw.tools.call("fake_abort_on_timeout", { value: "wait" });`,
       }),
     ).rejects.toThrow("tool_search_code timed out");
-    expect(observedSignal).toBeDefined();
-    expect(observedSignal?.aborted).toBe(true);
+    if (!observedSignal) {
+      throw new Error("Expected observed abort signal");
+    }
+    expect(observedSignal.aborted).toBe(true);
     expect(abortCount).toBe(1);
   });
 });

src/gateway/server-methods/skills.update.normalizes-api-key.test.ts

@@ -24,7 +24,9 @@ vi.mock("../../config/config.js", () => {
 const { skillsHandlers } = await import("./skills.js");
 
 function expectWrittenSkillEntry(skillKey: string, entry: unknown) {
-  expect(writtenConfig).toBeDefined();
+  if (!writtenConfig) {
+    throw new Error("Expected written config");
+  }
   const config = writtenConfig as {
     skills?: {
       entries?: Record<string, unknown>;