Skip to content

fix(public): point desk, pricing and specimen canonicals, og:url and JSON-LD at the clean non-307 URLs - #61

Closed
nish3451 wants to merge 2 commits into
mainfrom
fix/clean-canonical-urls-remaining-pages
Closed

fix(public): point desk, pricing and specimen canonicals, og:url and JSON-LD at the clean non-307 URLs#61
nish3451 wants to merge 2 commits into
mainfrom
fix/clean-canonical-urls-remaining-pages

Conversation

@nish3451

@nish3451 nish3451 commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

Finding

Live walk 2026-08-10: the worker 307-redirects every .html form to its
extensionless twin, yet three of the five public pages still declare their
canonical, og:url and JSON-LD WebPage @id/url in the redirecting
.html spelling:

  • /agents -> canonical https://tinystudio.io/agents.html (307 -> /agents)
  • /pricing -> canonical https://tinystudio.io/pricing.html (307 -> /pricing)
  • /specimen -> canonical https://tinystudio.io/specimen.html (307 -> /specimen)

PR #56 (fix/audit-canonical-clean-url) migrated /audit and its guard
explicitly left these three pages "in their .html form here until they are
migrated the same way" — this PR completes that migration. No open PR owns
these pages.

Change

  • public/agents.html, public/pricing.html, public/specimen.html:
    canonical, og:url and JSON-LD WebPage @id/url now name the clean
    URL that serves 200.
  • scripts/check-site.mjs: the social-share, structured-data and canonical
    guard lists now assert the clean URL for all three pages, so the
    redirecting spelling cannot return. (Audit page entries left untouched;
    they flip with fix(public): point appraisal-page canonicals and JSON-LD @ids at the clean /audit URL #56.)

Verify

  • npm run check — "TinyStudio.io checks passed."
  • npm test — 6/7/53/16 suites green.
  • npm run check:render-blocking — all six pages PASS under production CSP.
  • Live confirmation of the redirect shape: /agents.html, /pricing.html,
    /specimen.html each return 307 -> clean twin.

Summary by CodeRabbit

  • SEO Improvements
    • Updated Agents, Pricing, and Specimen pages to use clean URLs without .html.
    • Aligned canonical links, social sharing URLs, and structured data with the updated page addresses.
  • Quality Assurance
    • Updated site checks to verify the new clean URL format.

…JSON-LD at the clean non-307 URLs

The worker 307-redirects every .html form to its extensionless twin, so a
canonical that names the .html address names a redirecting URL. The audit
page was already migrated to /audit (PR #56); the desk, pricing and
specimen pages are migrated here the same way, and the check-site guards
now assert the clean URL for all three.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: b5ddd7ca-ad90-4b35-9c58-8bc9989eeba3

📥 Commits

Reviewing files that changed from the base of the PR and between ac05bec and 78b7311.

📒 Files selected for processing (4)
  • public/agents.html
  • public/pricing.html
  • public/specimen.html
  • scripts/check-site.mjs

📝 Walkthrough

Walkthrough

The change replaces .html URLs with extensionless URLs in agents, pricing, and specimen page metadata. Site validation checks now expect the same extensionless URLs for social-share metadata, structured data, and canonical links.

Changes

Extensionless page URLs

Layer / File(s) Summary
Public page metadata
public/agents.html, public/pricing.html, public/specimen.html
Canonical, Open Graph, and JSON-LD URLs now use /agents, /pricing, and /specimen.
Site URL validation
scripts/check-site.mjs
Social-share, structured-data, and canonical URL checks now expect the extensionless paths.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change to canonical, og:url, and JSON-LD URLs for the desk, pricing, and specimen pages.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/clean-canonical-urls-remaining-pages

Comment @coderabbitai help to get the list of available commands.

@nish3451 nish3451 closed this Aug 10, 2026
@nish3451 nish3451 reopened this Aug 10, 2026

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@nish3451

Copy link
Copy Markdown
Collaborator Author

Closing as the conflict-locked stale duplicate in the duplicate open-PR reconciliation (docs/evidence/duplicate-open-pr-clusters-2026-08-11): #95 re-lands the identical agents/pricing/specimen canonical + og:url + JSON-LD edits on the fresh base, while this branch's check-site.mjs guard still expects the redirecting /audit.html form main has since moved (PR #56), leaving it mergeStateStatus DIRTY. Verified: the per-page HTML edits are byte-identical between #61 and #95; #95 passes npm run check on its own tree. #95 is the superior delivery path.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant