Skip to content

Refactor AnyGen CLI for enhanced usability#82

Open
manucian-official wants to merge 2 commits intoHKUDS:mainfrom
manucian-official:patch-3
Open

Refactor AnyGen CLI for enhanced usability#82
manucian-official wants to merge 2 commits intoHKUDS:mainfrom
manucian-official:patch-3

Conversation

@manucian-official
Copy link
Contributor

Refactor AnyGen CLI to improve functionality and readability.

Refactor AnyGen CLI to improve functionality and readability.
@sehawq
Copy link
Collaborator

sehawq commented Mar 15, 2026

Thanks for the PR — but this looks like more than cleanup. It seems to remove a large portion of the CLI surface.

From the diff, several commands/options disappear (e.g. task poll/download/thumbnail/list/prepare, config delete/path, session status, and many task create/run options like language, template, ratio, export-format, etc.). That’s a breaking change unless intentionally deprecated.

Also: the raw file for anygen_cli.py appears to render as a single line. If that reflects the actual file contents, it would cause a syntax error. Can you confirm the file content/formatting is correct?

If these removals are intentional, could you document the deprecations and rationale? Otherwise I’d suggest keeping the existing commands/options.

@manucian-official
Copy link
Contributor Author

Thanks for taking the time to review the PR and for pointing this out.

You're right that the current diff appears larger than a simple cleanup. The intention of this change was internal refactoring and optimization only, not to remove or alter the public CLI surface.

If some commands or options appear to be missing in the diff (e.g. task poll, task download, task thumbnail, task list, task prepare, config delete, config path, session status, or options like language, template, ratio, export-format, etc.), that was not intentional. These commands are still meant to be supported, and I will review the patch to make sure the full CLI surface is preserved.

Regarding the anygen_cli.py formatting: the file should definitely not be a single line. This may be an issue with how the file was rendered in the diff or how it was generated/exported. I'll verify the actual file contents and push a corrected commit if necessary.

I'll follow up shortly with an updated patch that:

  • restores the full set of CLI commands and options,
  • keeps the intended refactoring/optimization changes,
  • and ensures the file formatting is correct.

Thanks again for catching this.

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.

2 participants