test-infra(1/6): add test infrastructure and CI tooling

[aRustyDev]

Summary

Important

MOST of this PRs changes are contained in #203
Merge #203 first before reviewing these changes

Test infrastructure and CI tooling (builds on docs reorganization):

• justfile: Modular task runner with recipes for testing, quality, docs
• CI: cargo-deny, nextest, pre-commit hooks, codecov
• GitHub: Updated workflows, dependabot, PR template
• Tests: Comprehensive MCP server integration tests
• ADR-0012: MCP server library selection documentation

Files Changed (10 commits)

• justfile, */justfile - Task runner
• .pre-commit-config.yaml, deny.toml - Quality tools
• .github/workflows/* - CI updates
• crates/adrs/src/mcp.rs - MCP integration tests

Test plan

• just test - All tests pass
• just test doc - All doctests pass
• All pre-commit hooks pass

depends-on: #203

🤖 Generated with Claude Code

[aRustyDev]

just test all
cargo nextest run
    Finished `test` profile [unoptimized + debuginfo] target(s) in 0.14s
────────────
 Nextest run ID 5d4af15b-237d-4d2d-ad05-aad638708b2d with nextest profile: default
    Starting 520 tests across 7 binaries
        PASS [   0.490s] adrs::cli test_cheatsheet_output
        ... removed for brevity ...
        PASS [   0.013s] adrs-core::real_world_corpus test_npryce_adr_tools_canonical
────────────
     Summary [   1.475s] 520 tests run: 520 passed, 0 skipped
cargo test --doc --package adrs-core
    Finished `test` profile [unoptimized + debuginfo] target(s) in 0.08s
   Doc-tests adrs_core

running 1 test
test crates/adrs-core/src/types.rs - types::LinkKind::reverse (line 293) ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

all doctests ran in 0.39s; merged doctests compilation took 0.19s
All tests passed!

[aRustyDev]

just quality all
cargo deny check

... removed for brevity ...

advisories ok, bans ok, licenses ok, sources ok
Security checks passed!
cargo machete
Analyzing dependencies of crates in this directory...
cargo-machete didn't find any unused dependencies in this directory. Good job!
Done!
cargo fmt --check
cargo clippy --all-targets --all-features -- -D warnings
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.22s
All quality checks passed!

[aRustyDev]

pre-commit run --all-files --hook-stage pre-push
cargo fmt................................................................Passed
cargo clippy.............................................................Passed
cargo nextest (unit tests)...............................................Passed
cargo test --doc.........................................................Passed
cargo deny (security, licenses)..........................................Passed
cargo machete (unused deps)..............................................Passed

pre-commit run --all-files
cargo fmt................................................................Passed
cargo clippy.............................................................Passed

[joshrotenberg]

On first glance this PR seems to include all of the doc changes of the first one. Is that intentional?

[aRustyDev]

On first glance this PR seems to include all of the doc changes of the first one. Is that intentional?

Yeah thats more of an error with how I broke up the work when submitting. I put a 'WARNING' block in the description to try and capture that.

I'd definitely recommend Merging #203 first, then looking at this one.

Note

I need to make another update too this too, the CI failure is b/c of how I broke up the changes too.
I'm pretty sure its b/c I added the doctest CI steps, but didn't add doctest until later?

[aRustyDev]

@joshrotenberg
Just pushed a change to fix the CI check error about 'coverage'. If you'd be so kind as to approve the workflow again?

[joshrotenberg]

Hey, I appreciate the effort but these PRs are huge and have a lot of changes I think require discussion first. Your recent commits also seem to include the entire doc changes despite claiming to change something else. There is no way I can review these in this state. I'm all for using Claude (I use it every day) but this is just too much at one time without clear value. Please close these and open up issues to discuss the changes you think will help the project.