Repository navigation
Live record of paper runs: NautilusTrader JSON log, decision stream, read-only manifest - #138
Merged
seathatflowsinourveins merged 4 commits intoSep 23, 2026
Merged
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
9 of 22 tasks
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7db1414332
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
seathatflowsinourveins
force-pushed
the
claude/paper-live-manifest-20260923
branch
from
September 23, 2026 18:55
7db1414 to
fbda240
Compare
…read-only manifest - runner.py --live-dir DIR (optional; absent = unchanged): the controller's event list also appends each decision and intent to DIR/events.jsonl as it happens (LiveEventLog; a write failure is counted, never raised), and the node gets NautilusTrader's own JSON file logger in DIR/nautilus (native_adapter.logger_config: FileWriterConfig(file_format="json"), INFO and above; stdout stays ERROR). - live_manifest.py: loopback page, /api/state and Prometheus /metrics from the run directory, the durable ledger opened sqlite3 mode=ro, and the STOP file. No broker access, no ledger writes; account ids and fingerprints redacted, no balances. - observability/paper-trading-live: collector file_log receivers and a Loki pipeline that keeps bodies and redacts account identifiers, a 5 s Prometheus scrape, and a Grafana dashboard (uid paper-trading-live). NautilusTrader 2.0.0rc5 has no live dashboard of its own (tearsheets are backtest-only, ReportProvider returns DataFrames, Nautilus Cloud is commercial and not yet available); its JSON file logger is the upstream live record used. Tests: 8 new (event log lines and failure counting, manifest state/metrics/redaction, read-only ledger, logger_config, and a native node in a fresh process writing SubmitOrder events to the JSON log). Engine suites with an isolated HOME: 686 OK (1 skipped). validate.py passed. Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
…ass LIVE_DIR through the trial wrapper (review) - NautilusTrader rc5 logs every AccountState at INFO with balances and margins (confirmed in a native JSON log); live_manifest.redact and the collector pipeline now replace balances=[...] and margins=[...], with a regression test. - Ledger intent status counts come from a GROUP BY over the whole ledger, not the 200 rows shown; the dashboard panel says so. - The request handler answers loopback Host names only and returns a 500 JSON error instead of an empty reply on a source error; limits parsing moved inside the guard. - scheduled_trial.sh: optional LIVE_DIR is passed to runner.py as --live-dir. - README: collector log times are read time (no timestamp parser). Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
… surface stream and ledger health (Codex review) - runner --live-dir writes run.json (trial, its ledger, the kill switch it checks); live_manifest uses that binding for the ledger and STOP status, else the ledger with the latest write including its WAL. - The default run directory is the one whose event stream or engine log changed last, not the directory mtime. - Gauges: adaptive_paper_ledger_readable, adaptive_paper_bound_to_run, adaptive_paper_events_age_seconds; the page shows ledger binding and stream age. - The run outcome records live_event_write_errors when --live-dir is used. - Collector: records carry their source time (events: at epoch seconds; Nautilus: RFC 3339 ns) and a run attribute from the run directory; the file path is dropped. Checked with otelcol-contrib 0.161.0 on sample files. Tests: 10 in the live-manifest module. Engine suites (isolated HOME): 688 OK (1 skipped). Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
…541678 Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
seathatflowsinourveins
force-pushed
the
claude/paper-live-manifest-20260923
branch
from
September 23, 2026 19:27
9b75a17 to
2d43fc6
Compare
seathatflowsinourveins
deleted the
claude/paper-live-manifest-20260923
branch
September 23, 2026 22:45
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.
Why
A live view of a paper run's trading and decision-making. NautilusTrader 2.0.0rc5 has no live dashboard of its own: tearsheets take a
BacktestEngine/BacktestResult,ReportProviderreturns DataFrames, and Nautilus Cloud's dashboards are commercial and "coming soon". Its JSON file logger (FileWriterConfig(file_format="json")) is the upstream live record.What
runner.py --live-dir DIR(optional; absent = unchanged): decisions and intents appended toDIR/events.jsonlas they happen; NautilusTrader JSON file log inDIR/nautilus/.live_manifest.py: loopback page,/api/state, Prometheus/metrics. Reads the run dir, the ledger (sqlite3mode=ro) and the STOP file only; redacts account identifiers; shows no balances.observability/paper-trading-live/: OTel Collectorfile_log→ Loki pipeline (bodies kept, ids redacted), Prometheus scrape job, Grafana dashboardpaper-trading-live.Checks
SubmitOrderevents to the JSON log.HOME: 686 OK (1 skipped).validate.pypassed. Guarded gitleaks: no leaks.otelcol-contrib validatepassed on the merged collector config;promtool check configpassed.🤖 Generated with Claude Code