feat: make Registry Stack authoritative for public identifiers - #710
Merged
Conversation
2 tasks
jeremi
force-pushed
the
agent/identifier-catalog
branch
3 times, most recently
from
August 11, 2026 07:52
004cadb to
b547df8
Compare
jeremi
marked this pull request as ready for review
August 11, 2026 07:54
Signed-off-by: Jeremi Joslin <jeremi@joslin.fr>
jeremi
force-pushed
the
agent/identifier-catalog
branch
from
August 11, 2026 08:03
b547df8 to
558c342
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b547df88b0
ℹ️ 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".
Signed-off-by: Jeremi Joslin <jeremi@joslin.fr>
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.
Pull Request
Summary
Make
registry-stackthe release-scoped source of truth for identifiers published underhttps://id.registrystack.org/.4 JSON Schemas, 1 namespace, 2 vocabularies, and 4 vocabulary terms.
retirement records.
and every identifier to an explicit compatibility line.
check now runs on every PR, while source-owning changes still select the full export and
reproducibility job.
files no longer affect catalog generation.
breaking pre-1.0 change; the changelog documents migration to absolute IRIs or an explicitly
declared institution-owned vocabulary.
definitions without changing runtime response behavior.
0.19.1onward to bind the catalog path, digest, andactive entry count. Historical manifests validate those bytes at their recorded source commit, so
later catalog changes cannot invalidate earlier releases. The V1 Relay/Notary release inventory
remains only for validating historical v0.10-v0.16 manifests; current v0.19+ releases use the
Relay V2 inventory.
documentation for removal and adopter-owned vocabulary semantics.
Companion publisher PR: registrystack-id#1.
It currently pins the earlier reviewed PR head,
558c3423c4cd20b73556653f80feb6a4a65dcbcc; the current PR head is9e36e67c63b95a1234853417ef234c14b394b1f6. Its catalog SHA-256 remains545c3f9d9910d5190194f8027a7adaa75e7d6fc826515ee686de34b1c2f2a467. That publisher pin isprovisional and must be replaced with the exact resulting Registry Stack
maincommit after thisPR merges.
Checks
products/identifiers/scripts/check.sh(12 generator and closure tests plus reproducibility)python3 .github/scripts/test_ci_changes.py(56 tests)python3 -m unittest release/scripts/test_registry_release.py(83 tests)python3 -m unittest release/scripts/test_registry_release_plans.py(20 tests)cargo fmt --checkcargo test --locked -p registry-manifest-core -p registry-manifest-clicargo test --locked -p registry-relaycargo test --locked -p registry-relay --all-featurescargo run --locked --profile ci -p registry-manifest-cli -- validate-profiles products/manifest/profilesproducts/relay-v2/scripts/check-contracts.shcargo test --locked -p registry-relay-v2 --all-featurescargo clippy --locked -p registry-relay-v2 --all-targets --all-features -- -D warningscd docs/site && npm run checkcd docs/site && npm test(371 tests)actionlintgit diff --checkAll passed. The explicitly network-dependent SDMX schema test remains ignored by the existing test
suite. Vale reported zero errors and zero warnings, with one existing suggestion outside this
change.
Review notes
Security review found no changed runtime principal, authorization, disclosure, audit ordering,
source access, credential, signing, route, or error-redaction boundary.
problem.rsemits thesame closed, value-free problem facts from one inventory macro.
The contracts page contains Tier-C trust-boundary wording. Maintainer source-pack sign-off against
the applicable Registry Stack security specifications is required before merge, specifically for
the claim that identifier resolution creates no runtime authentication, authorization, or
disclosure authority.
The catalog is a publication contract. Removing an entry makes the resolver return
404; aremoved URI must remain absent and must never be reused or repurposed.
DCO
Signed-off-bytrailer.