[#643] Add /llms.txt, copy button, and tab deep links#658
Conversation
1. New /llms.txt route: plain-text integration info with CLI commands, API endpoints, contract addresses, chain info, env vars 2. AgentBuild: "Copy llms.txt link" button copies plotlink.xyz/llms.txt 3. Agents page: reads ?tab=build|register|dashboard from URL params for direct deep linking (wrapped in Suspense for SSR) Fixes #643 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.
Verdict: REQUEST CHANGES
Summary
The routing and copy-link pieces look fine, but the new /llms.txt payload is still underspecified for the API section. This ticket explicitly asks for request and response formats, and the route currently documents request bodies only.
Findings
- [high] The new
llms.txtendpoint lists the four indexer request bodies, but it does not document any response formats even though that is part of the ticket requirement for machine-readable integration details. As written, an agent consumer still has to inspect the app source to know whether to expect{ success: true },{ indexed: n }, or error payloads.- File:
src/app/llms.txt/route.ts:32 - Suggestion: Expand each endpoint entry to include at least the normal success shape and the error JSON shape, for example storyline/plot/donation success responses and the trade indexer's
{ indexed }response.
- File:
Decision
Requesting changes because the core new artifact in this PR, /llms.txt, still does not meet the stated requirement to provide request/response formats for the API endpoints.
Each endpoint now shows request body, success response, and error response JSON shapes. Added 409 retry-safe note and tx validation info. Addresses T2a review: missing response formats in llms.txt. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: REQUEST CHANGES
Summary
The API request/response shapes are documented now, but the revised /llms.txt still includes a behavior note that does not match the actual indexer routes. Since this file is supposed to be machine-readable integration guidance, that remaining mismatch is still blocking.
Findings
- [high] The new note says
409 = already indexed (safe to retry), but none of the four indexer routes documented here actually return409. The current implementations return{ success: true },{ indexed }, or{ error }with other statuses, so an agent integrating against/llms.txtwould now be coded against a response contract the app does not implement.- File:
src/app/llms.txt/route.ts:54 - Suggestion: Remove the
409note, or update the underlying routes to actually return409for the already-indexed case before documenting it here.
- File:
Decision
Requesting changes because /llms.txt is now closer, but it still advertises response behavior that the app does not currently provide.
No indexer route returns 409. Replaced with accurate note that duplicate indexing is safe (upsert on tx_hash + log_index). Addresses T2a review on PR #658. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: APPROVE
Summary
The latest revision fixes the remaining /llms.txt contract mismatch: the API section now includes request and response shapes, and the behavior note no longer advertises a nonexistent 409 response. The tab deep-linking and copy-link pieces remain scoped and consistent with the ticket.
Findings
- No blocking findings.
Decision
Approving because the current revision addresses the prior review feedback and now matches the actual indexer behavior closely enough for the machine-readable integration contract required by Queue #643 / agent-os#321.
Summary
/llms.txtroute: Plain-text integration info — CLI commands, API endpoints, contract addresses, chain info, env vars. Cached 1 hour.https://plotlink.xyz/llms.txtto clipboard./agents?tab=buildauto-selects Build tab. Supportsregister,build,dashboard. UsesuseSearchParamswith Suspense boundary.Fixes #643
Tracks realproject7/agent-os#321
Test plan
/llms.txtreturns plain text with integration info/agents?tab=buildauto-selects Build tab/agents?tab=dashboardauto-selects Dashboard tab🤖 Generated with Claude Code