ReviewOS

home-lang/home

Account Socket.IO server attachment skipped corpus

#303
Closed chris opened this 10 days ago ·
10 days ago

Bun compatibility slice

Account for js/third_party/socket.io/socket.io-server-attachment.test.ts at exhaustive corpus position 4056.

Bun currently disables the complete server-attachment suite with describe.skip. Home should resolve the SuperTest and extensionless support imports, expose SuperTest through Home's HTTP compatibility layer, preserve the suite-level todo, avoid unsupported or allowed-empty classification, and pin the fixture in the permanent third-party matrix.

Tracked from #66.

10 days ago

Implemented and pushed in 4bec480cd.

Home now resolves the attachment fixture's extensionless Socket.IO support import and default SuperTest import through the existing shared, callable SuperTest implementation. Bun's intentional suite-level skip is preserved and permanently asserted.

During verification, the permanent Express matrix caught and prevented a duplicate SuperTest registration from shadowing the shared callable implementation; the duplicate was removed before commit.

Verification:

  • focused position 4056: 1 file, 0 failed, 1 todo, 0 unsupported/allowed-empty;
  • Socket.IO positions 4050–4056: 7 files, 13 passed, 8 explicit todos, 0 failed/unsupported/allowed-empty;
  • permanent gRPC error-stack, SQL adapter-contract, Express/third-party corpus, and 36-case diagnostic suites all pass.

Sign in to comment on this issue.