Merge pull request #136 from auths-dev/dev-initCleanupRegistryOutput

refactor: clean registry print statements from init

Author: bordumb

crates/auths-cli/src/commands/init/display.rs

@@ -8,21 +8,25 @@ pub(crate) fn display_developer_result(
     registered: Option<&str>,
 ) {
     out.newline();
-    out.print_heading("You are on the Web of Trust!");
+    if registered.is_some() {
+        out.print_heading("You are on the Web of Trust!");
+    } else {
+        out.print_heading("Your identity is ready!");
+    }
     out.newline();
     out.println(&format!("  Identity: {}", out.info(&result.identity_did)));
     out.println(&format!("  Key alias: {}", out.info(&result.key_alias)));
     if let Some(registry) = registered {
         out.println(&format!("  Registry: {}", out.info(registry)));
+        let did_prefix = result
+            .identity_did
+            .strip_prefix("did:keri:")
+            .unwrap_or(&result.identity_did);
+        out.println(&format!(
+            "  Profile: {}",
+            out.info(&format!("https://auths.dev/registry/identity/{did_prefix}"))
+        ));
     }
-    let did_prefix = result
-        .identity_did
-        .strip_prefix("did:keri:")
-        .unwrap_or(&result.identity_did);
-    out.println(&format!(
-        "  Profile: {}",
-        out.info(&format!("https://auths.dev/registry/identity/{did_prefix}"))
-    ));
     out.newline();
     out.print_success("Your next commit will be signed with Auths!");
     out.println("  Run `auths status` to check your identity");

crates/auths-cli/src/commands/init/mod.rs

@@ -30,7 +30,7 @@ use crate::config::CliConfig;
 use crate::factories::storage::build_auths_context;
 use crate::ux::format::Output;
 
-use super::signers::{SignersSyncArgs, handle_sync};
+use super::signers::sync_signers;
 use display::{
     display_agent_dry_run, display_agent_result, display_ci_result, display_developer_result,
 };
@@ -278,10 +278,15 @@ fn run_developer_setup(
     {
         let root = PathBuf::from(String::from_utf8_lossy(&output.stdout).trim());
         let repo_signers = root.join(".auths").join("allowed_signers");
-        let _ = handle_sync(&SignersSyncArgs {
-            repo: "~/.auths".into(),
-            output_file: Some(repo_signers),
-        });
+        if let Ok(auths_repo) = get_auths_repo_path()
+            && let Ok((path, report)) = sync_signers(&auths_repo, &repo_signers)
+        {
+            out.println(&format!(
+                "  Wrote {} allowed signer(s) to {}",
+                report.added,
+                path.display()
+            ));
+        }
     }
 
     // REGISTRATION & DISPLAY

crates/auths-cli/src/commands/signers.rs

@@ -2,7 +2,7 @@
 
 use anyhow::{Context, Result};
 use auths_sdk::workflows::allowed_signers::{
-    AllowedSigners, AllowedSignersError, EmailAddress, SignerPrincipal, SignerSource,
+    AllowedSigners, AllowedSignersError, EmailAddress, SignerPrincipal, SignerSource, SyncReport,
 };
 use auths_storage::git::RegistryAttestationStorage;
 use auths_verifier::core::Ed25519PublicKey;
@@ -208,26 +208,32 @@ fn handle_remove(args: &SignersRemoveArgs) -> Result<()> {
     Ok(())
 }
 
+/// Core sync logic — no printing. Reused by init and `auths signers sync`.
+pub(crate) fn sync_signers(
+    repo: &std::path::Path,
+    output_file: &std::path::Path,
+) -> Result<(PathBuf, SyncReport)> {
+    let storage = RegistryAttestationStorage::new(repo);
+    let mut signers = AllowedSigners::load(output_file, &FileAllowedSignersStore)
+        .with_context(|| format!("Failed to load {}", output_file.display()))?;
+    let report = signers
+        .sync(&storage)
+        .context("Failed to sync attestations")?;
+    signers
+        .save(&FileAllowedSignersStore)
+        .with_context(|| format!("Failed to write {}", output_file.display()))?;
+    Ok((output_file.to_path_buf(), report))
+}
+
 pub(crate) fn handle_sync(args: &SignersSyncArgs) -> Result<()> {
     let repo_path = expand_tilde(&args.repo)?;
-    let storage = RegistryAttestationStorage::new(&repo_path);
-
     let path = if let Some(ref output) = args.output_file {
         expand_tilde(output).map_err(|e| anyhow::anyhow!("{}", e))?
     } else {
         resolve_signers_path()?
     };
 
-    let mut signers = AllowedSigners::load(&path, &FileAllowedSignersStore)
-        .with_context(|| format!("Failed to load {}", path.display()))?;
-
-    let report = signers
-        .sync(&storage)
-        .context("Failed to sync attestations")?;
-
-    signers
-        .save(&FileAllowedSignersStore)
-        .with_context(|| format!("Failed to write {}", path.display()))?;
+    let (path, report) = sync_signers(&repo_path, &path)?;
 
     println!(
         "Synced: {} added, {} removed, {} manual preserved",