[ivanmilevtues]

Preserve Core's recovery guidance for both authentication failures (exit code 2) and exhausted quotas (exit code 3). The Action prefers Core's CodeBoarding: line and uses generic guidance only when Core provides no message. Successful stdout JSON remains separate from stderr diagnostics.

CodeBoarding quota failures use the current Core wording:

Your CodeBoarding quota is exhausted. Buy a CodeBoarding license at https://codeboarding.org/pricing, or wait for your weekly reset.

Hosted upstream quota failures identify a CodeBoarding service issue instead of asking the customer to replenish our provider balance. Own-provider quota failures retain provider billing guidance. Authentication failures preserve the masked key tail or hosted workflow instructions.

The shared prefix is a named constant pointing to Core's codeboarding_cli/main.py. Subprocess tests cover all three quota messages and both own-key and hosted authentication messages, including trailing unprefixed diagnostics.

Merge/release dependency: Core #648 must be merged and released first. Before merging or releasing this Action PR, update both codeboarding== in action.yml and the engine version/provider table in scripts/action/supported-providers.json to that actual release. The current 0.15.1 pin remains unchanged because the Core fix is not released. Publishing the Action must also update the moving v1 tag.

Both branches are fix/free-credits-message. No AWS changes are required.

Quota failure previews

These existing previews show the review comment and sync summary presentation using the earlier quota wording; the exact current message is quoted above.

Validation: all 257 Action tests passed. Black formatting and git diff --check passed.

[codeboarding-review]

CodeBoarding review

Status: 1 changed component

See the full change in CodeBoarding.

graph LR
    n_Action_Workflow_Scripts["Action Workflow Scripts"]
    classDef added fill:#1f883d,stroke:#0b5d23,color:#ffffff;
    classDef modified fill:#bf8700,stroke:#7d4e00,color:#ffffff;
    classDef deleted fill:#cf222e,stroke:#82071e,color:#ffffff,stroke-dasharray:5 3;
    class n_Action_Workflow_Scripts modified;

Loading

_{download artifacts · run 39707758110}

[chatgpt-codex-connector]

Codex Review Summary

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

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-10-20T22:22:44.625830Z	1f3a36a	New commits

ℹ️ 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.

[ivanmilevtues]

Clarification: this PR is already ready for review, not a draft. The earlier draft wording was stale.

The implementation is addressed: the Action preserves Core’s quota recovery message using the existing exit code 3. The remaining dependency is release sequencing, not an unfinished error-handling change.

The Action currently installs codeboarding==0.15.1. That version does not contain Core #648’s new CodeBoarding purchase/reset guidance, so publishing this Action against that pin would still show the older Core message. Core #648 needs to be merged and released first; then action.yml and scripts/action/supported-providers.json must both be updated to the actual released version before shipping the Action. We have not guessed a future version or published a release as part of this fix.

This does not block reviewing the PR now. It blocks shipping the complete user-visible fix until the engine pin is updated.

[ivanmilevtues]

CodeBoarding review · failed

CodeBoarding quota is exhausted. Buy a CodeBoarding license at https://codeboarding.org/pricing, or wait for your weekly reset.

See the workflow logs.

---

_{Example of the quota-failure comment for this PR’s screenshot. This is a preview, not a report that this PR’s workflow exhausted its quota.}

[Svilen-Stefanov]

Looks good. Main thing: this shouldn't merge or at least shouldn't release before the codeboarding== pin bump is in. Couple of smaller nits below.

[ivanmilevtues]

Svilen, pushed the Action review fixes in 1f3a36a:

• Exit-code handling: prefer Core's recovery message for both authentication (exit 2) and quota failures (exit 3), keeping ENGINE_REFUSALS as fallback. Tests cover own-key and hosted authentication, plus the three quota messages.
• Release sequencing: this is still pending. Core #648 is open, so no released version includes the fix yet. The PR description explicitly requires merging/releasing Core and updating action.yml plus supported-providers.json together before this PR merges or releases. The existing 0.15.1 pin is unchanged.
• Shared prefix: added ENGINE_ERROR_PREFIX with a pointer to Core's codeboarding_cli/main.py. Empty messages retain fallback guidance; tests cover trailing unprefixed diagnostics. Last-match selection remains unchanged.
• Fixture/PR wording: both now match the current Core message: “Your CodeBoarding quota is exhausted. Buy a CodeBoarding license at https://codeboarding.org/pricing, or wait for your weekly reset.” Core's latest review changes use “Your CodeBoarding” to avoid the doubled prefix in logs. Hosted upstream quota failures identify a service issue on our side.
• README: removed “free” from the quota wording and distinguished hosted service failures from own-provider billing guidance.

Validation: all 257 Action tests passed; Black and diff checks passed.

Posting these responses together because GitHub rejected inline replies while this account has an existing pending review. That draft review was left unchanged.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1f3a36af77

ℹ️ 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".

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".

[ivanmilevtues]

Follow-up verification for 1f3a36a: checked the parser against the matching Core branch and ran five actual Core CLI subprocess → Action parser cases with injected SDK errors. Customer quota, hosted upstream quota, own-provider quota, hosted authentication, and own-key authentication all preserved the expected recovery guidance in failure_reason.

Both sides use the exact CodeBoarding: prefix. The Action handles exits 2 and 3; substring matching accommodates Core's logging metadata before the prefix, and the later unprefixed authentication hint does not override the detailed message. All 257 Action tests passed.

The changes are already pushed. The release dependency is unchanged: Core #648 must be merged and released, then action.yml and supported-providers.json must be updated together before merging/releasing this Action PR. The current 0.15.1 pin remains unchanged.

[Svilen-Stefanov]

LGTM. Before merging, we should probably merge the core and release it so that we bump the version here.

[ivanmilevtues]

Svilen, the implementation comments are addressed, and Core #648 is now merged. We still need to publish the Core patch release.

I have bumped this PR’s engine pin and the mirrored supported-providers.json version together from 0.15.1 to the planned 0.15.2 in 8255268. All 257 Action tests pass locally, and the diff check passes.

This pin is intentionally ahead of publication: Core 0.15.2 must be available before this Action revision can install it or be released to users. The published-package installation/compatibility check remains dependent on that release. No Core or Action release was published as part of this update.

Posting here because this account has an existing pending review, which previously blocked inline replies; that draft review is left untouched.