My mentality: `dependencies` are for _runtime_ things, if you've got an app that runs and uses `expr

My mentality:

dependencies

dependencies

are for runtime things, if you've got an app that runs and uses

express

express

for example.

devDependencies

devDependencies

for dev tooling, or anything that's only used to build, like everything here, since it's all bundled into a single output worker.

DaniFoldi•6/19/23, 10:02 PM

I keep dependencies required in the main build step in regular, everything else in dev, so the main build step only needs to install non-dev stuff

JJames My mentality: `dependencies` are for _runtime_ things, if you've got an app that...

JamesOP•6/19/23, 10:02 PM

But I know people have differing opinions on that

DaniFoldi•6/19/23, 10:03 PM

Yup

JamesOP•6/19/23, 10:03 PM

Yeah, in your case I would personally throw it all in

devDependencies

devDependencies

, but you're not publishing to npm... so it doesn't overly matter.

JamesOP•6/19/23, 10:04 PM

perfect haha

kian•6/19/23, 10:04 PM

that's a James move

JamesOP•6/19/23, 10:06 PM

we're always happy to help!! Love to see it

captain•6/20/23, 2:42 AM

hey what's up team...trying to update from wrangler 2.20.0 to the latest...but running: sudo npm install -g wrangler sudo npm install -g wrangler but getting the following starting error:

npm ERR! code 1
npm ERR! path /Users/dev/.nvm/versions/node/v19.0.1/lib/node_modules/wrangler/node_modules/better-sqlite3
npm ERR! command failed

Any ideas?

captain•6/20/23, 2:49 AM

runing via m1 latest version of os

captain•6/20/23, 3:17 AM

follow up on this:
% sudo npm install -g wrangler
npm WARN deprecated rollup-plugin-inject@3.0.2: This package has been deprecated and is no longer maintained. Please use @rollup/plugin-inject.
npm WARN deprecated sourcemap-codec@1.4.8: Please use @jridgewell/sourcemap-codec instead
npm ERR! code 1
npm ERR! path /Users/dev/.nvm/versions/node/v19.0.1/lib/node_modules/wrangler/node_modules/better-sqlite3
npm ERR! command failed

captain•6/20/23, 5:13 AM

bueler...

Ccaptain hey what's up team...trying to update from wrangler 2.20.0 to the latest...but r...

Max (@rozenmd)•6/20/23, 8:48 AM

for some reason node 19 doesn't work, can you try node 18 or 20?

Erwin•6/20/23, 9:13 AM

Would someone be able to approve the test workflows on my PR for wrangler? https://github.com/cloudflare/workers-sdk/pull/3497

kian•6/20/23, 9:33 AM

Looks like it's been reported on

workerd

workerd

's repo (https://github.com/cloudflare/workerd/issues/777) - but no reply yet

GitHub

🐛 BUG: `deleteAlarm()` inside `alarm` handler causes seg fault · Is...

Which Cloudflare product(s) does this pertain to? Wrangler What version of Wrangler are you using? 3.1.0 What operating system are you using? Mac Describe the Bug Calling deleteAlarm() inside alarm...

Ssteveruizok Turns out we didn't need `deleteAlarm`!

Erwin•6/20/23, 11:47 AM

Funnily enough it is cheaper to have the alarm go off and not do anything than it is to delete the alarm

Original message was deleted

Erwin•6/20/23, 11:49 AM

Hmm.. Prettier runs when I save in my IDE, but might not have picked up the correct configuration file. Will double check.

Original message was deleted

Erwin•6/20/23, 12:18 PM

Ugg.. does indeed seems to have been an issue with configuration files or something. Now we have to wait for someone to approve running them again

MMax (@rozenmd)for some reason node 19 doesn't work, can you try node 18 or 20?

captain•6/20/23, 4:35 PM

just tried v20.3.0 and v18.16.0 but both area giving same issue of attempting something in /Users/dev/.nvm/versions/node/v18.16.0/lib/node_modules/wrangler/node_modules/better-sqlite3 but wrangler does't normally exist in that path does it?

MMax (@rozenmd)for some reason node 19 doesn't work, can you try node 18 or 20?

captain•6/20/23, 4:35 PM

still struggling here

Ccaptain just tried v20.3.0 and v18.16.0 but both area giving same issue of attempting so...

Max (@rozenmd)•6/20/23, 4:38 PM

if you're using

nvm

nvm

