From 74b8e09f9a94c348f3bc3fd6a38d77d0bb549fe0 Mon Sep 17 00:00:00 2001 From: "Bob.Hou" Date: Mon, 26 Oct 2026 07:10:12 -0400 Subject: [PATCH 1/2] test(perplexity-web): use a mapped model in the stream boundary delayed-hook case The delayed success hook case in the Perplexity stream error boundary fixture used the unmapped model name pplx-auto-delayed-success-proof so its persisted-failure lookup would not match rows from the earlier pplx-auto case. Since #17822 the executor answers an unmapped model with a local 404 before any stream starts, so the case never reached the boundary it is meant to cover and the parent test failed on release. Use pplx-sonar instead. It is mapped and is not used by any other case in the fixture, so the lookup stays isolated. Assertions and source are unchanged. --- .../0000-pplx-stream-error-boundary-mapped-model.md | 1 + .../fixtures/perplexity-web-stream-error-boundary.fixture.ts | 4 +++- 2 files changed, 4 insertions(+), 1 deletion(-) create mode 100644 changelog.d/maintenance/0000-pplx-stream-error-boundary-mapped-model.md diff --git a/changelog.d/maintenance/0000-pplx-stream-error-boundary-mapped-model.md b/changelog.d/maintenance/0000-pplx-stream-error-boundary-mapped-model.md new file mode 100644 index 00000000000..7d1063dd533 --- /dev/null +++ b/changelog.d/maintenance/0000-pplx-stream-error-boundary-mapped-model.md @@ -0,0 +1 @@ +- **test(perplexity-web):** the stream error boundary test's delayed success hook case uses the mapped `pplx-sonar` model instead of the unmapped `pplx-auto-delayed-success-proof`, which #17822 now rejects with a local 404 before the stream starts ([#0000](https://github.com/diegosouzapw/OmniRoute/pull/0000)) — thanks @HouMinXi diff --git a/tests/fixtures/perplexity-web-stream-error-boundary.fixture.ts b/tests/fixtures/perplexity-web-stream-error-boundary.fixture.ts index 4e1746b485b..8006ea10d9e 100644 --- a/tests/fixtures/perplexity-web-stream-error-boundary.fixture.ts +++ b/tests/fixtures/perplexity-web-stream-error-boundary.fixture.ts @@ -426,7 +426,9 @@ test("same-packet content and error preserve the prefix across repeated readines test("a delayed success hook cannot erase same-packet content before terminal failure", async () => { const partialAnswer = "prefix must survive delayed success bookkeeping"; - const model = "pplx-auto-delayed-success-proof"; + // A mapped model other than pplx-auto keeps the persisted-failure lookup below + // separate from the earlier pplx-auto test. Unmapped names get a local 404 (#17822). + const model = "pplx-sonar"; const requestStartedAt = new Date(Date.now() - 1_000); let finalizedFailure: StreamFailure | null = null; const result = await executeThroughChatCore( From e5641fee04946f8640be4075243bd4d708733486 Mon Sep 17 00:00:00 2001 From: "Bob.Hou" Date: Mon, 26 Oct 2026 07:14:53 -0400 Subject: [PATCH 2/2] docs(changelog): number the #18232 fragment --- ...odel.md => 18232-pplx-stream-error-boundary-mapped-model.md} | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) rename changelog.d/maintenance/{0000-pplx-stream-error-boundary-mapped-model.md => 18232-pplx-stream-error-boundary-mapped-model.md} (57%) diff --git a/changelog.d/maintenance/0000-pplx-stream-error-boundary-mapped-model.md b/changelog.d/maintenance/18232-pplx-stream-error-boundary-mapped-model.md similarity index 57% rename from changelog.d/maintenance/0000-pplx-stream-error-boundary-mapped-model.md rename to changelog.d/maintenance/18232-pplx-stream-error-boundary-mapped-model.md index 7d1063dd533..c44d3a33dac 100644 --- a/changelog.d/maintenance/0000-pplx-stream-error-boundary-mapped-model.md +++ b/changelog.d/maintenance/18232-pplx-stream-error-boundary-mapped-model.md @@ -1 +1 @@ -- **test(perplexity-web):** the stream error boundary test's delayed success hook case uses the mapped `pplx-sonar` model instead of the unmapped `pplx-auto-delayed-success-proof`, which #17822 now rejects with a local 404 before the stream starts ([#0000](https://github.com/diegosouzapw/OmniRoute/pull/0000)) — thanks @HouMinXi +- **test(perplexity-web):** the stream error boundary test's delayed success hook case uses the mapped `pplx-sonar` model instead of the unmapped `pplx-auto-delayed-success-proof`, which #17822 now rejects with a local 404 before the stream starts ([#18232](https://github.com/diegosouzapw/OmniRoute/pull/18232)) — thanks @HouMinXi