Skip to content

Fix split language release jobs after skipped gates - #163

Merged
konard merged 4 commits into
mainfrom
issue-162-0c186d5158ec
Jun 8, 2026
Merged

Fix split language release jobs after skipped gates#163
konard merged 4 commits into
mainfrom
issue-162-0c186d5158ec

Conversation

@konard

@konard konard commented Jun 8, 2026

Copy link
Copy Markdown
Member

Fixes #162

Summary

This PR fixes the split JavaScript/Rust release workflows introduced by PR #161. The post-merge runs after PR #161 succeeded overall, but the actual language release jobs were skipped because their needs graph included PR-only gate jobs that are skipped on push.

The release jobs now use explicit status-check guards so GitHub Actions evaluates them after skipped PR-only dependencies:

  • JavaScript release: always() && !cancelled() plus successful lint and test results.
  • Rust release: always() && !cancelled() plus successful lint, test, and build results.

Case Study

Full investigation and evidence:

https://github.com/link-foundation/command-stream/blob/issue-162-0c186d5158ec/docs/case-studies/issue-162/README.md

The case study includes the PR #161 timeline, downloaded CI logs, package registry snapshots, template snapshots, and before/after test logs.

Template Reports

The JavaScript and Rust upstream templates already protect their automatic release paths from this skipped-needs behavior. I found related workflow-dispatch risks in two other templates and opened upstream reports:

Verification

Local checks run:

bun test js/tests/repository-layout.test.mjs --timeout 10000
ruby -e "require 'yaml'; YAML.load_file('.github/workflows/js.yml'); YAML.load_file('.github/workflows/rust.yml'); puts 'ok'"
cd js && bun run format:check
cd js && bun run lint
cd js && bun run check:duplication
cd js && PATH=/tmp/issue-162-bin:$PATH bun run test

The full JS suite passed with 686 passing tests, 5 skipped, 0 failed. The container did not provide jq and passwordless sudo was unavailable, so the full local JS test run used a temporary uncommitted jq-1.7.1 binary under /tmp/issue-162-bin; CI installs jq in the workflow.

Release Notes

Patch release markers were added for both packages so the next merge to main has release input to consume.

Adding .gitkeep for PR creation (default mode).
This file will be removed when the task is complete.

Issue: #162
@konard konard self-assigned this Jun 8, 2026
@konard konard changed the title [WIP] Previous changes in https://github.com/link-foundation/command-stream/pull/161 didn't not result in Rust and JavaScript releases at GitHub and package managers Fix split language release jobs after skipped gates Jun 8, 2026
@konard
konard marked this pull request as ready for review June 8, 2026 19:49
@konard

konard commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

Working session summary

Implemented and pushed the fix to PR 163: #163

What changed:

  • Fixed JS and Rust release job if: conditions so they evaluate after skipped PR-only gate jobs on push to main.
  • Added regression coverage in js/tests/repository-layout.test.mjs.
  • Added a follow-up CI fix so the workflow test handles CRLF line endings on Windows.
  • Added JS/Rust release markers and the issue 162 case study evidence.

Verification:

PR is ready for review, not draft, merge state is CLEAN, and the working tree is clean.


This summary was automatically extracted from the AI working session output.

@konard

konard commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

🤖 Solution Draft Log

This log file contains the complete execution trace of the AI solution draft process.

💰 Cost estimation:

  • Model: GPT-5.5
  • Provider: OpenAI
  • Public pricing estimate: $19.077064

📊 Context and tokens usage:

  • 562.7K / 1.1M (54%) input tokens, 58.3K / 128K (46%) output tokens

Total: (562.7K + 10.8M cached) input tokens, 58.3K output tokens, $19.077064 cost

🤖 Models used:

  • Tool: OpenAI Codex
  • Requested: gpt-5.5
  • Model: GPT-5.5 (gpt-5.5)

📎 Log file uploaded as Repository (64054KB)


Now working session is ended, feel free to review and add any feedback on the solution draft.

@konard

konard commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

✅ Ready to merge

This pull request is now ready to be merged:

  • All CI checks have passed
  • No merge conflicts
  • No pending changes

Monitored by hive-mind with --auto-restart-until-mergeable flag

@konard
konard merged commit 89d47cf into main Jun 8, 2026
20 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

1 participant