Fix misleading AI activity messages that could be mistaken for actual database operations.#9722
Open
dpage wants to merge 1 commit intopgadmin-org:masterfrom
Open
Fix misleading AI activity messages that could be mistaken for actual database operations.#9722dpage wants to merge 1 commit intopgadmin-org:masterfrom
dpage wants to merge 1 commit intopgadmin-org:masterfrom
Conversation
WalkthroughRefactoring consolidates duplicated thinking message functionality into a new shared module. Two components that previously defined local THINKING_MESSAGES arrays and getRandomThinkingMessage functions now import these from a centralized module. Release notes are updated with a bug fix entry. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
dpage
force-pushed
the
ai_activity_messages
branch
from
89305ee to
5fe3be6
Compare
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
web/pgadmin/static/js/Explain/AIInsights.jsx (1)
5-5: Copyright year appears outdated.The copyright year shows "2025" while the other modified file (
NLQChatPanel.jsx) shows "2026". Consider updating this to maintain consistency.📅 Proposed fix
-// Copyright (C) 2013 - 2025, The pgAdmin Development Team +// Copyright (C) 2013 - 2026, The pgAdmin Development Team🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@web/pgadmin/static/js/Explain/AIInsights.jsx` at line 5, Update the copyright header string "Copyright (C) 2013 - 2025, The pgAdmin Development Team" to match the other file by changing the end year to 2026; locate the header in AIInsights.jsx (the top-of-file copyright comment) and replace 2025 with 2026 so it is consistent with NLQChatPanel.jsx.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@web/pgadmin/tools/sqleditor/static/js/components/sections/NLQChatPanel.jsx`:
- Around line 144-177: THINKING_MESSAGES in NLQChatPanel.jsx contains hard-coded
English strings; wrap each string in gettext() so they become translatable
(matching AIInsights.jsx). Locate the THINKING_MESSAGES constant and replace
each plain string entry with gettext('...') preserving the same message text;
ensure gettext is imported/available in NLQChatPanel.jsx (add import/gettext
usage consistent with surrounding components) and run lint/format.
---
Nitpick comments:
In `@web/pgadmin/static/js/Explain/AIInsights.jsx`:
- Line 5: Update the copyright header string "Copyright (C) 2013 - 2025, The
pgAdmin Development Team" to match the other file by changing the end year to
2026; locate the header in AIInsights.jsx (the top-of-file copyright comment)
and replace 2025 with 2026 so it is consistent with NLQChatPanel.jsx.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: fb1a6248-41e5-4d18-94a8-72b1690e9393
📒 Files selected for processing (5)
docs/en_US/release_notes.rstdocs/en_US/release_notes_9_13.rstdocs/en_US/release_notes_9_14.rstweb/pgadmin/static/js/Explain/AIInsights.jsxweb/pgadmin/tools/sqleditor/static/js/components/sections/NLQChatPanel.jsx
web/pgadmin/tools/sqleditor/static/js/components/sections/NLQChatPanel.jsx
Outdated
Show resolved
Hide resolved
dpage
force-pushed
the
ai_activity_messages
branch
2 times, most recently
from
0efb595 to
26da6bc
Compare
The previous messages like "Vacuuming the catalog..." and "Analyzing table statistics..." could be mistaken for actual database operations. Replace them with clearly whimsical elephant-themed messages, expand the pool to 32 messages, and consolidate them into a single shared module with gettext() support. Fixes pgadmin-org#9702 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
dpage
force-pushed
the
ai_activity_messages
branch
from
26da6bc to
c53c996
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@web/pgadmin/static/js/ai_thinking_messages.js`:
- Around line 14-47: The THINKING_MESSAGES array contains DB-adjacent verbs
("Joining the herds...", "Querying the watering hole...", "Rolling back the
peanuts...", "Migrating across the plains...") that can be misinterpreted;
update the entries in the THINKING_MESSAGES constant to fully whimsical,
non-operational alternatives (e.g., replace each offending string with a playful
phrase like "Chasing butterfly clouds...", "Whistling with the wind...",
"Balancing on a sunbeam...", "Skipping over moonbeams..."), preserving the
gettext(...) wrappers and array structure so localization remains intact.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: e2f9145f-5705-4325-aa97-fa478820609a
📒 Files selected for processing (4)
docs/en_US/release_notes_9_14.rstweb/pgadmin/static/js/Explain/AIInsights.jsxweb/pgadmin/static/js/ai_thinking_messages.jsweb/pgadmin/tools/sqleditor/static/js/components/sections/NLQChatPanel.jsx
🚧 Files skipped from review as they are similar to previous changes (1)
- docs/en_US/release_notes_9_14.rst
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit
Documentation
Refactor