remem-ai 0.6.61

Local-first coding agent memory for Claude Code and OpenAI Codex
Documentation
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
693
694
695
696
697
698
699
700
701
702
703
704
705
706
707
708
709
710
711
712
713
714
715
716
717
718
719
720
721
722
723
724
725
726
727
728
729
730
731
732
733
734
735
736
737
738
739
740
741
742
743
744
745
746
747
748
749
750
751
752
753
754
755
756
757
758
759
760
761
762
763
764
765
766
767
768
769
770
771
772
773
774
775
776
777
778
use rusqlite::params;

use crate::db::{self, test_support::ScopedTestDataDir};

use super::{lock, mark_successful_job, record_failed_job_transition, recover_expired_jobs, run};
use test_support::install_stub_codex;

pub(super) mod test_support;

#[tokio::test]
async fn worker_skips_legacy_observation_job_without_retry() -> anyhow::Result<()> {
    let _data_dir = ScopedTestDataDir::new("worker-skip-legacy-observation");
    let conn = db::open_db()?;
    let job_id = db::enqueue_job(
        &conn,
        "codex-cli",
        db::JobType::Observation,
        "/tmp/remem",
        Some("sess-legacy-observation"),
        r#"{"host":"codex-cli","session_id":"sess-legacy-observation","project":"/tmp/remem"}"#,
        50,
    )?;

    run(true, 10).await?;

    let conn = db::open_db()?;
    let (state, attempt_count, last_error): (String, i64, Option<String>) = conn.query_row(
        "SELECT state, attempt_count, last_error FROM jobs WHERE id = ?1",
        params![job_id],
        |row| Ok((row.get(0)?, row.get(1)?, row.get(2)?)),
    )?;
    anyhow::ensure!(
        state == "done",
        "expected skipped legacy job done, got {state}"
    );
    anyhow::ensure!(
        attempt_count == 0,
        "legacy job should not retry, got {attempt_count}"
    );
    anyhow::ensure!(
        last_error.is_none(),
        "legacy job should not record an error"
    );
    Ok(())
}

#[tokio::test]
async fn worker_rejects_legacy_summary_job_without_retry() -> anyhow::Result<()> {
    let _data_dir = ScopedTestDataDir::new("worker-reject-legacy-summary");
    let conn = db::open_db()?;
    let now = chrono::Utc::now().timestamp();
    conn.execute(
        "INSERT INTO jobs
         (host, job_type, project, session_id, payload_json, state, priority,
          attempt_count, max_attempts, next_retry_epoch, created_at_epoch, updated_at_epoch)
         VALUES ('codex-cli', 'summary', '/tmp/remem', 'sess-legacy-summary',
                 ?1, 'pending', 50, 0, 6, 0, ?2, ?2)",
        params![
            r#"{"host":"codex-cli","session_id":"sess-legacy-summary","project":"/tmp/remem"}"#,
            now
        ],
    )?;
    let job_id = conn.last_insert_rowid();
    drop(conn);

    run(true, 10).await?;

    let conn = db::open_db()?;
    let (state, attempt_count, next_retry, last_error, failure_class): (
        String,
        i64,
        i64,
        Option<String>,
        Option<String>,
    ) = conn.query_row(
        "SELECT state, attempt_count, next_retry_epoch, last_error, failure_class
             FROM jobs WHERE id = ?1",
        params![job_id],
        |row| {
            Ok((
                row.get(0)?,
                row.get(1)?,
                row.get(2)?,
                row.get(3)?,
                row.get(4)?,
            ))
        },
    )?;
    anyhow::ensure!(state == "failed", "expected failed job, got {state}");
    anyhow::ensure!(
        attempt_count == 1,
        "legacy Summary rejection should consume one attempt, got {attempt_count}"
    );
    anyhow::ensure!(next_retry == 0, "legacy Summary job should not retry");
    anyhow::ensure!(
        last_error
            .as_deref()
            .is_some_and(|err| err.contains("legacy Summary jobs are retired")),
        "expected retired Summary error, got {last_error:?}"
    );
    anyhow::ensure!(
        failure_class.as_deref() == Some("permanent"),
        "expected permanent failure, got {failure_class:?}"
    );
    Ok(())
}

fn read_worker_log(data_dir: &ScopedTestDataDir) -> anyhow::Result<String> {
    Ok(std::fs::read_to_string(data_dir.path.join("remem.log"))?)
}

