fix: replace process::exit in doctor

Author: junhsss

src/commands/doctor.rs

@@ -79,7 +79,9 @@ pub async fn run(args: Args) -> anyhow::Result<()> {
     }
 
     if has_fail {
-        std::process::exit(1);
+        return Err(
+            crate::util::output::SilentExit(crate::util::output::exit_code::GENERAL).into(),
+        );
     }
 
     Ok(())

src/commands/init/agents.rs

@@ -14,6 +14,9 @@ use crate::status;
 
 const SKILL_CONTENT: &str = include_str!("../../../skills/steel-browser/SKILL.md");
 
+const CURSOR_DESCRIPTION: &str =
+    "Steel browser automation CLI for web tasks (scrape, screenshot, interactive browser sessions)";
+
 pub struct DetectedAgent {
     pub name: &'static str,
     pub install_path: PathBuf,
@@ -70,9 +73,7 @@ pub fn detect_agents() -> Vec<DetectedAgent> {
 /// stripping its YAML frontmatter and prepending Cursor-specific frontmatter.
 fn cursor_rule_content() -> String {
     let body = strip_frontmatter(SKILL_CONTENT);
-    format!(
-        "---\ndescription: Steel browser automation CLI for web tasks (scrape, screenshot, interactive browser sessions)\nalwaysApply: false\n---\n\n{body}"
-    )
+    format!("---\ndescription: {CURSOR_DESCRIPTION}\nalwaysApply: false\n---\n\n{body}")
 }
 
 fn strip_frontmatter(s: &str) -> &str {

src/commands/init/mod.rs

@@ -29,7 +29,7 @@ pub async fn run(args: Args) -> anyhow::Result<()> {
     // Step 1: login (no-ops if already logged in).
     login::run(login::Args {}).await?;
 
-    // Step 2: preflight check (calls process::exit on failure).
+    // Step 2: preflight check.
     status!("");
     doctor::run(doctor::Args { preflight: true }).await?;
 

src/util/output.rs

@@ -69,6 +69,19 @@ pub mod exit_code {
     pub const API_SERVER: i32 = 6;
 }
 
+/// Sentinel error: the command already printed its own output and reported
+/// failure; the process should exit with `code` without printing anything else.
+#[derive(Debug)]
+pub struct SilentExit(pub i32);
+
+impl std::fmt::Display for SilentExit {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        write!(f, "exit {}", self.0)
+    }
+}
+
+impl std::error::Error for SilentExit {}
+
 // --- Pure formatting functions (no I/O) ---
 
 fn format_success(json_mode: bool, data: &Value, text: &str) -> Option<String> {
@@ -172,6 +185,10 @@ pub fn success_text(data: Value) {
 
 /// Format a top-level error for output, then exit. Called from main.
 pub fn handle_error(err: &anyhow::Error) -> ! {
+    if let Some(SilentExit(code)) = err.downcast_ref::<SilentExit>() {
+        std::process::exit(*code);
+    }
+
     let (code, error_code, hint) = classify_error(err);
     let msg = format!("{err:#}");