Remove unnecessary intermediate variable in test

When CookieMap was constructed with invalid arguments and then delete() was
called with the CookieMap instance itself, it caused a null pointer dereference.

The issue was in jsCookieMapPrototypeFunction_deleteBody where
getIfPropertyExists() was called but the return value wasn't properly checked
before being used. getIfPropertyExists() returns an empty JSValue when the
property doesn't exist, and this empty value was being assigned directly.

The fix checks if the returned JSValue is valid before assigning it to
nameValue, matching the pattern already used for domain and path extraction.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

src/bun.js/bindings/webcore/JSCookieMap.cpp

@@ -455,8 +455,12 @@ static inline JSC::EncodedJSValue jsCookieMapPrototypeFunction_deleteBody(JSC::J
         auto* options = optionsArg.getObject();
 
         // Extract name
-        if (nameValue.isUndefined()) nameValue = options->getIfPropertyExists(lexicalGlobalObject, PropertyName(vm.propertyNames->name));
-        RETURN_IF_EXCEPTION(throwScope, {});
+        if (nameValue.isUndefined()) {
+            auto nameValueFromOptions = options->getIfPropertyExists(lexicalGlobalObject, PropertyName(vm.propertyNames->name));
+            RETURN_IF_EXCEPTION(throwScope, {});
+            if (nameValueFromOptions)
+                nameValue = nameValueFromOptions;
+        }
 
         // Extract optional domain
         auto domainValue = options->getIfPropertyExists(lexicalGlobalObject, names.domainPublicName());

test/js/bun/cookie/cookie-map.test.ts

@@ -330,4 +330,11 @@ describe("iterator", () => {
     a=b"
   `);
   });
+
+  test("delete with invalid constructor args should not crash", () => {
+    // Regression test for null pointer deref when calling delete with invalid args
+    const cookieMap = new Bun.CookieMap(Bun.CookieMap, Bun.CookieMap, Bun, Bun.CookieMap);
+    // Should throw TypeError instead of crashing with null pointer dereference
+    expect(() => cookieMap.delete(cookieMap)).toThrow(new TypeError("Cookie name is required"));
+  });
 });