feat(tasks-panel): right-click task to open its Automation config#3385
Open
viniciusventura29 wants to merge 2 commits into
Open
feat(tasks-panel): right-click task to open its Automation config#3385viniciusventura29 wants to merge 2 commits into
viniciusventura29 wants to merge 2 commits into
Conversation
Adds a "Go to Automation" context menu item on tasks triggered by an automation. Backend exposes automation_id via a leftJoin on automation_triggers so the frontend can navigate to the automation detail panel without an extra round-trip. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Contributor
🧪 BenchmarkShould we run the Virtual MCP strategy benchmark for this PR? React with 👍 to run the benchmark.
Benchmark will run on the next push after you react. |
Contributor
Release OptionsSuggested: Minor ( React with an emoji to override the release type:
Current version:
|
Contributor
There was a problem hiding this comment.
1 issue found across 6 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="apps/mesh/src/web/layouts/tasks-panel/task-row.tsx">
<violation number="1" location="apps/mesh/src/web/layouts/tasks-panel/task-row.tsx:72">
P2: Manual tasks are still wrapped in `ContextMenuTrigger`, so right-click is intercepted even when no custom menu is available. Render the context-menu wrapper only when `automationId` is present (or disable the trigger) so manual tasks keep the native browser context menu.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
Re-trigger cubic
Manual tasks now render the row directly without ContextMenuTrigger so right-click falls through to the native browser context menu. Only automation-triggered tasks get the custom menu. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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
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.
What is this contribution about?
Adds a right-click context menu on items in the Tasks panel with a "Go to Automation" action — only shown for tasks triggered by an automation. Clicking it opens the Automation configuration screen in the right panel while preserving the current task in the URL.
To make this possible without an extra round-trip on the client, the backend now exposes
automation_idon threads via aleftJoinonautomation_triggersinthreads.list()andlistByTriggerIds(). The field flows throughThreadEntitySchema→ frontendTasktype →TaskRow, which renders the newContextMenuonly whentask.automation_idis set.Screenshots/Demonstration
N/A — screenshot in the workspace attachments shows the Tasks panel that gains the new menu.
How to Test
bun run dev/$org/<taskId>?virtualmcpid=...&main=automation:<id>and the right panel shows the Automation configuration (Active, Starter, Instructions, etc.).Migration Notes
No database migrations or config changes needed. The new
automation_idis derived at query time from the existingautomation_triggersjoin.Review Checklist
bun run check,bun run lint,bun testall pass)🤖 Generated with Claude Code
Summary by cubic
Add a right‑click “Go to Automation” on tasks triggered by an automation. It opens the Automation config in the right panel and keeps the current task in the URL; manual tasks use the native browser context menu.
automation_idis present; manual tasks are not wrapped and fall through to the native menu.automation_idby left‑joiningautomation_triggersinthreads.list()andlistByTriggerIds().ThreadEntitySchema→ frontendTasktype →TaskRow, enabling navigation without an extra fetch.Written for commit 5e014a7. Summary will update on new commits. Review in cubic