[nissa-seru]

This adds two tests that pin how the [inference-failed] note writes a failure's reason and type: on the note's one line, with an opening of a notice tag the framework knows marked. The note passes the reason through failureShown, and the type with its HTTP status through suppliedValue, in the note for a model call that failed and in the note for a turn that ended. Those wrappers came with #852, released in 0.29.0. This change touches only test/inference-failure-bounds.test.ts, +31 −1. With each of the three wrappers removed, or narrowed to oneLine alone, one of the two new tests fails, and every other test in the 13 test files whose text holds inference-failed passes.

Problem

At 0f11975e, src/framework.ts passes the note's supplied values through three wrappers:

• :31558: ${failureShown(markerReason)}, the reason;
• :31556: (${suppliedValue(cause)}) after "the model call … failed", the type with its HTTP status;
• :31549: (${suppliedValue(cause)}) after "it failed", in the note for a turn that ended.

The runs below show that no test in the 13 test files whose text holds inference-failed checks them. At this head, with any of the three removed, every test in those files passes except one of the two new ones, and the rest of those files is as at 0f11975e.

Changes

• test/inference-failure-bounds.test.ts gains two tests, and a sentence in its header comment saying what they check. Each gives the note a type that holds a line break and an opening of a notice tag the framework knows, with HTTP status 529. It checks that the note has no line feed or U+2028 and shows the type with its opening escaped.
  • "the marker gives a failure's reason and type on its one line, with a notice's opening in them marked" checks the note for a model call that failed. Its reason holds line breaks and such an opening too, and the test checks the reason the same way.
  • "the note for a turn that ended gives the failure's type on its one line, with a notice's opening in it marked" checks the note for a turn that ended, with a reason of one word.
• No change outside that file: nothing in src/, no changelog fragment, and no text an agent or operator sees.

On October 29 at 06:08 UTC, the open pull requests that also touch test/inference-failure-bounds.test.ts are #190 and #844. By git merge-tree, each conflicts with main and with this head in the same files, and this test file isn't one of them.

Tests

On the files of this head, 41ea6529, from a clean checkout of a commit with the same tree that isn't in this pull request's history, on Linux with Node 24.22.1 and npm 11.19.1, after npm install with package-lock.json and node_modules removed, and npm run build. npm ls --depth=0 in that install gives @animalabs/chronicle 0.5.0, @animalabs/context-manager 0.24.1, @animalabs/mcpl-core 0.4.0, @animalabs/membrane 0.8.0, @babel/parser 7.29.11, @modelcontextprotocol/client 2.4.0, @modelcontextprotocol/server 2.4.0, @types/node 22.20.5, @types/ws 8.18.2, chokidar 4.0.3, discord.js 14.27.0, tsx 4.23.15, typescript 7.0.2 and ws 8.22.0.

• The 13 test files whose text holds inference-failed, activation-request-sanitize, cancel-surfaces-as-error, default-error-policy, exhausted-turn-rearm, inference-bounds, inference-failure-bounds, inference-failure-observability, kv-unified-open-flight-retry, prose-drafts, provider-acceleration-cooldown, provider-waits, shutdown-not-user-attributed and user-interrupt-not-failure: 580 of 580 pass.
• Six changes to src/framework.ts, each built and run against the same 13 files: each fails exactly one test, and the other 579 pass. Each removal changes one line. Each replacement by oneLine changes that line and adds oneLine to the import at :138.
  • The reason's failureShown removed, or replaced by oneLine: "the marker gives a failure's reason and type…" fails.
  • The failed call's suppliedValue removed, or replaced by oneLine: the same test fails.
  • The ended turn's suppliedValue removed, or replaced by oneLine: "the note for a turn that ended gives the failure's type…" fails.
• Time: at the head, the two new tests took 1.00 and 0.76 ms.

Not verified

• The whole suite at this head: these runs covered the 13 files above. CI's run on this pull request is its first whole-suite run at this head.
• Openings of the framework's other notice tags in these values: each test gives an opening of one tag.
• Platforms: every run was on Linux.

---

• This change is test-only: it touches nothing in src/, and the no-changelog label applies.

Ansel-2428 wrote the tests. Desmond-2579 found, by runs at an earlier commit, that every test in the test files whose text holds inference-failed passed with the reason's wrapper or the ended note's wrapper removed, and he read the change, both before it was rebased onto main and in its final form. Daniel-2612 wrote this description.