Skip to content

fix: update install message#529

Merged
mazhe-nerd merged 1 commit intomainfrom
fix/install-message
Apr 20, 2026
Merged

fix: update install message#529
mazhe-nerd merged 1 commit intomainfrom
fix/install-message

Conversation

@mazhe-nerd
Copy link
Copy Markdown
Collaborator

@mazhe-nerd mazhe-nerd commented Apr 17, 2026

Summary

update install message

Summary by CodeRabbit

  • New Features
    • Improved installation wizard prompts (English and Chinese): clearer wording that explicitly frames accessed data as personal (messages, documents, calendar, etc.) and states the AI may perform actions on your behalf. Also updated the final completion message wording and punctuation for better clarity in Chinese and English.

@github-actions github-actions Bot added the size/M Single-domain feat or fix with limited business impact label Apr 17, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ce672ff5-0591-409d-8649-fe5f2c658de9

📥 Commits

Reviewing files that changed from the base of the PR and between 322bc75 and f1461ea.

📒 Files selected for processing (1)
  • scripts/install-wizard.js
✅ Files skipped from review due to trivial changes (1)
  • scripts/install-wizard.js

📝 Walkthrough

Walkthrough

Updated localized CLI install-wizard prompts (Chinese and English): authorization confirmation now explicitly names personal data types (messages, documents, calendar) and states the AI will perform actions on the user's behalf; Chinese completion outro wording and punctuation adjusted.

Changes

Cohort / File(s) Summary
Localization Text Updates
scripts/install-wizard.js
Adjusted step4Confirm localized strings in zh and en to explicitly mention access to personal data (messages, documents, calendar) and that the AI will "perform actions on your behalf"; reworded Chinese done outro punctuation and phrasing.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • Feature/easy setup #464 — Edits localized prompt/completion strings in scripts/install-wizard.js, directly related to these string tweaks.

Suggested labels

size/L

Suggested reviewers

  • liangshuo-1

Poem

🐰 小兔报喜话语新,
授权言明更分明,
私人消息日历与文档,
AI 代为动作心中定。
本地化润色来庆贺,祝君安装顺利! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is incomplete, missing the Changes, Test Plan, and Related Issues sections required by the repository template. Complete the description by adding detailed Changes section (list all modifications), Test Plan section (describe verification approach), and Related Issues section, following the provided template structure.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: update install message' is directly related to the changeset, which updates localized text strings in the installation wizard prompts.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/install-message

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@scripts/install-wizard.js`:
- Line 45: Update the translation string for the "done" message to use the
correct product casing: change "Claude code" to "Claude Code" in the value of
the done property in the translations/object in scripts/install-wizard.js;
ensure the rest of the string punctuation and escaping remain unchanged and run
a quick search for any other occurrences (e.g., the English line) to keep casing
consistent.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 145122fa-9871-4757-8028-4b1f807ed04e

📥 Commits

Reviewing files that changed from the base of the PR and between 6ad25cd and 322bc75.

📒 Files selected for processing (1)
  • scripts/install-wizard.js

Comment thread scripts/install-wizard.js Outdated
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.19%. Comparing base (6ad25cd) to head (f1461ea).
⚠️ Report is 22 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #529      +/-   ##
==========================================
+ Coverage   59.05%   60.19%   +1.13%     
==========================================
  Files         384      390       +6     
  Lines       32636    33433     +797     
==========================================
+ Hits        19274    20125     +851     
+ Misses      11553    11426     -127     
- Partials     1809     1882      +73     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 17, 2026

🚀 PR Preview Install Guide

🧰 CLI update

npm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@f1461ea327e2112349afd921e5c40276c39d31f1

🧩 Skill update

npx skills add larksuite/cli#fix/install-message -y -g

liangshuo-1
liangshuo-1 previously approved these changes Apr 20, 2026
@mazhe-nerd mazhe-nerd merged commit 9acd121 into main Apr 20, 2026
21 checks passed
@mazhe-nerd mazhe-nerd deleted the fix/install-message branch April 20, 2026 04:03
@liangshuo-1 liangshuo-1 mentioned this pull request Apr 20, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Single-domain feat or fix with limited business impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants