docs(integrations): stop recommending Corepack for pnpm - #2716
docs(integrations): stop recommending Corepack for pnpm#2716antonvishal wants to merge 1 commit into
Conversation
|
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 PR was approved by @miguelg719 and mirrored to #2764. All further discussion should happen on that PR. |
…pm (#2764) Mirrored from external contributor PR #2716 after approval by @miguelg719. Original author: @antonvishal Original PR: #2716 Approved source head SHA: `2efa8a946ad33c6430c4bc668aa2e4d31e2d3006` @antonvishal, please continue any follow-up discussion on this mirrored PR. When the external PR gets new commits, this same internal PR will be marked stale until the latest external commit is approved and refreshed here. ## Original description <img width="433" height="66" alt="Screenshot 2026-08-14 at 10 09 40 PM" src="https://github.com/user-attachments/assets/b9b4783f-3c1f-451a-aa15-69fe4568b87f" /> ## Summary - Replace `corepack pnpm@11.10.0` with plain `pnpm` in the v4 integration docs (overview, Mastra, Eve, Vercel AI SDK, CrewAI). - Link the integrations overview to the [official pnpm install docs](https://pnpm.io/installation). pnpm now says not to install via Corepack (Node also stopped shipping Corepack from v25). We already pin `pnpm@11.10.0` via `devEngines.packageManager`, and CI uses `pnpm/action-setup`, so the docs only used Corepack as a one-shot version pin. <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Stop recommending Corepack for `pnpm` in v4 integration docs and switch commands to plain `pnpm`. This aligns with `pnpm` guidance and Node 25 removing Corepack while keeping our version pinning and CI setup intact. **Review notes** - Replaced `corepack pnpm@11.10.0` with `pnpm` in `crewai.mdx`, `eve.mdx`, `mastra.mdx`, `vercel-ai-sdk.mdx`, and `overview.mdx`. - Linked the overview to the official `pnpm` installation docs and kept the requirement at `pnpm` 11.10.0. - No code, build, or runtime changes; docs only. <sup>Written for commit 2efa8a9. Summary will update on new commits.</sup> <a href="https://cubic.dev/pr/browserbase/stagehand/pull/2716?utm_source=github" target="_blank" rel="noopener noreferrer" data-no-image-dialog="true"><picture><source media="(prefers-color-scheme: dark)" srcset="https://www.cubic.dev/buttons/review-in-cubic-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://www.cubic.dev/buttons/review-in-cubic-light.svg"><img alt="Review in cubic" src="https://www.cubic.dev/buttons/review-in-cubic-dark.svg"></picture></a> <!-- End of auto-generated description by cubic. --> <!-- external-contributor-pr:owned source-pr=2716 source-sha=2efa8a946ad33c6430c4bc668aa2e4d31e2d3006 claimer=miguelg719 --> Co-authored-by: Vishal Anton <vishalanton@appexert.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
|
The mirrored PR #2764 has been merged into |
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.