Summary
The internal/cli/ package has zero test files. The build command
now has two code paths (runBuild for local, runBuildRemote for
URLs) and neither is tested at the CLI layer.
What to test
- Flag parsing:
--ref, --filter, --tag, --media-type
- URL detection routing (
source.IsRemote → runBuildRemote)
--tag rejected when building multiple skills- Error output for missing
git, bad URLs, etc.
- Local build regression (cobra args, output format)
Context
Found during code review of #24. The backend packages
(pkg/source/, pkg/oci/) have good test coverage, but the CLI
wiring is only verified via manual end-to-end testing.
Summary
The
internal/cli/package has zero test files. The build commandnow has two code paths (
runBuildfor local,runBuildRemoteforURLs) and neither is tested at the CLI layer.
What to test
--ref,--filter,--tag,--media-typesource.IsRemote→runBuildRemote)--tagrejected when building multiple skillsgit, bad URLs, etc.Context
Found during code review of #24. The backend packages
(
pkg/source/,pkg/oci/) have good test coverage, but the CLIwiring is only verified via manual end-to-end testing.