Distillation

Date: Sep 16, 2026

• 🟡 (16:38) Inspection of packages/gateway/test/openai-responses-recall-aware-stream.test.ts around lines 3956–4273 showed stream regressions covering: no forwarded response.completed on failure; controllable upstream SSE release via close()/error(err); draining a pre-buffered stream after a delayed first read; upstream cancellation; abort handling; downstream cancellation that does not await a hostile upstream cancel; malformed JSON suppression; multiple function calls such as item-two; output-text events; and provider-failure cases including response.done with status failed and secret provider code text provider-code-secret.
• 🟡 (16:39) Search in packages/gateway/test/openai-responses-recall-aware-stream.test.ts found 4 contradiction-related tests: line 3946 rejects terminal recall arguments that contradict an established final value; line 4726 rejects finalized reasoning that contradicts summary deltas; line 5490 rejects contradictory status in %s snapshots; line 7871 rolls back persistence without contradicting an already delivered terminal.
• 🟡 (16:41) Inspection of packages/gateway/test/openai-responses-recall-aware-stream.test.ts:5372-5421 showed two recall safeguards: terminal-only recall status failed returns PUBLIC_RECALL_ERROR without executing onRecall (recalled remains 0), and a terminal response.completed snapshot whose recall output is marked status: "failed" is likewise rejected.
• 🟡 (16:41) Test rejects nonterminal response.done status without executing recall at packages/gateway/test/openai-responses-recall-aware-stream.test.ts:5423-5445 sends doneWithStatus("resp_nonterminal_done", "in_progress"), expects PUBLIC_RECALL_ERROR, and verifies onRecall was never called (recalled === 0).
• 🟡 (16:41) Parameterized test rejects hidden output in %s snapshots at packages/gateway/test/openai-responses-recall-aware-stream.test.ts:5447-5487 covers response.created and response.in_progress; an in-progress snapshot containing hidden recall function call fc_snapshot_recall with arguments {"query":"leaked"} must produce response.failed, must not expose fc_snapshot_recall, and must leave recalled === 0.
• 🟡 (16:41) Parameterized test rejects contradictory status in %s snapshots at packages/gateway/test/openai-responses-recall-aware-stream.test.ts:5489-5514 covers response.created and response.in_progress snapshots whose embedded response has status: "failed" and expects the client stream to contain response.failed.
• 🟡 (16:41) Test preserves failed companion-tool status in rebuilt terminals at packages/gateway/test/openai-responses-recall-aware-stream.test.ts:5516-5585 combines a recall with companion tool read; its terminal item fc_failed_companion / call_failed_companion must remain status:"failed" in rebuilt output, must not become status:"completed", and must not produce PUBLIC_RECALL_ERROR.
• 🟡 (16:41) Test rejects reference identities reused by later output items begins at packages/gateway/test/openai-responses-recall-aware-stream.test.ts:5587; it reuses call_shared_reference first as an item_reference and later as a recall call ID, validating output-item identity uniqueness.
• 🟡 (16:43) Search found exactly 2 matching provider/status tests in packages/gateway/test/openai-responses-recall-aware-stream.test.ts: line 300 sanitizes a principal provider failure received as %s with status %s, and line 5490 rejects contradictory status in %s snapshots.
• 🟡 (16:45) Test accepts a partial Codex terminal output after streamed items at packages/gateway/test/openai-responses-recall-aware-stream.test.ts:1322-1358 streams two messages (msg_omitted text omitted from terminal, and msg_answer text answer) while the terminal snapshot contains only msg_answer; expected behavior is preserving answer, emitting exactly one response.completed, and emitting no response.failed.
• 🟡 (16:45) Parameterized test hides recall when the principal response.done status is %s at packages/gateway/test/openai-responses-recall-aware-stream.test.ts:1360-1390 covers statuses failed and cancelled; expected behavior is exactly one response.failed, no leaked "name":"recall" or call_0, no recall execution, no follow-up, and no recall identifiers in completedResponse.
• 🟡 (16:45) Test suppresses a recall function_call and emits a marker (mixed tools) at packages/gateway/test/openai-responses-recall-aware-stream.test.ts:1392-1471 suppresses recall query what is lore, retains companion read call fc_1 / call_1 with {"file":"x"}, inserts buildAnchor("what is lore") before the read item, excludes Searching, rebuilds response.completed, orders terminal IDs as ["msg_resp_mixed_0", "fc_1"], preserves the read item in rawOutputItems, and excludes what is lore, fc_0, and call_0 from completedResponse.
• 🟡 (16:45) Test binds parallel recalls to their own call IDs and fails before an invalid follow-up begins at packages/gateway/test/openai-responses-recall-aware-stream.test.ts:1473; it tracks RecallContinuationFailureCategory, streams two recalls with identical query same at output indexes 0 and 1, records each as ${outputIndex}:${toolUseId}, and forbids runFollowUp for multiple recalls.
• 🟡 (16:47) Attempting to run a standalone TypeScript test under Node.js v24.16.0 with tsx@4.22.4 failed with MODULE_NOT_FOUND: /home/byk/Code/opencode-lore-provider-error/node_modules/.pnpm/node_modules/@loreai/core/dist/node/index.js could not be resolved; the reported package path was /home/byk/Code/opencode-lore-provider-error/node_modules/.pnpm/node_modules/@loreai/core.
• 🟡 (16:49) Header comment in packages/gateway/test/openai-responses-recall-aware-stream.test.ts:1-11 documents the regression: buffered accumulateResponsesSSEStream withheld all client bytes until a slow, reasoning-heavy upstream completed, triggering opencode’s 10-second ProviderHeaderTimeoutError; streamResponsesRecallAware instead forwards events live, intercepts a recall function call, emits a marker, runs a follow-up, and rebuilds terminal response.completed.
• 🟡 (16:49) packages/gateway/test/openai-responses-recall-aware-stream.test.ts imports log from @loreai/core, streamResponsesRecallAware from ../src/pipeline, continuation hooks/types from ../src/recall-continuation-failure, principal transport hooks/types from ../src/principal-transport-failure, and GatewayResponse from ../src/translate/types; afterEach clears both hooks and registers silentLogSink.
• 🟡 (16:49) Test helpers in packages/gateway/test/openai-responses-recall-aware-stream.test.ts include: sseEvent(event, data) to add a matching type and format SSE; streamFrom(events) to return a status-200 text/event-stream response; drain(resp) to read full SSE text; created(id, model); recallCall(outputIndex, args, itemId, callId) producing added/arguments-done/item-done events with final status: "completed"; sparseVisibleFunctionCall(...) rebuilding a read call from sparse fields and argument deltas; completed(id, usage?); incomplete(id); doneIncomplete(id); doneWithStatus(id, status); and textItem(outputIndex, text, itemId).
• 🟡 (16:49) Constants and failure matrix in packages/gateway/test/openai-responses-recall-aware-stream.test.ts define PUBLIC_RECALL_ERROR = "Lore could not continue the response after recall" and PUBLIC_GATEWAY_ERROR = "Gateway request failed"; nonMonotonicProviderFailures combines 5 aliases/events—response.failed, failed response.done, cancelled response.done, error, and response.error—with source sequences [10, 9], yielding 10 cases.
• 🟡 (16:50) tsconfig.base.json contains target: "ESNext", module: "ESNext", moduleResolution: "bundler", allowImportingTsExtensions: true, strict: true, esModuleInterop: true, skipLibCheck: true, declaration: true, and types: [].
• 🟡 (16:51) vitest.config.ts resolves @loreai/core to packages/core/src and @loreai/gateway to packages/gateway/src using top-level resolve.alias; comments state placing aliases under test.resolve.alias silently resolved stale dist output and masked real failures.
• 🟡 (16:51) vitest.config.ts test inclusion is limited to packages/core/test/**/*.test.ts, packages/core/eval/**/*.test.ts, packages/gateway/test/**/*.test.ts, packages/opencode/test/**/*.test.ts, and packages/pi/test/**/*.test.ts; setup uses ./packages/core/test/setup.ts, environment node, pool forks, testTimeout: 300_000, hookTimeout: 300_000, and environment variables NODE_ENV="test", SENTRY_ENABLED="0", LORE_DEBUG="0".
• 🟡 (16:51) Vitest coverage configuration uses provider v8 and reporters ["text", "json", "html", "lcov"]; exclusions include **/*.test.ts, **/test/setup.ts, **/test/helpers/**, **/script/**, **/dist/**, packages/website/**, packages/pi/**, **/eval/**, **/*.eval.ts, *.config.ts, and scripts/**.
• 🟡 (16:53) Enumerated temporary workspace contents: base-pipeline-numbered.txt, base-test-numbered.txt, llm-v5-current.diff, local-v5-current.diff, lore-ui-search-response, oxfmt-opencode.json, projection-parent-pipeline.ts, publisher-v5-current.diff, restore-projector-v5-labeled.patch, restore-projector-v5.patch, restore-runner-v5-labeled.patch, restore-runner-v5.patch, session-recovery-final-v4, session-recovery-final-v4-diag, session-recovery-final-v4.index, session-recovery-final-v4.patch, session-recovery-final-v5, session-recovery-final-v5.index, session-recovery-final-v5.patch, session-recovery-final-v6, session-recovery-final-v6.patch, session-recovery-final-v7, session-recovery-final-v7.patch, session-recovery-final-v8, session-recovery-final-v8.patch, session-recovery-recheck.f3rLnS, session-recovery-review.1VQHG0, session-recovery-review-c2c8afb0, session-recovery-review-v5-49183f9e, session-recovery-review-v5-49183f9e.index, session-recovery-v5-review.r03hHL, session-recovery-v7-check, session-recovery-v7-processes.txt, session-recovery-v8-processes.txt, and session-recovery-v8-smoke.mjs.
• 🟡 (16:55) Created standalone regression test file /tmp/opencode/provider-log-failure.test.ts.
• 🟡 (16:57) Vitest v4.1.8 run from /home/byk/Code/opencode-lore-provider-error against /tmp/opencode/provider-log-failure.test.ts found no test files and exited with code 1; root cause was that the absolute /tmp test did not match any configured include glob, while standard exclusions included **/node_modules/** and **/.git/**.
• 🟡 (16:58) Created dedicated temporary Vitest configuration /tmp/opencode/provider-log-vitest.config.ts to support running the standalone provider-log regression outside the repository’s normal include globs.