Repository navigation
Use public identifiers for directory pages and publishing - #416
Conversation
Codex Review Summary: ✅ CompletedThis 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. |
There was a problem hiding this comment.
All reported issues were addressed across 39 files
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 6 files (changes from recent commits).
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 10 files (changes from recent commits).
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Requires human review: Moves directory filters, public extension/developer links, and publishing fields to public identifiers, dropping developer_id query support; the resulting public-link/query compatibility break requires human sign-off.
Turn on auto-fix | Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 9 files (changes from recent commits).
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Requires human review: Switches directory filters, public extension/developer links, and publishing payloads from UUIDs to public identifiers, dropping developer_id support; the resulting public-link/query compatibility break needs human sign-off.
Turn on auto-fix | Re-trigger cubic
Send the manifest's public extension ID as `identifier` in directory creation requests. Keep the manifest ID and update paths as public identifiers, and update the publishing regressions and setup notes for UUID-based developer bindings. Companion API change: FOSSBilling/api#309. Merge alongside the API contract update before running the publisher. Directory caller update: FOSSBilling/extensions#416. Package publishing continues separately in #12.
The API now separates UUIDv7 record IDs from public identifiers. Use
identifierfor extension/profile links, filters, form values and publishing requests, while retaining UUIDs for record identity and ownership comparisons. Regenerate the API client and keep create-form redisplay independent of an assigned database ID.Requires API FOSSBilling/api#309, which has been merged. Moderator audit-history links use the immutable developer UUID; public profile and extension links use identifiers.