Skip to content

TINYDOC-3594 - Scope LLM file generation to the latest version's attachments - #4338

Merged
kemister85 merged 1 commit into
mainfrom
TINYDOC-3594-scope-llm-output-to-latest
Sep 1, 2026
Merged

TINYDOC-3594 - Scope LLM file generation to the latest version's attachments#4338
kemister85 merged 1 commit into
mainfrom
TINYDOC-3594-scope-llm-output-to-latest

Conversation

@kemister85

@kemister85 kemister85 commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Ticket: TINYDOC-3594 — follows TINYDOC-3592.

Problem: the generator found its output directory by searching the build for _attachments. A production build has one per version, so it wrote the version 8 corpus into version 6 and 7 as well — publishing four URLs carrying the wrong version's content, and leaving copy-llms-files.sh with six candidates instead of two, so /docs/llms.txt was set by find order.

Seen in the staging deploy, run 33481753889:

✓ Wrote build/site/tinymce/6/_attachments/llms.txt
✓ Wrote build/site/tinymce/7/_attachments/llms.txt
✓ Wrote build/site/tinymce/latest/_attachments/llms.txt

Fix: derive the path from BASE_URL rather than searching for it. parseSitemap() keeps only the URLs beneath BASE_URL, so the generator emits one version's corpus and must write to one version's directory. Falls back to the build root with a warning if that directory is absent.

Impact: staging only. Generation has never run against production, so no incorrect URL was published there.

Testing: full multi-version build via the production playbook — 1,582 pages, four versions, three _attachments directories.

@kemister85
kemister85 requested a review from a team as a code owner September 1, 2026 07:37
@kemister85
kemister85 requested review from a team, MichaelFromin, ShiridiGandham, TheSpyder, ltrouton, metricjs and soritaheng and removed request for a team and MichaelFromin September 1, 2026 07:37
@kemister85
kemister85 merged commit 0e901bc into main Sep 1, 2026
4 checks passed
@kemister85
kemister85 deleted the TINYDOC-3594-scope-llm-output-to-latest branch September 1, 2026 07:53
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.

3 participants