Skip to content

fix(langgraph): resolve tool bank IDs from config#2443

Merged
nicoloboschi merged 2 commits into
vectorize-io:mainfrom
koriyoshi2041:rios/langgraph-tools-config-bank
Jun 30, 2026
Merged

fix(langgraph): resolve tool bank IDs from config#2443
nicoloboschi merged 2 commits into
vectorize-io:mainfrom
koriyoshi2041:rios/langgraph-tools-config-bank

Conversation

@koriyoshi2041

Copy link
Copy Markdown
Contributor

Summary

  • allow LangGraph tools to omit a static bank_id and resolve it from RunnableConfig configurable values
  • keep static bank_id precedence for existing single-bank usage
  • document the multi-user tools pattern and add regression coverage for retain/recall/reflect

Fixes #2442.

Tests

  • cd hindsight-integrations/langgraph && uv run pytest tests/test_tools.py
  • cd hindsight-integrations/langgraph && uv run pytest tests
  • cd hindsight-integrations/langgraph && uvx ruff format --check hindsight_langgraph tests
  • cd hindsight-integrations/langgraph && uvx ruff check --config ../../ruff.toml hindsight_langgraph tests
  • git diff --check

Note: the repository pre-commit hook could not complete locally because the checkout is missing control-plane eslint dependencies (@eslint/js) and uv run --no-project ruff could not spawn ruff for integrations; the focused checks above passed.

koriyoshi2041 and others added 2 commits June 30, 2026 10:19
Rename the injected RunnableConfig tool parameter to runnable_config so it
no longer shadows the outer Hindsight config = get_config().
@nicoloboschi nicoloboschi force-pushed the rios/langgraph-tools-config-bank branch from 3a48b23 to 7b9fb09 Compare June 30, 2026 08:20
@nicoloboschi nicoloboschi merged commit 760bfc7 into vectorize-io:main Jun 30, 2026
83 of 84 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants