fix(hindsight): overhaul hindsight memory plugin and memory setup wizard#5094
Merged
Conversation
- Dedicated asyncio event loop for Hindsight async calls (fixes aiohttp session leaks) - Client caching (reuse instead of creating per-call) - Local mode daemon management with config change detection and auto-restart - Memory mode support (hybrid/context/tools) and prefetch method (recall/reflect) - Proper shutdown with event loop and client cleanup - Disable HindsightEmbedded.__del__ to avoid GC loop errors - Update API URLs (app -> ui.hindsight.vectorize.io, api_url -> base_url) - Setup wizard: conditional fields (when clause), dynamic defaults (default_from) - Switch dependency install from pip to uv (correct for uv-based venvs) - Add hindsight-all to plugin.yaml and import mapping - 12 new tests for dispatch routing and setup field filtering Original PR #5044 by cdbartholomew.
Contributor
|
6 tasks
benfrank241
added a commit
to vectorize-io/hindsight
that referenced
this pull request
Apr 6, 2026
Reflects changes from NousResearch/hermes-agent#5094: - Replace pip install quick start with hermes memory setup wizard - Update config path to ~/.hermes/hindsight/config.json with flat snake_case keys - Add memory_mode (hybrid/context/tools) and prefetch_method (recall/reflect) - Add LLM provider config section for local mode with all 7 providers and default models - Add HINDSIGHT_MODE, HINDSIGHT_LLM_API_KEY, HINDSIGHT_API_URL env vars - Update local daemon section with auto-start behavior and log paths - Fix URL: app.hindsight.vectorize.io -> ui.hindsight.vectorize.io - Note tools only registered in hybrid/tools modes
benfrank241
added a commit
to vectorize-io/hindsight
that referenced
this pull request
Apr 6, 2026
Reflects changes from NousResearch/hermes-agent#5094: - Replace pip install quick start with hermes memory setup wizard - Update config path to ~/.hermes/hindsight/config.json with flat snake_case keys - Add memory_mode (hybrid/context/tools) and prefetch_method (recall/reflect) - Add LLM provider config section for local mode with all 7 providers and default models - Add HINDSIGHT_MODE, HINDSIGHT_LLM_API_KEY, HINDSIGHT_API_URL env vars - Update local daemon section with auto-start behavior and log paths - Fix URL: app.hindsight.vectorize.io -> ui.hindsight.vectorize.io - Note tools only registered in hybrid/tools modes
benfrank241
added a commit
to vectorize-io/hindsight
that referenced
this pull request
Apr 6, 2026
Reflects changes from NousResearch/hermes-agent#5094: - Replace pip install quick start with hermes memory setup wizard - Update config path to ~/.hermes/hindsight/config.json with flat snake_case keys - Add memory_mode (hybrid/context/tools) and prefetch_method (recall/reflect) - Add LLM provider config section for local mode with all 7 providers and default models - Add HINDSIGHT_MODE, HINDSIGHT_LLM_API_KEY, HINDSIGHT_API_URL env vars - Update local daemon section with auto-start behavior and log paths - Fix URL: app.hindsight.vectorize.io -> ui.hindsight.vectorize.io - Note tools only registered in hybrid/tools modes
benfrank241
added a commit
to vectorize-io/hindsight
that referenced
this pull request
Apr 6, 2026
Reflects changes from NousResearch/hermes-agent#5094: - Replace pip install quick start with hermes memory setup wizard - Update config path to ~/.hermes/hindsight/config.json with flat snake_case keys - Add memory_mode (hybrid/context/tools) and prefetch_method (recall/reflect) - Add LLM provider config section for local mode with all 7 providers and default models - Add HINDSIGHT_MODE, HINDSIGHT_LLM_API_KEY, HINDSIGHT_API_URL env vars - Update local daemon section with auto-start behavior and log paths - Fix URL: app.hindsight.vectorize.io -> ui.hindsight.vectorize.io - Note tools only registered in hybrid/tools modes
nicoloboschi
added a commit
to vectorize-io/hindsight
that referenced
this pull request
Apr 8, 2026
…mory provider Hermes Agent now ships with a native Hindsight memory provider (NousResearch/hermes-agent#5094), making our pip-installable hindsight-hermes package redundant. Removes: - hindsight-integrations/hermes/ (source, tests, config) - CI job, release script entry, changelog generator references - Cookbook page and pip package changelog (referenced deleted code) Keeps: - Integration docs (updated by #881 for native provider) - Blog posts (historical, already have deprecation notices) - Sidebar/banner entries (still valid for native integration)
3 tasks
nicoloboschi
added a commit
to vectorize-io/hindsight
that referenced
this pull request
Apr 8, 2026
…mory provider Hermes Agent now ships with a native Hindsight memory provider (NousResearch/hermes-agent#5094), making our pip-installable hindsight-hermes package redundant. Removes: - hindsight-integrations/hermes/ (source, tests, config) - CI job, release script entry, changelog generator references - Cookbook page and pip package changelog (referenced deleted code) Keeps: - Integration docs (updated by #881 for native provider) - Blog posts (historical, already have deprecation notices) - Sidebar/banner entries (still valid for native integration)
nicoloboschi
added a commit
to vectorize-io/hindsight
that referenced
this pull request
Apr 8, 2026
* chore: drop hindsight-hermes integration in favor of native Hermes memory provider Hermes Agent now ships with a native Hindsight memory provider (NousResearch/hermes-agent#5094), making our pip-installable hindsight-hermes package redundant. Removes: - hindsight-integrations/hermes/ (source, tests, config) - CI job, release script entry, changelog generator references - Cookbook page and pip package changelog (referenced deleted code) Keeps: - Integration docs (updated by #881 for native provider) - Blog posts (historical, already have deprecation notices) - Sidebar/banner entries (still valid for native integration) * fix(docs): remove broken cookbook link to deleted hermes-memory page
19 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Salvage of PR #5044 by @cdbartholomew — selective cherry-pick of the substantive hindsight memory plugin and setup wizard changes onto current main.
What changed
Hindsight plugin overhaul (
plugins/memory/hindsight/__init__.py):HindsightEmbedded.__del__to avoid GC loop errorsapp→ui.hindsight.vectorize.io,api_url→base_url)Memory setup wizard (
hermes_cli/memory_setup.py):whenclause) — show/hide fields based on modedefault_from) — model auto-defaults per providerhindsight-allimport mappingTests & docs:
plugin.yaml: addhindsight-alldependencyWhat was dropped
The original PR was 4 commits behind main and silently reverted:
All non-hindsight changes were dropped. Only 5 files changed.
Test results
Co-authored-by: Chris Bartholomew chris.bartholomew@vectorize.io