Skip to content

Fix an issue where the AI Assistant input textbox sometimes swallows the first character of input.#9741

Open
dpage wants to merge 1 commit intopgadmin-org:masterfrom
dpage:chat_input_char_loss
Open

Fix an issue where the AI Assistant input textbox sometimes swallows the first character of input.#9741
dpage wants to merge 1 commit intopgadmin-org:masterfrom
dpage:chat_input_char_loss

Conversation

@dpage
Copy link
Contributor

@dpage dpage commented Mar 11, 2026

Fixes #9740

Summary by CodeRabbit

  • Bug Fixes

    • Fixed issue where the AI Assistant input textbox would swallow the first character of input.
  • Documentation

    • Updated release notes to document the input textbox fix.

@coderabbitai
Copy link

coderabbitai bot commented Mar 11, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5c1e9b2a-db20-42ab-939c-555c74bd7a33

📥 Commits

Reviewing files that changed from the base of the PR and between c8bd75c and fdfe548.

📒 Files selected for processing (2)
  • docs/en_US/release_notes_9_14.rst
  • web/pgadmin/tools/sqleditor/static/js/components/sections/NLQChatPanel.jsx

Walkthrough

This PR fixes Issue #9740 where the AI Assistant input textbox sometimes swallows the first character of input. The fix adds auto-focus logic to the NLQChatPanel component that triggers when loading completes, and documents the change in release notes.

Changes

Cohort / File(s) Summary
Bug Fix Documentation
docs/en_US/release_notes_9_14.rst
Added release notes entry documenting the fix for Issue #9740 regarding input textbox character loss.
NLQChatPanel Auto-Focus Implementation
web/pgadmin/tools/sqleditor/static/js/components/sections/NLQChatPanel.jsx
Introduced inputRef to enable programmatic focus control and added useEffect hook to auto-focus the input field when loading completes, addressing the first-character swallowing issue.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and specifically describes the main bug fix addressed in the PR - resolving the issue where the AI Assistant input textbox loses the first character.
Linked Issues check ✅ Passed The code changes directly address issue #9740 by adding input ref and auto-focus logic to prevent character loss in the AI Assistant input textbox.
Out of Scope Changes check ✅ Passed All changes are scoped to fixing the reported issue: documentation update noting the fix and implementation changes to NLQChatPanel component to resolve the first-character loss bug.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

The AI Assistant input textbox sometimes swallows the first character of input.

1 participant