Generate project memory context from meetings#18
Conversation
ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR adds project-level memory aggregation capabilities alongside meeting-level context packets. It enables reading existing JSONL data, merging project state, and writing aggregated per-project artifacts (source index, memory objects, relations, evidence map, context file). Core data models now support deserialization, and provider HTTP handling is refined. ChangesProject Memory Aggregation
Poem
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
There was a problem hiding this comment.
Code Review
This pull request introduces a project memory system that aggregates meeting data into project-specific directories. It includes logic for slugifying project IDs, namespacing object IDs, and merging new records with existing ones in JSONL files. Feedback highlights potential race conditions and atomicity issues during file writes, as well as opportunities to optimize object categorization and reduce unnecessary cloning in the rendering and writing logic.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b3d8bb1100
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Summary
Adds Project Memory v0 aggregation so completed meeting context packets also update project-level agent context artifacts.
Why
Completed meetings already generate source-backed context packets. This PR rolls those packets up into project-level memory so future agent tasks can consume accumulated context instead of only per-meeting briefs.
Changes
projects/<project-id>/.cargo fmt --check, native host tests, agent server tests, and the desktop native-launch test.Summary by CodeRabbit
Release Notes
New Features
Improvements