Skip to content

Fix "fix in copilot" button#3931

Merged
TheodoreSpeaks merged 2 commits intostagingfrom
fix/fix-in-copilot
Apr 4, 2026
Merged

Fix "fix in copilot" button#3931
TheodoreSpeaks merged 2 commits intostagingfrom
fix/fix-in-copilot

Conversation

@TheodoreSpeaks
Copy link
Copy Markdown
Collaborator

Summary

Our fix in copilot button wasn't sending the message to copilot. Added back copilot listener

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Other: ___________

Testing

  • Validated that message is now sent to copilot chat.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Screenshots/Videos

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 4, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Apr 4, 2026 2:00am

Request Review

@TheodoreSpeaks
Copy link
Copy Markdown
Collaborator Author

@BugBot review @greptile review

@cursor
Copy link
Copy Markdown

cursor bot commented Apr 4, 2026

PR Summary

Low Risk
Low risk: a small UI-side change that only adds a window event listener to route messages into the Copilot chat; main risk is unintended message dispatch/auto-tab switching if the event fires unexpectedly.

Overview
Restores support for the “Fix in Copilot” flow by adding a window listener for the mothership-send-message custom event in the workflow Panel.

When the event is received, the panel now automatically switches to the Copilot tab and forwards the provided message via copilotSendMessage, ensuring external dispatches reach the Copilot chat.

Reviewed by Cursor Bugbot for commit 566cb8c. Bugbot is set up for automated code reviews on this repo. Configure here.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 4, 2026

Greptile Summary

This PR fixes the "Fix in Copilot" button by re-adding the missing mothership-send-message custom event listener to panel.tsx. The listener was previously absent, so messages were never reaching the workflow panel's copilot chat.

Confidence Score: 5/5

Safe to merge — targeted one-file bug fix with proper cleanup and correct dependency array.

The change is a minimal, well-scoped bug fix that re-adds a missing event listener. It follows the established pattern in home.tsx, includes proper cleanup via the return function, and the dependency array is correct. No logic issues, no type safety concerns, and no rule violations found.

No files require special attention.

Important Files Changed

Filename Overview
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/panel.tsx Adds back the missing mothership-send-message window event listener so the "Fix in Copilot" button correctly routes messages to the panel's copilot chat and switches to the copilot tab.

Reviews (3): Last reviewed commit: "Auto send message to copilot for fix in ..." | Re-trigger Greptile

@TheodoreSpeaks
Copy link
Copy Markdown
Collaborator Author

@BugBot review @greptile review

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 566cb8c. Configure here.

@TheodoreSpeaks TheodoreSpeaks marked this pull request as ready for review April 4, 2026 02:10
@TheodoreSpeaks TheodoreSpeaks merged commit 2e69f85 into staging Apr 4, 2026
12 checks passed
@TheodoreSpeaks TheodoreSpeaks deleted the fix/fix-in-copilot branch April 4, 2026 02:13
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.

1 participant