Repository navigation
site: remove leftover coming-soon curl tab styles and copy - #220
Conversation
Signed-off-by: Christopher <131207452+PetrefiedThunder@users.noreply.github.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes.
|
PetrefiedThunder
left a comment
There was a problem hiding this comment.
Delta review at 9e1d72d. 3 files, site only.
- Removes the unused .hero-v2-tab-disabled CSS and its contrast-test row. No markup on the PR head uses that class any more (git grep finds 0 hits), and test_home_key.mjs still asserts the coming-soon span is gone.
- llms-full.txt: the new line "The curl invoke path is not published yet." matches site/index.html, where the curl tab only issues a demo key and invoke runs through the Python demo. No new IDs, hosts, ports or numbers. No em dashes. No links added or changed.
- No conflict with #200 site move. Main has since moved only by #48 (integration packages, no site/ files), so the PR is BEHIND but needs only a clean merge of main.
- CI green (only matrix license jobs skipped), 0 unresolved threads.
REVIEW: PASS codex 9e1d72d
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Cleanup for the site's "coming soon" curl tab and the example receipt id.
What I found: origin/main's site/index.html already has no "coming soon" curl tab and no
rcpt_7Kq2mX9. #200 replaced the disabled tab with a working curl row, and site/test_home_key.mjs already guards against both coming back. The live site (agentmiddleware.dev) still shows both, so production is stale and needs a site redeploy.Changes:
.hero-v2-tab-disabledrules, which only styled the coming-soon badge.Receipt id:
git log -S rcpt_7Kq2mX9finds no occurrence anywhere in repo history, so there was nothing to replace in the repo. I did no live dispatch and invented no id. The example is dropped: the homepage shows no receipt id, and the existing test asserts that.Tests:
node --test site/test_*.mjs: 42/42 pass.python3 build_site.py: exit 0, and the built dist/ has no coming-soon, disabled-tab or rcpt_7Kq2mX9 strings.pytest site/test_verify_receipt.py: 4 passed.Note
Low Risk
Documentation, dead CSS, and test cleanup only; no runtime or API behavior changes.
Overview
Removes leftover homepage “coming soon” curl tab artifacts now that the hero shows a working curl row instead of a disabled tab.
site/styles.cssdeletes the unused.hero-v2-tab-disabledrules (muted tab + badge styling).site/test_home_contrast.mjsdrops the contrast assertion that targeted that selector.site/llms-full.txtupdates machine-facing copy: the curl invoke path is described as not published yet, aligned withllms.txt, instead of referring to a “Coming soon” tab.Reviewed by Cursor Bugbot for commit 9e1d72d. Bugbot is set up for automated code reviews on this repo. Configure here.