Skip to content

refactor(seo): delegate Agent discovery to nuxt-ai-ready - #650

Merged
harlan-zw merged 1 commit into
mainfrom
fix/module-agent-discovery
Oct 13, 2026
Merged

harlan-zw merged 1 commit into
mainfrom
fix/module-agent-discovery

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

Related to #639.

❓ Type of change

  • 📖 Documentation
  • 🐞 Bug fix
  • 👌 Enhancement
  • ✨ New feature
  • 🧹 Chore
  • ⚠️ Breaking change

📚 Description

The site duplicated discovery routing, sitemap parsing, caching, and formatting to work around gaps in nuxt-ai-ready.
Version 3.3.1 now handles those paths, so this change removes the duplicate implementation and configures the module instead.
It preserves setup links, named Skills and demo files, Markdown links, and the retired sitemap exclusion.

Site policy feeds bounded discovery files in Nuxt AI Ready. The duplicate site renderer is removed.

🤖 AI disclosure: Harlan Agent Kit modified this description. My AI open-source policy.

@harlan-zw harlan-zw added the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 13, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

🤖 Harlan Agent Kit checked the built Cloudflare Worker against migrated local D1.

All four Agent discovery HTTP tests passed.
An independent review crawled every root and child link: all 98 returned direct HTTP 200.
The three generated files stayed within 80,000 characters, with a measured maximum of 6,730.
HEAD returned empty plain-text responses. POST returned 405, and an unknown discovery file returned 404.
Repeated root and child reads were byte-identical after crawling their page links.
Retired sitemap references and full-content links were absent. The old full-content URL retained its 302 redirect.

This verifies the released module inside the site's real Worker build.
The production deployment and large live sitemap data remain to be checked after merge.

@harlan-github-agent harlan-github-agent Bot added harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. harlan-agent-running An Agent holds a Task on this issue or pull request right now. and removed harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. labels Oct 13, 2026
@harlan-github-agent

harlan-github-agent Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

🤖 MERGED

Harlan Agent Kit posted this automated review. It is not Harlan's personal review or approval. AI open source policy. Last updated: 2026-10-13 18:27 UTC.

GitHub merged this pull request.

No material findings were recorded.

Review completed. No finding above 80/100.

@harlan-zw
harlan-zw merged commit 732b02c into main Oct 13, 2026
8 checks passed
@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. and removed harlan-agent-running An Agent holds a Task on this issue or pull request right now. labels Oct 13, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

🤖 Harlan Agent Kit verified the merged migration on live skilld.dev.

A paced crawl resolved all 194 discovery links with direct HTTP 200.
The root and two child files measured 5,489, 29,370, and 14,631 characters.
HEAD, POST rejection, missing-file 404, and the old full-content redirect passed.
Retired sitemap and full-content references were absent from discovery.

Cloudflare serves version 34abb1d3-efb5-47ca-9a0f-c24c8b9b758e, deployed at 2026-10-13T16:53:51Z.
CI and the service review gate passed for head 00dc5f8, with no findings.
The service's mergeability gate stayed pending because the PR merged during review.

@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. label Oct 13, 2026
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