Skip to content

chore(i18n): refresh native locales - #140874

Merged
openclaw-mantis[bot] merged 1 commit into
mainfrom
automation/native-app-locale-refresh
Sep 7, 2026
Merged

chore(i18n): refresh native locales#140874
openclaw-mantis[bot] merged 1 commit into
mainfrom
automation/native-app-locale-refresh

Conversation

@openclaw-mantis

@openclaw-mantis openclaw-mantis Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

What Problem This Solves

Keeps generated native app locales synchronized without bypassing protected-branch checks.

Why This Change Was Made

The Native App Locale Refresh workflow generated this update from 331d6b7f686171fef54e3fe229474ef2095586be and published it through a reviewable automation branch.

User Impact

No direct user-facing change beyond refreshed translations.

Evidence

@clawsweeper

clawsweeper Bot commented Sep 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

🦞👀
ClawSweeper picked this up.

Pull request received. I will update this pull request when review starts.

ClawSweeper review complete

ClawSweeper finished reviewing this revision. The review result is being finalized.

View the workflow run.

@openclaw-mantis
openclaw-mantis Bot enabled auto-merge (squash) September 7, 2026 05:33
@clawsweeper clawsweeper Bot added P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels Sep 7, 2026
@clawsweeper

clawsweeper Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Codex review: needs maintainer review before merge. Reviewed September 7, 2026, 1:37 AM ET / 05:37 UTC.

ClawSweeper review

What this changes

Refreshes native translation artifacts and the macOS string catalog to match the current Connection window and diagnostic labels.

Merge readiness

✅ Ready for maintainer review

Keep open under the generated-locale publisher policy. Current main still lacks these catalog entries, and inspection found no actionable patch defect.

Likely related people: Peter Steinberger is a routing candidate based on recent macOS and localization work.

Priority: P3
Reviewed head: cc75788a533abccd0f3e86a60efc1eb4a82f02c9

Review scores

Measure Result What it means
Overall readiness 🐚 platinum hermit (4/6) A focused generated refresh with consistent source mappings and no actionable correctness or security findings.
Proof confidence 🌊 off-meta tidepool Not applicable: This repository-managed bot refresh is exempt from contributor runtime proof; catalog consistency and the reported native-i18n check support validation, without claiming native rendering proof.
Patch quality 🐚 platinum hermit (4/6) No actionable review findings were identified.

Verification

Check Result Evidence
Real behavior Not applicable Not applicable: This repository-managed bot refresh is exempt from contributor runtime proof; catalog consistency and the reported native-i18n check support validation, without claiming native rendering proof.
Evidence reviewed 7 items Canonical publisher ownership: The supplied GitHub snapshot identifies openclaw-mantis[bot] and automation/native-app-locale-refresh targeting main. The repository workflow publishes that exact branch after generating and validating locale artifacts; the review policy explicitly keeps these publisher-managed PRs open.
Complete semantic delta inspected: Read-only JSON comparisons of the pinned base and head found 24 added and 28 removed macOS catalog keys, with no modified retained entries. Each of the 21 locale artifacts adds 23 translations and removes 28, without changing retained translations or metadata. Every added translation matches its exact source-key catalog value; inventory membership and checked format tokens had no discrepancies.
Refresh remains useful on current main: Comparing the catalog at fetched main with the introduced additions showed all 24 added keys remain absent there. The source inventory maps those additions to existing macOS interface call sites, while none of the removed keys remains in the macOS inventory.
Findings None None.
Security None None.

How this fits together

OpenClaw’s native localization pipeline combines extracted interface strings with locale translations to generate Apple string catalogs. The macOS app uses those catalogs to display translated interface text.

flowchart LR
  A[Native interface strings] --> B[Source inventory]
  B --> C[Locale translations]
  C --> D[Catalog generator]
  B --> D
  D --> E[macOS string catalog]
  E --> F[Localized native interface]
Loading

Before merge

None.

Agent review details

Security

None.

PR surface

Other -109. Total -109 across 22 files.

View PR surface stats
Area Files Added Removed Net
Source 0 0 0 0
Tests 0 0 0 0
Docs 0 0 0 0
Config 0 0 0 0
Generated 0 0 0 0
Other 22 507 616 -109
Total 22 507 616 -109

Review metrics

None.

Technical review

Best possible solution:

Keep native translations synchronized through the existing inventory and catalog generator, with freshness and landing owned by the generated-PR publisher.

Do we have a high-confidence way to reproduce the issue?

Not applicable: this is a generated translation refresh rather than a reported runtime defect; source inspection confirms the catalog gaps.

Is this the best way to solve the issue?

Yes. Updating the existing locale artifacts and derived catalog is the established, narrow solution, and retained translations remain unchanged.

AGENTS.md: found and applied where relevant.

Codex review notes: model internal, reasoning medium; reviewed against b7848ca6b197.

Labels

Label changes:

  • add P3: This is routine generated translation maintenance with no introduced runtime or configuration changes.
  • add rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🌊 off-meta tidepool and patch quality is 🐚 platinum hermit.
  • add status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Not applicable: This repository-managed bot refresh is exempt from contributor runtime proof; catalog consistency and the reported native-i18n check support validation, without claiming native rendering proof.

Label justifications:

  • P3: This is routine generated translation maintenance with no introduced runtime or configuration changes.
  • rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🌊 off-meta tidepool and patch quality is 🐚 platinum hermit.
  • status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Not applicable: This repository-managed bot refresh is exempt from contributor runtime proof; catalog consistency and the reported native-i18n check support validation, without claiming native rendering proof.

Evidence

What I checked:

  • Canonical publisher ownership: The supplied GitHub snapshot identifies openclaw-mantis[bot] and automation/native-app-locale-refresh targeting main. The repository workflow publishes that exact branch after generating and validating locale artifacts; the review policy explicitly keeps these publisher-managed PRs open. (.github/workflows/native-app-locale-refresh.yml:244, cc75788a533a)
  • Complete semantic delta inspected: Read-only JSON comparisons of the pinned base and head found 24 added and 28 removed macOS catalog keys, with no modified retained entries. Each of the 21 locale artifacts adds 23 translations and removes 28, without changing retained translations or metadata. Every added translation matches its exact source-key catalog value; inventory membership and checked format tokens had no discrepancies. (apps/macos/Sources/OpenClaw/Resources/Localizable.xcstrings, cc75788a533a)
  • Refresh remains useful on current main: Comparing the catalog at fetched main with the introduced additions showed all 24 added keys remain absent there. The source inventory maps those additions to existing macOS interface call sites, while none of the removed keys remains in the macOS inventory. (apps/macos/Sources/OpenClaw/Resources/Localizable.xcstrings, b7848ca6b197)
  • Existing interface contract: The Connection window already uses the updated remote-auth help, remote test status, Nearby Gateways heading, and Dashboard settings button. This PR supplies their translations without introducing those behaviors or changing authorization. (apps/macos/Sources/OpenClaw/ConnectionSettingsView.swift:317, cc75788a533a)
  • Canonical generation and validation: The generator derives Apple catalogs from the source inventory and locale artifacts, preserving existing translated entries. Locale validation rejects unknown or missing identifiers, empty translations, and structural-token mismatches. (scripts/native-app-i18n.ts:1487, cc75788a533a)
  • Area-history routing: Current-main path history lists Peter Steinberger’s recent Connection-window rewrite and earlier move of settings into the Dashboard. Localization history also records his restoration of the canonical refresh pipeline. These support routing, without establishing source-line introduction. (apps/macos/Sources/OpenClaw/ConnectionSettingsView.swift, adf581865676)

Likely related people:

  • Peter Steinberger: Suggested for follow-up; no historical authorship or introduction is verified. (role: unverified routing candidate; confidence: low)

Rating scale

Score Internal tier Crab rank Meaning
6/6 S 🦀 challenger crab Exceptional readiness
5/6 A 🦞 diamond lobster Very strong readiness
4/6 B 🐚 platinum hermit Good normal PR; ordinary maintainer review
3/6 C 🦐 gold shrimp Useful, but confidence is limited
2/6 D 🦪 silver shellfish Proof or implementation needs work
1/6 F 🧂 unranked krab Not merge-ready
N/A NA 🌊 off-meta tidepool Rating does not apply

Overall follows the weaker of proof and patch quality.
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics.

Workflow

  • ClawSweeper keeps one durable marker-backed review comment per issue or PR.
  • Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
  • A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
  • PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
  • Maintainers can also comment @clawsweeper review to request a fresh review only.
  • Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
  • Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
  • Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

@openclaw-mantis
openclaw-mantis Bot merged commit f6494e3 into main Sep 7, 2026
128 checks passed
@openclaw-mantis
openclaw-mantis Bot deleted the automation/native-app-locale-refresh branch September 7, 2026 05:50
github-actions Bot added a commit to Desicool/openclaw that referenced this pull request Sep 8, 2026
@openclaw-mantis @github-actions
chore(i18n): refresh native locales (openclaw#140874)
0414de8 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

app: macos App: macos P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. size: XL status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

0 participants