Feature/Fix: Remove shebang from codes

[renhiyama]

I was thinking of running my cli package that is a nodejs app, directly running it on deno like this:

 deno run -A -r https://esm.sh/[email protected]/src/cli/index.js test

However, the file doesn't make use of shebang, as we can see the code at https://esm.sh/v115/[email protected]/deno/src/cli/index.js

/* esm.sh - esbuild bundle([email protected]/src/cli/index) deno production */
import __Process$ from "https://deno.land/[email protected]/node/process.ts";#!/usr/bin/env node
import{dirname as m}from"/v115/[email protected]/deno/src/imports/env.js";import a from"/v115/[email protected]/deno/src/imports/nativeImport.js";import"/v115/[email protected]/deno/src/utils/log.js";import{Import as e}from"/v115/[email protected]/deno/src/imports/URLImport.js";import i from"/v115/[email protected]/deno/src/imports/dynamicImport.js";await Promise.all([e("[email protected]"),e("[email protected]"),e("[email protected]")]);var s=await e("[email protected]"),p=await a("node:fs"),d=await a("node:path"),n=i(await import("/v115/[email protected]/deno/package.json.js",{assert:{type:"json"}})),l=p.readdirSync(d.join(m,"./cli/cmds")),r={},o=s("reejs");o.version(n.version);for(let t of l)r[t.split(".")[0]]=i(await import(`./cmds/${t}`));for(let t in r)r[t].default(o);o.parse(__Process$.argv);
//# sourceMappingURL=index.js.map

Esm.sh adds its own comment at the start of the code, followed by polyfills... then comes our shebang...

Also, if possible, modify the shebang from node to the User-Agent used (like bun/node/deno) so it makes sure that we run directly on the runtime we want, and not just running a child process in theory 😄

[ije]

since esm.sh serves module as url, i don't see the necessary to modify it for deno/bun.

[renhiyama]

maybe atleast remove the shebang from the code so it isnt doesnt result in a syntax error ?

[ije]

yes, i am working on it