Skip to content

Use public identifiers for directory pages and publishing - #416

Merged
admdly merged 6 commits into
mainfrom
codex/directory-identity
Nov 7, 2026
Merged

admdly merged 6 commits into
mainfrom
codex/directory-identity

Conversation

@admdly

@admdly admdly commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

The API now separates UUIDv7 record IDs from public identifiers. Use identifier for 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.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 7, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
🔒 Security Review ✅ Completed 2026-11-07T17:26:08.886388Z 9a1e33b PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Nov 7, 2026 •

Copy link
Copy Markdown

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://codex-directory-identity-extensions.fossbilling.workers.dev (commit ea4b4ea)

This URL reflects your latest Preview deployment

Preview Deployments by commit

Status Deployment URL Commit Updated (UTC) See this deployment's details
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://25208fa0-extensions.fossbilling.workers.dev ea4b4ea 2026-11-07T18:22:09.917Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://335ce04c-extensions.fossbilling.workers.dev 4ee17e7 2026-11-07T18:17:42.105Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://9e96b465-extensions.fossbilling.workers.dev c7cffb7 2026-11-07T18:10:01.442Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://3eaff56a-extensions.fossbilling.workers.dev 7e1e4cd 2026-11-07T18:05:15.269Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://9a7a1754-extensions.fossbilling.workers.dev 9971ec3 2026-11-07T17:39:33.584Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://1e692189-extensions.fossbilling.workers.dev 9a1e33b 2026-11-07T17:22:52.470Z Visit the dashboard ↗

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 39 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread src/pages/index.astro Resolved
Comment thread src/lib/directory-filters.ts Resolved
Comment thread test/extension-view.test.ts Resolved
Comment thread test/helpers/directory-fixtures.ts Resolved
Comment thread README.md Resolved
Comment thread src/lib/extension-form.ts Resolved
Comment thread src/lib/developer-form.ts Resolved

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@admdly admdly self-assigned this Nov 7, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Comment thread test/extension-view.test.ts Resolved

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Comment thread test/extensions-data.test.ts Resolved

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@admdly
admdly merged commit 255716b into main Nov 7, 2026
8 checks passed
@admdly
admdly deleted the codex/directory-identity branch November 7, 2026 18:23
admdly added a commit to FOSSBilling/modules that referenced this pull request Nov 7, 2026
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant