Skip to content

print stackql version in action logs and update test to validate regi…#30

Merged
jeffreyaven merged 6 commits intomainfrom
claude/update-node-dependencies-wRhWa
Mar 14, 2026
Merged

print stackql version in action logs and update test to validate regi…#30
jeffreyaven merged 6 commits intomainfrom
claude/update-node-dependencies-wRhWa

Conversation

@jeffreyaven
Copy link
Member

…stry list

  • index.js: run stackql --version after setup and log output to action logs (runs before wrapper install to use the original binary name)
  • workflow: replace version check test with stackql exec REGISTRY LIST test that validates the JSON response is an array of {provider, version} objects

https://claude.ai/code/session_01Y8NseCm28GqE8nQ1Yvg5aa

claude added 6 commits March 14, 2026 21:31
…stry list

- index.js: run stackql --version after setup and log output to action logs
  (runs before wrapper install to use the original binary name)
- workflow: replace version check test with stackql exec REGISTRY LIST test
  that validates the JSON response is an array of {provider, version} objects

https://claude.ai/code/session_01Y8NseCm28GqE8nQ1Yvg5aa
When use_wrapper=true, @actions/exec writes stackql's stdout directly to
the wrapper process's stdout AND the wrapper emits ::debug:: annotations
via core.debug(). Since the JSON output has no trailing newline, the debug
text is appended on the same line (e.g. [{...}]::debug::StackQL exited...).

Strip all ::name::value GitHub Actions workflow commands from captured
output before piping to jq.

https://claude.ai/code/session_01Y8NseCm28GqE8nQ1Yvg5aa
The wrapper prints the binary path + args as the first line of output,
which is not JSON. grep -E '^\s*[\[{]' keeps only lines that are actual
JSON, discarding both that command line and any leftover blank lines.

https://claude.ai/code/session_01Y8NseCm28GqE8nQ1Yvg5aa
Avoids jq dependency and wrapper stdout pollution. Captures table-format
output via GITHUB_ENV multiline syntax, then validates header and first
data row with regex (same pattern as the version check step).

https://claude.ai/code/session_01Y8NseCm28GqE8nQ1Yvg5aa
macOS ships bash 3.2 which does not support \s in =~ regex patterns.
Replace with POSIX [[:space:]] which works on all platforms.

https://claude.ai/code/session_01Y8NseCm28GqE8nQ1Yvg5aa
${{ env.registry_list_output }} causes GitHub Actions to inline the raw
table content (with | pipes and newlines) directly into the bash script
before the shell parses it, breaking [[ ]] conditionals on Windows.
Using the shell env var avoids the inline substitution entirely.

https://claude.ai/code/session_01Y8NseCm28GqE8nQ1Yvg5aa
@jeffreyaven jeffreyaven merged commit e461ed8 into main Mar 14, 2026
7 checks passed
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.

2 participants