fix(WeakRefObjectMap): make delete consistent with has for dead entries

[Copilot]

delete delegated directly to this.map.delete(key), bypassing liveness checks, so it returned true for GC'd entries while has returned false for the same key. All existing tests masked this because they called has/get first, which pruned dead entries before delete ran.

Changes

• WeakRefObjectMap.ts: Use this.has(key) (the public method, which runs isLiveOrDelete) before delegating to the internal map:

  // Before
  delete(key: string): boolean {
    return this.map.delete(key);
  }
  
  // After — consistent with has()
  delete(key: string): boolean {
    return this.has(key) && this.map.delete(key);
  }

  Dead entries now return false from both has and delete (and are pruned by delete itself). Behavior is unchanged for live entries and non-existent keys.

• WeakRefObjectMap.test.ts: Added a test that calls delete directly on a dead entry without any prior has/get call, covering the previously-undetected inconsistency.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.