Repository navigation
Fix manifest migration for existing extension releases - #296
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. |
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
api | 9d222b4 | Nov 05 2026, 03:38 PM |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
All reported issues were addressed across 6 files
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 4 files (changes from recent commits).
Requires human review: Fix for failed manifest migration 0029: legacy minimums become lower-bound constraints, PHP optional with runtime-inheritance semantics, plus tests/docs. Decisive issue: missing PHP inherits runtime requirements rather than requiring explicit review.
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: Migration fix makes requires.php optional with runtime inheritance and converts legacy minimums to lower bounds, changing the v2 publishing contract and production data migration policy; needs human sign-off.
Turn on auto-fix | Re-trigger cubic
Production deployment stops at migration 0029 because all 45 existing releases lack the new requirements object. Convert their recorded FOSSBilling minimums to lower-bound constraints and make PHP optional: omission inherits the installed FOSSBilling version’s runtime requirements; an explicit constraint adds a restriction.
Preserve existing constraints and carry verified PHP requirements from the BTCPay 0.1.3–0.1.5 and OpenReceive 0.4.29 packages. Invalid supplied PHP values still fail validation. Migration rollback, audit history, and resource accounting remain covered.
Includes the previously failed migration because it rolled back in production. The directory companion is in FOSSBilling/extensions#407; deploy the API fix first.