Repository navigation
chore(monorepo): make admin/scripts the @docusaurus/monorepo-scripts workspace package, call copy-untyped-files without relative path (AI-assisted) - #12593
Merged
Conversation
slorber
added this pull request to stack #12594
October 8, 2026 15:41
1 of 3 tasks
✅ [V2]
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Size Change: 0 B Total Size: 12.4 MB ℹ️ View Unchanged
|
⚡️ Lighthouse report for the deploy preview of this PR
|
|
Size Change: 0 B Total Size: 12.6 MB ℹ️ View Unchanged
|
slorber
force-pushed
the
claude/admin-build-scripts
branch
from
October 8, 2026 18:35
82ec224 to
48bc281
Compare
slorber
force-pushed
the
claude/admin-build-scripts
branch
from
October 9, 2026 13:34
48bc281 to
035e909
Compare
…workspace package, call copy-untyped-files without relative path (AI-assisted) Turn admin/scripts into a private workspace package that declares the dependencies of its scripts (moved from the root devDependencies) and exposes copyUntypedFiles.js as the copy-untyped-files command, so package scripts no longer need the ../../admin/scripts path. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
slorber
force-pushed
the
claude/admin-build-scripts
branch
from
October 9, 2026 13:54
035e909 to
1bc2702
Compare
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.
Pre-flight checklist
Motivation
Preliminary work for the oxfmt upgrade (#12571), which needs a second build command in the theme packages (AI-assisted).
Package scripts called
node ../../admin/scripts/copyUntypedFiles.jswith a relative path in 10 packages. This PR turnsadmin/scriptsinto a private workspace package,@docusaurus/monorepo-scripts, which exposes the script as thecopy-untyped-filescommand:The root depends on it (
"@docusaurus/monorepo-scripts": "workspace:*"), so pnpm links the command into the rootnode_modules/.bin, which is on thePATHof every workspace package script. That's how the theme packages already runoxfmtwithout depending on it.commander,execa,image-sizeandsharpmove there from the rootdevDependencies, since onlyadmin/scriptsuses them at the root.@docusaurus/fs(declared asworkspace:*) andlodashstay at the root too, because root tests use them.image-sizeis added to the websitedevDependencies: a website test imports it and used to get it from the root.admin/scripts/*.jsentry moves from the root workspace to the newadmin/scriptsworkspace. syncpack: the package is added to the sources.copyUntypedFiles.jsonly gets a shebang. The other scripts are still run withnode admin/scripts/...as before.Test Plan
pnpm build:packagespasses.lib/CSS files are still copied (checked ontheme-commonfrom a deletedlib/).sharp, import fromadmin/scripts, andnode admin/scripts/resizeImage.js --helpworks.pnpm lint(including knip and syncpack),pnpm lint:deps,pnpm format:diffandpnpm testpass.Test links
Deploy preview: https://deploy-preview-12593--docusaurus-2.netlify.app/
No site change.
Related issues/PRs
.git-blame-ignore-revs)🤖 Generated with Claude Code