Skip to content

fix(deps): repair broken Workers build — pin eslint 9, bump workers-types v5 - #8

Open
Aswinmcw wants to merge 3 commits into
mainfrom
fix/eslint9-workers-types-v5
Open

fix(deps): repair broken Workers build — pin eslint 9, bump workers-types v5#8
Aswinmcw wants to merge 3 commits into
mainfrom
fix/eslint9-workers-types-v5

Conversation

@Aswinmcw

Copy link
Copy Markdown
Member

Problem

main is red: both Workers Builds: shiptrack and Workers Builds: shiptrack-poller are failing. Three Dependabot PRs auto-merged this morning; two of them broke npm ci (Cloudflare runs npm ci before build/deploy, and it runs post-merge, so it didn't gate the merges):

Either alone fails npm ci with ERESOLVE.

Fix

  • Pin eslint back to ^9.39.4 (the pre-bump version).
  • Bump @cloudflare/workers-types to ^5.20260722.1 — its latest, and the intended companion to wrangler 4.11x. @opennextjs/cloudflare declares no peer on it, so this is safe.
  • Regenerated package-lock.json.

Verified locally

  • npm ci resolves cleanly ✅
  • next build succeeds ✅
  • opennextjs-cloudflare build (the exact Cloudflare deploy step) exits 0 ✅

This repairs both the web and poller Workers builds (shared install). eslint 10 should return only once Next 16 lands (see #3).

🤖 Generated with Claude Code

…o v5

Two Dependabot bumps that auto-merged today broke the Cloudflare Workers
build on main (npm ci fails with ERESOLVE, so both `shiptrack` and
`shiptrack-poller` deploys fail):

- eslint 10.8.0 (#5) — eslint-config-next@15 peer-requires eslint ^9;
  eslint 10 has no supporting config-next until Next 16. Pin back to ^9.39.4.
- wrangler 4.114.0 (#4) — peer-requires @cloudflare/workers-types ^5, but
  the project pinned ^4. Bump workers-types to ^5.20260722.1 (its latest,
  and the intended companion to wrangler 4.11x; opennextjs has no peer on it).

Verified locally: `npm ci` resolves, `next build` and
`opennextjs-cloudflare build` both succeed (exit 0).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@Aswinmcw
Aswinmcw requested review from a team and Aswin-coder as code owners July 27, 2026 08:01

@Aswincloud-Bot Aswincloud-Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Auto-approved: @Aswinmcw is a member of @Aswincloud/admins.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 27, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
shiptrack 9204433 Commit Preview URL

Branch Preview URL
Jul 27 2026, 08:20 AM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 27, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
shiptrack-poller 9204433 Jul 27 2026, 08:21 AM

Aswin-coder and others added 2 commits July 27, 2026 08:10
The npm ci ERESOLVE break is fixed (web build green); re-running the
poller build to confirm it recovers on a clean install.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The eslint-9 pin fixed npm ci and recovered the web (opennextjs) build,
but Workers Builds: shiptrack-poller stayed red. The poller is the only
build that deploys via raw `wrangler deploy`, and wrangler 4.114.0 (#4) is
the sole poller-relevant change since the last green poller build (07-05,
locked at wrangler 4.95.0). Pin wrangler back to 4.95.0 (last-known-good);
this also drops the forced @cloudflare/workers-types v5 bump, so revert
that to ^4 as well.

Local: npm ci clean, tsc --noEmit passes, `wrangler deploy --dry-run` for
both web and poller exit 0.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.

3 participants