Skip to content

docs: correct the report stream default in the changelog and date 1.9.0 - #639

Merged
luisleo526 merged 3 commits into
mainfrom
cx/docs-190-notes
Oct 29, 2026
Merged

luisleo526 merged 3 commits into
mainfrom
cx/docs-190-notes

Conversation

@luisleo526

Copy link
Copy Markdown
Collaborator

Changelog corrections before the 1.9.0 release. Documentation only (CHANGELOG.md).

  • The live runner's report stream default is stated correctly: plain --report-jsonl and --report-jsonl=full keep emitting the cumulative document; --report-jsonl=delta is opt-in. The previous text called delta the default, and an introductory sentence referred to a later entry that does not exist; both are removed (the commit message quotes them).
  • The 1.9.0 section gets its dated heading.
  • One line notes that the window-plan library now links on macOS (the 1.8.0 known issue); CI covers Linux only.

🤖 Generated with Claude Code

luisleo526 and others added 2 commits October 30, 2026 02:52
…e macOS link

The 1.9.0 live-runner entry said "Delta JSONL is the default" and that the
default stream changed, and its intro pointed at a "later entry" that does not
exist. The runner does the opposite (runner/main.cpp: plain --report-jsonl and
--report-jsonl=full select the cumulative document; --report-jsonl=delta is the
opt-in; runner/README.md says the same), and the 1.8.0 entry already states it.
The entry now says what the runner does.

Also stamp the block "1.9.0 — 2026-10-30" and add one line that the window-plan
library links on macOS again (the 1.8.0 known issue; Linux CI only).

Replaced sentences, named for the documentation-revert check (all in CHANGELOG.md
on main f0b6213, the 1.9.0 squash of #637, and replaced here):
- "The development entries below are retained in order. The cumulative stream
  default in the later entry supersedes the earlier delta default; the carried
  accumulator in the earlier entry closes the later entry's remaining native
  rebuild term."
- "Delta JSONL is the default. `pineforge-live run --report-jsonl` now emits one
  `pineforge-native-report-delta/v1` document for each committed input, matching
  the payload stored in the ledger. This deliberately changes the stream shape
  for existing consumers that read the old cumulative document; request
  `--report-jsonl=full` to retain that cumulative output."

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

@graphify-labs graphify-labs Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Graphify reviewed this change.

No issues found by static checks — no coupling regressions or blocking issues in the code graph. That is not the same as safe to merge: see what was not checked.

Not checked: tests were not run; no formal proof of the changed code.


Graphify review — findings

Cuts the 1.9.0 release section in CHANGELOG.md and rewrites the native live report stream notes so the default --report-jsonl output is documented as unchanged cumulative pineforge-native-report/v1 documents. --report-jsonl=delta is described as the opt-in mode, emitting the ledger's pineforge-native-report-delta/v1 patch plus a top-level integer input_cursor, and other values are refused. It also records that the shared window-plan library links on macOS again, via an Apple-only linker option, while macOS stays unsupported and no CI job builds it, and drops the stale "development entries" explanation that described the old delta default.

No blocking issues surfaced.

Analysis details — impact, health, verification

Impact & health

Graphify review

Impact — 72 functions depend on the 72 functions this change touches.

Health — grade A; no new coupling hotspots.

Verification — 72 functions in the blast radius were not formally verified this run (proofs are advisory here).

Gate & verification

graphify gate

PASS — objectively clean (no health regressions, tests not run — proofs not run this pass (advisory)). Grounded, not self-assessed.

Advisory (not blocking):

  • verification_scope: 72 function(s) in the blast radius were not formally verified this run

…ot as a result

The 1.9.0 macOS bullet said the shared window-plan library "links on macOS
again", but no macOS build or test ran. Say only what the change is: on macOS,
CMakeLists.txt lets one symbol, strategy_reads_trade_history, stay undefined
when linking the shared window-plan library (the 1.8.0 known issue), and macOS
is not supported and no CI job builds or tests it.

The replaced bullet is new in this pull request (not on main), so no sentence
from main is removed and no documentation-revert naming is needed.

Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
@luisleo526
luisleo526 merged commit 99bf1d5 into main Oct 29, 2026
14 checks passed
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