Skip to content

chore: tighten ui copy#76

Open
SurviveM wants to merge 2 commits intoSakuraByteCore:mainfrom
SurviveM:chore/tighten-ui-copy
Open

chore: tighten ui copy#76
SurviveM wants to merge 2 commits intoSakuraByteCore:mainfrom
SurviveM:chore/tighten-ui-copy

Conversation

@SurviveM
Copy link
Copy Markdown
Collaborator

@SurviveM SurviveM commented Apr 6, 2026

Summary

  • tighten copy in the OpenClaw config tab
  • tighten copy in the skills market tab
  • tighten copy in the CLI install modal

Tests

  • npm run lint

Summary by CodeRabbit

  • Style

    • Refined Chinese UI copy across the install modal, OpenClaw config panel, and Skills market for clearer, more concise wording and labels.
  • Tests

    • Tightened a parity test to treat previously-allowed UI state differences as failures, increasing strictness for UI parity checks.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: b25930cb-941e-4cd7-b4bf-52b45ec438f7

📥 Commits

Reviewing files that changed from the base of the PR and between f6fc1b0 and d3dd231.

📒 Files selected for processing (1)
  • tests/unit/web-ui-behavior-parity.test.mjs
📜 Recent review details
🔇 Additional comments (1)
tests/unit/web-ui-behavior-parity.test.mjs (1)

322-323: Strict parity enforcement is a solid improvement.

Setting both extra-key allowlists to empty correctly hardens the parity gate so new data()/computed drift cannot slip through silently.

Also applies to: 360-360


📝 Walkthrough

Walkthrough

Updated Chinese UI text in three HTML templates (install modal, OpenClaw panel, market panel) and tightened a unit parity test to disallow previously-allowed extra data/computed keys. No structural code or runtime control-flow changes introduced.

Changes

Cohort / File(s) Summary
UI Text Localization
web-ui/partials/index/modals-basic.html, web-ui/partials/index/panel-config-openclaw.html, web-ui/partials/index/panel-market.html
Refined Chinese copy: modal titles, button labels, hints, and section descriptions updated for clarity and conciseness. No logic, bindings, or structural changes.
Unit Test Tightening
tests/unit/web-ui-behavior-parity.test.mjs
Removed allowlisted extra data() and computed keys by replacing previously permitted arrays with empty arrays, causing stricter parity assertions against HEAD.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • ymkiux
  • awsl233777

Poem

🐰
I nibbled through the UI night,
Polished labels till they’re bright,
Hints trimmed neat, messages small,
Tests kept honest — fair to all. ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: tighten ui copy' directly and clearly describes the main change: condensing and refining user-facing text across multiple UI templates.
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 unit tests (beta)
  • Create PR with unit tests

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.

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