Skip to content

Record the report-ownership fix in the 1.5.0 changelog - #362

Merged
luisleo526 merged 2 commits into
mainfrom
px/native-attempt-docs150-20261009
Oct 9, 2026
Merged

luisleo526 merged 2 commits into
mainfrom
px/native-attempt-docs150-20261009

Conversation

@luisleo526

Copy link
Copy Markdown
Collaborator

What this documents

The report-ownership fix that landed in #361 is recorded under the planned 1.5.0 entry of the changelog:

  • a run that the engine refuses before it begins returns an empty report and can no longer report a previous run on the same handle as successful;
  • the early C handle / argument checks and the native-module argument preflight are stated as the exceptions they are: they leave the caller's output untouched;
  • a run that started and then failed keeps its partial report, unchanged;
  • explicit stream snapshots follow the same ownership rule.

Scope

CHANGELOG.md only (20 additions, 1 deletion). No product source, header, test, version or corpus change; no release date, tag or already-released claim: the entry stays under 1.5.0 — Unreleased. Earlier release notes are untouched.

Evidence

  • An independent source review of the exact head: no blocking finding.
  • Exact-head preflight and documentation verification on the shared build hosts, recorded for this request.
  • Parity by the documentation diff rule (no behaviour difference); no product sweep is needed for a one-file documentation change.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Ko17oHtRco8iZ9vJptBAj8

luisleo526 and others added 2 commits October 9, 2026 19:25
Rename the Unreleased heading to "1.5.0 — Unreleased" and add one bullet
for #361: a batch run call's report belongs to its own attempt, a call
that does not begin a run gets the empty report and a failure status
instead of OK or an earlier payload, a started-then-failed run and the
explicit stream snapshot keep their reports, and the engine-owned gate
covers the C, native-module and generated wrappers with no change to
emitted source.

The recorded-outputs entry is unchanged byte for byte. CHANGELOG.md only:
no source, version, date, tag or patch-line change.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
A call that reaches the engine answers its own attempt's report. The empty
report and PF_NATIVE_E_RUN_FAILED now cover only calls the engine refuses
before it begins a run. The C entry's PF_NATIVE_E_HANDLE and
PF_NATIVE_E_ARGUMENT checks and native-module argument preflight still
return before the engine runs and leave caller output untouched, as
native_c_api.h and native_module.hpp already state.

Changelog wording only: no source, header or test change.

Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
@luisleo526
luisleo526 marked this pull request as ready for review October 9, 2026 12:22
@luisleo526
luisleo526 merged commit 873b25d into main Oct 9, 2026
13 checks passed
@luisleo526
luisleo526 deleted the px/native-attempt-docs150-20261009 branch October 9, 2026 14:55
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