Skip to content

Move config document helpers into their own module#25110

Merged
jif-oai merged 1 commit into
mainfrom
jif/nit-move
May 29, 2026
Merged

Move config document helpers into their own module#25110
jif-oai merged 1 commit into
mainfrom
jif/nit-move

Conversation

@jif-oai
Copy link
Copy Markdown
Collaborator

@jif-oai jif-oai commented May 29, 2026

Why

core/src/config/edit.rs owns the config edit state machine, but it also carried the TOML document helper code inline as a nested module. Moving those helpers into their own file keeps the edit orchestration easier to scan without changing the config persistence behavior.

What changed

  • Moved the existing document_helpers module from core/src/config/edit.rs into core/src/config/edit/document_helpers.rs.
  • Added mod document_helpers; so the existing pub(super) helper API remains available to the rest of config::edit.

Testing

Not run; this is a refactor-only module extraction with no intended behavior change.

@jif-oai jif-oai requested a review from a team as a code owner May 29, 2026 16:30
@jif-oai jif-oai changed the title nit: move condig document helper Move config document helpers into their own module May 29, 2026
@jif-oai jif-oai merged commit f4e9d2c into main May 29, 2026
31 checks passed
@jif-oai jif-oai deleted the jif/nit-move branch May 29, 2026 16:49
@github-actions github-actions Bot locked and limited conversation to collaborators May 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant