Skip to content

Expose builder name output#104

Open
mkusaka wants to merge 1 commit into
useblacksmith:mainfrom
mkusaka:feat/name-output
Open

Expose builder name output#104
mkusaka wants to merge 1 commit into
useblacksmith:mainfrom
mkusaka:feat/name-output

Conversation

@mkusaka

@mkusaka mkusaka commented Jun 4, 2026

Copy link
Copy Markdown

Summary

  • Add a name output to the action metadata.
  • Set the name output to the selected builder name after the Blacksmith remote builder is configured.
  • Also set the output when reusing an existing non-default builder or creating the local fallback builder.
  • Document the new output in the README.

Fixes #103.

Verification

  • git diff --check
  • node --check dist/index.js
  • node -e "JSON.parse(require(\"fs\").readFileSync(\"dist/index.js.map\",\"utf8\")); console.log(\"ok\")"
  • prettier --check README.md action.yml src/main.ts src/main.test.ts

I could not run pnpm install, pnpm test, or pnpm run build locally because the private Buf generated package requires BUF_TOKEN; without it, pnpm install --frozen-lockfile fails with a 404 for @buf/blacksmith_vm-agent.bufbuild_es from https://buf.build/gen/npm/v1/.


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled. (Staging)

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.

Expose builder name as a name output for docker/setup-buildx-action compatibility

1 participant