Skip to content

fix(kanban): keep graph tools orchestrator-only - #30299

Closed
mosaiq-systems wants to merge 1 commit into
NousResearch:mainfrom
mosaiq-systems:fix/kanban-orchestrator-graph-tools
Closed

fix(kanban): keep graph tools orchestrator-only#30299
mosaiq-systems wants to merge 1 commit into
NousResearch:mainfrom
mosaiq-systems:fix/kanban-orchestrator-graph-tools

Conversation

@mosaiq-systems

Copy link
Copy Markdown

Summary

  • make kanban_create and kanban_link orchestrator-only graph-routing tools instead of exposing them to dispatcher-scoped workers
  • enforce the same orchestrator boundary inside the handlers, so stale schemas or direct dispatch cannot mutate graph topology from worker scope
  • mark environment-sensitive Kanban tool gates as uncached to prevent orchestrator visibility from leaking into later worker schema builds in long-lived processes
  • update worker prompt guidance to record follow-up work through comments, completion metadata, or blocking instead of spawning tasks

Tests

  • uv run --with pytest --with pytest-asyncio --with pytest-timeout python -m pytest tests/tools/test_kanban_tools.py tests/hermes_cli/test_kanban_db.py -q -o 'addopts=' (244 passed)
  • uv run python -m py_compile tools/registry.py tools/kanban_tools.py agent/prompt_builder.py tests/tools/test_kanban_tools.py
  • independent review checked the diff for project-specific leakage, local paths, secrets, and cache/gate regressions

Hide kanban_create and kanban_link from dispatcher-scoped workers, enforce the same boundary at handler runtime, and prevent env-sensitive Kanban tool gates from leaking through the registry check cache.
@alt-glitch alt-glitch added type/bug Something isn't working comp/tools Tool registry, model_tools, toolsets P3 Low — cosmetic, nice to have labels May 22, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

V2 resubmission after closed #30022 by the same author. Adds uncached tool gate flags and updated worker prompt guidance compared to the prior attempt.

@mosaiq-systems mosaiq-systems closed this by deleting the head repository Jun 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/tools Tool registry, model_tools, toolsets P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants