Skip to content

fix(ui): clarify run history, shared instructions, and delegate form - #526

Merged
astyfx merged 1 commit into
mainfrom
fix/task-results
Sep 11, 2026
Merged

astyfx merged 1 commit into
mainfrom
fix/task-results

Conversation

@astyfx

@astyfx astyfx commented Sep 11, 2026

Copy link
Copy Markdown
Member

Summary

Makes task run history, shared workspace instructions, and the delegate form easier to scan and cheaper to render. Run history no longer mounts every turn's evidence at once; shared instructions state when they apply; turn activity and run history show effort next to the model; the delegate form's provider/model/permission controls align and include effort.

Changes

Features

  • session: collapse run history into compact accordion rows and load evidence only for the expanded run
  • session: show dispatched effort next to the run model in turn activity and saved results
  • collaboration: add an effort control and a one-line send summary to the delegate form
  • information: state that shared instructions apply to every task, with save status, character count, and an abridgement warning

Fixes

  • collaboration: align provider, model, and permission selects on the delegate form

Refactors / Chores

  • reviews: fetch result summaries without evidence and narrow useResultReviews subscriptions to primitive signals

Breaking Changes

Test Evidence

  • bun run typecheck passed
  • bun run check:design-system passed (36 adapters; 0 utility-syntax files)
  • bun run check:max-lines-ratchet passed
  • bun run check:reliability-gates passed
  • bun run check:switch-exhaustiveness passed
  • bun test tests/delegation-drafts.test.ts tests/result-evidence.test.ts tests/result-review-store.test.ts tests/model-resolution-summary.test.tsx tests/task-result-reviews-render.test.tsx tests/workspace-resume-brief-render.test.tsx — 24 pass, 0 fail

Notes

  • Shared instructions were verified via SSR render tests; the Information panel has no Lens preview route.
  • No new theme tokens; existing semantic colors only (colorSuccessText, colorWarningText, colorSurfaceTint).
  • Optional modelInfo on saved result evidence is backward compatible with older stored rows.

Co-authored-by: Cursor <cursoragent@cursor.com>
@astyfx
astyfx requested a review from a team as a code owner September 11, 2026 03:21
@astyfx
astyfx requested review from JongSikLim and dodonmountain and removed request for a team September 11, 2026 03:21
@astyfx
astyfx enabled auto-merge (squash) September 11, 2026 03:21
@upwind-code-us

upwind-code-us Bot commented Sep 11, 2026

Copy link
Copy Markdown

Upwind Upwind Code Scan - ✅ Passed

0 newly introduced vulnerabilities · 0 resolved · 57 total in this PR vs main

Total breakdown: 🔶 6 High | 🟡 48 Medium | 🟢 3 Low

View full analysis in Upwind Console

Scan completed in 18s

Scan history (1 scan)
Commit Scanned at New Resolved Net
909b0d9 < 2026-09-11 03:21 UTC 0 0 0

Last scanned: 909b0d9 · 2026-09-11 03:21 UTC

@upwind-code-us

upwind-code-us Bot commented Sep 11, 2026

Copy link
Copy Markdown

Upwind Upwind IaC Scan - ✅ Passed

0 newly introduced misconfigurations · 0 resolved · 0 total in this PR vs main

View full analysis in Upwind Console →

Scan completed in 2s

Scan history (1 scan)
Commit Scanned at New Resolved Net
909b0d9 < 2026-09-11 03:22 UTC 0 0 0

Last scanned: 909b0d9 · 2026-09-11 03:22 UTC

@heath-s heath-s left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approved

@astyfx
astyfx merged commit ebc01f3 into main Sep 11, 2026
6 checks passed
@astyfx
astyfx deleted the fix/task-results branch September 11, 2026 03:26
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.

2 participants