Fix npm provenance metadata and bump version to 0.2.3#5
Conversation
📝 WalkthroughWalkthroughThis PR bumps the package to 0.2.3 and expands package.json metadata (repository, homepage, bugs.url), updates README install pin to 0.2.3, and updates the auto-release GitHub Actions workflow to grant ChangesVersion 0.2.3 Release
Sequence Diagram(s)sequenceDiagram
participant AutoRelease as Auto-release Workflow
participant GHCLI as gh CLI
participant GitHubAPI as GitHub API
participant PublishWF as publish.yml Workflow
AutoRelease->>GHCLI: gh release create --tag <tag>
GHCLI->>GitHubAPI: create release (uses GITHUB_TOKEN)
AutoRelease->>GHCLI: gh workflow run publish.yml --ref <tag> (pass ref input)
GHCLI->>GitHubAPI: trigger workflow run for publish.yml with ref=<tag>
GitHubAPI->>PublishWF: start publish.yml run (with passed ref)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Actionable comments posted: 0 |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
.github/workflows/auto-release.yml (1)
50-56: ⚡ Quick winUse environment variable to avoid template injection in shell context.
Directly interpolating
${{ steps.check.outputs.tag }}into therun:block is flagged by static analysis for potential command injection. While the risk is low here (the value originates frompackage.jsonand requires write access tomain), best practice is to pass it via an environment variable, which GitHub Actions properly escapes.♻️ Proposed fix
- name: Trigger publish workflow if: steps.check.outputs.released == 'true' - run: | - TAG="${{ steps.check.outputs.tag }}" - gh workflow run publish.yml --ref "$TAG" -f ref="$TAG" + run: gh workflow run publish.yml --ref "$TAG" -f ref="$TAG" env: GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} + TAG: ${{ steps.check.outputs.tag }}🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.github/workflows/auto-release.yml around lines 50 - 56, The run block currently interpolates `${{ steps.check.outputs.tag }}` directly, which risks template injection; modify the "Trigger publish workflow" step to pass the tag via an environment variable (e.g., set env: TAG: ${{ steps.check.outputs.tag }} or similar) and then reference that env var inside the run script as "$TAG" (used by the existing TAG variable and gh workflow run command) instead of using `${{ ... }}` inline; ensure GH_TOKEN remains in env and remove direct template interpolation from the run body so the shell only sees the escaped environment variable.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In @.github/workflows/auto-release.yml:
- Around line 50-56: The run block currently interpolates `${{
steps.check.outputs.tag }}` directly, which risks template injection; modify the
"Trigger publish workflow" step to pass the tag via an environment variable
(e.g., set env: TAG: ${{ steps.check.outputs.tag }} or similar) and then
reference that env var inside the run script as "$TAG" (used by the existing TAG
variable and gh workflow run command) instead of using `${{ ... }}` inline;
ensure GH_TOKEN remains in env and remove direct template interpolation from the
run body so the shell only sees the escaped environment variable.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: e48480c7-4c9a-4eb8-a0ca-df89f726c450
📒 Files selected for processing (1)
.github/workflows/auto-release.yml
Summary
Validation
Summary by CodeRabbit
Documentation
Chores