[piekstra]

Summary

The release workflow installs GoReleaser ~> v2, which now resolves to 2.19. That version deprecates homebrew_casks.hooks.post.install, and the workflow's goreleaser check fails on any deprecated property, so the v0.10.350 release failed before building.

• The cask's postflight moves to install_steps, with {{ .StagedPath }} in place of #{staged_path}. GoReleaser renders it as the postflight_steps block the shared alias job's modernizer used to produce, and the modernizer passes an already-modern cask through unchanged.
• scripts/verify-package-render.sh now asserts the rendered postflight_steps block holds only run steps with the absolute {{staged_path}} token, and that no legacy postflight block is rendered.
• The Arch package check pins GoReleaser for its snapshot render; it moves from v2.18.2 to v2.19.0, which reads install_steps.

Verification

• goreleaser check with 2.19.0 fails on main with the deprecation and passes on this branch.
• goreleaser release --snapshot --clean --skip=publish,before,sign: the rendered cask's postflight_steps block matches the one published in the tap line for line, and modernize.sh render from the pinned shared workflow leaves the cask unchanged.
• scripts/verify-package-render.sh passes on the rendered output. It fails with legacy postflight block present when the block is renamed back to postflight do, and with postflight_steps line is not a run step when the step is written as system_command.

Type of Change

• Bug fix
• New feature
• Documentation update
• Repo scaffolding / maintenance

Checklist

• I have read the contributing guide
• My change follows the project structure and conventions
• make tidy passes
• make lint passes
• make test passes
• make build passes

Model: Claude Opus 5.5 (1M context, xhigh effort) in Claude Code.

[monit-reviewer]

Automated PR Review

Reviewed commit: 40199aaf69da
Profile: claude-monit-reviewer - Posting as: monit-reviewer

Summary

Reviewer	Findings
automation:ci-release	0
harness-engineering:repo-health	0
policies:conventions	0
structure:repo-health	0
automation:ci-deploy	0

Reviewer Coverage

Complete coverage details are retained in the local coverage.json artifact.

• automation:ci-release — complete (broad); scope: 3 files; examples: .github/workflows/arch-package.yml, .goreleaser.yml, scripts/verify-package-render.sh; body-inspected: 3 files; examples: .github/workflows/arch-package.yml, .goreleaser.yml, scripts/verify-package-render.sh; skipped: none; constraints: Did not inspect the shared release workflow or modernizer; relied on the PR description for how it treats an already-modern cask. Static review of the diff only; did not run goreleaser or the verify script.
• harness-engineering:repo-health — complete (constrained); scope: 3 files; examples: .github/workflows/arch-package.yml, .goreleaser.yml, scripts/verify-package-render.sh; body-inspected: 3 files; examples: .github/workflows/arch-package.yml, .goreleaser.yml, scripts/verify-package-render.sh; skipped: none; constraints: Reviewed the three-file diff only; did not execute goreleaser or the verify script.
• policies:conventions — complete (broad); scope: 3 files; examples: .github/workflows/arch-package.yml, .goreleaser.yml, scripts/verify-package-render.sh; body-inspected: 3 files; examples: .github/workflows/arch-package.yml, .goreleaser.yml, scripts/verify-package-render.sh; skipped: none; constraints: Did not run goreleaser or the verify script. Shared cli-common docs and the pinned shared workflow were not available in context; review is based on the diff and PR description only.
• structure:repo-health — complete (broad); scope: 2 files; examples: .github/workflows/arch-package.yml, .goreleaser.yml; body-inspected: 2 files; examples: .github/workflows/arch-package.yml, .goreleaser.yml; skipped: none; constraints: Did not execute goreleaser or scripts/verify-package-render.sh; relied on static inspection of the diff and the verify script's postflight assertions.
• automation:ci-deploy — complete (broad); scope: 3 files; examples: .github/workflows/arch-package.yml, .goreleaser.yml, scripts/verify-package-render.sh; body-inspected: 3 files; examples: .github/workflows/arch-package.yml, .goreleaser.yml, scripts/verify-package-render.sh; skipped: none; constraints: Did not execute goreleaser or verify-package-render.sh; relied on static diff inspection and PR-described verification. Did not read the shared release workflow's modernize.sh; the PR states it passes the modern cask through unchanged.

0 PR discussion threads considered. 0 summarized; 0 resolved.

---

Completed in 1m 47s | ~$0.43 (est.) | claude-sonnet-5-5 | cr 0.10.349

Field	Value
Model	claude-sonnet-5-5
Reviewers	automation:ci-release, harness-engineering:repo-health, policies:conventions, structure:repo-health, automation:ci-deploy
Engine	claude_cli · claude-sonnet-5-5
Reviewed by	cr · monit-reviewer
Duration	1m 47s wall · 3m 56s compute
Cost	~$0.43 (est.)
Pricing basis	baseline-2026-10-02/pricing
Tokens	52 in / 7.1k out

Per-workstream usage

• orchestrator-selection — claude-sonnet-5-5
  • In: 6
  • Out: 1.3k
  • Cache read: 18.9k
  • Cache create: 11.4k
  • Cost: ~$0.06 (est.)
  • Duration: 17s
• automation:ci-release — claude-sonnet-5-5
  • In: 8
  • Out: 1.1k
  • Cache read: 42.6k
  • Cache create: 11.9k
  • Cost: ~$0.07 (est.)
  • Duration: 46s
• harness-engineering:repo-health — claude-sonnet-5-5
  • In: 8
  • Out: 977
  • Cache read: 43.7k
  • Cache create: 12.3k
  • Cost: ~$0.07 (est.)
  • Duration: 44s
• policies:conventions — claude-sonnet-5-5
  • In: 8
  • Out: 951
  • Cache read: 41.2k
  • Cache create: 11.1k
  • Cost: ~$0.06 (est.)
  • Duration: 33s
• structure:repo-health — claude-sonnet-5-5
  • In: 6
  • Out: 840
  • Cache read: 26.5k
  • Cache create: 10.3k
  • Cost: ~$0.05 (est.)
  • Duration: 33s
• automation:ci-deploy — claude-sonnet-5-5
  • In: 10
  • Out: 1.4k
  • Cache read: 64.3k
  • Cache create: 14.0k
  • Cost: ~$0.08 (est.)
  • Duration: 41s
• orchestrator-rollup — claude-sonnet-5-5
  • In: 6
  • Out: 483
  • Cache read: 45.8k
  • Cache create: 4.2k
  • Cost: ~$0.03 (est.)
  • Duration: 19s