Fix repo-wide prettier format:check drift on main

[BigSimmo]

Summary

npm run format:check (prettier --check .) was failing repo-wide on main, so the PR-template format:check box failed for every PR regardless of that PR's own diff. This restores a green check.

The failing set is churn from recent merges landing code that was not prettier-clean. Rebased onto current main and split into two commits:

• chore(format) — untrack + ignore .impeccable/, a machine-local tooling hook cache (session ids, timestamps, local absolute paths) that was git-tracked by mistake and fails prettier --check. Added to .prettierignore and .gitignore. (database.types.ts is already handled by .prettierignore on main, so it needs no change here.)
• style — mechanical prettier --write . of the 17 tracked files that were not prettier-clean on current main. Formatting only, no logic changes.

Verification

• npm run format:check — green (All matched files use Prettier code style!)
• npm run typecheck — passes
• npm run verify:ui / verify:release / check:production-readiness / check:deployment-readiness — N/A (no UI, runtime, env, or deploy behavior changed)

Clinical Governance Preflight

N/A — formatting-only change. Some reformatted files live under ingestion / privacy / auth / worker paths (src/lib/privacy.ts, src/lib/supabase/auth.ts, worker/main.ts, src/app/api/**), but only whitespace / line-wrapping / clarifying-parens changed — no logic, control flow, source-verification, privacy, Supabase target, service-role access, or SaMD-classification behavior was modified.

Notes

• .impeccable/hook.cache.json contained machine-local absolute paths and session ids; untracking + gitignoring it removes per-machine churn from the repo.
• Rebased onto current main (151e4e3f2); force-pushed with --force-with-lease.

🤖 Generated with Claude Code

[BigSimmo]

@copilot resolve the merge conflicts in this pull request

[Copilot]

@copilot resolve the merge conflicts in this pull request

Done. Merged origin/main into this branch, resolved the .prettierignore conflict (kept both the database.types.ts and .impeccable/ ignore entries), then ran prettier --write on the 13 files from the merged commits that weren't yet formatted. prettier --check . is green again.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8822c2033e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Update source-string assertion after formatting

When the test suite is run with dependencies installed, this assertion now searches for a one-line normalizeStructuredVisualProfile(redactCaptionMetadataValue(...), { call, but the same commit reformatted worker/main.ts so that call starts with normalizeStructuredVisualProfile( and puts redactCaptionMetadataValue(...) on the next line. The old substring is no longer present, so tests/worker-visual-capture.test.ts fails even though the intended behavior remains unchanged; update the assertion to match the multiline format or check for smaller stable substrings.

Useful? React with 👍 / 👎.

[BigSimmo]

Superseded by #180 (merged). #180 does the same repo-wide prettier cleanup and adds the durable fix this PR was missing: a Format check step in the CI verify job so drift can't re-accumulate. It also resolves the reason a format-only cleanup keeps failing verify — tests/worker-visual-capture.test.ts asserts on the exact source text of worker/main.ts, which prettier rewraps; #180 carves that one file out of prettier. Closing as redundant. Thanks!

[BigSimmo]

Superseded by #180 (already merged to main), which enforces prettier --check as a required CI gate in the verify job, reformats the same files, and untracks .impeccable/hook.cache.json.

This standalone cleanup is now redundant and conflicts with main, so closing it. The format:check-on-main problem is already resolved on main, and the new required gate prevents recurrence.