feat: add memory_type to FTS5 index (#662)

[ajianaz]

Summary

Add memory_type column to the FTS5 full-text search index so that keyword searches can match against memory types (fact, decision, procedure, etc.).

Changes

• FTS5 virtual table: Added memory_type as a new FTS5 column
• Sync triggers: Updated all 3 triggers (INSERT, DELETE, UPDATE) to include memory_type
• Migration v13->v14: Drops old FTS5 table + triggers, recreates with new schema, rebuilds index from existing data
• Tests: Added test_fts5_memory_type_column_search and test_fts5_memory_type_rebuild

Closes #662

[github-actions]

🔍 Cora AI Code Review

✅ No issues found. Code looks good!

---

Review powered by cora-cli · BYOK · MIT

[ajianaz]

CI Failure Analysis

Error: Format check fail

schema.rs:972 — migration code formatting doesn't match cargo fmt. The diff shows:

• Array literal ["memories_fts_ai", "memories_fts_ad", "memories_fts_au"] needs wrapping to single line
• Method chain .conn.execute(...) needs reformatting

Fix: Run cargo fmt --all locally and push.

Note: Check, Clippy, and Build passed on the latest run (29320742498). Only Format failed.

[ajianaz]

Updated CI Failure — Run 29321960300

Format: ✅ FIXED

Build/Check/Clippy: ✅ PASS

Test: ❌ FAIL — 4 migration tests broken

All same root cause: CURRENT_SCHEMA_VERSION is now 14 (due to v13→v14 FTS5 migration), but 4 test assertions still expect version 13.

Failed tests:

1. edges::tests::migration_dispatcher_reaches_v8 (edges.rs:1279)
2. memory::crud::content_type_tests::test_schema_migration_v5_to_v6_adds_content_type (crud.rs:708)
3. memory::store::tests::test_migration_v11_to_v12_from_v04x_db (store.rs:1781)
4. timeline::tests::migration_dispatcher_reaches_v9 (timeline.rs:299)

Fix: Update all 4 assertions from == 13 to == 14:

// edges.rs:1279, crud.rs:708, store.rs:1781, timeline.rs:299
assert_eq!(version, 14);  // was 13

Also check CURRENT_SCHEMA_VERSION constant — if it's now 14, all these tests need to match.