Hi, I'm requiring a node lib in package.json thusly:

"dependencies": {
  "pgsql-parser": "^13.4.0",

(:require
  ["pgsql-parser" :as pgsql]

Uncaught Error: Cannot find module 'pgsql_parser'

Looking at the shim generated I see:

goog.provide("shadow.js.shim.module$pgsql_parser");
goog.provide("module$shadow_js_shim_module$pgsql_parser");
shadow.js.shim.module$pgsql_parser = require("pgsql_parser");
module$shadow_js_shim_module$pgsql_parser.default = shadow.js.shim.module$pgsql_parser;

If I edit to :

shadow.js.shim.module$pgsql_parser = require("pgsql-parser");

All is well. I assume this a bug to raise, but thought I'd check if anyone has an insight because this confuses me. The Shadow code calls Closure libs to generate names.

@coltnz I don't know how that would happen? there is never any translation or conversion of - to _ for JS packages. I cannot reproduce it locally either? which shadow-cljs version do you use?

• shadow-cljs - server version: 2.20.10

It really is weird, I tried react-dom package and it is fine

do you maybe have a [pgsql_parser :as pgsql] or [pgsql-parser :as pgsql] require somewhere else in your code?

symbols might get translated, so use strings always for npm packages

Nope only the require as above. My build target is a node lib btw.

build target shouldn't matter for this. just tried with :node-script and :node-library. both totally fine

for shadow-cljs, for (shadow/watch :app-id), is it possible to say: you don't have to do highest level optimization, but please do a quick tree shake and output a .js that I can copy/paste elsewhere? [I'm working on a project involving both cljs and non-cljs js partts, and I'd like to copy/paste the generate .js file elsewhere]

(shadow/release :app-id) takes ~20 seconds

I have a file src/foo.js . Is there a way to tell shadow-cljs, even in optimizations:advanced, to not rewrite file src/foo.js ?

I am setting a variable like this (set! js/Foo bar) and getting a warning `

constant Foo assigned a value more than once.
Original definition at externs.shadow.js:4

what exactly are you trying to do there? you shouldn't really use set! if you want to export a global? use (js/goog.exportSymbol "Foo" bar) instead