fn enqueue_worker_job(
    conn: &rusqlite::Connection,
    job_type: db::JobType,
    project: &str,
) -> anyhow::Result<i64> {
    db::enqueue_job(conn, "codex-cli", job_type, project, None, "{}", 50)
}

#[test]
fn worker_transition_conflict_logs_error_without_done_or_retry_success() -> anyhow::Result<()> {
    let data_dir = ScopedTestDataDir::new("worker-transition-conflict");
    let mut conn = db::open_db()?;
    let job_id = enqueue_worker_job(&conn, db::JobType::Observation, "/tmp/remem")?;
    db::claim_next_job(&mut conn, "worker-a", 60)?.expect("job should claim");
    conn.execute(
        "UPDATE jobs SET lease_owner = 'worker-b' WHERE id = ?1",
        params![job_id],
    )?;
    let error = mark_successful_job(
        &conn,
        job_id,
        db::JobType::Observation,
        "/tmp/remem",
        "worker-a",
    )
    .expect_err("wrong owner transition must fail");
    assert!(error.to_string().contains("current_owner=worker-b"));
    let log = read_worker_log(&data_dir)?;
    assert!(log.contains(&format!("job transition failed id={job_id} operation=done")));
    assert!(log.contains("job_type=observation"));
    assert!(log.contains(&format!(
        "project_hash={:016x}",
        crate::db::deterministic_hash(b"/tmp/remem")
    )));
    assert!(log.contains("expected_owner=worker-a"));
    assert!(log.contains("current_owner=worker-b"));
    assert!(!log.contains("project=/tmp/remem"));
    assert!(!log.contains(&format!("done id={job_id}")));
    assert!(!log.contains("operation=retry success"));
    Ok(())
}

#[test]
fn worker_compile_rules_retry_collision_logs_safe_coalesced_result() -> anyhow::Result<()> {
    let data_dir = ScopedTestDataDir::new("worker-compile-collision");
    let mut conn = db::open_db()?;
    let source_id = enqueue_worker_job(&conn, db::JobType::CompileRules, "/tmp/remem")?;
    db::claim_next_job(&mut conn, "worker-a", 60)?.expect("source should claim");
    let canonical_id = enqueue_worker_job(&conn, db::JobType::CompileRules, "/tmp/remem")?;
    record_failed_job_transition(
        &conn,
        source_id,
        db::JobType::CompileRules,
        "/tmp/remem",
        "worker-a",
        "private-retry-error-must-not-log",
        5,
    )?;
    let log = read_worker_log(&data_dir)?;
    assert!(log.contains(&format!(
        "job retry coalesced source_id={source_id} canonical_id={canonical_id} identity=compile_rules"
    )));
    assert!(!log.contains("private-retry-error-must-not-log"));
    assert!(!log.contains(&format!("job id={source_id} failed:")));
    Ok(())
}

#[test]
fn worker_expired_job_recovery_logs_requeued_and_coalesced_outcomes() -> anyhow::Result<()> {
    let data_dir = ScopedTestDataDir::new("worker-expired-outcomes");
    let conn = db::open_db()?;
    let ordinary_id = enqueue_worker_job(&conn, db::JobType::Compress, "/ordinary")?;
    let compile_id = enqueue_worker_job(&conn, db::JobType::CompileRules, "/compile")?;
    let expired = chrono::Utc::now().timestamp() - 1;
    conn.execute(
        "UPDATE jobs SET state='processing', lease_owner='dead', lease_expires_epoch=?1,
         last_error='private-expired-error' WHERE id IN (?2, ?3)",
        params![expired, ordinary_id, compile_id],
    )?;
    let canonical_id = enqueue_worker_job(&conn, db::JobType::CompileRules, "/compile")?;

    recover_expired_jobs(&conn)?;
    let log = read_worker_log(&data_dir)?;
    assert!(log.contains(&format!(
        "expired job recovery requeued source_id={ordinary_id} identity=ordinary"
    )));
    assert!(log.contains(&format!(
        "expired job recovery coalesced source_id={compile_id} canonical_id={canonical_id} identity=compile_rules"
    )));
    assert!(!log.contains("private-expired-error"));
    assert!(!log.contains(&format!(
        "expired job recovery requeued source_id={compile_id}"
    )));
    Ok(())
}

