Skip to content

Bundle prek version manifest and automate refresh#80

Merged
j178 merged 8 commits intomainfrom
stack/prek-manifest-refresh-pr
Mar 15, 2026
Merged

Bundle prek version manifest and automate refresh#80
j178 merged 8 commits intomainfrom
stack/prek-manifest-refresh-pr

Conversation

@j178
Copy link
Copy Markdown
Owner

@j178 j178 commented Mar 15, 2026

Summary

  • resolve prek versions and asset URLs from a bundled release manifest
  • add a scheduled workflow to refresh the manifest and open an update PR
  • keep semver range parsing and archive checksum verification for later PRs in the stack

Stack

Depends on #79.

Copilot AI review requested due to automatic review settings March 15, 2026 12:17
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 43a91fd6f6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/update-version-manifest.mjs Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds automation to periodically refresh the bundled prek release manifest and open an update pull request (instead of pushing directly), keeping the action’s pinned runtime metadata up to date.

Changes:

  • Add scripts/update-version-manifest.mjs to regenerate src/version-manifest.json from the prek GitHub Releases API.
  • Add a scheduled + manually-triggerable workflow to run the updater, rebuild dist/, and open a PR via peter-evans/create-pull-request.
  • Filter release assets to only archive extensions when constructing the manifest.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
scripts/update-version-manifest.mjs New manifest generator that fetches releases and writes the bundled version manifest JSON.
.github/workflows/update-version-manifest.yml New scheduled workflow to run the generator, test/bundle on changes, and open an automated PR.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread scripts/update-version-manifest.mjs
Comment thread scripts/update-version-manifest.mjs
Comment thread .github/workflows/update-version-manifest.yml
@j178 j178 force-pushed the stack/prek-manifest-runtime branch from fdc6f14 to 6d6c9c3 Compare March 15, 2026 12:34
@j178 j178 force-pushed the stack/prek-manifest-refresh-pr branch from 43a91fd to 6281c2d Compare March 15, 2026 12:34
@j178 j178 changed the title Automate prek manifest refresh with pull requests Bundle prek version manifest and automate refresh Mar 15, 2026
@j178 j178 force-pushed the stack/prek-manifest-runtime branch from 6d6c9c3 to ab90fe7 Compare March 15, 2026 12:43
@j178 j178 force-pushed the stack/prek-manifest-refresh-pr branch from 6281c2d to 1e1d565 Compare March 15, 2026 12:43
Base automatically changed from stack/prek-manifest-runtime to main March 15, 2026 12:46
@j178 j178 added the enhancement New feature or request label Mar 15, 2026
@j178 j178 force-pushed the stack/prek-manifest-refresh-pr branch from 0fd300a to de011c0 Compare March 15, 2026 12:58
@j178 j178 merged commit 0683490 into main Mar 15, 2026
9 checks passed
@j178 j178 deleted the stack/prek-manifest-refresh-pr branch March 15, 2026 13:18
j178 added a commit that referenced this pull request Mar 15, 2026
## Summary
- verify downloaded prek archives against the SHA-256 digests stored in
the bundled manifest
- fail fast on checksum mismatches before extraction
- keep semver range parsing in the next PR in the stack

Closes #61 

## Stack
Depends on #80.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants