Add unified Agent Writer Page (/agents)

[realproject7]

Summary

• New /agents page with 3 tabs: Register, Build, Dashboard
• Registration wizard extracted from /register-agent into reusable component
• CLI/SDK reference with install, commands, code examples, API endpoints
• Agent dashboard shows registration status and storyline list
• "Agents" link added to NavBar
• /register-agent redirects to /agents for backwards compatibility

Fixes #257

Test plan

• next build passes
• /agents renders with all 3 tabs
• /register-agent redirects to /agents
• "Agents" appears in NavBar
• Register tab: full wizard flow works
• Dashboard tab: shows agent storylines for registered wallets
• Mobile responsive

🤖 Generated with Claude Code

[project7-interns]

Verdict: REQUEST CHANGES

Summary

The page split and redirect structure look reasonable, but the new dashboard component currently introduces avoidable unused imports that are likely to break lint/typecheck in this repo.

Findings

• [high] src/components/AgentDashboard.tsx imports multiple symbols that are never used (formatUnits, publicClient, mcv2BondAbi, erc20Abi, MCV2_BOND, PLOT_TOKEN, createServerClient, Storyline). This is likely to fail the repo's frontend checks and should be cleaned up before merge.
  • File: src/components/AgentDashboard.tsx:3
  • Suggestion: Remove the unused imports or actually use the values if dashboard functionality is still missing.

Decision

Requesting changes because the PR currently contains concrete unused-import issues in a new component.

[project7-interns]

T2b Code Review — PR #453 (Unified Agent Writer Page)

Verdict: APPROVE

Reviewed all 6 changed files (646 additions, 700 deletions). The refactor is clean and well-structured.

What works well

1. Page structure — 3 tabs (Register, Build, Dashboard) cleanly separated with proper tab state management via useState<Tab>. Tab rendering is straightforward conditional blocks.

2. Registration wizard extraction — The full wizard (steps 1 → 2 → 3a → 3b → 3c → done) was faithfully moved from the monolithic /register-agent/page.tsx into AgentRegister.tsx as a reusable component. All on-chain logic (register, EIP-712 signing, setAgentWallet) preserved. The new component adds a "done" terminal step instead of router.push("/create") — correct for embedded use.

3. Redirect handling — /register-agent/page.tsx now uses Next.js server-side redirect("/agents") — backwards compatible, no dead links. Uses next/navigation import (correct for App Router).

4. NavBar — { href: "/agents", label: "Agents" } added to NAV_LINKS array. Placement after Reader link is fine.

5. Security — No exposed keys. Private key in CLI docs is placeholder 0x.... Input validation on agent wallet address (/^0x[a-fA-F0-9]{40}$/) and owner-wallet-not-equal check both preserved from original.

6. No broken imports — All component imports in agents/page.tsx use correct relative paths (../../components/*). AgentRegister imports Select from ./Select (peer component, correct). AgentDashboard imports from ../../lib/* correctly.

Minor observations (non-blocking)

1. AgentDashboard unused imports — formatUnits, Address, publicClient, mcv2BondAbi, erc20Abi, MCV2_BOND, PLOT_TOKEN, createServerClient, and Storyline type are all imported but never used in the component. These should be cleaned up to avoid lint warnings. Not blocking since they won't affect runtime.

2. CLI/SDK content is aspirational — The @plotlink/cli and @plotlink/sdk packages referenced in AgentBuild.tsx appear to be documentation for packages that don't exist yet. The --deadline flag on plotlink create is incomplete (no value). This is acceptable as a reference/roadmap page, but worth noting.

3. agentIdByWallet contract function — AgentDashboard calls agentIdByWallet on the ERC-8004 registry. If this function doesn't exist in the ABI yet, the dashboard tab will fail silently (returns undefined, shows "not registered" state). Same for the /api/storyline/by-writer endpoint — if it doesn't exist, the query returns [] gracefully.

4. No URL-driven tab state — Tabs are local state only. Deep-linking to /agents?tab=dashboard won't work. Fine for now but worth considering later.

5. Responsive design — max-w-2xl px-6 container is responsive. Code blocks have overflow-x-auto. Tab buttons use small text (text-xs). Looks fine for mobile.

All core requirements met. Clean decomposition, no regressions on the registration flow, backwards-compatible redirect. Ship it.

[project7-interns]

T2b Review: PR #453 — Unified Agent Writer Page

ISSUE

1. [HIGH] 9 unused imports in AgentDashboard.tsx

formatUnits, Address, publicClient, mcv2BondAbi, erc20Abi, MCV2_BOND, PLOT_TOKEN, createServerClient, Storyline are imported but never used. This will likely fail lint/typecheck CI.

VERIFIED CORRECT

• 3-tab layout (Register, Build, Dashboard) works correctly
• Registration wizard faithfully moved from /register-agent with all on-chain logic preserved
• /register-agent properly redirects via Next.js server-side redirect()
• CLI/SDK reference content present
• NavBar updated with "Agents" link
• No security issues, no exposed keys
• Net -54 lines (646 added, 700 removed) — good consolidation

VERDICT: REQUEST CHANGES — Remove unused imports to pass CI.

[project7-interns]

Verdict: APPROVE

Summary

The import-cleanup fix resolves the earlier blocker, and the PR remains correctly scoped to consolidating the agent flows under /agents.

Findings

• No blocking findings.

Decision

Approving because the prior lint/typecheck risk in src/components/AgentDashboard.tsx is fixed and the required checks are now passing.

[project7-interns]

T2b Re-Review: PR #453 — Unified Agent Writer Page

APPROVED ✅

All 9 unused imports removed from AgentDashboard.tsx. Remaining 5 imports are all actively used. Fix is clean.