Skip to content

revert(docs): DR-7612 Remove floating Ask AI input#7634

Open
ArthurGamby wants to merge 1 commit intomainfrom
dr-7612-revert-remove-floating-ask-ai-input-from-docs
Open

revert(docs): DR-7612 Remove floating Ask AI input#7634
ArthurGamby wants to merge 1 commit intomainfrom
dr-7612-revert-remove-floating-ask-ai-input-from-docs

Conversation

@ArthurGamby
Copy link
Contributor

@ArthurGamby ArthurGamby commented Mar 12, 2026

Summary

Linear

https://linear.app/prisma-company/issue/DR-7612/revert-remove-floating-ask-ai-input-from-docs

Test plan

  • Verify no floating input bar appears on docs pages
  • Verify the AI chat sidebar still works correctly via the existing trigger
  • Verify no console errors on docs pages

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Refactor
    • Removed the floating question submission UI from the documentation interface.
    • Removed automatic pending message processing from the AI chat functionality.
    • Simplified the documentation layout structure.
    • Refactored the AI chat sidebar for improved code organization and clarity.

@vercel
Copy link

vercel bot commented Mar 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
blog Ready Ready Preview, Comment Mar 12, 2026 2:36pm
docs Ready Ready Preview, Comment Mar 12, 2026 2:36pm
eclipse Ready Ready Preview, Comment Mar 12, 2026 2:36pm

Request Review

@argos-ci
Copy link

argos-ci bot commented Mar 12, 2026

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) 👍 Changes approved 1 removed Mar 12, 2026, 2:41 PM

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 12, 2026

Walkthrough

These changes remove the FloatingAsk component and its supporting infrastructure from the docs application. The pendingMessageAtom is removed from the AI chat hook exports, the FloatingAsk component file is deleted entirely, and the layout and chat sidebar components are updated to eliminate related functionality and auto-submission logic.

Changes

Cohort / File(s) Summary
Component Removal
apps/docs/src/components/floating-ask.tsx
Deleted entire FloatingAsk React client component that provided a floating UI for submitting questions, including scroll-based visibility logic, input handling, and submit flow.
AI Chat Context
apps/docs/src/hooks/use-ai-chat.ts
Removed exported pendingMessageAtom and removed pendingMessage and setPendingMessage properties from useAIChatContext return interface. Minor clarifying comments added.
Layout & Chat Integration
apps/docs/src/app/(docs)/(default)/layout.tsx, apps/docs/src/components/ai-chat-sidebar.tsx
Removed FloatingAsk import and usage from layout; removed automatic pending message auto-submission logic from ai-chat-sidebar ChatInner; applied minor UI className formatting adjustments across multiple components.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 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 'revert(docs): DR-7612 Remove floating Ask AI input' accurately describes the main change: removing the floating Ask AI input component from docs.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

📝 Coding Plan for PR comments
  • Generate coding plan

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.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
apps/docs/src/components/ai-chat-sidebar.tsx (1)

532-533: Consider wrapping long kbd lines for readability.

These keyboard hint <kbd> elements are on single lines that extend quite long. While functionally correct, breaking them across lines would improve readability:

📝 Optional formatting improvement
         <div className="ms-auto inline-flex gap-0.5">
-          <kbd className="rounded-md border bg-fd-background px-1.5 text-fd-muted-foreground group-hover:text-fd-accent-foreground">{isMac ? "⌘" : "Ctrl"}</kbd>
-          <kbd className="rounded-md border bg-fd-background px-1.5 text-fd-muted-foreground group-hover:text-fd-accent-foreground">I</kbd>
+          <kbd className="rounded-md border bg-fd-background px-1.5 text-fd-muted-foreground group-hover:text-fd-accent-foreground">
+            {isMac ? "⌘" : "Ctrl"}
+          </kbd>
+          <kbd className="rounded-md border bg-fd-background px-1.5 text-fd-muted-foreground group-hover:text-fd-accent-foreground">
+            I
+          </kbd>
         </div>
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@apps/docs/src/components/ai-chat-sidebar.tsx` around lines 532 - 533, The
long inline <kbd> elements reduce readability; split the JSX onto multiple lines
so each <kbd> is on its own line and its attributes can be wrapped (e.g., break
the element start, props, and children onto separate lines) — update the
occurrences of the kbd elements rendering {isMac ? "⌘" : "Ctrl"} and the kbd
containing "I" in the ai-chat-sidebar component so each element and its
className/props are line-broken for clarity without changing behavior.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@apps/docs/src/components/ai-chat-sidebar.tsx`:
- Around line 532-533: The long inline <kbd> elements reduce readability; split
the JSX onto multiple lines so each <kbd> is on its own line and its attributes
can be wrapped (e.g., break the element start, props, and children onto separate
lines) — update the occurrences of the kbd elements rendering {isMac ? "⌘" :
"Ctrl"} and the kbd containing "I" in the ai-chat-sidebar component so each
element and its className/props are line-broken for clarity without changing
behavior.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 53a6f6ae-86f0-4c94-b4be-6a377a3e3b5a

📥 Commits

Reviewing files that changed from the base of the PR and between 855a669 and df765fa.

📒 Files selected for processing (4)
  • apps/docs/src/app/(docs)/(default)/layout.tsx
  • apps/docs/src/components/ai-chat-sidebar.tsx
  • apps/docs/src/components/floating-ask.tsx
  • apps/docs/src/hooks/use-ai-chat.ts
💤 Files with no reviewable changes (1)
  • apps/docs/src/components/floating-ask.tsx

@aidankmcalister aidankmcalister self-requested a review March 12, 2026 14:54
Copy link
Contributor

@carlagn carlagn left a comment

Choose a reason for hiding this comment

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

LGTM

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