Skip to content

feat(baoyu-image-gen): add Atlas Cloud provider - #190

Open
binyangzhu000-sudo wants to merge 2 commits into
JimLiu:mainfrom
binyangzhu000-sudo:codex/add-atlascloud-image-provider-20260702-1336
Open

feat(baoyu-image-gen): add Atlas Cloud provider#190
binyangzhu000-sudo wants to merge 2 commits into
JimLiu:mainfrom
binyangzhu000-sudo:codex/add-atlascloud-image-provider-20260702-1336

Conversation

@binyangzhu000-sudo

Copy link
Copy Markdown

Summary

  • Add an atlascloud provider for baoyu-image-gen with Nano Banana 2 text-to-image and edit flows.
  • Wire Atlas Cloud into provider detection, EXTEND defaults, batch rate limits, and CLI help.
  • Document Atlas Cloud usage and add provider/unit coverage for request mapping and validation.

Testing

  • node --import tsx --test $(rg --files skills/baoyu-image-gen/scripts | rg '\.test\.ts$' | sort)
  • Live smoke test with --provider atlascloud --quality normal --ar 1:1

No root README changes.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ed21e50b0c

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment on lines +326 to +327
if (args.responseFormat === "url") {
return new Uint8Array(Buffer.from(outputUrl, "utf-8"));

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Use .txt for Atlas URL outputs

When --response-format url is used with an extensionless output path, this branch returns UTF-8 URL bytes, but the Atlas provider does not export getDefaultOutputExtension, so prepareSingleTask/prepareBatchTasks normalize the path with the default .png. The command succeeds while writing a text URL into out.png, which downstream image consumers see as a corrupt image; mirror the Agnes provider by defaulting URL mode to .txt.

Useful? React with 👍 / 👎.

if (hasReplicate) return "replicate";
if (hasSeedream) return "seedream";
if (hasMinimax) return "minimax";
if (hasAtlascloud) return "atlascloud";

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Allow remote refs before Atlas auto-detection

With only ATLASCLOUD_API_KEY configured and a remote reference such as --ref https://.../ref.png, this auto-selection path is intended to choose Atlas Cloud, but both prepareSingleTask and prepareBatchTasks validate references before calling detectProvider and only allow remote URLs when args.provider is already a remote-capable provider. Since args.provider is null in auto mode, the URL is resolved as a local path and the Atlas branch is never reached; auto-detected Atlas edit workflows with remote refs fail unless users redundantly pass --provider atlascloud.

Useful? React with 👍 / 👎.

@binyangzhu000-sudo

Copy link
Copy Markdown
Author

Addressed the automated review feedback in 2baa42c: URL-only Atlas Cloud outputs now default to .txt, and remote references are validated after provider auto-detection so Atlas Cloud auto-detected edit flows can accept remote refs. Re-ran baoyu-image-gen tests: 115/115 passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant