mirror of
https://github.com/SuperClaude-Org/SuperClaude_Framework.git
synced 2025-12-29 16:16:08 +00:00
* feat: add missing install.sh script referenced in README\n\n- Create comprehensive installation script with POSIX compatibility\n- Add interactive and non-interactive installation modes\n- Include prerequisites checking and MCP server setup guidance\n- Replace echo -e with printf for better POSIX compliance * fix: resolve linting errors in install_mcp.py and clean_command_names.py Fix multiple ruff linting errors to ensure CI/CD pipeline passes: - install_mcp.py: Remove unused pathlib.Path import, replace bare except with specific exception types (ValueError, IndexError), remove extraneous f-string prefixes on lines without placeholders - clean_command_names.py: Remove unused os import, convert f-strings without placeholders to regular strings - pyproject.toml: Exclude docs/ directory from ruff checks to avoid N999 module naming violations in documentation templates All linting checks now pass successfully. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * style: apply ruff format to Python source files Apply ruff formatting rules to CLI and scripts modules to ensure consistent code style across the codebase. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix(ci): remove incompatible pip cache from quick-check workflow ## Problem GitHub Actions was failing with error: "Cache folder path is retrieved for pip but doesn't exist on disk: /home/runner/.cache/pip. This likely indicates that there are no dependencies to cache." ## Root Cause The quick-check.yml workflow specified `cache: 'pip'` in the Python setup step, but the workflow uses UV (not pip) for package management via `uv pip install --system -e ".[dev]"`. UV uses its own cache directory (~/.cache/uv), so the pip cache path was never created, causing the error. This was a migration oversight: - When UV was adopted as the project standard (commit00706f0), the CLAUDE.md established "CRITICAL: Never use pip directly" rule - The test.yml workflow was created correctly without pip cache - The quick-check.yml workflow incorrectly included pip cache from initial creation (commit8c0559c) and was not updated during migration ## Solution Remove `cache: 'pip'` line to align with: - Project's UV-first architecture (CLAUDE.md) - test.yml workflow (which runs successfully without pip cache) - readme-quality-check.yml workflow (no cache needed) Note: publish-pypi.yml intentionally uses pip cache as it directly runs `python -m pip install` commands, which is correct for that workflow. ## Impact - ✅ Eliminates GitHub Actions cache warning - ✅ Aligns all UV-based workflows consistently - ✅ Follows project standards documented in CLAUDE.md 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
GitHub Actions Workflows
This directory contains CI/CD workflows for SuperClaude Framework.
Workflows
1. test.yml - Comprehensive Test Suite
Triggers: Push/PR to master or integration, manual dispatch
Jobs:
- test: Run tests on Python 3.10, 3.11, 3.12
- Install UV and dependencies
- Run full test suite
- Generate coverage report (Python 3.10 only)
- Upload to Codecov
- lint: Run ruff linter and format checker
- plugin-check: Verify pytest plugin loads correctly
- doctor-check: Run
superclaude doctorhealth check - test-summary: Aggregate results from all jobs
Status Badge:
[](https://github.com/SuperClaude-Org/SuperClaude_Framework/actions/workflows/test.yml)
2. quick-check.yml - Fast PR Feedback
Triggers: Pull requests to master or integration
Jobs:
- quick-test: Fast check on Python 3.10 only
- Run unit tests only (faster)
- Run linter
- Check formatting
- Verify plugin loads
- 10 minute timeout
Purpose: Provide rapid feedback on PRs before running full test matrix.
3. publish-pypi.yml (Existing)
Triggers: Manual or release tags Purpose: Publish package to PyPI
4. readme-quality-check.yml (Existing)
Triggers: Push/PR affecting README files Purpose: Validate README quality and consistency
Local Testing
Before pushing, run these commands locally:
# Run full test suite
uv run pytest -v
# Run with coverage
uv run pytest --cov=superclaude --cov-report=term
# Run linter
uv run ruff check src/ tests/
# Check formatting
uv run ruff format --check src/ tests/
# Auto-fix formatting
uv run ruff format src/ tests/
# Verify plugin loads
uv run pytest --trace-config | grep superclaude
# Run doctor check
uv run superclaude doctor --verbose
CI/CD Pipeline
┌─────────────────────┐
│ Push/PR Created │
└──────────┬──────────┘
│
├─────────────────────────┐
│ │
┌──────▼──────┐ ┌───────▼────────┐
│ Quick Check │ │ Full Test │
│ (PR only) │ │ Matrix │
│ │ │ │
│ • Unit tests│ │ • Python 3.10 │
│ • Lint │ │ • Python 3.11 │
│ • Format │ │ • Python 3.12 │
│ │ │ • Coverage │
│ ~2-3 min │ │ • Lint │
└─────────────┘ │ • Plugin check │
│ • Doctor check │
│ │
│ ~5-8 min │
└────────────────┘
Coverage Reporting
Coverage reports are generated for Python 3.10 and uploaded to Codecov.
To view coverage locally:
uv run pytest --cov=superclaude --cov-report=html
open htmlcov/index.html
Troubleshooting
Workflow fails with "UV not found"
- UV is installed in each job via
curl -LsSf https://astral.sh/uv/install.sh | sh - If installation fails, check UV's status page
Tests fail locally but pass in CI (or vice versa)
- Check Python version:
python --version - Reinstall dependencies:
uv pip install -e ".[dev]" - Clear caches:
rm -rf .pytest_cache .venv
Plugin not loading in CI
- Verify entry point in
pyproject.toml:[project.entry-points.pytest11] - Check plugin is installed:
uv run pytest --trace-config
Coverage upload fails
- This is non-blocking (fail_ci_if_error: false)
- Check Codecov token in repository secrets
Maintenance
Adding a New Workflow
- Create new
.ymlfile in this directory - Follow existing structure (checkout, setup-python, install UV)
- Add status badge to README.md if needed
- Document in this file
Updating Python Versions
- Edit
matrix.python-versionintest.yml - Update
pyproject.tomlclassifiers - Test locally with new version first
Modifying Test Strategy
- quick-check.yml: For fast PR feedback (unit tests only)
- test.yml: For comprehensive validation (full matrix)
Best Practices
- Keep workflows fast: Use caching, parallel jobs
- Fail fast: Use
-xflag in pytest for quick-check - Clear names: Job and step names should be descriptive
- Version pinning: Pin action versions (@v4, @v5)
- Matrix testing: Test on multiple Python versions
- Non-blocking coverage: Don't fail on coverage upload errors
- Manual triggers: Add
workflow_dispatchfor debugging