refactor: improve code quality and consistency

- parser.go: compile regex once at package level (perf)
- parser.go: include response body in HTTP error messages (debug)
- main.go: use strings.HasPrefix for URI detection (safety)
- html.go: handle file close errors consistently
- docx.go: extract font size magic numbers to constants
- markdown.go: normalize item types to lowercase for consistency

.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/CODE_OF_CONDUCT.md

@@ -17,23 +17,23 @@ diverse, inclusive, and healthy community.
 Examples of behavior that contributes to a positive environment for our
 community include:
 
--   Demonstrating empathy and kindness toward other people
--   Being respectful of differing opinions, viewpoints, and experiences
--   Giving and gracefully accepting constructive feedback
--   Accepting responsibility and apologizing to those affected by our mistakes,
+- Demonstrating empathy and kindness toward other people
+- Being respectful of differing opinions, viewpoints, and experiences
+- Giving and gracefully accepting constructive feedback
+- Accepting responsibility and apologizing to those affected by our mistakes,
   and learning from the experience
--   Focusing on what is best not just for us as individuals, but for the
+- Focusing on what is best not just for us as individuals, but for the
   overall community
 
 Examples of unacceptable behavior include:
 
--   The use of sexualized language or imagery, and sexual attention or
+- The use of sexualized language or imagery, and sexual attention or
   advances of any kind
--   Trolling, insulting or derogatory comments, and personal or political attacks
--   Public or private harassment
--   Publishing others' private information, such as a physical or email
+- Trolling, insulting or derogatory comments, and personal or political attacks
+- Public or private harassment
+- Publishing others' private information, such as a physical or email
   address, without their explicit permission
--   Other conduct which could reasonably be considered inappropriate in a
+- Other conduct which could reasonably be considered inappropriate in a
   professional setting
 
 ## Enforcement Responsibilities

.github/CONTRIBUTING.md

@@ -12,73 +12,73 @@ This project and everyone participating in it is governed by our Code of Conduct
 
 Before creating bug reports, please check existing issues as you might find that the issue has already been reported. When creating a bug report, include as many details as possible:
 
--   Use the bug report template
--   Include sample Articulate Rise content that reproduces the issue
--   Provide your environment details (OS, Go version, etc.)
--   Include error messages and stack traces
+- Use the bug report template
+- Include sample Articulate Rise content that reproduces the issue
+- Provide your environment details (OS, Go version, etc.)
+- Include error messages and stack traces
 
 ### Suggesting Enhancements
 
 Enhancement suggestions are welcome! Please use the feature request template and include:
 
--   A clear description of the enhancement
--   Your use case and why this would be valuable
--   Any implementation ideas you might have
+- A clear description of the enhancement
+- Your use case and why this would be valuable
+- Any implementation ideas you might have
 
 ### Pull Requests
 
-1.  **Fork the repository** and create your branch from `master`
-2.  **Make your changes** following our coding standards
-3.  **Add tests** for any new functionality
-4.  **Ensure all tests pass** by running `go test ./...`
-5.  **Run `go fmt`** to format your code
-6.  **Run `go vet`** to check for common issues
-7.  **Update documentation** if needed
-8.  **Create a pull request** with a clear title and description
+1. **Fork the repository** and create your branch from `master`
+2. **Make your changes** following our coding standards
+3. **Add tests** for any new functionality
+4. **Ensure all tests pass** by running `go test ./...`
+5. **Run `go fmt`** to format your code
+6. **Run `go vet`** to check for common issues
+7. **Update documentation** if needed
+8. **Create a pull request** with a clear title and description
 
 ## Development Setup
 
-1.  **Prerequisites:**
+1. **Prerequisites:**
 
--   Go 1.21 or later
--   Git
+- Go 1.21 or later
+- Git
 
-2.  **Clone and setup:**
+2. **Clone and setup:**
 
-   ```bash
-   git clone https://github.com/your-username/articulate-parser.git
-   cd articulate-parser
-   go mod download
-   ```
+```bash
+git clone https://github.com/your-username/articulate-parser.git
+cd articulate-parser
+go mod download
+```
 
-3.  **Run tests:**
+3. **Run tests:**
 
-   ```bash
-   go test -v ./...
-   ```
+```bash
+go test -v ./...
+```
 
-4.  **Build:**
+4. **Build:**
 
-   ```bash
-   go build main.go
-   ```
+```bash
+go build main.go
+```
 
 ## Coding Standards
 
 ### Go Style Guide
 
--   Follow the [Go Code Review Comments](https://github.com/golang/go/wiki/CodeReviewComments)
--   Use `gofmt` to format your code
--   Use meaningful variable and function names
--   Add comments for exported functions and types
--   Keep functions focused and small
+- Follow the [Go Code Review Comments](https://github.com/golang/go/wiki/CodeReviewComments)
+- Use `gofmt` to format your code
+- Use meaningful variable and function names
+- Add comments for exported functions and types
+- Keep functions focused and small
 
 ### Testing
 
--   Write tests for new functionality
--   Use table-driven tests where appropriate
--   Aim for good test coverage
--   Test error cases and edge conditions
+- Write tests for new functionality
+- Use table-driven tests where appropriate
+- Aim for good test coverage
+- Test error cases and edge conditions
 
 ### Commit Messages
 
@@ -112,19 +112,19 @@ articulate-parser/
 
 ### New Content Types
 
-1.  Add the content type definition to `types/`
-2.  Implement parsing logic in `parser/`
-3.  Add export handling in `exporters/`
-4.  Write comprehensive tests
-5.  Update documentation
+1. Add the content type definition to `types/`
+2. Implement parsing logic in `parser/`
+3. Add export handling in `exporters/`
+4. Write comprehensive tests
+5. Update documentation
 
 ### New Export Formats
 
-1.  Create a new exporter in `exporters/`
-2.  Implement the `Exporter` interface
-3.  Add CLI support in `main.go`
-4.  Add tests with sample output
-5.  Update README with usage examples
+1. Create a new exporter in `exporters/`
+2. Implement the `Exporter` interface
+3. Add CLI support in `main.go`
+4. Add tests with sample output
+5. Update README with usage examples
 
 ## Testing
 
@@ -146,31 +146,31 @@ go test -run TestSpecificFunction ./...
 
 ### Test Data
 
--   Add sample Articulate Rise JSON files to `tests/data/`
--   Include both simple and complex content examples
--   Test edge cases and error conditions
+- Add sample Articulate Rise JSON files to `tests/data/`
+- Include both simple and complex content examples
+- Test edge cases and error conditions
 
 ## Documentation
 
--   Update the README for user-facing changes
--   Add inline code comments for complex logic
--   Update examples when adding new features
--   Keep the feature list current
+- Update the README for user-facing changes
+- Add inline code comments for complex logic
+- Update examples when adding new features
+- Keep the feature list current
 
 ## Release Process
 
 Releases are handled by maintainers:
 
-1.  Version bumping follows semantic versioning
-2.  Releases are created from the `master` branch
-3.  GitHub Actions automatically builds and publishes releases
-4.  Release notes are auto-generated from commits
+1. Version bumping follows semantic versioning
+2. Releases are created from the `master` branch
+3. GitHub Actions automatically builds and publishes releases
+4. Release notes are auto-generated from commits
 
 ## Questions?
 
--   Open a discussion for general questions
--   Use the question issue template for specific help
--   Check existing issues and documentation first
+- Open a discussion for general questions
+- Use the question issue template for specific help
+- Check existing issues and documentation first
 
 ## Recognition
 

.github/ISSUE_TEMPLATE/bug_report.yml

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

.github/PULL_REQUEST_TEMPLATE.md

@@ -5,31 +5,34 @@
 ## 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)
--   [ ] Documentation update
--   [ ] Performance improvement
--   [ ] Code refactoring (no functional changes)
--   [ ] Test updates
+
+- [ ] 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)
+- [ ] Documentation update
+- [ ] Performance improvement
+- [ ] Code refactoring (no functional changes)
+- [ ] Test updates
 
 ## 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
--   [ ] I have updated the documentation
--   [ ] I have added tests that prove my fix is effective or that my feature works
--   [ ] New and existing unit tests pass locally with my changes
--   [ ] I have checked for potential breaking changes
--   [ ] No new warnings are generated
--   [ ] The commit message follows our guidelines
+
+- [ ] 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
+- [ ] I have updated the documentation
+- [ ] I have added tests that prove my fix is effective or that my feature works
+- [ ] New and existing unit tests pass locally with my changes
+- [ ] I have checked for potential breaking changes
+- [ ] No new warnings are generated
+- [ ] The commit message follows our guidelines
 
 ## Screenshots (if appropriate)
 
@@ -42,6 +45,7 @@ Fixes #
 ## Testing Instructions
 
 <!-- Provide steps to test the changes, if applicable -->
+
 1.
 2.
 3.

.github/SECURITY.md

@@ -13,32 +13,32 @@ Currently, the following versions of Articulate Rise Parser are supported with s
 
 We take the security of Articulate Rise Parser seriously. If you believe you have found a security vulnerability, please follow these steps:
 
-1.  **Do not disclose the vulnerability publicly** - Please do not create a public GitHub issue for security vulnerabilities.
-2.  **Email the details to [security+articulate-parser@kjanat.com]** - Include as much information as possible about the vulnerability.
-3.  **Wait for a response** - We will acknowledge your email within 48 hours and provide an estimated timeline for a fix.
-4.  **Work with us** - We may ask for additional information to help us understand and address the issue.
+1. **Do not disclose the vulnerability publicly** - Please do not create a public GitHub issue for security vulnerabilities.
+2. **Email the details to [security+articulate-parser@kjanat.com]** - Include as much information as possible about the vulnerability.
+3. **Wait for a response** - We will acknowledge your email within 48 hours and provide an estimated timeline for a fix.
+4. **Work with us** - We may ask for additional information to help us understand and address the issue.
 
 ## What to Include in a Report
 
 When reporting a vulnerability, please include:
 
--   A clear description of the issue
--   Steps to reproduce the vulnerability
--   The potential impact of the vulnerability
--   Any possible mitigations you've identified
+- A clear description of the issue
+- Steps to reproduce the vulnerability
+- The potential impact of the vulnerability
+- Any possible mitigations you've identified
 
 ## What to Expect
 
--   We will acknowledge receipt of your vulnerability report within 48 hours.
--   We will provide regular updates about our progress.
--   We will notify you when the vulnerability is fixed.
--   With your permission, we will include your name in the acknowledgments.
+- We will acknowledge receipt of your vulnerability report within 48 hours.
+- We will provide regular updates about our progress.
+- We will notify you when the vulnerability is fixed.
+- With your permission, we will include your name in the acknowledgments.
 
 ## Security Measures
 
 This project follows these security practices:
 
--   Regular dependency updates via Dependabot
--   CodeQL security scanning
--   Automated testing for each pull request
--   Code review requirements for all changes
+- Regular dependency updates via Dependabot
+- CodeQL security scanning
+- Automated testing for each pull request
+- Code review requirements for all changes

.github/dependabot.yml

@@ -1,86 +1,86 @@
 version: 2
 updates:
   # Check for updates to GitHub Actions
-  - package-ecosystem: 'github-actions'
-    directory: '/'
+  - package-ecosystem: "github-actions"
+    directory: "/"
     schedule:
-      interval: 'weekly'
-      day: 'monday'
-      time: '07:00'
-      timezone: 'Europe/Amsterdam'
+      interval: "weekly"
+      day: "monday"
+      time: "07:00"
+      timezone: "Europe/Amsterdam"
     open-pull-requests-limit: 2
     labels:
-      - 'dependencies'
-      - 'dependencies/github-actions'
+      - "dependencies"
+      - "dependencies/github-actions"
     commit-message:
-      prefix: 'ci'
-      include: 'scope'
+      prefix: "ci"
+      include: "scope"
 
   # Check for updates to Docker
-  - package-ecosystem: 'docker'
-    directory: '/'
+  - package-ecosystem: "docker"
+    directory: "/"
     schedule:
-      interval: 'weekly'
-      day: 'monday'
-      time: '07:00'
-      timezone: 'Europe/Amsterdam'
+      interval: "weekly"
+      day: "monday"
+      time: "07:00"
+      timezone: "Europe/Amsterdam"
     open-pull-requests-limit: 2
     labels:
-      - 'dependencies'
-      - 'dependencies/docker'
+      - "dependencies"
+      - "dependencies/docker"
     commit-message:
-      prefix: 'docker'
-      include: 'scope'
+      prefix: "docker"
+      include: "scope"
     groups:
       docker:
         patterns:
-          - '*'
+          - "*"
         update-types:
-          - 'minor'
-          - 'patch'
+          - "minor"
+          - "patch"
 
   # Check for updates to Docker Compose
-  - package-ecosystem: 'docker-compose'
-    directory: '/'
+  - package-ecosystem: "docker-compose"
+    directory: "/"
     schedule:
-      interval: 'weekly'
-      day: 'monday'
-      time: '07:00'
-      timezone: 'Europe/Amsterdam'
+      interval: "weekly"
+      day: "monday"
+      time: "07:00"
+      timezone: "Europe/Amsterdam"
     open-pull-requests-limit: 2
     labels:
-      - 'dependencies'
-      - 'dependencies/docker-compose'
+      - "dependencies"
+      - "dependencies/docker-compose"
     commit-message:
-      prefix: 'docker'
-      include: 'scope'
+      prefix: "docker"
+      include: "scope"
     groups:
       docker:
         patterns:
-          - '*'
+          - "*"
         update-types:
-          - 'minor'
-          - 'patch'
+          - "minor"
+          - "patch"
 
   # Check for updates to Go modules
-  - package-ecosystem: 'gomod'
-    directory: '/'
+  - package-ecosystem: "gomod"
+    directory: "/"
     schedule:
-      interval: 'weekly'
-      day: 'monday'
-      time: '07:00'
-      timezone: 'Europe/Amsterdam'
+      interval: "weekly"
+      day: "monday"
+      time: "07:00"
+      timezone: "Europe/Amsterdam"
     open-pull-requests-limit: 2
     labels:
-      - 'dependencies'
-      - 'dependencies/go'
+      - "dependencies"
+      - "dependencies/go"
     commit-message:
-      prefix: 'deps'
-      include: 'scope'
+      prefix: "deps"
+      include: "scope"
     groups:
       go-modules:
         patterns:
-          - '*'
+          - "*"
         update-types:
-          - 'minor'
-          - 'patch'
+          - "minor"
+          - "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:
@@ -38,14 +38,11 @@ jobs:
     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 }}
         uses: actions/setup-go@v6
@@ -67,14 +64,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 }}
           - **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 +88,17 @@ jobs:
           SKIPPED_TESTS=$(grep -c "--- SKIP:" test-output.log || echo "0")
 
           # Generate test summary
