Feature/code orgnization

.flake8

@@ -8,4 +8,5 @@ statistics = true
 per-file-ignores =
     # Allow long lines in prompt files (they contain documentation strings)
     cli_tool/code_reviewer/prompt/*.py:E501
-
+    # Test files: Allow unused imports (fixtures) and unused variables (mocks)
+    tests/**/*.py:F401,F841,F811,F402

.github/workflows/docs.yml

@@ -29,11 +29,12 @@ jobs:
         uses: actions/setup-python@v5
         with:
           python-version: '3.12'
+          cache: 'pip'
+          cache-dependency-path: 'pyproject.toml'
 
       - name: Install dependencies
         run: |
-          pip install mkdocs-material mkdocs-click
-          pip install -e .
+          pip install ".[docs]"
 
       - name: Build documentation
         run: mkdocs build

.github/workflows/lint-reusable.yml

@@ -0,0 +1,27 @@
+name: Reusable Lint Workflow
+
+on:
+  workflow_call:
+
+jobs:
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Set up Python 3.12
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.12'
+          cache: 'pip'
+          cache-dependency-path: 'pyproject.toml'
+
+      - name: Install dependencies
+        run: |
+          pip install -e ".[dev]"
+
+      - name: Run pre-commit checks
+        run: pre-commit run --all-files

.github/workflows/release.yml

@@ -17,16 +17,29 @@ permissions:
   pull-requests: write
 
 jobs:
+  lint:
+    name: Lint
+    uses: ./.github/workflows/lint-reusable.yml
+
   test:
-    name: Run Tests
+    name: Test
+    needs: lint
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, windows-latest, macos-latest]
+        python-version: ['3.12', '3.13']
     uses: ./.github/workflows/test-reusable.yml
     with:
-      python-version: '3.12'
+      os: ${{ matrix.os }}
+      python-version: ${{ matrix.python-version }}
+    secrets:
+      CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
 
   check-version:
     name: Check next version (dry-run)
     runs-on: ubuntu-latest
-    needs: test
+    needs: [lint, test]
     outputs:
       should-release: ${{ steps.check.outputs.should-release }}
       next-version: ${{ steps.check.outputs.next-version }}
@@ -44,6 +57,7 @@ jobs:
         uses: actions/setup-python@v5
         with:
           python-version: '3.12'
+          cache: 'pip'
 
       - name: Install semantic-release
         run: pip install python-semantic-release
@@ -154,6 +168,8 @@ jobs:
         uses: actions/setup-python@v5
         with:
           python-version: '3.12'
+          cache: 'pip'
+          cache-dependency-path: 'pyproject.toml'
 
       - name: Build binary
         env:
@@ -268,6 +284,7 @@ jobs:
         uses: actions/setup-python@v5
         with:
           python-version: '3.12'
+          cache: 'pip'
 
       - name: Install semantic-release
         run: pip install python-semantic-release
@@ -348,19 +365,21 @@ jobs:
   notify-telegram:
     name: Notify Telegram
     runs-on: ubuntu-latest
-    needs: [test, check-version, build-binaries, create-release, upload-assets]
+    needs: [lint, test, check-version, build-binaries, create-release, upload-assets]
     if: always()
 
     steps:
       - name: Determine overall status
         id: status
         run: |
           # Check if any job failed or was cancelled
-          if [ "${{ needs.test.result }}" == "failure" ] || \
+          if [ "${{ needs.lint.result }}" == "failure" ] || \
+             [ "${{ needs.test.result }}" == "failure" ] || \
              [ "${{ needs.check-version.result }}" == "failure" ] || \
              [ "${{ needs.build-binaries.result }}" == "failure" ] || \
              [ "${{ needs.create-release.result }}" == "failure" ] || \
              [ "${{ needs.upload-assets.result }}" == "failure" ] || \
+             [ "${{ needs.lint.result }}" == "cancelled" ] || \
              [ "${{ needs.test.result }}" == "cancelled" ] || \
              [ "${{ needs.check-version.result }}" == "cancelled" ] || \
              [ "${{ needs.build-binaries.result }}" == "cancelled" ] || \
@@ -411,6 +430,7 @@ jobs:
             Commit: `${{ github.sha }}`
 
             Step Results:
+            • Lint: ${{ needs.lint.result }}
             • Tests: ${{ needs.test.result }}
             • Check Version: ${{ needs.check-version.result }}
             • Build Binaries: ${{ needs.build-binaries.result }}

.github/workflows/test-reusable.yml

@@ -8,29 +8,48 @@ on:
         required: false
         type: string
         default: '3.12'
+      os:
+        description: 'Operating system to test on'
+        required: false
+        type: string
+        default: 'ubuntu-latest'
+    secrets:
+      CODECOV_TOKEN:
+        description: 'Codecov token for uploading coverage'
+        required: false
 
 jobs:
   test:
-    name: Test on Python ${{ inputs.python-version }}
-    runs-on: ubuntu-latest
+    name: Test on ${{ inputs.os }} with Python ${{ inputs.python-version }}
+    runs-on: ${{ inputs.os }}
 
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
         with:
           fetch-depth: 0
+          fetch-tags: true
 
       - name: Set up Python ${{ inputs.python-version }}
         uses: actions/setup-python@v5
         with:
           python-version: ${{ inputs.python-version }}
+          cache: 'pip'
+          cache-dependency-path: 'pyproject.toml'
 
       - name: Install dependencies
         run: |
           pip install -e ".[dev]"
 
-      - name: Run pre-commit checks
-        run: pre-commit run --all-files
+      - name: Run tests with coverage
+        run: pytest --cov=cli_tool --cov-report=xml --cov-report=term-missing --cov-fail-under=40 -v
 
-      - name: Run tests
-        run: pytest --maxfail=1 --disable-warnings -v
+      - name: Upload coverage to Codecov
+        uses: codecov/codecov-action@v5
+        if: always()
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          files: ./coverage.xml
+          flags: ${{ inputs.os }}-python${{ inputs.python-version }}
+          fail_ci_if_error: false
+          verbose: true

.github/workflows/test.yml

@@ -11,16 +11,29 @@ on:
       - '.github/workflows/docs.yml'
 
 jobs:
+  lint:
+    name: Lint
+    uses: ./.github/workflows/lint-reusable.yml
+
   test:
-    name: Run Tests
+    name: Test
+    needs: lint
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, windows-latest, macos-latest]
+        python-version: ['3.12', '3.13']
     uses: ./.github/workflows/test-reusable.yml
     with:
-      python-version: '3.12'
+      os: ${{ matrix.os }}
+      python-version: ${{ matrix.python-version }}
+    secrets:
+      CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
 
   build-test:
     name: Test Binary Build (${{ matrix.platform }}-${{ matrix.arch }})
     runs-on: ${{ matrix.os }}
-    needs: test
+    needs: [lint, test]
     strategy:
       matrix:
         include:
@@ -31,7 +44,7 @@ jobs:
             build_script: bash scripts/build.sh --ci --release
 
           # macOS Intel
-          - os: macos-latest
+          - os: macos-15-intel
             platform: darwin
             arch: amd64
             build_script: bash scripts/build.sh --ci --release
@@ -65,6 +78,8 @@ jobs:
         uses: actions/setup-python@v5
         with:
           python-version: '3.12'
+          cache: 'pip'
+          cache-dependency-path: 'pyproject.toml'
 
       - name: Build binary
         env:

.gitignore

@@ -68,7 +68,7 @@ db.sqlite3
 db.sqlite3-journal
 
 # Flask stuff:
-instance/
+./instance/
 .webassets-cache
 
 # Scrapy stuff:
@@ -159,4 +159,6 @@ cython_debug/
 Thumbs.db
 
 config.*.json
-*.csv
+*.csv
+
+.kiro/specs