feat: shell command as bg job

.github/workflows/test-zsh-setup.yml

@@ -0,0 +1,274 @@
+# -------------------------------------------------------------------
+# ------------------------------- WARNING ---------------------------
+# -------------------------------------------------------------------
+#
+# This file was automatically generated by gh-workflows using the
+# gh-workflow-gen bin. You should add and commit this file to your
+# git repository. **DO NOT EDIT THIS FILE BY HAND!** Any manual changes
+# will be lost if the file is regenerated.
+#
+# To make modifications, update your `build.rs` configuration to adjust
+# the workflow description as needed, then regenerate this file to apply
+# those changes.
+#
+# -------------------------------------------------------------------
+# ----------------------------- END WARNING -------------------------
+# -------------------------------------------------------------------
+
+name: Test ZSH Setup
+'on':
+  pull_request:
+    types:
+    - opened
+    - synchronize
+    - reopened
+    paths:
+    - crates/forge_main/src/zsh/**
+    - crates/forge_main/src/ui.rs
+    - crates/forge_ci/tests/scripts/test-zsh-setup.sh
+    - crates/forge_ci/tests/scripts/test-zsh-setup-macos.sh
+    - crates/forge_ci/tests/scripts/test-zsh-setup-windows.sh
+    - '.github/workflows/test-zsh-setup.yml'
+  push:
+    branches:
+    - main
+  workflow_dispatch: {}
+jobs:
+  test_zsh_setup_amd64:
+    name: Test ZSH Setup (amd64)
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+    steps:
+    - name: Checkout Code
+      uses: actions/checkout@v6
+    - name: Cache Cargo registry and git
+      uses: actions/cache@v4
+      with:
+        path: |-
+          ~/.cargo/registry
+          ~/.cargo/git
+        key: cargo-registry-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}
+        restore-keys: |-
+          cargo-registry-${{ runner.os }}-${{ runner.arch }}-
+          cargo-registry-${{ runner.os }}-
+    - name: Cache Rust toolchains
+      uses: actions/cache@v4
+      with:
+        path: ~/.rustup
+        key: rustup-${{ runner.os }}-${{ runner.arch }}
+    - name: Cache build artifacts
+      uses: actions/cache@v4
+      with:
+        path: target
+        key: build-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}-${{ hashFiles('**/*.rs') }}
+        restore-keys: |-
+          build-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}-
+          build-${{ runner.os }}-${{ runner.arch }}-
+    - name: Setup Protobuf Compiler
+      uses: arduino/setup-protoc@v3
+      with:
+        repo-token: ${{ secrets.GITHUB_TOKEN }}
+    - name: Setup Cross Toolchain
+      uses: taiki-e/setup-cross-toolchain-action@v1
+      with:
+        target: x86_64-unknown-linux-musl
+    - name: Run ZSH setup test suite
+      run: bash crates/forge_ci/tests/scripts/test-zsh-setup.sh --native-build --no-cleanup --jobs 4
+    - name: Upload test results
+      if: always()
+      uses: actions/upload-artifact@v4
+      with:
+        name: zsh-setup-results-linux-amd64
+        path: test-results-linux/
+        retention-days: 7
+        if-no-files-found: ignore
+  test_zsh_setup_arm64:
+    name: Test ZSH Setup (arm64)
+    runs-on: ubuntu-24.04-arm
+    permissions:
+      contents: read
+    steps:
+    - name: Checkout Code
+      uses: actions/checkout@v6
+    - name: Cache Cargo registry and git
+      uses: actions/cache@v4
+      with:
+        path: |-
+          ~/.cargo/registry
+          ~/.cargo/git
+        key: cargo-registry-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}
+        restore-keys: |-
+          cargo-registry-${{ runner.os }}-${{ runner.arch }}-
+          cargo-registry-${{ runner.os }}-
+    - name: Cache Rust toolchains
+      uses: actions/cache@v4
+      with:
+        path: ~/.rustup
+        key: rustup-${{ runner.os }}-${{ runner.arch }}
+    - name: Cache build artifacts
+      uses: actions/cache@v4
+      with:
+        path: target
+        key: build-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}-${{ hashFiles('**/*.rs') }}
+        restore-keys: |-
+          build-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}-
+          build-${{ runner.os }}-${{ runner.arch }}-
+    - name: Setup Protobuf Compiler
+      uses: arduino/setup-protoc@v3
+      with:
+        repo-token: ${{ secrets.GITHUB_TOKEN }}
+    - name: Setup Cross Toolchain
+      uses: taiki-e/setup-cross-toolchain-action@v1
+      with:
+        target: aarch64-unknown-linux-musl
+    - name: Run ZSH setup test suite (exclude Arch)
+      run: bash crates/forge_ci/tests/scripts/test-zsh-setup.sh --native-build --no-cleanup --exclude "Arch Linux" --jobs 4
+    - name: Upload test results
+      if: always()
+      uses: actions/upload-artifact@v4
+      with:
+        name: zsh-setup-results-linux-arm64
+        path: test-results-linux/
+        retention-days: 7
+        if-no-files-found: ignore
+  test_zsh_setup_macos_arm64:
+    name: Test ZSH Setup (macOS arm64)
+    runs-on: macos-latest
+    permissions:
+      contents: read
+    steps:
+    - name: Checkout Code
+      uses: actions/checkout@v6
+    - name: Cache Cargo registry and git
+      uses: actions/cache@v4
+      with:
+        path: |-
+          ~/.cargo/registry
+          ~/.cargo/git
+        key: cargo-registry-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}
+        restore-keys: |-
+          cargo-registry-${{ runner.os }}-${{ runner.arch }}-
+          cargo-registry-${{ runner.os }}-
+    - name: Cache Rust toolchains
+      uses: actions/cache@v4
+      with:
+        path: ~/.rustup
+        key: rustup-${{ runner.os }}-${{ runner.arch }}
+    - name: Cache build artifacts
+      uses: actions/cache@v4
+      with:
+        path: target
+        key: build-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}-${{ hashFiles('**/*.rs') }}
+        restore-keys: |-
+          build-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}-
+          build-${{ runner.os }}-${{ runner.arch }}-
+    - name: Setup Protobuf Compiler
+      uses: arduino/setup-protoc@v3
+      with:
+        repo-token: ${{ secrets.GITHUB_TOKEN }}
+    - name: Install shellcheck
+      run: brew install shellcheck
+    - name: Run macOS ZSH setup test suite
+      run: bash crates/forge_ci/tests/scripts/test-zsh-setup-macos.sh --no-cleanup
+    - name: Upload test results
+      if: always()
+      uses: actions/upload-artifact@v4
+      with:
+        name: zsh-setup-results-macos-arm64
+        path: test-results-macos/
+        retention-days: 7
+        if-no-files-found: ignore
+  test_zsh_setup_windows:
+    name: Test ZSH Setup (Windows x86_64)
+    runs-on: windows-latest
+    permissions:
+      contents: read
+    steps:
+    - name: Checkout Code
+      uses: actions/checkout@v6
+    - name: Cache Cargo registry and git
+      uses: actions/cache@v4
+      with:
+        path: |-
+          ~/.cargo/registry
+          ~/.cargo/git
+        key: cargo-registry-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}
+        restore-keys: |-
+          cargo-registry-${{ runner.os }}-${{ runner.arch }}-
+          cargo-registry-${{ runner.os }}-
+    - name: Cache Rust toolchains
+      uses: actions/cache@v4
+      with:
+        path: ~/.rustup
+        key: rustup-${{ runner.os }}-${{ runner.arch }}
+    - name: Cache build artifacts
+      uses: actions/cache@v4
+      with:
+        path: target
+        key: build-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}-${{ hashFiles('**/*.rs') }}
+        restore-keys: |-
+          build-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}-
+          build-${{ runner.os }}-${{ runner.arch }}-
+    - name: Setup Protobuf Compiler
+      uses: arduino/setup-protoc@v3
+      with:
+        repo-token: ${{ secrets.GITHUB_TOKEN }}
+    - name: Run Windows ZSH setup test suite
+      run: bash crates/forge_ci/tests/scripts/test-zsh-setup-windows.sh --no-cleanup
+    - name: Upload test results
+      if: always()
+      uses: actions/upload-artifact@v4
+      with:
+        name: zsh-setup-results-windows
+        path: test-results-windows/
+        retention-days: 7
+        if-no-files-found: ignore
+  test_zsh_setup_windows_arm64:
+    name: Test ZSH Setup (Windows arm64)
+    runs-on: windows-11-arm
+    permissions:
+      contents: read
+    steps:
+    - name: Checkout Code
+      uses: actions/checkout@v6
+    - name: Cache Cargo registry and git
+      uses: actions/cache@v4
+      with:
+        path: |-
+          ~/.cargo/registry
+          ~/.cargo/git
+        key: cargo-registry-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}
+        restore-keys: |-
+          cargo-registry-${{ runner.os }}-${{ runner.arch }}-
+          cargo-registry-${{ runner.os }}-
+    - name: Cache Rust toolchains
+      uses: actions/cache@v4
+      with:
+        path: ~/.rustup
+        key: rustup-${{ runner.os }}-${{ runner.arch }}
+    - name: Cache build artifacts
+      uses: actions/cache@v4
+      with:
+        path: target
+        key: build-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}-${{ hashFiles('**/*.rs') }}
+        restore-keys: |-
+          build-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('**/Cargo.lock') }}-
+          build-${{ runner.os }}-${{ runner.arch }}-
+    - name: Setup Protobuf Compiler
+      uses: arduino/setup-protoc@v3
+      with:
+        repo-token: ${{ secrets.GITHUB_TOKEN }}
+    - name: Run Windows ARM64 ZSH setup test suite
+      run: bash crates/forge_ci/tests/scripts/test-zsh-setup-windows.sh --no-cleanup
+    - name: Upload test results
+      if: always()
+      uses: actions/upload-artifact@v4
+      with:
+        name: zsh-setup-results-windows-arm64
+        path: test-results-windows/
+        retention-days: 7
+        if-no-files-found: ignore
+concurrency:
+  group: test-zsh-setup-${{ github.ref }}
+  cancel-in-progress: true

crates/forge_api/src/api.rs

@@ -253,4 +253,10 @@ pub trait API: Sync + Send {
         &self,
         data_parameters: DataGenerationParameters,
     ) -> Result<BoxStream<'static, Result<serde_json::Value, anyhow::Error>>>;
+
+    /// Returns all tracked background processes with their alive status.
+    fn list_background_processes(&self) -> Result<Vec<(forge_domain::BackgroundProcess, bool)>>;
+
+    /// Kills a background process by PID and optionally deletes its log file.
+    fn kill_background_process(&self, pid: u32, delete_log: bool) -> Result<()>;
 }

crates/forge_api/src/forge_api.rs

@@ -8,8 +8,8 @@ use forge_app::{
     AgentProviderResolver, AgentRegistry, AppConfigService, AuthService, CommandInfra,
     CommandLoaderService, ConversationService, DataGenerationApp, EnvironmentInfra,
     EnvironmentService, FileDiscoveryService, ForgeApp, GitApp, GrpcInfra, McpConfigManager,
-    McpService, ProviderAuthService, ProviderService, Services, User, UserUsage, Walker,
-    WorkspaceService,
+    McpService, ProviderAuthService, ProviderService, Services, ShellService, User, UserUsage,
+    Walker, WorkspaceService,
 };
 use forge_domain::{Agent, ConsoleWriter, InitAuth, LoginInfo, *};
 use forge_infra::ForgeInfra;
@@ -416,6 +416,14 @@ impl<
         app.execute(data_parameters).await
     }
 
+    fn list_background_processes(&self) -> Result<Vec<(forge_domain::BackgroundProcess, bool)>> {
+        self.services.shell_service().list_background_processes()
+    }
+
+    fn kill_background_process(&self, pid: u32, delete_log: bool) -> Result<()> {
+        self.services.shell_service().kill_background_process(pid, delete_log)
+    }
+
     async fn get_default_provider(&self) -> Result<Provider<Url>> {
         let provider_id = self.services.get_default_provider().await?;
         self.services.get_provider(provider_id).await

crates/forge_app/src/fmt/fmt_input.rs

@@ -100,11 +100,14 @@ impl FormatContent for ToolCatalog {
                 let display_path = display_path_for(&input.path);
                 Some(TitleFormat::debug("Undo").sub_title(display_path).into())
             }
-            ToolCatalog::Shell(input) => Some(
-                TitleFormat::debug(format!("Execute [{}]", env.shell))
-                    .sub_title(&input.command)
-                    .into(),
-            ),
+            ToolCatalog::Shell(input) => {
+                let label = if input.background {
+                    format!("Spawned [{}]", env.shell)
+                } else {
+                    format!("Execute [{}]", env.shell)
+                };
+                Some(TitleFormat::debug(label).sub_title(&input.command).into())
+            }
             ToolCatalog::Fetch(input) => {
                 Some(TitleFormat::debug("GET").sub_title(&input.url).into())
             }