feat: surface 429s as typed BugSplatRateLimitError (#168)

* feat: surface 429s as typed BugSplatRateLimitError

Rate-limited (HTTP 429) responses were thrown as a generic Error with a
"too many requests" message, leaving consumers to match on the message
string to detect them.

Add a BugSplatRateLimitError (isRateLimitError marker + status) and throw
it from the three presigned-URL sites (symbols/versions getPresignedUrl,
crash-post getCrashUploadUrl) so consumers can detect rate limiting by
type. Exported via the common/client barrel.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* test: cover 429 in crash-post-client; drop es5 trailing comma

Addresses Copilot review feedback on #168:
- Remove trailing comma after the BugSplatRateLimitError `status`
  constructor param (incompatible with Prettier `trailingComma: es5`).
- Add a spec asserting CrashPostClient throws BugSplatRateLimitError
  with status 429 when getCrashUploadUrl returns 429.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* ci: bump actions to v6 and Node 24 to clear node20 deprecation

- actions/checkout@main -> @v6, actions/setup-node@v4 -> @v6 (both now
  run on the Node 24 runtime, clearing the node20 deprecation warning).
- node-version 22.x -> 24.x.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* chore: drop husky/commitlint for CI-based PR title linting

Husky's hook bootstrap is deprecated (fails in v10), and its commit-msg
hook only linted individual branch commits that get squashed away on
merge. Since PRs squash-merge with the PR title as the commit subject,
that title is what semantic-release parses for version bumps.

- Remove husky (dep, `prepare` script, .husky/, core.hooksPath).
- Remove @commitlint/cli, @commitlint/config-conventional, .commitlintrc.js
  (only ever invoked by the husky commit-msg hook).
- Add a PR Title workflow using amannn/action-semantic-pull-request to
  validate the PR title against Conventional Commits.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: bobbyg603

.commitlintrc.js

@@ -0,0 +1,24 @@
+name: PR Title
+
+on:
+  pull_request:
+    types:
+      - opened
+      - edited
+      - synchronize
+    branches:
+      - main
+
+permissions:
+  pull-requests: read
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+
+    steps:
+      # The PR title becomes the squash commit subject that semantic-release
+      # parses, so validate it follows Conventional Commits.
+      - uses: amannn/action-semantic-pull-request@v6
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/pr-title.yml

@@ -10,13 +10,13 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@main
+      - uses: actions/checkout@v6
         with:
           persist-credentials: false
-      
-      - uses: actions/setup-node@v4
+
+      - uses: actions/setup-node@v6
         with:
-          node-version: "22.x"
+          node-version: "24.x"
 
       - name: Install
         run: npm install

.github/workflows/release.yml

@@ -10,14 +10,14 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@main
+      - uses: actions/checkout@v6
         with:
           persist-credentials: false
-      
-      # Use latest Node.js 22 LTS
-      - uses: actions/setup-node@v4
+
+      # Use latest Node.js 24 LTS
+      - uses: actions/setup-node@v6
         with:
-          node-version: "22.x"
+          node-version: "24.x"
 
       - name: Install
         run: npm install