-          cat >> $GITHUB_STEP_SUMMARY << EOF
+          {
+            cat << EOF
           ## 🧪 Test Results (Go ${{ matrix.go }})
 
-          | Metric      | Value                                                       |
-          | ----------- | ----------------------------------------------------------- |
-          | Total Tests | $TOTAL_TESTS                                                |
-          | Passed      | $PASSED_TESTS                                               |
-          | Failed      | $FAILED_TESTS                                               |
-          | Skipped     | $SKIPPED_TESTS                                              |
-          | Status      | $([ $TEST_STATUS -eq 0 ] && echo "PASSED" || echo "FAILED") |
+          | Metric      | Value                                                         |
+          | ----------- | ------------------------------------------------------------- |
+          | Total Tests | $TOTAL_TESTS                                                  |
+          | Passed      | $PASSED_TESTS                                                 |
+          | Failed      | $FAILED_TESTS                                                 |
+          | Skipped     | $SKIPPED_TESTS                                                |
+          | Status      | $([ "$TEST_STATUS" -eq 0 ] && echo "PASSED" || echo "FAILED") |
 
           ### 📦 Package Test Results
 
@@ -104,42 +106,43 @@ jobs:
           |---------|--------|
           EOF
 
-          # Extract package results
-          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
-            elif [[ $line == FAIL* ]]; then
-              pkg=$(echo "$line" | awk '{print $2}')
-              echo "| $pkg | ❌ FAIL |" >> $GITHUB_STEP_SUMMARY
-            fi
-          done
+            # Extract package results
+            grep "^ok\|^FAIL" test-output.log | while read -r line; do
+              if [[ $line == ok* ]]; then
+                pkg=$(echo "$line" | awk '{print $2}')
+                echo "| $pkg | ✅ PASS |"
+              elif [[ $line == FAIL* ]]; then
+                pkg=$(echo "$line" | awk '{print $2}')
+                echo "| $pkg | ❌ FAIL |"
+              fi
+            done
 
-          echo "" >> $GITHUB_STEP_SUMMARY
+            echo ""
 
-          # Add detailed results if tests failed
-          if [ $TEST_STATUS -ne 0 ]; then
-            cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+            # Add detailed results if tests failed
+            if [ "$TEST_STATUS" -ne 0 ]; then
+              cat << 'EOF'
           ### ❌ Failed Tests Details
           ```
           EOF
-            grep -A 10 "--- FAIL:" test-output.log | head -100 >> $GITHUB_STEP_SUMMARY
-            cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+              grep -A 10 -- "--- FAIL:" test-output.log | head -100
+              cat << 'EOF'
           ```
 
           EOF
-          fi
+            fi
+          } >> "$GITHUB_STEP_SUMMARY"
 
           # Set outputs for other steps
-          cat >> $GITHUB_OUTPUT << EOF
-          test-status=$TEST_STATUS
-          total-tests=$TOTAL_TESTS
-          passed-tests=$PASSED_TESTS
-          failed-tests=$FAILED_TESTS
-          EOF
+          {
+            echo "test-status=$TEST_STATUS"
+            echo "total-tests=$TOTAL_TESTS"
+            echo "passed-tests=$PASSED_TESTS"
+            echo "failed-tests=$FAILED_TESTS"
+          } >> "$GITHUB_OUTPUT"
 
           # Exit with the original test status
-          exit $TEST_STATUS
+          exit "$TEST_STATUS"
 
       - name: Generate coverage report
         if: always()
@@ -147,7 +150,8 @@ 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
+            {
+              cat << EOF
           ## 📊 Code Coverage (Go ${{ matrix.go }})
 
           **Total Coverage: $COVERAGE**
@@ -159,45 +163,46 @@ jobs:
           | ------- | -------- |
           EOF
 
-            # Create temporary file for package coverage aggregation
-            temp_coverage=$(mktemp)
+              # Create temporary file for package coverage aggregation
+              temp_coverage=$(mktemp)
 
-            # Extract package-level coverage data
-            go tool cover -func=coverage/coverage.out | grep -v total | while read -r line; do
-              if [[ $line == *".go:"* ]]; then
-                # Extract package path from file path (everything before the filename)
-                filepath=$(echo "$line" | awk '{print $1}')
-                pkg_path=$(dirname "$filepath" | sed 's|github.com/kjanat/articulate-parser/||; s|^\./||')
-                coverage=$(echo "$line" | awk '{print $3}' | sed 's/%//')
+              # Extract package-level coverage data
+              go tool cover -func=coverage/coverage.out | grep -v total | while read -r line; do
+                if [[ $line == *".go:"* ]]; then
+                  # Extract package path from file path (everything before the filename)
+                  filepath=$(echo "$line" | awk '{print $1}')
+                  pkg_path=$(dirname "$filepath" | sed 's|github.com/kjanat/articulate-parser/||; s|^\./||')
+                  coverage=$(echo "$line" | awk '{print $3}' | sed 's/%//')
 
-                # Use root package if no subdirectory
-                [[ "$pkg_path" == "." || "$pkg_path" == "" ]] && pkg_path="root"
+                  # Use root package if no subdirectory
+                  [[ "$pkg_path" == "." || "$pkg_path" == "" ]] && pkg_path="root"
 
-                echo "$pkg_path $coverage" >> "$temp_coverage"
-              fi
-            done
+                  echo "$pkg_path $coverage" >> "$temp_coverage"
+                fi
+              done
 
-            # Aggregate coverage by package (average)
-            awk '{
-              packages[$1] += $2
-              counts[$1]++
-            }
-            END {
-              for (pkg in packages) {
-                avg = packages[pkg] / counts[pkg]
-                printf "| %s | %.1f%% |\n", pkg, avg
+              # Aggregate coverage by package (average)
+              awk '{
+                packages[$1] += $2
+                counts[$1]++
               }
-            }' "$temp_coverage" | sort >> $GITHUB_STEP_SUMMARY
+              END {
+                for (pkg in packages) {
+                  avg = packages[pkg] / counts[pkg]
+                  printf "| %s | %.1f%% |\n", pkg, avg
+                }
+              }' "$temp_coverage" | sort
 
-            rm -f "$temp_coverage"
+              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,7 +211,7 @@ jobs:
 
       - name: Upload test artifacts
         if: failure()
-        uses: actions/upload-artifact@v5
+        uses: actions/upload-artifact@v6
         with:
           name: test-results-go-${{ matrix.go }}
           path: |
@@ -216,52 +221,53 @@ jobs:
 
       - name: Run linters
         run: |
