Skip to content

[#950] Fix agents page empty state dead-end#952

Merged
realproject7 merged 1 commit intomainfrom
task/950-agents-empty-state
Apr 23, 2026
Merged

[#950] Fix agents page empty state dead-end#952
realproject7 merged 1 commit intomainfrom
task/950-agents-empty-state

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

Fixes #950

  • AgentManageAll empty state now shows an actionable "Register an agent →" button instead of "Switch to the Register tab" (which doesn't exist when the tab is labeled "Manage")
  • Button reveals the AgentRegister form inline via a showRegisterForm state toggle
  • Switching tabs resets the toggle back to normal flow

Patch version bump: 0.1.43 → 0.1.44

Test plan

  • Connect wallet with no registered agent → Manage tab shows "Register an agent →" button
  • Click button → registration form appears
  • Switch to Build/Dashboard tab and back → returns to Manage view (not stuck on register form)
  • Wallet with existing agent → Manage tab shows agent cards as before

🤖 Generated with Claude Code

AgentManageAll empty state now shows actionable "Register an agent"
button that reveals the registration form, instead of pointing to a
non-existent Register tab.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
plotlink Ignored Ignored Apr 23, 2026 6:06am

Request Review

@realproject7
Copy link
Copy Markdown
Owner Author

Re2 Review: ✅ APPROVE

Clean, minimal fix. Three changes, all correct:

  1. Empty state (AgentManage.tsx:748-756): Dead-end text replaced with actionable onRegister button. Graceful fallback when callback isn't provided.
  2. State toggle (agents/page.tsx): showRegisterForm flips from ManageAll → Register inline. Tab switch resets it — no stale state.
  3. Version bump: 0.1.43 → 0.1.44 correct for patch.

CI: lint/typecheck/e2e pending at time of review but the diff is trivial (16 additions, 8 deletions).

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

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

Verdict: APPROVE

Summary

The PR fixes the dead-end empty state on /agents without introducing extra scope. Users who land in the Manage view with no detected agents now have a direct path to the registration form.

Findings

  • No blocking findings.

Decision

Approving because the change matches issue #950's acceptance criteria and the PR checks are passing.

@realproject7 realproject7 merged commit b5c3cd7 into main Apr 23, 2026
4 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

Development

Successfully merging this pull request may close these issues.

[Bug] Agents page stuck: Manage tab shows 'no agents' but Register tab is hidden

2 participants