Add health-check coverage to domains/rate-review/setup-info.toml so that
mise run dev:doctor rate-review surfaces the status of every rate-review
service and infrastructure dependency -- matching the pattern already in place
for the select domain.
PR #2583 · final commit 1d253619
- Documents has a distinct processing/syncing case — rows are empty not because there are no documents, but because uploads haven't resolved yet. That's not "no documents yet," and collapsing it there was wrong.
- The five loosely-related props (
emptyHeading,emptySubtext,emptyAction,isFiltered,onClearSearch) push inference to call sites and let filter-empty silently fall through to true-empty copy or CTAs.
the current hard validation requiring Fee % to sum to exactly 100% is blocking us from making any edits or adding new law firms. even with Law Firm Admin and Tekmir Admin permissions, the system triggers a "Failed to update law firm" error because the total doesn't hit 100% mid-edit.
Diffly is a VS Code extension for reviewing git diffs. The Reviewer's Cockpit turns AI-generated review findings into a triageable workflow inside the panel: per-finding Agree / Disagree / Dismiss / Asked chips, an "Ask Author" list of absences (missing tests, missing rate limits, etc.), a suggested review order, and a one-click batched GitHub review post.
The cockpit is driven by a single file at the repo root: .diffly/findings.json. Diffly's file watcher picks up changes automatically — no reload needed.
This gist documents the JSON schema, the prompt for generating it with any coding agent, and a few examples.
Captured: 2026-05-04
PR: #630 — Modernization of bid /setup/ pages
PRE worktree: .worktrees/jus-1204-pre @ origin/main (15d626697d)
POST branch: alexlazarian/JUS-1204-followup-select (HEAD)
Date: 2026-05-01
Branch: alexlazarian/JUS-1204-followup-select
Scope: Confirm the TinyMCE → Tiptap migration introduces no user-visible regressions across the Select domain (15 editor surfaces, the shared message-form gem, the bidder text-answer flow, three Rails apps, and three email templates).
The plan is split into two phases. Before merge is everything a person on a dev laptop with the local stack running can confirm (build, click-through, dev mail-catcher). After deploy is everything that needs the staging environment, real mail clients, real production-shape data, or a longer soak window.
Tick a checkbox only when you have personally observed the expected behaviour or output. If a check is non-applicable (e.g. you have no permissions on a particular surface), strike it through with a note rather than ticking it.