sqlite_graphrag/commands/
forget.rs

1//! Handler for the `forget` CLI subcommand.
2
3use crate::errors::AppError;
4use crate::i18n::errors_msg;
5use crate::output;
6use crate::paths::AppPaths;
7use crate::storage::connection::open_rw;
8use crate::storage::memories;
9use rusqlite::{params, OptionalExtension};
10use serde::Serialize;
11
12#[derive(clap::Args)]
13#[command(after_long_help = "EXAMPLES:\n  \
14    # Soft-delete a memory by name (positional form)\n  \
15    sqlite-graphrag forget onboarding\n\n  \
16    # Soft-delete using the named flag form\n  \
17    sqlite-graphrag forget --name onboarding\n\n  \
18    # Soft-delete from a specific namespace\n  \
19    sqlite-graphrag forget onboarding --namespace my-project")]
20pub struct ForgetArgs {
21    /// Memory name as a positional argument. Alternative to `--name`.
22    #[arg(
23        value_name = "NAME",
24        conflicts_with = "name",
25        help = "Memory name to soft-delete; alternative to --name"
26    )]
27    pub name_positional: Option<String>,
28    /// Memory name to soft-delete. The row is preserved with `deleted_at` set, recoverable via `restore`.
29    /// Use `purge` to permanently remove soft-deleted memories.
30    #[arg(long)]
31    pub name: Option<String>,
32    #[arg(
33        long,
34        help = "Namespace (env: SQLITE_GRAPHRAG_NAMESPACE, default: global)"
35    )]
36    pub namespace: Option<String>,
37    #[arg(long, hide = true, help = "No-op; JSON is always emitted on stdout")]
38    pub json: bool,
39    #[arg(long, env = "SQLITE_GRAPHRAG_DB_PATH")]
40    pub db: Option<String>,
41}
42
43#[derive(Serialize)]
44struct ForgetResponse {
45    /// Outcome of the forget operation: `soft_deleted`, `already_deleted`, or `not_found`.
46    action: String,
47    /// True only when this invocation actively transitioned the memory from live to soft-deleted.
48    forgotten: bool,
49    name: String,
50    namespace: String,
51    /// Unix epoch seconds when the memory was soft-deleted; `None` when `action="not_found"`.
52    #[serde(skip_serializing_if = "Option::is_none")]
53    deleted_at: Option<i64>,
54    /// RFC 3339 UTC timestamp parallel to `deleted_at` for ISO 8601 parsers.
55    #[serde(skip_serializing_if = "Option::is_none")]
56    deleted_at_iso: Option<String>,
57    /// Total execution time in milliseconds from handler start to serialisation.
58    elapsed_ms: u64,
59}
60
61pub fn run(args: ForgetArgs) -> Result<(), AppError> {
62    let start = std::time::Instant::now();
63    tracing::debug!(target: "forget", name = ?args.name_positional.as_deref().or(args.name.as_deref()), "soft-deleting memory");
64    // Resolve name from positional or --name flag; both are optional, at least one is required.
65    let name = args.name_positional.or(args.name).ok_or_else(|| {
66        AppError::Validation("name required: pass as positional argument or via --name".to_string())
67    })?;
68    let namespace = crate::namespace::resolve_namespace(args.namespace.as_deref())?;
69    let paths = AppPaths::resolve(args.db.as_deref())?;
70    crate::storage::connection::ensure_db_ready(&paths)?;
71
72    let conn = open_rw(&paths.db)?;
73
74    // Probe state without filtering on `deleted_at` so we can distinguish
75    // `not_found` (no row) from `already_deleted` (row with deleted_at set)
76    // from the live case (deleted_at IS NULL) handled by `soft_delete`.
77    let probe: Option<(i64, Option<i64>)> = conn
78        .query_row(
79            "SELECT id, deleted_at FROM memories WHERE namespace = ?1 AND name = ?2",
80            params![namespace, name],
81            |r| Ok((r.get::<_, i64>(0)?, r.get::<_, Option<i64>>(1)?)),
82        )
83        .optional()?;
84
85    let (action, forgotten, deleted_at, memory_id) = match probe {
86        None => ("not_found", false, None, None),
87        Some((id, Some(existing))) => ("already_deleted", false, Some(existing), Some(id)),
88        Some((id, None)) => {
89            // G39 Passo 4 (v1.0.69): remove the embedding vector BEFORE the
90            // soft-delete so we do not leave a `vec_memories` row that will
91            // show up as `vec_memories_orphaned` in `health --json`. The
92            // operation is best-effort: a failure is logged but does not
93            // abort the soft-delete (the user-visible action is the same).
94            if let Err(e) = memories::delete_vec(&conn, id) {
95                tracing::warn!(
96                    target: "forget",
97                    memory_id = id,
98                    error = %e,
99                    "vec cleanup before soft-delete failed — orphan vector may be left",
100                );
101            }
102            let ok = memories::soft_delete(&conn, &namespace, &name)?;
103            if !ok {
104                // Race: row was concurrently soft-deleted between probe and update.
105                // Re-read to get the current `deleted_at`.
106                let current: Option<i64> = conn
107                    .query_row(
108                        "SELECT deleted_at FROM memories WHERE id = ?1",
109                        params![id],
110                        |r| r.get::<_, Option<i64>>(0),
111                    )
112                    .optional()?
113                    .flatten();
114                ("already_deleted", false, current, Some(id))
115            } else {
116                let ts: Option<i64> = conn
117                    .query_row(
118                        "SELECT deleted_at FROM memories WHERE id = ?1",
119                        params![id],
120                        |r| r.get::<_, Option<i64>>(0),
121                    )
122                    .optional()?
123                    .flatten();
124                ("soft_deleted", true, ts, Some(id))
125            }
126        }
127    };
128
129    if forgotten {
130        if let Some(id) = memory_id {
131            // FTS5 external-content: manual `DELETE FROM fts_memories WHERE rowid=?`
132            // corrupts the index. The correct cleanup happens via the `trg_fts_ad` trigger
133            // when `purge` physically removes the row from `memories`. Between soft-delete
134            // and purge, FTS queries filter `m.deleted_at IS NULL` in the JOIN.
135            if let Err(e) = memories::delete_vec(&conn, id) {
136                tracing::warn!(target: "forget", memory_id = id, error = %e, "vec cleanup failed — orphan vector left");
137            }
138        }
139    }
140
141    conn.execute_batch("PRAGMA wal_checkpoint(TRUNCATE);")?;
142
143    if action == "not_found" {
144        return Err(AppError::NotFound(errors_msg::memory_not_found(
145            &name, &namespace,
146        )));
147    }
148
149    let deleted_at_iso = deleted_at.map(crate::tz::epoch_to_iso);
150    let response = ForgetResponse {
151        action: action.to_string(),
152        forgotten,
153        name: name.clone(),
154        namespace: namespace.clone(),
155        deleted_at,
156        deleted_at_iso,
157        elapsed_ms: start.elapsed().as_millis() as u64,
158    };
159    output::emit_json(&response)?;
160
161    Ok(())
162}
163
164#[cfg(test)]
165mod tests {
166    use super::*;
167
168    #[test]
169    fn forget_response_serializes_basic_fields() {
170        let resp = ForgetResponse {
171            action: "soft_deleted".to_string(),
172            forgotten: true,
173            name: "my-memory".to_string(),
174            namespace: "global".to_string(),
175            deleted_at: Some(1_700_000_000),
176            deleted_at_iso: Some("2023-11-14T22:13:20+00:00".to_string()),
177            elapsed_ms: 5,
178        };
179        let json = serde_json::to_value(&resp).expect("serialization failed");
180        assert_eq!(json["action"], "soft_deleted");
181        assert_eq!(json["forgotten"], true);
182        assert_eq!(json["name"], "my-memory");
183        assert_eq!(json["namespace"], "global");
184        assert_eq!(json["deleted_at"], 1_700_000_000i64);
185        assert!(json["deleted_at_iso"].is_string());
186        assert!(json["elapsed_ms"].is_number());
187    }
188
189    #[test]
190    fn forget_response_action_soft_deleted_implies_forgotten_true() {
191        let resp = ForgetResponse {
192            action: "soft_deleted".to_string(),
193            forgotten: true,
194            name: "test".to_string(),
195            namespace: "ns".to_string(),
196            deleted_at: Some(42),
197            deleted_at_iso: Some(crate::tz::epoch_to_iso(42)),
198            elapsed_ms: 1,
199        };
200        assert_eq!(resp.action, "soft_deleted");
201        assert!(resp.forgotten);
202        assert_eq!(resp.deleted_at, Some(42));
203        assert!(resp.deleted_at_iso.is_some());
204    }
205
206    #[test]
207    fn forget_response_already_deleted_preserves_timestamp() {
208        let resp = ForgetResponse {
209            action: "already_deleted".to_string(),
210            forgotten: false,
211            name: "abc".to_string(),
212            namespace: "my-project".to_string(),
213            deleted_at: Some(1_650_000_000),
214            deleted_at_iso: Some(crate::tz::epoch_to_iso(1_650_000_000)),
215            elapsed_ms: 2,
216        };
217        let json = serde_json::to_value(&resp).expect("serialization failed");
218        assert_eq!(json["action"], "already_deleted");
219        assert_eq!(json["forgotten"], false);
220        assert_eq!(json["deleted_at"], 1_650_000_000i64);
221        assert!(json["deleted_at_iso"].is_string());
222    }
223
224    #[test]
225    fn forget_response_not_found_omits_deleted_at_fields() {
226        let resp = ForgetResponse {
227            action: "not_found".to_string(),
228            forgotten: false,
229            name: "phantom".to_string(),
230            namespace: "global".to_string(),
231            deleted_at: None,
232            deleted_at_iso: None,
233            elapsed_ms: 0,
234        };
235        let json = serde_json::to_value(&resp).expect("serialization failed");
236        assert_eq!(json["action"], "not_found");
237        assert_eq!(json["forgotten"], false);
238        // skip_serializing_if = "Option::is_none" means both fields are absent
239        assert!(json.get("deleted_at").is_none());
240        assert!(json.get("deleted_at_iso").is_none());
241        assert_eq!(json["elapsed_ms"], 0u64);
242    }
243}
sqlite_graphrag/commands/forget.rs

sqlite_graphrag/commands/
forget.rs
