Skip to content

Conversation

@ElMaxter99
Copy link
Owner

Summary

  • add a GitHub Actions workflow that installs dependencies, runs tests, and publishes to npm whenever a v* tag is pushed or the job is manually dispatched
  • remove the local postversion publish script and document the new release flow in the README

Testing

  • npm test

https://chatgpt.com/codex/tasks/task_e_6901eade197483258a6e5efd535a81bc

Copy link

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

ℹ️ 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 on lines +15 to +18
env:
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
steps:
- name: Checkout repository

Choose a reason for hiding this comment

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

P1 Badge Scope npm token to publish step

NODE_AUTH_TOKEN is configured at the job level, which means the secret is passed to npm ci and npm test as well as the publish step. Any dependency install script or test code could print or exfiltrate the token, giving the secret to untrusted code when the workflow runs. For least privilege, set the environment variable only on the Publish package step (or use actions/setup-node with token) so that only the actual publish command receives the secret.

Useful? React with 👍 / 👎.

@ElMaxter99 ElMaxter99 merged commit 04fa102 into master Oct 29, 2025
13 checks passed
@ElMaxter99 ElMaxter99 deleted the alvaromaxter/add-npm-publish-on-tag-creation branch October 29, 2025 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant