Fix setuptools pre-commit and pixi install

Author: sanders41

.github/workflows/testing.yml

@@ -383,16 +383,16 @@ jobs:
         uses: dtolnay/rust-toolchain@stable
       - name: Cache Rust dependencies
         uses: Swatinem/[email protected]
-      - name: Build package
-        run: cargo build --release
-      - name: Run creation
-        run: ./scripts/ci_run.sh ${{ matrix.project_type }} 5
-        shell: bash
       - name: Install Pixi
         uses: prefix-dev/[email protected]
         with:
           manifest-path: ${{ env.WORKING_DIR }}/pyproject.toml
           pixi-version: v0.57.0
+      - name: Build package
+        run: cargo build --release
+      - name: Run creation
+        run: ./scripts/ci_run.sh ${{ matrix.project_type }} 5
+        shell: bash
       - name: Set up Python
         working-directory: ${{ env.WORKING_DIR }}
         run: pixi add python=="${{ env.MIN_PYTHON_VERSION }}.*"
@@ -418,16 +418,16 @@ jobs:
         uses: dtolnay/rust-toolchain@stable
       - name: Cache Rust dependencies
         uses: Swatinem/[email protected]
-      - name: Build package
-        run: cargo build --release
-      - name: Run creation
-        run: ./scripts/ci_run.sh ${{ matrix.project_type }} 5
-        shell: bash
       - name: Install Pixi
         uses: prefix-dev/[email protected]
         with:
           manifest-path: ${{ env.WORKING_DIR }}/pyproject.toml
           pixi-version: v0.57.0
+      - name: Build package
+        run: cargo build --release
+      - name: Run creation
+        run: ./scripts/ci_run.sh ${{ matrix.project_type }} 5
+        shell: bash
       - name: Set up Python
         working-directory: ${{ env.WORKING_DIR }}
         run: pixi add python=="${{ env.MIN_PYTHON_VERSION }}.*"

src/dev_dependency_installer.rs

@@ -193,13 +193,7 @@ fn setuptools_precommit_autoupdate(project_info: &ProjectInfo) -> Result<()> {
         bail!("Virtual environment not found at {}", venv_path.display());
     }
 
-    let precommit_bin = if cfg!(windows) {
-        venv_path.join("Scripts").join("pre-commit.exe")
-    } else {
-        venv_path.join("bin").join("pre-commit")
-    };
-
-    let output = std::process::Command::new(&precommit_bin)
+    let output = std::process::Command::new(".venv/bin/pre-commit")
         .args(["autoupdate"])
         .current_dir(base_dir)
         .output()?;
@@ -285,13 +279,7 @@ fn setuptools_precommit_install(project_info: &ProjectInfo) -> Result<()> {
         bail!("Virtual environment not found at {}", venv_path.display());
     }
 
-    let precommit_bin = if cfg!(windows) {
-        venv_path.join("Scripts").join("pre-commit.exe")
-    } else {
-        venv_path.join("bin").join("pre-commit")
-    };
-
-    let output = std::process::Command::new(&precommit_bin)
+    let output = std::process::Command::new(".venv/bin/pre-commit")
         .args(["install"])
         .current_dir(base_dir)
         .output()?;

src/fastapi/fastapi_installer.rs

@@ -102,14 +102,17 @@ fn poetry_fastapi_dependency_installer(project_info: &ProjectInfo) -> Result<()>
 }
 
 fn setuptools_fastapi_dependency_installer(project_info: &ProjectInfo) -> Result<()> {
-    let venv_output = std::process::Command::new("python")
-        .args(["-m", "venv", ".venv"])
-        .current_dir(project_info.base_dir())
-        .output()?;
-
-    if !venv_output.status.success() {
-        let stderr = String::from_utf8_lossy(&venv_output.stderr);
-        bail!("Failed to create virtual environment: {stderr}");
+    let venv_path = project_info.base_dir().join(".venv");
+    if !venv_path.exists() {
+        let venv_output = std::process::Command::new("python")
+            .args(["-m", "venv", ".venv"])
+            .current_dir(project_info.base_dir())
+            .output()?;
+
+        if !venv_output.status.success() {
+            let stderr = String::from_utf8_lossy(&venv_output.stderr);
+            bail!("Failed to create virtual environment: {stderr}");
+        }
     }
 
     let dependencies = FASTAPI_BASE_DEPENDENCIES.to_vec();