Add list subcommand for manifest accounts (#494)

closes #391

## What changed
- added a new `list` subcommand to print account names from
`manifest.json`
- wired the command into clap subcommands and runtime dispatch
- added unit tests for list extraction behavior and kept CLI shape
validation covered by existing `verify_cli` test

## Notes
- `list` now runs before auto-upgrade logic so it does not mutate
manifest/account files
- if the manifest requires migration, `list` exits with a clear error
instead of modifying data

## Testing
- `cargo test list::tests::`
- `cargo test commands::tests::verify_cli`
- `cargo fmt --check`

<!-- PULLFROG_DIVIDER_DO_NOT_REMOVE_PLZ -->
<sup><a href="https://pullfrog.com"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://pullfrog.com/logos/frog-white-full-18px.png"><img
src="https://pullfrog.com/logos/frog-green-full-18px.png" width="9px"
height="9px" style="vertical-align: middle; "
alt="Pullfrog"></picture></a>&nbsp;&nbsp;｜ [View workflow
run](https://github.com/dyc3/steamguard-cli/actions/runs/24413687679/job/71317067164)
｜ Triggered by [Pullfrog](https://pullfrog.com) ｜ Using `GPT Codex` ｜
[𝕏](https://x.com/pullfrogai)</sup>

---------

Co-authored-by: pullfrog[bot] <226033991+pullfrog[bot]@users.noreply.github.com>

Author: pullfrog[bot]

src/commands.rs

@@ -17,6 +17,7 @@ pub mod debug;
 pub mod decrypt;
 pub mod encrypt;
 pub mod import;
+pub mod list;
 #[cfg(feature = "qr")]
 pub mod qr;
 pub mod qr_login;
@@ -32,6 +33,7 @@ pub use debug::DebugCommand;
 pub use decrypt::DecryptCommand;
 pub use encrypt::EncryptCommand;
 pub use import::ImportCommand;
+pub use list::ListCommand;
 #[cfg(feature = "qr")]
 pub use qr::QrCommand;
 pub use qr_login::QrLoginCommand;
@@ -172,6 +174,7 @@ pub(crate) enum Subcommands {
 	Completion(CompletionsCommand),
 	Setup(SetupCommand),
 	Import(ImportCommand),
+	List(ListCommand),
 	#[clap(alias = "trade")]
 	Confirm(ConfirmCommand),
 	Remove(RemoveCommand),

src/commands/list.rs

@@ -0,0 +1,75 @@
+use clap::Parser;
+use steamguard::transport::Transport;
+
+use crate::{accountmanager::ManifestEntry, AccountManager};
+
+use super::{GlobalArgs, ManifestCommand};
+
+#[derive(Debug, Clone, Parser)]
+#[clap(about = "List all accounts from the manifest.")]
+pub struct ListCommand;
+
+impl<T> ManifestCommand<T> for ListCommand
+where
+	T: Transport,
+{
+	fn execute(
+		&self,
+		_transport: T,
+		manager: &mut AccountManager,
+		_args: &GlobalArgs,
+	) -> anyhow::Result<()> {
+		let account_names = account_names(manager.iter());
+		if account_names.is_empty() {
+			println!("No accounts found in manifest.");
+			return Ok(());
+		}
+
+		for account_name in account_names {
+			println!("{}", account_name);
+		}
+
+		Ok(())
+	}
+}
+
+fn account_names<'a>(entries: impl Iterator<Item = &'a ManifestEntry>) -> Vec<&'a str> {
+	entries.map(|entry| entry.account_name.as_str()).collect()
+}
+
+#[cfg(test)]
+mod tests {
+	use crate::accountmanager::{manifest::Manifest, ManifestEntry};
+
+	use super::account_names;
+
+	#[test]
+	fn account_names_preserves_manifest_order() {
+		let entries = [
+			ManifestEntry {
+				filename: String::from("beta.maFile"),
+				steam_id: 2,
+				account_name: String::from("beta"),
+				encryption: None,
+			},
+			ManifestEntry {
+				filename: String::from("alpha.maFile"),
+				steam_id: 1,
+				account_name: String::from("alpha"),
+				encryption: None,
+			},
+		];
+
+		let names = account_names(entries.iter());
+
+		assert_eq!(names, vec!["beta", "alpha"]);
+	}
+
+	#[test]
+	fn account_names_empty_manifest() {
+		let manifest = Manifest::default();
+		let names = account_names(manifest.entries.iter());
+
+		assert!(names.is_empty());
+	}
+}

src/main.rs

@@ -78,13 +78,16 @@ fn main() {
 
 fn run(args: commands::Args) -> anyhow::Result<()> {
 	let globalargs = args.global;
+	let subcommand = args.sub.unwrap_or(Subcommands::Code(args.code));
+	let is_list_command = matches!(&subcommand, Subcommands::List(_));
 
-	let cmd: CommandType<WebApiTransport> = match args.sub.unwrap_or(Subcommands::Code(args.code)) {
+	let cmd: CommandType<WebApiTransport> = match subcommand {
 		Subcommands::Approve(args) => CommandType::Account(Box::new(args)),
 		Subcommands::Debug(args) => CommandType::Const(Box::new(args)),
 		Subcommands::Completion(args) => CommandType::Const(Box::new(args)),
 		Subcommands::Setup(args) => CommandType::Manifest(Box::new(args)),
 		Subcommands::Import(args) => CommandType::Manifest(Box::new(args)),
+		Subcommands::List(args) => CommandType::Manifest(Box::new(args)),
 		Subcommands::Encrypt(args) => CommandType::Manifest(Box::new(args)),
 		Subcommands::Decrypt(args) => CommandType::Manifest(Box::new(args)),
 		Subcommands::Confirm(args) => CommandType::Account(Box::new(args)),
@@ -128,6 +131,11 @@ fn run(args: commands::Args) -> anyhow::Result<()> {
 		manager = match accountmanager::AccountManager::load(path.as_path()) {
 			Ok(m) => m,
 			Err(ManifestLoadError::MigrationNeeded) => {
+				if is_list_command {
+					bail!(
+						"Manifest migration is needed before running `list`. Run another command to migrate the manifest first."
+					);
+				}
 				info!("Migrating manifest");
 				let manifest;
 				let accounts;
@@ -189,6 +197,14 @@ fn run(args: commands::Args) -> anyhow::Result<()> {
 	}
 
 	manager.submit_passkey(passkey);
+	if is_list_command {
+		let http_client = reqwest::blocking::Client::builder().build()?;
+		let transport = WebApiTransport::new(http_client);
+		if let CommandType::Manifest(cmd) = cmd {
+			cmd.execute(transport, &mut manager, &globalargs)?;
+			return Ok(());
+		}
+	}
 
 	loop {
 		match manager.auto_upgrade() {