Agentic Setup: Fix onboarding copy prompt CTA appearing without hover

[Sidnioulz]

Fixed with canary

Summary by CodeRabbit

• Refactor
  • Enhanced component flexibility by allowing additional properties to be passed and forwarded to underlying button elements, improving extensibility without affecting existing functionality.

[Copilot]

Pull request overview

This PR adjusts the CopyButton wrapper used in the sidebar onboarding checklist so it can forward additional props to the underlying ActionList.Button, which is needed for correct hover/targeting behavior (e.g., data-target-id used by the hover UI).

Changes:

• Add rest props (...props) to CopyButton and spread them onto ActionList.Button.
• Reformat the CopyButton return to support prop forwarding.

Comments suppressed due to low confidence (1)

code/core/src/manager/components/sidebar/ChecklistWidget.tsx:185

• CopyButton now forwards ...props to ActionList.Button, but the component’s props type still only declares label, copyContent, and onClick. As a result, JSX usages like data-target-id (and any other button props you intend to forward) will still be a TypeScript error. Update the CopyButton props type to extend the underlying ActionList.Button props (e.g., React.ComponentProps<typeof ActionList.Button>), omitting conflicting keys like children/onClick as needed so callers can pass through DOM/ARIA/data attributes safely.

  ...props
}: {
  label: string;
  copyContent: string;
  onClick: (e: SyntheticEvent) => void;

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 62e92c99-1099-4ee6-bd75-11132d4e6be7

📥 Commits

Reviewing files that changed from the base of the PR and between 1cf16c5 and 236994f.

📒 Files selected for processing (1)

• code/core/src/manager/components/sidebar/ChecklistWidget.tsx

---

📝 Walkthrough

Walkthrough

The CopyButton component in ChecklistWidget.tsx was modified to accept and forward additional props to its underlying ActionList.Button element, enabling greater flexibility in prop composition without explicit prop declaration.

Changes

Props Forwarding Enhancement

Layer / File(s)	Summary
Component Enhancement code/core/src/manager/components/sidebar/ChecklistWidget.tsx	CopyButton component destructures ...props from parameters and forwards them to the rendered ActionList.Button alongside copyButtonProps, allowing callers to pass additional props through the component.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

---

_{Review rate limit: 4/5 reviews remaining, refill in 12 minutes.}

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