Conversation
WalkthroughThe pull request updates version constraints for the Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip Migrating from UI to YAML configuration.Use the |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
starter/package.json (1)
56-56: Consider using tilde range to allow patch updates in devDependencies.Exact pinning (
2.0.2) prevents all patch updates, including potential security fixes and bug fixes for 2.0.x releases. If the intent is to avoid minor version changes (e.g., to block breaking changes in 2.1.0+), use~2.0.2instead, which allows patch updates while preventing minor and major version bumps.♻️ Proposed alternative using tilde range
- "@yext/pages-components": "2.0.2", + "@yext/pages-components": "~2.0.2",🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@starter/package.json` at line 56, The dependency "@yext/pages-components" is pinned to an exact version "2.0.2" which prevents patch updates; change its entry in package.json to use a tilde range (~2.0.2) so patch releases (2.0.x) are allowed while still blocking minor/major bumps—update the devDependencies entry for "@yext/pages-components" accordingly and run your lockfile update (npm/yarn/pnpm install) to refresh the lockfile.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@packages/visual-editor/package.json`:
- Line 156: The peerDependency "@yext/pages-components" is pinned to an exact
version ("2.0.2") which can cause install conflicts; update the package.json
peerDependencies entry for "@yext/pages-components" to allow patch updates but
block 2.1+ (e.g., use a tilde range "~2.0.2" or an explicit range ">=2.0.2
<2.1.0") so consumers can receive patch/bugfix releases without permitting a
minor version upgrade.
---
Nitpick comments:
In `@starter/package.json`:
- Line 56: The dependency "@yext/pages-components" is pinned to an exact version
"2.0.2" which prevents patch updates; change its entry in package.json to use a
tilde range (~2.0.2) so patch releases (2.0.x) are allowed while still blocking
minor/major bumps—update the devDependencies entry for "@yext/pages-components"
accordingly and run your lockfile update (npm/yarn/pnpm install) to refresh the
lockfile.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: fffa6933-b947-4588-82b4-762dae4a68b4
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (2)
packages/visual-editor/package.jsonstarter/package.json
I don't think we want to upgrade mapbox yet til every part of it is ready. See: https://yext.slack.com/archives/C02UVSE7P6W/p1773854139855149?thread_ts=1773334309.653709&cid=C02UVSE7P6W