feat: upgrade groq-js to latest version#401
feat: upgrade groq-js to latest version#401kbrabrand wants to merge 2 commits intoFormidableLabs:mainfrom
Conversation
🦋 Changeset detectedLatest commit: 9eb2c84 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@kbrabrand is attempting to deploy a commit to the formidable-labs Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Pull request overview
Updates the workspace to use a newer groq-js version, refreshing the lockfile and recording a release note for groqd.
Changes:
- Bump
groq-jsfrom^1.1.9to^1.26.0inpackages/groqd(devDependency). - Update
pnpm-lock.yamlto reflect the new dependency graph/resolutions. - Add a Changesets entry marking a
groqdminor release for the dependency update.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| pnpm-lock.yaml | Regenerates lockfile with groq-js@1.26.0 and related dependency resolution changes. |
| packages/groqd/package.json | Updates the groq-js version range used by groqd during development/tests. |
| .changeset/purple-pumas-occur.md | Adds a release note for a groqd minor bump tied to the dependency upgrade. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "devDependencies": { | ||
| "@sanity/client": "^6.24.1", | ||
| "groq-js": "^1.1.9", | ||
| "groq-js": "^1.26.0", | ||
| "rimraf": "^5.0.5", | ||
| "typescript": "^5.7.2", | ||
| "vitest": "^1.6.0" |
There was a problem hiding this comment.
groq-js is a devDependency here and (in this repo) appears to be used only under src/tests/** (e.g. src/tests/mocks/executeQuery.ts). If the goal is to ship groq-js improvements/bugfixes to groqd consumers, this change won’t affect runtime installs; consider either (a) clarifying the PR/changeset as a dev/test-only upgrade, or (b) moving groq-js to dependencies/peerDependencies if it’s intended to be part of the public runtime surface.
Description
Upgrade
groq-jsto latest version (minor bump) to get improvements and bugfixes into groqd.Type of Change
It should be backwards compatible, but a dep bump is still more than a bugfix.
How Has This Been Tested?
I've run the automated tests after upgrading
groq-jsto latest, and they all still pass.