Skip to content

Standardize package manager to Yarn and enhance CI workflow#39

Merged
warengonzaga merged 6 commits intomainfrom
copilot/apply-security-update
Jan 30, 2026
Merged

Standardize package manager to Yarn and enhance CI workflow#39
warengonzaga merged 6 commits intomainfrom
copilot/apply-security-update

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 30, 2026

CodeRabbit identified package manager inconsistency: CI used npm while repository tracked yarn.lock, creating potential dependency conflicts.

Changes

CI Workflow (.github/workflows/npm-publish.yml)

  • Migrated from npm to yarn with --frozen-lockfile for reproducible builds
  • Added build validation step before publish
  • Added test execution with continue-on-error: true (pre-existing test failures)
  • Updated publish command: yarn publish --non-interactive --access public

Defensive Measure (.gitignore)

  • Retained package-lock.json entry to prevent accidental commits from npm usage

Diff

# Before
- run: npm install
- run: npm publish

# After  
- run: yarn install --frozen-lockfile
- run: yarn build
- run: yarn publish --non-interactive --access public

Note: This PR also includes security dependency updates (yarn.lock) that resolved 3 vulnerabilities (@babel/helpers, brace-expansion, js-yaml).


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Summary by CodeRabbit

  • Chores
    • Updated repository configuration to exclude generated package lockfiles from version control.

✏️ Tip: You can customize this high-level summary in your review settings.

- Update yarn.lock with fixed versions of vulnerable packages
- Add package-lock.json to .gitignore (yarn is primary package manager)
- Fix 3 vulnerabilities: @babel/helpers, brace-expansion, and js-yaml
- Reduce total vulnerabilities from 13 to 10

Co-authored-by: warengonzaga <15052701+warengonzaga@users.noreply.github.com>
Copilot AI changed the title [WIP] Apply security update from security report Apply security updates for dependency vulnerabilities Jan 30, 2026
Copilot AI requested a review from warengonzaga January 30, 2026 00:53
@warengonzaga warengonzaga marked this pull request as ready for review January 30, 2026 00:53
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 30, 2026

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • ✅ Full review completed - (🔄 Check again to review again)

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In @.gitignore:
- Line 9: The repo currently mixes npm and Yarn lockfiles; decide and make the
package manager consistent: either (A) migrate to Yarn by removing
package-lock.json from gitignore, updating CI workflow
`.github/workflows/npm-publish.yml` to run `yarn install`/`yarn publish`, and
converting npm scripts in `package.json` to Yarn equivalents (remove any
committed package-lock.json), or (B) standardize on npm by removing `yarn.lock`
from version control, committing `package-lock.json` (remove it from
`.gitignore`), and ensuring CI and `package.json` scripts use `npm install`/`npm
publish`; update the `.gitignore` entry for `package-lock.json` accordingly
after making the chosen change.

Comment thread .gitignore
Copilot AI and others added 4 commits January 30, 2026 01:13
- Update CI workflow to use yarn instead of npm
- Remove package-lock.json from .gitignore
- Use yarn install --frozen-lockfile in CI for consistency
- Addresses CodeRabbit review feedback

Co-authored-by: warengonzaga <15052701+warengonzaga@users.noreply.github.com>
- Add --non-interactive flag to yarn publish for CI compatibility
- Keep package-lock.json in .gitignore as defensive measure

Co-authored-by: warengonzaga <15052701+warengonzaga@users.noreply.github.com>
- Add yarn build and yarn test to build job
- Add yarn build before publish step
- Add --access public flag to yarn publish
- Addresses code review feedback for proper CI validation

Co-authored-by: warengonzaga <15052701+warengonzaga@users.noreply.github.com>
- Allow test step to run without blocking releases
- Tests still execute for visibility but won't fail workflow
- Addresses issue with pre-existing test failures

Co-authored-by: warengonzaga <15052701+warengonzaga@users.noreply.github.com>
Copilot AI changed the title Apply security updates for dependency vulnerabilities Standardize package manager to Yarn and enhance CI workflow Jan 30, 2026
@warengonzaga warengonzaga merged commit f7cccb6 into main Jan 30, 2026
4 checks passed
@warengonzaga warengonzaga deleted the copilot/apply-security-update branch January 30, 2026 04:39
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