#[tokio::test]
async fn worker_retries_compress_job_when_ai_fails() -> anyhow::Result<()> {
    let _data_dir = ScopedTestDataDir::new("worker-compress-ai-failure");
    configure_codex_stub("/tmp/remem-missing-codex-for-compress")?;

    let conn = db::open_db()?;
    insert_compressible_observations(&conn, "/tmp/remem", 101)?;
    let job_id = db::enqueue_job(
        &conn,
        "codex-cli",
        db::JobType::Compress,
        "/tmp/remem",
        None,
        "{}",
        200,
    )?;

    run(true, 10).await?;

    let conn = db::open_db()?;
    let (state, attempt_count, next_retry, last_error): (String, i64, Option<i64>, Option<String>) =
        conn.query_row(
            "SELECT state, attempt_count, next_retry_epoch, last_error FROM jobs WHERE id = ?1",
            params![job_id],
            |row| Ok((row.get(0)?, row.get(1)?, row.get(2)?, row.get(3)?)),
        )?;
    anyhow::ensure!(state == "pending", "expected pending retry, got {state}");
    anyhow::ensure!(
        attempt_count == 1,
        "expected one attempt, got {attempt_count}"
    );
    anyhow::ensure!(next_retry.is_some(), "expected retry delay");
    anyhow::ensure!(
        last_error
            .as_deref()
            .is_some_and(|err| err.contains("failed to spawn")),
        "expected missing codex path error, got {last_error:?}"
    );
    Ok(())
}

#[tokio::test]
async fn worker_retries_unimplemented_extraction_task() -> anyhow::Result<()> {
    let _data_dir = ScopedTestDataDir::new("worker-extraction-unimplemented");
    let conn = db::open_db()?;
    let outcome = db::record_captured_event(
        &conn,
        &db::CaptureEventInput {
            host: "codex-cli",
            session_id: "sess-extract",
            project: "/tmp/remem",
            cwd: None,
            event_type: "tool_result",
            role: None,
            tool_name: Some("Bash"),
            content: r#"{"tool_name":"Bash"}"#,
            task_kind: Some(db::ExtractionTaskKind::RuleCandidate),
        },
    )?;
    let task_id = outcome
        .extraction_task_id
        .expect("capture should coalesce extraction task");

    run(true, 10).await?;

    let conn = db::open_db()?;
    let (status, attempts, next_retry, last_error): (String, i64, Option<i64>, Option<String>) = conn.query_row(
            "SELECT status, attempts, next_retry_epoch, last_error FROM extraction_tasks WHERE id = ?1",
            params![task_id],
            |row| Ok((row.get(0)?, row.get(1)?, row.get(2)?, row.get(3)?)),
        )?;
    anyhow::ensure!(status == "pending", "expected pending task, got {status}");
    anyhow::ensure!(attempts == 1, "expected one attempt, got {attempts}");
    anyhow::ensure!(next_retry.is_some(), "expected retry delay");
    anyhow::ensure!(
        last_error
            .as_deref()
            .is_some_and(|err| err.contains("not implemented")),
        "expected explicit unimplemented error"
    );
    Ok(())
}

#[tokio::test]
async fn worker_once_records_startup_heartbeat_without_work() -> anyhow::Result<()> {
    let _data_dir = ScopedTestDataDir::new("worker-once-startup-heartbeat");

    run(true, 10).await?;

    let conn = db::open_db()?;
    let Some(heartbeat) = db::latest_worker_heartbeat(&conn)? else {
        anyhow::bail!("worker --once should record startup heartbeat");
    };
    let expected_prefix = format!("worker-v{}-once-", env!("CARGO_PKG_VERSION"));
    anyhow::ensure!(
        heartbeat.owner.starts_with(&expected_prefix),
        "unexpected heartbeat owner {}",
        heartbeat.owner
    );
    anyhow::ensure!(
        heartbeat.started_at_epoch <= heartbeat.updated_at_epoch,
        "heartbeat should be valid immediately after singleton acquisition"
    );
    Ok(())
}

#[tokio::test]
async fn worker_once_backfills_pending_memory_embeddings() -> anyhow::Result<()> {
    let _data_dir = ScopedTestDataDir::new("worker-once-embedding-backfill");
    crate::runtime_config::init_config()?;
    crate::runtime_config::set_config_value("embeddings.provider", "feature-hash")?;
    let conn = db::open_db()?;
    conn.execute(
            "INSERT INTO memories
             (id, project, title, content, memory_type, created_at_epoch, updated_at_epoch, status)
             VALUES (1, '/tmp/remem', 'Credential store', 'SQLCipher encrypts secrets at rest.', 'architecture', 1, 1, 'active')",
            [],
        )?;
    drop(conn);

    run(true, 10).await?;

    let conn = db::open_db()?;
    let count: i64 = conn.query_row(
        "SELECT COUNT(*)
             FROM memory_embeddings
             WHERE memory_id = 1
               AND model = ?1
               AND dimensions = ?2",
        params![
            crate::retrieval::embedding::FEATURE_HASH_EMBEDDING_MODEL,
            crate::retrieval::embedding::FEATURE_HASH_EMBEDDING_DIMENSIONS as i64
        ],
        |row| row.get(0),
    )?;
    anyhow::ensure!(count == 1, "worker should backfill one embedding row");
    Ok(())
}

#[tokio::test]
async fn worker_once_refreshes_heartbeat_in_drain_loop() -> anyhow::Result<()> {
    let _data_dir = ScopedTestDataDir::new("worker-once-loop-heartbeat");
    let conn = db::open_db()?;
    conn.execute_batch(
        "CREATE TABLE heartbeat_updates (owner TEXT NOT NULL);
             CREATE TRIGGER record_worker_heartbeat_update
             AFTER UPDATE ON worker_heartbeats
             BEGIN
                 INSERT INTO heartbeat_updates (owner) VALUES (new.owner);
             END;",
    )?;
    drop(conn);

    run(true, 10).await?;

    let conn = db::open_db()?;
    let update_count: i64 = conn.query_row(
        "SELECT COUNT(*)
             FROM heartbeat_updates
             WHERE owner LIKE 'worker-once-%'
                OR owner LIKE 'worker-v%-once-%'",
        [],
        |row| row.get(0),
    )?;
    anyhow::ensure!(
        update_count >= 1,
        "worker --once should refresh its heartbeat inside the drain loop"
    );
    Ok(())
}

#[tokio::test]
async fn worker_once_exits_without_work_when_singleton_lock_is_held() -> anyhow::Result<()> {
    let _data_dir = ScopedTestDataDir::new("worker-once-lock-held");
    let Some(_guard) = lock::acquire_worker_singleton()? else {
        anyhow::bail!("test worker lock should acquire");
    };
    let conn = db::open_db()?;
    let job_id = db::enqueue_job(
        &conn,
        "codex-cli",
        db::JobType::Observation,
        "/tmp/remem",
        Some("sess-lock-held"),
        r#"{"host":"codex-cli","session_id":"sess-lock-held","project":"/tmp/remem"}"#,
        50,
    )?;

    run(true, 10).await?;

    let conn = db::open_db()?;
    let state: String = conn.query_row(
        "SELECT state FROM jobs WHERE id = ?1",
        params![job_id],
        |row| row.get(0),
    )?;
    anyhow::ensure!(
        state == "pending",
        "locked-out worker should not process job"
    );
    anyhow::ensure!(
        db::latest_worker_heartbeat(&conn)?.is_none(),
        "locked-out worker should exit before recording heartbeat"
    );
    Ok(())
}

#[tokio::test]
async fn old_version_daemon_lock_allows_current_once_heartbeat() -> anyhow::Result<()> {
    let _data_dir = ScopedTestDataDir::new("worker-daemon-once-mutual-exclusion");
    let Some(_daemon_lock) = lock::acquire_worker_singleton()? else {
        anyhow::bail!("test daemon lock should acquire");
    };
    let conn = db::open_db()?;
    let now = chrono::Utc::now().timestamp();
    let daemon_owner = "worker-daemon-test";
    db::upsert_worker_heartbeat(&conn, daemon_owner, i64::from(std::process::id()), now, now)?;

    run(true, 10).await?;

    let expected_prefix = format!("worker-v{}-once-", env!("CARGO_PKG_VERSION"));
    let current_once_count: i64 = conn.query_row(
        "SELECT COUNT(*) FROM worker_heartbeats WHERE owner LIKE ?1",
        params![format!("{expected_prefix}%")],
        |row| row.get(0),
    )?;
    anyhow::ensure!(
            current_once_count == 1,
            "worker --once should record one current-version heartbeat after bypass, got {current_once_count}"
        );
    Ok(())
}

