ci(deps): bump actions/upload-artifact from 6 to 7

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 6 to 7.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v6...v7)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

.dprint.jsonc

@@ -0,0 +1,42 @@
+{
+  "typescript": {
+  },
+  "json": {
+  },
+  "markdown": {
+  },
+  "toml": {
+  },
+  "dockerfile": {
+  },
+  "oxc": {
+  },
+  "ruff": {
+  },
+  "jupyter": {
+  },
+  "malva": {
+  },
+  "markup": {
+  },
+  "yaml": {
+  },
+  "excludes": [
+    "**/node_modules",
+    "**/*-lock.json",
+  ],
+  "plugins": [
+    "https://plugins.dprint.dev/typescript-0.95.13.wasm",
+    "https://plugins.dprint.dev/json-0.21.1.wasm",
+    "https://plugins.dprint.dev/markdown-0.20.0.wasm",
+    "https://plugins.dprint.dev/toml-0.7.0.wasm",
+    "https://plugins.dprint.dev/dockerfile-0.3.3.wasm",
+    "https://plugins.dprint.dev/oxc-0.1.0.wasm",
+    "https://plugins.dprint.dev/ruff-0.6.11.wasm",
+    "https://plugins.dprint.dev/jupyter-0.2.1.wasm",
+    "https://plugins.dprint.dev/g-plane/malva-v0.15.1.wasm",
+    "https://plugins.dprint.dev/g-plane/markup_fmt-v0.25.3.wasm",
+    "https://plugins.dprint.dev/g-plane/pretty_yaml-v0.5.1.wasm",
+    "https://plugins.dprint.dev/exec-0.6.0.json@a054130d458f124f9b5c91484833828950723a5af3f8ff2bd1523bd47b83b364",
+  ],
+}

.github/CONTRIBUTING.md

@@ -45,23 +45,23 @@ Enhancement suggestions are welcome! Please use the feature request template and
 
 2. **Clone and setup:**
 
-   ```bash
+```bash
-   git clone https://github.com/your-username/articulate-parser.git
+git clone https://github.com/your-username/articulate-parser.git
-   cd articulate-parser
+cd articulate-parser
-   go mod download
+go mod download
-   ```
+```
 
 3. **Run tests:**
 
-   ```bash
+```bash
-   go test -v ./...
+go test -v ./...
-   ```
+```
 
 4. **Build:**
 
-   ```bash
+```bash
-   go build main.go
+go build main.go
-   ```
+```
 
 ## Coding Standards
 

.github/ISSUE_TEMPLATE/bug_report.yml

@@ -1,7 +1,7 @@
 name: Bug Report
 description: Create a report to help us improve
-title: '[BUG] '
+title: "[BUG] "
-labels: ['bug', 'triage']
+labels: ["bug", "triage"]
 body:
   - type: markdown
     attributes:

.github/PULL_REQUEST_TEMPLATE.md

@@ -5,11 +5,13 @@
 ## Related Issue
 
 <!-- Link to the issue this PR addresses using the syntax: Fixes #issue_number -->
+
 Fixes #
 
 ## Type of Change
 
 <!-- Mark the appropriate option with an "x" -->
+
 - [ ] Bug fix (non-breaking change which fixes an issue)
 - [ ] New feature (non-breaking change which adds functionality)
 - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
@@ -21,6 +23,7 @@ Fixes #
 ## Checklist
 
 <!-- Mark the items you've completed with an "x" -->
+
 - [ ] My code follows the style guidelines of this project
 - [ ] I have performed a self-review of my code
 - [ ] I have added comments to complex logic
@@ -42,6 +45,7 @@ Fixes #
 ## Testing Instructions
 
 <!-- Provide steps to test the changes, if applicable -->
+
 1.
 2.
 3.

.github/dependabot.yml

@@ -1,86 +1,86 @@
 version: 2
 updates:
   # Check for updates to GitHub Actions
-  - package-ecosystem: 'github-actions'
+  - package-ecosystem: "github-actions"
-    directory: '/'
+    directory: "/"
     schedule:
-      interval: 'weekly'
+      interval: "weekly"
-      day: 'monday'
+      day: "monday"
-      time: '07:00'
+      time: "07:00"
-      timezone: 'Europe/Amsterdam'
+      timezone: "Europe/Amsterdam"
     open-pull-requests-limit: 2
     labels:
-      - 'dependencies'
+      - "dependencies"
-      - 'dependencies/github-actions'
+      - "dependencies/github-actions"
     commit-message:
-      prefix: 'ci'
+      prefix: "ci"
-      include: 'scope'
+      include: "scope"
 
   # Check for updates to Docker
-  - package-ecosystem: 'docker'
+  - package-ecosystem: "docker"
-    directory: '/'
+    directory: "/"
     schedule:
-      interval: 'weekly'
+      interval: "weekly"
-      day: 'monday'
+      day: "monday"
-      time: '07:00'
+      time: "07:00"
-      timezone: 'Europe/Amsterdam'
+      timezone: "Europe/Amsterdam"
     open-pull-requests-limit: 2
     labels:
-      - 'dependencies'
+      - "dependencies"
-      - 'dependencies/docker'
+      - "dependencies/docker"
     commit-message:
-      prefix: 'docker'
+      prefix: "docker"
-      include: 'scope'
+      include: "scope"
     groups:
       docker:
         patterns:
-          - '*'
+          - "*"
         update-types:
-          - 'minor'
+          - "minor"
-          - 'patch'
+          - "patch"
 
   # Check for updates to Docker Compose
-  - package-ecosystem: 'docker-compose'
+  - package-ecosystem: "docker-compose"
-    directory: '/'
+    directory: "/"
     schedule:
-      interval: 'weekly'
+      interval: "weekly"
-      day: 'monday'
+      day: "monday"
-      time: '07:00'
+      time: "07:00"
-      timezone: 'Europe/Amsterdam'
+      timezone: "Europe/Amsterdam"
     open-pull-requests-limit: 2
     labels:
-      - 'dependencies'
+      - "dependencies"
-      - 'dependencies/docker-compose'
+      - "dependencies/docker-compose"
     commit-message:
-      prefix: 'docker'
+      prefix: "docker"
-      include: 'scope'
+      include: "scope"
     groups:
       docker:
         patterns:
-          - '*'
+          - "*"
         update-types:
-          - 'minor'
+          - "minor"
-          - 'patch'
+          - "patch"
 
   # Check for updates to Go modules
-  - package-ecosystem: 'gomod'
+  - package-ecosystem: "gomod"
-    directory: '/'
+    directory: "/"
     schedule:
-      interval: 'weekly'
+      interval: "weekly"
-      day: 'monday'
+      day: "monday"
-      time: '07:00'
+      time: "07:00"
-      timezone: 'Europe/Amsterdam'
+      timezone: "Europe/Amsterdam"
     open-pull-requests-limit: 2
     labels:
-      - 'dependencies'
+      - "dependencies"
-      - 'dependencies/go'
+      - "dependencies/go"
     commit-message:
-      prefix: 'deps'
+      prefix: "deps"
-      include: 'scope'
+      include: "scope"
     groups:
       go-modules:
         patterns:
-          - '*'
+          - "*"
         update-types:
-          - 'minor'
+          - "minor"
-          - 'patch'
+          - "patch"

.github/workflows/autofix.yml

@@ -2,7 +2,7 @@ name: autofix.ci
 on:
   pull_request:
   push:
-    branches: [ "master" ]
+    branches: ["master"]
 permissions:
   contents: read
 
@@ -11,13 +11,13 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Install Task
         uses: go-task/setup-task@v1
 
       - uses: actions/setup-go@v6
-        with: { go-version-file: 'go.mod' }
+        with: { go-version-file: "go.mod" }
 
       - name: Setup go deps
         run: |
@@ -34,7 +34,7 @@ jobs:
         run: golangci-lint run --fix
 
       - name: Run golangci-lint format
-        run: golangci-lint format
+        run: golangci-lint fmt
 
       - name: Run go mod tidy
         run: go mod tidy

.github/workflows/ci.yml

@@ -2,7 +2,7 @@ name: CI
 
 on:
   push:
