fix: address Greptile review bugs in archive and status

- keys.rs: remove redundant pub_path.exists() guard after private key
  rename — public key existence is already ensured before the rename,
  so re-checking creates a silent partial-archive path that leaves old
  traces unverifiable after rotation
- status.rs: fix misleading ok() prefix in Gemini (true,true) arm when
  tools.enableHooks is not set; consolidate into a single warn() line

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: codeprakhar25

src/commands/status.rs

@@ -307,11 +307,7 @@ fn print_agent_hook_status() {
                         );
                     } else {
                         println!(
-                            "  {} agent hook     gemini-cli registered; antigravity rule set",
-                            prefix(ok())
-                        );
-                        println!(
-                            "  {} agent hook     gemini tools.enableHooks not set — re-run 'agentdiff configure'",
+                            "  {} agent hook     gemini-cli registered; antigravity rule set but tools.enableHooks not set — re-run 'agentdiff configure'",
                             prefix(warn())
                         );
                         any_missing = true;

src/keys.rs

@@ -64,10 +64,8 @@ pub fn archive_current_keypair() -> Result<Option<PathBuf>> {
     let dest_pub = dest.join("public.key");
     std::fs::rename(&priv_path, &dest_priv)
         .with_context(|| format!("archiving private key to {}", dest_priv.display()))?;
-    if pub_path.exists() {
-        std::fs::rename(&pub_path, &dest_pub)
-            .with_context(|| format!("archiving public key to {}", dest_pub.display()))?;
-    }
+    std::fs::rename(&pub_path, &dest_pub)
+        .with_context(|| format!("archiving public key to {}", dest_pub.display()))?;
 
     let meta = ArchivedKeyMeta {
         key_id: kid,