#[allow(clippy::await_holding_lock)]
#[tokio::test]
async fn worker_processes_session_rollup_task_on_codex_stub() -> anyhow::Result<()> {
    let _data_dir = ScopedTestDataDir::new("worker-session-rollup");
    let stub_codex = std::env::temp_dir().join(format!(
        "remem-test-codex-rollup-{}-{}.sh",
        std::process::id(),
        chrono::Utc::now().timestamp_nanos_opt().unwrap_or_default()
    ));
    install_stub_codex(&stub_codex);
    let stub_codex_str = stub_codex
        .as_os_str()
        .to_str()
        .expect("stub codex path should be valid utf-8");
    configure_codex_stub(stub_codex_str)?;

    let conn = db::open_db()?;
    let outcome = db::record_captured_event(
        &conn,
        &db::CaptureEventInput {
            host: "codex-cli",
            session_id: "sess-rollup-worker",
            project: "/tmp/remem",
            cwd: None,
            event_type: "session_stop",
            role: None,
            tool_name: None,
            content: r#"{"session_id":"sess-rollup-worker","result":"done"}"#,
            task_kind: Some(db::ExtractionTaskKind::SessionRollup),
        },
    )?;
    let task_id = outcome
        .extraction_task_id
        .expect("capture should coalesce extraction task");

    let test_result = async {
        run(true, 10).await?;

        let conn = db::open_db()?;
        let (status, cursor, high_watermark): (String, Option<i64>, Option<i64>) = conn.query_row(
            "SELECT status, cursor_event_id, high_watermark_event_id
                     FROM extraction_tasks WHERE id = ?1",
            params![task_id],
            |row| Ok((row.get(0)?, row.get(1)?, row.get(2)?)),
        )?;
        let summary_text: String = conn.query_row(
            "SELECT summary_text FROM session_summaries
                 WHERE session_row_id IS NOT NULL",
            [],
            |row| row.get(0),
        )?;

        anyhow::ensure!(status == "done", "expected done task, got {status}");
        anyhow::ensure!(cursor == high_watermark, "expected cursor to advance");
        anyhow::ensure!(
            summary_text.contains("Codex worker flush"),
            "expected stub summary text"
        );
        Ok::<(), anyhow::Error>(())
    }
    .await;

    let _ = std::fs::remove_file(&stub_codex);
    test_result
}

