refactor(desktop): split oversized files instead of raising the ratchet

The new file-size gate (#6187) pins each file to its base size when that is
already over the 1000-line budget, so this branch's additions pushed six
files past their ceiling. Every one is split rather than exempted:

- shared/api/tauri.ts: the per-turn routing API moves to
  shared/api/routingPolicy.ts. Self-contained apart from invokeTauri, and
  deliberately not re-exported from tauri.ts -- that would re-add the lines
  it sheds and create an import cycle.
- discovery.rs: the KNOWN_ACP_RUNTIMES table moves to
  discovery/known_runtimes.rs. Declared after windows_install so the
  macro_use macros its entries call are in scope.
- discovery/tests.rs and config_bridge/reader_tests.rs: the OpenCode tests
  move to sibling modules, following the #[path] split reader_tests.rs
  already carried for this reason.
- AgentInstanceEditDialog.tsx: drops a handleOpenChange wrapper that only
  forwarded to the onOpenChange prop.
- lib.rs: collapses three huddle imports into one nested use and globs
  deep_link, matching the globs the file already uses. The tauri::Listener
  and shutdown imports are left alone -- both sit under cfg attributes that
  merging would silently widen.

Signed-off-by: Michael Feth <michael@jira-flow.com>

Author: Michael Feth

desktop/src-tauri/src/lib.rs

@@ -50,24 +50,17 @@ use builderlab::*;
 #[doc(hidden)]
 pub use commands::print_agent_access_owner_only_probe_if_requested;
 use commands::*;
-use deep_link::{
-    acknowledge_pending_community_deep_link, acknowledge_pending_entity_deep_link,
-    acknowledge_pending_navigation_deep_link, clear_pending_navigation_deep_links,
-    handle_deep_link_url, take_pending_community_deep_link, take_pending_entity_deep_link,
-    take_pending_navigation_deep_link, PendingCommunityDeepLinks, PendingEntityDeepLinks,
-    PendingNavigationDeepLinks,
-};
-use huddle::audio_output::{
-    get_audio_output_device, list_audio_output_devices, set_audio_output_device,
-};
-use huddle::reconnect::reconnect_huddle_audio;
+use deep_link::*;
 use huddle::{
-    add_agent_to_huddle, check_pipeline_hotstart, close_huddle_companion, confirm_huddle_active,
-    download_voice_models, end_huddle, get_huddle_agent_pubkeys, get_huddle_state,
-    get_model_status, get_voice_input_mode, interrupt_huddle_speech, join_huddle, leave_huddle,
-    open_huddle_window, push_audio_pcm, remove_agent_from_huddle, set_huddle_manual_mic_unmuted,
-    set_huddle_transcription_enabled, set_tts_enabled, set_voice_input_mode, speak_agent_message,
-    start_huddle, start_stt_pipeline, HuddlePhase,
+    add_agent_to_huddle,
+    audio_output::{get_audio_output_device, list_audio_output_devices, set_audio_output_device},
+    check_pipeline_hotstart, close_huddle_companion, confirm_huddle_active, download_voice_models,
+    end_huddle, get_huddle_agent_pubkeys, get_huddle_state, get_model_status, get_voice_input_mode,
+    interrupt_huddle_speech, join_huddle, leave_huddle, open_huddle_window, push_audio_pcm,
+    reconnect::reconnect_huddle_audio,
+    remove_agent_from_huddle, set_huddle_manual_mic_unmuted, set_huddle_transcription_enabled,
+    set_tts_enabled, set_voice_input_mode, speak_agent_message, start_huddle, start_stt_pipeline,
+    HuddlePhase,
 };
 use initial_window::*;
 use managed_agents::{

desktop/src-tauri/src/managed_agents/config_bridge/reader_tests.rs

@@ -219,86 +219,6 @@ fn goose_mcp_config_path_follows_path_root_override() {
     );
 }
 
-static OPENCODE_CONFIG_LOCK: Mutex<()> = Mutex::new(());
-
-fn with_opencode_config<T>(path: &Path, body: impl FnOnce() -> T) -> T {
-    let _guard = OPENCODE_CONFIG_LOCK
-        .lock()
-        .unwrap_or_else(|err| err.into_inner());
-    let prior = std::env::var_os("OPENCODE_CONFIG");
-    std::env::set_var("OPENCODE_CONFIG", path);
-    let output = body();
-    match prior {
-        Some(value) => std::env::set_var("OPENCODE_CONFIG", value),
-        None => std::env::remove_var("OPENCODE_CONFIG"),
-    }
-    output
-}
-
-/// End-to-end wiring guard for the whole point of the OpenCode entry: the
-/// harness takes no `--model` flag and reads no model env var, so unless the
-/// bridge reaches its config file the model field is blank in the panel.
-#[test]
-fn opencode_surface_takes_its_model_from_the_config_file() {
-    let dir = tempfile::tempdir().expect("tempdir");
-    let config = dir.path().join("opencode.jsonc");
-    std::fs::write(
-        &config,
-        r#"{
-            // real OpenCode configs are JSONC with comments and trailing commas
-            "$schema": "https://opencode.ai/config.json",
-            "model": "anthropic/claude-sonnet-4-5",
-            "mcp": { "filesystem": { "type": "local" } },
-        }"#,
-    )
-    .expect("write config");
-
-    let record = test_record();
-    let runtime = &KnownAcpRuntime {
-        id: "opencode",
-        label: "OpenCode",
-        commands: &["opencode"],
-        model_env_var: None,
-        provider_env_var: None,
-        supports_acp_native_config: false,
-        thinking_env_var: None,
-        max_tokens_env_var: None,
-        context_limit_env_var: None,
-        required_normalized_fields: &[],
-        config_file_path: Some("~/.config/opencode/opencode.json"),
-        config_file_format: Some("json"),
-        ..*test_runtime()
-    };
-
-    let surface = with_opencode_config(&config, || {
-        read_config_surface(&record, Some(runtime), None, None)
-    });
-
-    let model = surface.normalized.model.expect("model field");
-    assert_eq!(model.value.as_deref(), Some("claude-sonnet-4-5"));
-    assert_eq!(model.origin, ConfigOrigin::ConfigFile);
-    // Nothing can write it back — no env var, no ACP model switching.
-    assert!(matches!(model.write_via, ConfigWriteMechanism::ReadOnly));
-
-    let provider = surface.normalized.provider.expect("provider field");
-    assert_eq!(provider.value.as_deref(), Some("anthropic"));
-
-    assert_eq!(surface.sources.config_file, ConfigTierStatus::Available);
-    assert_eq!(
-        surface.sources.config_file_path.as_deref().map(Path::new),
-        Some(config.as_path()),
-        "the reported path must be the file actually read, not the static default"
-    );
-    assert_eq!(
-        surface
-            .extensions
-            .iter()
-            .map(|e| e.name.as_str())
-            .collect::<Vec<_>>(),
-        vec!["filesystem"]
-    );
-}
-
 #[test]
 fn claude_surface_uses_mcp_config_path_not_settings_path() {
     let record = test_record();
@@ -1032,3 +952,6 @@ fn numeric_max_tokens_inherits_from_global_env() {
 // ── Extended tests (split file to respect line-count ratchet) ────────────────
 #[path = "reader_tests_ext.rs"]
 mod ext;
+
+#[path = "reader_tests_opencode.rs"]
+mod opencode_tests;

desktop/src-tauri/src/managed_agents/config_bridge/reader_tests_opencode.rs

@@ -0,0 +1,87 @@
+//! OpenCode config-bridge tests — split out of `reader_tests.rs` to keep it
+//! under the 1000-line file-size ratchet.
+//!
+//! Included as `mod opencode_tests` inside `reader_tests.rs`, so `use super::*`
+//! gives access to all helpers and types from that module.
+
+use super::*;
+
+static OPENCODE_CONFIG_LOCK: Mutex<()> = Mutex::new(());
+
+fn with_opencode_config<T>(path: &Path, body: impl FnOnce() -> T) -> T {
+    let _guard = OPENCODE_CONFIG_LOCK
+        .lock()
+        .unwrap_or_else(|err| err.into_inner());
+    let prior = std::env::var_os("OPENCODE_CONFIG");
+    std::env::set_var("OPENCODE_CONFIG", path);
+    let output = body();
+    match prior {
+        Some(value) => std::env::set_var("OPENCODE_CONFIG", value),
+        None => std::env::remove_var("OPENCODE_CONFIG"),
+    }
+    output
+}
+
+/// End-to-end wiring guard for the whole point of the OpenCode entry: the
+/// harness takes no `--model` flag and reads no model env var, so unless the
+/// bridge reaches its config file the model field is blank in the panel.
+#[test]
+fn opencode_surface_takes_its_model_from_the_config_file() {
+    let dir = tempfile::tempdir().expect("tempdir");
+    let config = dir.path().join("opencode.jsonc");
+    std::fs::write(
+        &config,
+        r#"{
+            // real OpenCode configs are JSONC with comments and trailing commas
+            "$schema": "https://opencode.ai/config.json",
+            "model": "anthropic/claude-sonnet-4-5",
+            "mcp": { "filesystem": { "type": "local" } },
+        }"#,
+    )
+    .expect("write config");
+
+    let record = test_record();
+    let runtime = &KnownAcpRuntime {
+        id: "opencode",
+        label: "OpenCode",
+        commands: &["opencode"],
+        model_env_var: None,
+        provider_env_var: None,
+        supports_acp_native_config: false,
+        thinking_env_var: None,
+        max_tokens_env_var: None,
+        context_limit_env_var: None,
+        required_normalized_fields: &[],
+        config_file_path: Some("~/.config/opencode/opencode.json"),
+        config_file_format: Some("json"),
+        ..*test_runtime()
+    };
+
+    let surface = with_opencode_config(&config, || {
+        read_config_surface(&record, Some(runtime), None, &no_tiers())
+    });
+
+    let model = surface.normalized.model.expect("model field");
+    assert_eq!(model.value.as_deref(), Some("claude-sonnet-4-5"));
+    assert_eq!(model.origin, ConfigOrigin::ConfigFile);
+    // Nothing can write it back — no env var, no ACP model switching.
+    assert!(matches!(model.write_via, ConfigWriteMechanism::ReadOnly));
+
+    let provider = surface.normalized.provider.expect("provider field");
+    assert_eq!(provider.value.as_deref(), Some("anthropic"));
+
+    assert_eq!(surface.sources.config_file, ConfigTierStatus::Available);
+    assert_eq!(
+        surface.sources.config_file_path.as_deref().map(Path::new),
+        Some(config.as_path()),
+        "the reported path must be the file actually read, not the static default"
+    );
+    assert_eq!(
+        surface
+            .extensions
+            .iter()
+            .map(|e| e.name.as_str())
+            .collect::<Vec<_>>(),
+        vec!["filesystem"]
+    );
+}