[sajor2000]

What

Locale packs for the data-boundary guard (#146): national identifier shapes as pure data, UK first as the template pack.

• hooks/locales/uk.json — NHS number (mod-11), Scottish CHI (mod-11 + DDMMYY plausibility), UK unit postcode (quasi-identifier), plus column-name patterns (nhs_number, chi, postcode, ...).
• Opt-in: phi_environment.locale: uk in config.local.yaml. Default unchanged (built-in US shapes); locale checks ride on US defaults, never replace them.
• The checksum rule doubles as the fixtures rule: a match counts as an identifier only when its checksum validates, so checksum-INVALID synthetic values (every test fixture) pass by design and nothing committed can name a real person.
• Malformed or unknown pack fails closed (same posture as a malformed identifier index); a bare locale: key is inert.
• Locale columns join the aggregate-table header scan; the shared identifier block stays byte-identical across the hook, indexer, and schema-peek copies (the contract test is untouched and passing).
• Residual (noted, follow-up material): schema-peek's own header masking learns locale columns in a later pass if a UK project shows up; the hook blocks those tables regardless.

Closes #146

Validation

• bun test tests/data-boundary-locale.test.ts — 13 tests: pack shape, validator truth tables, egress denies (valid NHS/CHI/postcode), checksum-invalid controls pass, US defaults still fire, opt-in explicitness, unknown-locale fail-closed, content-scan deny on a UK-column CSV upload
• bun test — full suite green (1762 tests), shared-block parity test passing
• bun run release:validate — in sync

[coderabbitai]

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

• Configuration used: Organization UI
• Review profile: CHILL
• Plan: Essentials
• Run ID: f66a4ba5-2420-4ac5-b89f-485778a55a7b

📥 Commits

Reviewing files that changed from the base of the PR and between af7dc69 and e75c079.

⛔ Files ignored due to path filters: 2

• demo/bcg-meta-analysis/output/effects.csv is excluded by !**/*.csv
• demo/bcg-meta-analysis/output/forest.png is excluded by !**/*.png

📒 Files selected for processing: 7

• demo/bcg-meta-analysis/README.md
• demo/bcg-meta-analysis/analysis.R
• demo/bcg-meta-analysis/output/summary.md
• plugins/methodsmith/README.md
• plugins/methodsmith/hooks/data-boundary-guard.py
• plugins/methodsmith/hooks/locales/uk.json
• tests/data-boundary-locale.test.ts

+---------------------------------+
| WMD: Weapons of Mass Debugging. |
+---------------------------------+
  \
   \     /)/)
        ( '')
        /  |
      o(___)

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

_{Comment @coderabbitai help to get the list of available commands.}