Skip to content

refactor: rename pack command to build#14

Merged
pavelanni merged 2 commits intomainfrom
refactor/rename-pack-to-build
Apr 27, 2026
Merged

refactor: rename pack command to build#14
pavelanni merged 2 commits intomainfrom
refactor/rename-pack-to-build

Conversation

@pavelanni
Copy link
Copy Markdown
Collaborator

@pavelanni pavelanni commented Apr 27, 2026

Summary

  • Rename skillctl pack to skillctl build to align with Docker/Podman conventions (build, push, pull, inspect)
  • Rename all internal types: PackOptionsBuildOptions, PackBundleBuildBundle, etc.
  • Update user-facing docs (README, CLAUDE.md, deploy/README, landing page)
  • Design docs and historical plans left unchanged as records

Test plan

  • go build ./... compiles cleanly
  • go test ./... all pass
  • make lint zero issues
  • Verify bin/skillctl build examples/hello-world/ works end-to-end

Summary by CodeRabbit

  • Refactor

    • Renamed CLI subcommand from pack to build for skill container operations
    • Updated OCI API methods and configuration types: Pack/PackOptions → Build/BuildOptions, PackBundle/BundlePackOptions → BuildBundle/BundleBuildOptions
  • Documentation

    • Updated terminology and examples from "pack" to "build" across all documentation files and deployment guides
  • Tests

    • Updated test names and assertions to reflect new "build" terminology

Align CLI with Docker/Podman conventions: build, push, pull,
inspect, promote. The tool isn't widely adopted yet, so now is
the right time for this rename.

Assisted-By: Claude (Anthropic AI) <noreply@anthropic.com>
Signed-off-by: Pavel Anni <panni@redhat.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: d481736c-a989-4983-ae04-37ebda803fec

📥 Commits

Reviewing files that changed from the base of the PR and between 985437c and 1cfa8ce.

📒 Files selected for processing (11)
  • CLAUDE.md
  • README.md
  • deploy/README.md
  • docs/index.html
  • internal/cli/build.go
  • internal/cli/root.go
  • pkg/oci/build.go
  • pkg/oci/bundle.go
  • pkg/oci/bundle_test.go
  • pkg/oci/client.go
  • pkg/oci/oci_test.go

📝 Walkthrough

Walkthrough

The pull request refactors terminology across the codebase, renaming the OCI packaging operation from "pack" to "build." This affects CLI commands, exported API methods/types, documentation, and test suite references, while preserving all underlying functionality and logic.

Changes

Cohort / File(s) Summary
Documentation Updates
README.md, CLAUDE.md, deploy/README.md, docs/index.html
Updated workflow descriptions, command examples, and module documentation to use "build" instead of "pack" terminology. Changes are primarily in comments, section titles, and command references.
CLI Implementation
internal/cli/build.go, internal/cli/root.go
Renamed CLI subcommand and dispatch functions from pack to build. Updated help text, function names (runPackrunBuild), method calls (PackBuild), and option types (PackOptionsBuildOptions) throughout the command implementation.
Core OCI Package API
pkg/oci/build.go, pkg/oci/bundle.go, pkg/oci/client.go
Renamed exported types and methods: PackBuild, PackBundleBuildBundle, PackOptionsBuildOptions, BundlePackOptionsBundleBuildOptions. Method signatures and field names updated while preserving parameter order and return types.
Test Suite
pkg/oci/oci_test.go, pkg/oci/bundle_test.go
Updated test function names and method calls to match new API naming. Replaced all client.Pack calls with client.Build, client.PackBundle with client.BuildBundle, and updated error message assertions to reference the new method names.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~18 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 3.70% 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
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'refactor: rename pack command to build' clearly and concisely describes the primary change across the entire changeset—renaming the pack command to build.
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 refactor/rename-pack-to-build

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Assisted-By: Claude (Anthropic AI) <noreply@anthropic.com>
Signed-off-by: Pavel Anni <panni@redhat.com>
@pavelanni pavelanni marked this pull request as ready for review April 27, 2026 21:53
@pavelanni pavelanni merged commit 9460484 into main Apr 27, 2026
6 checks passed
@pavelanni pavelanni deleted the refactor/rename-pack-to-build branch April 27, 2026 22:17
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