fix: do not error for bin entrypoint with existing shebang (#469)

Author: dsherret

Cargo.lock

@@ -13,8 +13,10 @@ members = [
 
 [workspace.dependencies]
 async-trait = "0.1.88"
+deno_cache_dir = "0.23.0"
 deno_config = "0.61.0"
 deno_error = { version = "0.6.1", features = ["serde", "serde_json", "url"] }
+deno_graph = { version = "0.96.2", features = ["swc"], default-features = false }
 deno_path_util = "0.4.0"
 deno_resolver = { version = "0.42.0", features = ["graph"] }
 serde_json = { version = "1.0.140", features = ["preserve_order"] }

Cargo.toml

@@ -53,13 +53,12 @@
     "tests/web_socket_project/npm"
   ],
   "imports": {
-    "@deno/cache-dir": "jsr:@deno/cache-dir@^0.20",
     "@std/assert": "jsr:@std/assert@1",
     "@std/fmt": "jsr:@std/fmt@1",
     "@std/fs": "jsr:@std/fs@1",
     "@std/path": "jsr:@std/path@1",
-    "@ts-morph/bootstrap": "jsr:@ts-morph/bootstrap@^0.25.0",
-    "code-block-writer": "jsr:@david/code-block-writer@^13.0.2"
+    "@ts-morph/bootstrap": "jsr:@ts-morph/bootstrap@^0.27.0",
+    "code-block-writer": "jsr:@david/code-block-writer@^13.0.3"
   },
   "exports": {
     ".": "./mod.ts",

deno.jsonc

@@ -332,7 +332,9 @@ export async function build(options: BuildOptions): Promise<void> {
       outputFile.filePath,
     );
     const outputFileText = binaryEntryPointPaths.has(outputFile.filePath)
-      ? `#!/usr/bin/env node\n${outputFile.fileText}`
+      ? `#!/usr/bin/env node\n${
+        outputFile.fileText.replace(/^#![^\n\r]*\r?\n?/, "")
+      }`
       : outputFile.fileText;
     const sourceFile = project.createSourceFile(
       outputFilePath,

deno.lock

@@ -8,18 +8,18 @@ repository = "https://github.com/denoland/dnt"
 description = "Deno to Node/canonical TypeScript transform."
 
 [features]
-default = ["tokio-loader", "serialization"]
-tokio-loader = ["tokio", "reqwest"]
+default = ["serialization"]
 serialization = ["serde"]
 
 [dependencies]
 anyhow = "1.0.70"
 async-trait.workspace = true
 base64 = "0.13.1"
 deno_ast = { version = "0.48.1", features = ["transforms", "view", "visit", "utils"] }
+deno_cache_dir.workspace = true
 deno_config.workspace = true
 deno_error.workspace = true
-deno_graph = { version = "0.96.2", features = ["swc"], default-features = false }
+deno_graph.workspace = true
 deno_path_util.workspace = true
 deno_resolver.workspace = true
 deno_semver = "0.7.1"
@@ -30,13 +30,12 @@ jsonc-parser = { version = "0.26.2", features = ["serde"] }
 once_cell = "1.17.1"
 pathdiff = "0.2.1"
 regex = "1.7"
-reqwest = { version = "0.11", features = ["rustls"], optional = true }
 serde = { version = "1.0.159", features = ["derive"], optional = true }
 serde_json.workspace = true
 sys_traits.workspace = true
-tokio = { version = "1", features = ["full"], optional = true }
 url.workspace = true
 
 [dev-dependencies]
 pretty_assertions = "1.3.0"
 sys_traits = { workspace = true, features = ["memory"] }
+tokio = { version = "1", features = ["full"] }

mod.ts

@@ -6,7 +6,6 @@ use std::fmt::Write;
 use std::rc::Rc;
 
 use crate::loader::get_all_specifier_mappers;
-use crate::loader::Loader;
 use crate::loader::SourceLoader;
 use crate::parser::ScopeAnalysisParser;
 use crate::specifiers::get_specifiers;
@@ -35,7 +34,7 @@ use sys_traits::impls::RealSys;
 pub struct ModuleGraphOptions<'a, TSys: WorkspaceFactorySys> {
   pub entry_points: Vec<ModuleSpecifier>,
   pub test_entry_points: Vec<ModuleSpecifier>,
-  pub loader: Rc<dyn Loader>,
+  pub loader: Rc<dyn deno_graph::source::Loader>,
   pub resolver: DefaultDenoResolverRc<TSys>,
   pub specifier_mappings: &'a HashMap<ModuleSpecifier, MappedSpecifier>,
   pub compiler_options_resolver: Rc<CompilerOptionsResolver>,