Skip to content

fix(developer-hub): unblock local build#3713

Open
keyvankhademi wants to merge 1 commit into
mainfrom
fix-devhub
Open

fix(developer-hub): unblock local build#3713
keyvankhademi wants to merge 1 commit into
mainfrom
fix-devhub

Conversation

@keyvankhademi
Copy link
Copy Markdown
Contributor

@keyvankhademi keyvankhademi commented May 18, 2026

  • Bump @internationalized/date in component-library to ^3.12.0 to match the version react-aria resolves to (avoids nominal #private type errors in DatePicker).
  • Update two pino logger.error calls to the {err, msg} pattern accepted by pino 9's stricter LogFn overloads.
  • Bump tsx to 4.22.2; 4.20.6 misclassified fumadocs-openapi's rolldown-bundled dereference-json-schema as CJS, breaking generate:docs.

Summary

Rationale

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

Open in Devin Review

- Bump @internationalized/date in component-library to ^3.12.0 to match
  the version react-aria resolves to (avoids nominal #private type errors
  in DatePicker).
- Update two pino logger.error calls to the {err, msg} pattern accepted
  by pino 9's stricter LogFn overloads.
- Bump tsx to 4.22.2; 4.20.6 misclassified fumadocs-openapi's
  rolldown-bundled dereference-json-schema as CJS, breaking generate:docs.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 18, 2026 19:39
@keyvankhademi keyvankhademi requested a review from a team as a code owner May 18, 2026 19:39
@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
api-reference Error Error May 18, 2026 7:44pm
component-library Ready Ready Preview, Comment May 18, 2026 7:44pm
developer-hub Ready Ready Preview, Comment May 18, 2026 7:44pm
entropy-explorer Error Error May 18, 2026 7:44pm
insights Error Error May 18, 2026 7:44pm
proposals Ready Ready Preview, Comment May 18, 2026 7:44pm
staking Error Error May 18, 2026 7:44pm

Request Review

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR unblocks local builds for the developer hub by aligning dependency/tooling versions with what the build actually resolves to, and by updating pino logging calls to match pino@9’s stricter TypeScript overloads.

Changes:

  • Bump tsx in the workspace catalog to 4.22.2 to avoid a CJS/ESM misclassification during docs generation.
  • Bump @internationalized/date in @pythnetwork/component-library to ^3.12.0 to avoid nominal type conflicts with the version pulled in by react-aria.
  • Update two logger.error call sites to the { err } + message form compatible with pino@9 typings.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

File Description
pnpm-workspace.yaml Updates the workspace catalog version for tsx to restore local/docs build behavior.
packages/component-library/src/useQueryParamsPagination/index.ts Adjusts a pino error log call signature to match pino@9 typing.
packages/component-library/src/AppShell/report-accessibility.ts Adjusts a pino error log call signature to match pino@9 typing.
packages/component-library/package.json Bumps @internationalized/date to align with react-aria-resolved version and avoid TS nominal type issues.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

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