refactor(core): source control in folders

.git-blame-ignore-revs

@@ -15,3 +15,9 @@ e5278fd577dadd1cd11800f0ca2b7abd908343cc # refactor(core): move the codex driver
 cb0d36fa7a89a7087a51eba5774fcc923ae78933 # refactor(core): move the acp driver tests into their own file
 8678212e2ca296ee089964e96848e26372567217 # refactor(web): move workspace files into sidebar, palette and app-chrome folders
 6f4e756ad452cf8492291a11c36a93363261c5b2 # refactor(web): move panel files into strip, browser, diff, files, terminal, activity and preview folders
+ff729cfa4ec1e3b71c956ed013aee5a9706c36cf # refactor(core): move forge into its own folder
+4a8a17919098a3bedd4132088a110d30b8a7c611 # refactor(core): move forge_cache into the forge folder as cache
+5c8eb42c8294b8e48a3263eae09184d7b83fb6af # refactor(core): move the forge tests into their own files
+0b6e3a36e79764b2d73854e25b4ded064b721910 # refactor(core): move scm into its own folder
+d6cb0514c515aa58c86a86a2b2c79dbf32f47325 # refactor(core): move scm_writing and its style into the scm folder as writing
+46cbc88a24a5cba1355a18f95b62ef1d1eeba64d # refactor(core): move the scm and scm writing tests into their own files

crates/orbyn-core/src/app.rs

@@ -122,7 +122,7 @@ pub struct Core {
     pub recovery: Recovery,
     pub(crate) keybinding_writer: Mutex<()>,
     /// Pull request details asked of GitHub, reused while a pull request is unchanged.
-    pub(crate) pull_details: Mutex<crate::source_control::forge_cache::DetailCache>,
+    pub(crate) pull_details: Mutex<crate::source_control::forge::cache::DetailCache>,
     /// What each forge's CLI said, reused briefly: asking costs a process per forge.
     pub(crate) forge_statuses:
         Mutex<Option<(std::time::Instant, Vec<orbyn_contracts::ForgeStatus>)>>,

crates/orbyn-core/src/settings/preferences.rs

@@ -190,7 +190,7 @@ fn check(definition: &Definition, value: &Value) -> Result<()> {
         crate::project::project_file::validate_scripts(value)?;
     }
     if fits && definition.key == "source_control_writing_style" {
-        crate::source_control::scm_writing::validate_style(value)?;
+        crate::source_control::scm::writing::validate_style(value)?;
     }
     if fits && definition.key == "default_model_options" && !model_options(value) {
         return Err(CoreError::Invalid(

crates/orbyn-core/src/source_control/README.md

@@ -4,12 +4,14 @@ Source control on a thread's checkout, always through the person's own installed
 
 | Module | Owns |
 | --- | --- |
-| `scm` | Source control actions on a thread's checkout |
+| `scm` | Source control actions on a thread's checkout: git actions and branches (`mod.rs`), pull request reads and linking (`pulls`), comments, reviews and merges (`review`), stacks and checkouts (`stack`), pull request watching (`watch`), forge status and publishing (`forges`) |
+| `scm::writing` | Diff-derived text for a confirmed Git action: the writing request (`mod.rs`), the bounded checkout reads (`candidate`) and the writing style (`style`) |
 | `git` | Git through the installed CLI, worktrees in the sibling `<repo>-wt` root |
 | `jj` | Non-colocated Jujutsu checkouts, read through the installed `jj` |
-| `forge` | GitHub and GitLab through their own CLIs |
-| `forge_cache` | Short-lived memory of GitHub pull request details |
-| `scm_writing` | Diff-derived text for a confirmed Git action |
+| `forge` | GitHub and GitLab through their own CLIs: `Forge` and the public functions (`mod.rs`), one CLI call (`process`), answer parsing (`parse`, `github`, `gitlab`), list details (`details`), reads (`pulls`), comments and reviews (`review`), edits and merges (`merge`), creating and publishing (`create`), sign-in status (`status`) |
+| `forge::cache` | Short-lived memory of GitHub pull request details |
 | `pull_guides` | An agent's reading order for one pull request |
 
+Each folder's `mod.rs` is its front door: the old paths such as `source_control::forge::list_pulls` still resolve through its re-exports. Tests sit beside the module in `tests.rs`.
+
 The crate's [group table](../../README.md#groups) lists every group.