Peter Steinberger
|
f717a13039
|
refactor(agent): dedupe harness and command workflows
|
2026-02-16 14:59:30 +00:00 |
|
the sun gif man
|
68ea063958
|
🤖 fix: preserve openai reasoning replay ids (#17792)
What:
- disable tool-call id sanitization for OpenAI/OpenAI Codex transcript policy
- gate id sanitization in image sanitizer to full mode only
- keep orphan reasoning downgrade scoped to OpenAI model-switch replay path
- update transcript policy, session-history, sanitizer, and reasoning replay tests
- document OpenAI model-switch orphan-reasoning cleanup behavior in transcript hygiene reference
Why:
- OpenAI Responses replay depends on canonical call_id|fc_id pairings for reasoning followers
- strict id rewriting in OpenAI path breaks follower matching and triggers rs_* orphan 400s
- limiting scope avoids behavior expansion while fixing the identified regression
Tests:
- pnpm vitest run src/agents/transcript-policy.test.ts src/agents/pi-embedded-runner.sanitize-session-history.test.ts src/agents/openai-responses.reasoning-replay.test.ts
- pnpm vitest run --config vitest.e2e.config.ts src/agents/transcript-policy.e2e.test.ts src/agents/pi-embedded-runner.sanitize-session-history.e2e.test.ts src/agents/pi-embedded-helpers.sanitize-session-messages-images.removes-empty-assistant-text-blocks-but-preserves.e2e.test.ts src/agents/pi-embedded-helpers.sanitizeuserfacingtext.e2e.test.ts
- pnpm lint
- pnpm format:check
- pnpm check:docs
- pnpm test (fails in current macOS bash 3.2 env at test/git-hooks-pre-commit.integration.test.ts: mapfile not found)
|
2026-02-15 22:45:01 -08:00 |
|
Peter Steinberger
|
5db579f2e0
|
refactor(test): reuse sanitize session history fixtures
|
2026-02-14 21:39:58 +00:00 |
|
Peter Steinberger
|
aa7fbf0488
|
perf(test): trim duplicate sanitize-session-history e2e cases
|
2026-02-13 16:21:59 +00:00 |
|
Peter Steinberger
|
b272158fe4
|
perf(test): eliminate resetModules via injectable seams
|
2026-02-13 16:20:37 +00:00 |
|
Peter Steinberger
|
9131b22a28
|
test: migrate suites to e2e coverage layout
|
2026-02-13 14:28:22 +00:00 |
|