refactor(c/M12): one UUID_RE, and put the numeric-id docblock on isClioId
Two defects in the same few lines. The docblock describing numeric Clio ids sat above UUID_RE, which is neither numeric nor Clio's - the function it documents (isClioId) had none, so the comment described the wrong constant. UUID_RE also existed twice, once in the seam and once in the routes, guarding the same values. Duplicated validation regexes drift, and the copy that drifts is the one that stops guarding. The seam now exports it and the route imports it. No behaviour change; the two suites over these files stay green (90 tests). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
| Repository | Sloth-ninja/JessicaOSS |
|---|---|
| Author | sloth-ninja <ezanahaddis@gmail.com> |
| Authored | |
| Parents | 2bfa0b57 |
| Stats | 2 files changed , +15 , -9 |
| Part of | Practice Management: Clio-backed Matters and workspace links |
Capture this commit into my fork
Download a Markdown prompt that tells Claude how to port this
exact commit into your working tree. Run it via
claude -p < capture-commit-d7091d34.md
from inside the repo you want the change in.