Skip to content

feat(critique): benchmark intake + apply-fixes in the session (v1.5 PR9) - #70

Merged
thisiscsim merged 1 commit into
mainfrom
feat/critique-chat
Aug 5, 2026
Merged

thisiscsim merged 1 commit into
mainfrom
feat/critique-chat

Conversation

@thisiscsim

Copy link
Copy Markdown
Owner

Summary

PR9 of the v1.5 train — the critique flow from 346:11571 (benchmark submit) and 346:12768 (feedback + apply).

  • Composer critique mode: benchmark dropzone above the field (imports via benchmark:import, then benchmarks:analyze so scoring context is fresh); the text field takes the "copy and paste a link" role — an http(s)-only submission fetches the post's video through the PR5 benchmark:fromUrl path instead of running a turn.
  • Session log: user-turn attachments (references + benchmarks) render as thumbnails under the bubble; when the acting agent flips, a "Generation/Critique has joined the session" separator draws (orb + hairlines).
  • Apply the suggested changes: the critique card gets the HTR widget — a new applyCritiqueFixes store action runs auto-tune with autotune:phase streaming into a fresh assistant turn, flips the card's applied flag (persisted), and reloads the cut.
  • Tests: markCritiqueApplied (session lib) + a full applyCritiqueFixes store test (145 total).

Renderer-only — HMR covers it.

Test plan

  • typecheck / test (145) / build / lint / format — green
  • Manual: switch to Critique, drop a benchmark + paste a TikTok link, run critique, apply fixes, re-critique

Made with Cursor

…ssion

Completes the critique flow per the V1 frames. In Critique mode the
composer grows a benchmark dropzone (imports into benchmarks/ and runs
feature extraction immediately) and its field doubles as a paste-a-link
intake — URLs fetch the related video through benchmark:fromUrl instead
of running a turn. Benchmark and reference attachments render as
thumbnails under the user bubble, agent switches draw a "Critique has
joined the session" separator, and the critique card gains the "Apply
the suggested changes" widget: a store action that runs autotune with
streamed status, marks the card applied, and logs the re-tune as its own
assistant turn.

Co-authored-by: Cursor <cursoragent@cursor.com>
@thisiscsim
thisiscsim merged commit b88bfc2 into main Aug 5, 2026
1 check passed
@thisiscsim
thisiscsim deleted the feat/critique-chat branch August 5, 2026 02:57
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