Skip to content

ci: harden releases and lockfiles - #233

Open
victor0602 wants to merge 1 commit into
mainfrom
codex/harden-release-lockfiles
Open

ci: harden releases and lockfiles#233
victor0602 wants to merge 1 commit into
mainfrom
codex/harden-release-lockfiles

Conversation

@victor0602

@victor0602 victor0602 commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • synchronize Bun and npm lockfiles and update Undici to the patched 6.x release
  • keep build-only bun-plugin-dts in development dependencies
  • use frozen installs and pin third-party Actions to verified commit SHAs
  • run typecheck, lint, tests, build, package-content checks, and Node 18/current-LTS smoke tests before publishing
  • publish the exact verified tarball before creating the GitHub release
  • add workflow permissions, concurrency controls, and execution timeouts

Validation

  • bun install --frozen-lockfile
  • npm ci --ignore-scripts
  • bun run typecheck && bun run lint && bun test && bun run build (449 passed)
  • npm package-content and CLI/SDK smoke checks
  • npm audit --registry=https://registry.npmjs.org --audit-level=high (0 vulnerabilities)
  • workflow YAML parsing and Action tag/SHA verification

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@NianJiuZst NianJiuZst left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

No blocking findings on the current head. The two lockfiles install reproducibly, package contents and Node entry points are checked before publish, and every pinned Action SHA matches the stated upstream tag; I recommend merging it.

Verified locally: frozen Bun install, typecheck, lint (one pre-existing warning), full suite (449/449), build, npm package contents, npm ci --ignore-scripts, and zero npm audit findings. GitHub checks are green. I did not execute the tag-triggered npm publish/release path.

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