Uh oh!
There was an error while loading. Please reload this page.
fix(desktop): fresh sweep on new tip — restraint consistency, bot primary, honest fade - #722
Merged
Conversation
…mary, honest fade (round 16) Post-pull discovery sweep over the externally-merged tip (LiteLLM, Goal #625, FormatJson, tool-row rework — all reviewed clean, incl. the Goal kill-switch pill which correctly absorbed the earlier review blockers). Three finds, fixed: - 每日回顾 已生成 chip still success-green — the EXPECTED outcome per the #651 restraint rule (健康 正常 and 权限 已授权 are already neutral); failures keep destructive. - 机器人对话 had NO primary at all — 测试并连接 (the form's completion action) promoted from secondary to default. - first-run provider list: the 28px bottom fade left a half-clipped row still half-legible — read as broken clipping; 56px fully veils it. Also re-learned: the desktop test suite does not compile renderer pages (a JSX syntax error sailed through 2309 green tests and only vite/ typecheck caught it) — full typecheck now part of this round's gate. Desktop 2309/2309 + typecheck + dead-css clean; CDP captures verified.
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 freeto 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.
Discovery sweep over today's externally-merged tip (LiteLLM #707, Goal #625, FormatJson #658, tool-row rework #712/#717/#721 — all reviewed clean; the Goal kill-switch pill correctly absorbed the earlier maintainer-review blockers: visible, one-click stoppable, Target icon through the funnel, mode-pill recipe).
Three finds, all fixed and CDP-verified:
Process note: a JSX syntax error passed the 2309-test suite untouched — the suite never compiles renderer pages; only vite/typecheck catches it. Full
npm run typecheckadded to this round's gate (and worth wiring into CI later).Desktop 2309/2309 + typecheck + dead-css clean.