Skip to content

[rush] Fix ENOENT in AutoinstallerPluginLoader when destination folder doesn't exist#5699

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/fix-instructions-ignoring
Closed

[rush] Fix ENOENT in AutoinstallerPluginLoader when destination folder doesn't exist#5699
Copilot wants to merge 2 commits intomainfrom
copilot/fix-instructions-ignoring

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 12, 2026

PR #5678 switched from FileSystem.copyFile() to readFile() + writeFile() for line-ending normalization, but unlike fs-extra.copySync, writeFile() doesn't create missing parent directories. This causes ENOENT when the per-package subfolder under rush-plugins/ hasn't been created yet.

Changes

  • AutoinstallerPluginLoader.ts — add ensureFolderExists: true to both writeFile() calls (manifest and command-line.json):
FileSystem.writeFile(destinationManifestPath, manifestContent, {
  convertLineEndings: NewlineKind.Lf,
  ensureFolderExists: true
});
  • Change file — patch bump for @microsoft/rush

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…sion bump

Co-authored-by: octogonz <4673363+octogonz@users.noreply.github.com>
Copilot AI changed the title [WIP] [PR-5697] Fix ignoring of provided instructions [rush] Fix ENOENT regression in autoinstaller plugin file writes; add patch change entry Mar 12, 2026
Copilot AI requested a review from octogonz March 12, 2026 21:39
Copilot AI changed the title [rush] Fix ENOENT regression in autoinstaller plugin file writes; add patch change entry [rush] Fix ENOENT regression in autoinstaller plugin loader; add patch change entry Mar 12, 2026
Copilot AI changed the title [rush] Fix ENOENT regression in autoinstaller plugin loader; add patch change entry [rush] Fix ENOENT in AutoinstallerPluginLoader when destination folder doesn't exist Mar 12, 2026
@octogonz octogonz closed this Mar 12, 2026
@github-project-automation github-project-automation bot moved this from Needs triage to Closed in Bug Triage Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Closed

Development

Successfully merging this pull request may close these issues.

2 participants