Improve the AI Assistant user prompt to be more descriptive of the actual functionality.

[dpage]

Summary by CodeRabbit

• Bug Fixes

  • Improved AI Assistant prompt clarity for SQL generation (Issue Improve the AI Assistant user prompt #9732).

• Improvements

  • Enhanced assistant interface messages to better guide users on asking questions or describing needed SQL, and clarified supported operations.

• Documentation

  • Updated release notes to reflect the improved prompts.

• Tests

  • Adjusted test expectations to match the updated UI copy.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 85319d0f-d844-42ab-83ce-be3a4b52bf52

📥 Commits

Reviewing files that changed from the base of the PR and between 4b10544 and e5fea0d.

📒 Files selected for processing (2)

• docs/en_US/release_notes_9_14.rst
• web/pgadmin/tools/sqleditor/static/js/components/sections/NLQChatPanel.jsx

---

Walkthrough

Updates user-facing copy for the NLQ Chat Panel: expands the empty-state introductory text and changes the input placeholder. Corresponding test assertions were updated and a release-note entry was added documenting the prompt clarity fix (Issue #9732).

Changes

Cohort / File(s)	Summary
Release Notes docs/en_US/release_notes_9_14.rst	Added a Bug fixes entry referencing Issue #9732 about improving the AI Assistant user prompt.
NLQ Chat Panel Component web/pgadmin/tools/sqleditor/static/js/components/sections/NLQChatPanel.jsx	Reworded UI copy: empty-state message expanded to mention asking database questions and supported SQL statement types; input placeholder changed to "Ask a question or describe the SQL you need...".
NLQ Chat Panel Tests web/regression/javascript/sqleditor/NLQChatPanel.spec.js	Updated test expectations to match the new placeholder and adjusted empty-state message assertions.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: improving the AI Assistant user prompt to be more descriptive. The modifications across three files all center on enhancing user-facing text and prompts in the NLQChatPanel component.
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

📝 Coding Plan

• Generate coding plan for human review comments

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

Tip

CodeRabbit can suggest fixes for GitHub Check annotations.

Configure the reviews.tools.github-checks setting to adjust the time to wait for GitHub Checks to complete.

[coderabbitai]

🧹 Nitpick comments (1)

docs/en_US/release_notes_9_14.rst (1)

34-35: Consider moving this entry to "Housekeeping" section.

This change improves user-facing copy for clarity rather than fixing a bug. The "Housekeeping" section (line 25) may be a more appropriate location for this entry.

Additionally, line 35 has trailing whitespace that could be removed.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@docs/en_US/release_notes_9_14.rst` around lines 34 - 35, Move the
release-note entry for "Issue `#9732` - Improve the AI Assistant user prompt..."
out of the current location and insert it under the "Housekeeping" section (the
heading titled "Housekeeping") since it is copy clarity rather than a bug fix,
and also remove the trailing whitespace at the end of the entry line (the extra
space after the closing backticks on the line that contains the Issue `#9732`
reference).

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@docs/en_US/release_notes_9_14.rst`:
- Around line 34-35: Move the release-note entry for "Issue `#9732` - Improve the
AI Assistant user prompt..." out of the current location and insert it under the
"Housekeeping" section (the heading titled "Housekeeping") since it is copy
clarity rather than a bug fix, and also remove the trailing whitespace at the
end of the entry line (the extra space after the closing backticks on the line
that contains the Issue `#9732` reference).

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e984af8b-d640-4dd2-ae9e-0f90d501d37b

📥 Commits

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

📒 Files selected for processing (3)

• docs/en_US/release_notes_9_14.rst
• web/pgadmin/tools/sqleditor/static/js/components/sections/NLQChatPanel.jsx
• web/regression/javascript/sqleditor/NLQChatPanel.spec.js

[dpage]

The failed test here seems to be infrastructure related - the suite failed entirely due to timeouts

[dpage]

The test failure here is the flakey test fixed in #9723