Skip to content

feat(studio): timeline comments with agent resolution#936

Open
virang-nimrobo wants to merge 2 commits into
heygen-com:mainfrom
virang-nimrobo:feat/timeline-comments
Open

feat(studio): timeline comments with agent resolution#936
virang-nimrobo wants to merge 2 commits into
heygen-com:mainfrom
virang-nimrobo:feat/timeline-comments

Conversation

@virang-nimrobo
Copy link
Copy Markdown

Summary

Adds source-backed timeline comments in Studio, addressing #749 — Studio
previously had no durable way to leave timeline-specific edit requests and hand
them off to a coding agent.

  • Timeline comments persisted per-project via a new studio-api route
    (packages/core/src/studio-api/routes/timelineComments.ts) and helpers.
  • Comments tray (TimelineCommentsTray.tsx) with timeline markers on the
    ruler/canvas, plus comment authoring from the EditModal.
  • Superconnector bridge to send comments to the configured coding agent
    for resolution, with an agentRunController that manages run lifecycle.
  • Run stop support and correct status reporting — successful runs now
    resolve to "Done." rather than "Stopped." (the agent signals completion by
    removing its comment marker).

Changes

  • packages/coretimelineComments route + helpers, wired into createStudioApi.
  • packages/studio — comments tray, agent run controller, useTimelineComments
    hook, and timeline/ruler/canvas marker integration.

Closes #749

virang-nimrobo and others added 2 commits May 18, 2026 22:54
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Successful timeline-comment agent runs were reported as "Stopped."
The agent signals completion by removing its comment marker, which
useTimelineComments detected and handled by aborting the fetch via
cancel() — indistinguishable from a user cancel in the controller's
abort branch.

- Add agentRunController.complete(): aborts the run like cancel() but
  marks intent so the abort branch resolves to "Done." not "Stopped."
- Use complete() for the marker-gone completion signal.
- cancelAgentRun now always POSTs /agent/cancel (the server cancels by
  commentId regardless of owner) so a run reattached from a previous
  page load can be stopped; its stale local status is cleared.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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.

Add timeline comments with agent resolution

1 participant