Repository navigation
Fix observation admission, Orrery state, and provider recovery - #124
Merged
Merged
Conversation
Separate activity observations from fixture defaults, reject unusable critical feeds, repair provider cache admission, and keep paused epochs and small-body facts coherent. Add regressions, deterministic series output, smoke-server recovery, and documentation corrections.
Require the native CLI for fresh, missing, and stale activity round-trips. Preserve strict JSON, snapshot, and readiness validation without requiring historical app data to qualify for assimilation.
Protonmatter
marked this pull request as ready for review
September 29, 2026 03:05
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 10 tasks
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.
Outcome
Prevent missing activity observations from changing the analysis or reducing uncertainty, reject unusable critical provider payloads, and keep Solar System time and selected facts consistent across navigation and date changes. Repair malformed provider-cache entries instead of repeatedly returning them.
For example, a bundle containing only wind/magnetic context previously changed a 0.2 activity prior to 0.76 and reduced variance from 0.04 to approximately 0.008. It now retains the prior, variance and Synthetic mode. A paused System scene no longer advances with wall time after leaving and returning; Halley's distance updates with its displayed epoch without reselection.
The patch also derives daily cycle-stage labels from established thresholds, makes series text output explicitly LF, removes unreferenced empty GPU prototypes and redundant helpers, repairs registry documentation links, and prevents idle browser preconnections from blocking the smoke-test server.
CI's assimilation round-trip now builds the native CLI and tests explicit fresh, missing and stale synthetic reports from the real Python producer. It requires the expected analysis or withheld mode, prior/variance behavior, strict JSON, full snapshot validation and research-only readiness after 48 transport steps; it cannot skip when the native CLI is required.
Specification and traceability
docs/rfcs/0002-correctness-delivery-and-experience.md; these corrections enforce the accepted evidence, recovery and experience boundaries.docs/SPEC.md,docs/TRANSACTIONAL_FEED.md,docs/EPHEMERIS_V3.md; compatibility details indocs/review_activity_fix.md.Correctness and risk
UI and UX
Validation evidence
Commands and results on the unchanged local patch:
Local environment: Windows ARM64, Python 3.14.3, Node 24.18.0, Cargo 1.96.0, Chrome 153.0.8010.48; local Coverage.py 7.13.5. These differ from the CI pins. The core Node/Python/Rust suites were rerun before publication, and file hashes were checked against the staged-browser/coverage-tested tree. Independent review found and verified additional contributor-evidence and cache-overflow edge cases; its final scoped review found no remaining actionable issues.
Hosted validation on
bba14d4f84b796aea164afc7b6977f753a62e1cf: all 20 checks passed, including the release gate. CI, Coverage, and Docs all completed successfully. Hosted Rust line coverage was 95.67%; hosted Python aggregate coverage was 91.47%. GitHub's tested PR merge tree was verified equal to the pushed head's tree.The existing Windows ARM64 moon-regeneration byte difference remains documented. The successful Linux ARM64 diagnostic does not qualify the separately pinned Linux x86_64 image. Authoritative moon data, expected hashes, tolerances and pending qualification status are unchanged. Local
cargo-llvm-covis unavailable, but the hosted Rust coverage gate passed. No deployment evidence is claimed for this PR.Documentation