Catch the docs up to a week in which the read path moved twice - #32
Merged
Conversation
Four documents, all stale in the same direction: they describe a price path that stopped being true when ADR-0007 step 4 landed, the bars were imported, and the boot guard shipped. data-path.md carried a sentence I wrote two days ago that is now wrong: "until the futures bars are imported into the marketdata store, a local checkout renders positioning without prices". They were imported on 2026-08-09. Rather than delete the section, it now records what closed and what did not: the bars exist in BOTH stores and nothing reconciles them, so every cotdata price run leaves this store behind until the import is re-run. A new "Refusing early" section documents the guard, including the deliberate asymmetry (nothing at all refuses, a subset warns) and why get_bars was NOT changed, since the next reader will reasonably ask why the loudness lives in a separate call. Half of the dotted edge that section describes is now built and half is not. The startup check shipped; the UI badge did not, so the history-depth leak is still silent inside a rendered chart even though it is loud at boot. Said so rather than letting "the fix for the leaks above" imply both. data_architecture.md called Databento the primary price provider with a Yahoo fallback, which described a deployment that has not existed for some time, and still named pardo_quant_framework. Norgate via marketdata is the live futures source and yfinance the equity one; databento is dormant, unowned, and read by nothing. It gets a scope note pointing at data-path.md, which is verified against the source rather than written from the design, so a reader who finds the two disagreeing knows which wins. README gained the setup step a fresh clone now needs. An empty marketdata store no longer boots, so "populate both stores" stopped being advice and became a precondition, with the import command for a box that still has the bars under $COTDATA_STORE/prices/. Also corrected the workspace CLAUDE.md, which is not in this repo and not in any repo, but is what every session here reads first. It still called /damage a live crowdmon consumer four months after that page was deleted, still said cotdata.get_prices was the live API, and described the marketdata store as equity-only and deliberately never refreshed. That last one needed splitting rather than replacing: the equities half IS still deliberately frozen, because npf's pinned snapshot compares updated_at, and the futures half is new and read live. The crowdmon section's open question is now answered in one direction: its two launchd jobs publish a panel nothing reads. ruff clean, 138 tests pass. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Four documents, all stale in the same direction: they describe a price path that stopped
being true when ADR-0007 step 4 landed, the bars were imported, and the boot guard shipped.
docs/data-path.mdCarried a sentence written two days ago that is now wrong: "until the futures bars are
imported into the marketdata store, a local checkout renders positioning without prices."
They were imported on 2026-08-09.
Rather than delete the section, it now records what closed and what did not. The gap is
closed; the split is not. The bars exist in both stores and nothing reconciles them, so
every cotdata price run leaves this store behind until the import is re-run, which the boot
guard reports as
stale.New "Refusing early" section documents the guard: the deliberate asymmetry (nothing at
all refuses, a subset warns) and why
get_barswas not changed. The next reader willreasonably ask why the loudness lives in a separate call, and the answer belongs next to
the thing it explains.
Also: half of the dotted edge that section describes is now built and half is not. The
startup check shipped; the UI badge did not, so the history-depth leak is still silent
inside a rendered chart even though it is loud at boot. Said so, rather than letting "the
fix for the leaks above" imply both.
docs/data_architecture.mdCalled Databento the primary price provider with a Yahoo fallback, describing a deployment
that has not existed for some time, and still named
pardo_quant_framework.marketdata(Windows-only producer); Databento listed as dormant, unowned, read by nothingpardo_quant_framework(3 places)npf, with a note on why the naming mattersIt also gains a scope note pointing at
data-path.md, which is verified against the sourcerather than written from the design, so a reader finding the two in disagreement knows
which wins.
README.mdAn empty marketdata store no longer boots, so "populate both stores" stopped being advice
and became a precondition. Added the error a fresh clone will see, the reason bars cannot
be fetched on macOS or Linux, and the import command for a box that still has them under
$COTDATA_STORE/prices/.Not in this repo: the workspace
CLAUDE.mdCorrected in place (it is not in any repo, but it is what every session here reads first):
/damagewas still listed as a live crowdmon consumer, four months after that page wasdeleted in Remove the /damage page now that crowdmon is deprecated #22 and The last crowdmon references leave the repo with the page #25. The crowdmon section's open question is now answered in one
direction: its two launchd jobs publish a panel nothing reads.
cotdata.get_priceswas still called the live API.one needed splitting rather than replacing: the equities half IS still deliberately
frozen, because npf's pinned snapshot compares
updated_at, and the futures half is newand read live.
ruffclean, 138 tests pass.🤖 Generated with Claude Code