Skip to content

[Studio] fix: preserve prompt submission integrity - #2148

Closed
yuluo-yx wants to merge 1 commit into
apache:rocketmq-studiofrom
yuluo-yx:0813-yuluo-yx/fix-13
Closed

[Studio] fix: preserve prompt submission integrity#2148
yuluo-yx wants to merge 1 commit into
apache:rocketmq-studiofrom
yuluo-yx:0813-yuluo-yx/fix-13

Conversation

@yuluo-yx

Copy link
Copy Markdown
Member

Summary

  • ignore Enter while Home and AI inputs are composing IME text
  • guard AI streaming with a synchronous in-flight reference
  • release the guard after every stream completion path

Fixes #2143

Validation

  • HomePage.test.tsx and AiPage.test.tsx: 11 passed
  • targeted Prettier and ESLint passed
  • scope check: 62 changed lines

Base

rocketmq-studio at 737a7b4

@RockteMQ-AI RockteMQ-AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary

This PR fixes AI prompt submission integrity by trimming whitespace, preventing empty submissions, and disabling the submit button during loading. The changes ensure consistent behavior between Enter key and button click submissions.

Clean fix with proper test coverage. LGTM.


Automated review by github-manager-bot

@lizhimins

Copy link
Copy Markdown
Member

Closing per contribution guideline #2107: these are fragmented one-off changes of the same theme (one PR per guard/fix/page). Please group related changes into cohesive pull requests and resubmit — your contributions are welcome in a consolidated shape.

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.

3 participants