Skip to content

fix: warn on config divergence after boot#581

Merged
brickfrog merged 1 commit into
mainfrom
main.ei4y-audit-fixes-1781376939187-302538-0
Jun 13, 2026
Merged

fix: warn on config divergence after boot#581
brickfrog merged 1 commit into
mainfrom
main.ei4y-audit-fixes-1781376939187-302538-0

Conversation

@brickfrog

Copy link
Copy Markdown
Owner

Supersedes PR #580; re-lands the warn-only config divergence check and fixes audit findings F1 empty-project paths, F2 prenotify signature collisions, and F3 full Config coverage.\nVerification: moon test --target native && moon run src/bin/choir_lint --target native passed (2017 tests; lint clean).

Verification

Generated by Choir from commands executed in the leaf workspace.

  • moon test --target native
    • exit: 0
    • head: c6dfc6c
    • output tail:
Warning: Main package `choir/src/bin/choir` uses blackbox-only test inputs (`_test.mbt` files) in package directory "/mnt/data/Code/choir/.choir/worktrees/ei4y-audit-fixes-1781376939187-302538-0/src/bin/choir". Main packages will stop generating blackbox tests in a future release. Move public behavior into a non-main package and keep the main package as an entrypoint.
4
Total tests: 2017, passed: 2017, failed: 0.
  • moon run src/bin/choir_lint --target native

@brickfrog

Copy link
Copy Markdown
Owner Author

Choir: GitHub Actions rollup reports all checks successful on the latest head — re-check review/merge policy on GitHub (not an approval or merge signal by itself).

@brickfrog brickfrog merged commit 3e45850 into main Jun 13, 2026
1 check passed
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.

1 participant