Skip to content

fix: verify native Arch package integrity - #36

Merged
rianjs merged 1 commit into
mainfrom
rianjs/arch-linux-packages
Oct 4, 2026
Merged

rianjs merged 1 commit into
mainfrom
rianjs/arch-linux-packages

Conversation

@rianjs

@rianjs rianjs commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

Keep package timestamps aligned with pacman integrity metadata. Add full GoReleaser rendering and clean Arch installation, ownership/integrity, uninstall, and reinstall checks for both gro and grw.

Validation: GoReleaser configuration and Actionlint pass. Google already publishes native packages; this closes the integrity regression and adds the same coverage established in codereview-cli #648.

@monit-reviewer monit-reviewer left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated PR Review

Reviewed commit: 1e175446e3f8
Profile: codex-monit-reviewer - Posting as: monit-reviewer

Summary

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

Reviewer Coverage

  • automation:ci-release — complete (broad); skipped: none; constraints: Review limited to the assigned CI and release changes. Actionlint passed; full GoReleaser rendering and Arch container installation were not executed locally.
  • policies:conventions — complete (broad); skipped: none; constraints: Review scoped to convention adherence in the assigned files, using repository development guidance, standards index, and existing automation. Shared cli-common standards and shared automation were unavailable: no sibling convenience copies were present, and remote retrieval failed.
  • structure:repo-health — complete (broad); skipped: none; constraints: Review limited to the assigned packaging configuration and workflow, with repository guidance and release configuration as context. Full GoReleaser builds and Arch container checks were not executed.
  • automation:ci-deploy — complete (broad); skipped: none; constraints: Review limited to the assigned packaging and CI changes. Actionlint passed; full GoReleaser rendering and Docker installation checks were not executed locally.
Inspected files (2)
  • .github/workflows/arch-package.yml
  • .goreleaser.yaml

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


Completed in 1m 06s | gpt-6.1-sol | cr 0.10.323
Field Value
Model gpt-6.1-sol
Reviewers automation:ci-release, policies:conventions, structure:repo-health, automation:ci-deploy
Engine codex_cli · gpt-6.1-sol
Reviewed by cr · monit-reviewer
Duration 1m 06s wall · 2m 14s compute
Cost unavailable
Tokens 476.3k in / 2.8k out

Per-workstream usage

  • orchestrator-selection — gpt-6.1-sol
    • In: 19.7k
    • Out: 418
    • Cache read: 12.3k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 17s
  • automation:ci-release — gpt-6.1-sol
    • In: 135.5k
    • Out: 585
    • Cache read: 110.3k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 31s
  • policies:conventions — gpt-6.1-sol
    • In: 66.0k
    • Out: 391
    • Cache read: 52.0k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 21s
  • structure:repo-health — gpt-6.1-sol
    • In: 118.3k
    • Out: 444
    • Cache read: 101.1k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 27s
  • automation:ci-deploy — gpt-6.1-sol
    • In: 95.6k
    • Out: 466
    • Cache read: 77.4k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 25s
  • orchestrator-rollup — gpt-6.1-sol
    • In: 41.2k
    • Out: 499
    • Cache read: 31.9k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 10s

@rianjs
rianjs marked this pull request as ready for review October 4, 2026 22:04
@rianjs
rianjs merged commit 36790d1 into main Oct 4, 2026
13 checks passed
@rianjs
rianjs deleted the rianjs/arch-linux-packages branch October 4, 2026 22:51
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.

2 participants