Skip to content

[playbook-phase-13] Testing strategy and behavioral confidence #200

Description

@githubrobbi

Playbook section: docs/dev/architecture/code_clean/world_class_rust_workspace_refactor_playbook.md § Phase 13.

Baseline anchor: Phase-0 final report docs/dev/baseline/2026-05-12/phase_0_final_report.md (local-only per .gitignore:135). Workspace commit at baseline: 710ca6044 (cast-cleanup, PR #186).

Goals (from playbook)

Backfill tests for the low-test-density crates. See F5 follow-up for the uffs-broker extraction.

Current state (from Phase-0 baseline § 7 entry-point map)

Estimated cost: 1-2 weeks

Recommended entry points: uffs-broker (0 tests, 23 unsafe — see F5 issue) · uffs-daemon::broker_client (0 tests) · uffs-format low coverage

Acceptance criteria

  • Read playbook § Phase 13 Check / Refactor / Validate subsections
  • Open a tracking branch (refactor/phase-13-...) and a draft PR
  • Land work in surgical commits — preserve behavior and contracts (workspace rules: no suppression hacks, no relaxing tests, signed commits)
  • Update tests where the refactor changes invariants
  • Re-run just baseline-snapshot after merge and compare deltas against docs/dev/baseline/2026-05-12/
  • Close this issue with a one-line summary of the LOC / API / risk-marker delta

Workspace policy reminders

  1. No suppression hacks (#[allow] blankets, disabled lints, commented-out tests, cfg(any()) hiding)
  2. Surgical correct fixes (root cause, not workaround)
  3. Preserve behavior & contracts (public API stable unless CI proves it wrong)
  4. Improve tests, do not dodge them

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    playbookWorld-class Rust workspace refactor playbook phase trackingrefactorRefactor work tracked by the world_class_rust_workspace_refactor_playbook

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions