Phase: List Source Display, Doctor UX & Verify Dedup Status: Closeout Completed Started: 2026-03-17 Completed: 2026-03-17
| Batch | Name | WP | Requirements | Status |
|---|---|---|---|---|
| 1 | All Implementation (List Source + Doctor UX + Verify Dedup) | WP-1 + WP-2 + WP-3 | LSR-001 |
completed |
| 2 | Documentation + Regression + Closeout | WP-4 | DOC-001 | completed |
- Batch 1 is independent.
- Batch 2 (documentation + regression) depends on Batch 1.
- Requirements:
LSR-001,LSR-002,LSR-003,DUX-001,DUX-002,DUX-003,DUX-004,DUX-005,DUX-006,VDD-001,VDD-002,VDD-003 - Completed in this pass:
- Updated
crates/eden-skills-core/src/verify.rsso missing target paths short-circuit non-path-existschecks, reducing duplicate verify/doctor findings while preserving the existing check behavior for present targets. - Updated
crates/eden-skills-cli/src/commands/config_ops.rsto replace the humanlisttablePathcolumn with aSourcecolumn rendered asowner/repo (subpath)or~/local-path (subpath)using cyan styling, while leavinglist --jsonunchanged. - Added
DoctorArgsincrates/eden-skills-cli/src/lib.rs, wired the newdoctor --no-warningflag through command dispatch, and updatedcrates/eden-skills-cli/src/commands/diagnose.rsto filter warning findings before both human/JSON rendering and strict exit evaluation. - Renamed the doctor summary-table severity column from
SevtoLevel, expanded its width forwarning, and colorizederror/warning/infovalues red / yellow / dim to match the existing card-level severity semantics. - Added
crates/eden-skills-cli/tests/list_source_tests.rs,crates/eden-skills-cli/tests/doctor_ux_tests.rs,crates/eden-skills-cli/tests/verify_dedup_tests.rs, andcrates/eden-skills-core/tests/verify_dedup_tests.rs, and refreshed superseded legacy assertions in the affected CLI test files so historical coverage now matches the Phase 2.98 contracts.
- Updated
- Validation:
cargo fmt --all -- --check✅cargo clippy --workspace -- -D warnings✅cargo test --workspace✅- Test inventory:
476
- Notes:
doctor --no-warningintentionally remains behavior-neutral for exit codes and JSON schema beyond filtering warning findings out of the existing payload.- The Batch 1 regression pass stayed within Phase 2.98 scope only; root
STATUS.yaml, rootEXECUTION_TRACKER.md, and end-user docs remain untouched until Batch 2 closeout.
- Requirements:
DOC-001 - Completed in this pass:
- Updated
README.mdto mentiondoctor --no-warning, clarify thatlistshows source origins, refresh the Phase status line through 2.98, and sync the headline test/spec counts. - Updated
docs/07-cli-reference.mdto document the humanlistSourcecolumn semantics plus the newdoctor --no-warningoption, while explicitly noting that doctor JSON output keeps the existing schema. - Updated
docs/06-troubleshooting.mdwithdoctor --no-warningusage, aLevel-column example, and an error-focused JSON command example for automation. - Completed Phase 2.98 closeout by marking
DOC-001complete inspec/phase2.98/SPEC_TRACEABILITY.md, updatingtrace/phase2.98/status.yamlandtrace/phase2.98/tracker.md, and synchronizing rootSTATUS.yamlandEXECUTION_TRACKER.md.
- Updated
- Validation:
cargo fmt --all -- --check✅cargo clippy --workspace -- -D warnings✅cargo test --workspace --all-targets✅cargo check --workspace --all-targets --target x86_64-pc-windows-msvc✅- JSON output contracts unchanged ✅
- Exit code semantics unchanged ✅
- Test inventory:
476
- Notes:
- Phase 2.98 now closes with both batches complete; no additional implementation or documentation work remains in this phase.