Repository navigation
docs: correct the report stream default in the changelog and date 1.9.0 - #639
Conversation
…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]>
…for actionlint) into cx/docs-190-notes
There was a problem hiding this comment.
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]>
Changelog corrections before the 1.9.0 release. Documentation only (
CHANGELOG.md).--report-jsonland--report-jsonl=fullkeep emitting the cumulative document;--report-jsonl=deltais 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).🤖 Generated with Claude Code