Skip to content

chore: Update pnpm & dev dependencies - #15

Merged
stefashkaa merged 4 commits into
mainfrom
chore/update-dependencies
Sep 12, 2026
Merged

chore: Update pnpm & dev dependencies#15
stefashkaa merged 4 commits into
mainfrom
chore/update-dependencies

Conversation

@stefashkaa

Copy link
Copy Markdown
Owner

Summary

I've upgraded pnpm itself & updated all dev dependencies

Problem / Motivation

To utilize the latest possible dev tools

Type Of Change

  • Bug fix (non-breaking)
  • Feature (non-breaking)
  • Breaking change
  • Docs
  • Refactor / perf
  • CI / build / tooling

Testing

  • pnpm build:all
  • pnpm generate
  • pnpm generate:dev

Visual Diff (if applicable)

  • N/A

Checklist

  • I ran pnpm lint
  • I ran pnpm typecheck
  • I ran pnpm generate:dev
  • I updated docs and .env.example when needed
  • I added visual proof for rendering changes
  • I verified action behavior if action.yml or src/action.ts changed

Copilot AI lite review requested due to automatic review settings September 12, 2026 00:05
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 12, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-12T00:09:16.915215Z 12ef032 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@sonarqubecloud

Copy link
Copy Markdown

Copilot AI 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.

🟡 Changes recommended

The updated pnpm-lock.yaml appears to contain two YAML documents / duplicated lockfile sections, which is likely to break pnpm install and CI.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

This PR updates the repository’s tooling stack by bumping pnpm and various (mostly dev) dependencies, along with a small build-config adjustment to align path resolution with ESM usage.

Changes:

  • Bump pnpm version and refresh dependency versions (incl. svgo, eslint/typescript-eslint, vite, etc.).
  • Update Vite config entrypoint path resolution from __dirname to import.meta.dirname.
  • Update the workflow to use a newer pinned SHA for actions/checkout.
File summaries
File Description
vite.config.ts Adjusts Rollup input path resolution for an ESM config environment.
pnpm-lock.yaml Updates lockfile for new pnpm/dev dependency versions (but currently appears malformed).
package.json Bumps pnpm + dependencies and adds helper scripts for peer/audit checks.
.github/workflows/generate-crt-contributions.yml Updates pinned actions/checkout revision.
Review details

Files not reviewed (1)

  • pnpm-lock.yaml: Generated file
  • Files reviewed: 3/5 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread pnpm-lock.yaml

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 12ef0322ff

ℹ️ 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 thread pnpm-lock.yaml
@stefashkaa
stefashkaa merged commit e1910d4 into main Sep 12, 2026
3 checks passed
@stefashkaa
stefashkaa deleted the chore/update-dependencies branch September 12, 2026 00:11
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