fix: correctly detect hyphenated bin artifact file path (#4069)

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 # UNRELEASED
 
+### fix: correctly detects hyphenated Rust bin crates
+
 ### fix: removes unnecessary tsc step in sveltekit build script
 
 ### feat!: `dfx info pocketic-config-port`

e2e/assets/hyphenated-project/Cargo.toml

@@ -0,0 +1,3 @@
+[workspace]
+resolver = "2"
+members = ["hyphenated-lib", "hyphenated-bin"]

e2e/assets/hyphenated-project/dfx.json

@@ -0,0 +1,15 @@
+{
+    "version": 1,
+    "canisters": {
+        "hyphenated-bin": {
+            "type": "rust",
+            "candid": "name.did",
+            "package": "hyphenated-bin"
+        },
+        "hyphenated-lib": {
+            "type": "rust",
+            "candid": "name.did",
+            "package": "hyphenated-lib"
+        }
+    }
+}

e2e/assets/hyphenated-project/hyphenated-bin/Cargo.toml

@@ -0,0 +1,8 @@
+[package]
+name = "hyphenated-bin"
+version = "0.0.0"
+publish = false
+
+[dependencies]
+ic-cdk = "0.17"
+candid = "0.10"

e2e/assets/hyphenated-project/hyphenated-bin/src/main.rs

@@ -0,0 +1,6 @@
+#![no_main]
+
+#[ic_cdk::query]
+fn name() -> String {
+    "bin".into()
+}

e2e/assets/hyphenated-project/hyphenated-lib/Cargo.toml

@@ -0,0 +1,11 @@
+[package]
+name = "hyphenated-lib"
+version = "0.0.0"
+publish = false
+
+[lib]
+crate-type = ["cdylib"]
+
+[dependencies]
+ic-cdk = "0.17"
+candid = "0.10"

e2e/assets/hyphenated-project/hyphenated-lib/src/lib.rs

@@ -0,0 +1,4 @@
+#[ic_cdk::query]
+fn name() -> String {
+    "lib".into()
+}

e2e/assets/hyphenated-project/name.did

@@ -0,0 +1,3 @@
+service : {
+    name : () -> (text) query;
+}

e2e/tests-dfx/deploy.bash

@@ -280,3 +280,14 @@ teardown() {
   assert_command_fail dfx deploy --mode reinstall --wasm-memory-persistence keep
   assert_contains "--skip-pre-upgrade and --wasm-memory-persistence can only be used with mode 'upgrade' or 'auto'."
 }
+
+@test "can deploy a hyphenated project" {
+  setup_rust
+  install_asset hyphenated-project
+  dfx_start
+  assert_command dfx deploy
+  assert_command dfx canister call hyphenated-bin name
+  assert_contains bin
+  assert_command dfx canister call hyphenated-lib name
+  assert_contains lib
+}

src/dfx/src/lib/canister_info/rust.rs

@@ -85,7 +85,11 @@ impl CanisterInfoFactory for RustCanisterInfo {
             "More than one bin/cdylib {phrasing} found"
         );
 
-        let wasm_name = target.name.replace('-', "_");
+        let wasm_name = if target.kind.iter().any(|t| t == "bin") {
+            target.name.clone()
+        } else {
+            target.name.replace('-', "_")
+        };
         let output_wasm_path = metadata
             .target_directory
             .join(format!("wasm32-unknown-unknown/release/{wasm_name}.wasm"))