fix: restore catalog synchronization and publish breg identifiers - #3
Conversation
Signed-off-by: Jeremi Joslin <jeremi@joslin.fr>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 27d9e0360d
ℹ️ 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".
| "artifact_sha256": "581cd380fa3c33ed81d56ef4d88a514a34ee1e7665128f3b657a7f1124e182fc", | ||
| "immutable_uri": "https://id.registrystack.org/artifacts/sha256/581cd380fa3c33ed81d56ef4d88a514a34ee1e7665128f3b657a7f1124e182fc.json" | ||
| "artifact_sha256": "6b660ca268062996a138839051ba35b306673f1720256b40419f48a7abb1c44f", | ||
| "immutable_uri": "https://id.registrystack.org/artifacts/sha256/6b660ca268062996a138839051ba35b306673f1720256b40419f48a7abb1c44f.json" |
There was a problem hiding this comment.
When a client retains one of the three immutable_uri values published by the previous release, this update makes that URL return 404 because the corresponding files under both src/artifacts/sha256/ and public/artifacts/sha256/ are deleted. These content-addressed URLs were published as immutable artifacts and may be referenced independently of the moving canonical schema URI, so add the new digest artifacts without removing the old ones.
Useful? React with 👍 / 👎.
1 participant
The daily catalog sync failed on a missing PAT, leaving all 19 Base Registry Engine problem identifiers unpublished and all three breg configuration schemas stale. Import the exact merged Registry Stack commit
3a1e05d886eb2a738f5be3154664b806b7388c54, publishing 77 active identifiers and the source-bound schema bytes.Use the sync job's temporary GitHub token with repository-scoped Contents, Pull requests, and Actions write permissions. Keep draft synchronization PRs and explicitly dispatch publisher CI on their branch, since token-created pushes do not trigger CI and PR runs may require approval. The repository must allow GitHub Actions to create pull requests. No Cloudflare credentials or deployment permissions are added to sync.
Validation: 17 publisher tests pass; exact upstream source and artifact checks pass for all 77 identifiers; generated output matches; local availability smoke and all 59 problem URLs pass. Hosted CI also checks the pinned actionlint workflow rules before merge. After deployment, verify all breg problem URLs and canonical schema hashes live, then exercise synchronization with the temporary token.