Skip to content

feat(wfctl): add wfctl api extract and wfctl ui scaffold commands #165

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
intel352 merged 5 commits into from
Feb 25, 2026
Merged

feat(wfctl): add wfctl api extract and wfctl ui scaffold commands #165

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b67de36
feat: add step.build_from_config CI/CD pipeline step
intel352 Feb 25, 2026
090c9cf
feat: add step.statemachine_transition and step.statemachine_get pipe…
intel352 Feb 25, 2026
e94f433
feat: add cache.redis module and step.cache_get/set/delete pipeline s…
intel352 Feb 25, 2026
e91abec
feat(wfctl): add wfctl api extract command for offline OpenAPI 3.0 sp…
intel352 Feb 25, 2026
a2e9701
feat(wfctl): add wfctl ui scaffold command for Vite+React+TypeScript …
intel352 Feb 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
530 changes: 530 additions & 0 deletions cmd/wfctl/api_extract.go
View file
Open in desktop

Large diffs are not rendered by default.

583 changes: 583 additions & 0 deletions cmd/wfctl/api_extract_test.go
View file
Open in desktop

Large diffs are not rendered by default.

4 changes: 4 additions & 0 deletions cmd/wfctl/main.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,10 @@ var commands = map[string]func([]string) error{
"manifest": runManifest,
"migrate": runMigrate,
"build-ui": runBuildUI,
"ui": runUI,
Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

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

The command map registers "ui" with runUI function, but this function is not defined anywhere in the codebase. This will cause a compilation error. Either remove this entry or implement the runUI function.

Copilot uses AI. Check for mistakes.
"publish": runPublish,
"deploy": runDeploy,
"api": runAPI,
Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

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

The PR title and description focus exclusively on adding the wfctl api extract command for OpenAPI spec generation. However, this PR also includes several unrelated substantial features: a Redis cache module with three new pipeline steps (cache_get, cache_set, cache_delete), two new state machine pipeline steps (statemachine_transition, statemachine_get), and a new build_from_config CI/CD step. These additions represent significant new functionality beyond what is described in the PR. Consider splitting these into separate PRs, each with focused scope and description, or updating the PR description to accurately reflect all changes being introduced.

Copilot uses AI. Check for mistakes.
}

func usage() {
Expand All @@ -39,8 +41,10 @@ Commands:
manifest Analyze config and report infrastructure requirements
migrate Manage database schema migrations
build-ui Build the application UI (npm install + npm run build + validate)
ui UI tooling (scaffold: generate Vite+React+TypeScript SPA from OpenAPI spec)
publish Prepare and publish a plugin manifest to the workflow-registry
deploy Deploy the workflow application (docker, kubernetes, cloud)
api API tooling (extract: generate OpenAPI 3.0 spec from config)

Run 'wfctl <command> -h' for command-specific help.
`, version)
Expand Down
Loading
Loading