Skip to content

docs: list the value-failure codes this tree raises - #395

Merged
luisleo526 merged 1 commit into
mainfrom
lvd/cr125-failure-codes-doc
Oct 15, 2026
Merged

luisleo526 merged 1 commit into
mainfrom
lvd/cr125-failure-codes-doc

Conversation

@luisleo526

Copy link
Copy Markdown
Collaborator

The run-failure-codes page said that twelve of the 24 codes have a raise
site in this tree, all in the Python wrappers, and that
strategy_value_invalid, engine_command_value_invalid and
report_result_invalid have none. All three do: the first is thrown by the
source layer's value admission, the second by the engine kernel, the native
engine and the Pine adapter for invalid execution values (the kernel guards
fee.commission), and the third by docker/run_json.py for a
non-finite canonical result.

The page now says fifteen of the 24 have a raise site, names where each of
the three is raised, and keeps the nine codes with no producer in this tree.

Co-Authored-By: Claude Sonnet 5.5 [email protected]

Docs only (docs/pages/run-failure-codes.md). Checks: preflight 51 stages + docs (0 Doxygen warnings) passed on this exact tree; two independent reads, no open findings.

🤖 Generated with Claude Code

The run-failure-codes page said that twelve of the 24 codes have a raise
site in this tree, all in the Python wrappers, and that
strategy_value_invalid, engine_command_value_invalid and
report_result_invalid have none. All three do: the first is thrown by the
source layer's value admission, the second by the engine kernel, the native
engine and the Pine adapter for invalid execution values (the kernel guards
fee.commission), and the third by docker/run_json.py for a
non-finite canonical result.

The page now says fifteen of the 24 have a raise site, names where each of
the three is raised, and keeps the nine codes with no producer in this tree.

Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
@luisleo526
luisleo526 merged commit c60bf27 into main Oct 15, 2026
13 checks passed
luisleo526 added a commit that referenced this pull request Oct 17, 2026
* docs: the 1.6.0 release (changelog, current pair, HPO note)

Prepare the final 1.6.0 release documentation dated 2026-10-17, paired with pineforge-codegen 1.6.0 and the matching pineforge-hpo release for selected-window contract revision 1.3.

CHANGELOG.md adds the 1.6.0 — 2026-10-17 entry above 1.6.0-rc.1 — 2026-10-14, carrying forward the release-candidate entry's substance and naming the six documentation and test-only changes merged since v1.6.0-rc.1 (#390, #393, #394, #395, #396, #397).

README.md updates the current-release block and the codegen command-block pointer to 1.6.0 and adds the v1.6.0 bullet to the Releases section. The current-release block's 1.5.0 sentence from 4c78f39 ("The release notes cover recorded outputs, the call-scoped report rule and the test inputs' move to a private repository") is replaced on purpose by the 1.6.0 sentence that lists what 1.6.0 covers; that 1.5.0 summary stays in the Releases list (the v1.5.0 bullet) and in CHANGELOG.md's 1.5.0 entry.

docs/pages/public-contract.md adds the 1.6.0 paragraph after the 1.5.0 paragraph and updates the same-version pairing line to 1.6.0.

* docs: 1.6.0 notes review fixes (HPO pairing line, follow-up figures, value-ownership wording)

The paired-HPO line no longer claims HPO support for selected-window
revision 1.3; HPO 1.6.0 does not implement selected windows
(docs/return-stats.md:9 in pineforge-hpo). The landed follow-ups #396/#397
are said to be checked against their own deliberate bugs, and the figures
remain the candidate's. README.md and docker/README.md replace
"complete value ownership with" by "value ownership through", the wording
of the 1.6.0-rc.1 changelog entry, since no completeness proof was run.
The changelog's no-code-change sentence names the docker/README.md rewording.

Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>

---------

Co-authored-by: HaoLiangWen <[email protected]>
Co-authored-by: CORE-TOP <[email protected]>
Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant