Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: stackql/setup-stackql
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v2.3.0
Choose a base ref
...
head repository: stackql/setup-stackql
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v2.3.1
Choose a head ref
  • 8 commits
  • 5 files changed
  • 3 contributors

Commits on Mar 14, 2026

  1. print stackql version in action logs and update test to validate regi… 

    …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
    claude committed Mar 14, 2026
    Configuration menu
    Copy the full SHA
    7accc79 View commit details
    Browse the repository at this point in the history

  2. fix: strip wrapper debug annotations before jq validation in test 

    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
    claude committed Mar 14, 2026
    Configuration menu
    Copy the full SHA
    c2157b8 View commit details
    Browse the repository at this point in the history

  3. fix: strip wrapper command line from stackql output before jq parsing 

    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
    claude committed Mar 14, 2026
    Configuration menu
    Copy the full SHA
    e2161d3 View commit details
    Browse the repository at this point in the history

  4. fix: use table output and GITHUB_ENV for registry list validation 

    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
    claude committed Mar 14, 2026
    Configuration menu
    Copy the full SHA
    86853c4 View commit details
    Browse the repository at this point in the history

  5. fix: use [[:space:]] instead of \s for macOS bash 3.2 compatibility 

    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
    claude committed Mar 14, 2026
    Configuration menu
    Copy the full SHA
    3b6d999 View commit details
    Browse the repository at this point in the history

  6. fix: use shell var $registry_list_output instead of ${{ }} expansion 

    ${{ 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
    @claude
    claude committed Mar 14, 2026
    Configuration menu
    Copy the full SHA
    c06ca6b View commit details
    Browse the repository at this point in the history

  7. Merge pull request #30 from stackql/claude/update-node-dependencies-w… 

    …RhWa

print stackql version in action logs and update test to validate regi…
    @jeffreyaven
    jeffreyaven authored Mar 14, 2026
    Configuration menu
    Copy the full SHA
    e461ed8 View commit details
    Browse the repository at this point in the history

  8. updated output and tests

    @jeffreyaven
    jeffreyaven committed Mar 14, 2026
    Configuration menu
    Copy the full SHA
    2ccca03 View commit details
    Browse the repository at this point in the history
Loading