docs(integrations): stop recommending Corepack for pnpm - #2716
Open
antonvishal wants to merge 1 commit into
Open
docs(integrations): stop recommending Corepack for pnpm#2716antonvishal wants to merge 1 commit into
antonvishal wants to merge 1 commit into
Conversation
|
Contributor
There was a problem hiding this comment.
No issues found across 5 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Architecture diagram
sequenceDiagram
participant Dev as Developer
participant Docs as Integration Docs
participant Pnpm as pnpm CLI
participant Turbo as Turbo Build
participant Node as Node.js Runtime
participant CI as CI Pipeline
Note over Dev,CI: Integration Setup Flow (Docs Updated)
Dev->>Docs: Follow integration guide (Overview, CrewAI, Mastra, etc.)
Docs->>Dev: Clone Stagehand repo instructions
Dev->>Pnpm: pnpm install --frozen-lockfile
Note over Pnpm: Uses pinned pnpm@11.10.0<br/>via devEngines.packageManager
alt Corepack available (Node < 25)
Pnpm->>Pnpm: Falls back to Corepack if needed
else Corepack unavailable (Node >= 25)
Pnpm->>Pnpm: Uses standalone pnpm binary
end
Pnpm-->>Dev: Dependencies installed
Dev->>Pnpm: pnpm exec turbo run build
Pnpm->>Turbo: Execute turbo build
Turbo->>Node: Build integration packages
Node-->>Turbo: Build artifacts
Turbo-->>Dev: Build complete
Note over Dev,CI: Runtime Execution (Integration-Specific)
Dev->>Pnpm: pnpm --dir packages/integrations/<name> start
Pnpm->>Node: Launch integration process
Node->>Node: Load Stagehand MCP server
Note over Node: CrewAI, Mastra, Vercel AI SDK<br/>use shared TS facade MCP server
CI->>Pnpm: pnpm/action-setup (CI only)
CI->>Pnpm: pnpm install --frozen-lockfile
Pnpm-->>CI: Dependencies ready
CI->>Turbo: Run CI build checks
Turbo-->>CI: Validation results
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
corepack pnpm@11.10.0with plainpnpmin the v4 integration docs (overview, Mastra, Eve, Vercel AI SDK, CrewAI).pnpm now says not to install via Corepack (Node also stopped shipping Corepack from v25). We already pin
pnpm@11.10.0viadevEngines.packageManager, and CI usespnpm/action-setup, so the docs only used Corepack as a one-shot version pin.Summary by cubic
Stop recommending Corepack for
pnpmin v4 integration docs and switch commands to plainpnpm. This aligns withpnpmguidance and Node 25 removing Corepack while keeping our version pinning and CI setup intact.Review notes
corepack pnpm@11.10.0withpnpmincrewai.mdx,eve.mdx,mastra.mdx,vercel-ai-sdk.mdx, andoverview.mdx.pnpminstallation docs and kept the requirement atpnpm11.10.0.Written for commit 2efa8a9. Summary will update on new commits.