Skip to content

fix(flow-chat): clarify permission scope menu - #2710

Merged
GCWing merged 1 commit into
1.0.0-explorefrom
gcwing/chatinput-permission-scope-menu
Aug 31, 2026
Merged

fix(flow-chat): clarify permission scope menu#2710
GCWing merged 1 commit into
1.0.0-explorefrom
gcwing/chatinput-permission-scope-menu

Conversation

@GCWing

@GCWing GCWing commented Aug 31, 2026

Copy link
Copy Markdown
Owner

Summary

  • keep the current session permission modes as the primary single-select list, with a check only on the selected mode
  • move next-message/current-turn permission choices into a click-open secondary level with Right/Left Arrow navigation
  • add an explicit "Follow this session" choice to clear a one-off override while preserving existing permission behavior

Verification

  • pnpm exec vitest run src/flow_chat/components/ChatInputWorkspaceStrip.test.tsx (30/30)
  • pnpm run type-check:web
  • pnpm run i18n:audit
  • pnpm run appearance:contract-audit
  • focused ESLint for the two changed source components
  • git diff --check origin/1.0.0-explore...HEAD

Notes

  • An earlier full pnpm run check:web attempt was blocked by pre-existing stale generated outputs for MiniApp Git Graph and native mobile token projection; those files are outside this six-file PR and were not modified.
  • Browser/manual visual checks and E2E were not run, following the repository guidance for this focused menu change.

@GCWing
GCWing merged commit 84b6fa9 into 1.0.0-explore Aug 31, 2026
8 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.

1 participant