Skip to content

Improve build times for entire corpus - #348

Merged
Blargian merged 5 commits into
mainfrom
dummy-test-change
Sep 9, 2026
Merged

Improve build times for entire corpus#348
Blargian merged 5 commits into
mainfrom
dummy-test-change

Conversation

@Blargian

@Blargian Blargian commented Sep 9, 2026

Copy link
Copy Markdown
Member

This changes the documentation build architecture so English and each requested translation compile in isolated, sequential Astro processes before being merged into one Vercel artifact. This bounds peak memory, retains a single deployment, keeps ordinary previews scoped and fast, and still allows production to include every locale.

The build now:

  • compiles only the active translated collection in each locale process;
  • generates missing-translation fallbacks from the English process;
  • emits one canonical English .md agent representation per page and no duplicate .mdx or localized agent surfaces;
  • parses and indexes sidebar navigation once per locale;
  • uses namespaced locale assets and stable incremental-build cache keys; and
  • reports preview deployment state and selected components through a single sticky pull-request comment without allowing comment failures to interrupt Vercel polling.

The full production-shaped build completed in 18 minutes 44 seconds. Type checking, combined English and Spanish output generation, incremental cache restoration, workflow YAML validation, and shell syntax validation also passed.

Changelog category (leave one):

  • Build/Testing/Packaging Improvement

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

Reduce documentation build time and peak memory by compiling localized content in isolated sequential Astro processes and merging the results into one deployment.

@cursor cursor 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.

Stale Bugbot comment from a previous run.

Comment thread clickstack/architecture.mdx Outdated
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

Nimbus documentation preview (trusted-branch, English only): https://clickhouse-docs-1qqugsof0-clickhouse.vercel.app

@Blargian Blargian added the docs-translations-all Include all translations in the Vercel docs preview label Sep 9, 2026
@Blargian Blargian changed the title Test Improve build times for entire corpus Sep 9, 2026

@cursor cursor 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.

Stale Bugbot comment from a previous run.

Comment thread src/pages/[locale]/index.astro
@Blargian Blargian removed the docs-translations-all Include all translations in the Vercel docs preview label Sep 9, 2026

@cursor cursor 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.

Cursor Bugbot has reviewed your changes using high effort and found 2 potential issues.

There are 3 total unresolved issues (including 1 from previous review).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 10f3e7b. Configure here.

Comment thread .github/workflows/site-preview.yml Outdated
Comment thread .github/workflows/site-preview.yml
@Blargian
Blargian merged commit a2566b9 into main Sep 9, 2026
2 checks passed
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