TINYDOC-3616: Remove the committed LLM attachments and the dead generator. - #4364
Merged
Merged
Conversation
kemister85
requested review from
MitchC1999,
ShiridiGandham,
metricjs and
tiny-ben-tran
September 28, 2026 01:21
metricjs
approved these changes
Sep 28, 2026
tiny-ben-tran
approved these changes
Sep 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ticket: TINYDOC-3616
Site: Staging branch
Changes:
modules/ROOT/attachments/llms.txtandllms-full.txt. The deploy onmainnow generates both files on every build (TINYDOC-3613), and the committed copies had already drifted from what the site serves.-scripts/generate-llm-files.js,generate-llm-files.shandREADME-llm-files.md, and the twogenerate-llm-files*entries inpackage.jsonthat pointed at them. The generator and its documentation live onmain.ai-coding-agents.adoc:llms-full.txtnow contains the full content of every page, not an index of URLs, so its description in the table is updated to match.How to test locally:
Run these from your
tinymce-docscheckout. They do not depend on folder names.This branch alone (about a minute, plus the install):
git fetch origin && git switch hotfix/8/TINYDOC-3616 yarn install --frozen-lockfile npx antora ./antora-playbook-local.yml npx http-server build/site -p 4000ls build/site/tinymce/latest/_attachments/lists onlytinymceai-mcp-demoandurl-dialog-demo, with nollms*.txt.llms-full.txtas the full content of every page.git grep -nE "generate-llm-files|README-llm-files|attachments/llms"returns nothing.The deploy steps with this branch's content (about 5 minutes; stop the server first). This shows the build no longer needs the committed files:
llms-full.txtis about 3.9 MB.identical.The
ERRORlines during the Antora step come from the archivedtinymce/5content and appear onmaintoo. Clean up afterwards withcd "$DOCS" && git worktree remove --force ../docs-main-3616.These steps were run as written on fresh clones on 2026-09-28, and every check passed.
Pre-checks:
Review: