Skip to content

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

Merged
siarheidudko merged 4 commits into
mainfrom
chore/autoupdate-28272165725
Jun 27, 2026
Merged

chore(deps): autoupdate (needs claude fix)#22
siarheidudko merged 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.

@siarheidudko siarheidudko merged commit 9150e9a into main Jun 27, 2026
5 checks passed
@siarheidudko siarheidudko deleted the chore/autoupdate-28272165725 branch June 27, 2026 07:33
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