Skip to content

Clarify dependency removal PR policy in AGENTS.md#228

Merged
alexkroman merged 2 commits into
mainfrom
claude/serene-bardeen-tz5vu2
Jun 17, 2026
Merged

Clarify dependency removal PR policy in AGENTS.md#228
alexkroman merged 2 commits into
mainfrom
claude/serene-bardeen-tz5vu2

Conversation

@alexkroman

Copy link
Copy Markdown
Collaborator

Summary

Updated the dependency management guidance in AGENTS.md to explicitly clarify that removing dependencies requires a dedicated PR, even when the removal is paired with deleting the code that used the dependency.

Changes

  • Expanded the dependency change policy to explicitly mention "remove" alongside "add" and "bump"
  • Added clarification that dependency removals rewrite uv.lock and therefore warrant their own PR to avoid merge conflicts with feature branches
  • Emphasized that this applies even when a dependency removal is coupled with deleting the code that depended on it

Rationale

The original guidance only mentioned adding or bumping dependencies as requiring dedicated PRs. This update makes explicit that the same principle applies to removals, since uv.lock changes are the root cause of merge conflicts — not the semantic nature of the change. This prevents feature branches from colliding in the lockfile when one removes a dependency and another adds one.

https://claude.ai/code/session_01BBoBygev93Rkj1AcowNsjA

@alexkroman alexkroman enabled auto-merge June 17, 2026 20:57
@alexkroman alexkroman added this pull request to the merge queue Jun 17, 2026
Merged via the queue into main with commit ca6541c Jun 17, 2026
19 checks passed
@alexkroman alexkroman deleted the claude/serene-bardeen-tz5vu2 branch June 17, 2026 21:21
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.

2 participants