Skip to content

Document Dependabot workflow and dependency conventions - #65

Merged
mitchallen merged 1 commit into
mainfrom
docs/dependabot-notes
Aug 5, 2026
Merged

Document Dependabot workflow and dependency conventions#65
mitchallen merged 1 commit into
mainfrom
docs/dependabot-notes

Conversation

@mitchallen

Copy link
Copy Markdown
Owner

Adds a Dependencies & Dependabot section to CLAUDE.md, capturing the decisions from the recent dependency work that aren't derivable from reading the code.

Note Why it's worth recording
dependabot.yml grouping policy Majors stay individual on purpose — someone "tidying" the config could silently start batching an Express or TypeScript major into a routine PR
Transitive CVEs pinned via overrides Including the nested copyfiles → minimatch → brace-expansion entry, which exists because copyfiles pins an old minimatch the root override can't reach. Plus the rule to bump the existing pin rather than add a second one — that pin has moved twice already (5.0.6 → 5.0.7 → 5.0.9)
LTS-only Docker base image Node 25 was declined in #60; explains why it was closed without an ignore rule, so Node 26 LTS still gets a PR
TS7016 on untyped JS deps TypeScript 7 rejects what 5.x inferred as any — the reason @mitchallen/uptime is vendored as src/uptime.ts, with a note not to re-add it

Also documents that version-bump commits go straight to main (bare 2.x.x message), the one existing exception to the branch-and-PR rule.

Every factual claim was verified against the current tree: the overrides block, the 24-alpine base image, the three configured ecosystems, src/uptime.ts, and the pin's git history.

Docs only — no source or config changes.

🤖 Generated with Claude Code

Adds a "Dependencies & Dependabot" section covering the things that
aren't derivable from reading the code:

- the dependabot.yml grouping policy (majors stay individual on purpose)
- why transitive CVEs are pinned with `overrides`, including the nested
  copyfiles → minimatch entry the root override can't reach, and the rule
  to bump the existing pin instead of adding a second one
- the LTS-only policy for the Docker base image, and why #60 (Node 25)
  was closed without an `ignore` rule
- that untyped JS deps now fail under TypeScript 7 with TS7016, which is
  why @mitchallen/uptime is vendored as src/uptime.ts

Also notes that version-bump commits go straight to main, which is the
one documented exception to the branch-and-PR rule.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@mitchallen
mitchallen merged commit 842d8dc into main Aug 5, 2026
1 check passed
@mitchallen
mitchallen deleted the docs/dependabot-notes branch August 5, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant