Repository navigation
Improve verdict clarity, draft recovery, and mobile navigation - #1
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Evidence targeting and focus restoration issues remain unresolved.
Get a fresh assessment by requesting another Copilot review.
Review effort: Lite
Findings: 1
What changed in this PR
Improves verdict clarity, draft recovery, evidence navigation, and narrow-screen workspace usability.
Changes:
- Adds confidence labels, filters, details, and clearer summaries.
- Adds synthetic examples, memory-only undo, inline previews, and judged-text exports.
- Adds mobile navigation, accessibility coverage, documentation, and regressions.
| File | Description |
|---|---|
| README.md | Documents new workspace flows. |
| lib/sampleText.ts | Adds synthetic writing examples. |
| lib/results.ts | Separates low-confidence and unavailable results. |
| lib/results.test.ts | Tests updated summary behavior. |
| e2e/ux-judge.spec.ts | Covers UX and mobile regressions. |
| e2e/keyboard.spec.ts | Updates disclosure keyboard coverage. |
| e2e/judge.spec.ts | Tests filters, exports, examples, and evidence. |
| e2e/custom-checks.spec.ts | Tests inline check previews. |
| e2e/a11y.spec.ts | Extends accessibility coverage. |
| docs/superpowers/plans/2026-09-20-workspace-ux.md | Records implementation and validation. |
| CONTRIBUTING.md | Updates UI review guidance. |
| components/SourcePanel.tsx | Adds examples, undo, previews, and methodology. |
| components/ResultsPanel.tsx | Adds filters, stale handling, and evidence actions. |
| components/JudgeWorkspace.tsx | Coordinates recovery, navigation, and exports. |
| components/CheckChips.tsx | Adds inline check details. |
| components/AxisResultCard.tsx | Adds clarified verdict details and evidence location. |
| app/globals.css | Styles responsive navigation and new controls. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| onLocateEvidence={(snippet, trigger, name) => { | ||
| const editor = document.getElementById("judge-text") as HTMLTextAreaElement | null; | ||
| if (!editor) return; | ||
| const start = editor.value.indexOf(snippet); | ||
| if (start < 0) return; |
There was a problem hiding this comment.
Fixed in 950168c. The card/panel callbacks now retain Evidence.index, and the editor locates sentences in source order before selecting the indexed occurrence. A mocked live-provider regression selects sentence 3 when it duplicates sentence 1; it failed on desktop and mobile before the fix and now passes. Full verification passed again: 74 unit tests and 88 browser tests, with five expected platform skips.
Clarity Judge now makes uncertain verdicts explicit, keeps evidence close to its source, and reduces the scrolling needed to edit and review writing.
Validation:
pnpm check:secrets,pnpm lint,pnpm typecheck,pnpm test(74 passed),pnpm build, andpnpm test:e2e --workers=4(88 passed; five expected platform skips). Both themes and narrow layouts were inspected. Independent code review completed.Live-mode tests use mocked provider responses. Real-provider behavior and human VoiceOver/keyboard-only acceptance remain unverified.