-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tracking issue for maximum compatibility #19
Comments
It's the same for skypack: https://cdn.skypack.dev/error/build:@starbeam/[email protected] /*
* [Package Error] "@starbeam/[email protected]" could not be built.
*
* [1/5] Verifying package is valid…
* [2/5] Installing dependencies from npm…
* [3/5] Building package using esinstall…
* Running esinstall...
* Failed to load node_modules/@starbeam/core/dist/index.js
* Unexpected token (67:10) in @starbeam/core/dist/index.js
* Install failed.
* Install failed.
*
* How to fix:
* - If you believe this to be an error in Skypack, file an issue here: https://github.com/skypackjs/skypack-cdn/issues
* - If you believe this to be an issue in the package, share this URL with the package authors to help them debug & fix.
* - Use https://skypack.dev/ to find a web-friendly alternative to find another package.
*/
console.warn("[Package Error] \"@starbeam/[email protected]\" could not be built. \n[1/5] Verifying package is valid…\n[2/5] Installing dependencies from npm…\n[3/5] Building package using esinstall…\nRunning esinstall...\nFailed to load node_modules/@starbeam/core/dist/index.js\n Unexpected token (67:10) in @starbeam/core/dist/index.js\nInstall failed.\nInstall failed.");
throw new Error("[Package Error] \"@starbeam/[email protected]\" could not be built. ");
export default null; |
Looks like these tools are choking on private fields. oofta |
I think this suggests that we should make a dedicated build for this sort of use. One question: is there an exports-map entry that these tools will prefer? Is it just a matter of making the |
how do you mean? the jsdeliver folks said they were working on updating their terser config so that private properties were supported -- that would allow us to use things as they are (or until we uncover a new problem) |
@NullVoxPopuli what about other CDNs, etc? I think we need a "maximum compatibility" build (maybe UMD 🤮) that we make the |
is it possible to avoid duplication in non-ESM environments? irrc, the main thing for interop between "things using starbeam" is that the auto-tracking implementation can't be duplicated. 🤔 Though, maybe if we're clear about the intend and purpose of these builds, it's fine? It probably makes sense to clarify in documentation (and comment in the built file itself) that UMD/etc is intended for small demos and not for libraries or bigger things, etc? |
I'm hopeful that using export maps will help clarify the role of each build, but a lot depends on how each part of the ecosystem interprets |
Skypack is unlikely to receive new updates. It is based on Snowpack which is since deprecated, so it's not going to work with modern packages going forward. I would suggest using esm.sh or something else instead. |
esm.run is a delight on what I have used it for so far. 10/10 can recommend |
esm.sh over here: https://jsbin.com/kivuqal/1/edit?html,output Also, going to remove all the CJS checkboxes, because ESM in node is really good these days |
❌ = confirmed not working
✔️ = confirmed working
⏳ = pending further investigation
require
21Skypack errors on modern ES Syntax. See #48
2
data-uri-to-buffer
(a dependency of@starbeam/debug
) is ESM only, and it seems Starbeam can't be used in a CJS environment (unlessdata-uri-to-buffer
is transpiled to CJS)Please comment here if the build doesn't work out of the box in some environment. That includes bundlers and CDNs.
Original Report
✂️ issues reported on #48 ✂️
I believe supporting this would be good, as we want "just trying it" to be as easy as possible.
At first glance tho, I don't know why this error occurred -- they don't give enough information and we build with rollup -- so I'm going to open an issue on their repo as well.
The text was updated successfully, but these errors were encountered: