Pre-cache MCP server packages and Playwright browser for Copilot#108
Conversation
Warm the npm cache for the stdio MCP servers the Copilot coding agent launches (@playwright/mcp, fast-filesystem-mcp, octocode-mcp, yggdrasil-mcp) and install Chromium for the playwright server, so they start reliably once the agent runtime firewall is enabled. https://claude.ai/code/session_01AfnF51garpZ7ajmV6M7ZMF
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
There was a problem hiding this comment.
Pull request overview
Updates the Copilot agent bootstrap workflow to pre-fetch MCP server npm packages and Playwright Chromium binaries before the agent runtime firewall is enabled, aiming to keep local MCP servers and Playwright usable without network access.
Changes:
- Added documentation clarifying the role and required name of the
copilot-setup-stepsjob. - Added a step to globally install several MCP server npm packages to warm caches ahead of firewall enablement.
- Added a step to install Playwright Chromium (and OS deps) ahead of firewall enablement.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge The incremental changes address the previous review concern about the MCP configuration comment by clarifying that MCP servers are configured in repo Settings -> Copilot -> coding agent (not in a file in this repo). This is a documentation improvement that resolves the issue. All previously flagged issues remain unchanged and will carry forward. Files Reviewed
Reviewed by laguna-m.1-20260312:free · 258,297 tokens |
Address PR review: the inline comment implied an in-repo Copilot MCP config file. It is configured under repo Settings -> Copilot -> coding agent, so reword to avoid pointing maintainers at a non-existent path. https://claude.ai/code/session_01AfnF51garpZ7ajmV6M7ZMF
|
Thanks for the review. Addressing the three points:
For the record, the original agent-run failures were not caused by this workflow — they were Generated by Claude Code |
Summary
Extends the
copilot-setup-stepsworkflow to pre-install and cache MCP (Model Context Protocol) server packages and Playwright browser binaries before the Copilot agent's runtime firewall is enabled. This ensures the agent can launch its local stdio MCP servers and use Playwright without network access once the firewall is active.Key Changes
Added clarifying comment on the
copilot-setup-stepsjob explaining its role in the agent setup pipeline and the requirement to maintain the job name for agent discovery.Pre-install local MCP server packages via
npm install -g:@playwright/mcp@latestfast-filesystem-mcp@latestoctocode-mcp@latestyggdrasil-mcp@latestPre-install Playwright browser (Chromium):
npx --yes playwright install --with-deps chromiumto fetch the browser binary and OS dependenciesImplementation Details
Both new steps run before the agent's firewall is enabled, warming the npm cache and pre-staging runtime dependencies so the agent can operate offline once network restrictions are in place. The steps include inline documentation to clarify their purpose and maintenance requirements.
https://claude.ai/code/session_01AfnF51garpZ7ajmV6M7ZMF