fix: close build release hygiene findings#1201
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5eab0c1f1c
ℹ️ 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".
| - name: Attest release assets | ||
| if: steps.validate.outputs.valid == 'true' && github.event.inputs.dryRun != 'true' | ||
| uses: actions/attest-build-provenance@v2 | ||
| with: | ||
| subject-path: | | ||
| main.js | ||
| manifest.json | ||
| styles.css |
There was a problem hiding this comment.
Attest final release files after version bump
Move the attestation step to run after semantic-release, because @semantic-release/npm updates package.json in prepare and this repo’s version script (version-bump.mjs) rewrites manifest.json and versions.json; attesting manifest.json before Release means the generated provenance is for a pre-bump digest and will not match the asset ultimately published to GitHub releases. This causes verification failures for real releases even when the workflow succeeds.
Useful? React with 👍 / 👎.
5eab0c1 to
c4112b5
Compare
9dcf3fa to
88107e5
Compare
c4112b5 to
aa3f2b7
Compare
3f0ffa9 to
ca3d313
Compare
aa3f2b7 to
964ff2f
Compare
ca3d313 to
8ba8dc1
Compare
964ff2f to
1392394
Compare
8ba8dc1 to
8500a3a
Compare
1392394 to
0786e1a
Compare
8500a3a to
2882e0c
Compare
0786e1a to
c39fe67
Compare
2882e0c to
1047b0a
Compare
c39fe67 to
4a5f472
Compare
1047b0a to
1963257
Compare
4a5f472 to
fd4ece4
Compare
1963257 to
d95a35c
Compare
fd4ece4 to
a1f83f9
Compare
d95a35c to
b131734
Compare
a1f83f9 to
ccd77c8
Compare
b131734 to
e48be50
Compare
ccd77c8 to
6a75b43
Compare
e48be50 to
6f53e0f
Compare
6a75b43 to
46bf3f2
Compare
6f53e0f to
6cd69b2
Compare
46bf3f2 to
fa47a89
Compare
6cd69b2 to
e680dcf
Compare
fa47a89 to
d835f8f
Compare
e680dcf to
fc1a1f5
Compare
d835f8f to
8636b5f
Compare
8636b5f to
ee370e5
Compare
Close build, package, and release-hygiene scorecard findings.
This removes the unused
builtin-modulesdependency, adjusts the bundle config accordingly, deduplicates/validates icon type data, and adds release artifact attestation/provenance steps to the GitHub release workflow.Review focus:
esbuilddependency cleanup after removingbuiltin-modules.Stack position: 15/17, based on
scorecard/14-markdown-render-clear.Validated as part of the completed scorecard mission with
bun run build-with-lint,bun run test,bun run build,bun run test:e2e, and Obsidiandevvault reload/smoke checks.