Skip to content

build(frontend): remove deprecated pnpm runtime path - #623

Merged
seonghobae merged 1 commit into
mainfrom
fix/pnpm-node24-deprecation
Aug 25, 2026
Merged

build(frontend): remove deprecated pnpm runtime path#623
seonghobae merged 1 commit into
mainfrom
fix/pnpm-node24-deprecation

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Outcome

  • pins the current stable pnpm 11.24.0 with registry integrity, removing pnpm 9's Node 24 url.parse() deprecation path
  • uses React 19 useEffectEvent for the requested-post transition so the Hook dependency warning is fixed without repeated post-open callbacks

Verification

  • corepack pnpm install --frozen-lockfile --ignore-scripts (supply-chain policy passed, no deprecation warning)
  • corepack pnpm run lint (no Hook warning)
  • corepack pnpm exec tsc -b
  • corepack pnpm run build (passes; the existing 539.41 kB chunk warning remains tracked as a separate performance gap in PR docs(product): establish current PRD and semantic gap baseline #621)

The isolated ontology continuation interaction passes with a diagnostic 15 s ceiling but exceeds Vitest's existing 5 s default under current local load; hosted exact-head tests remain authoritative for this package-manager-only runtime change.


Open in Devin Review

@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 7 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: c5f94b65-4117-450f-82fa-4398fe529629

📥 Commits

Reviewing files that changed from the base of the PR and between 48f013a and 930bf3a.

📒 Files selected for processing (2)
  • frontend/package.json
  • frontend/src/App.tsx

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@seonghobae
seonghobae enabled auto-merge (squash) August 25, 2026 11:53

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Open in Devin Review

Comment thread frontend/package.json
@seonghobae
seonghobae merged commit c8131a2 into main Aug 25, 2026
19 of 22 checks passed
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.

1 participant