Skip to content

refactor(exec): centralize subprocess execution - #18

Merged
jmgilman merged 1 commit into
mainfrom
refactor/shared-exec
Aug 20, 2026
Merged

refactor(exec): centralize subprocess execution#18
jmgilman merged 1 commit into
mainfrom
refactor/shared-exec

Conversation

@jmgilman

Copy link
Copy Markdown
Contributor

Summary

  • add internal/execx as the single production subprocess mechanism
  • migrate apko, cosign, ghup, gitx, melange, and GoReleaser execution
  • centralize path resolution, stream routing, bounded stderr capture, WaitDelay, and typed exit metadata
  • keep argv, ordering, credentials, parsing, retries, and domain errors in each consumer
  • remove the six duplicated execution helpers and consolidate their mechanism tests

Verification

  • mise exec -- go test ./internal/execx -count=1
  • targeted tests for all six consumer packages
  • mise exec -- moon run root:check
  • docker run --rm --cpus 4 -v "$PWD:/src" -w /src -e GOFLAGS=-mod=mod golang:1.26 go test ./... -count=1
  • structural check: production os/exec remains only in internal/execx; the one other use is test-only Git fixture setup

@jmgilman
jmgilman merged commit 0fd8483 into main Aug 20, 2026
2 checks passed
@jmgilman
jmgilman deleted the refactor/shared-exec branch August 20, 2026 04:16
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