Skip to content

fix(deps): add explicit PostCSS dependency#601

Closed
rwjblue wants to merge 1 commit into
gleanwork:mainfrom
rwjblue:rwjblue/deps-add-postcss-dependency
Closed

fix(deps): add explicit PostCSS dependency#601
rwjblue wants to merge 1 commit into
gleanwork:mainfrom
rwjblue:rwjblue/deps-add-postcss-dependency

Conversation

@rwjblue

@rwjblue rwjblue commented Jul 1, 2026

Copy link
Copy Markdown

BLUF: Add PostCSS explicitly to stabilize Docusaurus peer resolution.

Recent Dependabot PRs regenerated pnpm-lock.yaml with a Docusaurus/OpenAPI peer graph that no longer attached PostCSS to the top-level Docusaurus package snapshots. That caused pnpm build to fail during static generation for docs routes with ReactContextError from useDoc()/DocItemMetadata. Making PostCSS an explicit dev dependency keeps regenerated lockfiles stable across those dependency bumps.

Validation

  • Reproduced the failing Dependabot shape locally by applying PR chore(deps): bump react-tooltip from 5.30.1 to 6.0.5 #598's package/lockfile changes onto latest main; pnpm build failed with the same ReactContextError.
  • Confirmed latest main builds before the repro.
  • Confirmed the minimal PostCSS dependency fix builds successfully.
  • mise x -- pnpm install --frozen-lockfile
  • mise x -- pnpm build
  • mise x -- pnpm test
  • mise x -- pnpm prettier --check .

Dependabot lockfile refreshes can resolve the Docusaurus/OpenAPI peer graph
without PostCSS attached to the top-level Docusaurus packages. That makes
static generation render docs pages without the expected docs context.

Add PostCSS explicitly so regenerated lockfiles keep the peer graph stable.
@vercel

vercel Bot commented Jul 1, 2026

Copy link
Copy Markdown

@rwjblue-glean is attempting to deploy a commit to the Glean Team on Vercel.

A member of the Team first needs to authorize it.

@rwjblue-glean rwjblue-glean marked this pull request as ready for review July 1, 2026 21:15
@rwjblue-glean rwjblue-glean requested a review from a team as a code owner July 1, 2026 21:15
@rwjblue-glean

Copy link
Copy Markdown
Member

Closing this duplicate fork-based PR. Recreated from the upstream branch as #602.

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