Make ngit non-interactive by default

Implements non-interactive mode as the default behavior for ngit.
Users must now use -i flag for interactive prompts, or provide
all required arguments explicitly. Adds -d flag for sensible
defaults and -f flag for force operations.

Changes:
- CLI interactor infrastructure supports non-interactive mode
- Global flags: -i (interactive), --defaults (use defaults), -f (force)
- ngit init: requires --name or --identifier, supports --defaults
- ngit account: new signup command, login supports non-interactive
- ngit send: validates required fields, supports --defaults
- git-remote-nostr: fixed to prevent interactive prompts during push
- Comprehensive test coverage: 234 unit tests + integration tests

1 files changed, 1171 insertions, 588 deletions

diff --git a/src/bin/ngit/sub_commands/init.rs b/src/bin/ngit/sub_commands/init.rs
index 39fe670..827acf8 100644
--- a/src/bin/ngit/sub_commands/init.rs
+++ b/src/bin/ngit/sub_commands/init.rs
@@ -3,6 +3,7 @@ use std::{
     env,
     process::{Command, Stdio},
     str::FromStr,
+    sync::Arc,
     thread,
     time::Duration,
 };
@@ -13,7 +14,7 @@ use git2::Oid;
 use ngit::{
     UrlWithoutSlash,
     cli_interactor::{
-        PromptChoiceParms, PromptConfirmParms, multi_select_with_custom_value,
+        PromptChoiceParms, PromptConfirmParms, cli_error, multi_select_with_custom_value,
         show_multi_input_prompt_success,
     },
     client::{Params, get_state_from_cache, send_events},
@@ -44,112 +45,638 @@ use crate::{
     },
 };
 
+// ---------------------------------------------------------------------------
+// InitState: determines what scenario we're in
+// ---------------------------------------------------------------------------
+
+enum InitState {
+    /// No coordinate found anywhere (State A)
+    Fresh,
+    /// Coordinate found but no announcement event on relays (State B)
+    CoordinateOnly { coordinate: Nip19Coordinate },
+    /// Announcement exists, I am the trusted maintainer (State C)
+    MyAnnouncement {
+        coordinate: Nip19Coordinate,
+        repo_ref: RepoRef,
+    },
+    /// Announcement exists, I'm in the maintainer set (State D)
+    CoMaintainer {
+        coordinate: Nip19Coordinate,
+        repo_ref: RepoRef,
+    },
+    /// Announcement exists, I'm not in the maintainer set (State E)
+    NotListed {
+        coordinate: Nip19Coordinate,
+        repo_ref: RepoRef,
+    },
+}
+
+impl InitState {
+    fn coordinate(&self) -> Option<&Nip19Coordinate> {
+        match self {
+            Self::Fresh => None,
+            Self::CoordinateOnly { coordinate }
+            | Self::MyAnnouncement { coordinate, .. }
+            | Self::CoMaintainer { coordinate, .. }
+            | Self::NotListed { coordinate, .. } => Some(coordinate),
+        }
+    }
+
+    fn repo_ref(&self) -> Option<&RepoRef> {
+        match self {
+            Self::Fresh | Self::CoordinateOnly { .. } => None,
+            Self::MyAnnouncement { repo_ref, .. }
+            | Self::CoMaintainer { repo_ref, .. }
+            | Self::NotListed { repo_ref, .. } => Some(repo_ref),
+        }
+    }
+
+    /// Extract my own announcement's `RepoRef` from the events map.
+    /// Returns `None` if no coordinate, no announcement, or I have no event.
+    fn my_repo_ref(&self, my_pubkey: &PublicKey) -> Option<RepoRef> {
+        self.repo_ref()
+            .and_then(|rr| my_event_repo_ref(rr, my_pubkey))
+    }
+
+    fn has_coordinate(&self) -> bool {
+        !matches!(self, Self::Fresh)
+    }
+}
+
+struct ResolvedFields {
+    identifier: String,
+    name: String,
+    description: String,
+    git_servers: Vec<String>,
+    relays: Vec<RelayUrl>,
+    blossoms: Vec<Url>,
+    web: Vec<String>,
+    maintainers: Vec<PublicKey>,
+    earliest_unique_commit: String,
+    hashtags: Vec<String>,
+    selected_grasp_servers: Vec<String>,
+}
+
+/// Extract my own announcement's `RepoRef` from the events map.
+fn my_event_repo_ref(repo_ref: &RepoRef, my_pubkey: &PublicKey) -> Option<RepoRef> {
+    repo_ref
+        .events
+        .values()
+        .find(|e| e.pubkey == *my_pubkey)
+        .and_then(|e| RepoRef::try_from((e.clone(), None)).ok())
+}
+
+/// Find the latest event (by `created_at`) across all maintainer events and
+/// parse it into a `RepoRef` for shared metadata (name, description, web).
+fn latest_event_repo_ref(repo_ref: &RepoRef) -> Option<RepoRef> {
+    repo_ref
+        .events
+        .values()
+        .max_by_key(|e| e.created_at)
+        .and_then(|e| RepoRef::try_from((e.clone(), None)).ok())
+}
+
+/// Check if a grasp-format clone URL belongs to the given public key.
+fn is_my_grasp_clone_url(url: &str, my_pubkey: &PublicKey) -> bool {
+    if !is_grasp_server_clone_url(url) {
+        return false;
+    }
+    if let Ok(npub) = extract_npub(url) {
+        if let Ok(url_pk) = PublicKey::from_bech32(npub) {
+            return url_pk == *my_pubkey;
+        }
+    }
+    false
+}
+
+/// Check if a relay URL corresponds to one of the given grasp servers.
+fn is_grasp_derived_relay(relay: &str, grasp_servers: &[String]) -> bool {
+    let Ok(relay_normalized) = normalize_grasp_server_url(relay) else {
+        return false;
+    };
+    grasp_servers.iter().any(|gs| {
+        normalize_grasp_server_url(gs).is_ok_and(|gs_normalized| gs_normalized == relay_normalized)
+    })
+}
+
+/// Check if a blossom URL corresponds to one of the given grasp servers.
+fn is_grasp_derived_blossom(blossom: &str, grasp_servers: &[String]) -> bool {
+    // Blossom URLs are https://{grasp_server} — same normalization as relays
+    is_grasp_derived_relay(blossom, grasp_servers)
+}
+
+fn dir_name_fallback() -> String {
+    env::current_dir()
+        .ok()
+        .and_then(|p| p.file_name().map(|n| n.to_string_lossy().to_string()))
+        .unwrap_or_default()
+}
+
+fn identifier_from_name(name: &str) -> String {
+    name.replace(' ', "-")
+        .chars()
+        .map(|c| {
+            if c.is_ascii_alphanumeric() || c.eq(&'/') {
+                c
+            } else {
+                '-'
+            }
+        })
+        .collect()
+}
+
+fn build_gitworkshop_url(
+    public_key: &PublicKey,
+    identifier: &str,
+    first_relay: Option<&RelayUrl>,
+) -> String {
+    NostrUrlDecoded {
+        original_string: String::new(),
+        coordinate: Nip19Coordinate {
+            coordinate: Coordinate {
+                public_key: *public_key,
+                kind: Kind::GitRepoAnnouncement,
+                identifier: identifier.to_string(),
+            },
+            relays: first_relay.into_iter().cloned().collect(),
+        },
+        protocol: None,
+        ssh_key_file: None,
+        nip05: None,
+    }
+    .to_string()
+    .replace("nostr://", "https://gitworkshop.dev/")
+}
+
+/// Resolve the `web` field from args, existing announcement, or gitworkshop
+/// default.
+fn resolve_web(
+    args_web: &[String],
+    state: &InitState,
+    identifier: &str,
+    gitworkshop_url: &str,
+) -> Vec<String> {
+    if !args_web.is_empty() {
+        return args_web.to_vec();
+    }
+    if let Some(rr) = state.repo_ref() {
+        let latest_web = latest_event_repo_ref(rr).map_or_else(|| rr.web.clone(), |lr| lr.web);
+        let joined = latest_web.join(" ");
+        // replace legacy gitworkshop.dev url format
+        if joined.contains(&format!("https://gitworkshop.dev/repo/{identifier}")) {
+            return vec![gitworkshop_url.to_string()];
+        }
+        return latest_web;
+    }
+    vec![gitworkshop_url.to_string()]
+}
+
+/// Derive clone-urls, relays, and blossoms from selected grasp servers.
+///
+/// For each grasp server, adds/replaces the corresponding clone URL in
+/// `git_servers`, adds a relay URL to `relays`, and adds a blossom URL to
+/// `blossoms`. Grasp-derived infrastructure is always added — the other
+/// lists (`git_servers`, `relays`, `blossoms`) contain *additional*
+/// infrastructure beyond what grasp servers provide.
+fn apply_grasp_infrastructure(
+    grasp_servers: &[String],
+    git_servers: &mut Vec<String>,
+    relays: &mut Vec<String>,
+    blossoms: &mut Vec<String>,
+    public_key: &PublicKey,
+    identifier: &str,
+) -> Result<()> {
+    for grasp_server in grasp_servers {
+        // Always add grasp-derived clone URL
+        let clone_url = format_grasp_server_url_as_clone_url(grasp_server, public_key, identifier)?;
+
+        let grasp_server_clone_root = if clone_url.contains("https://") {
+            format!("https://{grasp_server}")
+        } else {
+            grasp_server.to_string()
+        };
+
+        let matching_positions: Vec<usize> = git_servers
+            .iter()
+            .enumerate()
+            .filter_map(|(idx, url)| {
+                if url.contains(&grasp_server_clone_root) {
+                    Some(idx)
+                } else {
+                    None
+                }
+            })
+            .collect();
+
+        if matching_positions.is_empty() {
+            git_servers.push(clone_url);
+        } else {
+            git_servers[matching_positions[0]] = clone_url;
+            for &position in matching_positions.iter().skip(1).rev() {
+                git_servers.remove(position);
+            }
+        }
+
+        // Always add grasp-derived relay
+        let relay_url = format_grasp_server_url_as_relay_url(grasp_server)?;
+        if !relays.contains(&relay_url) {
+            relays.push(relay_url);
+        }
+
+        // Always add grasp-derived blossom
+        let blossom = format_grasp_server_url_as_blossom_url(grasp_server)?;
+        if !blossoms.contains(&blossom) {
+            blossoms.push(blossom);
+        }
+    }
+    Ok(())
+}
+
+/// Resolve which grasp servers to use. Handles flag overrides, detection from
+/// existing URLs, user grasp list / system fallbacks, and interactive
+/// prompting.
+fn resolve_grasp_servers(
+    args: &SubCommandArgs,
+    cli: &Cli,
+    state: &InitState,
+    user_ref: &ngit::login::user::UserRef,
+    client: &Client,
+    identifier: &str,
+    interactive: bool,
+) -> Result<Vec<String>> {
+    if !args.grasp_servers.is_empty() {
+        return Ok(args.grasp_servers.clone());
+    }
+
+    let has_both_relays_and_clone_url = !args.relays.is_empty() && !args.clone.is_empty();
+    if has_both_relays_and_clone_url {
+        return Ok(vec![]);
+    }
+
+    // Use my own announcement (not the consolidated union) for grasp detection.
+    // Infrastructure is personal — each maintainer has their own servers.
+    let my_ref = state.my_repo_ref(&user_ref.public_key);
+
+    if !args.clone.is_empty() {
+        return Ok(detect_existing_grasp_servers(
+            my_ref.as_ref(),
+            &args.relays,
+            &args.clone,
+            identifier,
+        ));
+    }
+
+    if !interactive || cli.defaults || state.has_coordinate() || cli.force {
+        // Prefer grasp servers from my existing announcement, then user's grasp
+        // list, then system fallbacks
+        let existing =
+            detect_existing_grasp_servers(my_ref.as_ref(), &args.relays, &[], identifier);
+        if !existing.is_empty() {
+            return Ok(existing);
+        }
+        return Ok(grasp_servers_from_user_or_fallback(user_ref, client));
+    }
+
+    // Interactive prompt
+    let mut options: Vec<String> =
+        detect_existing_grasp_servers(my_ref.as_ref(), &args.relays, &args.clone, identifier);
+    let mut selections: Vec<bool> = vec![true; options.len()];
+    let empty = options.is_empty();
+    for user_grasp_option in &user_ref.grasp_list.urls {
+        if !options
+            .iter()
+            .any(|option| option.contains(user_grasp_option.as_str()))
+        {
+            options.push(user_grasp_option.to_string());
+            selections.push(empty);
+        }
+    }
+    let empty = options.is_empty();
+    let fallback_grasp_servers = client.get_grasp_default_set();
+    for fallback in fallback_grasp_servers {
+        if !options.iter().any(|option| option.contains(fallback)) {
+            options.push(fallback.clone());
+            selections.push(empty);
+        }
+    }
+    let selected = multi_select_with_custom_value(
+        "grasp servers (ideally use between 2-4)",
+        "grasp server",
+        options,
+        selections,
+        normalize_grasp_server_url,
+    )?;
+    show_multi_input_prompt_success("grasp servers", &selected);
+    Ok(selected)
+}
+
+fn grasp_servers_from_user_or_fallback(
+    user_ref: &ngit::login::user::UserRef,
+    client: &Client,
+) -> Vec<String> {
+    if user_ref.grasp_list.urls.is_empty() {
+        client
+            .get_grasp_default_set()
+            .iter()
+            .map(std::string::ToString::to_string)
+            .collect()
+    } else {
+        user_ref
+            .grasp_list
+            .urls
+            .iter()
+            .map(std::string::ToString::to_string)
+            .collect()
+    }
+}
+
+// ---------------------------------------------------------------------------
+// Validation
+// ---------------------------------------------------------------------------
+
+/// Validation for State A (Fresh): no existing coordinate.
+fn validate_fresh(cli: &Cli, args: &SubCommandArgs, user_has_grasp_list: bool) -> Result<()> {
+    // -d or -f with no substantive flags: proceed with all defaults
+    if !args.has_substantive_flags() && (cli.defaults || cli.force) {
+        return Ok(());
+    }
+
+    // Substantive flags provided: -d fills any gaps
+    if cli.defaults {
+        return Ok(());
+    }
+
+    // Validate essential fields
+    let mut missing: Vec<(&str, &str)> = Vec::new();
+
+    let missing_name = args.identifier.is_none() && args.name.is_none();
+    if missing_name {
+        missing.push(("--name <NAME>", "repository name or identifier"));
+    }
+
+    let has_grasp_servers = !args.grasp_servers.is_empty();
+    let has_both_relays_and_clone_url = !args.relays.is_empty() && !args.clone.is_empty();
+    let missing_servers =
+        !has_grasp_servers && !user_has_grasp_list && !has_both_relays_and_clone_url;
+    if missing_servers {
+        missing.push((
+            "--grasp-servers <URL>...",
+            "where your git+nostr data is hosted",
+        ));
+    }
+
+    if missing.is_empty() {
+        return Ok(());
+    }
+
+    let message = if missing.len() == 1 {
+        let (flag, desc) = missing[0];
+        format!("missing {flag} ({desc})")
+    } else {
+        "missing required fields".to_string()
+    };
+
+    let mut details: Vec<(&str, &str)> = if missing.len() > 1 {
+        missing.clone()
+    } else {
+        vec![]
+    };
+
+    details.push(("-d, --defaults", "or just use sensible defaults"));
+    let name_part = if missing_name {
+        " --name \"My Project\""
+    } else {
+        ""
+    };
+    let suggestion =
+        format!("ngit init{name_part} --description \"my project description\" --defaults");
+
+    Err(cli_error(&message, &details, &[&suggestion]))
+}
+
 #[derive(Debug, clap::Args)]
 pub struct SubCommandArgs {
-    #[clap(short, long)]
-    /// name of repository
-    title: Option<String>,
-    #[clap(short, long)]
+    #[clap(long)]
+    /// name of repository (preferred over --identifier)
+    name: Option<String>,
+    #[clap(long)]
+    /// shortname with no spaces or special characters
+    identifier: Option<String>,
+    #[clap(long)]
     /// optional description
     description: Option<String>,
-    #[clap(long)]
-    /// git server url users can clone from
-    clone_url: Vec<String>,
     #[clap(short, long, value_parser, num_args = 1..)]
-    /// homepage
-    web: Vec<String>,
-    #[clap(short, long, value_parser, num_args = 1..)]
-    /// relays contributors push patches and comments to
+    /// where your git+nostr data is hosted
+    grasp_servers: Vec<String>,
+    #[clap(long, value_parser, num_args = 1..)]
+    /// additional relays beyond grasp servers
     relays: Vec<String>,
-    #[clap(short, long, value_parser, num_args = 1..)]
-    /// blossom servers
+    #[clap(long)]
+    /// additional git server URLs beyond grasp servers
+    clone: Vec<String>,
+    #[clap(long, value_parser, num_args = 1..)]
+    /// additional blossom servers beyond grasp servers
     blossoms: Vec<String>,
-    #[clap(short, long, value_parser, num_args = 1..)]
+    #[clap(long, value_parser, num_args = 1..)]
+    /// homepage
+    web: Vec<String>,
+    #[clap(long, value_parser, num_args = 1..)]
     /// npubs of other maintainers
     other_maintainers: Vec<String>,
     #[clap(long)]
     /// usually root commit but will be more recent commit for forks
     earliest_unique_commit: Option<String>,
-    #[clap(short, long)]
-    /// shortname with no spaces or special characters
-    identifier: Option<String>,
 }
 
-#[allow(clippy::too_many_lines)]
-pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
-    let git_repo = Repo::discover().context("failed to find a git repository")?;
-    let git_repo_path = git_repo.get_path()?;
+impl SubCommandArgs {
+    fn has_substantive_flags(&self) -> bool {
+        self.name.is_some()
+            || self.identifier.is_some()
+            || self.description.is_some()
+            || !self.clone.is_empty()
+            || !self.relays.is_empty()
+            || !self.grasp_servers.is_empty()
+            || !self.web.is_empty()
+            || !self.blossoms.is_empty()
+            || !self.other_maintainers.is_empty()
+            || self.earliest_unique_commit.is_some()
+    }
+}
 
-    let root_commit = git_repo
-        .get_root_commit()
-        .context("failed to get root commit of the repository")?;
+// ---------------------------------------------------------------------------
+// Pre/post-fetch validation
+// ---------------------------------------------------------------------------
+
+fn validate_pre_fetch(
+    cli: &Cli,
+    args: &SubCommandArgs,
+    repo_coordinate: Option<&Nip19Coordinate>,
+    user_has_grasp_list: bool,
+) -> Result<()> {
+    // Interactive mode bypasses pre-fetch validation
+    if cli.interactive {
+        return Ok(());
+    }
 
-    // TODO: check for empty repo
-    // TODO: check for existing maintaiers file
+    // If no coordinate exists, we're in State A (Fresh) - validate now
+    if repo_coordinate.is_none() {
+        return validate_fresh(cli, args, user_has_grasp_list);
+    }
 
-    let mut client = Client::new(Params::with_git_config_relay_defaults(&Some(&git_repo)));
+    // Coordinate exists - we need to fetch before we can validate further
+    Ok(())
+}
 
-    let repo_coordinate = (try_and_get_repo_coordinates_when_remote_unknown(&git_repo).await).ok();
+fn validate_post_fetch(cli: &Cli, args: &SubCommandArgs, state: &InitState) -> Result<()> {
+    // Interactive mode bypasses all validation
+    if cli.interactive {
+        return Ok(());
+    }
 
-    let repo_ref = if let Some(repo_coordinate) = &repo_coordinate {
-        fetching_with_report(git_repo_path, &client, repo_coordinate).await?;
-        (get_repo_ref_from_cache(Some(git_repo_path), repo_coordinate).await).ok()
+    match state {
+        InitState::Fresh => {
+            // Already validated in pre-fetch
+            Ok(())
+        }
+        InitState::CoordinateOnly { coordinate } => {
+            if cli.force {
+                Ok(())
+            } else {
+                let id = &coordinate.identifier;
+                Err(cli_error(
+                    &format!(
+                        "no announcement found for coordinate '{id}'\n\n\
+                         \x20 This could be a relay or network issue. Only proceed with --force\n\
+                         \x20 if you are sure there isn't an existing announcement event."
+                    ),
+                    &[],
+                    &["ngit init --force"],
+                ))
+            }
+        }
+        InitState::MyAnnouncement { repo_ref, .. } => {
+            if let Some(new_id) = &args.identifier {
+                if *new_id != repo_ref.identifier && !cli.force {
+                    let suggestion = format!("ngit init --identifier {new_id} --force");
+                    return Err(cli_error(
+                        "changing identifier creates a new repository",
+                        &[],
+                        &[&suggestion],
+                    ));
+                }
+            }
+            if !args.has_substantive_flags() && !cli.force {
+                return Err(cli_error(
+                    "no arguments specified, use --force to publish with new timestamp",
+                    &[],
+                    &["ngit init --force"],
+                ));
+            }
+            Ok(())
+        }
+        InitState::CoMaintainer { repo_ref, .. } => {
+            if let Some(new_id) = &args.identifier {
+                if *new_id != repo_ref.identifier && !cli.force {
+                    let suggestion = format!("ngit init --identifier {new_id} --force");
+                    return Err(cli_error(
+                        "changing identifier creates a new repository",
+                        &[],
+                        &[&suggestion],
+                    ));
+                }
+            }
+            Ok(())
+        }
+        InitState::NotListed { .. } => {
+            if cli.force {
+                Ok(())
+            } else {
+                Err(cli_error(
+                    "you are not listed as a maintainer",
+                    &[],
+                    &["ngit init --force"],
+                ))
+            }
+        }
+    }
+}
+
+#[allow(clippy::too_many_lines)]
+#[allow(clippy::too_many_arguments)]
+fn resolve_fields(
+    state: &InitState,
+    user_ref: &ngit::login::user::UserRef,
+    args: &SubCommandArgs,
+    cli: &Cli,
+    git_repo: &Repo,
+    root_commit: &str,
+    client: &Client,
+    repo_config_result: &Result<ngit::repo_ref::RepoConfigYaml>,
+    interactive: bool,
+) -> Result<ResolvedFields> {
+    let my_pubkey = &user_ref.public_key;
+
+    // Shared lookups used by multiple fields below
+    let latest = state.repo_ref().and_then(latest_event_repo_ref);
+    let my_ref = state.my_repo_ref(my_pubkey);
+
+    // --- Identifier default ---
+    let identifier_default = if let Some(coord) = state.coordinate() {
+        coord.identifier.clone()
+    } else if let Ok(config) = repo_config_result {
+        if let Some(id) = &config.identifier {
+            id.clone()
+        } else {
+            dir_name_fallback()
+        }
     } else {
-        None
+        dir_name_fallback()
     };
 
-    let (signer, user_ref, _) = login::login_or_signup(
-        &Some(&git_repo),
-        &extract_signer_cli_arguments(cli_args).unwrap_or(None),
-        &cli_args.password,
-        Some(&client),
-        true,
-    )
-    .await?;
-
-    let repo_config_result = get_repo_config_from_yaml(&git_repo);
-    // TODO: check for other claims
+    // --- Name ---
+    let name_default = if let Some(ref lr) = latest {
+        lr.name.clone()
+    } else if let Some(coord) = state.coordinate() {
+        coord.identifier.clone()
+    } else {
+        dir_name_fallback()
+    };
 
-    let name = match &args.title {
-        Some(t) => t.clone(),
-        None => Interactor::default().input(
+    let name = if let Some(v) = &args.name {
+        v.clone()
+    } else if interactive {
+        Interactor::default().input(
             PromptInputParms::default()
                 .with_prompt("repo name")
-                .with_default(if let Some(repo_ref) = &repo_ref {
-                    repo_ref.name.clone()
-                } else if let Some(coordinate) = &repo_coordinate {
-                    coordinate.identifier.clone()
-                } else if let Ok(path) = env::current_dir() {
-                    if let Some(current_dir_name) = path.file_name() {
-                        current_dir_name.to_string_lossy().to_string()
-                    } else {
-                        String::new()
-                    }
-                } else {
-                    String::new()
-                }),
-        )?,
+                .with_default(name_default.clone())
+                .with_flag_name("--name"),
+        )?
+    } else {
+        name_default.clone()
     };
 
-    let description = match &args.description {
-        Some(t) => t.clone(),
-        None => Interactor::default().input(
+    // --- Description ---
+    let description_default = latest
+        .as_ref()
+        .map_or_else(String::new, |lr| lr.description.clone());
+
+    let description = if let Some(v) = &args.description {
+        v.clone()
+    } else if interactive {
+        Interactor::default().input(
             PromptInputParms::default()
-                .with_prompt("repo description (one sentance)")
+                .with_prompt("repo description (one sentence)")
                 .optional()
-                .with_default(if let Some(repo_ref) = &repo_ref {
-                    repo_ref.description.clone()
-                } else {
-                    String::new()
-                }),
-        )?,
+                .with_default(description_default.clone())
+                .with_flag_name("--description"),
+        )?
+    } else {
+        description_default
     };
 
-    // this is important so init can be completed done without prompts
-    let has_server_and_relay_flags = !args.clone_url.is_empty() && !args.relays.is_empty();
-
-    let simple_mode = if has_server_and_relay_flags {
-        false
+    // --- Simple mode (interactive only) ---
+    let simple_mode = if !interactive || (!args.clone.is_empty() && !args.relays.is_empty()) {
+        false // not used in non-interactive, but avoids Option
     } else {
         Interactor::default().choice(
             PromptChoiceParms::default()
@@ -162,216 +689,142 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
         )? == 0
     };
 
-    let identifier_default = if let Some(repo_ref) = &repo_ref {
-        repo_ref.identifier.clone()
-    } else if let Some(repo_coordinate) = &repo_coordinate {
-        repo_coordinate.identifier.clone()
-    } else {
-        let fallback = name
-            .clone()
-            .replace(' ', "-")
-            .chars()
-            .map(|c| {
-                if c.is_ascii_alphanumeric() || c.eq(&'/') {
-                    c
-                } else {
-                    '-'
-                }
-            })
-            .collect();
-        if let Ok(config) = &repo_config_result {
-            if let Some(identifier) = &config.identifier {
-                identifier.to_string()
-            } else {
-                fallback
-            }
+    // --- Identifier ---
+    let identifier = if let Some(id) = &args.identifier {
+        id.clone()
+    } else if state.has_coordinate() {
+        identifier_default.clone()
+    } else if !interactive || cli.defaults {
+        if args.name.is_some() && !state.has_coordinate() {
+            identifier_from_name(&name)
         } else {
-            fallback
+            identifier_default.clone()
         }
+    } else {
+        let id_default = if args.name.is_some() || name != name_default {
+            identifier_from_name(&name)
+        } else {
+            identifier_default.clone()
+        };
+        Interactor::default().input(
+            PromptInputParms::default()
+                .with_prompt("repo identifier")
+                .with_default(id_default)
+                .with_flag_name("--identifier"),
+        )?
     };
 
-    let identifier = match &args.identifier {
-        Some(t) => t.clone(),
-        None => {
-            if simple_mode {
-                identifier_default
+    // --- Grasp servers ---
+    let selected_grasp_servers =
+        resolve_grasp_servers(args, cli, state, user_ref, client, &identifier, interactive)?;
+
+    // --- Base infrastructure (flag > my event > fallback) ---
+    // Grasp-derived infrastructure (my clone URLs, relays, blossoms) is handled
+    // by apply_grasp_infrastructure below. Defaults here are *additional*
+    // infrastructure only. My own grasp-format clone URLs are filtered out so
+    // they get re-derived from the resolved grasp servers. Grasp-format clone
+    // URLs belonging to other maintainers are kept as additional git servers.
+    let no_state = git_repo
+        .get_git_config_item("nostr.nostate", None)
+        .ok()
+        .flatten()
+        .is_some_and(|s| s == "true");
+
+    // Detect my grasp servers from my existing announcement (for filtering)
+    let my_existing_grasp_servers: Vec<String> = my_ref
+        .as_ref()
+        .map(|mr| detect_existing_grasp_servers(Some(mr), &[], &[], &identifier))
+        .unwrap_or_default();
+
+    let git_servers_default = if let Some(ref mr) = my_ref {
+        // Keep non-grasp URLs and grasp URLs from other maintainers;
+        // filter out my own grasp-derived clone URLs (re-derived from grasp servers)
+        mr.git_server
+            .iter()
+            .filter(|url| !is_my_grasp_clone_url(url, my_pubkey))
+            .cloned()
+            .collect()
+    } else if no_state {
+        // Only fall back to origin URL when nostate is set (user pushes directly
+        // to a traditional git server rather than through grasp servers)
+        if let Ok(url) = git_repo.get_origin_url() {
+            if let Ok(fetch_url) = convert_clone_url_to_https(&url) {
+                vec![fetch_url]
+            } else if url.starts_with("nostr://") {
+                vec![]
             } else {
-                Interactor::default().input(
-                PromptInputParms::default()
-                    .with_prompt(
-                        "repo identifier (typically the short name with hypens instead of spaces)",
-                    )
-                    .with_default(identifier_default),
-            )?
+                vec![url]
             }
-        }
-    };
-
-    let mut git_server_defaults: Vec<String> = if !args.clone_url.is_empty() {
-        args.clone_url.clone()
-    } else if let Some(repo_ref) = &repo_ref {
-        // TODO dont default to git servers of other maintainers (?)
-        repo_ref.git_server.clone()
-    } else if let Ok(url) = git_repo.get_origin_url() {
-        if let Ok(fetch_url) = convert_clone_url_to_https(&url) {
-            vec![fetch_url]
-        } else if url.starts_with("nostr://") {
-            // nostr added as origin remote before repo announcement sent
-            vec![]
         } else {
-            // local repo or custom protocol
-            vec![url]
+            vec![]
         }
     } else {
         vec![]
     };
 
-    let mut relay_defaults = if args.relays.is_empty() {
-        if let Ok(config) = &repo_config_result {
-            config.relays.clone()
-        } else if let Some(repo_ref) = &repo_ref {
-            repo_ref
-                .relays
-                .iter()
-                .map(std::string::ToString::to_string)
-                .collect::<Vec<String>>()
-        } else {
+    let relays_default = if let Some(ref mr) = my_ref {
+        // Keep relays that don't correspond to my grasp servers
+        // (grasp-derived relays are re-added by apply_grasp_infrastructure)
+        mr.relays
+            .iter()
+            .map(std::string::ToString::to_string)
+            .filter(|r| !is_grasp_derived_relay(r, &my_existing_grasp_servers))
+            .collect()
+    } else if let Ok(config) = repo_config_result {
+        if config.relays.is_empty() {
             client.get_relay_default_set().clone()
+        } else {
+            config.relays.clone()
         }
     } else {
-        args.relays.clone()
+        client.get_relay_default_set().clone()
     };
 
-    let mut blossoms_defaults = if args.blossoms.is_empty() {
-        if let Some(repo_ref) = &repo_ref {
-            repo_ref
-                .blossoms
-                .iter()
-                .map(UrlWithoutSlash::to_string_without_trailing_slash)
-                .collect::<Vec<String>>()
-        // } else if user_ref.blossoms.read().is_empty() {
-        //     client.get_fallback_relays().clone()
-        } else {
-            vec![]
-            // user_ref.relays.read().clone()
-        }
+    let blossoms_default: Vec<String> = if let Some(ref mr) = my_ref {
+        // Keep blossoms that don't correspond to my grasp servers
+        mr.blossoms
+            .iter()
+            .map(UrlWithoutSlash::to_string_without_trailing_slash)
+            .filter(|b| !is_grasp_derived_blossom(b, &my_existing_grasp_servers))
+            .collect()
     } else {
-        args.blossoms.clone()
+        vec![]
     };
 
-    let fallback_grasp_servers = client.get_grasp_default_set();
-
-    let selected_grasp_servers = if has_server_and_relay_flags {
-        // ignore so a script running `ngit init` can contiue without prompts
-        vec![]
+    let mut git_servers = if args.clone.is_empty() {
+        git_servers_default
     } else {
-        let mut options: Vec<String> = detect_existing_grasp_servers(
-            repo_ref.as_ref(),
-            &args.relays,
-            &args.clone_url,
-            &identifier,
-        );
-        let mut selections: Vec<bool> = vec![true; options.len()]; // Initialize selections based on existing options
-        let empty = options.is_empty();
-        for user_grasp_option in user_ref.grasp_list.urls {
-            // Check if any option contains the user_grasp_option as a substring
-            if !options
-                .iter()
-                .any(|option| option.contains(user_grasp_option.as_str()))
-            {
-                options.push(user_grasp_option.to_string()); // Add if not found
-                selections.push(empty); // mark as selected if no existing grasp otherwise not
-            }
-        }
-
-        let empty = options.is_empty();
-        for fallback in fallback_grasp_servers {
-            // Check if any option contains the fallback as a substring
-            if !options.iter().any(|option| option.contains(fallback)) {
-                options.push(fallback.clone()); // Add fallback if not found
-                selections.push(empty); // mark as selected if no existing selections otherwise not
-            }
-        }
-        let selected = multi_select_with_custom_value(
-            "grasp servers (ideally use between 2-4)",
-            "grasp server",
-            options,
-            selections,
-            normalize_grasp_server_url,
-        )?;
-        show_multi_input_prompt_success("grasp servers", &selected);
-        selected
+        args.clone.clone()
     };
-
-    // ensure ngit relays are added as git server, relay and blossom entries
-    for grasp_server in &selected_grasp_servers {
-        if args.clone_url.is_empty() {
-            let clone_url = format_grasp_server_url_as_clone_url(
-                grasp_server,
-                &user_ref.public_key,
-                &identifier,
-            )?;
-
-            let grasp_server_clone_root = if clone_url.contains("https://") {
-                format!("https://{grasp_server}")
-            } else {
-                grasp_server.to_string()
-            };
-
-            // Find all positions of entries containing the relay root
-            let matching_positions: Vec<usize> = git_server_defaults
-                .iter()
-                .enumerate()
-                .filter_map(|(idx, url)| {
-                    if url.contains(&grasp_server_clone_root) {
-                        Some(idx)
-                    } else {
-                        None
-                    }
-                })
-                .collect();
-
-            // If we found any matches
-            if matching_positions.is_empty() {
-                // No existing entries found, so add a new one
-                git_server_defaults.push(clone_url);
-            } else {
-                // Replace the first occurrence
-                git_server_defaults[matching_positions[0]] = clone_url;
-
-                // Remove any subsequent occurrences (in reverse order to avoid index issues)
-                for &position in matching_positions.iter().skip(1).rev() {
-                    git_server_defaults.remove(position);
-                }
-            }
-        }
-        if args.relays.is_empty() {
-            let relay_url = format_grasp_server_url_as_relay_url(grasp_server)?;
-            if !relay_defaults.contains(&relay_url) {
-                relay_defaults.push(relay_url);
-            }
-        }
-        if args.blossoms.is_empty() {
-            let blossom = format_grasp_server_url_as_blossom_url(grasp_server)?;
-            if !blossoms_defaults.contains(&blossom) {
-                blossoms_defaults.push(blossom);
-            }
-        }
-    }
-
-    let no_state = if let Ok(Some(s)) = git_repo.get_git_config_item("nostr.nostate", None) {
-        s == "true"
+    let mut relay_strings = if args.relays.is_empty() {
+        relays_default
     } else {
-        false
+        args.relays.clone()
+    };
+    let mut blossom_strings = if args.blossoms.is_empty() {
+        blossoms_default
+    } else {
+        args.blossoms.clone()
     };
-    if no_state
+
+    apply_grasp_infrastructure(
+        &selected_grasp_servers,
+        &mut git_servers,
+        &mut relay_strings,
+        &mut blossom_strings,
+        &user_ref.public_key,
+        &identifier,
+    )?;
+
+    // --- Interactive: nostr.nostate prompt ---
+    if interactive
+        && no_state
         && Interactor::default().confirm(
             PromptConfirmParms::default()
                 .with_prompt("store state on nostr? required for nostr-permissioned git servers")
                 .with_default(true),
         )?
     {
-        // TODO check if grasp servers in use and if so turn this off:
         if git_repo
             .get_git_config_item("nostr.nostate", Some(true))
             .unwrap_or(None)
@@ -383,214 +836,146 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
         }
     }
 
-    let git_server = if args.clone_url.is_empty() {
-        let grasp_server_git_servers: Vec<String> = git_server_defaults
+    // --- Git servers (interactive prompting) ---
+    let git_servers = if !args.clone.is_empty() || !interactive {
+        git_servers
+    } else {
+        prompt_git_servers(git_servers, &selected_grasp_servers, simple_mode)?
+    };
+
+    // --- Relays ---
+    let relays: Vec<RelayUrl> = if !args.relays.is_empty() || !interactive {
+        relay_strings
             .iter()
-            .filter(|s| is_grasp_server_clone_url(s))
-            .cloned()
+            .filter_map(|r| parse_relay_url(r).ok())
+            .collect()
+    } else if simple_mode {
+        let grasp_relay_urls: Vec<String> = selected_grasp_servers
+            .iter()
+            .filter_map(|r| format_grasp_server_url_as_relay_url(r).ok())
             .collect();
-        let mut additional_server_options: Vec<String> = git_server_defaults
+        let mut options: Vec<String> = relay_strings
             .iter()
-            .filter(|s| !is_grasp_server_clone_url(s))
+            .filter(|s| !grasp_relay_urls.iter().any(|r| s.as_str() == r))
             .cloned()
             .collect();
-
-        if simple_mode && !selected_grasp_servers.is_empty() {
-            if additional_server_options.is_empty() {
-                git_server_defaults
-            } else {
-                // additional git servers were listed
-                let selected = loop {
-                    let selections: Vec<bool> = vec![true; additional_server_options.len()];
-                    let selected = multi_select_with_custom_value(
-                        "additional git server(s) on top of grasp servers",
-                        "git server remote url",
-                        additional_server_options,
-                        selections,
-                        |s| {
-                            CloneUrl::from_str(s)
-                                .map(|_| s.to_string())
-                                .context(format!("Invalid git server URL format: {s}"))
-                        },
-                    )?;
-
-                    if selected.is_empty() || Interactor::default().choice(
-                    PromptChoiceParms::default()
-                        .with_prompt("if you or another maintainer start pushing directly to these, nostr will be out of date")
-                        .dont_report()
-                        .with_choices(vec![
-                            "I'll always push to the nostr remote".to_string(),
-                            "change setup".to_string(),
-                        ])
-                        .with_default(0),
-                        )? == 1 {
-                        additional_server_options = selected;
-                        continue
-                    }
-                    break selected;
-                };
-                show_multi_input_prompt_success("additional git servers", &selected);
-                let mut combined = grasp_server_git_servers;
-                combined.extend(selected);
-                combined
+        let mut selections: Vec<bool> = vec![true; options.len()];
+        for relay in client.get_relay_default_set().clone() {
+            if !options.iter().any(|r| r.contains(&relay))
+                && !grasp_relay_urls.iter().any(|r| relay.contains(r))
+            {
+                options.push(relay);
+                selections.push(selections.is_empty());
             }
-        } else {
-            // show all git servers
-            let selections: Vec<bool> = vec![true; git_server_defaults.len()];
-
-            let selected = multi_select_with_custom_value(
-                "git server remote url(s)",
-                "git server remote url",
-                git_server_defaults,
-                selections,
-                |s| {
-                    CloneUrl::from_str(s)
-                        .map(|_| s.to_string())
-                        .context(format!("Invalid git server URL format: {s}"))
-                },
-            )?;
-            show_multi_input_prompt_success("git servers", &selected);
-            selected
         }
+        let selected = multi_select_with_custom_value(
+            "additional nostr relays on top of nostr-relays - 1 or 2 public relays are reccomended",
+            "nostr relay",
+            options,
+            selections,
+            |s| {
+                parse_relay_url(s)
+                    .map(|_| s.to_string())
+                    .context(format!("Invalid relay URL format: {s}"))
+            },
+        )?;
+        show_multi_input_prompt_success("additional nostr relays", &selected);
+        [
+            grasp_relay_urls
+                .iter()
+                .filter_map(|r| parse_relay_url(r).ok())
+                .collect::<Vec<RelayUrl>>(),
+            selected
+                .iter()
+                .filter_map(|r| parse_relay_url(r).ok())
+                .collect::<Vec<RelayUrl>>(),
+        ]
+        .concat()
     } else {
-        git_server_defaults
+        // advanced interactive
+        let selections: Vec<bool> = vec![true; relay_strings.len()];
+        let selected = multi_select_with_custom_value(
+            "nostr relays",
+            "nostr relay",
+            relay_strings,
+            selections,
+            |s| {
+                parse_relay_url(s)
+                    .map(|_| s.to_string())
+                    .context(format!("Invalid relay URL format: {s}"))
+            },
+        )?;
+        show_multi_input_prompt_success("nostr relays", &selected);
+        selected
+            .iter()
+            .filter_map(|r| parse_relay_url(r).ok())
+            .collect()
     };
 
-    let relays: Vec<RelayUrl> = {
-        if simple_mode {
-            let formatted_selected_grasp_servers: Vec<String> = selected_grasp_servers
-                .iter()
-                .filter_map(|r| format_grasp_server_url_as_relay_url(r).ok())
-                .collect();
-            let mut options: Vec<String> = relay_defaults
-                .iter()
-                .filter(|s| {
-                    !formatted_selected_grasp_servers
-                        .iter()
-                        .any(|r| s.as_str() == r)
-                })
-                .cloned()
-                .collect();
-
-            let mut selections: Vec<bool> = vec![true; options.len()];
-
-            // add fallback relays as options
-            for relay in client.get_relay_default_set().clone() {
-                if !options.iter().any(|r| r.contains(&relay))
-                    && !formatted_selected_grasp_servers
-                        .iter()
-                        .any(|r| relay.contains(r))
-                {
-                    options.push(relay);
-                    selections.push(selections.is_empty());
-                }
-            }
+    // --- Blossoms ---
+    let blossoms: Vec<Url> = if !args.blossoms.is_empty() || !interactive {
+        blossom_strings
+            .iter()
+            .filter_map(|b| Url::parse(b).ok())
+            .collect()
+    } else if !simple_mode {
+        let selections: Vec<bool> = vec![true; blossom_strings.len()];
+        let selected = multi_select_with_custom_value(
+            "blossom servers",
+            "blossom server",
+            blossom_strings,
+            selections,
+            |s| {
+                format_grasp_server_url_as_blossom_url(s)
+                    .context(format!("Invalid blossom URL format: {s}"))
+            },
+        )?;
+        show_multi_input_prompt_success("blossom servers", &selected);
+        selected.iter().filter_map(|b| Url::parse(b).ok()).collect()
+    } else {
+        blossom_strings
+            .iter()
+            .filter_map(|b| Url::parse(b).ok())
+            .collect()
+    };
 
-            let selected = multi_select_with_custom_value(
-                "additional nostr relays on top of nostr-relays - 1 or 2 public relays are reccomended",
-                "nostr relay",
-                options,
-                selections,
-                |s| {
-                    parse_relay_url(s)
-                        .map(|_| s.to_string())
-                        .context(format!("Invalid relay URL format: {s}"))
-                },
-            )?;
-            show_multi_input_prompt_success("additional nostr relays", &selected);
-            [
-                formatted_selected_grasp_servers
-                    .iter()
-                    .filter_map(|r| parse_relay_url(r).ok())
-                    .collect::<Vec<RelayUrl>>(),
-                selected
-                    .iter()
-                    .filter_map(|r| parse_relay_url(r).ok())
-                    .collect::<Vec<RelayUrl>>(),
-            ]
-            .concat()
-        } else {
-            let selections: Vec<bool> = vec![true; relay_defaults.len()];
-            if args.relays.is_empty() {
-                let selected = multi_select_with_custom_value(
-                    "nostr relays",
-                    "nostr relay",
-                    relay_defaults,
-                    selections,
-                    |s| {
-                        parse_relay_url(s)
-                            .map(|_| s.to_string())
-                            .context(format!("Invalid relay URL format: {s}"))
-                    },
-                )?;
-                show_multi_input_prompt_success("nostr relays", &selected);
-                selected
-                    .iter()
-                    .filter_map(|r| parse_relay_url(r).ok())
-                    .collect()
-            } else {
-                relay_defaults
-                    .iter()
-                    .filter_map(|r| parse_relay_url(r).ok())
-                    .collect()
+    // --- Maintainers ---
+    let maintainers_default = if let Some(ref mr) = my_ref {
+        let mut m = vec![*my_pubkey];
+        for pk in &mr.maintainers {
+            if !m.contains(pk) {
+                m.push(*pk);
             }
         }
-    };
-
-    let blossoms: Vec<Url> = {
-        if simple_mode || has_server_and_relay_flags {
-            blossoms_defaults
-                .iter()
-                .filter_map(|b| Url::parse(b).ok())
-                .collect()
+        m
+    } else if let Some(coord) = state.coordinate() {
+        let trusted = coord.coordinate.public_key;
+        if trusted == *my_pubkey {
+            vec![*my_pubkey]
         } else {
-            let selections: Vec<bool> = vec![true; blossoms_defaults.len()];
-            if args.blossoms.is_empty() {
-                let selected = multi_select_with_custom_value(
-                    "blossom servers",
-                    "blossom server",
-                    blossoms_defaults,
-                    selections,
-                    |s| {
-                        format_grasp_server_url_as_blossom_url(s)
-                            .context(format!("Invalid blossom URL format: {s}"))
-                    },
-                )?;
-                show_multi_input_prompt_success("nostr relays", &selected);
-                selected.iter().filter_map(|b| Url::parse(b).ok()).collect()
-            } else {
-                blossoms_defaults
-                    .iter()
-                    .filter_map(|b| Url::parse(b).ok())
-                    .collect()
-            }
+            vec![*my_pubkey, trusted]
         }
+    } else {
+        vec![*my_pubkey]
     };
 
-    let default_maintainers = {
-        let mut maintainers = vec![user_ref.public_key];
-        if args.other_maintainers.is_empty() {
-            if let Some(repo_ref) = &repo_ref {
-                for m in &repo_ref.maintainers {
-                    if !maintainers.contains(m) {
-                        maintainers.push(*m);
-                    }
-                }
-            }
-        } else {
-            for m in &args.other_maintainers {
-                if let Ok(pubkey) = PublicKey::from_bech32(m).context("invalid npub") {
-                    if !maintainers.contains(&pubkey) {
-                        maintainers.push(pubkey);
-                    }
+    let base_maintainers = if args.other_maintainers.is_empty() {
+        maintainers_default
+    } else {
+        let mut m = vec![user_ref.public_key];
+        for npub in &args.other_maintainers {
+            if let Ok(pk) = PublicKey::from_bech32(npub) {
+                if !m.contains(&pk) {
+                    m.push(pk);
                 }
             }
         }
-        maintainers
+        m
     };
 
-    let maintainers: Vec<PublicKey> = if args.other_maintainers.is_empty() {
-        if default_maintainers.len() == 1
+    let maintainers = if !args.other_maintainers.is_empty()
+        || !interactive
+        || (base_maintainers.len() == 1
             && Interactor::default().choice(
                 PromptChoiceParms::default()
                     .with_prompt("add other maintainers now?")
@@ -600,41 +985,44 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
                         "add maintainers".to_string(),
                     ])
                     .with_default(0),
-            )? == 0
-        {
-            default_maintainers
-        } else {
-            let selections: Vec<bool> = vec![true; default_maintainers.len()];
-
-            let selected = multi_select_with_custom_value(
-                "maintainers",
-                "maintainer npub",
-                default_maintainers
-                    .iter()
-                    .filter_map(|m| m.to_bech32().ok())
-                    .collect(),
-                selections,
-                |s| {
-                    extract_npub(s)
-                        .map(|_| s.to_string())
-                        .context(format!("Invalid npub: {s}"))
-                },
-            )?;
-            show_multi_input_prompt_success("maintainers", &selected);
-            selected
-                .iter()
-                .filter_map(|npub| PublicKey::parse(npub).ok())
-                .collect()
-        }
+            )? == 0)
+    {
+        base_maintainers
     } else {
-        default_maintainers
+        let selections: Vec<bool> = vec![true; base_maintainers.len()];
+        let selected = multi_select_with_custom_value(
+            "maintainers",
+            "maintainer npub",
+            base_maintainers
+                .iter()
+                .filter_map(|m| m.to_bech32().ok())
+                .collect(),
+            selections,
+            |s| {
+                extract_npub(s)
+                    .map(|_| s.to_string())
+                    .context(format!("Invalid npub: {s}"))
+            },
+        )?;
+        show_multi_input_prompt_success("maintainers", &selected);
+        selected
+            .iter()
+            .filter_map(|npub| PublicKey::parse(npub).ok())
+            .collect()
     };
 
-    if selected_grasp_servers.is_empty() && git_server.iter().any(|s| s.contains("github.com") || s.contains("codeberg.org")) && Interactor::default().confirm(
+    // --- Interactive: github/codeberg warning ---
+    if interactive
+        && selected_grasp_servers.is_empty()
+        && git_servers
+            .iter()
+            .any(|s| s.contains("github.com") || s.contains("codeberg.org"))
+        && Interactor::default().confirm(
             PromptConfirmParms::default()
                 .with_prompt("you have listed github / codeberg. Are you or other maintainers planning on pushing directly to github / codeberg rather than using your shiny new nostr clone url which will do this for you?")
                 .with_default(false),
-        )? {
+        )?
+    {
         println!("This means people using the nostr URL won't get your latest branch updates.");
         if Interactor::default().confirm(
             PromptConfirmParms::default()
@@ -645,124 +1033,228 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
         }
     }
 
-    let gitworkshop_url = NostrUrlDecoded {
-        original_string: String::new(),
-        coordinate: Nip19Coordinate {
-            coordinate: Coordinate {
-                public_key: user_ref.public_key,
-                kind: Kind::GitRepoAnnouncement,
-                identifier: identifier.clone(),
-            },
-            relays: if let Some(relay) = relays.first() {
-                vec![relay.clone()]
-            } else {
-                vec![]
-            },
-        },
-        protocol: None,
-        ssh_key_file: None,
-        nip05: None,
-    }
-    .to_string()
-    .replace("nostr://", "https://gitworkshop.dev/");
-
-    let web: Vec<String> = if args.web.is_empty() {
-        let web_default = if let Some(repo_ref) = &repo_ref {
-            if repo_ref
-                .web
-                .clone()
-                .join(" ")
-                // replace legacy gitworkshop.dev url format with new one
-                .contains(format!("https://gitworkshop.dev/repo/{}", &identifier).as_str())
-            {
-                gitworkshop_url.clone()
-            } else {
-                repo_ref.web.clone().join(" ")
-            }
-        } else {
-            gitworkshop_url.clone()
-        };
+    // --- Web ---
+    let gitworkshop_url = build_gitworkshop_url(&user_ref.public_key, &identifier, relays.first());
+    let web_default = resolve_web(&args.web, state, &identifier, &gitworkshop_url);
 
-        if simple_mode {
-            web_default
-        } else {
-            Interactor::default().input(
+    let web = if !args.web.is_empty() || !interactive || simple_mode {
+        web_default
+    } else {
+        // advanced interactive
+        let web_default_str = web_default.join(" ");
+        Interactor::default()
+            .input(
                 PromptInputParms::default()
                     .with_prompt("repo website")
                     .optional()
-                    .with_default(web_default),
+                    .with_default(web_default_str)
+                    .with_flag_name("--web"),
             )?
-        }
-        .split(' ')
-        .map(std::string::ToString::to_string)
-        .collect()
-    } else {
-        args.web.clone()
+            .split(' ')
+            .map(std::string::ToString::to_string)
+            .collect()
     };
 
-    let earliest_unique_commit = if let Some(t) = &args.earliest_unique_commit {
-        t.clone()
-    } else {
-        let mut earliest_unique_commit = if let Some(repo_ref) = &repo_ref {
-            repo_ref.root_commit.clone()
-        } else {
-            root_commit.to_string()
-        };
-        if simple_mode {
-            earliest_unique_commit
+    // --- Earliest unique commit ---
+    // Cascade: my event -> consolidated RepoRef (trusted maintainer's) -> local
+    // root commit
+    let my_euc = my_ref
+        .as_ref()
+        .map(|mr| &mr.root_commit)
+        .filter(|c| !c.is_empty());
+    let repo_euc = state
+        .repo_ref()
+        .map(|rr| &rr.root_commit)
+        .filter(|c| !c.is_empty());
+    let euc_default = my_euc
+        .or(repo_euc)
+        .cloned()
+        .unwrap_or_else(|| root_commit.to_string());
+
+    let earliest_unique_commit = if let Some(commit) = &args.earliest_unique_commit {
+        if let Ok(exists) = git_repo.does_commit_exist(commit) {
+            if !exists {
+                bail!("earliest unique commit does not exist on current repository");
+            }
         } else {
-            println!(
-                "the earliest unique commit helps with discoverability. It defaults to the root commit. Only change this if your repo has completely forked off an has formed its own identity."
-            );
-            loop {
-                earliest_unique_commit = Interactor::default().input(
-                    PromptInputParms::default()
-                        .with_prompt("earliest unique commit (to help with discoverability)")
-                        .with_default(earliest_unique_commit.clone()),
-                )?;
-                if let Ok(exists) = git_repo.does_commit_exist(&earliest_unique_commit) {
-                    if exists {
-                        break earliest_unique_commit;
-                    }
-                    println!("commit does not exist on current repository");
-                } else {
-                    println!("commit id not formatted correctly");
+            bail!("earliest unique commit id not formatted correctly");
+        }
+        if commit.len() != 40 {
+            bail!("earliest unique commit id must be 40 characters long");
+        }
+        commit.clone()
+    } else if interactive && !simple_mode {
+        println!(
+            "the earliest unique commit helps with discoverability. It defaults to the root commit. Only change this if your repo has completely forked off an has formed its own identity."
+        );
+        let mut result = euc_default.clone();
+        loop {
+            result = Interactor::default().input(
+                PromptInputParms::default()
+                    .with_prompt("earliest unique commit (to help with discoverability)")
+                    .with_default(result.clone())
+                    .with_flag_name("--earliest-unique-commit"),
+            )?;
+            if let Ok(exists) = git_repo.does_commit_exist(&result) {
+                if exists && result.len() == 40 {
+                    break;
                 }
-                if earliest_unique_commit.len().ne(&40) {
-                    println!("commit id must be 40 characters long");
+                if !exists {
+                    println!("commit does not exist on current repository");
                 }
+            } else {
+                println!("commit id not formatted correctly");
+            }
+            if result.len() != 40 {
+                println!("commit id must be 40 characters long");
             }
         }
+        result
+    } else {
+        euc_default
     };
 
-    println!("publishing repostory announcement to nostr...");
+    // --- Hashtags (shared metadata — from latest event, like name/description/web)
+    // ---
+    let hashtags = latest
+        .as_ref()
+        .map_or_else(Vec::new, |lr| lr.hashtags.clone());
 
-    let repo_ref = RepoRef {
-        identifier: identifier.clone(),
+    Ok(ResolvedFields {
+        identifier,
         name,
         description,
-        root_commit: earliest_unique_commit,
-        git_server,
-        web,
-        relays: relays.clone(),
+        git_servers,
+        relays,
         blossoms,
-        hashtags: if let Some(repo_ref) = repo_ref {
-            repo_ref.hashtags
-        } else {
-            vec![]
-        },
+        web,
+        maintainers,
+        earliest_unique_commit,
+        hashtags,
+        selected_grasp_servers,
+    })
+}
+
+/// Interactive prompt for git server selection with simple/advanced modes.
+fn prompt_git_servers(
+    git_servers: Vec<String>,
+    selected_grasp_servers: &[String],
+    simple_mode: bool,
+) -> Result<Vec<String>> {
+    let grasp_server_git_servers: Vec<String> = git_servers
+        .iter()
+        .filter(|s| is_grasp_server_clone_url(s))
+        .cloned()
+        .collect();
+    let mut additional_server_options: Vec<String> = git_servers
+        .iter()
+        .filter(|s| !is_grasp_server_clone_url(s))
+        .cloned()
+        .collect();
+
+    if simple_mode && !selected_grasp_servers.is_empty() {
+        if additional_server_options.is_empty() {
+            return Ok(git_servers);
+        }
+        let selected = loop {
+            let selections: Vec<bool> = vec![true; additional_server_options.len()];
+            let selected = multi_select_with_custom_value(
+                "additional git server(s) on top of grasp servers",
+                "git server remote url",
+                additional_server_options,
+                selections,
+                |s| {
+                    CloneUrl::from_str(s)
+                        .map(|_| s.to_string())
+                        .context(format!("Invalid git server URL format: {s}"))
+                },
+            )?;
+
+            if selected.is_empty()
+                || Interactor::default().choice(
+                    PromptChoiceParms::default()
+                        .with_prompt("if you or another maintainer start pushing directly to these, nostr will be out of date")
+                        .dont_report()
+                        .with_choices(vec![
+                            "I'll always push to the nostr remote".to_string(),
+                            "change setup".to_string(),
+                        ])
+                        .with_default(0),
+                )? == 1
+            {
+                additional_server_options = selected;
+                continue;
+            }
+            break selected;
+        };
+        show_multi_input_prompt_success("additional git servers", &selected);
+        let mut combined = grasp_server_git_servers;
+        combined.extend(selected);
+        Ok(combined)
+    } else {
+        let selections: Vec<bool> = vec![true; git_servers.len()];
+        let selected = multi_select_with_custom_value(
+            "git server remote url(s)",
+            "git server remote url",
+            git_servers,
+            selections,
+            |s| {
+                CloneUrl::from_str(s)
+                    .map(|_| s.to_string())
+                    .context(format!("Invalid git server URL format: {s}"))
+            },
+        )?;
+        show_multi_input_prompt_success("git servers", &selected);
+        Ok(selected)
+    }
+}
+
+#[allow(clippy::too_many_lines)]
+async fn publish_and_finalize(
+    fields: ResolvedFields,
+    signer: Arc<dyn nostr::prelude::NostrSigner>,
+    user_ref: &ngit::login::user::UserRef,
+    client: &mut Client,
+    cli: &Cli,
+    git_repo: &Repo,
+    repo_config_result: &Result<ngit::repo_ref::RepoConfigYaml>,
+) -> Result<()> {
+    let git_repo_path = git_repo.get_path()?;
+
+    // Step 1: Build RepoRef
+    let repo_ref = RepoRef {
+        identifier: fields.identifier.clone(),
+        name: fields.name,
+        description: fields.description,
+        root_commit: fields.earliest_unique_commit,
+        git_server: fields.git_servers,
+        web: fields.web,
+        relays: fields.relays.clone(),
+        blossoms: fields.blossoms,
+        hashtags: fields.hashtags,
         trusted_maintainer: user_ref.public_key,
         maintainers_without_annoucnement: None,
-        maintainers: maintainers.clone(),
+        maintainers: fields.maintainers.clone(),
         events: HashMap::new(),
         nostr_git_url: None,
     };
+
+    // Step 2: Create event
+    println!("publishing repostory announcement to nostr...");
     let repo_event = repo_ref.to_event(&signer).await?;
 
-    let nostr_url_decoded = repo_ref.to_nostr_git_url(&Some(&git_repo));
+    // Step 3: Build nostr URL
+    let nostr_url_decoded = repo_ref.to_nostr_git_url(&Some(git_repo));
 
     let mut events = vec![repo_event];
 
+    // Step 4: Handle state events and push/sync logic
+    let no_state = if let Ok(Some(s)) = git_repo.get_git_config_item("nostr.nostate", None) {
+        s == "true"
+    } else {
+        false
+    };
+
     let (need_push, need_sync) = if std::env::var("NGITTEST").is_ok() || no_state {
         // dont push or sync during tests as git-remote-nostr isn't installed during
         // ngit binary tests
@@ -785,7 +1277,7 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
         if let Some(url) = remote.url() {
             // issue a state event with origin state, to all (inc. new) repo relays
             if let Ok(mut origin_state) =
-                list_from_remote(&Term::stdout(), &git_repo, url, &nostr_url_decoded, false)
+                list_from_remote(&Term::stdout(), git_repo, url, &nostr_url_decoded, false)
             {
                 origin_state.retain(|key, _| {
                     key.starts_with("refs/heads/")
@@ -809,7 +1301,7 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
                 if required_oids.is_empty() {
                     println!("fetching refs missing locally from existing origin...");
                     if let Err(error) = fetch_from_git_server(
-                        &git_repo,
+                        git_repo,
                         &required_oids,
                         url,
                         &nostr_url_decoded,
@@ -839,27 +1331,28 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
         (true, false)
     };
 
+    // Step 5: Publish events
     client.set_signer(signer).await;
 
     send_events(
-        &client,
+        client,
         Some(git_repo_path),
         events,
         user_ref.relays.write(),
-        relays.clone(),
-        !cli_args.disable_cli_spinners,
+        fields.relays.clone(),
+        !cli.disable_cli_spinners,
         false,
     )
     .await?;
 
-    // TODO - does this git config item do more harm than good?
+    // Step 6: Set git config
     git_repo.save_git_config_item(
         "nostr.repo",
         &Nip19Coordinate {
             coordinate: Coordinate {
                 kind: Kind::GitRepoAnnouncement,
                 public_key: user_ref.public_key,
-                identifier: identifier.clone(),
+                identifier: fields.identifier.clone(),
             },
             relays: vec![],
         }
@@ -867,9 +1360,8 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
         false,
     )?;
 
-    // set origin remote
+    // Step 7: Set origin remote
     let nostr_url = nostr_url_decoded.to_string();
-
     if git_repo.git_repo.find_remote("origin").is_ok() {
         git_repo.git_repo.remote_set_url("origin", &nostr_url)?;
     } else {
@@ -877,8 +1369,9 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
     }
     println!("set remote origin to nostr url");
 
+    // Step 8: Push/sync
     if need_push {
-        if selected_grasp_servers.is_empty() {
+        if fields.selected_grasp_servers.is_empty() {
             println!("running `ngit push` to publish your repository data");
         } else {
             let countdown_start = 5;
@@ -894,14 +1387,14 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
             term.flush().unwrap(); // Ensure the output is flushed to the terminal
         }
 
-        if let Err(err) = push_main_or_master_branch(&git_repo) {
+        if let Err(err) = push_main_or_master_branch(git_repo) {
             println!(
                 "your repository announcement was published to nostr but git push exited with an error: {err}"
             );
         }
     }
     if need_sync {
-        if selected_grasp_servers.is_empty() {
+        if fields.selected_grasp_servers.is_empty() {
             println!(
                 "running `ngit sync` to ensure your repository data is available on repository git servers"
             );
@@ -926,27 +1419,25 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
         }
     }
 
-    // println!(
-    //     "any remote branches beginning with `pr/` are open PRs from contributors.
-    // they can submit these by simply pushing a branch with this `pr/` prefix."
-    // );
+    // Step 9: Print share URLs
+    let gitworkshop_url = nostr_url_decoded
+        .to_string()
+        .replace("nostr://", "https://gitworkshop.dev/");
     println!("share your repository: {gitworkshop_url}");
     println!("clone url: {nostr_url}");
 
-    // no longer create a new maintainers.yaml file - its too confusing for users
-    // as it falls out of sync with data in nostr event . update if it already
-    // exists
-
-    let relays = relays
+    // Step 10: Update maintainers.yaml if needed
+    let relays = fields
+        .relays
         .iter()
         .map(std::string::ToString::to_string)
         .collect::<Vec<String>>();
-    if match &repo_config_result {
+    if match repo_config_result {
         Ok(config) => {
             !<std::option::Option<std::string::String> as Clone>::clone(&config.identifier)
                 .unwrap_or_default()
-                .eq(&identifier)
-                || !extract_pks(config.maintainers.clone())?.eq(&maintainers)
+                .eq(&fields.identifier)
+                || !extract_pks(config.maintainers.clone())?.eq(&fields.maintainers)
                 || !config.relays.eq(&relays)
         }
         Err(_) => false,
@@ -954,9 +1445,9 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
         let title_style = Style::new().bold().fg(console::Color::Yellow);
         println!("{}", title_style.apply_to("maintainers.yaml"));
         save_repo_config_to_yaml(
-            &git_repo,
-            identifier.clone(),
-            maintainers.clone(),
+            git_repo,
+            fields.identifier.clone(),
+            fields.maintainers.clone(),
             relays.clone(),
         )?;
         println!(
@@ -974,6 +1465,98 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
     Ok(())
 }
 
+#[allow(clippy::too_many_lines)]
+pub async fn launch(cli_args: &Cli, args: &SubCommandArgs) -> Result<()> {
+    // Phase 1: Local-only setup
+    let git_repo = Repo::discover().context("failed to find a git repository")?;
+    let git_repo_path = git_repo.get_path()?;
+    let root_commit = git_repo
+        .get_root_commit()
+        .context("failed to get root commit of the repository")?;
+    let mut client = Client::new(Params::with_git_config_relay_defaults(&Some(&git_repo)));
+    let (signer, user_ref, _) = login::login_or_signup(
+        &Some(&git_repo),
+        &extract_signer_cli_arguments(cli_args).unwrap_or(None),
+        &cli_args.password,
+        Some(&client),
+        false,
+    )
+    .await?;
+
+    let repo_coordinate = (try_and_get_repo_coordinates_when_remote_unknown(&git_repo).await).ok();
+
+    // Phase 2: Pre-fetch validation (fail fast)
+    let user_has_grasp_list = !user_ref.grasp_list.urls.is_empty();
+    validate_pre_fetch(
+        cli_args,
+        args,
+        repo_coordinate.as_ref(),
+        user_has_grasp_list,
+    )?;
+
+    // Phase 3: Network fetch (only if coordinate exists)
+    let repo_ref = if let Some(repo_coordinate) = &repo_coordinate {
+        fetching_with_report(git_repo_path, &client, repo_coordinate).await?;
+        (get_repo_ref_from_cache(Some(git_repo_path), repo_coordinate).await).ok()
+    } else {
+        None
+    };
+
+    // Phase 4: Determine state + post-fetch validation
+    let state = match (&repo_coordinate, &repo_ref) {
+        (None, _) => InitState::Fresh,
+        (Some(coord), None) => InitState::CoordinateOnly {
+            coordinate: coord.clone(),
+        },
+        (Some(coord), Some(rr)) => {
+            if coord.coordinate.public_key == user_ref.public_key {
+                InitState::MyAnnouncement {
+                    coordinate: coord.clone(),
+                    repo_ref: rr.clone(),
+                }
+            } else if rr.maintainers.contains(&user_ref.public_key) {
+                InitState::CoMaintainer {
+                    coordinate: coord.clone(),
+                    repo_ref: rr.clone(),
+                }
+            } else {
+                InitState::NotListed {
+                    coordinate: coord.clone(),
+                    repo_ref: rr.clone(),
+                }
+            }
+        }
+    };
+
+    validate_post_fetch(cli_args, args, &state)?;
+
+    // Phase 5: Resolve all fields
+    let repo_config_result = get_repo_config_from_yaml(&git_repo);
+    let fields = resolve_fields(
+        &state,
+        &user_ref,
+        args,
+        cli_args,
+        &git_repo,
+        &root_commit.to_string(),
+        &client,
+        &repo_config_result,
+        cli_args.interactive,
+    )?;
+
+    // Phase 6: Build and publish
+    publish_and_finalize(
+        fields,
+        signer,
+        &user_ref,
+        &mut client,
+        cli_args,
+        &git_repo,
+        &repo_config_result,
+    )
+    .await
+}
+
 fn format_grasp_server_url_as_clone_url(
     url: &str,
     public_key: &PublicKey,