Skip to content

fix(action): read INPUT_* env vars via printenv to handle dashes#18

Merged
Cro22 merged 1 commit into
developfrom
feature/action-self-test
May 9, 2026
Merged

fix(action): read INPUT_* env vars via printenv to handle dashes#18
Cro22 merged 1 commit into
developfrom
feature/action-self-test

Conversation

@Cro22

@Cro22 Cro22 commented May 9, 2026

Copy link
Copy Markdown
Owner

The GitHub Actions runner forwards inputs as INPUT_<NAME> env vars with dashes preserved verbatim — plan-file becomes INPUT_PLAN-FILE, not INPUT_PLAN_FILE — because only spaces are converted to underscores. POSIX parameter expansion can't reference names with - (the dash is the default-value operator), so the previous reads of ${INPUT_PLAN_FILE} always saw an empty string and the entrypoint exited with "plan-file input is required" before anything else ran.

Caught by the in-repo self-test workflow added on this branch.

The GitHub Actions runner forwards inputs as `INPUT_<NAME>` env vars
with dashes preserved verbatim — `plan-file` becomes `INPUT_PLAN-FILE`,
not `INPUT_PLAN_FILE` — because only spaces are converted to
underscores. POSIX parameter expansion can't reference names with `-`
(the dash is the default-value operator), so the previous reads of
`${INPUT_PLAN_FILE}` always saw an empty string and the entrypoint
exited with "plan-file input is required" before anything else ran.

Caught by the in-repo self-test workflow added on this branch.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Cro22
Cro22 merged commit ae30963 into develop May 9, 2026
2 checks passed
Cro22 added a commit that referenced this pull request Jun 9, 2026
fix(action): read INPUT_* env vars via printenv to handle dashes
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