Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 345f359a06
ℹ️ 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".
There was a problem hiding this comment.
Pull request overview
This PR moves prek version metadata out of the bundled action code by downloading a repo-root version-manifest.json at runtime, and introduces a separate generated checksum map to verify downloaded assets.
Changes:
- Replace bundled manifest usage with runtime download/caching of
version-manifest.jsonfor version/asset resolution. - Introduce generated
knownChecksumsByAssetkeyed byversion:assetNameand update install-time checksum verification to use it. - Update updater script/workflow, tests, and types/docs to match the new manifest/checksum flow.
Reviewed changes
Copilot reviewed 10 out of 13 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
src/manifest.ts |
Downloads and caches the version manifest at runtime; updates version/asset resolution APIs. |
src/install.ts |
Switches checksum verification to use a generated known-checksum map keyed by version+asset. |
src/types.ts |
Simplifies ManifestAsset shape (removes checksum/size fields). |
src/known-checksums.ts |
Adds generated checksum map for assets by version. |
scripts/update-known-versions.mjs |
Generates repo-root manifest + checksum map; updates PR output metadata. |
.github/workflows/update-known-versions.yml |
Updates scheduled workflow to regenerate new manifest/checksum outputs and open PRs. |
test/manifest.test.ts |
Updates tests for new manifest-driven resolution APIs. |
test/install.test.ts |
Updates checksum validation tests for the new checksum lookup scheme. |
package.json |
Renames updater script to update-known-versions. |
README.md |
Updates documentation references from bundled manifest to runtime resolution wording. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
345f359 to
fea8b93
Compare
Summary