Remove curation

[dacorvo]

This removes the curation verb and the whole concept of project memory store. My own experiments with huggingface/funes-memory showed me that it is far easier to do an agent assisted curation than to use the awkward tools I had developped. Lesson learnt: don't develop a user interface when the agent is a better interface.

[Copilot]

Pull request overview

This PR removes the curate command / “project memory” concept (including the TUI review flow and its gating behavior) and replaces it with an explicit publishing mechanism: funes push --sessions …, where the session list itself is the publish decision.

Changes:

• Remove funes curate command, curation state/files, and the in-process TUI picker used for interactive review.
• Extend funes push to optionally publish only explicitly named sessions (--sessions), updating CLI, docs, and tests accordingly.
• Simplify UI rendering by keeping only the stable agent-format output for read verbs; remove now-unused human-view rendering paths and dependencies (e.g., nucleo-matcher).

Reviewed changes

Copilot reviewed 19 out of 20 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
tests/push_round_trip.rs	Updates push invocations to pass the new sessions slice parameter.
tests/push_card.rs	Updates push invocations to pass the new sessions slice parameter.
tests/curate_gating.rs	Deletes the e2e test that depended on project-memory curation gating.
src/ui/tui/curate.rs	Deletes the curation-specific TUI picker model.
src/ui/tui.rs	Deletes the generic list+preview TUI engine used by curate.
src/ui/render.rs	Updates docs/comments and removes the human get rendering helpers/tests, keeping agent-format rendering.
src/ui.rs	Removes the tui module export now that TUI curation is gone.
src/memory/remote.rs	Removes schema-metadata amend helper used for project-memory “naming”.
src/main.rs	Removes Curate subcommand; adds --sessions support to Push and wires through to run_push.
src/commands/recall.rs	Removes curate dependency; inlines is_scaffolding logic needed for session prompt extraction and updates status output.
src/commands/push.rs	Implements --sessions-based selection, removes curation gating logic, and adds helpers for session→chunk resolution.
src/commands/curate.rs	Deletes the entire curation/project-memory implementation.
src/commands.rs	Removes the curate module export.
README.md	Updates docs pointers/command list to remove curate and describe selection publishing.
docs/storage.md	Updates terminology from “curated project memory” to “published selection”.
docs/push.md	Replaces the “Project memories: funes curate” section with “Publishing a selection: --sessions”.
docs/configuration.md	Removes curation/ from $FUNES_HOME layout and related environment variable docs.
docs/add.md	Removes mention of preserving curation decisions.
Cargo.toml	Removes nucleo-matcher; updates ratatui comment to reflect remaining usage.
Cargo.lock	Removes nucleo-matcher from the lockfile.

Suppressed comments (1)

src/commands/push.rs:413

• When --sessions is used on a first publish (remote missing/empty), rows_to_push(..., first_publish=true) disables the id IN (...) filter and ends up scanning/publishing the entire local dataset, ignoring the session selection. This can leak unintended sessions on the initial publish.

        return Ok(format!("{}: up to date ({} chunks)\n{note}", target.label(), remote_ids.len()).into());

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

Copilot reviewed 19 out of 20 changed files in this pull request and generated no new comments.

Suppressed comments (3)

Previously missed (3) — in code that hasn't changed since the last review.

src/commands/push.rs:324

• The error message for unknown session ids is grammatically incomplete ("lists what is"), which makes it harder to understand what the user should do next.

        bail!(
            "not in your local memory: {} — `funes sessions` lists what is",
            unknown.join(", ")
        );

src/commands/push.rs:289

• ids_by_session always scans the entire local dataset (id + session_id for every row). For large memories, funes push --sessions ... will pay an O(N) full-table scan even when publishing a single session; this is a noticeable performance regression for the selection workflow.

/// A memory's rows as session → chunk ids, the shape a selection is resolved against.
pub(crate) async fn ids_by_session(ds: &Dataset) -> Result<HashMap<String, Vec<String>>> {
    let batches = dataset::scan_rows(ds, &["id", "session_id"], None, None).await?;

docs/storage.md:18

• This table header still says "Published project", but the surrounding text and the rest of the table now describe a published selection. Updating the header keeps the terminology consistent.

| | Local development | Published project |