Merge the CI part of dev/changelog into main (#647)

Part of #448 and #208.

---------

Co-authored-by: Zachary Vander Velden <[email protected]>

Author: ia0

crates/cli-tools/CHANGELOG.md

@@ -10,6 +10,7 @@
 
 ### Minor
 
+- Add `cargo` and `changelog` modules and features
 - Handle more errors during platform discovery and `action::PlatformReboot`
 - Extend `fs::write()` first parameter to set the `OpenOptions` too
 - Add `error::root_cause_is()` to check the `anyhow::Error` root cause
@@ -34,4 +35,4 @@
 
 ## 0.1.0
 
-<!-- Increment to skip CHANGELOG.md test: 6 -->
+<!-- Increment to skip CHANGELOG.md test: 7 -->

crates/cli-tools/Cargo.lock

@@ -22,6 +22,7 @@ humantime = { version = "2.1.0", default-features = false, optional = true }
 indicatif = { version = "0.17.8", default-features = false, optional = true }
 log = { version = "0.4.21", default-features = false }
 rusb = { version = "0.9.4", default-features = false, optional = true }
+semver = { version = "1.0.23", default-features = false, optional = true }
 serde = { version = "1.0.202", default-features = false, features = ["derive"] }
 toml = { version = "0.8.13", default-features = false, features = ["display", "parse"] }
 wasefire-wire = { version = "0.1.1-git", path = "../wire", optional = true }
@@ -57,7 +58,7 @@ optional = true
 
 [features]
 action = [
-  "dep:cargo_metadata",
+  "cargo",
   "dep:clap",
   "dep:data-encoding",
   "dep:humantime",
@@ -69,6 +70,8 @@ action = [
   "dep:wasefire-wire",
   "tokio/time",
 ]
+cargo = ["dep:cargo_metadata"]
+changelog = ["cargo", "dep:clap", "dep:semver"]
 
 [lints]
 clippy.unit-arg = "allow"

crates/cli-tools/Cargo.toml

@@ -17,13 +17,13 @@ use std::path::{Path, PathBuf};
 use std::time::Duration;
 
 use anyhow::{bail, ensure, Result};
-use cargo_metadata::{Metadata, MetadataCommand};
 use clap::{ValueEnum, ValueHint};
 use rusb::GlobalContext;
 use tokio::process::Command;
 use wasefire_protocol::{self as service, applet, Connection, ConnectionExt};
 use wasefire_wire::{self as wire, Yoke};
 
+use crate::cargo::metadata;
 use crate::error::root_cause_is;
 use crate::{cmd, fs};
 
@@ -633,15 +633,6 @@ pub async fn optimize_wasm(applet: impl AsRef<Path>, opt_level: Option<OptLevel>
     Ok(())
 }
 
-async fn metadata(dir: impl Into<PathBuf>) -> Result<Metadata> {
-    let dir = dir.into();
-    let metadata =
-        tokio::task::spawn_blocking(|| MetadataCommand::new().current_dir(dir).no_deps().exec())
-            .await??;
-    ensure!(metadata.packages.len() == 1, "not exactly one package");
-    Ok(metadata)
-}
-
 fn wasefire_feature(
     package: &cargo_metadata::Package, feature: &str, cargo: &mut Command,
 ) -> Result<()> {

crates/cli-tools/src/action.rs

@@ -0,0 +1,27 @@
+// Copyright 2024 Google LLC
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+use std::path::PathBuf;
+
+use anyhow::{ensure, Result};
+use cargo_metadata::{Metadata, MetadataCommand};
+
+pub async fn metadata(dir: impl Into<PathBuf>) -> Result<Metadata> {
+    let dir = dir.into();
+    let metadata =
+        tokio::task::spawn_blocking(|| MetadataCommand::new().current_dir(dir).no_deps().exec())
+            .await??;
+    ensure!(metadata.packages.len() == 1, "not exactly one package");
+    Ok(metadata)
+}