-          # Initialize summary
-          cat >> $GITHUB_STEP_SUMMARY << EOF
+          {
+            cat << EOF
           ## 🔍 Static Analysis (Go ${{ matrix.go }})
 
           EOF
 
-          # Run go vet
-          VET_OUTPUT=$(task lint:vet 2>&1 || echo "")
-          VET_STATUS=$?
+            # Run go vet
+            VET_OUTPUT=$(task lint:vet 2>&1 || echo "")
+            VET_STATUS=$?
 
-          if [ $VET_STATUS -eq 0 ]; then
-            echo "✅ **go vet:** No issues found" >> $GITHUB_STEP_SUMMARY
-          else
-            cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+            if [ "$VET_STATUS" -eq 0 ]; then
+              echo "✅ **go vet:** No issues found"
+            else
+              cat << 'EOF'
           ❌ **go vet:** Issues found
 
           ```
           EOF
-            echo "$VET_OUTPUT" >> $GITHUB_STEP_SUMMARY
-            echo '```' >> $GITHUB_STEP_SUMMARY
-          fi
-          echo "" >> $GITHUB_STEP_SUMMARY
+              echo "$VET_OUTPUT"
+              echo '```'
+            fi
+            echo ""
 
-          # Run go fmt check
-          FMT_OUTPUT=$(task lint:fmt 2>&1 || echo "")
-          FMT_STATUS=$?
+            # Run go fmt check
+            FMT_OUTPUT=$(task lint:fmt 2>&1 || echo "")
+            FMT_STATUS=$?
 
-          if [ $FMT_STATUS -eq 0 ]; then
-            echo "✅ **go fmt:** All files properly formatted" >> $GITHUB_STEP_SUMMARY
-          else
-            cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+            if [ "$FMT_STATUS" -eq 0 ]; then
+              echo "✅ **go fmt:** All files properly formatted"
+            else
+              cat << 'EOF'
           ❌ **go fmt:** Files need formatting
 
           ```
           EOF
-            echo "$FMT_OUTPUT" >> $GITHUB_STEP_SUMMARY
-            echo '```' >> $GITHUB_STEP_SUMMARY
-          fi
+              echo "$FMT_OUTPUT"
+              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'
+          cat >> "$GITHUB_STEP_SUMMARY" << 'EOF'
           ## 📋 Job Summary (Go ${{ matrix.go }})
 
           | Step            | Status                                                          |
@@ -297,7 +303,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,24 +322,26 @@ jobs:
 
       - name: Test Docker image using Task
         run: |
-          cat >> $GITHUB_STEP_SUMMARY << 'EOF'
+          {
+            cat << 'EOF'
           ## 🧪 Docker Image Tests
 
           EOF
 
-          # Run Task docker test
-          task docker:test
+            # Run Task docker test
+            task docker:test
 
-          echo "**Testing help command:**" >> $GITHUB_STEP_SUMMARY
-          echo '```terminaloutput' >> $GITHUB_STEP_SUMMARY
-          docker run --rm articulate-parser:latest --help >> $GITHUB_STEP_SUMMARY
-          echo '```' >> $GITHUB_STEP_SUMMARY
-          echo "" >> $GITHUB_STEP_SUMMARY
+            echo "**Testing help command:**"
+            echo '```terminaloutput'
+            docker run --rm articulate-parser:latest --help
+            echo '```'
+            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 "" >> $GITHUB_STEP_SUMMARY
+            # Test image size
+            IMAGE_SIZE=$(docker image inspect articulate-parser:latest --format='{{.Size}}' | numfmt --to=iec-i --suffix=B)
+            echo "**Image size:** $IMAGE_SIZE"
+            echo ""
+          } >> "$GITHUB_STEP_SUMMARY"
 
   dependency-review:
     name: Dependency Review
@@ -342,10 +350,10 @@ jobs:
       contents: read
     if: github.event_name == 'pull_request'
     steps:
-      - name: 'Checkout Repository'
-        uses: actions/checkout@v5
+      - name: "Checkout Repository"
+        uses: actions/checkout@v6
 
-      - name: 'Dependency Review'
+      - name: "Dependency Review"
         uses: actions/dependency-review-action@v4
         with:
           fail-on-severity: moderate
@@ -357,14 +365,14 @@ 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
@@ -436,7 +444,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:
@@ -47,58 +47,58 @@ jobs:
       fail-fast: false
       matrix:
         include:
-        - language: actions
-          build-mode: none
-        - language: go
-          build-mode: autobuild
-        # CodeQL supports the following values keywords for 'language': 'actions', 'c-cpp', 'csharp', 'go', 'java-kotlin', 'javascript-typescript', 'python', 'ruby', 'swift'
-        # Use `c-cpp` to analyze code written in C, C++ or both
-        # Use 'java-kotlin' to analyze code written in Java, Kotlin or both
-        # Use 'javascript-typescript' to analyze code written in JavaScript, TypeScript or both
-        # To learn more about changing the languages that are analyzed or customizing the build mode for your analysis,
-        # see https://docs.github.com/en/code-security/code-scanning/creating-an-advanced-setup-for-code-scanning/customizing-your-advanced-setup-for-code-scanning.
-        # If you are analyzing a compiled language, you can modify the 'build-mode' for that language to customize how
-        # 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
+          - language: actions
+            build-mode: none
+          - language: go
+            build-mode: autobuild
+          # CodeQL supports the following values keywords for 'language': 'actions', 'c-cpp', 'csharp', 'go', 'java-kotlin', 'javascript-typescript', 'python', 'ruby', 'swift'
+          # Use `c-cpp` to analyze code written in C, C++ or both
+          # Use 'java-kotlin' to analyze code written in Java, Kotlin or both
+          # Use 'javascript-typescript' to analyze code written in JavaScript, TypeScript or both
+          # To learn more about changing the languages that are analyzed or customizing the build mode for your analysis,
+          # see https://docs.github.com/en/code-security/code-scanning/creating-an-advanced-setup-for-code-scanning/customizing-your-advanced-setup-for-code-scanning.
+          # If you are analyzing a compiled language, you can modify the 'build-mode' for that language to customize how
+          # 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
+      - name: Checkout repository
+        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`
-    # or others). This is typically only required for manual builds.
-    # - name: Setup runtime (example)
-    #   uses: actions/setup-example@v1
+      # Add any setup steps before running the `github/codeql-action/init` action.
+      # This includes steps like installing compilers or runtimes (`actions/setup-node`
+      # or others). This is typically only required for manual builds.
+      # - name: Setup runtime (example)
+      #   uses: actions/setup-example@v1
 
-    # Initializes the CodeQL tools for scanning.
-    - name: Initialize CodeQL
-      uses: github/codeql-action/init@v4
-      with:
-        languages: ${{ matrix.language }}
-        build-mode: ${{ matrix.build-mode }}
-        # If you wish to specify custom queries, you can do so here or in a config file.
-        # By default, queries listed here will override any specified in a config file.
-        # Prefix the list here with "+" to use these queries and those in the config file.
+      # Initializes the CodeQL tools for scanning.
+      - name: Initialize CodeQL
+        uses: github/codeql-action/init@v4
+        with:
+          languages: ${{ matrix.language }}
+          build-mode: ${{ matrix.build-mode }}
+          # If you wish to specify custom queries, you can do so here or in a config file.
+          # By default, queries listed here will override any specified in a config file.
+          # Prefix the list here with "+" to use these queries and those in the config file.
 
-        # For more details on CodeQL's query packs, refer to: https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-code-scanning#using-queries-in-ql-packs
-        # queries: security-extended,security-and-quality
+          # For more details on CodeQL's query packs, refer to: https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-code-scanning#using-queries-in-ql-packs
+          # queries: security-extended,security-and-quality
 
-    # If the analyze step fails for one of the languages you are analyzing with
-    # "We were unable to automatically build your code", modify the matrix above
-    # to set the build mode to "manual" for that language. Then modify this step
-    # to build your code.
-    # ℹ️ Command-line programs to run using the OS shell.
-    # 📚 See https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsrun
-    - if: matrix.build-mode == 'manual'
-      shell: bash
-      run: |
-        echo 'If you are using a "manual" build mode for one or more of the' \
-          'languages you are analyzing, replace this with the commands to build' \
-          'your code, for example:'
-        echo '  make bootstrap'
-        echo '  make release'
-        exit 1
+      # If the analyze step fails for one of the languages you are analyzing with
+      # "We were unable to automatically build your code", modify the matrix above
+      # to set the build mode to "manual" for that language. Then modify this step
+      # to build your code.
+      # ℹ️ Command-line programs to run using the OS shell.
+      # 📚 See https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsrun
+      - if: matrix.build-mode == 'manual'
+        shell: bash
+        run: |
+          echo 'If you are using a "manual" build mode for one or more of the' \
+            'languages you are analyzing, replace this with the commands to build' \
+            'your code, for example:'
+          echo '  make bootstrap'
+          echo '  make release'
+          exit 1
 
-    - name: Perform CodeQL Analysis
-      uses: github/codeql-action/analyze@v4
-      with:
-        category: "/language:${{matrix.language}}"
+      - name: Perform CodeQL Analysis
+        uses: github/codeql-action/analyze@v4
+        with:
+          category: "/language:${{matrix.language}}"

.github/workflows/dependency-review.yml

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

.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
 
@@ -82,13 +82,13 @@ 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

.golangci.yml

@@ -79,68 +79,68 @@ linters:
   # Enable specific linters
   enable:
     # Default/standard linters
-    - errcheck      # Check for unchecked errors
-    - govet         # Go vet
-    - ineffassign   # Detect ineffectual assignments
-    - staticcheck   # Staticcheck
-    - unused        # Find unused code
+    - errcheck # Check for unchecked errors
+    - govet # Go vet
+    - ineffassign # Detect ineffectual assignments
+    - staticcheck # Staticcheck
+    - unused # Find unused code
 
     # Code quality
-    - revive        # Fast, configurable linter
-    - gocritic      # Opinionated Go source code linter
-    - godot         # Check comment periods
-    - godox         # Detect TODO/FIXME comments
-    - gocognit      # Cognitive complexity
-    - gocyclo       # Cyclomatic complexity
-    - funlen        # Function length
-    - maintidx      # Maintainability index
+    - revive # Fast, configurable linter
+    - gocritic # Opinionated Go source code linter
+    - godot # Check comment periods
+    - godox # Detect TODO/FIXME comments
+    - gocognit # Cognitive complexity
+    - gocyclo # Cyclomatic complexity
+    - funlen # Function length
+    - maintidx # Maintainability index
 
     # Security
-    - gosec         # Security problems
+    - gosec # Security problems
 
     # Performance
-    - prealloc      # Find slice preallocation opportunities
-    - bodyclose     # Check HTTP response body closed
+    - prealloc # Find slice preallocation opportunities
+    - bodyclose # Check HTTP response body closed
 
     # Style and formatting
-    - goconst       # Find repeated strings
-    - misspell      # Find misspellings
-    - whitespace    # Find unnecessary blank lines
-    - unconvert     # Remove unnecessary type conversions
-    - dupword       # Check for duplicate words
+    - goconst # Find repeated strings
+    - misspell # Find misspellings
+    - whitespace # Find unnecessary blank lines
+    - unconvert # Remove unnecessary type conversions
+    - dupword # Check for duplicate words
 
     # Error handling
-    - errorlint     # Error handling improvements
-    - wrapcheck     # Check error wrapping
+    - errorlint # Error handling improvements
+    - wrapcheck # Check error wrapping
 
     # Testing
-    - testifylint   # Testify usage
-    - tparallel     # Detect improper t.Parallel() usage
-    - thelper       # Detect test helpers without t.Helper()
+    - testifylint # Testify usage
+    - tparallel # Detect improper t.Parallel() usage
+    - thelper # Detect test helpers without t.Helper()
 
     # Best practices
-    - exhaustive    # Check exhaustiveness of enum switches
-    - nolintlint    # Check nolint directives
-    - nakedret      # Find naked returns
-    - nilnil        # Check for redundant nil checks
-    - noctx         # Check sending HTTP requests without context
-    - contextcheck  # Check context propagation
-    - asciicheck    # Check for non-ASCII identifiers
-    - bidichk       # Check for dangerous unicode sequences
+    - exhaustive # Check exhaustiveness of enum switches
+    - nolintlint # Check nolint directives
+    - nakedret # Find naked returns
+    - nilnil # Check for redundant nil checks
+    - noctx # Check sending HTTP requests without context
+    - contextcheck # Check context propagation
+    - asciicheck # Check for non-ASCII identifiers
+    - bidichk # Check for dangerous unicode sequences
     - durationcheck # Check for multiplied durations
-    - makezero      # Find slice declarations with non-zero length
-    - nilerr        # Find code returning nil with non-nil error
-    - predeclared   # Find code shadowing predeclared identifiers
-    - promlinter    # Check Prometheus metrics naming
-    - reassign      # Check reassignment of package variables
+    - makezero # Find slice declarations with non-zero length
+    - nilerr # Find code returning nil with non-nil error
+    - predeclared # Find code shadowing predeclared identifiers
+    - promlinter # Check Prometheus metrics naming
+    - reassign # Check reassignment of package variables
     - usestdlibvars # Use variables from stdlib
-    - wastedassign  # Find wasted assignments
+    - wastedassign # Find wasted assignments
 
     # Documentation
-    - godoclint     # Check godoc comments
+    - godoclint # Check godoc comments
 
     # New
-    - modernize     # Suggest simplifications using new Go features
+    - modernize # Suggest simplifications using new Go features
 
   # Exclusion rules for linters
   exclusions:
@@ -221,8 +221,8 @@ linters:
     govet:
       enable-all: true
       disable:
-        - fieldalignment  # Too many false positives
-        - shadow         # Can be noisy
+        - fieldalignment # Too many false positives
+        - shadow # Can be noisy
 
     # goconst settings
     goconst:
@@ -286,8 +286,8 @@ linters:
       severity: medium
       confidence: medium
       excludes:
-        - G104  # Handled by errcheck
-        - G304  # File path provided as taint input
+        - G104 # Handled by errcheck
+        - G304 # File path provided as taint input
 
     # revive settings
     revive:
@@ -349,7 +349,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 ./...`
-- **Run single test**: `go test -v -race -run ^TestName$ ./path/to/package`
-- **Test with coverage**: 
-  - `task test:coverage` or 
-  - `go test -race -coverprofile=coverage/coverage.out -covermode=atomic ./...`
-- **Lint**: `task lint` (runs vet, fmt check, staticcheck, golangci-lint)
-- **Format**: `task fmt` or `gofmt -s -w .`
-- **CI checks**: `task ci` (deps, lint, test with coverage, build)
+
+### Primary Commands (using Taskfile)
+
+```bash
+task build              # Build binary to bin/articulate-parser
+task test               # Run all tests with race detection
+task lint               # Run all linters (vet, fmt, staticcheck, golangci-lint)
+task fmt                # Format all Go files
+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
-- Group related imports together
+- Order: stdlib, blank line, external packages, blank line, internal packages
+
+```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
-- Cognitive complexity: max 20
+- Cyclomatic complexity: max 15; 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

DOCKER.md

@@ -49,17 +49,17 @@ docker run --rm ghcr.io/kjanat/articulate-parser:latest --version
 
 ## Available Tags
 
--   `latest` - Latest stable release
--   `v1.x.x` - Specific version tags
--   `main` - Latest development build
+- `latest` - Latest stable release
+- `v1.x.x` - Specific version tags
+- `main` - Latest development build
 
 ## Image Details
 
--   **Base Image**: `scratch` (minimal attack surface)
--   **Architecture**: Multi-arch support (amd64, arm64)
--   **Size**: < 10MB (optimized binary)
--   **Security**: Runs as non-root user
--   **Features**: SBOM and provenance attestation included
+- **Base Image**: `scratch` (minimal attack surface)
+- **Architecture**: Multi-arch support (amd64, arm64)
+- **Size**: < 10MB (optimized binary)
+- **Security**: Runs as non-root user
+- **Features**: SBOM and provenance attestation included
 
 ## Development
 
@@ -77,6 +77,6 @@ docker-compose up --build
 
 ## Repository
 
--   **Source**: [github.com/kjanat/articulate-parser](https://github.com/kjanat/articulate-parser)
--   **Issues**: [Report bugs or request features](https://github.com/kjanat/articulate-parser/issues)
--   **License**: See repository for license details
+- **Source**: [github.com/kjanat/articulate-parser](https://github.com/kjanat/articulate-parser)
+- **Issues**: [Report bugs or request features](https://github.com/kjanat/articulate-parser/issues)
+- **License**: See repository for license details

README.md

@@ -78,32 +78,32 @@ flowchart TD
 
 The system follows **Clean Architecture** principles with clear separation of concerns:
 
--   **🎯 Entry Point**: Command-line interface handles user input and coordinates operations
--   **🏗️ Application Layer**: Core business logic with dependency injection
--   **📋 Interface Layer**: Contracts defining behavior without implementation details  
--   **🔧 Service Layer**: Concrete implementations of parsing and utility services
--   **📤 Export Layer**: Factory pattern for format-specific exporters
--   **📊 Data Layer**: Domain models representing course structure
+- **🎯 Entry Point**: Command-line interface handles user input and coordinates operations
+- **🏗️ Application Layer**: Core business logic with dependency injection
+- **📋 Interface Layer**: Contracts defining behavior without implementation details
+- **🔧 Service Layer**: Concrete implementations of parsing and utility services
+- **📤 Export Layer**: Factory pattern for format-specific exporters
+- **📊 Data Layer**: Domain models representing course structure
 
 ## Features
 
--   Parse Articulate Rise JSON data from URLs or local files
--   Export to Markdown (.md) format
--   Export to HTML (.html) format with professional styling
--   Export to Word Document (.docx) format
--   Support for various content types:
-    -   Text content with headings and paragraphs
-    -   Lists and bullet points
-    -   Multimedia content (videos and images)
-    -   Knowledge checks and quizzes
-    -   Interactive content (flashcards)
-    -   Course structure and metadata
+- Parse Articulate Rise JSON data from URLs or local files
+- Export to Markdown (.md) format
+- Export to HTML (.html) format with professional styling
+- Export to Word Document (.docx) format
+- Support for various content types:
+  - Text content with headings and paragraphs
+  - Lists and bullet points
+  - Multimedia content (videos and images)
+  - Knowledge checks and quizzes
+  - Interactive content (flashcards)
+  - Course structure and metadata
 
 ## Installation
 
 ### Prerequisites
 
--   Go, I don't know the version, but I have [![Go version](https://img.shields.io/github/go-mod/go-version/kjanat/articulate-parser?label=)][gomod] configured right now, and it works, see the [CI][Build] workflow where it is tested.
+- Go, I don't know the version, but I have [![Go version](https://img.shields.io/github/go-mod/go-version/kjanat/articulate-parser?label=)][gomod] configured right now, and it works, see the [CI][Build] workflow where it is tested.
 
 ### Install from source
 
@@ -124,7 +124,7 @@ go install github.com/kjanat/articulate-parser@latest
 
 The parser uses the following external library:
 
--   `github.com/fumiama/go-docx` - For creating Word documents (MIT license)
+- `github.com/fumiama/go-docx` - For creating Word documents (MIT license)
 
 ## Testing
 
@@ -164,25 +164,25 @@ go run main.go <input_uri_or_file> <output_format> [output_path]
 
 #### Examples
 
-1.  **Parse from URL and export to Markdown:**
+1. **Parse from URL and export to Markdown:**
 
 ```bash
 go run main.go "https://rise.articulate.com/share/N_APNg40Vr2CSH2xNz-ZLATM5kNviDIO#/" md
 ```
 
-2.  **Parse from local file and export to Word:**
+2. **Parse from local file and export to Word:**
 
 ```bash
 go run main.go "articulate-sample.json" docx "my-course.docx"
 ```
 
-3.  **Parse from local file and export to HTML:**
+3. **Parse from local file and export to HTML:**
 
 ```bash
 go run main.go "articulate-sample.json" html "output.html"
 ```
 
-4.  **Parse from local file and export to Markdown:**
+4. **Parse from local file and export to Markdown:**
 
 ```bash
 go run main.go "articulate-sample.json" md "output.md"
@@ -225,14 +225,14 @@ 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 |
-| `develop` | Development branch builds | Development/testing |
-| `feature/docker-ghcr` | Feature branch builds | Feature testing |
-| `master` | Latest master branch build | Continuous integration |
+| 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     |
+| `develop`             | Development branch builds                   | Development/testing    |
+| `feature/docker-ghcr` | Feature branch builds                       | Feature testing        |
+| `master`              | Latest master branch build                  | Continuous integration |
 
 ### Usage Examples
 
@@ -313,11 +313,11 @@ 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 |
+| Argument     | Description                           | Default        |
+| ------------ | ------------------------------------- | -------------- |
+| `VERSION`    | Version string embedded in the binary | `dev`          |
+| `BUILD_TIME` | Build timestamp                       | Current time   |
+| `GIT_COMMIT` | Git commit hash                       | Current commit |
 
 ### Docker Security
 
@@ -332,88 +332,88 @@ The Docker image supports the following build-time arguments:
 
 The project maintains high code quality standards:
 
--   Cyclomatic complexity ≤ 15 (checked with [gocyclo](https://github.com/fzipp/gocyclo))
--   Race condition detection enabled
--   Comprehensive test coverage
--   Code formatting with `gofmt`
--   Static analysis with `go vet`
+- Cyclomatic complexity ≤ 15 (checked with [gocyclo](https://github.com/fzipp/gocyclo))
+- Race condition detection enabled
+- Comprehensive test coverage
+- Code formatting with `gofmt`
+- Static analysis with `go vet`
 
 ### Contributing
 
-1.  Fork the repository
-2.  Create a feature branch
-3.  Make your changes
-4.  Run tests: `go test ./...`
-5.  Submit a pull request
+1. Fork the repository
+2. Create a feature branch
+3. Make your changes
+4. Run tests: `go test ./...`
+5. Submit a pull request
 
 ## Output Formats
 
 ### Markdown (`.md`)
 
--   Hierarchical structure with proper heading levels
--   Clean text content with HTML tags removed
--   Lists and bullet points preserved
--   Quiz questions with correct answers marked
--   Media references included
--   Course metadata at the top
+- Hierarchical structure with proper heading levels
+- Clean text content with HTML tags removed
+- Lists and bullet points preserved
+- Quiz questions with correct answers marked
+- Media references included
+- Course metadata at the top
 
 ### HTML (`.html`)
 
--   Professional styling with embedded CSS
--   Interactive and visually appealing layout
--   Proper HTML structure with semantic elements
--   Responsive design for different screen sizes
--   All content types beautifully formatted
--   Maintains course hierarchy and organization
+- Professional styling with embedded CSS
+- Interactive and visually appealing layout
+- Proper HTML structure with semantic elements
+- Responsive design for different screen sizes
+- All content types beautifully formatted
+- Maintains course hierarchy and organization
 
 ### Word Document (`.docx`)
 
--   Professional document formatting
--   Bold headings and proper typography
--   Bulleted lists
--   Quiz questions with answers
--   Media content references
--   Maintains course structure
+- Professional document formatting
+- Bold headings and proper typography
+- Bulleted lists
+- Quiz questions with answers
+- Media content references
+- Maintains course structure
 
 ## Supported Content Types
 
 The parser handles the following Articulate Rise content types:
 
--   **Text blocks**: Headings and paragraphs
--   **Lists**: Bullet points and numbered lists
--   **Multimedia**: Videos and images (references only)
--   **Knowledge Checks**: Multiple choice, multiple response, fill-in-the-blank, matching
--   **Interactive Content**: Flashcards and interactive scenarios
--   **Dividers**: Section breaks
--   **Sections**: Course organization
+- **Text blocks**: Headings and paragraphs
+- **Lists**: Bullet points and numbered lists
+- **Multimedia**: Videos and images (references only)
+- **Knowledge Checks**: Multiple choice, multiple response, fill-in-the-blank, matching
+- **Interactive Content**: Flashcards and interactive scenarios
+- **Dividers**: Section breaks
+- **Sections**: Course organization
 
 ## Data Structure
 
 The parser works with the standard Articulate Rise JSON format which includes:
 
--   Course metadata (title, description, settings)
--   Lesson structure
--   Content items with various types
--   Media references
--   Quiz/assessment data
--   Styling and layout information
+- Course metadata (title, description, settings)
+- Lesson structure
+- Content items with various types
+- Media references
+- Quiz/assessment data
+- Styling and layout information
 
 ## URL Pattern Recognition
 
 The parser automatically extracts share IDs from Articulate Rise URLs:
 
--   Input: `https://rise.articulate.com/share/N_APNg40Vr2CSH2xNz-ZLATM5kNviDIO#/`
--   API URL: `https://rise.articulate.com/api/rise-runtime/boot/share/N_APNg40Vr2CSH2xNz-ZLATM5kNviDIO`
+- Input: `https://rise.articulate.com/share/N_APNg40Vr2CSH2xNz-ZLATM5kNviDIO#/`
+- API URL: `https://rise.articulate.com/api/rise-runtime/boot/share/N_APNg40Vr2CSH2xNz-ZLATM5kNviDIO`
 
 ## Error Handling
 
 The parser includes error handling for:
 
--   Invalid URLs or share IDs
--   Network connection issues
--   Malformed JSON data
--   File I/O errors
--   Unsupported content types
+- Invalid URLs or share IDs
+- Network connection issues
+- Malformed JSON data
+- File I/O errors
+- Unsupported content types
 
 <!-- ## Code coverage
 
@@ -425,28 +425,28 @@ The parser includes error handling for:
 
 ## Limitations
 
--   Media files (videos, images) are referenced but not downloaded
--   Complex interactive elements may be simplified in export
--   Styling and visual formatting is not preserved
--   Assessment logic and interactivity is lost in static exports
+- Media files (videos, images) are referenced but not downloaded
+- Complex interactive elements may be simplified in export
+- Styling and visual formatting is not preserved
+- Assessment logic and interactivity is lost in static exports
 
 ## Performance
 
--   Lightweight with minimal dependencies
--   Fast JSON parsing and export
--   Memory efficient processing
--   No external license requirements
+- Lightweight with minimal dependencies
+- Fast JSON parsing and export
+- Memory efficient processing
+- No external license requirements
 
 ## Future Enhancements
 
 Potential improvements could include:
 
--   [ ]  PDF export support
--   [ ]  Media file downloading
--   [x]  ~~HTML export with preserved styling~~
--   [ ]  SCORM package support
--   [ ]  Batch processing capabilities
--   [ ]  Custom template support
+- [ ] PDF export support
+- [ ] Media file downloading
+- [x] ~~HTML export with preserved styling~~
+- [ ] SCORM package support
+- [ ] Batch processing capabilities
+- [ ] Custom template support
 
 ## License
 
@@ -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_GOARCH: '{{.ITEM.GOARCH}}'
+          TARGET_GOOS: "{{.ITEM.GOOS}}"
+          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}}'
-      GOARCH: '{{.TARGET_GOARCH}}'
+      GOOS: "{{.TARGET_GOOS}}"
+      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:
@@ -202,13 +210,13 @@ tasks:
         sh: gofmt -s -l .
     cmds:
       - |
-        {{if ne .UNFORMATTED ""}}
-        echo "❌ The following files need formatting:"
-        echo "{{.UNFORMATTED}}"
-        exit 1
-        {{else}}
-        echo "All files are properly formatted"
-        {{end}}
+          {{if ne .UNFORMATTED ""}}
+          echo "❌ The following files need formatting:"
+          echo "{{.UNFORMATTED}}"
+          exit 1
+          {{else}}
+          echo "All files are properly formatted"
+          {{end}}
 
   lint:staticcheck:
     desc: Run staticcheck (install if needed)
@@ -296,22 +304,22 @@ tasks:
     aliases: [db]
     cmds:
       - |
-        docker build \
-          --build-arg VERSION={{.VERSION}} \
-          --build-arg BUILD_TIME={{.BUILD_TIME}} \
-          --build-arg GIT_COMMIT={{.GIT_COMMIT}} \
-          -t {{.APP_NAME}}:{{.VERSION}} \
-          -t {{.APP_NAME}}:latest \
-          .
+          docker build \
+            --build-arg VERSION={{.VERSION}} \
+            --build-arg BUILD_TIME={{.BUILD_TIME}} \
+            --build-arg GIT_COMMIT={{.GIT_COMMIT}} \
+            -t {{.APP_NAME}}:{{.VERSION}} \
+            -t {{.APP_NAME}}:latest \
+            .
       - >
-        echo "Docker image built: {{.APP_NAME}}:{{.VERSION}}"
+          echo "Docker image built: {{.APP_NAME}}:{{.VERSION}}"
 
   docker:build:dev:
     desc: Build development Docker image
     cmds:
       - docker build -f Dockerfile.dev -t {{.APP_NAME}}:dev .
       - >
-        echo "Development Docker image built: {{.APP_NAME}}:dev"
+          echo "Development Docker image built: {{.APP_NAME}}:dev"
 
   docker:run:
     desc: Run Docker container
@@ -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,15 +418,15 @@ tasks:
     requires:
       vars: [VERSION]
     preconditions:
-      - sh: 'git diff --exit-code'
-        msg: 'Working directory is not clean'
-      - sh: 'git diff --cached --exit-code'
-        msg: 'Staging area is not clean'
+      - sh: "git diff --exit-code"
+        msg: "Working directory is not clean"
+      - sh: "git diff --cached --exit-code"
+        msg: "Staging area is not clean"
     cmds:
       - git tag -a v{{.VERSION}} -m "Release v{{.VERSION}}"
       - echo "Tagged v{{.VERSION}}"
       - >
-        echo "Push with: git push origin v{{.VERSION}}"
+          echo "Push with: git push origin v{{.VERSION}}"
 
   # Documentation tasks
   docs:serve:
@@ -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

@@ -2,13 +2,15 @@ module github.com/kjanat/articulate-parser
 
 go 1.24.0
 
+toolchain go1.25.5
+
 require (
 	github.com/fumiama/go-docx v0.0.0-20250506085032-0c30fd09304b
-	golang.org/x/net v0.46.0
-	golang.org/x/text v0.30.0
+	golang.org/x/net v0.48.0
+	golang.org/x/text v0.32.0
 )
 
 require (
 	github.com/fumiama/imgsz v0.0.4 // indirect
-	golang.org/x/image v0.32.0 // indirect
+	golang.org/x/image v0.34.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.32.0/go.mod h1:/R37rrQmKXtO6tYXAjtDLwQgFLHmhW+V6ayXlxzP2Pc=
-golang.org/x/net v0.46.0 h1:giFlY12I07fugqwPuWJi68oOnpfqFnJIJzaIIm2JVV4=
-golang.org/x/net v0.46.0/go.mod h1:Q9BGdFy1y4nkUwiLvT5qtyhAnEHgnQ/zd8PfU6nc210=
-golang.org/x/text v0.30.0 h1:yznKA/E9zq54KzlzBEAWn1NXSQ8DIp/NYMy88xJjl4k=
-golang.org/x/text v0.30.0/go.mod h1:yDdHFIX9t+tORqspjENWgzaCVXgk0yYnYuSZ8UzzBVM=
+golang.org/x/image v0.34.0 h1:33gCkyw9hmwbZJeZkct8XyR11yH889EQt/QH4VmXMn8=
+golang.org/x/image v0.34.0/go.mod h1:2RNFBZRB+vnwwFil8GkMdRvrJOFd1AzdZI6vOY+eJVU=
+golang.org/x/net v0.48.0 h1:zyQRTTrjc33Lhh0fBgT/H3oZq9WuvRR5gPC70xpDiQU=
+golang.org/x/net v0.48.0/go.mod h1:+ndRgGjkh8FGtu1w1FGbEC31if4VrNVMuKTgcAAnQRY=
+golang.org/x/text v0.32.0 h1:ZD01bjUt1FQ9WJ0ClOL5vxgxOI/sVCNgX1YtKwcY0mU=
+golang.org/x/text v0.32.0/go.mod h1:o/rUWzghvpD5TXrTIBuJU77MTaN0ljMWE47kxGJQ7jY=

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/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,173 +1,175 @@
 body {
-    font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, Oxygen, Ubuntu, Cantarell, sans-serif;
-    line-height: 1.6;
-    color: #333;
-    max-width: 800px;
-    margin: 0 auto;
-    padding: 20px;
-    background-color: #f9f9f9;
+  font-family:
+    -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, Oxygen, Ubuntu,
+    Cantarell, sans-serif;
+  line-height: 1.6;
+  color: #333;
+  max-width: 800px;
+  margin: 0 auto;
+  padding: 20px;
+  background-color: #f9f9f9;
 }
 header {
-    background: linear-gradient(135deg, #667eea 0%, #764ba2 100%);
-    color: white;
-    padding: 2rem;
-    border-radius: 10px;
-    margin-bottom: 2rem;
-    box-shadow: 0 4px 6px rgba(0, 0, 0, 0.1);
+  background: linear-gradient(135deg, #667eea 0%, #764ba2 100%);
+  color: white;
+  padding: 2rem;
+  border-radius: 10px;
+  margin-bottom: 2rem;
+  box-shadow: 0 4px 6px rgba(0, 0, 0, 0.1);
 }
 header h1 {
-    margin: 0;
-    font-size: 2.5rem;
-    font-weight: 300;
+  margin: 0;
+  font-size: 2.5rem;
+  font-weight: 300;
 }
 .course-description {
-    margin-top: 1rem;
-    font-size: 1.1rem;
-    opacity: 0.9;
+  margin-top: 1rem;
+  font-size: 1.1rem;
+  opacity: 0.9;
 }
 .course-info {
-    background: white;
-    padding: 1.5rem;
-    border-radius: 8px;
-    margin-bottom: 2rem;
-    box-shadow: 0 2px 4px rgba(0, 0, 0, 0.1);
+  background: white;
+  padding: 1.5rem;
+  border-radius: 8px;
+  margin-bottom: 2rem;
+  box-shadow: 0 2px 4px rgba(0, 0, 0, 0.1);
 }
 .course-info h2 {
-    margin-top: 0;
-    color: #4a5568;
-    border-bottom: 2px solid #e2e8f0;
-    padding-bottom: 0.5rem;
+  margin-top: 0;
+  color: #4a5568;
+  border-bottom: 2px solid #e2e8f0;
+  padding-bottom: 0.5rem;
 }
 .course-info ul {
-    list-style: none;
-    padding: 0;
+  list-style: none;
+  padding: 0;
 }
 .course-info li {
-    margin: 0.5rem 0;
-    padding: 0.5rem;
-    background: #f7fafc;
-    border-radius: 4px;
+  margin: 0.5rem 0;
+  padding: 0.5rem;
+  background: #f7fafc;
+  border-radius: 4px;
 }
 .course-section {
-    background: #4299e1;
-    color: white;
-    padding: 1.5rem;
-    border-radius: 8px;
-    margin: 2rem 0;
-    box-shadow: 0 2px 4px rgba(0, 0, 0, 0.1);
+  background: #4299e1;
+  color: white;
+  padding: 1.5rem;
+  border-radius: 8px;
+  margin: 2rem 0;
+  box-shadow: 0 2px 4px rgba(0, 0, 0, 0.1);
 }
 .course-section h2 {
-    margin: 0;
-    font-weight: 400;
+  margin: 0;
+  font-weight: 400;
 }
 .lesson {
-    background: white;
-    padding: 2rem;
-    border-radius: 8px;
-    margin: 2rem 0;
-    box-shadow: 0 2px 4px rgba(0, 0, 0, 0.1);
-    border-left: 4px solid #4299e1;
+  background: white;
+  padding: 2rem;
+  border-radius: 8px;
+  margin: 2rem 0;
+  box-shadow: 0 2px 4px rgba(0, 0, 0, 0.1);
+  border-left: 4px solid #4299e1;
 }
 .lesson h3 {
-    margin-top: 0;
-    color: #2d3748;
-    font-size: 1.5rem;
+  margin-top: 0;
+  color: #2d3748;
+  font-size: 1.5rem;
 }
 .lesson-description {
-    margin: 1rem 0;
-    padding: 1rem;
-    background: #f7fafc;
-    border-radius: 4px;
-    border-left: 3px solid #4299e1;
+  margin: 1rem 0;
+  padding: 1rem;
+  background: #f7fafc;
+  border-radius: 4px;
+  border-left: 3px solid #4299e1;
 }
 .item {
-    margin: 1.5rem 0;
-    padding: 1rem;
-    border-radius: 6px;
-    background: #fafafa;
-    border: 1px solid #e2e8f0;
+  margin: 1.5rem 0;
+  padding: 1rem;
+  border-radius: 6px;
+  background: #fafafa;
+  border: 1px solid #e2e8f0;
 }
 .item h4 {
-    margin-top: 0;
-    color: #4a5568;
-    font-size: 1.2rem;
-    text-transform: capitalize;
+  margin-top: 0;
+  color: #4a5568;
+  font-size: 1.2rem;
+  text-transform: capitalize;
 }
 .text-item {
-    background: #f0fff4;
-    border-left: 3px solid #48bb78;
+  background: #f0fff4;
+  border-left: 3px solid #48bb78;
 }
 .list-item {
-    background: #fffaf0;
-    border-left: 3px solid #ed8936;
+  background: #fffaf0;
+  border-left: 3px solid #ed8936;
 }
 .knowledge-check {
-    background: #e6fffa;
-    border-left: 3px solid #38b2ac;
+  background: #e6fffa;
+  border-left: 3px solid #38b2ac;
 }
 .multimedia-item {
-    background: #faf5ff;
-    border-left: 3px solid #9f7aea;
+  background: #faf5ff;
+  border-left: 3px solid #9f7aea;
 }
 .interactive-item {
-    background: #fff5f5;
-    border-left: 3px solid #f56565;
+  background: #fff5f5;
+  border-left: 3px solid #f56565;
 }
 .unknown-item {
-    background: #f7fafc;
-    border-left: 3px solid #a0aec0;
+  background: #f7fafc;
+  border-left: 3px solid #a0aec0;
 }
 .answers {
-    margin: 1rem 0;
+  margin: 1rem 0;
 }
 .answers h5 {
-    margin: 0.5rem 0;
-    color: #4a5568;
+  margin: 0.5rem 0;
+  color: #4a5568;
 }
 .answers ol {
-    margin: 0.5rem 0;
-    padding-left: 1.5rem;
+  margin: 0.5rem 0;
+  padding-left: 1.5rem;
 }
 .answers li {
-    margin: 0.3rem 0;
-    padding: 0.3rem;
+  margin: 0.3rem 0;
+  padding: 0.3rem;
 }
 .correct-answer {
-    background: #c6f6d5;
-    border-radius: 3px;
-    font-weight: bold;
+  background: #c6f6d5;
+  border-radius: 3px;
+  font-weight: bold;
 }
 .correct-answer::after {
-    content: " ✓";
-    color: #38a169;
+  content: " ✓";
+  color: #38a169;
 }
 .feedback {
-    margin: 1rem 0;
-    padding: 1rem;
-    background: #edf2f7;
-    border-radius: 4px;
-    border-left: 3px solid #4299e1;
-    font-style: italic;
+  margin: 1rem 0;
+  padding: 1rem;
+  background: #edf2f7;
+  border-radius: 4px;
+  border-left: 3px solid #4299e1;
+  font-style: italic;
 }
 .media-info {
-    background: #edf2f7;
-    padding: 1rem;
-    border-radius: 4px;
-    margin: 0.5rem 0;
+  background: #edf2f7;
+  padding: 1rem;
+  border-radius: 4px;
+  margin: 0.5rem 0;
 }
 .media-info strong {
-    color: #4a5568;
+  color: #4a5568;
 }
 hr {
-    border: none;
-    height: 2px;
-    background: linear-gradient(to right, #667eea, #764ba2);
-    margin: 2rem 0;
-    border-radius: 1px;
+  border: none;
+  height: 2px;
+  background: linear-gradient(to right, #667eea, #764ba2);
+  margin: 2rem 0;
+  border-radius: 1px;
 }
 ul {
-    padding-left: 1.5rem;
+  padding-left: 1.5rem;
 }
 li {
-    margin: 0.5rem 0;
+  margin: 0.5rem 0;
 }

internal/exporters/markdown.go

@@ -96,7 +96,10 @@ 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
+	itemType := strings.ToLower(item.Type)
+
+	switch itemType {
 	case "text":
 		e.processTextItem(buf, item, headingPrefix)
 	case "list":
@@ -105,7 +108,7 @@ func (e *MarkdownExporter) processItemToMarkdown(buf *bytes.Buffer, item models.
 		e.processMultimediaItem(buf, item, headingPrefix)
 	case "image":
 		e.processImageItem(buf, item, headingPrefix)
-	case "knowledgeCheck":
+	case "knowledgecheck":
 		e.processKnowledgeCheckItem(buf, item, headingPrefix)
 	case "interactive":
 		e.processInteractiveItem(buf, item, headingPrefix)

internal/services/parser.go

@@ -15,6 +15,9 @@ import (
 	"github.com/kjanat/articulate-parser/internal/models"
 )
 
+// 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 {
@@ -78,15 +81,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)
@@ -133,8 +136,7 @@ func (p *ArticulateParser) extractShareID(uri string) (string, error) {
 		return "", fmt.Errorf("invalid domain for Articulate Rise URI: %s", parsedURL.Host)
 	}
 
-	re := regexp.MustCompile(`/share/([a-zA-Z0-9_-]+)`)
-	matches := re.FindStringSubmatch(uri)
+	matches := shareIDRegex.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.