fix(ci): make cargo-make tasks cross-platform

- Use taiki-e/install-action@cargo-deny instead of cargo-make install
- Replace bash scripts with cross-platform cargo-make commands
- Split test-node, test-python, coverage-* tasks into subtasks
- Use cwd and command/args instead of script blocks
- Fixes Windows CI failures due to missing /bin/bash

Author: bug-ops

.github/workflows/ci.yml

@@ -74,11 +74,11 @@ jobs:
         with:
           shared-key: "security"
 
-      - name: Install cargo-make
-        uses: taiki-e/install-action@cargo-make
+      - name: Install cargo-deny
+        uses: taiki-e/install-action@cargo-deny
 
-      - name: Run security checks
-        run: cargo make ci-security
+      - name: Run cargo-deny
+        run: cargo deny check
 
   # Cross-platform Rust tests
   test-rust:

Makefile.toml

@@ -129,49 +129,49 @@ dependencies = ["test-rust", "doctest"]
 
 [tasks.test-python]
 description = "Build and test Python bindings"
-dependencies = ["build-python", "test-python-run"]
+condition = { files_exist = ["${CARGO_MAKE_WORKING_DIRECTORY}/crates/feedparser-rs-py/Cargo.toml"] }
+dependencies = ["test-python-deps", "test-python-build", "test-python-install", "test-python-run", "test-python-import"]
+
+[tasks.test-python-deps]
+description = "Install Python dependencies"
+command = "uv"
+args = ["pip", "install", "--system", "maturin", "pytest"]
 
-[tasks.build-python]
+[tasks.test-python-build]
 description = "Build Python wheel with maturin"
-condition = { files_exist = ["${CARGO_MAKE_WORKING_DIRECTORY}/crates/feedparser-rs-py/Cargo.toml"] }
-script = '''
-#!/bin/bash
-set -e
-cd crates/feedparser-rs-py
+cwd = "./crates/feedparser-rs-py"
+command = "maturin"
+args = ["build", "--release", "--out", "dist"]
 
-# Check if uv is available, fallback to pip
-if command -v uv &> /dev/null; then
-    uv pip install --system maturin pytest
-else
-    pip install maturin pytest
-fi
+[tasks.test-python-install]
+description = "Install built wheel"
+cwd = "./crates/feedparser-rs-py"
+command = "uv"
+args = ["pip", "install", "--system", "--reinstall", "--no-deps", "dist/feedparser_rs-0.1.0-cp39-abi3-${CARGO_MAKE_RUST_TARGET_TRIPLE}.whl"]
+# Note: wheel name pattern may need adjustment per platform
 
-maturin build --release --out dist
-'''
+[tasks.test-python-install.linux]
+command = "uv"
+args = ["pip", "install", "--system", "--reinstall", "--no-deps", "--find-links", "dist", "feedparser-rs"]
 
-[tasks.test-python-run]
-description = "Run pytest on Python bindings"
-condition = { files_exist = ["${CARGO_MAKE_WORKING_DIRECTORY}/crates/feedparser-rs-py/tests"] }
-script = '''
-#!/bin/bash
-set -e
-cd crates/feedparser-rs-py
+[tasks.test-python-install.mac]
+command = "uv"
+args = ["pip", "install", "--system", "--reinstall", "--no-deps", "--find-links", "dist", "feedparser-rs"]
 
-# Install the built wheel
-if command -v uv &> /dev/null; then
-    uv pip install --system dist/*.whl
-else
-    pip install dist/*.whl
-fi
+[tasks.test-python-install.windows]
+command = "uv"
+args = ["pip", "install", "--system", "--reinstall", "--no-deps", "--find-links", "dist", "feedparser-rs"]
 
-# Run tests if they exist
-if [ -d "tests" ] && [ "$(ls -A tests/*.py 2>/dev/null)" ]; then
-    pytest tests/ -v
-fi
+[tasks.test-python-run]
+description = "Run pytest"
+cwd = "./crates/feedparser-rs-py"
+command = "pytest"
+args = ["tests/", "-v"]
 
-# Test import
-python -c "import feedparser_rs; print('feedparser-rs loaded successfully')"
-'''
+[tasks.test-python-import]
+description = "Test Python import"
+command = "python"
+args = ["-c", "import feedparser_rs; print('feedparser-rs loaded successfully')"]
 
 # ============================================================================
 # Testing Tasks - Node.js
@@ -180,15 +180,25 @@ python -c "import feedparser_rs; print('feedparser-rs loaded successfully')"
 [tasks.test-node]
 description = "Build and test Node.js bindings"
 condition = { files_exist = ["${CARGO_MAKE_WORKING_DIRECTORY}/crates/feedparser-rs-node/package.json"] }
-script = '''
-#!/bin/bash
-set -e
-cd crates/feedparser-rs-node
-
-npm ci
-npm run build
-npm test
-'''
+dependencies = ["test-node-install", "test-node-build", "test-node-run"]
+
+[tasks.test-node-install]
+description = "Install Node.js dependencies"
+cwd = "./crates/feedparser-rs-node"
+command = "npm"
+args = ["ci"]
+
+[tasks.test-node-build]
+description = "Build Node.js bindings"
+cwd = "./crates/feedparser-rs-node"
+command = "npm"
+args = ["run", "build"]
+
+[tasks.test-node-run]
+description = "Run Node.js tests"
+cwd = "./crates/feedparser-rs-node"
+command = "npm"
+args = ["test"]
 
 # ============================================================================
 # Coverage Tasks
@@ -219,47 +229,53 @@ echo "Coverage report generated: lcov.info"
 [tasks.coverage-python]
 description = "Generate Python code coverage"
 condition = { files_exist = ["${CARGO_MAKE_WORKING_DIRECTORY}/crates/feedparser-rs-py/tests"] }
-script = '''
-#!/bin/bash
-set -e
-cd crates/feedparser-rs-py
-
-# Install dependencies
-if command -v uv &> /dev/null; then
-    uv pip install --system maturin pytest pytest-cov
-else
-    pip install maturin pytest pytest-cov
-fi
-
-# Build and install wheel
-maturin build --release --out dist
-if command -v uv &> /dev/null; then
-    uv pip install --system --force-reinstall dist/*.whl
-else
-    pip install --force-reinstall dist/*.whl
-fi
-
-# Run tests with coverage if tests exist
-if [ -d "tests" ] && [ "$(ls -A tests/*.py 2>/dev/null)" ]; then
-    pytest tests/ --cov=feedparser_rs --cov-report=xml --cov-report=term
-    echo "Python coverage report generated: coverage.xml"
-fi
-'''
+dependencies = ["coverage-python-deps", "coverage-python-build", "coverage-python-install", "coverage-python-run"]
+
+[tasks.coverage-python-deps]
+description = "Install Python coverage dependencies"
+command = "uv"
+args = ["pip", "install", "--system", "maturin", "pytest", "pytest-cov"]
+
+[tasks.coverage-python-build]
+description = "Build Python wheel for coverage"
+cwd = "./crates/feedparser-rs-py"
+command = "maturin"
+args = ["build", "--release", "--out", "dist"]
+
+[tasks.coverage-python-install]
+description = "Install built wheel for coverage"
+cwd = "./crates/feedparser-rs-py"
+command = "uv"
+args = ["pip", "install", "--system", "--reinstall", "--no-deps", "--find-links", "dist", "feedparser-rs"]
+
+[tasks.coverage-python-run]
+description = "Run pytest with coverage"
+cwd = "./crates/feedparser-rs-py"
+command = "pytest"
+args = ["tests/", "--cov=feedparser_rs", "--cov-report=xml", "--cov-report=term"]
 
 [tasks.coverage-node]
 description = "Generate Node.js code coverage with c8"
 condition = { files_exist = ["${CARGO_MAKE_WORKING_DIRECTORY}/crates/feedparser-rs-node/package.json"] }
-script = '''
-#!/bin/bash
-set -e
-cd crates/feedparser-rs-node
-
-npm ci
-npm run build
-npm run test:coverage
-
-echo "Node.js coverage report generated in coverage/"
-'''
+dependencies = ["coverage-node-install", "coverage-node-build", "coverage-node-run"]
+
+[tasks.coverage-node-install]
+description = "Install Node.js dependencies for coverage"
+cwd = "./crates/feedparser-rs-node"
+command = "npm"
+args = ["ci"]
+
+[tasks.coverage-node-build]
+description = "Build Node.js bindings for coverage"
+cwd = "./crates/feedparser-rs-node"
+command = "npm"
+args = ["run", "build"]
+
+[tasks.coverage-node-run]
+description = "Run Node.js tests with coverage"
+cwd = "./crates/feedparser-rs-node"
+command = "npm"
+args = ["run", "test:coverage"]
 
 # ============================================================================
 # MSRV Check