-    branches: ['master', 'develop']
+    branches: ["master", "develop"]
   pull_request:
 
 env:
@@ -21,12 +21,12 @@ jobs:
       contents: read
       pull-requests: read
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
       - uses: actions/setup-go@v6
         with:
           go-version: stable
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v8
+        uses: golangci/golangci-lint-action@v9
         with: { version: latest }
 
   test:
@@ -35,24 +35,24 @@ jobs:
     runs-on: ubuntu-latest
     permissions:
       contents: write
-    strategy:
-      matrix:
-        go:
-          - 1.21.x
-          - 1.22.x
-          - 1.23.x
-          - 1.24.x
-          - 1.25.x
 
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
-      - name: Set up Go ${{ matrix.go }}
+      - name: Set up Go
+        id: setup-go
         uses: actions/setup-go@v6
         with:
-          go-version: ${{ matrix.go }}
+          go-version-file: go.mod
           check-latest: true
 
+      # Expose the resolved Go version as an env var so it is never
+      # interpolated directly into a shell script (avoids script injection).
+      - name: Export Go version
+        env:
+          GO_VERSION: ${{ steps.setup-go.outputs.go-version }}
+        run: echo "GO_VERSION=$GO_VERSION" >> "$GITHUB_ENV"
+
       - name: Install Task
         uses: go-task/setup-task@v1
 
@@ -67,14 +67,18 @@ jobs:
 
       - name: Run tests with enhanced reporting
         id: test
+        env:
+          CGO_ENABLED: 1
         run: |
-          cat >> $GITHUB_STEP_SUMMARY << EOF
+          {
+            cat << EOF
           ## 🔧 Test Environment
-          - **Go Version:** ${{ matrix.go }}
+          - **Go Version:** $GO_VERSION
           - **OS:** ubuntu-latest
           - **Timestamp:** $(date -u)
 
           EOF
+          } >> "$GITHUB_STEP_SUMMARY"
 
           echo "Running tests with coverage..."
           task test:coverage 2>&1 | tee test-output.log
@@ -87,16 +91,17 @@ jobs:
           SKIPPED_TESTS=$(grep -c "--- SKIP:" test-output.log || echo "0")
 
           # Generate test summary
-          cat >> $GITHUB_STEP_SUMMARY << EOF
+          {
-          ## 🧪 Test Results (Go ${{ matrix.go }})
+            cat << EOF
+          ## 🧪 Test Results (Go $GO_VERSION)
 
           | Metric      | Value                                                         |
-          | ----------- | ----------------------------------------------------------- |
+          | ----------- | ------------------------------------------------------------- |
           | Total Tests | $TOTAL_TESTS                                                  |
           | Passed      | $PASSED_TESTS                                                 |
           | Failed      | $FAILED_TESTS                                                 |
           | Skipped     | $SKIPPED_TESTS                                                |
-          | Status      | $([ $TEST_STATUS -eq 0 ] && echo "PASSED" || echo "FAILED") |
+          | Status      | $([ "$TEST_STATUS" -eq 0 ] && echo "PASSED" || echo "FAILED") |
 
           ### 📦 Package Test Results
 
@@ -108,38 +113,39 @@ jobs:
             grep "^ok\|^FAIL" test-output.log | while read -r line; do
               if [[ $line == ok* ]]; then
                 pkg=$(echo "$line" | awk '{print $2}')
-              echo "| $pkg | ✅ PASS |" >> $GITHUB_STEP_SUMMARY
+                echo "| $pkg | ✅ PASS |"
               elif [[ $line == FAIL* ]]; then
                 pkg=$(echo "$line" | awk '{print $2}')
-              echo "| $pkg | ❌ FAIL |" >> $GITHUB_STEP_SUMMARY
+                echo "| $pkg | ❌ FAIL |"
               fi
             done
 
-          echo "" >> $GITHUB_STEP_SUMMARY
+            echo ""
 
             # Add detailed results if tests failed
-          if [ $TEST_STATUS -ne 0 ]; then
+            if [ "$TEST_STATUS" -ne 0 ]; then
-            cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+              cat << 'EOF'
           ### ❌ Failed Tests Details
           ```
           EOF
-            grep -A 10 "--- FAIL:" test-output.log | head -100 >> $GITHUB_STEP_SUMMARY
+              grep -A 10 -- "--- FAIL:" test-output.log | head -100
-            cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+              cat << 'EOF'
           ```
 
           EOF
             fi
+          } >> "$GITHUB_STEP_SUMMARY"
 
           # Set outputs for other steps
-          cat >> $GITHUB_OUTPUT << EOF
+          {
-          test-status=$TEST_STATUS
+            echo "test-status=$TEST_STATUS"
-          total-tests=$TOTAL_TESTS
+            echo "total-tests=$TOTAL_TESTS"
-          passed-tests=$PASSED_TESTS
+            echo "passed-tests=$PASSED_TESTS"
-          failed-tests=$FAILED_TESTS
+            echo "failed-tests=$FAILED_TESTS"
-          EOF
+          } >> "$GITHUB_OUTPUT"
 
           # Exit with the original test status
-          exit $TEST_STATUS
+          exit "$TEST_STATUS"
 
       - name: Generate coverage report
         if: always()
@@ -147,8 +153,9 @@ jobs:
           if [ -f coverage/coverage.out ]; then
             COVERAGE=$(go tool cover -func=coverage/coverage.out | grep total | awk '{print $3}')
 
-            cat >> $GITHUB_STEP_SUMMARY << EOF
+            {
-          ## 📊 Code Coverage (Go ${{ matrix.go }})
+              cat << EOF
+          ## 📊 Code Coverage (Go $GO_VERSION)
 
           **Total Coverage: $COVERAGE**
 
@@ -187,17 +194,18 @@ jobs:
                   avg = packages[pkg] / counts[pkg]
                   printf "| %s | %.1f%% |\n", pkg, avg
                 }
-            }' "$temp_coverage" | sort >> $GITHUB_STEP_SUMMARY
+              }' "$temp_coverage" | sort
 
               rm -f "$temp_coverage"
 
-            cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+              cat << 'EOF'
 
           </details>
 
           EOF
+            } >> "$GITHUB_STEP_SUMMARY"
           else
-            cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+            cat >> "$GITHUB_STEP_SUMMARY" << 'EOF'
           ## ⚠️ Coverage Report
           No coverage file generated
 
@@ -206,9 +214,9 @@ jobs:
 
       - name: Upload test artifacts
         if: failure()
-        uses: actions/upload-artifact@v5
+        uses: actions/upload-artifact@v7
         with:
-          name: test-results-go-${{ matrix.go }}
+          name: test-results-go-${{ env.GO_VERSION }}
           path: |
             test-output.log
             coverage/
@@ -216,9 +224,9 @@ jobs:
 
       - name: Run linters
         run: |
-          # Initialize summary
+          {
-          cat >> $GITHUB_STEP_SUMMARY << EOF
+            cat << EOF
-          ## 🔍 Static Analysis (Go ${{ matrix.go }})
+          ## 🔍 Static Analysis (Go $GO_VERSION)
 
           EOF
 
@@ -226,44 +234,46 @@ jobs:
             VET_OUTPUT=$(task lint:vet 2>&1 || echo "")
             VET_STATUS=$?
 
-          if [ $VET_STATUS -eq 0 ]; then
+            if [ "$VET_STATUS" -eq 0 ]; then
-            echo "✅ **go vet:** No issues found" >> $GITHUB_STEP_SUMMARY
+              echo "✅ **go vet:** No issues found"
             else
-            cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+              cat << 'EOF'
           ❌ **go vet:** Issues found
 
           ```
           EOF
-            echo "$VET_OUTPUT" >> $GITHUB_STEP_SUMMARY
+              echo "$VET_OUTPUT"
-            echo '```' >> $GITHUB_STEP_SUMMARY
+              echo '```'
             fi
-          echo "" >> $GITHUB_STEP_SUMMARY
+            echo ""
 
             # Run go fmt check
             FMT_OUTPUT=$(task lint:fmt 2>&1 || echo "")
             FMT_STATUS=$?
 
-          if [ $FMT_STATUS -eq 0 ]; then
+            if [ "$FMT_STATUS" -eq 0 ]; then
-            echo "✅ **go fmt:** All files properly formatted" >> $GITHUB_STEP_SUMMARY
+              echo "✅ **go fmt:** All files properly formatted"
             else
-            cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+              cat << 'EOF'
           ❌ **go fmt:** Files need formatting
 
           ```
           EOF
-            echo "$FMT_OUTPUT" >> $GITHUB_STEP_SUMMARY
+              echo "$FMT_OUTPUT"
-            echo '```' >> $GITHUB_STEP_SUMMARY
+              echo '```'
             fi
+          } >> "$GITHUB_STEP_SUMMARY"
 
           # Exit with error if any linter failed
-          [ $VET_STATUS -eq 0 ] && [ $FMT_STATUS -eq 0 ] || exit 1
+          [ "$VET_STATUS" -eq 0 ] && [ "$FMT_STATUS" -eq 0 ] || exit 1
 
       - name: Job Summary
         if: always()
         run: |
-          cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+          {
-          ## 📋 Job Summary (Go ${{ matrix.go }})
+            echo "## 📋 Job Summary (Go $GO_VERSION)"
-
+            echo ""
+            cat << 'EOF'
           | Step            | Status                                                          |
           | --------------- | --------------------------------------------------------------- |
           | Dependencies    | Success                                                         |
@@ -273,12 +283,13 @@ jobs:
           | Static Analysis | ${{ job.status == 'success' && 'Clean' || 'Issues' }}           |
           | Code Formatting | ${{ job.status == 'success' && 'Clean' || 'Issues' }}           |
           EOF
+          } >> "$GITHUB_STEP_SUMMARY"
 
       - name: Upload coverage reports to Codecov
         uses: codecov/codecov-action@v5
         with:
           files: ./coverage/coverage.out
-          flags: Go ${{ matrix.go }}
+          flags: Go ${{ env.GO_VERSION }}
           slug: kjanat/articulate-parser
           token: ${{ secrets.CODECOV_TOKEN }}
 
@@ -286,7 +297,7 @@ jobs:
         if: ${{ !cancelled() }}
         uses: codecov/test-results-action@v1
         with:
-          flags: Go ${{ matrix.go }}
+          flags: Go ${{ env.GO_VERSION }}
           token: ${{ secrets.CODECOV_TOKEN }}
 
   docker-test:
@@ -297,7 +308,7 @@ jobs:
       contents: read
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Set up Go
         uses: actions/setup-go@v6
@@ -316,7 +327,8 @@ jobs:
 
       - name: Test Docker image using Task
         run: |
-          cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+          {
+            cat << 'EOF'
           ## 🧪 Docker Image Tests
 
           EOF
@@ -324,29 +336,31 @@ jobs:
             # Run Task docker test
             task docker:test
 
-          echo "**Testing help command:**" >> $GITHUB_STEP_SUMMARY
+            echo "**Testing help command:**"
-          echo '```terminaloutput' >> $GITHUB_STEP_SUMMARY
+            echo '```terminaloutput'
-          docker run --rm articulate-parser:latest --help >> $GITHUB_STEP_SUMMARY
+            docker run --rm articulate-parser:latest --help
-          echo '```' >> $GITHUB_STEP_SUMMARY
+            echo '```'
-          echo "" >> $GITHUB_STEP_SUMMARY
+            echo ""
 
             # Test image size
             IMAGE_SIZE=$(docker image inspect articulate-parser:latest --format='{{.Size}}' | numfmt --to=iec-i --suffix=B)
-          echo "**Image size:** $IMAGE_SIZE" >> $GITHUB_STEP_SUMMARY
+            echo "**Image size:** $IMAGE_SIZE"
-          echo "" >> $GITHUB_STEP_SUMMARY
+            echo ""
+          } >> "$GITHUB_STEP_SUMMARY"
 
   dependency-review:
     name: Dependency Review
     runs-on: ubuntu-latest
     permissions:
       contents: read
+      pull-requests: write
     if: github.event_name == 'pull_request'
     steps:
-      - name: 'Checkout Repository'
+      - name: "Checkout Repository"
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
-      - name: 'Dependency Review'
+      - name: "Dependency Review"
-        uses: actions/dependency-review-action@v4
+        uses: actions/dependency-review-action@v5
         with:
           fail-on-severity: moderate
           comment-summary-in-pr: always
@@ -357,30 +371,30 @@ jobs:
     permissions:
       contents: read
       packages: write
-    needs: [test, docker-test, dependency-review]
+    needs: [test, docker-test]
     if: |
       github.event_name == 'push' && (github.ref == 'refs/heads/master' ||
       github.ref == 'refs/heads/develop' ||
       startsWith(github.ref, 'refs/heads/feature/docker'))
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Login to Docker Hub
-        uses: docker/login-action@v3
+        uses: docker/login-action@v4
         with:
           username: ${{ vars.DOCKERHUB_USERNAME }}
           password: ${{ secrets.DOCKERHUB_TOKEN }}
 
       - name: Log in to GitHub Container Registry
-        uses: docker/login-action@v3
+        uses: docker/login-action@v4
         with:
           registry: ${{ env.REGISTRY }}
           username: ${{ github.actor }}
           password: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Set up QEMU
-        uses: docker/setup-qemu-action@v3
+        uses: docker/setup-qemu-action@v4
 
       - name: Set up Docker Buildx
         uses: docker/setup-buildx-action@v3
@@ -436,7 +450,7 @@ jobs:
 
       - name: Generate Docker summary
         run: |
-          cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+          cat >> "$GITHUB_STEP_SUMMARY" << 'EOF'
           ## 🐳 Docker Build Summary
 
           **Image:** `ghcr.io/${{ github.repository }}`

.github/workflows/codeql.yml

@@ -17,7 +17,7 @@ name: "CodeQL"
 on:
   workflow_call:
   schedule:
-    - cron: '44 16 * * 6'
+    - cron: "44 16 * * 6"
   # push:
   #   branches: [ "master" ]
   # pull_request:
@@ -61,7 +61,7 @@ jobs:
           # your codebase is analyzed, see https://docs.github.com/en/code-security/code-scanning/creating-an-advanced-setup-for-code-scanning/codeql-code-scanning-for-compiled-languages
     steps:
       - name: Checkout repository
-      uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       # Add any setup steps before running the `github/codeql-action/init` action.
       # This includes steps like installing compilers or runtimes (`actions/setup-node`

.github/workflows/dependency-review.yml

@@ -16,11 +16,11 @@ jobs:
   dependency-review:
     runs-on: ubuntu-latest
     steps:
-      - name: 'Checkout Repository'
+      - name: "Checkout Repository"
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
-      - name: 'Dependency Review'
+      - name: "Dependency Review"
-        uses: actions/dependency-review-action@v4
+        uses: actions/dependency-review-action@v5
         with:
           fail-on-severity: moderate
           comment-summary-in-pr: always

.github/workflows/release.yml

@@ -20,7 +20,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         with:
           fetch-depth: 0
 
@@ -62,7 +62,7 @@ jobs:
           done
 
       - name: Create Release
-        uses: softprops/action-gh-release@v2
+        uses: softprops/action-gh-release@v3
         with:
           files: |
             build/articulate-parser-linux-amd64
@@ -82,29 +82,29 @@ jobs:
   docker:
     name: Docker Build & Push
     runs-on: ubuntu-latest
-    needs: ['release']
+    needs: ["release"]
     permissions:
       contents: read
       packages: write
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
 
       - name: Login to Docker Hub
-        uses: docker/login-action@v3
+        uses: docker/login-action@v4
         with:
           username: ${{ vars.DOCKERHUB_USERNAME }}
           password: ${{ secrets.DOCKERHUB_TOKEN }}
 
       - name: Log in to GitHub Container Registry
-        uses: docker/login-action@v3
+        uses: docker/login-action@v4
         with:
           registry: ${{ env.REGISTRY }}
           username: ${{ github.actor }}
           password: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Set up QEMU
-        uses: docker/setup-qemu-action@v3
+        uses: docker/setup-qemu-action@v4
 
       - name: Set up Docker Buildx
         uses: docker/setup-buildx-action@v3

.golangci.yml

@@ -8,11 +8,6 @@ run:
   # Timeout for total work
   timeout: 5m
 
-  # Skip directories (not allowed in config v2, will use issues exclude instead)
-
-  # Go version
-  go: "1.24"
-
   # Include test files
   tests: true
 
@@ -349,7 +344,15 @@ linters:
 
     # stylecheck settings
     staticcheck:
-      checks: ["all", "-ST1000", "-ST1003", "-ST1016", "-ST1020", "-ST1021", "-ST1022"]
+      checks: [
+        "all",
+        "-ST1000",
+        "-ST1003",
+        "-ST1016",
+        "-ST1020",
+        "-ST1021",
+        "-ST1022",
+      ]
 
     # maintidx settings
     maintidx:

.pre-commit-config.yaml

@@ -0,0 +1,75 @@
+# See https://pre-commit.com for more information
+# See https://pre-commit.com/hooks.html for more hooks
+repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v6.0.0
+    hooks:
+      # File quality
+      - id: trailing-whitespace
+        exclude: '^\.github/ISSUE_TEMPLATE/.*\.yml$'
+      - id: end-of-file-fixer
+      - id: mixed-line-ending
+        args: ["--fix=lf"]
+
+      # File validation
+      - id: check-yaml
+      - id: check-json
+      - id: check-toml
+
+      # Security
+      - id: detect-private-key
+
+      # Git safety
+      - id: check-merge-conflict
+      - id: check-case-conflict
+      - id: no-commit-to-branch
+        args: ["--branch=master", "--branch=main"]
+
+      # File structure
+      - id: check-added-large-files
+      - id: check-symlinks
+      - id: check-executables-have-shebangs
+  - repo: local
+    hooks:
+      - id: actionlint
+        name: Lint GitHub Actions workflow files
+        description: Runs actionlint to lint GitHub Actions workflow files
+        language: golang
+        types: ["yaml"]
+        files: ^\.github/workflows/
+        entry: actionlint
+        minimum_pre_commit_version: 3.0.0
+  - repo: https://github.com/golangci/golangci-lint
+    rev: v2.7.2
+    hooks:
+      - id: golangci-lint
+        name: golangci-lint
+        description: Fast linters runner for Go. Note that only modified files are linted, so linters like 'unused' that need to scan all files won't work as expected.
+        entry: golangci-lint run --new-from-rev HEAD --fix
+        types: [go]
+        language: golang
+        require_serial: true
+        pass_filenames: false
+      # - id: golangci-lint-full
+      #   name: golangci-lint-full
+      #   description: Fast linters runner for Go. Runs on all files in the module. Use this hook if you use pre-commit in CI.
+      #   entry: golangci-lint run --fix
+      #   types: [go]
+      #   language: golang
+      #   require_serial: true
+      #   pass_filenames: false
+      - id: golangci-lint-fmt
+        name: golangci-lint-fmt
+        description: Fast linters runner for Go. Formats all files in the repo.
+        entry: golangci-lint fmt
+        types: [go]
+        language: golang
+        require_serial: true
+        pass_filenames: false
+      - id: golangci-lint-config-verify
+        name: golangci-lint-config-verify
+        description: Verifies the configuration file
+        entry: golangci-lint config verify
+        files: '\.golangci\.(?:yml|yaml|toml|json)'
+        language: golang
+        pass_filenames: false

AGENTS.md

@@ -1,56 +1,183 @@
 # Agent Guidelines for articulate-parser
 
+A Go CLI tool that parses Articulate Rise courses from URLs or local JSON files and exports them to Markdown, HTML, or DOCX formats.
+
+## Repository Info
+
+- **GitHub**: https://github.com/kjanat/articulate-parser
+- **Default branch**: `master` (not `main`)
+
 ## Build/Test Commands
-- **Build**: `task build` or `go build -o bin/articulate-parser main.go`
+
-- **Run tests**: `task test` or `go test -race -timeout 5m ./...`
+### Primary Commands (using Taskfile)
-- **Run single test**: `go test -v -race -run ^TestName$ ./path/to/package`
+
-- **Test with coverage**: 
+```bash
-  - `task test:coverage` or 
+task build              # Build binary to bin/articulate-parser
-  - `go test -race -coverprofile=coverage/coverage.out -covermode=atomic ./...`
+task test               # Run all tests with race detection
-- **Lint**: `task lint` (runs vet, fmt check, staticcheck, golangci-lint)
+task lint               # Run all linters (vet, fmt, staticcheck, golangci-lint)
-- **Format**: `task fmt` or `gofmt -s -w .`
+task fmt                # Format all Go files
-- **CI checks**: `task ci` (deps, lint, test with coverage, build)
+task ci                 # Full CI pipeline: deps, lint, test with coverage, build
+task qa                 # Quick QA: fmt + lint + test
+```
+
+### Direct Go Commands
+
+```bash
+# Build
+go build -o bin/articulate-parser main.go
+
+# Run all tests
+go test -race -timeout 5m ./...
+
+# Run single test by name
+go test -v -race -run ^TestMarkdownExporter_Export$ ./internal/exporters
+
+# Run tests in specific package
+go test -v -race ./internal/services
+
+# Run tests matching pattern
+go test -v -race -run "TestParser" ./...
+
+# Test with coverage
+go test -race -coverprofile=coverage/coverage.out -covermode=atomic ./...
+go tool cover -html=coverage/coverage.out -o coverage/coverage.html
+
+# Benchmarks
+go test -bench=. -benchmem ./...
+go test -bench=BenchmarkMarkdownExporter ./internal/exporters
+```
+
+### Security & Auditing
+
+```bash
+task security:check     # Run gosec security scanner
+task security:audit     # Run govulncheck for vulnerabilities
+```
 
 ## Code Style Guidelines
 
 ### Imports
+
 - Use `goimports` with local prefix: `github.com/kjanat/articulate-parser`
-- Order: stdlib, external, internal packages
+- Order: stdlib, blank line, external packages, blank line, internal packages
-- Group related imports together
+
+```go
+import (
+    "context"
+    "fmt"
+
+    "github.com/fumiama/go-docx"
+
+    "github.com/kjanat/articulate-parser/internal/interfaces"
+)
+```
 
 ### Formatting
+
 - Use `gofmt -s` (simplify) and `gofumpt` with extra rules
 - Function length: max 100 lines, 50 statements
-- Cyclomatic complexity: max 15
+- Cyclomatic complexity: max 15; Cognitive complexity: max 20
-- Cognitive complexity: max 20
 
 ### Types & Naming
+
 - Use interface-based design (see `internal/interfaces/`)
-- Export types/functions with clear godoc comments ending with period
+- Exported types/functions require godoc comments ending with period
 - Use descriptive names: `ArticulateParser`, `MarkdownExporter`
 - Receiver names: short (1-2 chars), consistent per type
 
 ### Error Handling
+
 - Always wrap errors with context: `fmt.Errorf("operation failed: %w", err)`
 - Use `%w` verb for error wrapping to preserve error chain
 - Check all error returns (enforced by `errcheck`)
 - Document error handling rationale in defer blocks when ignoring close errors
 
+```go
+// Good: Error wrapping with context
+if err := json.Unmarshal(body, &course); err != nil {
+    return nil, fmt.Errorf("failed to unmarshal JSON: %w", err)
+}
+
+// Good: Documented defer with error handling
+defer func() {
+    if err := resp.Body.Close(); err != nil {
+        p.Logger.Warn("failed to close response body", "error", err)
+    }
+}()
+```
+
 ### Comments
+
 - All exported types/functions require godoc comments
 - End sentences with periods (`godot` linter enforced)
 - Mark known issues with TODO/FIXME/HACK/BUG/XXX
 
 ### Security
-- Use `#nosec` with justification for deliberate security exceptions (G304 for CLI file paths, G306 for export file permissions)
+
-- Run `gosec` and `govulncheck` for security audits
+- Use `#nosec` with justification for deliberate security exceptions
+- G304: File paths from CLI args; G306: Export file permissions
+
+```go
+// #nosec G304 - File path provided by user via CLI argument
+data, err := os.ReadFile(filePath)
+```
 
 ### Testing
-- Enable race detection: `-race` flag
+
+- Enable race detection: `-race` flag always
 - Use table-driven tests where applicable
 - Mark test helpers with `t.Helper()`
+- Use `t.TempDir()` for temporary files
 - Benchmarks in `*_bench_test.go`, examples in `*_example_test.go`
+- Test naming: `Test<Type>_<Method>` or `Test<Function>`
+
+```go
+func TestMarkdownExporter_ProcessItemToMarkdown_AllTypes(t *testing.T) {
+    tests := []struct {
+        name, itemType, expectedText string
+    }{
+        {"text item", "text", ""},
+        {"divider item", "divider", "---"},
+    }
+    for _, tt := range tests {
+        t.Run(tt.name, func(t *testing.T) {
+            // test implementation
+        })
+    }
+}
+```
 
 ### Dependencies
-- Minimal external dependencies (currently: go-docx, golang.org/x/net, golang.org/x/text)
+
+- Minimal external dependencies (go-docx, golang.org/x/net, golang.org/x/text)
 - Run `task deps:tidy` after adding/removing dependencies
+- CGO disabled by default (`CGO_ENABLED=0`)
+
+## Project Structure
+
+```
+articulate-parser/
+  internal/
+    config/              # Configuration loading
+    exporters/           # Export implementations (markdown, html, docx)
+    interfaces/          # Core interfaces (Exporter, CourseParser, Logger)
+    models/              # Data models (Course, Lesson, Item, Media)
+    services/            # Core services (parser, html cleaner, app, logger)
+    version/             # Version information
+  main.go                # Application entry point
+```
+
+## Common Patterns
+
+### Creating a new exporter
+
+1. Implement `interfaces.Exporter` interface
+2. Add factory method to `internal/exporters/factory.go`
+3. Register format in `NewFactory()`
+4. Add tests following existing patterns
+
+### Adding configuration options
+
+1. Add field to `Config` struct in `internal/config/config.go`
+2. Load from environment variable with sensible default
+3. Document in config struct comments

Dockerfile

@@ -1,5 +1,5 @@
 # Build stage
-FROM golang:1.25-alpine AS builder
+FROM golang:1.26-alpine AS builder
 
 # Install git and ca-certificates (needed for fetching dependencies and HTTPS)
 RUN apk add --no-cache git ca-certificates tzdata file

Dockerfile.dev

@@ -2,7 +2,7 @@
 # Uses Alpine instead of scratch for debugging
 
 # Build stage - same as production
-FROM golang:1.25-alpine AS builder
+FROM golang:1.26-alpine AS builder
 
 # Install git and ca-certificates (needed for fetching dependencies and HTTPS)
 RUN apk add --no-cache git ca-certificates tzdata file

README.md

@@ -78,12 +78,12 @@ flowchart TD
 
 The system follows **Clean Architecture** principles with clear separation of concerns:
 
--   **🎯 Entry Point**: Command-line interface handles user input and coordinates operations
+- **Entry Point**: Command-line interface handles user input and coordinates operations
--   **🏗️ Application Layer**: Core business logic with dependency injection
+- **Application Layer**: Core business logic with dependency injection
--   **📋 Interface Layer**: Contracts defining behavior without implementation details  
+- **Interface Layer**: Contracts defining behavior without implementation details
--   **🔧 Service Layer**: Concrete implementations of parsing and utility services
+- **Service Layer**: Concrete implementations of parsing and utility services
--   **📤 Export Layer**: Factory pattern for format-specific exporters
+- **Export Layer**: Factory pattern for format-specific exporters
--   **📊 Data Layer**: Domain models representing course structure
+- **Data Layer**: Domain models representing course structure
 
 ## Features
 
@@ -206,7 +206,7 @@ Then run:
 
 The application is available as a Docker image from GitHub Container Registry.
 
-### 🐳 Docker Image Information
+### Docker Image Information
 
 - **Registry**: `ghcr.io/kjanat/articulate-parser`
 - **Platforms**: linux/amd64, linux/arm64
@@ -226,7 +226,7 @@ docker run --rm ghcr.io/kjanat/articulate-parser:latest --help
 ### Available Tags
 
 | Tag                   | Description                                 | Use Case               |
-|-----|-------------|----------|
+| --------------------- | ------------------------------------------- | ---------------------- |
 | `latest`              | Latest stable release from master branch    | Production use         |
 | `edge`                | Latest development build from master branch | Testing new features   |
 | `v1.x.x`              | Specific version releases                   | Production pinning     |
@@ -314,7 +314,7 @@ docker build --build-arg VERSION=local --build-arg BUILD_TIME=$(date -u +%Y-%m-%
 The Docker image supports the following build-time arguments:
 
 | Argument     | Description                           | Default        |
-|----------|-------------|---------|
+| ------------ | ------------------------------------- | -------------- |
 | `VERSION`    | Version string embedded in the binary | `dev`          |
 | `BUILD_TIME` | Build timestamp                       | Current time   |
 | `GIT_COMMIT` | Git commit hash                       | Current commit |
@@ -460,7 +460,9 @@ This is a utility tool for educational content conversion. Please ensure you hav
 [Go report]: https://goreportcard.com/report/github.com/kjanat/articulate-parser
 [gomod]: go.mod
 [Issues]: https://github.com/kjanat/articulate-parser/issues
+
 <!-- [Latest release]: https://github.com/kjanat/articulate-parser/releases/latest -->
+
 [MIT License]: LICENSE
 [Package documentation]: https://godoc.org/github.com/kjanat/articulate-parser
 [Tags]: https://github.com/kjanat/articulate-parser/tags

Taskfile.yml

@@ -1,7 +1,7 @@
 # yaml-language-server: $schema=https://taskfile.dev/schema.json
 # Articulate Parser - Task Automation
 # https://taskfile.dev
-version: '3'
+version: "3"
 
 # Global output settings
 output: prefixed
@@ -47,11 +47,11 @@ vars:
 
 # Environment variables
 env:
-  CGO_ENABLED: '{{.CGO_ENABLED}}'
+  CGO_ENABLED: "{{.CGO_ENABLED}}"
   GO111MODULE: on
 
 # Load .env files if present
-dotenv: ['.env', '.env.local']
+dotenv: [".env", ".env.local"]
 
 # Task definitions
 tasks:
@@ -69,12 +69,12 @@ tasks:
     interactive: true
     watch: true
     sources:
-      - '**/*.go'
+      - "**/*.go"
       - go.mod
       - go.sum
     cmds:
       - task: build
-      - '{{.OUTPUT_DIR}}/{{.APP_NAME}}{{.EXE_EXT}} --help'
+      - "{{.OUTPUT_DIR}}/{{.APP_NAME}}{{.EXE_EXT}} --help"
 
   # Build tasks
   build:
@@ -82,14 +82,14 @@ tasks:
     aliases: [b]
     deps: [clean-bin]
     sources:
-      - '**/*.go'
+      - "**/*.go"
       - go.mod
       - go.sum
     generates:
-      - '{{.OUTPUT_DIR}}/{{.APP_NAME}}{{.EXE_EXT}}'
+      - "{{.OUTPUT_DIR}}/{{.APP_NAME}}{{.EXE_EXT}}"
     cmds:
       - task: mkdir
-        vars: { DIR: '{{.OUTPUT_DIR}}' }
+        vars: { DIR: "{{.OUTPUT_DIR}}" }
       - go build {{.GO_FLAGS}} -ldflags="{{.LDFLAGS}}" -o {{.OUTPUT_DIR}}/{{.APP_NAME}}{{.EXE_EXT}} {{.MAIN_FILE}}
     method: checksum
 
@@ -99,25 +99,25 @@ tasks:
     deps: [clean-bin]
     cmds:
       - task: mkdir
-        vars: { DIR: '{{.OUTPUT_DIR}}' }
+        vars: { DIR: "{{.OUTPUT_DIR}}" }
       - for:
           matrix:
             GOOS: [linux, darwin, windows]
             GOARCH: [amd64, arm64]
         task: build:platform
         vars:
-          TARGET_GOOS: '{{.ITEM.GOOS}}'
+          TARGET_GOOS: "{{.ITEM.GOOS}}"
-          TARGET_GOARCH: '{{.ITEM.GOARCH}}'
+          TARGET_GOARCH: "{{.ITEM.GOARCH}}"
       - echo "Built binaries for all platforms in {{.OUTPUT_DIR}}/"
 
   build:platform:
     internal: true
     vars:
       TARGET_EXT: '{{if eq .TARGET_GOOS "windows"}}.exe{{end}}'
-      OUTPUT_FILE: '{{.OUTPUT_DIR}}/{{.APP_NAME}}-{{.TARGET_GOOS}}-{{.TARGET_GOARCH}}{{.TARGET_EXT}}'
+      OUTPUT_FILE: "{{.OUTPUT_DIR}}/{{.APP_NAME}}-{{.TARGET_GOOS}}-{{.TARGET_GOARCH}}{{.TARGET_EXT}}"
     env:
-      GOOS: '{{.TARGET_GOOS}}'
+      GOOS: "{{.TARGET_GOOS}}"
-      GOARCH: '{{.TARGET_GOARCH}}'
+      GOARCH: "{{.TARGET_GOARCH}}"
     cmds:
       - echo "Building {{.OUTPUT_FILE}}..."
       - go build {{.GO_FLAGS}} -ldflags="{{.LDFLAGS}}" -o "{{.OUTPUT_FILE}}" {{.MAIN_FILE}}
@@ -134,6 +134,8 @@ tasks:
   test:
     desc: Run all tests
     aliases: [t]
+    env:
+      CGO_ENABLED: 1
     cmds:
       - go test {{.GO_FLAGS}} -race -timeout {{.TEST_TIMEOUT}} ./...
 
@@ -141,9 +143,11 @@ tasks:
     desc: Run tests with coverage report
     aliases: [cover, cov]
     deps: [clean-coverage]
+    env:
+      CGO_ENABLED: 1
     cmds:
       - task: mkdir
-        vars: { DIR: '{{.COVERAGE_DIR}}' }
+        vars: { DIR: "{{.COVERAGE_DIR}}" }
       - go test {{.GO_FLAGS}} -race -coverprofile={{.COVERAGE_DIR}}/coverage.out -covermode=atomic -timeout {{.TEST_TIMEOUT}} ./...
       - go tool cover -html={{.COVERAGE_DIR}}/coverage.out -o {{.COVERAGE_DIR}}/coverage.html
       - go tool cover -func={{.COVERAGE_DIR}}/coverage.out
@@ -152,6 +156,8 @@ tasks:
   test:verbose:
     desc: Run tests with verbose output
     aliases: [tv]
+    env:
+      CGO_ENABLED: 1
     cmds:
       - go test -v -race -timeout {{.TEST_TIMEOUT}} ./...
 
@@ -160,7 +166,7 @@ tasks:
     aliases: [tw]
     watch: true
     sources:
-      - '**/*.go'
+      - "**/*.go"
     cmds:
       - task: test
 
@@ -172,6 +178,8 @@ tasks:
 
   test:integration:
     desc: Run integration tests
+    env:
+      CGO_ENABLED: 1
     status:
       - '{{if eq OS "windows"}}if not exist "main_test.go" exit 1{{else}}test ! -f "main_test.go"{{end}}'
     cmds:
@@ -352,14 +360,14 @@ tasks:
     internal: true
     cmds:
       - task: rmdir
-        vars: { DIR: '{{.OUTPUT_DIR}}' }
+        vars: { DIR: "{{.OUTPUT_DIR}}" }
 
   clean-coverage:
     desc: Remove coverage files
     internal: true
     cmds:
       - task: rmdir
-        vars: { DIR: '{{.COVERAGE_DIR}}' }
+        vars: { DIR: "{{.COVERAGE_DIR}}" }
 
   clean-cache:
     desc: Clean Go build and test cache
@@ -410,10 +418,10 @@ tasks:
     requires:
       vars: [VERSION]
     preconditions:
-      - sh: 'git diff --exit-code'
+      - sh: "git diff --exit-code"
-        msg: 'Working directory is not clean'
+        msg: "Working directory is not clean"
-      - sh: 'git diff --cached --exit-code'
+      - sh: "git diff --cached --exit-code"
-        msg: 'Staging area is not clean'
+        msg: "Staging area is not clean"
     cmds:
       - git tag -a v{{.VERSION}} -m "Release v{{.VERSION}}"
       - echo "Tagged v{{.VERSION}}"
@@ -498,11 +506,11 @@ tasks:
       - '{{if eq OS "windows"}}if not exist "articulate-sample.json" exit 1{{else}}test ! -f "articulate-sample.json"{{end}}'
     deps: [build]
     cmds:
-      - '{{.OUTPUT_DIR}}/{{.APP_NAME}}{{.EXE_EXT}} articulate-sample.json md output-demo.md'
+      - "{{.OUTPUT_DIR}}/{{.APP_NAME}}{{.EXE_EXT}} articulate-sample.json md output-demo.md"
       - echo "Demo Markdown created{{:}} output-demo.md"
       - defer:
           task: rmfile
-          vars: { FILE: 'output-demo.md' }
+          vars: { FILE: "output-demo.md" }
 
   demo:html:
     desc: Demo - Convert sample to HTML
@@ -510,11 +518,11 @@ tasks:
       - '{{if eq OS "windows"}}if not exist "articulate-sample.json" exit 1{{else}}test ! -f "articulate-sample.json"{{end}}'
     deps: [build]
     cmds:
-      - '{{.OUTPUT_DIR}}/{{.APP_NAME}}{{.EXE_EXT}} articulate-sample.json html output-demo.html'
+      - "{{.OUTPUT_DIR}}/{{.APP_NAME}}{{.EXE_EXT}} articulate-sample.json html output-demo.html"
       - echo "Demo HTML created{{:}} output-demo.html"
       - defer:
           task: rmfile
-          vars: { FILE: 'output-demo.html' }
+          vars: { FILE: "output-demo.html" }
 
   demo:docx:
     desc: Demo - Convert sample to DOCX
@@ -522,11 +530,11 @@ tasks:
       - '{{if eq OS "windows"}}if not exist "articulate-sample.json" exit 1{{else}}test ! -f "articulate-sample.json"{{end}}'
     deps: [build]
     cmds:
-      - '{{.OUTPUT_DIR}}/{{.APP_NAME}}{{.EXE_EXT}} articulate-sample.json docx output-demo.docx'
+      - "{{.OUTPUT_DIR}}/{{.APP_NAME}}{{.EXE_EXT}} articulate-sample.json docx output-demo.docx"
       - echo "Demo DOCX created{{:}} output-demo.docx"
       - defer:
           task: rmfile
-          vars: { FILE: 'output-demo.docx' }
+          vars: { FILE: "output-demo.docx" }
 
   # Performance profiling
   profile:cpu:
@@ -536,7 +544,7 @@ tasks:
       - go tool pprof -http=:8080 cpu.prof
       - defer:
           task: rmfile
-          vars: { FILE: 'cpu.prof' }
+          vars: { FILE: "cpu.prof" }
 
   profile:mem:
     desc: Run memory profiling
@@ -545,14 +553,14 @@ tasks:
       - go tool pprof -http=:8080 mem.prof
       - defer:
           task: rmfile
-          vars: { FILE: 'mem.prof' }
+          vars: { FILE: "mem.prof" }
 
   # Git hooks
   hooks:install:
     desc: Install git hooks
     cmds:
       - task: mkdir
-        vars: { DIR: '.git/hooks' }
+        vars: { DIR: ".git/hooks" }
       - '{{if eq OS "windows"}}echo "#!/bin/sh" > .git/hooks/pre-commit && echo "task lint:fmt" >> .git/hooks/pre-commit{{else}}cat > .git/hooks/pre-commit << ''EOF''{{printf "\n"}}#!/bin/sh{{printf "\n"}}task lint:fmt{{printf "\n"}}EOF{{printf "\n"}}chmod +x .git/hooks/pre-commit{{end}}'
       - echo "Git hooks installed"
 

go.mod

@@ -1,14 +1,14 @@
 module github.com/kjanat/articulate-parser
 
-go 1.24.0
+go 1.26.0
 
 require (
 	github.com/fumiama/go-docx v0.0.0-20250506085032-0c30fd09304b
-	golang.org/x/net v0.46.0
+	golang.org/x/net v0.56.0
-	golang.org/x/text v0.30.0
+	golang.org/x/text v0.38.0
 )
 
 require (
 	github.com/fumiama/imgsz v0.0.4 // indirect
-	golang.org/x/image v0.32.0 // indirect
+	golang.org/x/image v0.42.0 // indirect
 )

go.sum

@@ -2,9 +2,9 @@ github.com/fumiama/go-docx v0.0.0-20250506085032-0c30fd09304b h1:/mxSugRc4SgN7Xg
 github.com/fumiama/go-docx v0.0.0-20250506085032-0c30fd09304b/go.mod h1:ssRF0IaB1hCcKIObp3FkZOsjTcAHpgii70JelNb4H8M=
 github.com/fumiama/imgsz v0.0.4 h1:Lsasu2hdSSFS+vnD+nvR1UkiRMK7hcpyYCC0FzgSMFI=
 github.com/fumiama/imgsz v0.0.4/go.mod h1:bISOQVTlw9sRytPwe8ir7tAaEmyz9hSNj9n8mXMBG0E=
-golang.org/x/image v0.32.0 h1:6lZQWq75h7L5IWNk0r+SCpUJ6tUVd3v4ZHnbRKLkUDQ=
+golang.org/x/image v0.42.0 h1:1gSs6ehNWXLbkHBIPcWztk3D/6aIA/8hauiAYtlodVY=
-golang.org/x/image v0.32.0/go.mod h1:/R37rrQmKXtO6tYXAjtDLwQgFLHmhW+V6ayXlxzP2Pc=
+golang.org/x/image v0.42.0/go.mod h1:rrpelvGFt+kLPAjPM4HeWPgrl0FtafueU//e5N0qk/Q=
-golang.org/x/net v0.46.0 h1:giFlY12I07fugqwPuWJi68oOnpfqFnJIJzaIIm2JVV4=
+golang.org/x/net v0.56.0 h1:Rw8j/hFzGvJUZwNBXnAtf5sVDVt+65SK2C7IxCxZt5o=
-golang.org/x/net v0.46.0/go.mod h1:Q9BGdFy1y4nkUwiLvT5qtyhAnEHgnQ/zd8PfU6nc210=
+golang.org/x/net v0.56.0/go.mod h1:D3Ku6r+V6JROoZK144D2XfMHFcMq/0zSfLelVTCFKec=
-golang.org/x/text v0.30.0 h1:yznKA/E9zq54KzlzBEAWn1NXSQ8DIp/NYMy88xJjl4k=
+golang.org/x/text v0.38.0 h1:sXmwo9DwP3OK9EZ7PqAdaooSGozfl/3a6/xJcbzPRhE=
-golang.org/x/text v0.30.0/go.mod h1:yDdHFIX9t+tORqspjENWgzaCVXgk0yYnYuSZ8UzzBVM=
+golang.org/x/text v0.38.0/go.mod h1:YXZt3QhHUKYT53r2lLKFIVi6Ao1jdzrTR/KQ09qyxF4=

internal/exporters/docx.go

@@ -16,6 +16,13 @@ import (
 	"github.com/kjanat/articulate-parser/internal/services"
 )
 
+// Font sizes for DOCX document headings (in half-points, so "32" = 16pt).
+const (
+	docxTitleSize  = "32" // Course title (16pt)
+	docxLessonSize = "28" // Lesson heading (14pt)
+	docxItemSize   = "24" // Item heading (12pt)
+)
+
 // DocxExporter implements the Exporter interface for DOCX format.
 // It converts Articulate Rise course data into a Microsoft Word document
 // using the go-docx package.
@@ -53,7 +60,7 @@ func (e *DocxExporter) Export(course *models.Course, outputPath string) error {
 
 	// Add title
 	titlePara := doc.AddParagraph()
-	titlePara.AddText(course.Course.Title).Size("32").Bold()
+	titlePara.AddText(course.Course.Title).Size(docxTitleSize).Bold()
 
 	// Add description if available
 	if course.Course.Description != "" {
@@ -106,7 +113,7 @@ func (e *DocxExporter) Export(course *models.Course, outputPath string) error {
 func (e *DocxExporter) exportLesson(doc *docx.Docx, lesson *models.Lesson) {
 	// Add lesson title
 	lessonPara := doc.AddParagraph()
-	lessonPara.AddText(fmt.Sprintf("Lesson: %s", lesson.Title)).Size("28").Bold()
+	lessonPara.AddText(fmt.Sprintf("Lesson: %s", lesson.Title)).Size(docxLessonSize).Bold()
 
 	// Add lesson description if available
 	if lesson.Description != "" {
@@ -132,7 +139,7 @@ func (e *DocxExporter) exportItem(doc *docx.Docx, item *models.Item) {
 	if item.Type != "" {
 		itemPara := doc.AddParagraph()
 		caser := cases.Title(language.English)
-		itemPara.AddText(caser.String(item.Type)).Size("24").Bold()
+		itemPara.AddText(caser.String(item.Type)).Size(docxItemSize).Bold()
 	}
 
 	// Add sub-items

internal/exporters/factory.go

@@ -13,6 +13,11 @@ const (
 	FormatMarkdown = "markdown"
 	FormatDocx     = "docx"
 	FormatHTML     = "html"
+
+	// Format aliases accepted by CreateExporter.
+	formatAliasMarkdown = "md"
+	formatAliasDocx     = "word"
+	formatAliasHTML     = "htm"
 )
 
 // Factory implements the ExporterFactory interface.
@@ -41,11 +46,11 @@ func NewFactory(htmlCleaner *services.HTMLCleaner) interfaces.ExporterFactory {
 // Format strings are case-insensitive (e.g., "markdown", "DOCX").
 func (f *Factory) CreateExporter(format string) (interfaces.Exporter, error) {
 	switch strings.ToLower(format) {
-	case FormatMarkdown, "md":
+	case FormatMarkdown, formatAliasMarkdown:
 		return NewMarkdownExporter(f.htmlCleaner), nil
-	case FormatDocx, "word":
+	case FormatDocx, formatAliasDocx:
 		return NewDocxExporter(f.htmlCleaner), nil
-	case FormatHTML, "htm":
+	case FormatHTML, formatAliasHTML:
 		return NewHTMLExporter(f.htmlCleaner), nil
 	default:
 		return nil, fmt.Errorf("unsupported export format: %s", format)
@@ -55,5 +60,9 @@ func (f *Factory) CreateExporter(format string) (interfaces.Exporter, error) {
 // SupportedFormats returns a list of all supported export formats,
 // including both primary format names and their aliases.
 func (f *Factory) SupportedFormats() []string {
-	return []string{FormatMarkdown, "md", FormatDocx, "word", FormatHTML, "htm"}
+	return []string{
+		FormatMarkdown, formatAliasMarkdown,
+		FormatDocx, formatAliasDocx,
+		FormatHTML, formatAliasHTML,
+	}
 }

internal/exporters/html.go

@@ -15,7 +15,7 @@ import (
 //go:embed html_styles.css
 var defaultCSS string
 
-//go:embed html_template.html
+//go:embed html_template.gohtml
 var htmlTemplate string
 
 // HTMLExporter implements the Exporter interface for HTML format.
@@ -69,7 +69,16 @@ func (e *HTMLExporter) Export(course *models.Course, outputPath string) error {
 	if err != nil {
 		return fmt.Errorf("failed to create file: %w", err)
 	}
-	defer f.Close()
+	defer func() {
+		// Close errors are logged but not fatal since the content has already been written.
+		// The file must be closed to flush buffers, but a close error doesn't invalidate
+		// the data already written to disk.
+		if closeErr := f.Close(); closeErr != nil {
+			// Note: In production, this should log via a logger passed to the exporter.
+			// For now, we silently ignore close errors as they're non-fatal.
+			_ = closeErr
+		}
+	}()
 
 	return e.WriteHTML(f, course)
 }

internal/exporters/html_styles.css

@@ -1,5 +1,7 @@
 body {
-    font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, Oxygen, Ubuntu, Cantarell, sans-serif;
+  font-family:
+    -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, Oxygen, Ubuntu,
+    Cantarell, sans-serif;
   line-height: 1.6;
   color: #333;
   max-width: 800px;

internal/exporters/html_template_data.go

@@ -21,6 +21,9 @@ const (
 	itemTypeDivider        = "divider"
 )
 
+// lessonTypeSection identifies a lesson that acts as a section header.
+const lessonTypeSection = "section"
+
 // templateData represents the data structure passed to the HTML template.
 type templateData struct {
 	Course   models.CourseInfo
@@ -74,7 +77,7 @@ func prepareTemplateData(course *models.Course, htmlCleaner *services.HTMLCleane
 			Description: lesson.Description,
 		}
 
-		if lesson.Type != "section" {
+		if lesson.Type != lessonTypeSection {
 			lessonCounter++
 			section.Number = lessonCounter
 			section.Items = prepareItems(lesson.Items, htmlCleaner)

internal/exporters/markdown.go

@@ -40,35 +40,35 @@ func (e *MarkdownExporter) Export(course *models.Course, outputPath string) erro
 	var buf bytes.Buffer
 
 	// Write course header
-	buf.WriteString(fmt.Sprintf("# %s\n\n", course.Course.Title))
+	fmt.Fprintf(&buf, "# %s\n\n", course.Course.Title)
 
 	if course.Course.Description != "" {
-		buf.WriteString(fmt.Sprintf("%s\n\n", e.htmlCleaner.CleanHTML(course.Course.Description)))
+		fmt.Fprintf(&buf, "%s\n\n", e.htmlCleaner.CleanHTML(course.Course.Description))
 	}
 
 	// Add metadata
 	buf.WriteString("## Course Information\n\n")
-	buf.WriteString(fmt.Sprintf("- **Course ID**: %s\n", course.Course.ID))
+	fmt.Fprintf(&buf, "- **Course ID**: %s\n", course.Course.ID)
-	buf.WriteString(fmt.Sprintf("- **Share ID**: %s\n", course.ShareID))
+	fmt.Fprintf(&buf, "- **Share ID**: %s\n", course.ShareID)
-	buf.WriteString(fmt.Sprintf("- **Navigation Mode**: %s\n", course.Course.NavigationMode))
+	fmt.Fprintf(&buf, "- **Navigation Mode**: %s\n", course.Course.NavigationMode)
 	if course.Course.ExportSettings != nil {
-		buf.WriteString(fmt.Sprintf("- **Export Format**: %s\n", course.Course.ExportSettings.Format))
+		fmt.Fprintf(&buf, "- **Export Format**: %s\n", course.Course.ExportSettings.Format)
 	}
 	buf.WriteString("\n---\n\n")
 
 	// Process lessons
 	lessonCounter := 0
 	for _, lesson := range course.Course.Lessons {
-		if lesson.Type == "section" {
+		if lesson.Type == lessonTypeSection {
-			buf.WriteString(fmt.Sprintf("# %s\n\n", lesson.Title))
+			fmt.Fprintf(&buf, "# %s\n\n", lesson.Title)
 			continue
 		}
 
 		lessonCounter++
-		buf.WriteString(fmt.Sprintf("## Lesson %d: %s\n\n", lessonCounter, lesson.Title))
+		fmt.Fprintf(&buf, "## Lesson %d: %s\n\n", lessonCounter, lesson.Title)
 
 		if lesson.Description != "" {
-			buf.WriteString(fmt.Sprintf("%s\n\n", e.htmlCleaner.CleanHTML(lesson.Description)))
+			fmt.Fprintf(&buf, "%s\n\n", e.htmlCleaner.CleanHTML(lesson.Description))
 		}
 
 		// Process lesson items
@@ -96,20 +96,23 @@ func (e *MarkdownExporter) SupportedFormat() string {
 func (e *MarkdownExporter) processItemToMarkdown(buf *bytes.Buffer, item models.Item, level int) {
 	headingPrefix := strings.Repeat("#", level)
 
-	switch item.Type {
+	// Normalize item type to lowercase for consistent matching
-	case "text":
+	itemType := strings.ToLower(item.Type)
+
+	switch itemType {
+	case itemTypeText:
 		e.processTextItem(buf, item, headingPrefix)
-	case "list":
+	case itemTypeList:
 		e.processListItem(buf, item)
-	case "multimedia":
+	case itemTypeMultimedia:
 		e.processMultimediaItem(buf, item, headingPrefix)
-	case "image":
+	case itemTypeImage:
 		e.processImageItem(buf, item, headingPrefix)
-	case "knowledgeCheck":
+	case itemTypeKnowledgeCheck:
 		e.processKnowledgeCheckItem(buf, item, headingPrefix)
-	case "interactive":
+	case itemTypeInteractive:
 		e.processInteractiveItem(buf, item, headingPrefix)
-	case "divider":
+	case itemTypeDivider:
 		e.processDividerItem(buf)
 	default:
 		e.processUnknownItem(buf, item, headingPrefix)

internal/services/parser.go

@@ -15,6 +15,15 @@ import (
 	"github.com/kjanat/articulate-parser/internal/models"
 )
 
+// Default endpoint configuration for the Articulate Rise API.
+const (
+	riseHost       = "rise.articulate.com"
+	defaultBaseURL = "https://" + riseHost
+)
+
+// shareIDRegex is compiled once at package init for extracting share IDs from URIs.
+var shareIDRegex = regexp.MustCompile(`/share/([a-zA-Z0-9_-]+)`)
+
 // ArticulateParser implements the CourseParser interface specifically for Articulate Rise courses.
 // It can fetch courses from the Articulate Rise API or load them from local JSON files.
 type ArticulateParser struct {
@@ -34,7 +43,7 @@ func NewArticulateParser(logger interfaces.Logger, baseURL string, timeout time.
 		logger = NewNoOpLogger()
 	}
 	if baseURL == "" {
-		baseURL = "https://rise.articulate.com"
+		baseURL = defaultBaseURL
 	}
 	if timeout == 0 {
 		timeout = 30 * time.Second
@@ -78,15 +87,15 @@ func (p *ArticulateParser) FetchCourse(ctx context.Context, uri string) (*models
 		}
 	}()
 
-	if resp.StatusCode != http.StatusOK {
-		return nil, fmt.Errorf("API returned status %d", resp.StatusCode)
-	}
-
 	body, err := io.ReadAll(resp.Body)
 	if err != nil {
 		return nil, fmt.Errorf("failed to read response body: %w", err)
 	}
 
+	if resp.StatusCode != http.StatusOK {
+		return nil, fmt.Errorf("API returned status %d: %s", resp.StatusCode, string(body))
+	}
+
 	var course models.Course
 	if err := json.Unmarshal(body, &course); err != nil {
 		return nil, fmt.Errorf("failed to unmarshal JSON: %w", err)
@@ -129,12 +138,11 @@ func (p *ArticulateParser) extractShareID(uri string) (string, error) {
 	}
 
 	// Validate that it's an Articulate Rise domain
-	if parsedURL.Host != "rise.articulate.com" {
+	if parsedURL.Host != riseHost {
 		return "", fmt.Errorf("invalid domain for Articulate Rise URI: %s", parsedURL.Host)
 	}
 
-	re := regexp.MustCompile(`/share/([a-zA-Z0-9_-]+)`)
+	matches := shareIDRegex.FindStringSubmatch(uri)
-	matches := re.FindStringSubmatch(uri)
 	if len(matches) < 2 {
 		return "", fmt.Errorf("could not extract share ID from URI: %s", uri)
 	}

main.go

@@ -92,7 +92,7 @@ func run(args []string) int {
 // Returns:
 //   - true if the string appears to be a URI, false otherwise
 func isURI(str string) bool {
-	return len(str) > 7 && (str[:7] == "http://" || str[:8] == "https://")
+	return strings.HasPrefix(str, "http://") || strings.HasPrefix(str, "https://")
 }
 
 // printUsage prints the command-line usage information.