Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions pr-review/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion pr-review/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
"mollitia": "0.2.0",
"octokit": "5.0.3",
"parse-diff": "0.11.1",
"zod": "4.0.16"
"zod": "4.0.17"
Copy link

Choose a reason for hiding this comment

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

Consider using a caret (^) or tilde (~) version range instead of pinning to an exact patch number. This allows non-breaking updates (e.g., minor and patch releases) to be pulled in automatically, which can include important security fixes and performance improvements.

Example:

{
  "dependencies": {
    "zod": "^4.0.17"
  }
}

},
"devDependencies": {
"@eslint/js": "9.33.0",
Comment on lines 27 to 33
Copy link

Choose a reason for hiding this comment

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

I see that multiple dependencies are repeated across both pr-review and pr-summary packages. To avoid duplication and ensure consistency, you could consolidate shared dependencies in a workspace root package.json (if you're using a mono repo) or extract them into a shared config. This reduces maintenance overhead and potential version drift.

Example root package.json using npm workspaces:

{
  "private": true,
  "workspaces": [
    "pr-review",
    "pr-summary"
  ],
  "dependencies": {
    "mollitia": "^0.2.0",
    "octokit": "^5.0.3",
    "parse-diff": "^0.11.1",
    "zod": "^4.0.17"
  }
}

Expand Down
8 changes: 4 additions & 4 deletions pr-summary/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion pr-summary/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
"mollitia": "0.2.0",
"octokit": "5.0.3",
"parse-diff": "0.11.1",
"zod": "4.0.16"
"zod": "4.0.17"
Copy link

Choose a reason for hiding this comment

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

Similarly here, switch the exact patch version to a caret range to automatically receive non-breaking updates:

{
  "dependencies": {
    "zod": "^4.0.17"
  }
}

},
"devDependencies": {
"@eslint/js": "9.33.0",
Expand Down