[i18n] Fix missing subscription.* translations #6610
Conversation
🎭 Playwright Test Results⏰ Completed at: 11/10/2025, 10:50:45 AM UTC 📈 Summary
📊 Test Reports by Browser
🎉 Click on the links above to view detailed test results for each browser configuration. |
🎨 Storybook Build Status✅ Build completed successfully! ⏰ Completed at: 11/10/2025, 10:38:31 AM UTC 🔗 Links🎨 Chromatic Visual Tests🎉 Your Storybook is ready for review! |
Bundle Size ReportSummary
Category Glance Per-category breakdownApp Entry Points — 3.25 MB (baseline 3.25 MB) • ⚪ 0 BMain entry bundles and manifests
Graph Workspace — 796 kB (baseline 796 kB) • ⚪ 0 BGraph editor runtime, canvas, workflow orchestration
Views & Navigation — 8.18 kB (baseline 8.18 kB) • ⚪ 0 BTop-level views, pages, and routed surfaces
Panels & Settings — 293 kB (baseline 293 kB) • ⚪ 0 BConfiguration panels, inspectors, and settings screens
UI Components — 12.6 kB (baseline 12.6 kB) • ⚪ 0 BReusable component library chunks
Data & Services — 10.4 kB (baseline 10.4 kB) • ⚪ 0 BStores, services, APIs, and repositories
Utilities & Hooks — 1.07 kB (baseline 1.07 kB) • ⚪ 0 BHelpers, composables, and utility bundles
Vendor & Third-Party — 5.32 MB (baseline 5.32 MB) • ⚪ 0 BExternal libraries and shared vendor chunks
Other — 2.55 MB (baseline 2.55 MB) • ⚪ 0 BBundles that do not match a named category
|
christian-byrne
added
the
claude-review
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
| git diff --staged --quiet || git commit -m "Update locales" | ||
| git push origin HEAD:${{ github.head_ref }} | ||
| if ! git diff --staged --quiet; then | ||
| git commit -m "Update locales" |
There was a problem hiding this comment.
[quality] high Priority
Issue: Potential error silencing with || true
Context: The removal of git stash pop || true removes error handling, but this is actually good as it was masking real failures
Suggestion: The new approach is better - explicit conditional commit logic is much clearer than git stash operations
| git checkout -B ${{ github.head_ref }} origin/${{ github.head_ref }} | ||
| # Apply the stashed changes if any | ||
| git stash pop || true | ||
| git checkout ${{ github.head_ref }} |
There was a problem hiding this comment.
[architecture] low Priority
Issue: Git workflow simplification is good but comments should explain the change
Context: Removing git stash/pop operations simplifies the workflow but the rationale could be better documented
Suggestion: The commit message explains it well, but adding a comment in the YAML would help future maintainers understand why this approach was chosen
There was a problem hiding this comment.
Comprehensive PR Review
This review is generated by Claude. It may not always be accurate, as with human reviewers. If you believe that any of the comments are invalid or incorrect, please state why for each. For others, please implement the changes in one way or another.
Review Summary
PR: [i18n] Fix missing subscription.* translations (#6610)
Impact: Minimal changes across 3 files - primarily workflow fixes and locale updates
Issue Distribution
- Critical: 0
- High: 1
- Medium: 0
- Low: 1
Category Breakdown
- Architecture: 1 issue
- Security: 0 issues
- Performance: 0 issues
- Code Quality: 1 issue
Key Findings
Architecture & Design
This PR appropriately addresses a specific i18n workflow issue where subscription translations were missing. The approach of creating trigger files and simplifying git operations in the workflow is sound architectural practice for CI/CD pipelines.
Workflow Improvements
The git workflow simplification removing stash/pop operations is a good improvement that eliminates potential failure points. The previous || true operator was masking failures that could result in lost translations.
Security Considerations
No security issues identified in the workflow changes. The changes maintain proper authentication and authorization patterns.
Performance Impact
Minimal performance impact - the workflow changes should actually be more reliable and potentially faster by removing unnecessary git operations.
Positive Observations
- Clear problem identification and targeted solution
- Good commit message documentation explaining the rationale
- Minimal, focused changes that don't introduce unnecessary complexity
- Proper use of conditional logic in GitHub Actions workflow
References
Next Steps
- Consider adding workflow documentation comments as suggested in inline feedback
- Monitor the i18n workflow execution to ensure translations are generated correctly
- Verify the trigger mechanism works as expected for future i18n updates
This is a comprehensive automated review focused on the i18n workflow improvements. The changes are minimal and targeted, addressing a specific operational issue.
github-actions
bot
removed
the
claude-review
This commit triggers the i18n workflow to generate missing translations for subscription.* keys that were added after the last locale update. The subscription.titleUnsubscribed key was added in f2aea9c on Nov 1, but the last locale update was on Oct 30 (a189e51). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
snomiao
force-pushed
the
version-bump-fix-subscription-i18n
branch
from
77a8610 to
abf7e72
Compare
The previous workflow was resetting the branch which discarded the generated locale changes. This fix simplifies the commit logic to just add and commit the changes directly without branch manipulation. The issue was: - git stash + git checkout -B would reset to remote, losing changes - The detached HEAD in GitHub Actions made stash operations unreliable The fix: - Remove unnecessary git stash/checkout operations - Directly add, commit, and push the locale changes 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Add debug step to check: - If subscription section exists in source (en) locale - If subscription section exists in target (zh) locale - Git working tree status before commit This will help diagnose why lobe-i18n isn't generating changes. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Summary
Fixes missing translations for
subscription.*keys that were added after the last automatic locale update.Problem
The
subscription.titleUnsubscribedkey was added on Nov 1, 2025 (commit f2aea9c), but the last automatic locale update ran on Oct 30, 2025. The 1.32.2 release on Nov 6 did not include these translations because:.github/workflows/i18n-update-core.yaml) only runs onversion-bump-*branchesAs a result, the
subscription.*keys exist in English (src/locales/en/main.json) but are missing from all other locales (zh, zh-TW, ru, ja, ko, fr, es, ar, tr).Solution
This PR creates a
version-bump-*branch to trigger the automatic i18n workflow, which will:pnpm collect-i18nto collect i18n strings from the UIpnpm localeto generate translations using OpenAI (lobe-i18n)The workflow will generate translations for the missing keys:
subscription.titleUnsubscribedRelated
Testing
Once the i18n workflow completes, verify:
subscription.*keys are translated in all locales (zh, zh-TW, ru, ja, ko, fr, es, ar, tr)Workflow Details
The i18n workflow is configured to:
version-bump-*branches.i18nrc.cjs(including Chinese character requirements)┆Issue is synchronized with this Notion page by Unito