Standardize project configuration

pdugan20 · pdugan20 · commit e97dbf5c1060 · 2026-03-04T10:22:05.000-08:00
- Standardize README badges (CI, Node.js, License) and remove redundant license section
- Fix 8 stale file references in CLAUDE.md
- Standardize Dependabot config with grouping, labels, assignees
- Add Dependabot auto-merge workflow for patch/minor updates
- Add claudelint to CI pipeline, install config and SessionStart hook
- Add .claudelint-cache/ to .gitignore
diff --git a/.claude/hooks/hooks.json b/.claude/hooks/hooks.json
@@ -0,0 +1,14 @@
+{
+  "hooks": {
+    "SessionStart": [
+      {
+        "hooks": [
+          {
+            "type": "command",
+            "command": "npx claudelint check-all --format json"
+          }
+        ]
+      }
+    ]
+  }
+}
diff --git a/.claudelintignore b/.claudelintignore
@@ -0,0 +1,11 @@
+# Dependencies
+node_modules/
+vendor/
+
+# Build outputs
+dist/
+build/
+
+# Testing
+coverage/
+*.log
diff --git a/.claudelintrc.json b/.claudelintrc.json
@@ -0,0 +1,3 @@
+{
+  "extends": "claudelint:recommended"
+}
diff --git a/.github/dependabot.yml b/.github/dependabot.yml
@@ -1,44 +1,56 @@
 version: 2
 updates:
-  # Enable version updates for npm
   - package-ecosystem: "npm"
     directory: "/"
     schedule:
       interval: "weekly"
       day: "monday"
+      time: "06:00"
+      timezone: "America/New_York"
     open-pull-requests-limit: 10
-    groups:
-      # Group all TypeScript ESLint packages together
-      typescript-eslint:
-        patterns:
-          - "@typescript-eslint/*"
-      # Group all Jest/testing packages together
-      jest:
-        patterns:
-          - "jest"
-          - "@types/jest"
-          - "ts-jest"
-      # Group Puppeteer packages together
-      puppeteer:
-        patterns:
-          - "puppeteer*"
-      # Group ESLint packages together
-      eslint:
-        patterns:
-          - "eslint*"
-        exclude-patterns:
-          - "@typescript-eslint/*"
-    # Automatically merge patch and minor updates for dev dependencies
+    commit-message:
+      prefix: "deps"
+      prefix-development: "deps-dev"
+      include: "scope"
     labels:
       - "dependencies"
-      - "automated"
+      - "npm"
+    assignees:
+      - "pdugan20"
+    reviewers:
+      - "pdugan20"
+    groups:
+      dev-dependencies:
+        dependency-type: "development"
+        update-types:
+          - "minor"
+          - "patch"
+      production-dependencies:
+        dependency-type: "production"
+        update-types:
+          - "minor"
+          - "patch"
 
-  # Enable version updates for GitHub Actions
   - package-ecosystem: "github-actions"
     directory: "/"
     schedule:
       interval: "weekly"
       day: "monday"
+      time: "06:00"
+      timezone: "America/New_York"
+    open-pull-requests-limit: 5
+    commit-message:
+      prefix: "ci"
     labels:
+      - "dependencies"
       - "github-actions"
-      - "automated"
+    assignees:
+      - "pdugan20"
+    reviewers:
+      - "pdugan20"
+    groups:
+      github-actions:
+        patterns: ["*"]
+        update-types:
+          - "minor"
+          - "patch"
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -43,6 +43,9 @@ jobs:
           flags: unittests
           name: codecov-umbrella
 
+      - name: Lint Claude Code config
+        run: npx claudelint
+
       - name: Build
         run: npm run build
 
diff --git a/.github/workflows/dependabot-auto-merge.yml b/.github/workflows/dependabot-auto-merge.yml
@@ -0,0 +1,32 @@
+name: Dependabot Auto-Merge
+
+on: pull_request
+
+permissions:
+  contents: write
+  pull-requests: write
+
+jobs:
+  auto-merge:
+    runs-on: ubuntu-latest
+    if: github.event.pull_request.user.login == 'dependabot[bot]'
+    steps:
+      - name: Fetch Dependabot metadata
+        id: metadata
+        uses: dependabot/fetch-metadata@v2
+        with:
+          github-token: "${{ secrets.GITHUB_TOKEN }}"
+
+      - name: Approve PR
+        if: steps.metadata.outputs.update-type == 'version-update:semver-minor' || steps.metadata.outputs.update-type == 'version-update:semver-patch'
+        run: gh pr review --approve "$PR_URL"
+        env:
+          PR_URL: ${{ github.event.pull_request.html_url }}
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+
+      - name: Enable auto-merge
+        if: steps.metadata.outputs.update-type == 'version-update:semver-minor' || steps.metadata.outputs.update-type == 'version-update:semver-patch'
+        run: gh pr merge --auto --squash "$PR_URL"
+        env:
+          PR_URL: ${{ github.event.pull_request.html_url }}
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
diff --git a/.gitignore b/.gitignore
@@ -48,3 +48,4 @@ Thumbs.db
 # Testing
 coverage/
 .nyc_output/
+.claudelint-cache/
diff --git a/CLAUDE.md b/CLAUDE.md
@@ -46,9 +46,9 @@ npm run extension:lint            # Lint with warnings-as-errors
 - `src/commands/list.ts` - List downloaded books
 - `src/commands/tag.ts` - Tag MP3 files with metadata
 - `src/commands/merge.ts` - Merge chapters into M4B audiobook
-- `src/metadata/embedder.ts` - ID3 tag embedding
+- `src/services/metadata-service.ts` - ID3 tag embedding
 - `src/services/merge-service.ts` - M4B audiobook merging with FFmpeg
-- `src/utils/books.ts` - Book discovery and status checking
+- `src/services/book-service.ts` - Book discovery and status checking
 - `src/utils/logger.ts` - Logging utilities
 
 **Chrome Extension (TypeScript Source):**
@@ -106,7 +106,7 @@ npm run extension:lint            # Lint with warnings-as-errors
    - Saves metadata.json alongside MP3 files
 
 2. **CLI Tags (Optional):**
-   - CLI auto-discovers books in `~/Downloads/libby-downloads/`
+   - CLI auto-discovers books in ~/Downloads/libby-downloads/
    - Reads metadata.json from book folders
    - Embeds ID3 tags into MP3 files (title, author, narrator, cover art)
    - Shows book status (tagged/untagged)
@@ -174,11 +174,11 @@ export const DEBUG_MODE = true; // Set to false for production
 
 **Test structure:** `src/__tests__/`
 
-- `__tests__/mocks/` - Chrome API mocks
-- `__tests__/shared/` - Shared utility tests (validators, logger, errors)
-- `__tests__/background/` - Background service tests (download-service, download-tracker)
-- `__tests__/types/` - Type definition tests
-- `utils/__tests__/` - CLI utility tests
+- `src/__tests__/mocks/` - Chrome API mocks
+- `src/__tests__/shared/` - Shared utility tests (validators, logger, errors)
+- `src/__tests__/background/` - Background service tests (download-service, download-tracker)
+- `src/__tests__/types/` - Type definition tests
+- `src/utils/__tests__/` - CLI utility tests
 
 **Test environment:** Jest with jsdom (for browser-like environment)
 
diff --git a/README.md b/README.md
@@ -1,8 +1,8 @@
 # Libby Downloader
 
-[![CI](https://github.com/pdugan20/libby-downloader/actions/workflows/ci.yml/badge.svg)](https://github.com/pdugan20/libby-downloader/actions/workflows/ci.yml)
-[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
-[![Node.js Version](https://img.shields.io/badge/node-%3E%3D18.0.0-brightgreen)](https://nodejs.org/)
+[![CI](https://github.com/pdugan20/libby-downloader/workflows/CI/badge.svg)](https://github.com/pdugan20/libby-downloader/actions)
+[![Node.js](https://img.shields.io/node/v/next?logo=node.js)](https://nodejs.org)
+[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 
 Download audiobooks from Libby to your computer for offline listening.
 
@@ -206,6 +206,3 @@ This tool is provided for **educational purposes only**. Users are responsible f
 - Complying with their library's terms of service
 - Respecting copyright and licensing agreements
 
-## License
-
-MIT - See [LICENSE](LICENSE)
diff --git a/package.json b/package.json
@@ -26,7 +26,9 @@
     "check-all": "npm run typecheck && npm run lint && npm run format:check && npm run test",
     "prepare": "husky",
     "extension:lint": "web-ext lint --source-dir=chrome-extension --warnings-as-errors",
-    "extension:validate": "node scripts/validate-extension.js"
+    "extension:validate": "node scripts/validate-extension.js",
+    "lint:claude": "claudelint check-all",
+    "lint:claude:fix": "claudelint format --fix"
   },
   "keywords": [
     "libby",

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+{`
	`2`	`+ "extends": "claudelint:recommended"`
	`3`	`+}`