Skip to content

chore(deps): autoupdate (needs claude fix)#22

Open
github-actions[bot] wants to merge 4 commits into
mainfrom
chore/autoupdate-28272165725
Open

chore(deps): autoupdate (needs claude fix)#22
github-actions[bot] wants to merge 4 commits into
mainfrom
chore/autoupdate-28272165725

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

The dependency autoupdater failed during builds-and-checks.

Run log: https://github.com/dudko-dev/agent/actions/runs/28272165725

A Claude session has been dispatched to push fixes onto this branch
so the following commands all exit 0:

  npm run typecheck
  npm run format:check
  npm run build
  npm test

Claude will leave a status comment on this PR when it finishes.
PR-checks will re-run on each new commit.

@github-actions github-actions Bot requested a review from siarheidudko June 27, 2026 00:06
@github-actions github-actions Bot added autoupdate Automated dependency update PRs needs-claude Needs Claude GitHub App to fix labels Jun 27, 2026
github-actions Bot added 3 commits June 27, 2026 00:10
…ider conflict

workers-ai-provider@3.2.0 declares ai@^6 as a peer dependency, but the
project now requires ai@^7. The packages are functionally compatible (all
130 tests pass), so --legacy-peer-deps bypasses the spurious semver
conflict until workers-ai-provider publishes a version that supports ai@^7.
…lict

workers-ai-provider@3.2.x declares ai@^6 as a peer dependency, but the
project now uses ai@^7. The packages are functionally compatible (all
130 tests pass). The overrides field tells npm to use the project-level
ai version when resolving workers-ai-provider, silencing the peer dep
conflict so that npm ci succeeds without --legacy-peer-deps.
@claude

claude Bot commented Jun 27, 2026

Copy link
Copy Markdown

Autoupdate fix summary

Initial failures: npm ci failed (exit 1) because workers-ai-provider@3.2.0 declares ai@^6 as a peer dependency, but the autoupdater bumped ai to ^7.0.2. npm ci in npm v7+ enforces peer dep constraints strictly, so install failed before any of the four checks could run.

Root cause: The autoupdater bumped ai to ^7.0.2 (a major bump) while workers-ai-provider@3.2.x hasn't yet been updated to declare support for ai@^7. The packages are functionally compatible — once installed (e.g. with --legacy-peer-deps), all 130 tests pass.

Fix applied: Added an overrides block to package.json:

"overrides": {
  "workers-ai-provider": {
    "ai": "$ai"
  }
}

This tells npm to use the project-level ai version when resolving workers-ai-provider's peer dep, so npm ci succeeds without any flags.

All four checks pass locally:

  • npm run typecheck
  • npm run format:check
  • npm run build
  • npm test ✅ (130/130 pass)

Commits pushed: Two intermediate commits (a failed workflow approach + its revert) plus the real fix commit 314d29e.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autoupdate Automated dependency update PRs needs-claude Needs Claude GitHub App to fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant