fix: preserve OpenHuman checkpoint alignment on latest main

[senamakel]

Summary

Merge upstream main into the TinyFlows revision currently pinned by OpenHuman, preserving its checkpoint schema alignment with TinyAgents. This lets OpenHuman advance this submodule without dropping the existing fix.

Related issue

None.

API or behavior changes

The checkpoint alignment on the pinned branch is preserved.

Validation

• OpenHuman root cargo check --locked --quiet
• TinyFlows CI

Tests

No test code changed in the merge.

Documentation

No documentation change needed for the merge.

Checklist

• The change is focused on the OpenHuman submodule update
• No new #[allow(...)], #[ignore], or relaxed lints
• No secrets, tokens, or .env contents in the diff or the description

Summary by CodeRabbit

• New Features
  • Checkpoint records now include format version and creation-time metadata.
  • Expired thread execution leases can be reclaimed, including after the process that held them has stopped running.

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 3de3c741-fa98-4a0f-bbe9-e162cd2b9576

📥 Commits

Reviewing files that changed from the base of the PR and between 3276dd1 and ce8467c.

📒 Files selected for processing (1)

• crates/tinyflows-sqlite/src/checkpoint/mod.rs

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

The SQLite schema adds format_version and created_at columns to checkpoints. It also creates thread_leases with columns for a thread ID, owner, and expiration time.

Changes

SQLite checkpoint schema

Layer / File(s)	Summary
Checkpoint and lease tables crates/tinyflows-sqlite/src/checkpoint/mod.rs	The checkpoints table adds format_version and created_at. The new thread_leases table stores thread_id, owner, and expires_at.

Estimated code review effort: 2 (Simple) | ~8 minutes

Merge Risk: ⚪ Minimal · up to ce846

Existing checkpoint databases remain usable with the current read and write paths. No actionable merge-blocking issue remains after normal checks.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main change: preserving OpenHuman checkpoint schema alignment with the latest upstream main changes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

---

_{A rabbit reads the schema bright,
Two fields join checkpoints in the night.
A lease table records each thread,
Its owner and expiry date.
The database sprouts a row of green,
While bunny hops between.}

_{Comment @coderabbitai help to get the list of available commands.}

[tinysweeper]

Tiny Sweeper review

Tiny Sweeper reviewed this change across 6 lane(s) and found 0 active actionable finding(s). Detailed lane evidence and any incomplete work are listed below.

State: Ready for maintainer review
Priority: none
Reviewed head: ce8467c9bcbb
Updated: 1790268936 (Unix time)

Review snapshot

Change surface	Files	Review signal	Count
Production	1	Active findings	0
Tests	0	Noted findings	0
Documentation	0	Resolved findings	0
Configuration	0	Pending checks/questions	0

Completeness: Complete
Test assessment: No supported feature-to-test mapping was available; this does not mean tests are absent or passed.

What changed

The review could not produce a supported behavioral summary; inspect the cited changed surface and lane details below.

Features

None identified with supported citations.

Tests

No supported feature-to-test mapping was produced. Test execution is not inferred.

Findings

No active actionable findings.

Before merge

None.

How this fits together

flowchart LR
  n0["sqlite_err"]:::impacted
  n1["State"]:::impacted
  n2["lock_conn"]:::impacted
  n3["Checkpointer"]:::impacted
  n4["Checkpoint"]:::impacted
  n5["Send"]:::impacted
  n0 -->|calls| n4
  n2 -->|calls| n4
  n3 -->|uses| n1
  n3 -->|uses| n4
  n3 -->|uses| n5
  n3 -->|implements| n5
  n4 -->|uses| n1
  classDef changed fill:#0d4429,stroke:#238636,color:#e6edf3
  classDef impacted fill:#161b22,stroke:#6e7681,color:#c9d1d9
  classDef flagged fill:#5a1e02,stroke:#d93f0b,color:#ffffff
  classDef blocking fill:#67060c,stroke:#f85149,color:#ffffff

