Skip to content

[auto] update translations - #519

Merged
NiveditJain merged 1 commit into
mainfrom
auto/translate-docs-20260716-0919
Jul 16, 2026
Merged

NiveditJain merged 1 commit into
mainfrom
auto/translate-docs-20260716-0919

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Summary

Automated translation update triggered by changes to English documentation sources.

  • Only changed pages were re-translated (content-hash cache)
  • All 14 languages across 3 tiers (parallel matrix)

@hermes-exosphere

Copy link
Copy Markdown
Contributor

Automated code review started - full review. Results will be posted here.

@hermes-exosphere

Copy link
Copy Markdown
Contributor

⏳ Automated review in progress... Phase 0–3 complete. No existing review threads found. Build: green (2064/2064 tests, lint clean, tsc clean). Posting detailed review next.

@hermes-exosphere

Copy link
Copy Markdown
Contributor

Automated Code Review

Executive Summary

This is an automated translation update PR triggered by changes to the English documentation source (README.md). It updates all 14 translated README files (ar, de, es, fr, he, hi, it, ja, ko, pt-br, ru, tr, vi, zh) to match the latest English source. The key content changes include: (1) replacing the inline <img> logo grid with a 2-row-by-6-column responsive HTML <table> layout, (2) adding detailed per-CLI blockquote descriptions for all 12 supported agent CLIs (expanding from the previous 6), and (3) updating product copy for consistency. Overall code quality is good -- the translation pipeline is working correctly, structural consistency across all 14 languages is maintained.


Change Architecture

graph TD
    A[English README.md source] -->|"content-hash cache detects change"| B[translate-docs CI workflow]
    B -->|"matrix: 14 languages x 1 file"| C[Translation Pipeline]
    C --> D[ar: Arabic]
    C --> E[zh: Chinese]
    C --> F[ja: Japanese]
    C --> G[ko: Korean]
    C --> H["es/fr/de/ru/hi/tr/vi/it/pt-br/he"]
    D -->|verified| I[PR #519]
    E -->|verified| I
    F -->|verified| I
    G -->|verified| I
    H -->|verified| I
    style A fill:#87CEEB
    style B fill:#90EE90
    style I fill:#FFD700
Loading

Legend: Source | Pipeline | PR Deliverable


Breaking Changes

No breaking changes detected. This is a documentation-only update.


Issues Found

No blocking issues found. All checks passed across the 14 translation files:

  1. Info -- Structural consistency verified: all 14 files have identical 2-row-by-6-column <table> layout with 20 logo references each and 12 CLI links. Blockquote count varies (9-13 lines) due to natural language length differences -- not an issue.

Logical / Bug Analysis

Reviewed all 14 files for:

  • No merge conflict markers
  • All referenced logo assets (assets/logos/*.svg, *.png) exist on disk
  • All URLs updated consistently (old URLs like developers.openai.com/codex to learn.chatgpt.com, cursor.com/docs/hooks to cursor.com)
  • --cli lists updated from 7 to 12 agent names
  • Policy count consistently "30" across all languages
  • New CLI descriptions for Hermes, OpenClaw, Factory, Devin, Antigravity, Goose present in all languages
  • Translation disclaimer header present in all files
  • Language bar (emoji flags + links) present in all files

No logical errors, no copy-paste mistakes, no broken links found.


Evidence -- Build & Test Results

Lint Output
$ bun run lint
5 warnings, 0 errors (all pre-existing <img> warnings unrelated to this PR)
TypeScript Check
$ bunx tsc --noEmit
(clean -- no errors)
Test Results
$ bun run test:run
Test Files  121 passed (121)
     Tests  2064 passed (2064)
Duration  54.47s

Issue Linkage

No linked issues -- this is an automated translation sync. The previous translation PR pattern (e.g., 89fa603 docs: update translations) indicates this is a routine CI-triggered update.


Human Review Feedback

No human review comments on this PR. One previous hermes-exosphere bot comment (id=4990321358) is informational only ("Automated code review started").


Suggestions

  1. Consider adding a summary comment -- The PR body mentions "All 14 languages across 3 tiers" but does not detail what content changed. Adding a brief summary of what changed in the English source (e.g., "12-CLI table layout, new CLI descriptions") would help reviewers understand the scope at a glance.

  2. Logo consistency check -- openclaw.svg, devin.svg, and antigravity.svg do not support dark/light variants while all other logos do. This is consistent with the English README source but worth noting.


Verdict

VERDICT: APPROVED


Automated code review · 2026-07-16 15:20:00 UTC

@hermes-exosphere hermes-exosphere left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Automated review: Approved. All 14 translation files are structurally consistent. 2064 tests pass, lint clean, tsc clean. No breaking changes.

@hermes-exosphere hermes-exosphere left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Automated review: Approved. ✅

@NiveditJain
NiveditJain merged commit 446484b into main Jul 16, 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.

2 participants