Skip to content

Add cloud shorthand for ricky run#60

Merged
khaliqgant merged 1 commit into
mainfrom
codex/ricky-run-cloud-shorthand
May 7, 2026
Merged

Add cloud shorthand for ricky run#60
khaliqgant merged 1 commit into
mainfrom
codex/ricky-run-cloud-shorthand

Conversation

@khaliqgant

Copy link
Copy Markdown
Member

Summary

  • parse ricky run <artifact> --cloud as Cloud execution shorthand
  • render Cloud follow-up commands with the shorter ricky run ... --cloud form
  • update CLI help, README, product CLI spec, and tests

Tests

  • npx vitest run src/surfaces/cli/flows/power-user-parser.test.ts src/surfaces/cli/flows/workflow-summary.test.ts src/surfaces/cli/commands/cli-main.test.ts
  • npm run typecheck
  • git diff --check

@coderabbitai

coderabbitai Bot commented May 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack
No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 5f4cd835-4faf-4c30-b9b7-bfafd36d7fb4

📥 Commits

Reviewing files that changed from the base of the PR and between 4f1089d and 50f9e73.

📒 Files selected for processing (8)
  • README.md
  • docs/product/ricky-simplified-workflow-cli-spec.md
  • src/surfaces/cli/commands/cli-main.test.ts
  • src/surfaces/cli/commands/cli-main.ts
  • src/surfaces/cli/flows/power-user-parser.test.ts
  • src/surfaces/cli/flows/power-user-parser.ts
  • src/surfaces/cli/flows/workflow-summary.test.ts
  • src/surfaces/cli/flows/workflow-summary.ts

📝 Walkthrough

Walkthrough

This PR introduces a --cloud shorthand flag for the ricky run command, enabling direct Cloud workflow execution via ricky run <artifact> --cloud. The parser recognizes the flag, workflow summaries generate the new command syntax, help text documents the option, and user documentation reflects the change.

Changes

CLI --cloud Flag Support

Layer / File(s) Summary
Parser Logic
src/surfaces/cli/flows/power-user-parser.ts, src/surfaces/cli/flows/power-user-parser.test.ts
When run command includes --cloud flag, parsePowerUserArgs sets mode to 'cloud'. New test verifies parsing produces surface: 'legacy', mode: 'cloud', and runRequested: true.
Run Command Generation
src/surfaces/cli/flows/workflow-summary.ts, src/surfaces/cli/flows/workflow-summary.test.ts
cloudPowerUserWorkflowSummary now generates ricky run <artifact> --cloud instead of ricky cloud --workflow <artifact> --run. Test expectations updated accordingly.
Help Text & CLI Routing
src/surfaces/cli/commands/cli-main.ts, src/surfaces/cli/commands/cli-main.test.ts
Help output documents the new --cloud variant. Tests validate parseArgs, help text inclusion, Cloud routing, and "run when ready" output with the updated command syntax.
Documentation
README.md, docs/product/ricky-simplified-workflow-cli-spec.md
CLI reference and power-user spec examples updated to show ricky run <artifact> --cloud for Cloud execution.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • AgentWorkforce/ricky#52: Modifies the same CLI parsing and help files for cloud/workflow/run command behavior changes.

Poem

🐰 A flag called --cloud hops into sight,
Shorthand for workflows soaring in flight,
No more verbose syntax to bear,
Just ricky run with flair!
Cloud execution, simple and right! ☁️

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding a cloud shorthand syntax for the ricky run command.
Description check ✅ Passed The description is directly related to the changeset, explaining the cloud shorthand feature, updated syntax, and test coverage.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/ricky-run-cloud-shorthand

Comment @coderabbitai help to get the list of available commands and usage tips.

@khaliqgant khaliqgant merged commit c1d3264 into main May 7, 2026
1 check passed
@khaliqgant khaliqgant deleted the codex/ricky-run-cloud-shorthand branch May 7, 2026 09:47
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