Skip to content

fix: check live config before store init (#10) - #11

Merged
StarsailsClover merged 4 commits into
mainfrom
codex/issue-10-doctor-uninitialized
Nov 6, 2026
Merged

StarsailsClover merged 4 commits into
mainfrom
codex/issue-10-doctor-uninitialized

Conversation

@StarsailsClover

Copy link
Copy Markdown
Owner

Fixes #10

Root cause: Store::status only parsed live config inside the initialized branch, leaving LiveInfo.parsed false before init even when the file was valid. This made doctor report fail / present and parseable.

Changes:

  • Parse live TOML for status reporting regardless of store initialization.
  • Keep baseline evaluation and drift classification gated on initialization.
  • Report invalid pre-init config with its actual TOML parse error.
  • Add valid and invalid uninitialized-config regressions.

Verification: cargo fmt --all -- --check; cargo test --workspace; cargo clippy --workspace --all-targets --offline -- -D warnings; real fresh-home doctor/status repro.

@StarsailsClover
StarsailsClover merged commit 1c26b4f into main Nov 6, 2026
2 checks passed
@StarsailsClover
StarsailsClover deleted the codex/issue-10-doctor-uninitialized branch November 6, 2026 18:40
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.

[BUG] doctor marks a valid config.toml as failed before init

1 participant