you shouldn't need to use

sudo

sudo

, might be related

MMax (@rozenmd)if you're using `nvm` you shouldn't need to use `sudo`, might be related

captain•6/20/23, 4:39 PM

using sudo, but again look at that path is that where wrangler is installed?

MMax (@rozenmd)if you're using `nvm` you shouldn't need to use `sudo`, might be related

captain•6/20/23, 4:40 PM

i pretty much do everything i'm told in the cloudflare support docs and no change was made when upgrading from different versions of v2, so something was possibly overlooked on the cloudflare end

MMax (@rozenmd)if you're using `nvm` you shouldn't need to use `sudo`, might be related

captain•6/20/23, 4:49 PM

any ideas would be appreciated...banging on this for a while. Where is wrangler normally installed globally?

captain•6/20/23, 6:33 PM

anybody else having issues upgrading to latest version from v2.20

JamesOP•6/20/23, 6:55 PM

What kind of issues specifically? The biggest change in v3 is with local dev now using

workerd

workerd

instead of a node.js simulator

JJames What kind of issues specifically? The biggest change in v3 is with local dev now...

captain•6/21/23, 5:04 AM

James! I get a bunch of errors, but this is what I see in the beginning:

% sudo npm install -g wrangler
npm WARN deprecated rollup-plugin-inject@3.0.2: This package has been deprecated and is no longer maintained. Please use @rollup/plugin-inject.
npm WARN deprecated sourcemap-codec@1.4.8: Please use @jridgewell/sourcemap-codec instead
npm ERR! code 1
npm ERR! path /Users/dev/.nvm/versions/node/v19.0.1/lib/node_modules/wrangler/node_modules/better-sqlite3
npm ERR! command failed

Murder Chicken•6/21/23, 2:59 PM

Is there any way to catch a miniflare error? If I have a fetch and pass it a non-existent URL, the .catch.catch handler on the

fetch

fetch

is triggered but the error object is empty. Miniflare describes the error though in my console. As I'm making dynamic fetches, it would be nice to be able to handle these properly in my local environment knowing it matches how production would handle it.

vassbence•6/21/23, 5:26 PM

Hi! Could you please add a —local flag to KV? In the wrangler@3 cli there is no way currently to access/edit KV data locally

vassbence•6/21/23, 5:27 PM

it would come really handy for me as i need to update some kv tags before starting tests for my worker via unstable_dev()

Vvassbence Hi! Could you please add a —local flag to KV? In the wrangler@3 cli there is no ...

Cyb3r-Jak3•6/21/23, 5:31 PM

Best place is a feature request on the wrangler repo if one doesn’t exist already. https://github.com/cloudflare/workers-sdk

GitHub

GitHub - cloudflare/workers-sdk: ⛅️ Home to Wrangler, the CLI for C...

Home to Wrangler, the CLI for Cloudflare Workers® - GitHub - cloudflare/workers-sdk:

Home to Wrangler, the CLI for Cloudflare Workers®

GitHub - cloudflare/workers-sdk: ⛅️ Home to Wrangler, the CLI for C...

vassbence•6/21/23, 5:51 PM

Seems to be already

vassbence•6/21/23, 5:51 PM

https://github.com/cloudflare/workers-sdk/issues/3332

GitHub

🚀 Feature Request: Examples of using KV with local dev · Issue #333...

Describe the solution I am trying to develop a workers app. I wish there were examples of using the kv commands in a tutorial My project.toml looks like this kv_namespaces = [ { binding = "FLA...

Vvassbence Seems to be already

DaniFoldi•6/21/23, 5:55 PM

yeah, quite a few people have asked if this exists just after v3 was released, and sadly I haven't seen any movement from the team around this one

vassbence•6/21/23, 5:58 PM

Is there a limitation with miniflare 3?

vassbence•6/21/23, 5:59 PM

if not then i’d be happy to contribute depending on how hard this will be

captain•6/21/23, 6:46 PM

could there be an issue where better-sqlite3 is now required by cloudflare wrangler?

captain•6/21/23, 6:48 PM

and if a person is upgrading from v2 to v3 for some reason the cloudflare wrangler install routine isn't adding that better-sqlite3 package creating an error? And perhaps all the QA and devs at cloudflare already have better-sqlite3 installed so they don't notice?

My mentality: `dependencies` are for _runtime_ things, if you've got an app that runs and uses `expr

Similar Threads