Skip to content

fix(feishu): restore naming paths and split control UI context notices#48898

Closed
futuremind2026 wants to merge 7 commits intoopenclaw:mainfrom
MaGonglei:codex/upstream-feishu-control-ui-20260317
Closed

fix(feishu): restore naming paths and split control UI context notices#48898
futuremind2026 wants to merge 7 commits intoopenclaw:mainfrom
MaGonglei:codex/upstream-feishu-control-ui-20260317

Conversation

@futuremind2026
Copy link
Copy Markdown
Contributor

Summary

This PR ports a small fork-maintained patch set onto current main to restore two Feishu UX/runtime behaviors and one Control UI visibility improvement:

  1. restore Feishu group-name and topic/thread label display
  2. restore Feishu direct-message sender-name fallback
  3. split Control UI context notices between model context ceiling and higher-rate pricing threshold

What changed

Feishu

  • preserve Feishu group display names instead of normalizing them into slug-like labels
  • append the chat id only when it is not already visible in the label
  • resolve topic labels from the root message and surface them in thread context labels
  • use chatMembers for DM peer-name fallback
  • prefer user_id before open_id when resolving group sender names

Control UI

  • add gateway.controlUi.chat.showPricingThreshold
  • split the chat context notice into:
    • model context usage / limit
    • higher-rate pricing threshold
  • wire the setting through gateway/bootstrap/UI layers
  • add the corresponding tests

Validation

Local validation on the upstream-port branch:

  • passed: src/gateway/control-ui.http.test.ts
  • passed: pnpm ui:build
  • passed: newly added Feishu coverage for DM member-name fallback and group sender user_id fallback
  • note: 3 ACP-routing tests in extensions/feishu/src/bot.test.ts also fail when re-run locally on current origin/main, so they are not introduced by this PR branch
  • note: browser UI tests were not re-run in this temporary worktree because the local environment currently lacks @vitest/browser-playwright

Scope

This PR only carries the functional patch set. It does not include any deploy/update automation or local runtime-management changes.

@openclaw-barnacle openclaw-barnacle Bot added app: web-ui App: web-ui gateway Gateway runtime channel: feishu Channel integration: feishu size: L labels Mar 17, 2026
@futuremind2026
Copy link
Copy Markdown
Contributor Author

Closing this draft because it overlaps with existing upstream PRs #47540 and #38958. I will continue by updating those existing PR branches instead of maintaining a duplicate combined PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app: web-ui App: web-ui channel: feishu Channel integration: feishu gateway Gateway runtime size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant