Skip to content

refactor: move profile state helpers (R2j) - #97

Merged
KeyffMS merged 1 commit into
mainfrom
refactor/r2j-profile-state-helpers
Sep 20, 2026
Merged

KeyffMS merged 1 commit into
mainfrom
refactor/r2j-profile-state-helpers

Conversation

@KeyffMS

@KeyffMS KeyffMS commented Sep 19, 2026

Copy link
Copy Markdown
Owner

R2j

Small move/visibility-only continuation of #83 R2.

Scope:

  • move only the read-only ApplicationState::selected_profile and ApplicationState::profile_candidates helpers from application.rs into the existing application/state.rs;
  • expose them only as pub(super) so the existing sibling connection.rs reducer can keep calling the same helpers;
  • keep exactly one public ApplicationState definition and the top-level ApplicationState::reduce in application.rs.

Safety/architecture:

  • no reducer logic or behavior changed;
  • write/restore exact operator-confirmation comparisons remain unchanged in application.rs and directly covered by the existing architecture gate;
  • no session/write authority, scheduler/budget/fairness/QueueFull, filesystem boundary, or conformance semantics changed;
  • no gate/test weakening.

Refs #83

@KeyffMS
KeyffMS marked this pull request as ready for review September 20, 2026 08:29
@KeyffMS
KeyffMS merged commit 3c2b121 into main Sep 20, 2026
1 check passed
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