Skip to content

Fix changeset release parsing for quoted text#85

Merged
konard merged 3 commits into
mainfrom
issue-84-d3cc33387574
May 1, 2026
Merged

Fix changeset release parsing for quoted text#85
konard merged 3 commits into
mainfrom
issue-84-d3cc33387574

Conversation

@konard
Copy link
Copy Markdown
Member

@konard konard commented Apr 30, 2026

Summary

  • Fix scripts/release/apply-changesets.sh so changeset descriptions are trimmed without xargs, preventing apostrophes like buildx's from aborting release CI.
  • Read changeset files with NUL-delimited find ... -print0 and Bash arrays, which is safer for quoted text and filenames.
  • Add a focused regression experiment plus an issue-84 case study with preserved CI logs and template comparison.

Validation

  • GITHUB_BASE_REF=main GITHUB_HEAD_REF=issue-84-d3cc33387574 scripts/release/validate-changeset.sh
  • GITHUB_BASE_REF=main GITHUB_HEAD_REF=issue-84-d3cc33387574 scripts/release/check-version.sh
  • bash -n scripts/release/apply-changesets.sh
  • experiments/test-issue84-apply-changesets-quotes.sh
  • experiments/test-issue82-pr-parallel-tests.sh
  • git show --check --stat --oneline HEAD

Evidence

  • Case study: docs/case-studies/issue-84/CASE-STUDY.md
  • Failed job log: docs/case-studies/issue-84/ci-logs/job-73637479652.log
  • Reproducer: experiments/test-issue84-apply-changesets-quotes.sh

Closes #84

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

Issue: #84
@konard konard self-assigned this Apr 30, 2026
@konard konard changed the title [WIP] Fix CI/CD bugs Fix changeset release parsing for quoted text Apr 30, 2026
@konard konard marked this pull request as ready for review April 30, 2026 07:39
@konard
Copy link
Copy Markdown
Member Author

konard commented Apr 30, 2026

Solution summary

Implemented and pushed the fix to PR #85: #85

Root cause was the release script using xargs to trim changeset descriptions, which broke on apostrophes like buildx's. I updated apply-changesets.sh to avoid xargs, use safer NUL-delimited changeset discovery, and added a regression test in experiments/test-issue84-apply-changesets-quotes.sh.

Also added the issue #84 investigation artifacts and CI logs under docs/case-studies/issue-84, plus a patch changeset.

Verified locally:

  • bash -n on changed release/test scripts
  • quote regression experiment passed
  • issue Fix all CI/CD bugs #82 PR test layout experiment passed
  • release checks passed: validate-changeset.sh, check-version.sh
  • dry-run apply against real pending changesets no longer hits the xargs failure

PR is ready for review, working tree is clean, and the latest GitHub Actions run 25153457364 passed on commit 25782a5c08dd7346fbf72a62df468f9f4e2d79c8.


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

@konard
Copy link
Copy Markdown
Member Author

konard commented Apr 30, 2026

🤖 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.359732

📊 Context and tokens usage:

  • 13.7M / 1.1M (1306%) input tokens, 48.5K / 128K (38%) output tokens

Total: (385.3K + 13.3M cached) input tokens, 48.5K output tokens, $19.359732 cost

🤖 Models used:

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

📎 Log file uploaded as Repository (55466KB)


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

@konard
Copy link
Copy Markdown
Member Author

konard commented Apr 30, 2026

✅ 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 0e16792 into main May 1, 2026
50 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.

Fix CI/CD bugs

1 participant