Skip to content

Fix CI deployment workflow by adding @cloudflare/workers-types#151

Merged
jaypatrick merged 2 commits intomasterfrom
fix/ci-cloudflare-workers-types
Jan 14, 2026
Merged

Fix CI deployment workflow by adding @cloudflare/workers-types#151
jaypatrick merged 2 commits intomasterfrom
fix/ci-cloudflare-workers-types

Conversation

@jaypatrick
Copy link
Owner

Problem

The deployment workflow was failing with:

error: Failed resolving types. Could not resolve "@cloudflare/workers-types", but found it in a package.json. Deno expects the node_modules/ directory to be up to date. Did you forget to run `deno install`?

Solution

  • Added @cloudflare/workers-types to deno.json imports as an npm dependency
  • Added deno install step before type checking in CI workflow to ensure npm dependencies are installed
  • Updated type check command to explicitly include worker/tail.ts

Testing

Verified locally that:

  • deno install successfully installs the types
  • deno check worker/tail.ts passes without errors
  • deno check src/index.ts still works correctly

Co-Authored-By: Warp agent@warp.dev

jaypatrick and others added 2 commits January 13, 2026 23:20
- Add @cloudflare/workers-types to deno.json imports as npm dependency
- Add deno install step before type checking in CI workflow
- Update type check to include worker/tail.ts
- Resolves: Failed resolving types error in deployment workflow

Co-Authored-By: Warp <agent@warp.dev>
@github-actions
Copy link

Version bumped: 0.7.18 -> 0.7.19

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 14, 2026

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 successful!
View logs
adblock-compiler 777ad27 Jan 14 2026, 04:21 AM

@jaypatrick jaypatrick merged commit d41c6cd into master Jan 14, 2026
1 check passed
@jaypatrick jaypatrick deleted the fix/ci-cloudflare-workers-types branch January 14, 2026 04:23
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