Skip to content

πŸ‘· Prepare automated package releases - #7

Merged
nbsp1221 merged 4 commits into
mainfrom
chore/prepare-first-release
Sep 4, 2026
Merged

πŸ‘· Prepare automated package releases#7
nbsp1221 merged 4 commits into
mainfrom
chore/prepare-first-release

Conversation

@nbsp1221

@nbsp1221 nbsp1221 commented Sep 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • Set the initial package version to 0.1.0 and source runtime version metadata from the package manifest.
  • Add release-it automation for version commits and tags, then publish tagged releases through GitHub Actions using npm Trusted Publishing without a long-lived npm token.
  • Generate the corresponding GitHub release after a successful package publish and let npm determine provenance for each publishing environment.

Verification

  • pnpm check β€” formatting, linting, typechecking, 40 unit and integration tests, and the production build passed.
  • pnpm release --ci --dry-run --no-increment --no-git.requireBranch --no-git.requireUpstream --no-git.push β€” simulated the initial release flow without changing Git state or pushing.
  • npm pack --dry-run --json β€” produced the expected seven package files for chat2shell@0.1.0.

@leverframe

leverframe Bot commented Sep 4, 2026

Copy link
Copy Markdown

Leverframe review

🟒 Incremental review completed in 317s

  • 0 new findings
  • 2 existing findings were verified fixed
  • 3 of 3 changed files reviewed
  • Verification: 6 passed

Changes d3aca4c..50d8251 Β· Commit 50d8251 Β· View check run

@leverframe leverframe Bot 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.

Leverframe review

The code, lockfile, package contents, and stable-version build pass under Node v24.20.0, npm 11.19.0, and the declared pnpm 11.23.0. Two release-workflow defects remain: prerelease tags cannot publish, and a post-publish failure cannot be recovered by rerunning the workflow.

Findings

2 findings were published inline.

Checks

6 passed Β· 1 failed Β· 0 not run

Show 7 checks
Status Check Evidence
🟒 passed node --version; pnpm --version; npm --version Selected repository/CI-compatible environment: Node v24.20.0, pnpm 11.23.0, npm 11.19.0.
🟒 passed pnpm install --frozen-lockfile Installed 464 packages; the lockfile was current and passed pnpm supply-chain policy verification.
🟒 passed pnpm check Formatting, oxlint/eslint, TypeScript checking, 40 unit/integration tests, and the tsdown production build passed.
🟒 passed node dist/cli.mjs --version; npm pack --dry-run --json The built CLI reported 0.1.0 and the package dry run produced a valid chat2shell@0.1.0 tarball containing the executable and declared files.
🟒 passed pnpm exec release-it --dry-run --ci --increment=prepatch --preRelease=beta In a temporary main-branch clone with an upstream, release-it resolved 0.1.0 to 0.1.1-beta.0 and showed the matching commit, annotated tag, and push operations.
πŸ”΄ failed npm publish --dry-run --loglevel notice Against a temporary 0.1.1-beta.0 build, npm 11.19.0 reproduced the defect: You must specify a tag using --tag when publishing a prerelease version.
🟒 passed git diff --check 3169bd993f8492089316aecf6a2c1e6e39dc21ec...d3aca4cf4e3d13b802516fc28cd3295d3d74e39e No whitespace errors were reported.

Limitations

  • No real npm publication or GitHub release was performed because review scope forbids publishing and GitHub credential use. The release state transitions were verified through dry runs, installed implementation behavior, and focused temporary-clone simulations.

  • Docker Sandbox E2E tests were not run; repository guidance excludes them from the normal quality gate, and this diff does not alter sandbox behavior.

Comment thread .github/workflows/release.yml Outdated
Comment thread .github/workflows/release.yml Outdated
@nbsp1221
nbsp1221 merged commit 207d1b6 into main Sep 4, 2026
2 checks passed
@nbsp1221
nbsp1221 deleted the chore/prepare-first-release branch September 4, 2026 15:09
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.

1 participant