Skip to content

[codex] refactor(config): extract harness posture module#3506

Draft
Hmbown wants to merge 1 commit into
mainfrom
codex/config-extract-harness-posture
Draft

[codex] refactor(config): extract harness posture module#3506
Hmbown wants to merge 1 commit into
mainfrom
codex/config-extract-harness-posture

Conversation

@Hmbown

@Hmbown Hmbown commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • Moves the harness posture/profile data model out of crates/config/src/lib.rs into a focused crates/config/src/harness.rs module.
  • Moves the built-in harness profile seeds and profile matching helpers with the type they support, keeping the boundary cohesive.
  • Re-exports the harness types and built_in_harness_profiles from the crate root so existing callers keep the same public API.

Refs #3311

Testing

  • cargo fmt --all -- --check
  • cargo clippy -p codewhale-config --all-targets --locked
  • cargo test -p codewhale-config --locked harness
  • cargo test -p codewhale-config --locked config
  • cargo test -p codewhale-config --locked
  • git diff --check
  • cargo clippy --workspace --all-targets --all-features
  • cargo test --workspace --all-features

Checklist

  • Updated docs or comments as needed (moved existing rustdoc with the types)
  • Added or updated tests where relevant (existing harness/config coverage preserved)
  • Verified TUI behavior manually if UI changes (N/A, config crate extraction only)
  • Harvested/co-authored credit uses a GitHub numeric noreply address (N/A, no harvested code)

Move the harness posture/profile data model, built-in seeds, and profile matching helpers out of lib.rs into a focused harness module.\n\nRe-export the public harness types and built_in_harness_profiles from the crate root so existing callers and tests keep the same API.\n\nThis is a behavior-preserving config split slice for #3311.
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