Skip to content

Conversation

@christian-byrne
Copy link
Contributor

@christian-byrne christian-byrne commented Jun 4, 2025

Adds GitHub workflow to automatically generate TypeScript types from ComfyUI-Manager's OpenAPI specification, replacing manually maintained API types and ensuring consistency with upstream API changes.

Fixes #4069

┆Issue is synchronized with this Notion page by Unito

Implements automated type generation from ComfyUI-Manager's openapi.yaml to replace manually maintained API types and ensure consistency with upstream changes.
@christian-byrne christian-byrne requested a review from a team as a code owner June 4, 2025 09:01
- Uses [email protected] for consistent setup
- Adopts same git config and branch handling as browser expectations workflow
- Works within ComfyUI_frontend directory structure for proper paths
@christian-byrne
Copy link
Contributor Author

- Uses same setup pattern: checkout, Node.js setup, npm ci
- Generates types in main repo directory structure
- Checks for changes before creating PR
- Uses peter-evans/create-pull-request for automated PR creation
- Includes proper permissions for contents and pull-requests
- Removes incorrect ComfyUI_frontend_setup_action usage
@christian-byrne
Copy link
Contributor Author

Going to merge so I can start testing it and seeing and see how difficult it will be to fully replace manual types with the generated ones.

@christian-byrne christian-byrne merged commit 841e3f7 into main Jun 4, 2025
10 checks passed
@christian-byrne christian-byrne deleted the feat/manager-types-openapi-workflow branch June 4, 2025 11:31
@webfiltered
Copy link
Contributor

Sounds good! We should not worry about merging new dispatch workflows.

imo In the event that it has to be left incomplete (e.g. to focus on another project), we should create a tracking issue, or just revert until ready.

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.

Use openapi-driven types for ComfyUI-Manager API

3 participants