Skip to content

test(issue-261): complete Sprint 3 deprecated-path cleanup lane#264

Merged
graysurf merged 1 commit intomainfrom
issue/s3-t1-identify-deprecated-removed-path-tests-and-fixtu
Feb 26, 2026
Merged

test(issue-261): complete Sprint 3 deprecated-path cleanup lane#264
graysurf merged 1 commit intomainfrom
issue/s3-t1-identify-deprecated-removed-path-tests-and-fixtu

Conversation

@graysurf
Copy link
Owner

Summary

  • Complete issue Plan: Workspace obsolete test cleanup and de-risking #261 Sprint 3 lane cleanup for deprecated/removed-path test coverage.
  • Remove obsolete parity and fixture artifacts that no longer guard reachable behavior.
  • Rebaseline plan-issue-cli help parity checks to assert the current command contract directly.

Scope

  • Removed obsolete deprecated-path parity cases from:
    • crates/codex-cli/tests/parity_oracle.rs
    • crates/gemini-cli/tests/parity_oracle.rs
  • Reworked crates/plan-issue-cli/tests/parity_guardrails.rs help-surface test to use explicit command-surface assertions.
  • Removed stale fixtures:
    • crates/plan-issue-cli/tests/fixtures/shell_parity/help.txt
    • crates/api-test/tests/fixtures/grpc/smoke.suite.json
  • Updated shell-parity fixture docs/regenerator:
    • crates/plan-issue-cli/tests/fixtures/shell_parity/README.md
    • crates/plan-issue-cli/tests/fixtures/shell_parity/regenerate.sh
  • Produced Sprint 3 classification artifact:
    • /Users/terry/.agents/out/workspace-test-cleanup/deprecated-path-candidates.tsv

Testing

  • agent-docs resolve --context startup --strict --format checklist (pass)
  • agent-docs resolve --context project-dev --strict --format checklist (pass)
  • test -s "$AGENT_HOME/out/workspace-test-cleanup/deprecated-path-candidates.tsv" plus header/field/remove-path checks (pass)
  • cargo test -p nils-codex-cli -p nils-gemini-cli -p nils-plan-issue-cli -p nils-api-rest -p nils-api-gql (pass)
  • ./.agents/skills/nils-cli-verify-required-checks/scripts/nils-cli-verify-required-checks.sh (pass)
  • cargo llvm-cov nextest --profile ci --workspace --lcov --output-path target/coverage/lcov.info --fail-under-lines 85 (not run by request for immediate handoff)

Issue

@github-actions
Copy link

Coverage

Total line coverage: 86.45% (58994/68243 lines hit).

@graysurf graysurf marked this pull request as ready for review February 26, 2026 02:04
@graysurf graysurf merged commit 7060c2a into main Feb 26, 2026
6 checks passed
@graysurf graysurf deleted the issue/s3-t1-identify-deprecated-removed-path-tests-and-fixtu branch February 26, 2026 02:04
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