[WRONG BRANCH] fix(responses): bound orphan call reordering work - #319
[WRONG BRANCH] fix(responses): bound orphan call reordering work#319luvs01 wants to merge 1 commit into
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
✅ Deterministic PR hygiene checks passed. |
⏳ DRAFT
What to do
Its title has been prefixed with |
Motivation
includes/indexOf, allowing admitted Requests to cause superlinear CPU and deep recursion.Description
Mapof output indexes and emits matched outputs by keyed lookup instead ofbatchKeys.includes/batchKeys.indexOflookups. The new algorithm tracks emitted indexes with aSetto avoid repeated work and eliminates the recursive call path. (changed:src/adapters/openai-responses.ts)tests/responses-stateless-dangling-call-repair.test.ts)Testing
./node_modules/.bin/bun test tests/responses-stateless-dangling-call-repair.test.tswhich passed all 9 tests in that file.bun x tsc --noEmit(typecheck) which completed successfully.PATH="$PWD/node_modules/.bin:$PATH" bun run privacy:scanwhich reportedPrivacy scan passed.bun run testof the whole suite but stopped after the focused adapter coverage because unrelated integration/concurrency tests timed out or failed; the targeted adapter tests and typecheck that validate this change passed.Codex Task