Unable to bundle graphql about esm.sh HOT 8 CLOSED

I've worked out that the input going into esbuild is incorrect. Here is effectively the cause of the compiler fail:

export const { version,versionInfo,graphql, ....... } = graphql;

Note this is passed in at https://github.com/postui/esm.sh/blob/c61d1a05462a1d413f2b69797093e3ae54de7912/server/build.go#L465

An example of what the compiler is trying to compile:

const graphql = {
	graphql: "test"
};
export const { graphql } = graphql;
export default graphql;

Which results in

$ node script.js
file:///tmp/script.js:4
export const { graphql } = graphql;
               ^

SyntaxError: Identifier 'graphql' has already been declared
    at Loader.moduleStrategy (node:internal/modules/esm/translators:145:18)
    at async link (node:internal/modules/esm/module_job:47:21)

Now to find out why esbuild is being passed this.

from esm.sh.

Esbuild seems to have no problem bundling it locally.

app.tsx:

import * as Graphql from "graphql";
console.log(Graphql)

$ yarn add esbuild graphql
$ node_modules/.bin/esbuild app.tsx --bundle --outfile=out.js

from esm.sh.

Looks like it is partially a problem with the graphql package and how they wrote it

from esm.sh.

Strange...

from esm.sh.

Maybe ESM is using an older version of esbuild?

from esm.sh.

@FallingSnow thanks, i will figure it out what happened.

from esm.sh.

@FallingSnow thanks for the details, it is vary useful for me, and i just fixed it! please try it agian: https://esm.sh/graphql.

from esm.sh.

Great, it's working great! I'm finally only using esm.sh and denoland.

from esm.sh.