vxcontrol/pentagi
1
0
Fork 0
mirror of https://github.com/vxcontrol/pentagi.git synced 2026-07-20 23:57:11 -04:00
Code Issues 45 Packages Projects Releases 9 Wiki Activity

[PR #89] [MERGED] feat: graphiti search #152

New Issue
Closed
opened 2026-06-06 22:09:26 -04:00 by yindo · 0 comments
yindo commented 2026-06-06 22:09:26 -04:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/vxcontrol/pentagi/pull/89
Author: @zavgorodnii
Created: 12/2/2025
Status: ✅ Merged
Merged: 12/30/2025
Merged by: @asdek

Base: master ← Head: feat/graphiti-search

📝 Commits (10+)

  • 65232f3 feat: added graphiti integration
  • 338daff feat: graphiti-search
  • d414203 graphiti uses global llm provider
  • 13b7150 added proper graphiti disabling + set up logging
  • da9019b feat: added patch refiner
  • adf89ab Merge pull request #93 from vxcontrol/feat/patch-refiner
  • fca31d3 feat: integrate Graphiti with Docker Compose and update environment configurations
  • 4b90669 feat: update Anthropic and Gemini configurations for model enhancements and performance improvements
  • 558bcb1 fix: add missing username for private scraper on installer form
  • ae80dc0 Merge branch 'master' into feat/graphiti-search

📊 Changes

52 files changed (+4413 additions, -1253 deletions)

View changed files

📝 .env.example (+12 -1)
📝 README.md (+114 -11)
📝 backend/cmd/ftester/mocks/tools.go (+126 -0)
📝 backend/cmd/ftester/worker/args.go (+1 -0)
📝 backend/cmd/ftester/worker/executor.go (+32 -20)
📝 backend/cmd/ftester/worker/tester.go (+2 -0)
➕ backend/cmd/installer/files/links/docker-compose-graphiti.yml (+1 -0)
📝 backend/cmd/installer/wizard/models/scraper_form.go (+4 -0)
📝 backend/cmd/pentagi/main.go (+8 -0)
📝 backend/docs/config.md (+51 -1)
📝 backend/go.mod (+1 -0)
📝 backend/go.sum (+2 -0)
📝 backend/pkg/config/config.go (+5 -0)
📝 backend/pkg/controller/assistant.go (+1 -0)
📝 backend/pkg/controller/flow.go (+2 -0)
📝 backend/pkg/docker/client.go (+4 -1)
➕ backend/pkg/graphiti/client.go (+141 -0)
📝 backend/pkg/observability/langfuse/api/internal/caller_test.go (+1 -1)
📝 backend/pkg/observability/langfuse/api/internal/error_decoder_test.go (+1 -1)
📝 backend/pkg/observability/langfuse/api/internal/retrier_test.go (+1 -1)

...and 32 more files

📄 Description

Description of the Change

Problem

Solution

Closes #80

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • 🚀 New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • 🔧 Configuration change
  • 🧪 Test update
  • 🛡️ Security update

Areas Affected

  • Core Services (Frontend UI/Backend API)
  • AI Agents (Researcher/Developer/Executor)
  • Security Tools Integration
  • Memory System (Vector Store/Knowledge Base)
  • Monitoring Stack (Grafana/OpenTelemetry)
  • Analytics Platform (Langfuse)
  • External Integrations (LLM/Search APIs)
  • Documentation
  • Infrastructure/DevOps

Testing and Verification

Test Configuration

PentAGI Version:
Docker Version:
Host OS:
LLM Provider:
Enabled Features: [Langfuse/Grafana/etc]

Test Steps

Test Results

Security Considerations

Performance Impact

Documentation Updates

  • README.md updates
  • API documentation updates
  • Configuration documentation updates
  • GraphQL schema updates
  • Other:

Deployment Notes

Checklist

Code Quality

  • My code follows the project's coding standards
  • I have added/updated necessary documentation
  • I have added tests to cover my changes
  • All new and existing tests pass
  • I have run go fmt and go vet (for Go code)
  • I have run npm run lint (for TypeScript/JavaScript code)

Security

  • I have considered security implications
  • Changes maintain or improve the security model
  • Sensitive information has been properly handled

Compatibility

  • Changes are backward compatible
  • Breaking changes are clearly marked and documented
  • Dependencies are properly updated

Documentation

  • Documentation is clear and complete
  • Comments are added for non-obvious code
  • API changes are documented

Additional Notes

Note

Graphiti knowledge graph integration

  • New pkg/graphiti client (using graphiti-go-client), wired through providers/executors; agents now store responses and tool executions to Graphiti via templates
  • New graphiti_search tool with multiple search modes (recent_context, successful_tools, episode_context, entity_relationships, temporal_window, diverse_results, entity_by_label) and prompt updates to leverage Graphiti
  • Config/env: GRAPHITI_* and Neo4j settings; docker-compose-graphiti.yml and installer link; README and docs updated (architecture, quick start, usage)

Planning/refinement changes

  • Add delta-based subtask refinement: new subtask_patch tool, validation and application logic, tests; prompt updated from list replacement to patch operations

Other

  • Browser scraper URL selection improved with private/public fallback + tests
  • Docker containers now use on-failure restart policy; main sets log level from DEBUG
  • Provider model configs refreshed; minor installer field additions and mocks

Written by Cursor Bugbot for commit ae80dc063e. This will update automatically on new commits. Configure here.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/vxcontrol/pentagi/pull/89 **Author:** [@zavgorodnii](https://github.com/zavgorodnii) **Created:** 12/2/2025 **Status:** ✅ Merged **Merged:** 12/30/2025 **Merged by:** [@asdek](https://github.com/asdek) **Base:** `master` ← **Head:** `feat/graphiti-search` --- ### 📝 Commits (10+) - [`65232f3`](https://github.com/vxcontrol/pentagi/commit/65232f30dccda329662eb0a7e70826697ad8d057) feat: added graphiti integration - [`338daff`](https://github.com/vxcontrol/pentagi/commit/338daff29a889e5b43d2bc1ebd39bec0fba60cea) feat: graphiti-search - [`d414203`](https://github.com/vxcontrol/pentagi/commit/d41420374a7abe6206a9bcc80d79d6e16cd43c4b) graphiti uses global llm provider - [`13b7150`](https://github.com/vxcontrol/pentagi/commit/13b7150c7b5a80bd78d95f5805230360cf374f27) added proper graphiti disabling + set up logging - [`da9019b`](https://github.com/vxcontrol/pentagi/commit/da9019bdf3c651d83b24faa49510b170d4b251d2) feat: added patch refiner - [`adf89ab`](https://github.com/vxcontrol/pentagi/commit/adf89ab6f0ed70e90681137e953ba70445fdf829) Merge pull request #93 from vxcontrol/feat/patch-refiner - [`fca31d3`](https://github.com/vxcontrol/pentagi/commit/fca31d39634ef2e1237f53b2e0eb96f981dc8483) feat: integrate Graphiti with Docker Compose and update environment configurations - [`4b90669`](https://github.com/vxcontrol/pentagi/commit/4b9066980c5592994aa84361c1fa75d7beacdf67) feat: update Anthropic and Gemini configurations for model enhancements and performance improvements - [`558bcb1`](https://github.com/vxcontrol/pentagi/commit/558bcb17b81a8d7ceead5ae3619c26690e534adc) fix: add missing username for private scraper on installer form - [`ae80dc0`](https://github.com/vxcontrol/pentagi/commit/ae80dc063e074becef3df9ca59d6f7b64addabeb) Merge branch 'master' into feat/graphiti-search ### 📊 Changes **52 files changed** (+4413 additions, -1253 deletions) <details> <summary>View changed files</summary> 📝 `.env.example` (+12 -1) 📝 `README.md` (+114 -11) 📝 `backend/cmd/ftester/mocks/tools.go` (+126 -0) 📝 `backend/cmd/ftester/worker/args.go` (+1 -0) 📝 `backend/cmd/ftester/worker/executor.go` (+32 -20) 📝 `backend/cmd/ftester/worker/tester.go` (+2 -0) ➕ `backend/cmd/installer/files/links/docker-compose-graphiti.yml` (+1 -0) 📝 `backend/cmd/installer/wizard/models/scraper_form.go` (+4 -0) 📝 `backend/cmd/pentagi/main.go` (+8 -0) 📝 `backend/docs/config.md` (+51 -1) 📝 `backend/go.mod` (+1 -0) 📝 `backend/go.sum` (+2 -0) 📝 `backend/pkg/config/config.go` (+5 -0) 📝 `backend/pkg/controller/assistant.go` (+1 -0) 📝 `backend/pkg/controller/flow.go` (+2 -0) 📝 `backend/pkg/docker/client.go` (+4 -1) ➕ `backend/pkg/graphiti/client.go` (+141 -0) 📝 `backend/pkg/observability/langfuse/api/internal/caller_test.go` (+1 -1) 📝 `backend/pkg/observability/langfuse/api/internal/error_decoder_test.go` (+1 -1) 📝 `backend/pkg/observability/langfuse/api/internal/retrier_test.go` (+1 -1) _...and 32 more files_ </details> ### 📄 Description <!-- Thank you for your contribution to PentAGI! Please fill out this template completely to help us review your changes effectively. Any PR that does not include enough information may be closed at maintainers' discretion. --> ### Description of the Change <!-- We must be able to understand the design of your change from this description. Please provide as much detail as possible. --> #### Problem <!-- Describe the problem this PR addresses --> #### Solution <!-- Describe your solution and its key aspects --> <!-- Enter any applicable Issue number(s) here that will be closed/resolved by this PR. --> Closes #80 ### Type of Change <!-- Mark with an `x` all options that apply --> - [ ] 🐛 Bug fix (non-breaking change which fixes an issue) - [x] 🚀 New feature (non-breaking change which adds functionality) - [ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] 📚 Documentation update - [x] 🔧 Configuration change - [ ] 🧪 Test update - [ ] 🛡️ Security update ### Areas Affected <!-- Mark with an `x` all components that are affected --> - [x] Core Services (Frontend UI/Backend API) - [x] AI Agents (Researcher/Developer/Executor) - [ ] Security Tools Integration - [ ] Memory System (Vector Store/Knowledge Base) - [ ] Monitoring Stack (Grafana/OpenTelemetry) - [ ] Analytics Platform (Langfuse) - [ ] External Integrations (LLM/Search APIs) - [ ] Documentation - [ ] Infrastructure/DevOps ### Testing and Verification <!-- Please describe the tests that you ran to verify your changes and provide instructions so we can reproduce. --> #### Test Configuration ```yaml PentAGI Version: Docker Version: Host OS: LLM Provider: Enabled Features: [Langfuse/Grafana/etc] ``` #### Test Steps 1. 2. 3. #### Test Results <!-- Include relevant screenshots, logs, or test outputs --> ### Security Considerations <!-- Describe any security implications of your changes. For security-related changes, please note any new dependencies, changed permissions, etc. --> ### Performance Impact <!-- Describe any performance implications and testing done to verify acceptable performance. Especially important for changes affecting AI agents, memory systems, or data processing. --> ### Documentation Updates <!-- Note any documentation changes required by this PR --> - [ ] README.md updates - [ ] API documentation updates - [ ] Configuration documentation updates - [ ] GraphQL schema updates - [ ] Other: <!-- specify --> ### Deployment Notes <!-- Describe any special considerations for deploying this change. Include any new environment variables, configuration changes, or migration steps. --> ### Checklist <!--- Go over all the following points, and put an `x` in all the boxes that apply. --> #### Code Quality - [ ] My code follows the project's coding standards - [ ] I have added/updated necessary documentation - [ ] I have added tests to cover my changes - [ ] All new and existing tests pass - [ ] I have run `go fmt` and `go vet` (for Go code) - [ ] I have run `npm run lint` (for TypeScript/JavaScript code) #### Security - [ ] I have considered security implications - [ ] Changes maintain or improve the security model - [ ] Sensitive information has been properly handled #### Compatibility - [ ] Changes are backward compatible - [ ] Breaking changes are clearly marked and documented - [ ] Dependencies are properly updated #### Documentation - [ ] Documentation is clear and complete - [ ] Comments are added for non-obvious code - [ ] API changes are documented ### Additional Notes <!-- Any additional information that would be helpful for reviewers --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Graphiti knowledge graph integration** > > - New `pkg/graphiti` client (using `graphiti-go-client`), wired through providers/executors; agents now store responses and tool executions to Graphiti via templates > - New `graphiti_search` tool with multiple search modes (`recent_context`, `successful_tools`, `episode_context`, `entity_relationships`, `temporal_window`, `diverse_results`, `entity_by_label`) and prompt updates to leverage Graphiti > - Config/env: `GRAPHITI_*` and Neo4j settings; `docker-compose-graphiti.yml` and installer link; README and docs updated (architecture, quick start, usage) > > **Planning/refinement changes** > > - Add delta-based subtask refinement: new `subtask_patch` tool, validation and application logic, tests; prompt updated from list replacement to patch operations > > **Other** > > - Browser scraper URL selection improved with private/public fallback + tests > - Docker containers now use on-failure restart policy; main sets log level from `DEBUG` > - Provider model configs refreshed; minor installer field additions and mocks > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit ae80dc063e074becef3df9ca59d6f7b64addabeb. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-06-06 22:09:26 -04:00
yindo closed this issue 2026-06-06 22:09:26 -04:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
enhancement
help wanted
pull-request
Mirrored from GitHub Pull Request
 question
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: vxcontrol/pentagi#152
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?