Skip to content

fix(canary): compact nx affected JSON for GITHUB_OUTPUT - #1213

Merged
ryanmcafee merged 1 commit into
mainfrom
fix/canary-compact-affected-json
Oct 7, 2026
Merged

fix(canary): compact nx affected JSON for GITHUB_OUTPUT#1213
ryanmcafee merged 1 commit into
mainfrom
fix/canary-compact-affected-json

Conversation

@ryanmcafee

@ryanmcafee ryanmcafee commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Canary release on #1212 fails at "Detect affected packages": nx show projects --json prints multi-line JSON and GITHUB_OUTPUT rejects it (Invalid format). #1193 fixed this under lerna; the nx switch in #1196 regressed it. Pipe through jq -c and stop masking nx errors as "no changes".

Since issue_comment workflows run from main, this must merge before /canary-release on #1212 can succeed.

Failed run: https://github.com/JupiterOne/sdk/actions/runs/37663172433
JIRA: https://jupiterone.atlassian.net/browse/PLATENG-1343

@ryanmcafee Ryan McAfee - AI
fix(canary): compact nx affected JSON before writing step output
d878b0d 
nx show projects --json emits multi-line JSON, which GITHUB_OUTPUT rejects
with 'Invalid format'. #1193 fixed this for lerna; #1196 reintroduced it.
Also stop masking nx failures as an empty affected list.

Co-Authored-By: Ryan McAfee - AI <ryan.mcafee+ai@jupiterone.com>
@ryanmcafee
ryanmcafee requested a review from a team as a code owner October 7, 2026 18:34
@ryanmcafee
ryanmcafee enabled auto-merge October 7, 2026 18:40
@ryanmcafee
ryanmcafee merged commit fac9847 into main Oct 7, 2026
5 checks passed
@ryanmcafee
ryanmcafee deleted the fix/canary-compact-affected-json branch October 7, 2026 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@VDubber VDubber VDubber approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ryanmcafee @VDubber