Skip to content

chore: promote develop -> stage (docs SEO fixes) - #29

Merged
evereq merged 8 commits into
stagefrom
develop
Aug 30, 2026
Merged

chore: promote develop -> stage (docs SEO fixes)#29
evereq merged 8 commits into
stagefrom
develop

Conversation

@evereq

@evereq evereq commented Aug 30, 2026

Copy link
Copy Markdown
Member

Promotion only.

🤖 Generated with Claude Code

@evereq
evereq merged commit 7ab7996 into stage Aug 30, 2026
1 of 3 checks passed
@greptile-apps

greptile-apps Bot commented Aug 30, 2026

Copy link
Copy Markdown

Greptile Summary

The change limits the deployed documentation site to English, updates web-server redirect handling, and refreshes CI configuration. The Docusaurus configuration contains uncommented Yarn output at line 56, causing yarn build --locale en to fail before static documentation is generated. Commenting or removing that text restores the build path; the maintenance note should also identify the --locale en flag used by the container build.

Confidence Score: 4/5

Not safe to merge until the invalid configuration line is corrected, because the documentation build cannot complete.

The failing build was reproduced with the same locale command used by the container, and an isolated corrected configuration completed the build and generated static files.

Files Needing Attention: website/docusaurus.config.ts requires correction at line 56 and clarification of the locale-restoration instructions near line 60.

T-Rex T-Rex Logs

What T-Rex did

  • T-Rex produced proofs for two posted P1 findings, and attached a Validation script, a stack trace, and an After output log to support the findings.
  • T-Rex validated locale-config behavior with a focused non-mutating script; before output the locale build failed with a parse error, and after output the locale build succeeded.

View all artifacts

T-Rex Ran code and verified through T-Rex

Comments Outside Diff (1)

  1. General comment

    P1 Uncommented Yarn output in locale maintenance block prevents all site builds

    • Bug
      • At website/docusaurus.config.ts:56, info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. ... is unprefixed text rather than a // comment. Running the Dockerfile's build command (yarn build --locale en) fails before site generation with a parse error at that exact location. Separately, line 60 says drop the flag, so it does not identify the --locale en flag used by website/Dockerfile.everk8s:32; locale-restoration instructions are incomplete.
    • Cause
      • Yarn command output was inserted into a TypeScript configuration comment block without comment syntax, and the intended command-line flag was omitted from the restoration text.
    • Fix
      • Prefix the line-56 Yarn output with // or remove it. Replace the incomplete wording at line 60 with explicit guidance such as remove the --locale en flag in Dockerfile.everk8s, while retaining the actual container command context.

    T-Rex Ran code and verified through T-Rex

Reviews (1): Last reviewed commit: "Merge pull request #28 from ever-co/fix/..." | Re-trigger Greptile

//
// Docusaurus renders a language dropdown for EVERY locale listed here, but the Docker build runs
// yarn run v1.22.22
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. and so only emits English. The other twelve were linked from every page

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Uncommented Yarn output breaks configuration

info Visit https://yarnpkg.com/... is not prefixed with //, so it is parsed as TypeScript configuration rather than maintenance guidance. Consequently, the container's yarn build --locale en command fails with a parse error before the documentation site can be generated. Remove the pasted output or comment it out. Also update the nearby restoration note to explicitly name the --locale en flag used in Dockerfile.everk8s.

Artifacts

Validation script

  • This non-mutating script ran the exact repository locale build and a corrected isolated-copy comparison, showing the malformed line and documentation flag behavior.

Before output

  • Running `yarn build --locale en` in `/home/user/repo/website` exited 1 with Docusaurus's parse error at `docusaurus.config.ts:56:5`, proving the text is code rather than a comment.

After output

  • The same `yarn build --locale en` command in an isolated corrected copy exited 0 and generated static files, proving syntactically commented guidance restores the build path.

View artifacts

T-Rex Ran code and verified through T-Rex

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