Skip to content

web-ui: stabilize rich text input sync#323

Merged
wgqqqqq merged 2 commits intoGCWing:mainfrom
wgqqqqq:fix/rich-text-input-sync-regression
Mar 31, 2026
Merged

web-ui: stabilize rich text input sync#323
wgqqqqq merged 2 commits intoGCWing:mainfrom
wgqqqqq:fix/rich-text-input-sync-regression

Conversation

@wgqqqqq
Copy link
Copy Markdown
Collaborator

@wgqqqqq wgqqqqq commented Mar 31, 2026

Summary

  • replace the RichTextInput external sync skip-flag with explicit sync action selection based on parent value vs current DOM content
  • add focused unit tests for sync action selection and component-level regression coverage for local echo vs true external replacement
  • add jsdom to src/web-ui devDependencies so the DOM-level regression test runs in Vitest

Testing

  • pnpm --dir src/web-ui exec vitest run src/flow_chat/components/richTextInputSync.test.ts src/flow_chat/components/RichTextInput.test.tsx

@wgqqqqq wgqqqqq force-pushed the fix/rich-text-input-sync-regression branch from ce1884d to 453f3a2 Compare March 31, 2026 13:05
@wgqqqqq wgqqqqq merged commit 4453a95 into GCWing:main Mar 31, 2026
4 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