chore(v2): remove dead update_ssh_session method and fix stale comment

- Remove unused SessionDb::update_ssh_session() and its test (SshDialog
  was deleted in P10, this method had no callers)
- Fix stale TilingGrid reference in AgentPane comment
- Eliminates the last Rust compiler warning
This commit is contained in:
Hibryda 2026-03-07 16:51:15 +01:00
parent 67c416dc10
commit 6ea1ed1dfd
2 changed files with 1 additions and 41 deletions

View file

@ -363,25 +363,6 @@ impl SessionDb {
Ok(()) Ok(())
} }
pub fn update_ssh_session(&self, session: &SshSession) -> Result<(), String> {
let conn = self.conn.lock().unwrap();
conn.execute(
"UPDATE ssh_sessions SET name = ?1, host = ?2, port = ?3, username = ?4, key_file = ?5, folder = ?6, color = ?7, last_used_at = ?8 WHERE id = ?9",
params![
session.name,
session.host,
session.port,
session.username,
session.key_file,
session.folder,
session.color,
session.last_used_at,
session.id,
],
).map_err(|e| format!("SSH update failed: {e}"))?;
Ok(())
}
// --- v3: Agent message persistence --- // --- v3: Agent message persistence ---
pub fn save_agent_messages( pub fn save_agent_messages(
@ -790,27 +771,6 @@ mod tests {
assert_eq!(sessions[0].id, "ssh2"); assert_eq!(sessions[0].id, "ssh2");
} }
#[test]
fn test_update_ssh_session() {
let db = make_db();
let mut s = make_ssh_session("ssh1", "Old Name");
db.save_ssh_session(&s).unwrap();
s.name = "New Name".to_string();
s.host = "new.example.com".to_string();
s.port = 2222;
s.last_used_at = 9999;
db.update_ssh_session(&s).unwrap();
let sessions = db.list_ssh_sessions().unwrap();
assert_eq!(sessions.len(), 1);
assert_eq!(sessions[0].name, "New Name");
assert_eq!(sessions[0].host, "new.example.com");
assert_eq!(sessions[0].port, 2222);
assert_eq!(sessions[0].last_used_at, 9999);
// created_at should be unchanged (UPDATE doesn't touch it)
assert_eq!(sessions[0].created_at, 1000);
}
#[test] #[test]
fn test_ssh_session_upsert() { fn test_ssh_session_upsert() {

View file

@ -95,7 +95,7 @@
}); });
// NOTE: Do NOT stop agents in onDestroy — it fires on layout changes/remounts, // NOTE: Do NOT stop agents in onDestroy — it fires on layout changes/remounts,
// not just explicit close. Stop-on-close is handled by TilingGrid. // not just explicit close. Stop-on-close is handled by workspace teardown.
let followUpPrompt = $state(''); let followUpPrompt = $state('');