Normalize formatting line endings#15
Merged
Merged
Conversation
This was referenced Jul 21, 2026
BigSimmo
added a commit
that referenced
this pull request
Jul 22, 2026
…solve #15) (#1057) * test(clinical): content-first + governance-badge regression tests (#15) Cover the content-first transitions flagged in #1024's CodeRabbit review for the two record loaders, locking in the clinical-safety invariant that no authoritative verification badge is asserted before live governance reconciles: - registry-record-loader.dom.test.tsx (8 tests): fallback paint during loading, the fixture-flag neutralization on the provisional paint, spinner when no fallback, live swap-in reconciling the badge on/off against governance, passthrough when governance is absent, and not_found / unauthorized states. - medication-record-page.dom.test.tsx (6 tests): fallback paint vs skeleton, live swap-in, error-empty panel, and the governance-drop-on-error invariant (a fixture "Reviewed" badge must not persist after an error) contrasted with the in-flight loading state that trusts the SSR governance. Test-only; no source/behaviour change. 14/14 pass under the jsdom project. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * docs(issues): resolve #15 — content-first regression tests added Move #15 to Resolved/archive; covered by the two new jsdom suites in this PR. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
BigSimmo
added a commit
that referenced
this pull request
Jul 22, 2026
BigSimmo
pushed a commit
that referenced
this pull request
Jul 22, 2026
Resolve the merge conflict against main's concurrent archive edits (keep both the #15 and #4 resolved rows), and address the review finding: archiving #4 (provider-gated RAG safety ideas) while process-hardening.md still advertised the same rescope as pending left conflicting durable guidance. Update that bullet to record the closure (rescue source unrecoverable; thresholds + deep-health already on main; cost-cap preflight dropped) so /issues and the process-hardening doc agree. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LRZ1EyBZW1ADXrZvMxEEsC
4 tasks
BigSimmo
added a commit
that referenced
this pull request
Jul 22, 2026
11 tasks
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 free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
.gitattributespolicy so text files are checked out with LF endings.core.autocrlf=trueinteraction that caused localnpm run format:checkfailures without a content diff.Verification
Local verification under Node 22:
npm run format:checkpassednpm run lintpassednpm run typecheckpassednpm run testpassed: 39 files, 240 testsnpm run buildpassednpm run test:e2e:chromiumpassed: 20 testsgit diff --check HEAD --passedRisk notes