Skip to content

[3.x] Revert generated robots.txt and llms.txt (stacked PR 6 and PR 7) - #2580

Merged
emmadesilva merged 5 commits into
v3/non-html-pagesfrom
v3/revert-non-html-pages-robots
Aug 5, 2026
Merged

[3.x] Revert generated robots.txt and llms.txt (stacked PR 6 and PR 7)#2580
emmadesilva merged 5 commits into
v3/non-html-pagesfrom
v3/revert-non-html-pages-robots

Conversation

@emmadesilva

Copy link
Copy Markdown
Member

Summary

Reverts both #2531 (generated robots.txt) and #2534 (generated llms.txt). Both were overengineered; they're being redesigned from scratch, starting from a clean baseline instead of iterating on top of the current implementation.

llms.txt is included in the same PR rather than a separate one because it was explicitly built "wired like robots.txt" — reverting only one would have left the epic and several docs full of references to a sibling feature that no longer exists.

  • Removes RobotsTxtGenerator, LlmsTxtGenerator, their Features::hasRobotsTxt()/hasLlmsTxt() gates, HydeCoreExtension wiring, and the hyde.robots/hyde.llms config blocks (kept in lockstep between config/hyde.php and the framework config stub).
  • Deletes the feature-specific test classes, and trims the now-dead hyde.robots.enabled/hyde.llms.enabled toggles out of collection/manifest/realtime-compiler tests that only needed them to keep exact-count assertions accurate.
  • Rolls back the public docs (customization.md, build-tasks.md, in-memory-pages.md) and UPGRADE.md/HYDEPHP_V3_PLANNING.md. Illustrative uses of robots.txt as an example non-HTML filename (InMemoryPage output inference, showInSitemap()) are untouched — those describe PR 1/PR 4 behavior, not the reverted generators.
  • Updates EPIC_NON_HTML_PAGES.md: PR 6 and PR 7 are marked reverted rather than implemented, their implementation notes (which described deleted code as existing) replaced with a short record of what was tried and why it was cut, and downstream references that claimed the D5 user-override paths were verified for both pages are corrected.

Test plan

  • vendor/bin/pest --testsuite FeatureFramework,UnitFramework,FeatureHyde,"Realtime Compiler" — ran before and after the revert; identical FAILED output both times (only the known pre-existing FeaturedImageUnitTest PHP 8.5 failure)
  • php monorepo/HydeStan/run.php — no errors
  • Verified config/hyde.php and packages/framework/config/hyde.php stay in lockstep (HydeConfigFilesAreMatchingTest)

Removes RobotsTxtGenerator, LlmsTxtGenerator, their Features::hasRobotsTxt()/
hasLlmsTxt() gates, HydeCoreExtension wiring, and the hyde.robots/hyde.llms
config blocks. Both features (PRs #2531 and #2534) were overengineered; they
are being redesigned from scratch on this branch.
Deletes the feature-specific generator/page test classes and drops the now
dead hyde.robots/hyde.llms config toggles from collection and manifest tests
that only needed them to keep exact-count assertions accurate.
Hyde no longer generates these files automatically, so drop the generator
customization example, the config reference, and the feature mentions in
the generated-discovery-files and build-tasks docs. Illustrative uses of
robots.txt as an example non-HTML filename (InMemoryPage output inference,
showInSitemap()) are unaffected and stay.
Drops the upgrade steps and checklist items for features that no longer
exist on this branch, and renumbers the subsequent upgrade steps. Removes
the corresponding HYDEPHP_V3_PLANNING.md changelog entries.
Marks PR 6 and PR 7 as reverted rather than implemented, replaces their
implementation notes (which described deleted code as existing) with a
short record of what was reverted and why, and fixes downstream references
that claimed the D5 user-override paths were verified for both pages.
@emmadesilvaemmadesilva changed the title [3.x] Revert generated robots.txt and llms.txt (PR 6 and PR 7)[3.x] Revert generated robots.txt and llms.txt (stacked PR 6 and PR 7)Aug 5, 2026
@codecov

codecovBot commented Aug 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (0899467) to head (d3ddfaf).

Additional details and impacted files
@@ Coverage Diff @@## v3/non-html-pages #2580 +/- ##
=====================================================
Coverage 100.00% 100.00% + Complexity 1824 1792 -32 
=====================================================
Files 184 182 -2 Lines 4539 4453 -86 =====================================================
- Hits 4539 4453 -86 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@emmadesilva
emmadesilva merged commit 3dd5e3b into v3/non-html-pagesAug 5, 2026
8 checks passed
@emmadesilva
emmadesilva deleted the v3/revert-non-html-pages-robots branch August 5, 2026 22:07
Sign up for freeto 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

@emmadesilva