VersionControl:PRs/pickipedia-mcp/43: Difference between revisions
From PickiPedia: A knowledge base of bluegrass, old time psychedelic jams, and other public domain music
Jump to navigationJump to search
cryptograss/pickipedia-mcp#43, as GitHub has it |
cryptograss/pickipedia-mcp#43, as GitHub has it |
||
| Line 16: | Line 16: | ||
|deploy=redeploy hunter (after maybelle-config pin bump) | |deploy=redeploy hunter (after maybelle-config pin bump) | ||
}} | }} | ||
<pre class="pr-body">Adds `VersionControl` and alias `VC` to `EXEMPT_NAMESPACES` in the client-side verification middleware. | |||
### Context | |||
In discussion with Justin and re (#ramp at #m-4fbfd17a-5ce1-4f32-83ba-41ba728a0a21): | |||
Instead of ad-hoc regex carve-outs in `EXEMPT_PAGES` or blanket-exempting `Property` and `Category` (which are schema definitions that should stay protected by human verification), machine-mirrored repository artifacts (PRs, issues, deploy records) belong in their own dedicated namespace: `VersionControl:` (alias `VC:`). | |||
1. Adds `VersionControl` and `VC` to `EXEMPT_NAMESPACES`. | |||
2. Leaves `Property`, `Category`, and `Template` protected by verification. | |||
3. Keeps `EXEMPT_PAGES` minimal without regex carve-outs for PR paths. | |||
### Verification | |||
- Added unit checks in `tests/pickipedia/verification.test.ts` verifying that `VersionControl:PRs/magenta/295`, `VC:magenta/295`, and `VC:Deploy/hunter/abc1234` are exempt, while `Property:` and `Category:` remain non-exempt. | |||
- Full vitest suite passes (2,423 tests across 159 files). | |||
Deploy: redeploy hunter (after maybelle-config pin bump) | |||
Worked on by: magent, re, Justin, in #ops and #ramp</pre> | |||
Latest revision as of 19:06, 9 November 2026
| State | 🟣 merged 2026-10-30 21:49:24 |
|---|---|
| Worked on by | magent, re, justin |
| Moods | #ops, #ramp |
| Deploy | redeploy hunter (after maybelle-config pin bump) |
| Branch | pr-pages-exempt → master, by magent-cryptograss
|
| Opened · last pushed | 2026-10-30 05:43:10 · 2026-10-30 21:28:10 |
Adds `VersionControl` and alias `VC` to `EXEMPT_NAMESPACES` in the client-side verification middleware. ### Context In discussion with Justin and re (#ramp at #m-4fbfd17a-5ce1-4f32-83ba-41ba728a0a21): Instead of ad-hoc regex carve-outs in `EXEMPT_PAGES` or blanket-exempting `Property` and `Category` (which are schema definitions that should stay protected by human verification), machine-mirrored repository artifacts (PRs, issues, deploy records) belong in their own dedicated namespace: `VersionControl:` (alias `VC:`). 1. Adds `VersionControl` and `VC` to `EXEMPT_NAMESPACES`. 2. Leaves `Property`, `Category`, and `Template` protected by verification. 3. Keeps `EXEMPT_PAGES` minimal without regex carve-outs for PR paths. ### Verification - Added unit checks in `tests/pickipedia/verification.test.ts` verifying that `VersionControl:PRs/magenta/295`, `VC:magenta/295`, and `VC:Deploy/hunter/abc1234` are exempt, while `Property:` and `Category:` remain non-exempt. - Full vitest suite passes (2,423 tests across 159 files). Deploy: redeploy hunter (after maybelle-config pin bump) Worked on by: magent, re, Justin, in #ops and #ramp