Skip to content

feat(kanban): surface task outcomes and blocked context - #29241

Closed
Lypidri wants to merge 1 commit into
NousResearch:mainfrom
Lypidri:feat/kanban-summary-source-pr
Closed

feat(kanban): surface task outcomes and blocked context#29241
Lypidri wants to merge 1 commit into
NousResearch:mainfrom
Lypidri:feat/kanban-summary-source-pr

Conversation

@Lypidri

@Lypidri Lypidri commented May 20, 2026

Copy link
Copy Markdown

Summary

Reopens the Kanban dashboard improvements from #25655 as a fresh branch against current main, without replaying the stale old bundle wholesale.

This updates the selected-task drawer/API so reviewers can see the useful task outcome information first:

  • Adds a summary-tree payload for a selected task and descendants.
  • Surfaces result/handoff text, latest run summary, run metadata, subtask status/progress, and important comments.
  • Adds structured blocked-task context on board/detail/summary-tree payloads: reason, missing info, latest relevant comment, and a comment prompt.
  • Adds artifact availability metadata so unavailable/missing/scratch paths render as explicit empty states instead of misleading open/copy actions.
  • Keeps local artifact opening restricted to backend-derived, user-actionable artifacts; comment-derived paths are context-only and not openable.
  • Updates the current Kanban dashboard bundle carefully rather than reverting it to the old PR's stale state.

Notes on the previous PR feedback

The old PR was closed because its plugins/kanban/dashboard/dist/index.js was stale against current main. This branch starts from current origin/main and ports the behavior onto the current Kanban dashboard code, preserving newer scheduled/review/workflow/orchestration changes.

I looked for a maintained Kanban source file under web/src/plugins/kanban; this bundled dashboard plugin currently ships as a plain non-minified IIFE in plugins/kanban/dashboard/dist, so the frontend changes are applied there while keeping the diff focused.

Test Plan

  • git diff --check -- plugins/kanban/dashboard/plugin_api.py plugins/kanban/dashboard/dist/index.js plugins/kanban/dashboard/dist/style.css tests/plugins/test_kanban_dashboard_plugin.py
  • node --check plugins/kanban/dashboard/dist/index.js
  • $HOME/.hermes/hermes-agent/venv/bin/python -m pytest tests/plugins/test_kanban_dashboard_plugin.py -q -o 'addopts=' — 102 passed
  • $HOME/.hermes/hermes-agent/venv/bin/python -m pytest tests/hermes_cli/test_kanban*.py tests/tools/test_kanban_tools.py -q -o 'addopts=' — 620 passed, 1 skipped
  • npm --prefix web ci
  • npm --prefix web run build

Additional check:

  • pytest tests/plugins -q -o 'addopts=' currently reports 751 passed / 1 failed. The failure is unrelated to this diff: tests/plugins/web/test_web_search_provider_plugins.py::TestBundledPluginsRegister::test_all_seven_plugins_present_in_registry expects seven web search providers, but current plugin discovery includes xai as an eighth provider.

@SatoDri

SatoDri commented May 20, 2026

Copy link
Copy Markdown

Superseded by #29247, opened from the SatoDri fork/account.

1 similar comment
@Lypidri

Lypidri commented May 20, 2026

Copy link
Copy Markdown
Author

Superseded by #29247, opened from the SatoDri fork/account.

@Lypidri Lypidri closed this May 20, 2026
@alt-glitch alt-glitch added type/feature New feature or request P3 Low — cosmetic, nice to have comp/plugins Plugin system and bundled plugins labels May 20, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Resubmission of closed #25655 rebased onto current main. Same feature scope: task outcome summaries, blocked context, and artifact availability metadata for the Kanban dashboard.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants