Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@apps/web/src/components/channel-setup/feishu-setup-view.tsx`:
- Around line 432-441: The help sentence is missing the trailing translation key
and becomes incomplete; update the JSX in feishu-setup-view.tsx to render the
missing t("feishuSetup.helpSuffix") immediately after
t("feishuSetup.helpLinkText") (preserving spacing/punctuation) so the full
sentence is produced alongside the ExternalLink icon; ensure you reference the
same translation key t("feishuSetup.helpSuffix") used in locales/en.ts and keep
target="_blank" rel attributes intact.

In `@apps/web/src/components/model-picker-dropdown.tsx`:
- Line 393: The selected-model label currently uses the low-contrast utility
"text-brand-primary" on a near-white surface; update the JSX in the
ModelPickerDropdown component where "text-brand-primary" is applied (adjacent to
"bg-brand-primary/5" and the check/highlight logic) to use the primary
foreground utility (e.g., "text-foreground" or "text-primary") instead, leaving
the check/highlight classes (like "bg-brand-primary/5") unchanged so the brand
accent remains only in the check/highlight while the label uses the readable
primary text color.

---

Nitpick comments:
In `@apps/desktop/src/runtime-page.css`:
- Line 1339: Extract the hard-coded success hex (`#00a365`) into a shared CSS
variable (e.g., --desktop-success-color) defined at a common scope (for example
:root or the top-level .desktop runtime container) and replace the two
occurrences of background: `#00a365`; with background:
var(--desktop-success-color); to ensure a single source of truth and prevent
future drift.

In `@apps/web/src/i18n/locales/en.ts`:
- Line 687: Update the English copy for the "wechatSetup.desc" locale entry so
the value reads "Scan with WeChat on your phone to connect." instead of "Scan in
WeChat on your phone to connect."; locate the "wechatSetup.desc" key in the
locales file and replace the string accordingly to improve readability.

In `@apps/web/src/pages/sessions.tsx`:
- Around line 414-427: Replace the hardcoded background rgba in the success
badge with the design token variable to match surrounding usage: change the span
using bg-[rgba(0,163,101,0.12)] (the element rendering the CheckCircle2 icon
inside the success badge in sessions.tsx) to use a CSS variable like
bg-[var(--color-success-muted)] (or the appropriate --color-success token
available in the codebase), keeping the rest of the classes and semantics
(formattedSummary, CheckCircle2, and t("sessions.chat.toolCompleted"))
unchanged; ensure the chosen CSS variable exists or add it to the theme tokens
before updating the class.

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@apps/web/src/components/skills/import-skill-modal.tsx`:
- Around line 121-134: The two identical className strings on the TabsTrigger
components should be extracted to a single constant to avoid duplication; define
a descriptive constant (e.g. tabTriggerClass) near the component (in the same
module or component scope) and replace both TabsTrigger className props with
that constant so styling changes only need to be made in one place; ensure the
constant is exported or scoped appropriately if used elsewhere and keep the
original class string exactly as-is when moving it into the constant.

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@apps/web/src/i18n/locales/zh-CN.ts`:
- Line 217: Some i18n keys were renamed but call sites still use the old IDs,
causing runtime missing translations; add alias entries in the zh-CN locale
mapping so old IDs point to the same strings as the new IDs. Locate the new keys
such as "home.starnexu" and create duplicates for the old IDs used in code
(e.g., add "home.starNexu" mapping to the same value), and similarly add aliases
for claim.teamUsesNexu.* and claim.exploreNexu, feishuBind.exploreNexu,
slackSetup.addNexuTitle, and slackSetup.addNexuDesc so callers continue to
resolve until callers are migrated. Ensure each alias key uses the identical
string value as its new counterpart to preserve behavior.

In `@apps/web/src/pages/models.tsx`:
- Around line 1152-1155: The "Active" badge is hardcoded; replace the literal
text with a localized string using the app's i18n pattern (e.g., useTranslation
or existing t() function used elsewhere in this file) and a key like
"status.active". Update both occurrences (the span containing <Check /> at the
shown block and the same block around lines 1520-1523) to call
t('status.active') (or the project's equivalent) so the badge text is
translated; add the translation key to the locale files if missing and
import/use the same translation hook used in this component (e.g., const { t } =
useTranslation()).
- Around line 1013-1050: The button currently only renders for cloudConnected ||
loginBusy and its onClick only handles disconnect; add the missing "connect"
path so the same header action appears when not connected and clicking initiates
a connect flow. Specifically, keep the existing props (cloudToggleBusy,
aria-label, loader, translations) but extend the onClick handler to: if
(!cloudConnected) return early when already connecting, set a new
cloudConnecting state (e.g., cloudConnecting / setCloudConnecting), call the
appropriate connect API (e.g., postApiInternalDesktopCloudConnect) with
try/catch/finally, setCloudConnected(true) on success and invalidate the same
queries via queryClient.invalidateQueries(["models"]) and
["desktop-default-model"], and clear cloudConnecting in finally; also update
aria-label and button text to use the "connect" translations when cloudConnected
is false and ensure the loader shows when cloudConnecting || loginBusy. Use the
existing symbols cloudConnected, loginBusy, cloudToggleBusy, cloudDisconnecting,
setCloudDisconnecting, setCloudConnected, queryClient.invalidateQueries and add
cloudConnecting / setCloudConnecting and postApiInternalDesktopCloudConnect to
locate where to implement the fix.

---

Nitpick comments:
In `@apps/web/src/pages/models.tsx`:
- Around line 869-870: Guard the model-switch mutation by checking the
updateModel mutation state before calling it: in the onSelectModel handlers that
currently call updateModel.mutate(modelId) (and the similar handler at the other
occurrence), return early if updateModel.isPending (or otherwise use a boolean
flag) to ignore clicks while a switch is in progress, and also consider
disabling the model selection UI prop when updateModel.isPending to prevent
further input.

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@apps/web/src/i18n/locales/en.ts`:
- Around line 708-710: Update the slack-oauth-view component to use the new
lowercase locale keys: replace any references to slackSetup.addNexuTitle and
slackSetup.addNexuDesc in the Slack OAuth view with slackSetup.addnexuTitle and
slackSetup.addnexuDesc (i.e., update the getString/useTranslation calls or JSX
t('...') usages inside the slack-oauth-view component to the new key names so
they match the updated en locale).
- Line 221: Update the translation keys used in components to match the new
lowercase keys: replace occurrences of "home.starNexu" with "home.starnexu"
(e.g., any t('home.starNexu') calls in the home component), and replace
"slackSetup.addNexuTitle" and "slackSetup.addNexuDesc" with
"slackSetup.addnexuTitle" and "slackSetup.addnexuDesc" respectively (e.g., the
t(...) usages in the Slack OAuth view component); ensure all string literals
passed to the i18n translator (t or similar) use the new lowercase keys so
lookups succeed.

---

Nitpick comments:
In `@apps/web/src/i18n/locales/en.ts`:
- Around line 290-293: Rename the i18n key models.managed.connected (value "Log
out") to a semantically correct key like models.managed.logOut and update every
reference to it across the codebase (components, hooks, tests) to use
models.managed.logOut instead of models.managed.connected; also update other
locale files and any snapshot/tests that expect the old key, and ensure no
fallback relies on models.managed.connected to avoid runtime missing-key errors.

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@apps/web/src/pages/sessions.tsx`:
- Line 414: The background uses a hardcoded rgba value inside the className on
the Sessions page which conflicts with the border's use of the CSS token
var(--color-success) via color-mix; replace the hardcoded background
(rgba(0,163,101,0.06)) with a CSS token or color-mix (for example define and use
--color-success-subtle or use color-mix(in srgb, var(--color-success) 6%,
transparent)) so both border and background derive from the same theme
token—update the className in apps/web/src/pages/sessions.tsx where that long
className string appears to reference the new token/mix instead of the rgba
literal.