Skip to content

docs(kanban): RFC task-scoped skill injection - #37304

Draft
hehehe0803 wants to merge 1 commit into
NousResearch:mainfrom
hehehe0803:rfc/task-scoped-skill-injection
Draft

docs(kanban): RFC task-scoped skill injection#37304
hehehe0803 wants to merge 1 commit into
NousResearch:mainfrom
hehehe0803:rfc/task-scoped-skill-injection

Conversation

@hehehe0803

Copy link
Copy Markdown
Contributor

Summary

  • Adds an RFC/developer-guide page for Kanban task-scoped read-only skill injection.
  • Defines how required task.skills should preflight against profile-local roots plus approved read-only task overlay roots.
  • Specifies resolver precedence/collision behavior, provenance/audit events, and future optional_skills semantics.
  • Keeps the design upstream-generic: no dependency on local Hermes OS overlay/profile-sync scripts.

Relationship to existing work

Test plan

  • cd website && npm run build
    • Build succeeded.
    • Existing Docusaurus warnings remain for unrelated broken links/anchors in web-dashboard and zh-Hans docs; no new failure from this RFC page.

@alt-glitch alt-glitch added type/docs Documentation improvements P3 Low — cosmetic, nice to have tool/skills Skills system (list, view, manage) labels Jun 2, 2026

@teknium1 teknium1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for documenting the broader overlay design. The RFC remains useful, but its baseline needs an update before it can serve as a reliable developer guide.

Problems

  • website/docs/developer-guide/kanban-task-scoped-skill-injection.md:46 and :171 describe a spawned --skills kanban-worker. Current dispatch forwards only declared task skills (hermes_cli/kanban_db.py:8031-8039); the current regression test explicitly says no skill is auto-loaded (tests/hermes_cli/test_kanban_core_functionality.py:3043-3079).
  • website/docs/developer-guide/kanban-task-scoped-skill-injection.md:48 says missing required skills fail at startup. Current CLI skips missing names when at least one requested skill loads, and raises only when all are missing (cli.py:15931-15947).

Suggested changes

  • Refresh the current-state comparison and label fail-closed handling as proposed RFC behavior, not current behavior.

Automated hermes-sweeper review.


1. Dispatcher claims a Kanban task.
2. Dispatcher resolves the assignee profile home.
3. Dispatcher builds a child command like `hermes -p <profile> --skills kanban-worker --skills <task-skill> chat -q "work kanban task <id>"`.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is stale against current main: _default_spawn() forwards only task.skills (hermes_cli/kanban_db.py:8031-8039), and tests/hermes_cli/test_kanban_core_functionality.py:3043-3079 explicitly asserts no skill is auto-loaded. Please remove kanban-worker from the current-state command and explain that lifecycle guidance is injected separately.

@teknium1 teknium1 added the sweeper:blast-contained Sweeper blast radius: contained — one narrow path / opt-in / few users label Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have sweeper:blast-contained Sweeper blast radius: contained — one narrow path / opt-in / few users tool/skills Skills system (list, view, manage) type/docs Documentation improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants