Repository navigation
Preserve authored campaign template-variable CLI policy - #95
Conversation
|
Warning Review limit reachedThe included review limit has been reached and this organization has disabled usage-based review continuation. Wait for reviews to reset or ask a billing admin to change After included review limits.
Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing. Next included review available in 28 minutes. View limit details
|
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. |
…gn-template-policy-20261029 # Conflicts: # test/commands/campaigns.test.ts
0.1.0-dev.20261004024706 (the main pin) lacks Campaign.variableMapping and 0.1.0-dev.20261017182310 drops apiKeys and projectTokens, so pin 0.1.0-dev.20261010020302, the published build that has both. Its recipient list reason is a closed union without frequency_cap, so the CLI keeps its own string reason parameter. Refresh the CLI reference.
|
@codex review Please review the current head |
|
Codex Review: Didn't find any major issues. 🎉 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
🛡️ Codex Security Review · Automatically triggeredSecurity review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
Summary
Publish the frozen Campaigns custody candidate
e7a17e2b47189f5cf0484e416d53dc85b7a844a0, preserving its original parent, author and source bytes. It adds explicit missing-template-variable launch policy, template mapping display/help and its authored command assertion. No implementation or reconciliation was added in this publication sweep.Verification
No new tests or review. The retained patch is absent from the accepted
044flaunch-command blob, which still equals its original pre-change blob. Its original assertions remain untouched. Current-main integration, generated command reference, browser/source checks and installed/environment qualification are not established for this historical candidate.Release status
Keep draft; Release owns merge sequencing. API and SDK contract compatibility must be revalidated before integration. CLI source and README change here; package pins are not accepted by this frozen snapshot. Docs/reference adoption, feature availability and package publication remain held. Admin controls are unchanged. No activation, provider acceptance or staging deployment.
Need help on this PR? Tag
@codesmith-botwith what you need.