Skip to content

feat(studio): add reusable Feishu bot setup card - #1050

Merged
yaozheng-fang merged 4 commits into
volcengine:mainfrom
evanlowe:feat/feishu-bot-setup-card
Sep 3, 2026
Merged

feat(studio): add reusable Feishu bot setup card#1050
yaozheng-fang merged 4 commits into
volcengine:mainfrom
evanlowe:feat/feishu-bot-setup-card

Conversation

@evanlowe

@evanlowe evanlowe commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a reusable Feishu bot setup card to the final Agent Studio deployment step
  • keep the card at a fixed 112 px height before and after flipping, with automatic and manual configuration tabs
  • integrate Feishu's official PersonalAgent app-registration device flow for QR creation and polling
  • return credentials only after user authorization, then atomically refill App ID and App Secret
  • isolate reusable backend setup logic under frontend/server/feishu_bot_setup/
  • include rebuilt Studio web assets

Integration notes

  • POST accounts.feishu.cn/oauth/v1/app/registration with action=begin creates the official registration session
  • action=poll is rate-limited according to the provider interval and handles pending, slow-down, denial, expiry, and Feishu/Lark tenant brand switching
  • credentials are cached only in the in-memory server session after a successful authorization; App Secret is never logged
  • the existing manual App ID/App Secret path remains available

Verification

  • real Feishu QR authorization completed successfully; the backend returned non-empty App ID and App Secret without exposing either value in logs
  • frontend credential refill now updates both fields atomically; regression tests cover the contract
  • npm test — 966 passed
  • npm run build — passed
  • backend Feishu setup tests — 11 passed
  • Ruff — passed
  • Pyright — 0 errors, 0 warnings
  • pre-commit and hardcoded-secret scan — passed
  • visual QA reviewed at 1440×900 and 390×844; the flipped card remains 112 px high with no card-level clipping or horizontal overflow
  • GitHub CI — all required checks passed

@evanlowe
evanlowe force-pushed the feat/feishu-bot-setup-card branch 3 times, most recently from 30fedef to 4e046db Compare September 3, 2026 05:16
@evanlowe
evanlowe force-pushed the feat/feishu-bot-setup-card branch from 4e046db to 9c8423f Compare September 3, 2026 10:04
@evanlowe
evanlowe force-pushed the feat/feishu-bot-setup-card branch from 9c8423f to f122f15 Compare September 3, 2026 10:12
@yaozheng-fang
yaozheng-fang merged commit 84e1cde into volcengine:main Sep 3, 2026
10 of 11 checks passed
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.

2 participants