What's Changed
Mermaid files and the renderer registry
.mermaid and .mmd files open with a Diagram / Source toggle. In files mode they open on Diagram; in git mode they open on Source, which is the diff. Click a node, edge label or actor to comment on the line that defines it. Expand opens the diagram fullscreen with pan and zoom.
This runs on a new renderer registry, and Mermaid fences in markdown now use it too. New renderers (for example Arazzo workflows) are added to Crit the same way. Crit does not load renderer scripts from outside its embedded assets. See docs/renderers.md. Mermaid is now on v12.
Find in diff
The review list only keeps rows near the viewport in the page, so the browser's own find missed most matches. Cmd+F (Ctrl+F off macOS) now opens a find bar that searches all files, loads lazy files in the background, and jumps to each match. Enter / Shift+Enter or Cmd+G / Shift+Cmd+G step through matches, Alt+C toggles match case, and Esc closes. Select text first to fill the bar. Press Cmd+F a second time to get the browser's find. Plan and Document reviews keep the browser's find.
- feat(web): add a pluggable renderer registry with mermaid files render (#1043) by @akaguny in https://github.com/tomasz-tomczyk/crit/pull/1043 - Thank you!
- feat: find in diff that reaches rows outside the viewport (#1059) by @tomasz-tomczyk in https://github.com/tomasz-tomczyk/crit/pull/1059 - Thank you @radeusgd for reporting!
- fix(web): Esc peels one layer with find open, cache rendered diagrams (#1066) by @tomasz-tomczyk in https://github.com/tomasz-tomczyk/crit/pull/1066
Integrations
- feat(integrations): add omp review skills (#1038) by @endophysics in https://github.com/tomasz-tomczyk/crit/pull/1038 - Thank you!
- feat(integrations): add OMO review skills (#1022) by @endophysics in https://github.com/tomasz-tomczyk/crit/pull/1022 - Thank you!
- fix: plan-hook tells the agent to resubmit the plan, not run crit plan (#1041) by @tomasz-tomczyk in https://github.com/tomasz-tomczyk/crit/pull/1041 - Thank you @mariaines for reporting!
- fix(integrations): keep reply explanations in Crit, not chat (#1042) by @tomasz-tomczyk in https://github.com/tomasz-tomczyk/crit/pull/1042
Fixes
- fix(web): keep focus in a reply draft when comments change (#1051) by @schrek1 in https://github.com/tomasz-tomczyk/crit/pull/1051 - Thank you!
- fix: stop test helpers from clearing GIT_* env vars in the crit binary (#1064) by @tomasz-tomczyk in https://github.com/tomasz-tomczyk/crit/pull/1064
Dependencies
- chore(deps): bump mermaid from 11.17.2 to 12.1.0 (#1047) by @dependabot in https://github.com/tomasz-tomczyk/crit/pull/1047
- chore(deps): bump @pierre/diffs from 1.5.1 to 1.5.2 (#1048) by @dependabot in https://github.com/tomasz-tomczyk/crit/pull/1048
- chore(deps): bump dompurify from 3.4.15 to 3.4.16 (#1016) by @dependabot in https://github.com/tomasz-tomczyk/crit/pull/1016
- chore(deps-dev): bump shiki from 4.4.3 to 4.5.0 (#1044) by @dependabot in https://github.com/tomasz-tomczyk/crit/pull/1044
- chore(deps-dev): bump eslint from 10.11.0 to 10.12.0 (#1045) by @dependabot in https://github.com/tomasz-tomczyk/crit/pull/1045
- chore(deps-dev): bump stylelint from 17.15.0 to 17.16.0 (#1046) by @dependabot in https://github.com/tomasz-tomczyk/crit/pull/1046
Internal refactors
- test: finish test isolation from #1053 and #1056, drop dead hint code (#1065) by @tomasz-tomczyk in https://github.com/tomasz-tomczyk/crit/pull/1065
- test(e2e): merge E2E specs to cut page loads in half (#1055) by @tomasz-tomczyk in https://github.com/tomasz-tomczyk/crit/pull/1055
- test: stop git auto-maintenance in test repos (#1056) by @tomasz-tomczyk in https://github.com/tomasz-tomczyk/crit/pull/1056
- test(e2e): start each project when its own fixture is ready (#1054) by @tomasz-tomczyk in https://github.com/tomasz-tomczyk/crit/pull/1054
- test: speed up and isolate the Go unit tests (#1053) by @tomasz-tomczyk in https://github.com/tomasz-tomczyk/crit/pull/1053
Contributors