Skip to content

fix: P0 release blockers - JSON order, help, goreleaser#6

Merged
muras3 merged 1 commit into
mainfrom
claude/p0-release-blockers
Feb 27, 2026
Merged

fix: P0 release blockers - JSON order, help, goreleaser#6
muras3 merged 1 commit into
mainfrom
claude/p0-release-blockers

Conversation

@muras3

@muras3 muras3 commented Feb 27, 2026

Copy link
Copy Markdown
Owner

Summary

OSS公開前のP0リリースブロッカー3件を修正。

Changes

1. JSON layers順序を実行順に固定

  • Result.MarshalJSON() カスタム実装で dns→tcp→tls→http を保証
  • core.LayerOrder でcanonical順序を一元管理
  • 契約テストで順序を検証

2. --help充実

  • USAGE / TARGETS / OPTIONS / EXIT CODES / EXAMPLES の全セクション
  • stdout出力(stderrではない)
  • 4テスト追加

3. goreleaser + GitHub Actions

  • .goreleaser.yml: linux/darwin/windows × amd64/arm64
  • .github/workflows/release.yml: tag push → 自動リリース
  • Makefile: release-dry, release-check
  • README: 存在しないHomebrew tap削除、TODOコメント除去

Test plan

  • go test ./... -count=1 ALL PASS
  • JSON順序の契約テスト追加
  • --help出力の内容テスト追加

Memo

  • eval-tech/eval-adoption/eval-market 3エージェント評価 + Codex合議の結果、P0として特定
  • JSON map順序問題はAgent利用時の機械可読契約を壊すため最優先で修正
  • goreleaser設定により git tag v0.1.0 && git push --tags でリリース可能に

🤖 Generated with Claude Code

## JSON layers order (fixer-json-order)
- Custom MarshalJSON on Result to guarantee dns→tcp→tls→http order
- Added core.LayerOrder canonical sequence, replaced runner hardcode
- Contract test verifying key order in raw JSON output

## --help充実 (fixer-help)
- Full help text with USAGE, TARGETS, OPTIONS, EXIT CODES, EXAMPLES
- Help output to stdout (not stderr)
- Config.Help field + HelpText() function
- 4 new CLI tests for help content

## goreleaser + GitHub Actions (fixer-release)
- .goreleaser.yml: linux/darwin/windows × amd64/arm64, ldflags -s -w
- .github/workflows/release.yml: tag-triggered release via goreleaser-action
- Makefile: release-dry and release-check targets
- README: removed non-existent Homebrew tap, cleaned TODO comments

All tests passing (go test ./... -count=1).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@muras3
muras3 merged commit 1cc3d40 into main Feb 27, 2026
1 check failed
@muras3
muras3 deleted the claude/p0-release-blockers branch March 3, 2026 02:31
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.

1 participant