Skip to content

Update dependabot.yml#2447

Merged
robert-inkeep merged 1 commit intomainfrom
robert-inkeep-patch-2
Feb 27, 2026
Merged

Update dependabot.yml#2447
robert-inkeep merged 1 commit intomainfrom
robert-inkeep-patch-2

Conversation

@robert-inkeep
Copy link
Copy Markdown
Collaborator

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agents-api Building Building Preview, Comment Feb 27, 2026 11:46pm
agents-docs Building Building Preview, Comment Feb 27, 2026 11:46pm
agents-manage-ui Building Building Preview, Comment Feb 27, 2026 11:46pm

Request Review

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Feb 27, 2026

⚠️ No Changeset found

Latest commit: 4290979

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@pullfrog
Copy link
Copy Markdown
Contributor

pullfrog Bot commented Feb 27, 2026

Bumps the create-agents-template Dependabot config from weekly to daily checks and tightens the dependency scope.

  • .github/dependabot.yml: Changed schedule interval from weekly to daily for the create-agents-template npm ecosystem
  • .github/dependabot.yml: Removed the allow filter for @inkeep/* (Dependabot will now rely solely on the group pattern)
  • .github/dependabot.yml: Narrowed the group pattern from @inkeep/* to @inkeep/agents-* to avoid pulling in unrelated Inkeep packages

Pullfrog  | View workflow run | Using Claude Code | Triggered by Pullfrogpullfrog.com𝕏

Copy link
Copy Markdown
Contributor

@pullfrog pullfrog Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. The three changes are straightforward and consistent:

  1. weeklydaily — faster pickup of new @inkeep/agents-* releases in the template.
  2. allow block removed — Dependabot now tracks all deps in create-agents-template, not just @inkeep/*. Given the template has only ~10 deps (mostly @inkeep/agents-* plus standard build tooling), this is fine and avoids silently missing updates to things like zod, drizzle-kit, etc.
  3. Group pattern @inkeep/*@inkeep/agents-* — tighter grouping that matches the actual dependency names (agents-core, agents-sdk, agents-cli, agents-manage-ui).

Pullfrog  | View workflow runpullfrog.com𝕏

Copy link
Copy Markdown
Contributor

@claude claude Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Review Summary

(1) Total Issues | Risk: Medium

🟠⚠️ Major (1) 🟠⚠️

🟠 1) .github/dependabot.yml Removing allow filter expands scope to ALL npm dependencies

Issue: The previous configuration had an allow filter that restricted Dependabot to only updating @inkeep/* packages. By removing lines 31-32 (allow: - dependency-name: '@inkeep/*'), Dependabot will now propose updates for all npm dependencies in /create-agents-template, not just @inkeep packages.

Why: This is a significant behavioral change. The affected packages now include: dotenv, zod, @tailwindcss/vite, tailwindcss, concurrently, drizzle-kit, mprocs, pg, tsx, turbo, @types/node, and any transitive dependencies. Combined with the schedule change from weekly to daily, this could result in a substantial increase in Dependabot PR volume.

Fix:

  • If intentional (you want all dependencies updated daily): This is valid but document this decision in the PR description and ensure the team has capacity for increased PR volume. Consider enabling auto-merge for minor/patch updates.
  • If unintentional (you only wanted to update @inkeep packages more frequently): Re-add the allow filter:
    allow:
      - dependency-name: '@inkeep/*'

Refs:

💭 Consider (2) 💭

💭 1) .github/dependabot.yml:32 Daily schedule significantly increases PR volume
Issue: Changing from weekly to daily for all dependencies (with allow removed) may generate many more PRs.
Why: Higher maintenance burden; ensure this aligns with team capacity.
Fix: Consider auto-merge for minor/patch, or keep allow filter if only @inkeep packages need daily updates.

💭 2) .github/dependabot.yml:34-36 Narrowed group pattern may not future-proof
Issue: Pattern changed from @inkeep/* to @inkeep/agents-*. Currently matches all 4 packages, but future @inkeep packages (e.g., @inkeep/ai-sdk-provider) wouldn't be grouped.
Why: Could cause ungrouped PRs later if template adds non-agents packages.
Fix: Consider keeping @inkeep/* for the group pattern if future packages are expected.


🚫 REQUEST CHANGES

Summary: The removal of the allow filter appears to significantly change the intended scope of Dependabot updates. Please clarify whether updating all npm dependencies daily (not just @inkeep packages) is the intended behavior. If only @inkeep packages should be updated with the new daily schedule, the allow filter should be restored.

Discarded (0)

No findings were discarded.

Reviewers (1)
Reviewer Returned Main Findings Consider While You're Here Inline Comments Pending Recs Discarded
pr-review-devops 4 1 2 0 0 0 1
Total 4 1 2 0 0 0 1

Note: One finding was self-corrected by the reviewer (pattern matching validation).

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