Repository navigation
docs(adr): record the metadatastician package-owner widening as ADR 0003 - #110
Merged
Merged
Conversation
Layer 2b of ci/validate.jl failed any package whose Package.toml repo was not a hyperpolymath slug. The owner's second organisation, metadatastician, now holds the Marid repositories (D54-A), and the Marid Tier-1 packages are to register here (D321). Split the single owner constant in two. REGISTRY_SLUG_OWNER keeps Registry.toml's own repo field pinned to hyperpolymath, where the registry lives. PACKAGE_SLUG_OWNERS is the closed set a package repo may name: hyperpolymath or metadatastician. Any other owner still fails, and the error names both allowed owners. Move the check_repo_slug! docstring onto that function; it sat above extract_github_slug, which now has its own. Update REGISTRY-POLICY and replace its stale "37 packages" count. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016XJoh8YA2sCWQmq93CF8RX
Contributor
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
📒 Files selected for processing (4)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
11 of 15 tasks
#109 shipped the Layer 2b widening (EXPECTED_SLUG_OWNER for the registry's own repo, ALLOWED_PACKAGE_OWNERS for packages), so this branch drops its parallel constants and takes main's ci/validate.jl, docs/REGISTRY-POLICY.adoc and CHANGELOG.adoc. What remains: - docs/decisions/0003-package-repos-in-metadatastician.adoc, rewritten to record the decision #109 implemented, under main's names; - the set of allowed package owners is closed: adding an organisation takes an ADR (REGISTRY-POLICY, and the comment on ALLOWED_PACKAGE_OWNERS); - one CHANGELOG [Unreleased] Added entry for the ADR. No code change: validate.jl differs from main only in that comment. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016XJoh8YA2sCWQmq93CF8RX
|
6 of 12 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
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
#109 registered the nine Marid Tier-1 packages and, in the same change, widened
ci/validate.jlLayer 2b so a packagerepomay be owned byhyperpolymathormetadatastician. It carried no ADR. This PR adds that ADR. It records the decision #109 implemented and makes one rule explicit: the set of allowed owners is closed, so adding an organisation takes another ADR.It contains no code change.
ci/validate.jldiffers frommainonly in the comment onALLOWED_PACKAGE_OWNERS.On 2026-10-09 the owner ruled that the metadatastician widening be recorded in its own PR with ADR 0003. This PR was opened before #109 merged, with its own implementation. When #109 landed first, I merged
mainin, kept main's code (EXPECTED_SLUG_OWNER,ALLOWED_PACKAGE_OWNERS,allowed_package_owner) and dropped my parallel constants. What remains is the record.It changes registry policy text, so it needs BDFL review. Automerge is not armed.
Note
Merged 2026-10-09T00:26:21Z as
991a5b7(signed byweb-flow), after #107 (e8383fb). Before merging, GitHub mergedmaininto the branch ascdc8e9b3, somainis #107 plus exactly the four files below (+85/−2). This body was updated after the merge.On
main991a5b7,ci/validate.jlgives rc=0: 41 packages, 0 custom errors and 0 owner mismatches. ADR numbers do not collide: #107 carries ADR 0002 and this PR carries ADR 0003.Changes
docs/decisions/0003-package-repos-in-metadatastician.adoc(new):repostayshyperpolymath;7c52894as the implementation.docs/REGISTRY-POLICY.adoc: one sentence after the Layer 2b paragraph says the set is closed and links ADR 0003.ci/validate.jl: the comment onALLOWED_PACKAGE_OWNERSgains the same sentence. No executable line changes.CHANGELOG.adoc[Unreleased] → Added: one entry for the ADR, directly below feat(registry): register the nine Marid Tier-1 packages at v0.1.0 (D321/D322) #109's Marid entry.📌 New pins
cdc8e9b3dbec0bc7aea480e81c667047c26294a2. This is GitHub's "Update branch" merge ofmaine8383fb(feat(registry)!: re-mint ten placeholder UUIDs as UUIDv8 and re-register (ADR 0002) #107) into8d7eeb36, my signed merge ofmain7c52894. The branch's first commit,a4a4bef, is also signed.actions.lock, lockfile or container digest changes.RSR Quality Checklist
Required
CC-BY-SA-4.0, matching ADR 0001..envfiles.mainat7c52894too (and at9e4dc83before it), so this PR does not cause them. They are deferred below.Deferred checks (§5c)
actions.lock is in sync with the workflow YAML: deferred to CI: actions.lock out of sync after Dependabot #106 bumped smtp-notify-action to v0.5.0 #108. Dependabot chore(deps): bump hyperpolymath/smtp-notify-action from 0.3.0 to 0.5.0 in the actions group #106 bumpedsmtp-notify-actionto v0.5.0 without refreshing.github/workflows/actions.lock.governance / Allowlist Preflight: deferred to CI: governance linter calls a missing scripts/update-actions-lock.sh; Allowlist Preflight runs with an empty GH_TOKEN #105.GH_TOKENis empty in the step, so it reportscould not read live Actions permissions.governance / Workflow security linter: deferred to CI: governance linter calls a missing scripts/update-actions-lock.sh; Allowlist Preflight runs with an empty GH_TOKEN #105.scripts/update-actions-lock.sh: No such file or directory, exit 127.As Applicable — Registry Changes
Registry.tomlindex updated: N/A. No package is added, removed or rebound.As Applicable — Documentation / Infra Changes
.machine_readable/6a2/*.a2ml: N/A. A2ML is retired estate-wide (D99/D269c), so no.a2mlfile is created or edited.TOPOLOGY.md: N/A. No architectural change.CHANGELOG.adoc.Testing
Every run is
ci/validate.jl <tree>on Julia 1.10.10 with RegistryCI v11.0.0, against agit ls-filescopy of this head. Each copy is run withGIT_CEILING_DIRECTORIESset, so no enclosing repository'soriginleaks into the slug comparison.metadatastician/marid)repo→evilcorp/marid(planted negative)MaridIR: Package.toml repo resolves to slug "evilcorp/marid", expected an owner in ("hyperpolymath", "metadatastician")hyperpolymath-owned"repoowners of the 32 non-MaridPackage.tomlfiles onmain: 32 ×hyperpolymathmain's pre-#109 validator (9e4dc83): rc=1, 9 owner mismatchesThe only executable file touched is
ci/validate.jl, and only in a comment.standards/.githooks/docstring-scan.shskips Julia files, so it gives no verdict here, and no function is added or changed.🤖 Generated with Claude Code
https://claude.ai/code/session_016XJoh8YA2sCWQmq93CF8RX