Repository navigation
Phase 0 (8/9): 'azureclaw convert' CLI skeleton (exit-3) - #42
Closed
Pal Lakatos-Toth (pallakatos) wants to merge 2 commits into
Closed
Pal Lakatos-Toth (pallakatos) wants to merge 2 commits into
Pal Lakatos-Toth (pallakatos) wants to merge 2 commits into
Conversation
Per docs/implementation-plan.md §2.2 + §6 item 13. Lock in the CLI
surface now; real translation lands Phase 2 per the mapping table in
docs/sigs-agent-sandbox-compat.md §4.
cli/src/commands/convert.ts
- Commander subcommand with --file / --to / --sandbox-ref / --dry-run /
--allow-lossy. Targets: clawsandbox | upstream-sandbox | overlay.
- All successful parses exit 3 ('not yet implemented') with a clear
stderr pointing at the Phase 2 mapping doc. No path exits 0 —
prevents scripts silently depending on a no-op (principle §0.2 #8).
- parseTarget helper exported via __test for unit tests.
cli/src/commands/convert.test.ts
- 3 unit tests on parseTarget (accept all three, reject unknown,
reject undefined).
cli/src/cli.ts
- Register convertCommand() under a new 'Interop' section. One import,
one addCommand call. No other command affected.
docs/security-audits/2026-04-24-phase0-convert-cli-skeleton.md
- 11-section audit: no network, no secrets, no cluster writes.
Negative-test coverage lists the two argv-reject paths + the two
exit-2 smoke cases.
Verification
- npm run typecheck (cli): clean
- npm run build (cli): clean
- npx vitest run convert.test.ts: 3 passed
- node dist/index.js convert --help: renders expected surface
- node dist/index.js convert --to bogus -f x: exit 2 as expected
- All 5 non-signoff gates: PASS
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Approved by repository maintainer for Phase 0 merge. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Collaborator
Author
|
Closing in favor of an integrated validation on the |
Pal Lakatos-Toth (pallakatos)
deleted the
phase0/kubectl-convert-skeleton
branch
April 27, 2026 12:58
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Phase 0 ·
azureclaw convertCLI skeletonAdds the Commander surface for
azureclaw convertahead of Phase 2 translator work.cli/src/commands/convert.ts--file <path>,--to <clawsandbox|sandbox>,--sandbox-ref <name>,--dry-run,--allow-lossy.cli/src/cli.tsconverttoday, will hold future interop commands.Tests
parseTarget(acceptsclawsandbox,sandbox, rejects others).Security
docs/security-audits/2026-04-24-phase0-convert-cli-skeleton.md— surface-only audit; re-audit triggers: real YAML parse, file I/O, CR emission,--apply.Verification
npm run typecheck+npm run buildclean.--helprenders; all rejection paths exit with the right code.Stack
PR 8/9 — bases on
phase0/conformance-corpus-signal(PR 7).