Skip to content

feat: add JSON and shell dry-run plans for build-container#201

Draft
wyli wants to merge 2 commits into
mainfrom
feature/dryrun-json-command-plan
Draft

feat: add JSON and shell dry-run plans for build-container#201
wyli wants to merge 2 commits into
mainfrom
feature/dryrun-json-command-plan

Conversation

@wyli

@wyli wyli commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Summary

Add host-resolved structured dry-run output for build-container.

  • --dryrun --json returns ordered probe and action steps.
  • --dryrun --shell returns labeled, multiline Bash steps for copying or editing.
  • Plain --dryrun remains unchanged.
  • Literal credential values are redacted.

This first slice is enabled only for build-container.

Usage

holoscan build-container my_app --dryrun --json
holoscan build-container my_app --dryrun --shell

Example JSON shape:

{
  "schema_version": 1,
  "steps": [
    {"role": "probe", "argv": ["docker", "buildx", "version"]},
    {"role": "action", "argv": ["docker", "build", "..."]}
  ]
}

The plan reflects the current host and environment. Review generated shell output before running it.

Validation

445 tests passed, 1 skipped. All PR checks pass.

AI-assisted: Created with Codex/GPT at the user's request.

Co-authored-by: Codex <noreply@openai.com>
@copy-pr-bot

copy-pr-bot Bot commented Jul 20, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Render replay steps as labeled, multiline subshell blocks while preserving exact argv, cwd, and environment semantics.

Co-authored-by: Codex <noreply@openai.com>
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