Skip to content

Solver rule hint text, difficulty fixes, and web rule naming - #18

Merged
daniel-jones-dev merged 5 commits into
mainfrom
core/solver-rule-hints
Sep 4, 2026
Merged

daniel-jones-dev merged 5 commits into
mainfrom
core/solver-rule-hints

Conversation

@daniel-jones-dev

Copy link
Copy Markdown
Owner

Summary

  • Improve and fix hint text/logic across solver rules (mark_queen, mark_empty, pointers, naked_set, hidden_set)
  • Fix an inverted unknown-cell filter bug in naked_set and hidden_set that caused incorrect rule firing
  • Add regression tests for naked_set and hidden_set covering real generated puzzle states
  • Rename displayed rule names on the web Solver Rules page to match the Rust code_name identifiers, and split mark_queen/mark_empty into their own "Trivial" difficulty tier to match Difficulty::Trivial in core/src/solver.rs

Test plan

  • cargo test --workspace passes (15 tests)
  • cargo fmt --check and cargo clippy --all-targets -- -D warnings pass
  • tsc --noEmit and eslint pass on web changes
  • Manually verify the Rules page and Solve page render the "Trivial" section correctly in the browser

daniel-jones-dev and others added 5 commits July 14, 2026 09:57
Cover the naked-pair (column with two remaining candidates) and
hidden-pair (two regions confined to the rightmost two columns) cases
that motivated the recent filter fix, using real generated puzzle
states decoded from share links.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Rename displayed rule labels to mirror the Rust code_name identifiers,
and split mark_queen/mark_empty into their own Trivial difficulty tier
to match Difficulty::Trivial in core/src/solver.rs.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@daniel-jones-dev
daniel-jones-dev merged commit b6cf586 into main Sep 4, 2026
5 checks 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