Skip to content

Make memory provider pluggable (MemPalace) and add bundle script#1

Open
mandarab76 wants to merge 1 commit intoazd-build-deployfrom
codex/add-mempalace-feature
Open

Make memory provider pluggable (MemPalace) and add bundle script#1
mandarab76 wants to merge 1 commit intoazd-build-deployfrom
codex/add-mempalace-feature

Conversation

@mandarab76
Copy link
Copy Markdown
Owner

Motivation

  • Allow switching the memory backend at runtime via a configuration setting (MEMORY_PROVIDER) so deployments can opt into the new MemPalace provider without changing agent code.
  • Provide an easy way to package and install the MemPalace backend changes via a portable zip bundle (scripts/mempalace_bundle.sh).

Description

  • Add MEMORY_PROVIDER to AppConfig with default value cosmos and load it from environment variables.
  • Introduce a memory provider factory context/memory_provider.py with create_memory_context(session_id, user_id) that returns either CosmosMemoryContext or MemPalaceMemoryContext based on MEMORY_PROVIDER.
  • Add a context/mempalace_memory_kernel.py stub that subclasses CosmosMemoryContext to represent the MemPalace provider implementation.
  • Update codepaths in kernel_agents/agent_factory.py and utils_kernel.py to use create_memory_context instead of constructing CosmosMemoryContext directly.
  • Add scripts/mempalace_bundle.sh to create and install a zip bundle containing the MemPalace backend files and document usage in src/backend/README.md.

Testing

  • No automated tests were run for this change.

Codex Task

@ecc-tools
Copy link
Copy Markdown

ecc-tools Bot commented Apr 22, 2026

Analyzing 200 commits...

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 22, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5184c0cc-b8f7-46a0-bf50-ca73cf80387e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/add-mempalace-feature

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ecc-tools
Copy link
Copy Markdown

ecc-tools Bot commented Apr 22, 2026

Analysis Complete

Generated ECC bundle from 1 commits | Confidence: 50%

View Pull Request #2

Repository Profile
Attribute Value
Language Python
Framework Not detected
Commit Convention freeform
Test Directory separate
Changed Files (7)
Metric Value
Files changed 7
Additions 124
Deletions 3

Top hotspots

Path Status +/-
scripts/mempalace_bundle.sh added +73 / -0
src/backend/context/memory_provider.py added +19 / -0
src/backend/README.md modified +14 / -1
src/backend/context/mempalace_memory_kernel.py added +13 / -0
src/backend/kernel_agents/agent_factory.py modified +2 / -1

Top directories

Directory Files Total changes
scripts 1 73
src/backend/context 2 32
src/backend 3 19
src/backend/kernel_agents 1 3
Likely Future Issues (1)
Severity Signal Why it may show up
HIGH Regression coverage may lag behind the diff 7 generic code paths changed; 0 test files changed
  • Regression coverage may lag behind the diff: The PR changes multiple code paths but does not touch any obvious test files.
Suggested Follow-up Work (1)
Type Suggested title Targets
PR test: add regression coverage for scripts/mempalace_bundle.sh + src/backend/README.md scripts/mempalace_bundle.sh, src/backend/README.md
  • test: add regression coverage for scripts/mempalace_bundle.sh + src/backend/README.md: Backfill regression coverage before another change set lands on the touched code paths.

Copy-ready bodies

test: add regression coverage for scripts/mempalace_bundle.sh + src/backend/README.md

## Summary
- Add regression coverage for the recently touched code paths before more changes stack on top.

## Why
- Backfill regression coverage before another change set lands on the touched code paths.

## Touched paths
- `scripts/mempalace_bundle.sh`
- `src/backend/README.md`

## Validation
- Add or extend focused tests that exercise the touched paths.
- Run the affected test suite and verify the new coverage closes the gap.
Generated Instincts (13)
Domain Count
git 2
code-style 9
testing 2

After merging, import with:

/instinct-import .claude/homunculus/instincts/inherited/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator-instincts.yaml

Files

  • .claude/ecc-tools.json
  • .claude/skills/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator/SKILL.md
  • .agents/skills/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator/SKILL.md
  • .agents/skills/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator/agents/openai.yaml
  • .claude/identity.json
  • .codex/config.toml
  • .codex/AGENTS.md
  • .codex/agents/explorer.toml
  • .codex/agents/reviewer.toml
  • .codex/agents/docs-researcher.toml
  • .claude/homunculus/instincts/inherited/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator-instincts.yaml

ECC Tools | Everything Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant