Skip to content

Fix discard confirmation for evolution review#268

Draft
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/ENG-523-linear-issue-triage-automation-7d2f
Draft

Fix discard confirmation for evolution review#268
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/ENG-523-linear-issue-triage-automation-7d2f

Conversation

@cursor
Copy link
Copy Markdown
Contributor

@cursor cursor Bot commented Jun 1, 2026

Summary

  • Updates the evolution review Discard confirmation copy to match the Linear acceptance criteria.
  • Adds widget coverage that Discard is visible/enabled alongside Build & Test and opens the required confirmation prompt.

Resolves ENG-523

Test Plan

  • bun -F native test:unit -- widget.test.tsx
  • bun -F native lint
  • bun -F native build

Docs

  • Docs updated (companion PR in darkmatter/nixmac-web: #___)
  • No docs update needed
Open in Web View Automation 

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag @codesmith with what you need. Autofix is disabled.

Co-authored-by: cooper <czxtm@users.noreply.github.com>
Co-authored-by: cooper <czxtm@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 1, 2026

Warnings
⚠️ PR is marked WIP / draft — do not merge until ready for review.
⚠️ Please assign this PR to someone (usually yourself).

📋 PR Overview

Lines changed 61 (+59 / -2)
Files 0 added, 2 modified, 0 deleted
Draft / WIP yes
Has Test Plan yes
New UI components no
New Storybook stories no
New Rust modules no
New TS source files no
New tests no
package.json touched no
Cargo.toml touched no
Infra / CI touched no

🔬 Coverage

Report Lines Statements Functions Branches
apps/native/coverage/coverage-summary.json 21.9% 21.9% 33.6% 57.7%

Generated by 🚫 dangerJS against 9968981

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 1, 2026

Warnings
⚠️ PR is marked WIP / draft — do not merge until ready for review.
⚠️ Please assign this PR to someone (usually yourself).

📋 PR Overview

Lines changed 61 (+59 / -2)
Files 0 added, 2 modified, 0 deleted
Draft / WIP yes
Has Test Plan yes
New UI components no
New Storybook stories no
New Rust modules no
New TS source files no
New tests no
package.json touched no
Cargo.toml touched no
Infra / CI touched no

🔬 Coverage

Report Lines Statements Functions Branches
apps/native/coverage/coverage-summary.json 21.9% 21.9% 33.6% 57.7%

Generated by 🚫 dangerJS against 9968981

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