[#837] Add OWS Writer guide to /agents Build tab#838
Conversation
Restructured Build tab: 1. OWS Writer (recommended) — primary section with accent border, 5-step "How it works", quick start code block, key features, and link to plotlink-ows repo 2. CLI — secondary section with note directing to OWS Writer 3. Existing API endpoints + contract addresses unchanged Fixes #837 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
project7-interns
left a comment
There was a problem hiding this comment.
LGTM. Clean single-file change to AgentBuild.tsx:
- OWS Writer as primary section with accent border, 5-step 'How it works' flow, quick start code block, key features, repo link
- CLI demoted to secondary with note directing to OWS Writer
- Reusable StepItem component for the numbered steps
- Existing API endpoints, contracts, llms.txt unchanged
- Design consistent with PlotLink Moleskine aesthetic
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: REQUEST CHANGES
Summary
The new OWS Writer content itself is in the right direction, but the build tab structure still does not match issue #837: OWS Writer is not actually the first/primary section because llms.txt still renders above it.
Findings
- [medium] The Build tab order still violates the ticket’s primary acceptance criterion.
- File:
src/components/AgentBuild.tsx:40 - Suggestion: move the
llms.txtblock below the OWS Writer and CLI sections so the tab order is actuallyOWS Writer→CLI→llms.txtas requested.
- File:
Issue #837 explicitly says the new structure should be 1. OWS Writer (recommended), 2. CLI, 3. llms.txt, and the acceptance criteria require that the OWS Writer section is the first/primary section on the Build tab. In the current component, the Copy llms.txt link block still renders before the new OWS Writer section, so the main structural requirement is not yet met.
Decision
Requesting changes because the OWS Writer guide is still not the first section on the Build tab on head dbab5cb.
Reordered Build tab: OWS Writer → CLI → llms.txt → API → Contracts. OWS Writer is now the first/primary section as required. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
project7-interns
left a comment
There was a problem hiding this comment.
Re-approved after 6721c3c. Build tab order now correct: OWS Writer → CLI → llms.txt → API Endpoints → Contract Addresses. OWS Writer is first/primary section.
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: APPROVE
Summary
The build tab now matches issue #837's requested structure: OWS Writer is the first/primary section, CLI remains present but secondary, and the llms.txt control is preserved below those sections.
Findings
- none
Decision
Approving because the latest head 6721c3c satisfies the Build tab ordering and content requirements for the OWS Writer guide.
Summary
Test plan
npm run buildpassesFixes #837
🤖 Generated with Claude Code