Repository navigation
Unify extension email settings and deployment setup - #315
Conversation
Deploying Preview to Cloudflare (api)Preview URL: https://codex-package-deployment-setup-api.fossbilling.workers.dev/ (commit
|
| Status | Deployment URL | Commit | Updated (UTC) | Details |
|---|---|---|---|---|
| Build: Success ✅ Deployment: Success ✅ |
https://87e8e7f2-api.fossbilling.workers.dev/ | 2803bc5 |
2026-11-14T10:04:01.047Z | View logs ↗ |
| Build: Success ✅ Deployment: Success ✅ |
https://b9e90745-api.fossbilling.workers.dev/ | ea4b551 |
2026-11-14T09:38:39.597Z | View logs ↗ |
| Build: Success ✅ Deployment: Success ✅ |
https://564a1a5f-api.fossbilling.workers.dev/ | 8329b15 |
2026-11-14T08:50:21.589Z | View logs ↗ |
| Build: Success ✅ Deployment: Success ✅ |
https://e3fe153b-api.fossbilling.workers.dev/ | 7daa4e5 |
2026-11-13T02:05:21.107Z | View logs ↗ |
| Build: Success ✅ Deployment: Success ✅ |
https://40cd628c-api.fossbilling.workers.dev/ | 8ba0941 |
2026-11-13T01:39:55.034Z | View logs ↗ |
| Build: Success ✅ Deployment: Success ✅ |
https://8a9a2120-api.fossbilling.workers.dev/ | eec1257 |
2026-11-12T10:44:04.374Z | View logs ↗ |
| Build: Success ✅ Deployment: Success ✅ |
https://54824b1d-api.fossbilling.workers.dev/ | 14f4985 |
2026-11-10T16:51:38.455Z | View logs ↗ |
| Build: Success ✅ Deployment: Success ✅ |
https://7e1759b0-api.fossbilling.workers.dev/ | 75c1eb4 |
2026-11-10T16:08:22.597Z | View logs ↗ |
| Build: Success ✅ Deployment: Success ✅ |
https://ed3f3e62-api.fossbilling.workers.dev/ | ff3179a |
2026-11-10T14:13:17.772Z | View logs ↗ |
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
Security findingsAdvisory findings (1)
ℹ️ 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.
No issues found across 1 file
Auto-approved: Documentation-only update clarifying deployment token permissions, preview/staging secrets, and frontend transport setup; no code or behavior changes, so the change is bounded and low-risk.
Turn on auto-fix | Re-trigger cubic
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
🛡️ Codex Security Review · Automatically triggered
Here are some automated security review suggestions for this pull request.
Reviewed commit: 75c1eb4a9c
ℹ️ About Codex security reviews in GitHub
This is an experimental Codex feature. Security reviews are triggered when:
- You comment "@codex security review"
- A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review
Once complete, Codex will leave suggestions, or a comment if no findings are found.
There was a problem hiding this comment.
0 issues found across 2 files (changes from recent commits).
Auto-approved: Documentation-only PR adding explanatory workflow comments and expanding README deployment setup guidance. It changes no workflow behavior or security configuration, so the change is bounded and clearly beneficial.
Turn on auto-fix | Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 1 file
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
Dismissed because Cubic found issues in a newer review.
admdly
changed the title
There was a problem hiding this comment.
All reported issues were addressed across 10 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Turn on auto-fix | Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 10 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 1 file (changes from recent commits).
Requires human review: Renames email configuration to EXTENSIONS_EMAIL_* across code, tests, Wrangler config and docs; deployment depends on a dashboard variable migration that is not visible in the diff.
Turn on auto-fix | Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 4 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: Renames email env vars to EXTENSIONS_EMAIL_* across code, tests, and configs; deployment depends on an external dashboard-variables migration claimed complete but not visible in the diff.
Turn on auto-fix | Re-trigger cubic
Rename extension email configuration to the provider-neutral
EXTENSIONS_EMAIL_*names across both adapters, existing tests, examples, Wrangler configuration and generated bindings. MXroute reads server, username and password; Resend reads the same password setting as its API key. From and Reply-To remain optional shared settings.Refresh deployment guidance for the trusted-maintainer model, required Preview App credentials, aligned Node versions and the tested local-OAuth artifact deployment for dashboard-related domain-metadata failures.