GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — print stackql version in action logs and update test to validate regi… by jeffreyaven · Pull Request #30 · stackql/setup-stackql

Conversation

@jeffreyaven

@jeffreyaven jeffreyaven commented Mar 14, 2026

Copy link
Copy Markdown
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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
claude added 6 commits March 14, 2026 21:31
@claude
print stackql version in action logs and update test to validate regi…
7accc79
…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
fix: strip wrapper debug annotations before jq validation in test
c2157b8
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
@claude
fix: strip wrapper command line from stackql output before jq parsing
e2161d3
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
@claude
fix: use table output and GITHUB_ENV for registry list validation
86853c4
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
@claude
fix: use [[:space:]] instead of \s for macOS bash 3.2 compatibility
3b6d999
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
@claude
fix: use shell var $registry_list_output instead of ${{ }} expansion
c06ca6b
${{ 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

Uh oh!

There was an error while loading. Please reload this page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@jeffreyaven @claude
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.