feat: add Feedo decentralized memory backend plugin - #19
Conversation
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 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. Comment |
|
@copilot Do a thorough review of this PR. Read ALL existing reviewer comments above from Qodo, Coderabbit, and Gemini first — incorporate their findings. Review areas:
|
Greptile SummaryThe PR adds Feedo as a lazily loaded decentralized memory backend and registers it through the PraisonAI plugin entry-point system.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| src/praisonai_plugins/integrations/feedo.py | Adds the Feedo memory adapter and plugin registration, including a narrowed import-error handler that addresses the previous masking issue. |
| tests/test_feedo_plugin.py | Covers registration, identity validation, SDK guidance, transitive import failures, incompatible exports, and core adapter forwarding. |
| pyproject.toml | Registers the Feedo integration as a PraisonAI plugin entry point. |
| README.md | Adds Feedo to the documented integrations list. |
Reviews (3): Last reviewed commit: "fix: re-raise non-missing ImportError in..." | Re-trigger Greptile
|
@claude You are the FINAL architecture reviewer. If the branch is under MervinPraison/PraisonAI-Plugins (not a fork), you are able to make modifications to this branch and push directly. SCOPE: Focus ONLY on PraisonAI-Plugins (src/praisonai_plugins, tests). Lifecycle plugins use praisonai.plugins; sandbox backends use praisonai.sandbox. Do NOT add agent-callable tools here — those belong in PraisonAI-Tools. Read ALL comments above from Gemini, Qodo, CodeRabbit, and Copilot carefully before responding. Phase 1: Review per AGENTS.md
Phase 2: FIX Valid Issues Phase 3: Final Verdict |
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
|
@MervinPraison Happy to add a quick getting-started / usage guide if needed! Feedo is designed around crypto-native identity (DID derived directly from the user's wallet, enabling seamless on-chain balance top-ups without traditional accounts or API key management). Let me know if you'd like me to add a short setup snippet + code example to the docs/README before merging. |
Signed-off-by: Andrii Shumko <andrii@feedo.ink>
7293520 to
64ee1e8
Compare
|
@claude You are the FINAL architecture reviewer. If the branch is under MervinPraison/PraisonAI-Plugins (not a fork), you are able to make modifications to this branch and push directly. SCOPE: Focus ONLY on PraisonAI-Plugins (src/praisonai_plugins, tests). Lifecycle plugins use praisonai.plugins; sandbox backends use praisonai.sandbox. Do NOT add agent-callable tools here — those belong in PraisonAI-Tools. Read ALL comments above from Gemini, Qodo, CodeRabbit, and Copilot carefully before responding. Phase 1: Review per AGENTS.md
Phase 2: FIX Valid Issues Phase 3: Final Verdict |
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
What
Adds Feedo as a memory backend for PraisonAI agents, recreated here in
praisonai-pluginsas a plugin, per your request on #4032.Feedo is a decentralized storage + semantic search network. An agent's
identity is the user's crypto wallet (
did:feedo:0x...), no accounts, noKYC, no vendor lock-in.
Changes
src/praisonai_plugins/integrations/feedo.py— self-contained plugin:FeedoMemoryAdapter(implementsMemoryProtocol) +FeedoPluginthatregisters the factory via
register_memory_factory("feedo", ...)onON_INIT.feedo-sdkis imported lazily, only when an agent selectsprovider: "feedo".pyproject.toml— entry pointfeedo = "praisonai_plugins.integrations.feedo:FeedoPlugin".tests/test_feedo_plugin.py— 5 tests (registration, graceful degradation,fail-fast identity check, missing-SDK guidance, happy-path round-trip).
README.md— addedfeedoto the integrations list.Links
feedo-sdk): https://pypi.org/project/feedo-sdk/feedo-protocol-sdk): https://www.npmjs.com/package/feedo-protocol-sdkHow to test
pip install feedo-sdk feedo init # registers a DID + generates a delegated usage key