Repository navigation
Conversation
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. |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
1 issue found and verified against the latest diff
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="src/app/index.ts">
<violation number="1" location="src/app/index.ts:28">
P1: This route change makes the existing FOSSBilling client call a dead URL: FOSSBilling/FOSSBilling/src/core/Remote/ExtensionManager.php:28 still builds detail and list requests under `/extensions/v2/`, which this deployment intentionally makes return 404. Update and deploy the installer/client in lockstep before removing the old base, or retain a temporary alias.</violation>
</file>
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
There was a problem hiding this comment.
No issues found across 24 files
Requires human review: Moves the extension v2 API base from /extensions/v2 to /v2 and removes the old path, changing a public contract with coordinated external deployment required.
Turn on auto-fix | Re-trigger cubic
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
api | 40f537d | Nov 05 2026, 06:04 PM |
Withdrawn: retain the existing
/extensions/v2/extensions/{id}layout. The service prefix and extension collection represent different scopes; keeping both avoids reserving extension IDs for service routes and changing all callers.No API routing change is needed. Badges from merged #297 retain their original URLs. FOSSBilling/extensions#408 and FOSSBilling/extensions-extra#2 use those URLs; the companion publisher and installer namespace PRs are also withdrawn.