added back dry run fix

Author: mkatychev

crates/wasm-pkg-client/src/local.rs

@@ -140,7 +140,7 @@ impl PackagePublisher for LocalBackend {
         let mut out = tokio::fs::File::create(&path)
             .await
             .map_err(|e| registry_path_context(e, &path))?;
-        println!("writing to {}", path.display());
+        tracing::info!("publishing to {}", path.display());
         tokio::io::copy(&mut data, &mut out)
             .await
             .map_err(Error::IoError)

crates/wasm-pkg-common/src/config.rs

@@ -4,7 +4,6 @@ use std::{
     path::{Path, PathBuf},
 };
 
-use ::toml::Value;
 use serde::{Deserialize, Serialize};
 
 use crate::{

crates/wkg/src/main.rs

@@ -273,11 +273,16 @@ impl PublishArgs {
                 wit::build_wit_dir(&self.path, client.clone(), &mut lock_file).await?;
             // There is no way to check if we are in a git repository unlike `cargo publish --allow-dirty` so
             // check against previous values.
-            if lock_file != prev_lock_ref {
+            if lock_file != prev_lock_ref && !self.dry_run {
                 return Err(anyhow::anyhow!(
                     "wkg.lock would be updated during publish, aborting"
                 ))
-                .context("Run `wkg wit fetch` before attempting to publish");
+                .with_context(|| {
+                    format!(
+                        "Run `wkg wit build {}` before attempting to publish",
+                        self.path.to_string_lossy()
+                    )
+                });
             }
 
             let tmp = tempfile::Builder::new()

crates/wkg/src/wit.rs

@@ -115,7 +115,7 @@ impl BuildArgs {
 pub async fn build_wit_dir(
     dir: impl AsRef<Path>,
     client: CachingClient<FileCache>,
-    mut lock_file: &mut LockFile,
+    lock_file: &mut LockFile,
 ) -> anyhow::Result<(PackageRef, Option<Version>, Vec<u8>)> {
     check_dir(&dir).await?;
     let wkg_config = wasm_pkg_core::config::Config::load().await?;