Repository navigation
chore(deps): update npm dependencies - #205
ejscribner merged 5 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Code Review
This pull request updates several dependencies in package.json, notably upgrading React to v19 and Tailwind CSS to v4, and updates postcss.config.cjs to use @tailwindcss/postcss. The review feedback recommends removing autoprefixer from both package.json and postcss.config.cjs because Tailwind CSS v4 handles vendor prefixing automatically.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| "@tailwindcss/postcss": "^4.3.2", | ||
| "autoprefixer": "^10.5.2", |
There was a problem hiding this comment.
| module.exports = { | ||
| plugins: { | ||
| tailwindcss: {}, | ||
| '@tailwindcss/postcss': {}, |
There was a problem hiding this comment.
@ejscribner Yes, and it's already done. Gemini is right: Tailwind 4's @tailwindcss/postcss plugin runs Lightning CSS, which adds vendor prefixes itself, so autoprefixer isn't needed anymore. It was removed in b53f6fe (part of the Tailwind 4 migration):
postcss.config.cjsnow has only'@tailwindcss/postcss': {}autoprefixeris gone frompackage.jsondevDependencies, and nothing inpackage-lock.jsondepends on it (npm ls autoprefixerreturns empty)
I checked the current head (cc1cad3) again: npm ci, npm run check (prettier + eslint) and next build all pass. The compiled CSS still has -webkit-* prefixes (for example -webkit-appearance), so we don't lose prefixing. No new commit was needed.
…0260703T172451Z Resolve the package-lock.json conflict by keeping this branch's lockfile and re-resolving it with npm on Node 24. Hermes task: t_93148fc1 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Bump direct dependencies to their latest stable in-range releases (next/eslint-config-next 16.3.7, react/react-dom 19.3.0, axios 1.20.0, qs 6.16.0, jest 30.5.2, tailwindcss 4.3.3, postcss 8.5.28, prettier 3.9.9, and others) and raise the tar override to 7.5.22 to clear the critical node-tar advisories. eslint stays on 9.x because eslint-plugin-react does not yet support eslint 10. Hermes task: t_93148fc1 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The tailwindcss 3 -> 4 bump left styles/globals.css on the v3 '@tailwind' directives, which Tailwind 4 no longer processes, so most utility classes were not generated and the UI rendered unstyled (e.g. invisible heroicons, no spacing). Apply the official @tailwindcss/upgrade codemod output: - switch globals.css to '@import "tailwindcss"' plus the v3 border-color compatibility layer - rename utilities whose scale changed (shadow-sm -> shadow-xs, rounded -> rounded-sm, flex-shrink-0 -> shrink-0, arbitrary left offset -> left-128) - drop tailwind.config.cjs (content is auto-detected in v4) - drop autoprefixer, which Tailwind 4 handles internally Hermes task: t_93148fc1 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Hermes refreshed this PR for Kanban task t_93148fc1 (board
Local validation on Node 24.21.0 against a local Couchbase 7.6.4: |
There was a problem hiding this comment.
@dex-the-ai please remove this file, it is unwanted clutter
There was a problem hiding this comment.
@ejscribner Done — removed .github/dependency-update-evidence/depupdate-20260703T172451Z-tests-passing.png in cc1cad3. Nothing else in the repo referenced it, and the PR diff no longer includes any .github/ files. This is a file deletion only, so no application code changed.
Addresses review feedback: the file is unwanted clutter. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
@dex-the-ai can you address the comment above from gemini-code-assist:
|
|
@ejscribner This Gemini suggestion was already applied in b53f6fe: I checked the current head (cc1cad3) again: |
Summary
Updates direct npm dependencies for
couchbase-examples/nextjs-quickstartand regeneratespackage-lock.jsonwith the existing npm workflow (Node 24 / npm 11).This PR was opened by @deniswsrosa on 2026-07-06. It was refreshed on 2026-09-29 as part of Hermes Kanban task t_93148fc1 (board
couchbase-examples) rather than opening a second dependency PR. The refresh:main, which fixes thepackage-lock.jsonconflict left by Bump ws from 8.20.1 to 8.21.1 #206 (wsbump).taroverride to7.5.22.npm auditnow reports 0 vulnerabilities; before, there was a criticalnode-taradvisory reached throughcouchbase→cmake-js.tailwindcssto v4 and switched the PostCSS plugin, butstyles/globals.cssstill used the v3@tailwinddirectives. Tailwind 4 ignores those, so most utilities were never generated and the app rendered unstyled: invisible heroicons, no spacing or colors. CI didn't catch this because the Jest suite only covers the API routes. This PR applies the official@tailwindcss/upgradecodemod output:@import 'tailwindcss'plus a border-color compatibility layer so borders keep their v3 colorshadow-sm→shadow-xs,rounded→rounded-sm,flex-shrink-0→shrink-0, and an arbitrary left offset→left-128tailwind.config.cjs(v4 detects content automatically)autoprefixer, which Tailwind 4 handles itself (the Gemini review suggested this too)Updated dependencies (vs
main)@headlessui/react^1.7.19^2.2.10Dialog.Panel/Transition.Childstill supported, UI verifiedaxios^1.18.0^1.20.0couchbase^4.7.0^4.7.1next^16.2.9^16.3.7qs^6.15.2^6.16.0react^18.3.1^19.3.0react-dom^18.3.1^19.3.0@tailwindcss/postcss^4.3.3autoprefixer^10.5.0env-cmd^10.1.0^11.0.0-fusage unchangedeslint^9.39.4^9.39.5eslint-config-next^16.2.9^16.3.7eslint-config-prettier^8.10.2^10.1.8jest^29.7.0^30.5.2jest-environment-jsdom^29.7.0^30.5.2next-test-api-route-handler^5.0.5^5.0.7postcss^8.5.15^8.5.28prettier^3.8.4^3.9.9tailwindcss^3.4.19^4.3.3tar(override)7.5.167.5.22couchbase→cmake-js)Overlapping Dependabot PRs: this PR covers the direct bumps from #217 and #212, and the lockfile regeneration resolves the transitive bumps in #216, #215, #214, #213, #211, #210 and #209. Those PRs can be closed once this merges.
Validation
All commands ran locally from a clean
npm cion Node v24.21.0 / npm 11.19.0, against a disposable local Couchbase Server 7.6.4 container set up the same way as CI (kv, n1ql and index services, memory-optimized indexes,Administrator/password, bucketuser_profile).npm ci npm run init-db:default npm run check npm test CB_USERNAME=Administrator CB_PASSWORD=password CB_CONNECT_STRING=couchbase://localhost CB_BUCKET=user_profile npm run build npm audit npm outdatednpm cinpm run init-db:defaultnpm run check(prettier + eslint--max-warnings=0)npm testnpm run build(Next.js 16.3.7 Turbopack, sample data + indexes)npm auditnpm outdatedeslint10.x remains (held intentionally)Browser evidence
I ran
npm run dev:defaultagainst the local Couchbase and drove it with Playwright (Chromium, 1280×800). The walkthrough loads the directory list, searches for a user, opens their profile, adds a new user through the Headless UI modal, then searches for the new record, which was written to Couchbase and read back.Profile view (after search)
Add-user modal (Headless UI v2 + Tailwind 4)
Created user read back from Couchbase
Initial list view: screenshot
Walkthrough video: Open the UI walkthrough (webm)
The browser logged no page errors. The only console messages were dev-mode HMR websocket handshake failures, caused by the Docker port mapping used locally; they are unrelated to the app.
Risk notes
eslint-plugin-react@7.37.5, pulled in byeslint-config-next, only declares peer support up to eslint^9.7, and an earlier eslint 10 attempt failed withscopeManager.addGlobals is not a function..github/dependency-update-evidence/depupdate-20260703T172451Z-tests-passing.png. This refresh left it alone; the owner can drop it before merge if the repo doesn't want committed evidence images.npm audit fix --forcewas used.Rollback
Revert this PR to restore the previous
package.json,package-lock.json,postcss.config.cjs,tailwind.config.cjs,styles/globals.cssand component class names.Hermes task:
t_93148fc1(boardcouchbase-examples): reconcile the dependency sweep through this existing PR instead of opening a duplicate.🤖 Generated with Claude Code