#[allow(clippy::await_holding_lock)]
#[tokio::test]
async fn late_session_stop_evidence_links_without_replaying_rollup() -> anyhow::Result<()> {
    let _data_dir = ScopedTestDataDir::new("worker-late-session-stop-git-link");
    configure_codex_stub("/tmp/remem-missing-codex-for-late-git-link")?;
    let conn = db::open_db()?;
    let input = db::CaptureEventInput {
        host: "codex-cli",
        session_id: "sess-late-session-stop-git-link",
        project: "/tmp/remem",
        cwd: None,
        event_type: "session_stop",
        role: None,
        tool_name: None,
        content: r#"{"session_id":"sess-late-session-stop-git-link"}"#,
        task_kind: Some(db::ExtractionTaskKind::SessionRollup),
    };
    let first = db::record_captured_event_with_id_and_reference_time_and_git_evidence(
        &conn,
        &input,
        Some("late-session-stop-git-link"),
        None,
        &[],
    )?;
    let rollup_task_id = first
        .extraction_task_id
        .expect("initial Stop should enqueue SessionRollup");
    conn.execute(
        "UPDATE extraction_tasks
         SET status = 'done', cursor_event_id = high_watermark_event_id
         WHERE id = ?1",
        params![rollup_task_id],
    )?;
    let evidence = crate::git_util::GitCommitEvidence {
        kind: crate::git_util::GitEvidenceKind::ObservedCommit,
        metadata: crate::git_util::GitCommitMetadata {
            repo_path: "/tmp/remem".to_string(),
            sha: "abcdef1234567890abcdef1234567890abcdef12".to_string(),
            short_sha: "abcdef1".to_string(),
            branch: Some("main".to_string()),
            message: Some("late commit evidence".to_string()),
            authored_at_epoch: Some(1_700_000_000),
            changed_files: vec!["src/lib.rs".to_string()],
        },
        locator: Some("replayed_stop".to_string()),
    };
    let second = db::record_captured_event_with_id_and_reference_time_and_git_evidence(
        &conn,
        &input,
        Some("late-session-stop-git-link"),
        None,
        std::slice::from_ref(&evidence),
    )?;
    db::record_captured_event_with_id_and_reference_time_and_git_evidence(
        &conn,
        &input,
        Some("late-session-stop-git-link"),
        None,
        &[evidence],
    )?;
    let link_task_id = second
        .extraction_task_id
        .expect("late evidence should enqueue link-only compensation");
    anyhow::ensure!(link_task_id != rollup_task_id);
    let (task_kind, cursor, high_watermark, task_count): (String, i64, i64, i64) = conn.query_row(
        "SELECT task_kind, cursor_event_id, high_watermark_event_id,
                    (SELECT COUNT(*) FROM extraction_tasks)
             FROM extraction_tasks
             WHERE id = ?1",
        params![link_task_id],
        |row| Ok((row.get(0)?, row.get(1)?, row.get(2)?, row.get(3)?)),
    )?;
    anyhow::ensure!(task_kind == "captured_git_link", "got {task_kind}");
    anyhow::ensure!(cursor == first.event_row_id - 1);
    anyhow::ensure!(high_watermark == first.event_row_id);
    anyhow::ensure!(
        task_count == 2,
        "duplicate replay created {task_count} tasks"
    );
    drop(conn);

    anyhow::ensure!(
        crate::extraction_worker::run_next("worker-late-git-link", 60, 60).await?,
        "link-only task should be processed"
    );

    let conn = db::open_db()?;
    let (status, final_cursor, final_high_watermark): (String, i64, i64) = conn.query_row(
        "SELECT status, cursor_event_id, high_watermark_event_id
         FROM extraction_tasks
         WHERE id = ?1",
        params![link_task_id],
        |row| Ok((row.get(0)?, row.get(1)?, row.get(2)?)),
    )?;
    let summary_count: i64 =
        conn.query_row("SELECT COUNT(*) FROM session_summaries", [], |row| {
            row.get(0)
        })?;
    let job_count: i64 = conn.query_row("SELECT COUNT(*) FROM jobs", [], |row| row.get(0))?;
    let extraction_task_count: i64 =
        conn.query_row("SELECT COUNT(*) FROM extraction_tasks", [], |row| {
            row.get(0)
        })?;
    let (link_count, linked_sha): (i64, String) = conn.query_row(
        "SELECT COUNT(*), COALESCE(MIN(commits.sha), '')
         FROM git_commit_sessions links
         JOIN git_commits commits ON commits.id = links.commit_id",
        [],
        |row| Ok((row.get(0)?, row.get(1)?)),
    )?;
    anyhow::ensure!(status == "done", "link task status was {status}");
    anyhow::ensure!(final_cursor == final_high_watermark);
    anyhow::ensure!(summary_count == 0, "link-only task wrote a summary");
    anyhow::ensure!(job_count == 0, "link-only task enqueued {job_count} jobs");
    anyhow::ensure!(
        extraction_task_count == 2,
        "link-only task enqueued follow-up extraction work"
    );
    anyhow::ensure!(link_count == 1);
    anyhow::ensure!(linked_sha == "abcdef1234567890abcdef1234567890abcdef12");
    Ok(())
}

#[allow(clippy::await_holding_lock)]
#[tokio::test]
async fn worker_processes_observation_extract_task_on_codex_stub() -> anyhow::Result<()> {
    let _data_dir = ScopedTestDataDir::new("worker-observation-extract");
    let stub_codex = std::env::temp_dir().join(format!(
        "remem-test-codex-observation-{}-{}.sh",
        std::process::id(),
        chrono::Utc::now().timestamp_nanos_opt().unwrap_or_default()
    ));
    install_stub_codex(&stub_codex);
    let stub_codex_str = stub_codex
        .as_os_str()
        .to_str()
        .expect("stub codex path should be valid utf-8");
    configure_codex_stub(stub_codex_str)?;

    let conn = db::open_db()?;
    let outcome = db::record_captured_event(
        &conn,
        &db::CaptureEventInput {
            host: "codex-cli",
            session_id: "sess-observe-worker",
            project: "/tmp/remem",
            cwd: None,
            event_type: "tool_result",
            role: None,
            tool_name: Some("Bash"),
            content: r#"{"tool_name":"Bash","output":"important"}"#,
            task_kind: Some(db::ExtractionTaskKind::ObservationExtract),
        },
    )?;
    let task_id = outcome
        .extraction_task_id
        .expect("capture should coalesce extraction task");

    let test_result = async {
        run(true, 10).await?;

        let conn = db::open_db()?;
        let (status, cursor, high_watermark): (String, Option<i64>, Option<i64>) = conn.query_row(
            "SELECT status, cursor_event_id, high_watermark_event_id
                     FROM extraction_tasks WHERE id = ?1",
            params![task_id],
            |row| Ok((row.get(0)?, row.get(1)?, row.get(2)?)),
        )?;
        let (text, evidence): (String, String) = conn.query_row(
            "SELECT text, evidence_event_ids FROM observations
                 WHERE session_row_id IS NOT NULL",
            [],
            |row| Ok((row.get(0)?, row.get(1)?)),
        )?;
        let pending_memory_candidates: i64 = conn.query_row(
            "SELECT COUNT(*) FROM memory_candidates WHERE review_status = 'pending_review'",
            [],
            |row| row.get(0),
        )?;
        let (graph_status, graph_attempts, graph_error): (String, i64, Option<String>) = conn
            .query_row(
                "SELECT status, attempts, last_error
                     FROM extraction_tasks
                     WHERE task_kind = ?1",
                params![db::ExtractionTaskKind::GraphCandidate.as_str()],
                |row| Ok((row.get(0)?, row.get(1)?, row.get(2)?)),
            )?;

        anyhow::ensure!(status == "done", "expected done task, got {status}");
        anyhow::ensure!(cursor == high_watermark, "expected cursor to advance");
        anyhow::ensure!(
            text.contains("Queued Codex observation persisted"),
            "expected stub observation text"
        );
        anyhow::ensure!(evidence.contains('1'), "expected captured event evidence");
        anyhow::ensure!(
            pending_memory_candidates == 1,
            "expected pending memory candidate review"
        );
        anyhow::ensure!(
            graph_status == "pending",
            "expected graph task pending after memory review gate, got {graph_status}"
        );
        anyhow::ensure!(
            graph_attempts == 0,
            "expected graph task to wait without consuming attempts, got {graph_attempts}"
        );
        anyhow::ensure!(
            graph_error
                .as_deref()
                .is_some_and(|err| err.contains("pending review")),
            "expected graph task defer reason to mention pending review"
        );
        Ok::<(), anyhow::Error>(())
    }
    .await;

    let _ = std::fs::remove_file(&stub_codex);
    test_result
}

#[tokio::test]
async fn worker_heartbeat_updates_in_loop() -> anyhow::Result<()> {
    let _data_dir = ScopedTestDataDir::new("worker-heartbeat-loop");

    let timed = tokio::time::timeout(std::time::Duration::from_millis(40), run(false, 10)).await;
    anyhow::ensure!(timed.is_err(), "daemon worker should keep running");
    let conn = db::open_db()?;
    let heartbeat = db::latest_worker_heartbeat(&conn)?;
    let heartbeat = heartbeat.expect("daemon worker should emit heartbeat");
    anyhow::ensure!(
        heartbeat.owner.starts_with("worker-"),
        "unexpected heartbeat owner {}",
        heartbeat.owner
    );
    anyhow::ensure!(
        heartbeat.updated_at_epoch >= heartbeat.started_at_epoch,
        "heartbeat should advance updated_at_epoch"
    );
    Ok(())
}

fn configure_codex_stub(stub_codex: &str) -> anyhow::Result<()> {
    crate::runtime_config::init_config()?;
    crate::runtime_config::set_config_value("memory_ai.profiles.codex.path", stub_codex)?;
    Ok(())
}

fn insert_compressible_observations(
    conn: &rusqlite::Connection,
    project: &str,
    count: usize,
) -> anyhow::Result<()> {
    for idx in 0..count {
        db::insert_observation(
            conn,
            &format!("compress-source-{idx}"),
            project,
            "discovery",
            Some(&format!("Source {idx}")),
            None,
            Some(&format!("Source observation {idx}")),
            None,
            None,
            None,
            None,
            None,
            0,
        )?;
    }
    Ok(())
}