Skip to content

Agent status mapping, parallel dashboard loading, rig parsing, bd exec fixes#27

Merged
web3dev1337 merged 1 commit intoweb3dev1337:masterfrom
yougotborked:master
Apr 9, 2026
Merged

Agent status mapping, parallel dashboard loading, rig parsing, bd exec fixes#27
web3dev1337 merged 1 commit intoweb3dev1337:masterfrom
yougotborked:master

Conversation

@yougotborked
Copy link
Copy Markdown
Contributor

Changes

  • Richer agent status: proper state mapping, rig agent distinction, current_task tracking
  • Dashboard: parallel data loading via Promise.allSettled for status/convoys/work/mail
  • Server: robust rig list parsing with JSON fallback to text parsing
  • BDGateway: use bd executable, keep --no-daemon compat, use create command
  • Various UI/component improvements across sidebar, agent-grid, work-list, etc.

21 files changed, 603 insertions, 310 deletions

…ashboard loading, rig parsing, bd exec fixes

- Richer agent status: proper state mapping, rig agent distinction, current_task tracking
- Dashboard: parallel data loading via Promise.allSettled for status/convoys/work/mail
- Server: robust rig list parsing with JSON fallback to text parsing
- BDGateway: use 'bd' executable, keep --no-daemon compat, use 'create' command
- Various UI/component improvements across sidebar, agent-grid, work-list, etc.
Copy link
Copy Markdown
Owner

@web3dev1337 web3dev1337 left a comment

Choose a reason for hiding this comment

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

This PR introduces regressions that should block merge. I opened a replacement PR with the fixes and full validation here: #28

Key issues in #27:

  • websocket refresh handling now relies on state.currentView, but that field is never defined or updated, and several emitted events are no longer handled, so the UI can go stale until a manual refresh
  • /api/agents and the dashboard assume rig.agents, but the current backend contract and tests still use legacy rig.hooks, which drops rig workers from summaries and agent listings
  • BDGateway bypasses the resolved executable path and hardcodes bd, which breaks configured executable overrides and weakens the current trust boundary
  • main refresh paths no longer force cache-busting even though the affected endpoints are still cached, so refreshes can legitimately return stale data

Replacement PR #28 keeps the useful parts of the submission, restores compatibility, and adds regression coverage.

@yougotborked yougotborked requested a review from web3dev1337 April 8, 2026 14:27
@web3dev1337
Copy link
Copy Markdown
Owner

Thanks for the revision, @yougotborked. I reran the full suite and kept your authored commit (afeb7b0) preserved in the salvage path. We’re merging PR #28 via merge commit (not squash) so your contribution remains directly in project history.

@web3dev1337 web3dev1337 merged commit afeb7b0 into web3dev1337:master Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants