feat: realtime scenario tests#154
Conversation
rogeriochaves
force-pushed
the
main
branch
2 times, most recently
from
77a92af to
9fdb87c
Compare
|
[grinder] BLOCKED — manual rebase required `git merge-tree` found 157 conflict markers between this branch and current `main`. The branch has 10+ commits adding `javascript/examples/vitest/tests/realtime/` infrastructure that predates significant structural changes to the examples directory on main. Autonomous rebase would require resolving 157 conflicts across JavaScript, TypeScript, and configuration files — this needs Drew's judgment to decide whether to:
Keeping `grinding` label. Will unblock when Drew makes the call. |
- Extract audio file writing logic from test into reusable utility - Follow JavaScript utils guidelines (Utils suffix, .utils.ts naming) - Use named parameters as destructurable object per TypeScript guidelines - Create single-responsibility AudioOutputUtils namespace object
drewdrewthis
force-pushed
the
realtime-scenario-tests
branch
from
d8edf1f to
a638c29
Compare
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
…inder] - Exclude realtime-client Vite sub-project from root ESLint config - Add eslint-disable-any to client/main.ts for untyped SDK event handlers - Fix import order and @ts-ignore→@ts-expect-error in realtime-agent-adapter - Fix import order in realtime-user-simulator.agent.ts - Revert react-refresh disable comment from button.tsx (now excluded) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…der] - Fix import order in vegetarian-recipe-realtime.test.ts - Re-add react-refresh disable to button.tsx (inner ESLint needs it; outer ignores file) - Rename event→_event at response.done listener (unused param warning) - Remove unused react-hooks/immutability disable in orb.tsx Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
| } | ||
|
|
||
| let targetIn = 0 | ||
| let targetOut = 0.3 |
| t = Math.imul(t, 0x21f0aaad) | ||
| t = t ^ (t >>> 15) | ||
| t = Math.imul(t, 0x735a2d97) | ||
| return ((t = t ^ (t >>> 15)) >>> 0) / 4294967296 |
…der] voice-contract-surface.test.ts uses top-level await at line 79. TypeScript requires target >= ES2017 when module=ESNext to allow top-level await. The missing target field defaulted to ES3, causing TS1378 errors. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
[grinder] BLOCKED — needs manual rebase Multiple CI layers (lint sub-projects, TS top-level-await tsconfig, then test-examples) kept failing across 4+ fix iterations. Root cause: deep integration of realtime sub-packages with mismatched ESLint/TS configs requires hands-on dev attention. Removing |
|
Automated low-risk assessment This PR was evaluated against the repository's Low-Risk Pull Requests procedure and does not qualify as low risk.
This PR requires a manual review before merging. |
|
Closing — superseded and the approach is no longer supported. The OpenAI Realtime adapter is now GA on main ( |
1 participant
No description provided.