Audit and polish CommandLayer public site#394
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9d73405fbf
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <meta property="og:type" content="website" /> | ||
| <meta property="og:url" content="https://www.commandlayer.org/" /> | ||
| <meta property="og:image" content="https://www.commandlayer.org/commandlayer-logo.png" /> | ||
| <meta property="og:image" content="https://www.commandlayer.org/og-image.png" /> |
There was a problem hiding this comment.
Add the referenced social preview image
When social crawlers render any of the newly tagged pages, this URL resolves to no deployed asset: a repo-wide search and the commit tree show no public/og-image.png. Consequently both the Open Graph and Twitter large-image cards point at a 404, defeating the preview metadata added here; add the asset under public/ or reference an existing image.
Useful? React with 👍 / 👎.
| <div class="container nav-inner"> | ||
| <a href="/" class="brand"><img src="/commandlayer-logo.png" alt="CommandLayer" /><span>CommandLayer</span></a> | ||
| <ul class="nav-links"><li><a href="/">Home</a></li><li><a href="/protocol.html">Protocol</a></li><li><a href="/capabilities.html">Capabilities</a></li><li><a href="/receipts.html">Receipts</a></li><li><a href="/proof.html">Live Proof</a></li><li><a href="/sdk-records.html">SDK</a></li><li class="nav-drop"><a href="/docs.html" class="active" aria-haspopup="true">Docs ▾</a><div class="nav-drop-menu"><a href="/docs.html">Docs Home</a><a href="/docs/wrap-your-agent.html">Wrap Your Agent</a><a href="/runtime.html">Runtime</a><a href="/mcp.html">MCP Bridge</a><a href="/schemas.html">Schemas</a><a href="/api.html">API Reference</a><a href="/integrations.html">Integrations</a><a href="/verify.html">Verifier</a><a href="/playground.html">Playground</a><a href="/webhook-auto-verify.html">Webhook Demo</a></div></li><li><a href="/claim.html">Claim</a></li><li><a href="https://github.com/commandlayer" target="_blank" rel="noopener">GitHub</a></li></ul> | ||
| <ul class="nav-links"><li><a href="/">Home</a></li><li><a href="/protocol.html">Protocol</a></li><li><a href="/capabilities.html">Capabilities</a></li><li><a href="/receipts.html">Receipts</a></li><li><a href="/proof.html">Live Proof</a></li><li><a href="/verify.html">Verifier</a></li><li><a href="/sdk-records.html">SDK</a></li><li class="nav-drop"><a href="/docs.html" class="active" aria-haspopup="true">Docs ▾</a><div class="nav-drop-menu"><a href="/docs.html">Docs Home</a><a href="/docs/wrap-your-agent.html">Wrap Your Agent</a><a href="/runtime.html">Runtime</a><a href="/mcp.html">MCP Bridge</a><a href="/schemas.html">Schemas</a><a href="/api.html">API Reference</a><a href="/integrations.html">Integrations</a><a href="/verify.html">Verifier</a><a href="/playground.html">Playground</a><a href="/webhook-auto-verify.html">Webhook Demo</a></div></li><li><a href="/claim.html">Claim</a></li><li><a href="https://github.com/commandlayer" target="_blank" rel="noopener">GitHub</a></li></ul> |
There was a problem hiding this comment.
Wrap the expanded navigation at tablet widths
On pages that rely on public/css/site.css, the navigation remains a non-wrapping flex row until the max-width: 900px breakpoint. Adding another top-level item makes the roughly 204px-wide logo, grid gaps, and ten links exceed the 90vw container across an additional band of tablet/small-desktop widths; because body also sets overflow-x:hidden, the trailing Claim/GitHub links are clipped rather than wrapped. Apply the responsive layout earlier or allow .nav-links to wrap.
Useful? React with 👍 / 👎.
| <section class="section stack-section" aria-labelledby="stack-heading"> | ||
| <div class="container"> | ||
| <p class="section-eyebrow">The verification stack</p> | ||
| <h2 class="section-h2" id="stack-heading">ERC-8004 proves who the agent is. CLAS proves what the agent did. CommandLayer verifies the receipt.</h2> |
There was a problem hiding this comment.
Describe ERC-8004 as discovery rather than proof
For namespaces created through the current Claim Beta flow, this headline overstates what ERC-8004 establishes: claim.html says the flow only prepares discovery tracking and marks registration verified after onchain confirmation, while the cards immediately below label ENS as identity and ERC-8004 as discovery. Saying ERC-8004 unconditionally “proves who the agent is” can therefore present an unregistered, merely tracked agent as identity-proven; align the headline with the discovery/registry semantics used elsewhere on the site.
Useful? React with 👍 / 👎.
Motivation
Description
public/index.htmlto change the<title>, homepage meta description, Open Graph and Twitter metadata, add a compact hero (Verifiable proof for AI agent actions.), new hero subheadline, and prioritized CTAs (/claim.html,/verify.html,/proof.html).SoftwareApplicationJSON-LD structured data plus small CSS for the stack layout.og:*, andtwitter:*tags to the requested key pages (index,claim,verify,proof,receipts,protocol,capabilities,docs,sdk-records,runtime,mcp,schemas,integrations) and updated navigation to surface the Verifier link while preserving existing routes and demos.public/to remove internal phrases and replace with customer-safe language (kept Claim Beta truth about prepared/pinned/tracking vs confirmed registration and ENSIP-25 verification); preserved Claim UI IDs/classes and JS behavior.public/robots.txtandpublic/sitemap.xmlcontaining only the approved public pages and avoided admin/API/private or preview URLs.Testing
npm test: 165 tests passed and 4 tests failed; the failing tests relate to ERC-8004 registration/pinning and test fixtures (agent-registrations.test.js, twoapi-admin-pin-agent-cards.test.jscases, andclaim-activation-pipeline.test.js) and are unrelated to static public-site content changes.npm run check:links: passed; all local links and assets resolved across the site.public/(passed).sitemap.xmlparse and ran a simple SEO presence check showing requiredcanonical/OG/Twitter tags present on required pages (passed).https://www.commandlayer.org/claim.htmlfrom this environment because outbound HTTPS CONNECT returned403and DNS resolution was unavailable, so no deployment staleness root cause could be determined here.Codex Task