Loading

Agent review details

critique

• Conclusion: Success
• Scope reviewed: all assigned evidence
• Lane summary: The change adds optional checkpoint metadata columns and a thread lease table to the schema without altering existing checkpoint read/write behavior. It looks safe to merge as a schema-only change, though the newly declared lease table is not used by this file. _The code index is behind this pull request (indexed at `a94e5a29dce3`), so retrieved context may be out of date._ _Memory was unavailable (model: cortex: v1/recall: timed out after 10s), so this review ran without it._

security

• Conclusion: Success
• Scope reviewed: all assigned evidence
• Lane summary: The schema additions are inert and do not introduce a reachable security or authorization problem; this change looks safe to merge. _The code index is behind this pull request (indexed at `a94e5a29dce3`), so retrieved context may be out of date._ _Memory was unavailable (model: cortex: v1/recall: timed out after 10s), so this review ran without it._

tests

• Conclusion: Success
• Scope reviewed: all assigned evidence
• Lane summary: Adds two new columns (`format_version`, `created_at`) to the checkpoints table and a new `thread_leases` table, all with defaults so existing code is unaffected. The change is additive and backward-compatible; no regression would be silent because nothing yet depends on the new columns or table. Safe to merge. _The code index is behind this pull request (indexed at `a94e5a29dce3`), so retrieved context may be out of date._ _Memory was unavailable (model: cortex: v1/recall: timed out after 10s), so this review ran without it._

commits

• Conclusion: Neutral
• Scope reviewed: all assigned evidence
• Lane summary: Nothing sensitive found in what this pull request commits.

description

• Conclusion: Success
• Scope reviewed: all assigned evidence
• Lane summary: This merge adds two columns (`format_version`, `created_at`) to the `checkpoints` table and creates a new `thread_leases` table. The changes are backward-compatible via defaults and `IF NOT EXISTS`, and no new problems are introduced. _The code index is behind this pull request (indexed at `a94e5a29dce3`), so retrieved context may be out of date._ _Memory was unavailable (model: cortex: v1/recall: timed out after 10s), so this review ran without it._

e2e

• Conclusion: Success
• Scope reviewed: all assigned evidence
• Lane summary: Adds two columns and a new table to the SQLite checkpoint schema without any end-to-end test validating the change. The additions are backward-compatible via defaults, but a regression could go undetected in production. (1 finding discarded for not matching a changed line) _The code index is behind this pull request (indexed at `a94e5a29dce3`), so retrieved context may be out of date._ _Memory was unavailable (model: cortex: v1/recall: timed out after 10s), so this review ran without it._

Evidence and run details

• Models: flash, ladder/vectors, gpt-5.6-luna, deepseek-v4-flash
• Spend: $0.002178
• Tokens: 80973 input · 11789 output · 6892 cached · 136 embedding

Head	State	Pass summary
ce8467c9bcbb	ready for maintainer review	0 active finding(s), 0 resolved finding(s) (at 1790268936)

[tinysweeper]

tinysweeper found nothing blocking. Approving.

             $0.0022 · 80,973 in / 11,789 out · 6,892 cached (9%)  · flash, ladder/vectors, gpt-5.6-luna, deepseek-v4-flash · 136 embedded
critique:    $0.0007 · 24,700 in / 859 out    · 2,032 cached (8%)  · gpt-5.6-luna
security:    $0.0006 · 24,162 in / 459 out    · 1,788 cached (7%)  · gpt-5.6-luna
tests:       $0.0002 · 11,006 in / 1,140 out  · 1,024 cached (9%)  · deepseek-v4-flash
description: $0.0001 · 3,409 in  / 1,288 out  · 1,024 cached (30%) · deepseek-v4-flash
e2e:         $0.0003 · 11,866 in / 3,052 out  · 1,024 cached (9%)  · deepseek-v4-flash