revert(docs): DR-7612 Remove floating Ask AI input#7634
revert(docs): DR-7612 Remove floating Ask AI input#7634ArthurGamby wants to merge 1 commit intomainfrom
Conversation
This reverts commit e867ae3.
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
WalkthroughThese 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
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 📝 Coding Plan for PR 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. Comment |
There was a problem hiding this comment.
🧹 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
📒 Files selected for processing (4)
apps/docs/src/app/(docs)/(default)/layout.tsxapps/docs/src/components/ai-chat-sidebar.tsxapps/docs/src/components/floating-ask.tsxapps/docs/src/hooks/use-ai-chat.ts
💤 Files with no reviewable changes (1)
- apps/docs/src/components/floating-ask.tsx
Summary
floating-ask.tsxand restores original state oflayout.tsx,ai-chat-sidebar.tsx, anduse-ai-chat.tsLinear
https://linear.app/prisma-company/issue/DR-7612/revert-remove-floating-ask-ai-input-from-docs
Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit