refactor: rename dts2-extern command to bindgen

- Rename command from dts2-extern to bindgen for better clarity
- Update all references throughout the codebase:
  - Rename files: dts2extern.rs → bindgen.rs
  - Rename struct: Dts2ExternCommand → BindgenCommand
  - Update CLI command: dts2-extern → bindgen
  - Update user agent string to husk-bindgen/0.1.1
- Update documentation and examples
- All tests passing (610 total)

Author: fcoury

examples/fs-example/README.md

@@ -1,6 +1,6 @@
 # Node.js fs Module Example
 
-This example demonstrates how to use Node.js's `fs` module from Husk using extern type declarations generated by the `dts2-extern` command.
+This example demonstrates how to use Node.js's `fs` module from Husk using extern type declarations generated by the `bindgen` command.
 
 ## How this example was created
 
@@ -12,7 +12,7 @@ This example demonstrates how to use Node.js's `fs` module from Husk using exter
 2. Converted the Node.js fs type definitions directly from a URL:
    ```bash
    cd fs-example
-   husk dts2-extern https://raw.githubusercontent.com/DefinitelyTyped/DefinitelyTyped/master/types/node/fs.d.ts -o src/fs-types.husk
+   husk bindgen https://raw.githubusercontent.com/DefinitelyTyped/DefinitelyTyped/master/types/node/fs.d.ts -o src/fs-types.husk
    ```
 
 This generated a complete `fs-types.husk` file with all the extern declarations needed to use the fs module.
@@ -71,7 +71,7 @@ cargo run -- build && node add-bindings.js && node dist/main.js
 
 ## Key takeaways
 
-- The `dts2-extern` command can convert TypeScript declaration files directly from URLs
+- The `bindgen` command can convert TypeScript declaration files directly from URLs
 - The generated extern declarations provide type-safe access to JavaScript libraries
 - Complex Node.js APIs like the fs module work seamlessly with Husk
 - The full fs API is available, not just a simplified subset

src/commands/dts2extern.rs renamed to src/commands/bindgen.rs

@@ -88,7 +88,7 @@ pub(crate) fn create_secure_client() -> Result<reqwest::blocking::Client> {
     reqwest::blocking::Client::builder()
         .timeout(Duration::from_secs(30)) // 30 second timeout
         .redirect(reqwest::redirect::Policy::limited(3)) // Limit redirects to prevent redirect loops
-        .user_agent("husk-dts2extern/0.1.1") // Proper user agent
+        .user_agent("husk-bindgen/0.1.1") // Proper user agent
         .build()
         .context("Failed to create HTTP client")
 }
@@ -129,7 +129,7 @@ fn validate_response(response: &reqwest::blocking::Response) -> Result<()> {
 }
 
 #[derive(Debug, Args)]
-pub struct Dts2ExternCommand {
+pub struct BindgenCommand {
     /// Input .d.ts file path or URL
     input: String,
 
@@ -150,7 +150,7 @@ pub struct Dts2ExternCommand {
     filter: Option<String>,
 }
 
-impl Dts2ExternCommand {
+impl BindgenCommand {
     pub fn execute(&self) -> Result<()> {
         // Check if input is a URL
         let (content, filename) = if self.input.starts_with("http://")

src/commands/dts2extern_security_tests.rs renamed to src/commands/bindgen_security_tests.rs

@@ -1,6 +1,6 @@
 #[cfg(test)]
-mod dts2extern_security_tests {
-    use super::super::dts2extern::{create_secure_client, validate_url};
+mod bindgen_security_tests {
+    use super::super::bindgen::{create_secure_client, validate_url};
 
     #[test]
     fn test_validate_url_allows_https() {

src/commands/mod.rs

@@ -1,6 +1,6 @@
-pub mod dts2extern;
+pub mod bindgen;
 
 #[cfg(test)]
-mod dts2extern_security_tests;
+mod bindgen_security_tests;
 
-pub use dts2extern::Dts2ExternCommand;
+pub use bindgen::BindgenCommand;

src/main.rs

@@ -4,7 +4,7 @@ use clap::{Parser, Subcommand};
 use husk_lang::repl;
 
 mod commands;
-use commands::Dts2ExternCommand;
+use commands::BindgenCommand;
 
 #[derive(Parser, Debug)]
 #[command(author, version, about, long_about = None)]
@@ -110,8 +110,8 @@ enum Command {
     /// Run tests
     Test(Test),
 
-    /// Convert TypeScript .d.ts file to Husk extern declarations
-    Dts2Extern(Dts2ExternCommand),
+    /// Generate Husk bindings from TypeScript .d.ts files
+    Bindgen(BindgenCommand),
 }
 
 #[derive(Parser, Debug)]
@@ -134,7 +134,7 @@ fn main() -> anyhow::Result<()> {
         Some(Command::Build(build)) => build_command(build, no_color)?,
         Some(Command::New(new)) => new_command(new)?,
         Some(Command::Test(test)) => test_command(test, no_color)?,
-        Some(Command::Dts2Extern(dts2extern)) => dts2extern.execute()?,
+        Some(Command::Bindgen(bindgen)) => bindgen.execute()?,
         Some(Command::Repl) | None => repl()?,
     };
     Ok(())

tests/dts2extern_error_test.rs renamed to tests/bindgen_error_test.rs

@@ -2,9 +2,9 @@ use std::io::Write;
 use std::process::Command;
 use tempfile::NamedTempFile;
 
-// Helper to run dts2extern command
-fn run_dts2extern(args: &[&str]) -> std::process::Output {
-    let mut cmd_args = vec!["run", "-q", "--", "dts2-extern"];
+// Helper to run bindgen command
+fn run_bindgen(args: &[&str]) -> std::process::Output {
+    let mut cmd_args = vec!["run", "-q", "--", "bindgen"];
     cmd_args.extend_from_slice(args);
 
     Command::new("cargo")
@@ -22,7 +22,7 @@ fn create_temp_file(content: &str, suffix: &str) -> NamedTempFile {
 
 #[test]
 fn test_non_existent_file() {
-    let output = run_dts2extern(&["/path/that/does/not/exist.d.ts"]);
+    let output = run_bindgen(&["/path/that/does/not/exist.d.ts"]);
 
     if output.status.success() {
         eprintln!("Expected failure for non-existent file, but got success");
@@ -46,7 +46,7 @@ fn test_non_existent_file() {
 fn test_non_dts_extension() {
     let file = create_temp_file("export function test(): void;", ".ts");
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap()]);
+    let output = run_bindgen(&[file.path().to_str().unwrap()]);
 
     assert!(!output.status.success());
     let stderr = String::from_utf8_lossy(&output.stderr);
@@ -58,7 +58,7 @@ fn test_non_dts_extension() {
 fn test_empty_file() {
     let file = create_temp_file("", ".d.ts");
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap()]);
+    let output = run_bindgen(&[file.path().to_str().unwrap()]);
 
     // Empty file should succeed
     assert!(output.status.success());
@@ -71,7 +71,7 @@ fn test_invalid_typescript_syntax() {
         ".d.ts",
     );
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap()]);
+    let output = run_bindgen(&[file.path().to_str().unwrap()]);
 
     assert!(!output.status.success());
     let stderr = String::from_utf8_lossy(&output.stderr);
@@ -92,7 +92,7 @@ fn test_malformed_type_declarations() {
         ".d.ts",
     );
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap()]);
+    let output = run_bindgen(&[file.path().to_str().unwrap()]);
 
     assert!(!output.status.success());
 }
@@ -108,7 +108,7 @@ fn test_circular_type_reference() {
         ".d.ts",
     );
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap()]);
+    let output = run_bindgen(&[file.path().to_str().unwrap()]);
 
     // Should handle circular references gracefully
     assert!(output.status.success());
@@ -128,7 +128,7 @@ fn test_reserved_keyword_parameter_names() {
         ".d.ts",
     );
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap()]);
+    let output = run_bindgen(&[file.path().to_str().unwrap()]);
 
     // Should handle reserved keywords
     assert!(output.status.success());
@@ -141,7 +141,7 @@ fn test_reserved_keyword_parameter_names() {
 fn test_output_to_readonly_directory() {
     let file = create_temp_file("export function test(): void;", ".d.ts");
 
-    let output = run_dts2extern(&[
+    let output = run_bindgen(&[
         file.path().to_str().unwrap(),
         "-o",
         "/root/readonly_output.husk",
@@ -168,7 +168,7 @@ fn test_filter_with_no_matches() {
         ".d.ts",
     );
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap(), "--filter", "nonexistent.*"]);
+    let output = run_bindgen(&[file.path().to_str().unwrap(), "--filter", "nonexistent.*"]);
 
     // Should succeed but produce module with no exports
     assert!(output.status.success());
@@ -178,7 +178,7 @@ fn test_filter_with_no_matches() {
 fn test_invalid_regex_filter() {
     let file = create_temp_file("export function test(): void;", ".d.ts");
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap(), "--filter", "[invalid(regex"]);
+    let output = run_bindgen(&[file.path().to_str().unwrap(), "--filter", "[invalid(regex"]);
 
     // TODO: Filter validation is not yet implemented
     // For now, the command succeeds even with invalid regex
@@ -202,7 +202,7 @@ fn test_no_exports_in_file() {
         ".d.ts",
     );
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap()]);
+    let output = run_bindgen(&[file.path().to_str().unwrap()]);
 
     // Should succeed
     assert!(output.status.success());
@@ -220,7 +220,7 @@ fn test_namespace_only_declarations() {
         ".d.ts",
     );
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap()]);
+    let output = run_bindgen(&[file.path().to_str().unwrap()]);
 
     // Should handle namespace-only files
     assert!(output.status.success());
@@ -237,7 +237,7 @@ fn test_complex_generic_types_with_simplify() {
         ".d.ts",
     );
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap(), "--simplify"]);
+    let output = run_bindgen(&[file.path().to_str().unwrap(), "--simplify"]);
 
     // With simplify, should succeed
     assert!(output.status.success());
@@ -250,7 +250,7 @@ fn test_complex_generic_types_with_simplify() {
 fn test_module_name_with_special_chars() {
     let file = create_temp_file("export function test(): void;", ".d.ts");
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap(), "-m", "[email protected]"]);
+    let output = run_bindgen(&[file.path().to_str().unwrap(), "-m", "[email protected]"]);
 
     // Module names with special characters should be handled
     assert!(output.status.success());
@@ -261,7 +261,7 @@ fn test_very_long_module_name() {
     let file = create_temp_file("export function test(): void;", ".d.ts");
 
     let very_long_name = "a".repeat(1000);
-    let output = run_dts2extern(&[file.path().to_str().unwrap(), "-m", &very_long_name]);
+    let output = run_bindgen(&[file.path().to_str().unwrap(), "-m", &very_long_name]);
 
     // Should handle very long module names
     assert!(output.status.success());
@@ -275,7 +275,7 @@ fn test_binary_file_input() {
         .unwrap();
     file.flush().unwrap();
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap()]);
+    let output = run_bindgen(&[file.path().to_str().unwrap()]);
 
     assert!(!output.status.success());
 }
@@ -288,7 +288,7 @@ fn test_file_with_bom() {
     file.write_all(b"export function test(): void;").unwrap();
     file.flush().unwrap();
 
-    let output = run_dts2extern(&[file.path().to_str().unwrap()]);
+    let output = run_bindgen(&[file.path().to_str().unwrap()]);
 
     // Should handle BOM correctly
     assert!(output.status.success());