Build: Refactor root of yarn monorepo from code to the git root#33194
Conversation
kasperpeulen
requested review from
jonniebigodes and
kylegach
as code owners
|
View your CI Pipeline Execution ↗ for commit a4f5eda
☁️ Nx Cloud last updated this comment at |
kasperpeulen
changed the title
Package BenchmarksCommit: The following packages have significant changes to their size or dependencies:
|
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 18 | 18 | 0 |
| Self size | 1.65 MB | 1.80 MB | 🚨 +149 KB 🚨 |
| Dependency size | 9.25 MB | 9.25 MB | 🎉 -280 B 🎉 |
| Bundle Size Analyzer | Link | Link |
@storybook/builder-vite
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 17 | 17 | 0 |
| Self size | 304 KB | 325 KB | 🚨 +20 KB 🚨 |
| Dependency size | 2.00 MB | 2.00 MB | 🎉 -152 B 🎉 |
| Bundle Size Analyzer | Link | Link |
storybook
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 39 | 39 | 0 |
| Self size | 19.64 MB | 20.51 MB | 🚨 +871 KB 🚨 |
| Dependency size | 16.40 MB | 16.40 MB | 0 B |
| Bundle Size Analyzer | Link | Link |
@storybook/html-vite
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 20 | 20 | 0 |
| Self size | 22 KB | 22 KB | 🎉 -128 B 🎉 |
| Dependency size | 2.34 MB | 2.36 MB | 🚨 +20 KB 🚨 |
| Bundle Size Analyzer | Link | Link |
@storybook/nextjs-vite
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 128 | 128 | 0 |
| Self size | 1.12 MB | 1.12 MB | 🚨 +796 B 🚨 |
| Dependency size | 21.95 MB | 21.97 MB | 🚨 +20 KB 🚨 |
| Bundle Size Analyzer | Link | Link |
@storybook/preact-vite
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 20 | 20 | 0 |
| Self size | 13 KB | 13 KB | 🎉 -140 B 🎉 |
| Dependency size | 2.33 MB | 2.35 MB | 🚨 +20 KB 🚨 |
| Bundle Size Analyzer | Link | Link |
@storybook/react-native-web-vite
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 160 | 160 | 0 |
| Self size | 30 KB | 30 KB | 🎉 -140 B 🎉 |
| Dependency size | 23.13 MB | 23.15 MB | 🚨 +20 KB 🚨 |
| Bundle Size Analyzer | Link | Link |
@storybook/react-vite
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 118 | 118 | 0 |
| Self size | 35 KB | 35 KB | 🎉 -140 B 🎉 |
| Dependency size | 19.75 MB | 19.77 MB | 🚨 +20 KB 🚨 |
| Bundle Size Analyzer | Link | Link |
@storybook/svelte-vite
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 24 | 24 | 0 |
| Self size | 56 KB | 55 KB | 🎉 -152 B 🎉 |
| Dependency size | 27.00 MB | 27.02 MB | 🚨 +20 KB 🚨 |
| Bundle Size Analyzer | Link | Link |
@storybook/sveltekit
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 25 | 25 | 0 |
| Self size | 56 KB | 56 KB | 🎉 -164 B 🎉 |
| Dependency size | 27.06 MB | 27.08 MB | 🚨 +20 KB 🚨 |
| Bundle Size Analyzer | Link | Link |
@storybook/vue3-vite
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 114 | 114 | 0 |
| Self size | 35 KB | 35 KB | 🎉 -164 B 🎉 |
| Dependency size | 44.15 MB | 44.17 MB | 🚨 +20 KB 🚨 |
| Bundle Size Analyzer | Link | Link |
@storybook/web-components-vite
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 21 | 21 | 0 |
| Self size | 19 KB | 19 KB | 🎉 -152 B 🎉 |
| Dependency size | 2.37 MB | 2.39 MB | 🚨 +20 KB 🚨 |
| Bundle Size Analyzer | Link | Link |
@storybook/cli
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 173 | 173 | 0 |
| Self size | 773 KB | 774 KB | 🚨 +802 B 🚨 |
| Dependency size | 66.73 MB | 67.61 MB | 🚨 +871 KB 🚨 |
| Bundle Size Analyzer | Link | Link |
@storybook/codemod
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 166 | 166 | 0 |
| Self size | 30 KB | 30 KB | 🎉 -140 B 🎉 |
| Dependency size | 65.31 MB | 66.18 MB | 🚨 +871 KB 🚨 |
| Bundle Size Analyzer | Link | Link |
create-storybook
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 40 | 40 | 0 |
| Self size | 999 KB | 999 KB | 🎉 -59 B 🎉 |
| Dependency size | 36.04 MB | 36.91 MB | 🚨 +871 KB 🚨 |
| Bundle Size Analyzer | node | node |
📝 WalkthroughWalkthroughWidespread CI and workspace reorganization plus sandbox orchestration: CircleCI jobs and GitHub actions paths/caching adjusted; SANDBOX_DIRECTORY and related envs centralized; new prepare-sandbox and sandbox caching/copy/port handling added; many Nx/project manifests and package.json workspace/layout changes; numerous scripts reworked to use ROOT_DIRECTORY and per-template ports. Changes
Sequence Diagram(s)mermaid Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Areas to focus during review:
Possibly related PRs
✨ Finishing touches
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (10)
🚧 Files skipped from review as they are similar to previous changes (7)
🧰 Additional context used📓 Path-based instructions (1)**/*.{ts,tsx,js,jsx,json,html,mjs}📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Files:
🧠 Learnings (8)📓 Common learnings📚 Learning: 2025-11-24T17:49:31.838ZApplied to files:
📚 Learning: 2025-11-24T17:49:47.659ZApplied to files:
📚 Learning: 2025-11-24T17:49:47.659ZApplied to files:
📚 Learning: 2025-11-24T17:49:47.659ZApplied to files:
📚 Learning: 2025-11-24T17:49:47.659ZApplied to files:
📚 Learning: 2025-11-24T17:49:47.659ZApplied to files:
📚 Learning: 2025-11-24T17:49:47.659ZApplied to files:
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
🔇 Additional comments (8)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 11
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (3)
scripts/knip.config.ts (1)
84-93: Workspace pattern mismatch breaks bundler-entry enrichmentWith
baseDirnow pointing at the git root, the workspace matching logic is broken:
fg(rootManifest.workspaces.packages, { cwd: baseDir })yields dirs likecode/addons/a11y,code/renderers/react, etc., because the rootpackage.jsonworkspaces globs all start withcode/(e.g.,"code/addons/*","code/renderers/*").wsDirtherefore includes thecode/prefix (e.g.,"code/addons/a11y").match([wsDir], configKey)compares against keys like"addons/*","renderers/*"(nocode/prefix).- The pattern
"addons/*"does not match"code/addons/a11y", so no workspace directory matches any config key.As a result, bundler entries from each
package.jsonare never merged into the Knip config. Static analysis will still run, but without those enriched entrypoints.Fix this by stripping the
code/prefix before matching:const workspaceDirectories = workspaceDirs.map((dir) => relative(baseDir, join(baseDir, dir)).replace(/^code\//, ''));Or update
baseConfig.workspaceskeys to include thecode/prefix to match the root workspace globs..github/copilot-instructions.md (1)
20-34: Clarify sandbox location vs new SANDBOX_ROOT behaviorThe docs still talk about a root
sandbox/directory (and contrast it withcode/sandbox/), but the new tooling introducesSANDBOX_ROOT(e.g.,../storybook-sandboxesfor agents). It would help to explicitly call out:
- Where sandboxes live by default for contributors running
yarn task --task sandboxlocally.- How that differs in CI (Nx agents) where
SANDBOX_ROOTis overridden.- How to opt back into the “old”
./sandboxbehavior via env.Right now it’s easy to assume everything always lands in
./sandbox, which may not match the new defaults.Also applies to: 193-201, 214-230
scripts/release/version.ts (1)
54-86: GuardpreIdvalidation against missingreleaseTypeIn
optionsSchema.superRefine, this condition can throw ifpreIdis provided withoutreleaseType:if (schema.preId && !schema.releaseType.startsWith('pre')) {
schema.releaseTypeis optional, so calling.startsWithonundefinedwill crash before Zod can surface a helpful validation error.A small defensive tweak avoids that:
- if (schema.preId && !schema.releaseType.startsWith('pre')) { + if (schema.preId && !schema.releaseType?.startsWith('pre')) {This preserves the intended validation while preventing a runtime TypeError for invalid option combinations.
♻️ Duplicate comments (8)
code/addons/onboarding/project.json (1)
5-8: Consistent with monorepo standardization; verify executor mappings upstream.The changes follow the established pattern across library and addon projects in this PR. Ensure the
checktarget executor is defined in the parent Nx configuration.code/presets/react-webpack/project.json (1)
5-8: Consistent with preset standardization pattern.The changes align with the monorepo-wide refactoring. Verify the
checktarget executor is mapped in parent Nx configuration.code/frameworks/vue3-vite/project.json (1)
5-8: Framework standardization pattern confirmed.Changes follow the repo-wide pattern. Confirm
checktarget executor is defined upstream in Nx configuration.code/sandbox/react-native-web-vite-expo-ts/project.json (1)
1-27: Consistent configuration pattern.This follows the same target structure as other sandbox projects in this PR. The empty target definitions should be verified similarly (see comment on bench-react-webpack-18-ts-test-build/project.json).
code/sandbox/nextjs-14-ts/project.json (1)
1-25: Consistent configuration pattern.This Next.js 14 TypeScript sandbox follows the established pattern with similar target definitions and dependency structure.
.circleci/src/jobs/vitest-integration.yml (1)
18-20: Verify sandbox path resolution (same pattern as e2e-production.yml).The
sandbox/prefix was removed. Ensure thatyarn get-sandbox-dir --template $TEMPLATEreturns the correct full path..circleci/src/jobs/bench-sandboxes.yml (1)
18-20: Verify sandbox path resolution (same pattern as other jobs).The
sandbox/prefix was removed. Ensure thatyarn get-sandbox-dir --template $TEMPLATEreturns the correct full path..circleci/src/jobs/create-sandboxes.yml (1)
34-34: Verify sandbox path resolution (same pattern as other jobs).The
sandbox/prefix was removed. Ensure thatyarn get-sandbox-dir --template $TEMPLATEreturns the correct full path so thecdandrm -rf node_modulescommands work correctly.
🧹 Nitpick comments (23)
code/lib/csf-plugin/project.json (1)
4-8: Verify that empty targets are properly inherited or configured elsewhere.The "build" and "check" targets (lines 5–6) are defined with empty configurations. Per the PR's goal of moving build/check scripts to Nx targets, please confirm these targets are either:
- Inheriting configuration from a base/shared Nx config (e.g., a root preset or extends field)
- Configured via environment-specific Nx settings or workspace plugins
- Intentionally empty as placeholders pending explicit wiring
The addition of
"tags": ["ci:normal"](line 8) suggests this file is part of a new CI orchestration strategy. Ensure this tag is recognized by the Nx CI pipeline and that agent routing works as intended. Also verify that the removal of$schemadoes not degrade IDE validation or Nx CLI introspection for developers.To help verify the broader configuration strategy, could you confirm:
- Which files define the actual implementation of these "check" and "build" targets (e.g., a root nx.json, base config, or workspace preset)?
- Whether the schema reference should be retained or centralized at the workspace level for IDE support.
code/sandbox/angular-cli-default-ts/project.json (1)
4-12: Verify implicitDependencies comprehensiveness and necessity.The list expanded significantly to include 7 packages. While these appear to be relevant storybook packages, please confirm that:
- All listed dependencies are actually used or transitively required by this sandbox
- No additional dependencies are missing (e.g., are all @storybook/* addons that this sandbox needs included?)
- This matches the pattern used in other sandbox configurations
docs/_snippets/addon-consume-and-update-globaltype.md (1)
17-21: Clarify the comma operator usage in documentation example.The comma operator pattern
(updateGlobals({...}), addons.getChannel().emit(...))is valid but semantically unclear in a documentation context. Developers reading this example may question why the result of the comma expression is discarded or misunderstand the intent. For documentation clarity, use two separate statements instead.const refreshAndUpdateGlobal = () => { - (updateGlobals({ + updateGlobals({ ['my-param-key']: !isActive, - }), - // Invokes Storybook's addon API method (with the FORCE_RE_RENDER) event to trigger a UI refresh - addons.getChannel().emit(FORCE_RE_RENDER)); + }); + // Invokes Storybook's addon API method (with the FORCE_RE_RENDER) event to trigger a UI refresh + addons.getChannel().emit(FORCE_RE_RENDER); };docs/_snippets/storybook-addons-api-getchannel.md (1)
16-20: Clarify the comma operator usage in documentation example (duplicate pattern).Same issue as
addon-consume-and-update-globaltype.md: the comma operator pattern is semantically unclear for documentation examples. This pattern appears across multiple snippet files and should be simplified for clarity. Use two separate statements instead.const refreshAndUpdateGlobal = () => { - (updateGlobals({ + updateGlobals({ ['my-param-key']: !isActive, - }), - // Invokes Storybook's addon API method (with the FORCE_RE_RENDER) event to trigger a UI refresh - addons.getChannel().emit(FORCE_RE_RENDER)); + }); + // Invokes Storybook's addon API method (with the FORCE_RE_RENDER) event to trigger a UI refresh + addons.getChannel().emit(FORCE_RE_RENDER); };scripts/build/utils/generate-type-mappers.ts (1)
39-39: LGTM! Path resolution correctly updated for monorepo restructuring.The addition of
join(cwd, ...)properly makes the generated type mapper file paths absolute, which is necessary given the monorepo root is moving from the code directory to the git root. This ensures the type declaration files are written to the correct location regardless of where the build script is invoked from.Optional: Consider more robust path transformation.
While not introduced by this PR, the string replacement pattern
.replace('src', 'dist')replaces only the first occurrence of 'src', which could be fragile if entry point paths contain 'src' in multiple locations (e.g.,src/source-maps/file.ts). Consider using path manipulation utilities or regex with anchors for more robust transformation in a future refactor.code/renderers/vue3/template/stories_vue3-vite-default-ts/ScopedSlots.stories.ts (1)
14-17: Avoidanyon the global addons channel (use a minimal structural type instead).Changing
__STORYBOOK_ADDONS_CHANNEL__toanyweakens type safety in a template file users are likely to copy from. You can keep strictness without importing internal types by defining a small structural type for just what this story needs:-declare global { - // eslint-disable-next-line no-var - var __STORYBOOK_ADDONS_CHANNEL__: any; -} +type StorybookAddonsChannel = { + emit: (event: string, ...args: any[]) => void; + once: (event: string, callback: (...args: any[]) => void) => void; +}; + +declare global { + // eslint-disable-next-line no-var + var __STORYBOOK_ADDONS_CHANNEL__: StorybookAddonsChannel; +}This preserves the looseness you likely want in a template while still aligning with the repo’s strict TypeScript guidelines.
code/sandbox/solid-vite-default-js/project.json (1)
13-22: Addvitest-integrationtarget for consistency with TS variant.The TypeScript counterpart includes a
vitest-integrationtarget, but this file omits it. If both variants should support the same workflows, add this target. If the JS variant genuinely does not need it, document the rationale in the PR description or add a comment explaining the divergence.If this target should be included, apply this diff:
"targets": { "sandbox": { "options": { "dir": "solid-vite-default-js" } }, "prepared-sandbox": {}, "dev": {}, + "vitest-integration": {}, "build-sandbox": { "options": { "dir": "solid-vite-default-js" } },scripts/check/check-package.ts (1)
1-31: Cwd normalization and scoped type-checking flow look solidDeriving
normalizedCwdfrom the optional--cwdflag (with sensible handling of absolute vs. repo-relative paths) and using it consistently fortsconfigPath,getTSFilesAndConfig, andgetTSDiagnosticsgives you a clear per-package check that works from the git root without breaking existing usage when no cwd is passed. The added"no type errors"branch also makes the script’s success path explicit. As per coding guidelines, just remember to runyarn prettier --write scripts/check/check-package.tsandyarn lint:js:cmd scripts/check/check-package.tsafter any further edits.code/core/src/telemetry/anonymous-id.ts (1)
50-52: Consider logging the error for debugging purposes.The error variable has been renamed from
_toe, but it's still unused in the catch block. Since this function is related to telemetry and project identification, consider logging the error (using the appropriate logger fromstorybook/internal/node-logger) to aid debugging when git command execution fails.Apply this diff to add error logging:
- } catch (e) { - // + } catch (e) { + // Silently fail if git command fails (e.g., not a git repository) + // Telemetry will proceed without project ID }As per coding guidelines, if you choose to log the error, use the logger from
storybook/internal/node-loggerinstead of console methods.scripts/sandbox/utils/getPort.ts (1)
5-6: Refactor index lookups for efficiency.The current implementation uses
Object.values().indexOf()to find the index of an item you already have a key for. This is inefficient as it creates an array of all values and then searches linearly.Apply this diff to improve the index lookup:
- const templateIndex = Object.values(allTemplates).indexOf(allTemplates[template.key]); - const taskIndex = Object.values(tasks).indexOf(tasks[template.selectedTask]); + const templateIndex = Object.keys(allTemplates).indexOf(template.key); + const taskIndex = Object.keys(tasks).indexOf(template.selectedTask);This directly searches for the key in the keys array, which is more straightforward and avoids the unnecessary value array creation.
.nx/workflows/distribution-config.yaml (2)
22-30: Clarify the commented-out e2e and test-runner targets.The commented-out assignment rules for
e2e-tests,e2e-tests-dev,test-runner, andtest-runner-devsuggest these features are planned but not yet enabled. Consider adding a TODO comment explaining why these are disabled and when they should be re-enabled, or remove them if they're not part of the immediate roadmap.Apply this diff to add clarifying context:
+ # TODO: Enable once e2e and test-runner infrastructure is ready # - targets: # - e2e-tests # - e2e-tests-dev
35-35: Add trailing newline to YAML file.The file is missing a trailing newline, which is a common convention for text files and can prevent issues with certain tools.
Ensure the file ends with a newline character.
scripts/get-template.ts (1)
304-306: Remove obvious comment.The comment "Convert to integer" is unnecessary as the unary
+operator's purpose is self-evident in this context.Apply this diff:
- // Convert to integer index: +CIRCLE_NODE_INDEX, total: +CIRCLE_NODE_TOTAL,.github/copilot-instructions.md (1)
150-156: Tighten duplicated compile steps in “For Code Changes”Steps 2 and 4 both compile packages, but use different working directories (
yarn task --task compilevscd code && yarn task --task compile). If only one of these is required with the new root layout, consider collapsing to a single, authoritative compile command to avoid confusion.code/sandbox/cra-default-ts/project.json (1)
4-26: Emptytagsmeans this sandbox is excluded from tag-based Nx CI runsThe
implicitDependenciesanddir: "cra-default-ts"wiring look consistent withtemplate: "cra/default-ts"andprepare-sandbox’stemplate.replace('/', '-')behavior.However,
tags: []means this project won’t be picked up by the newnx.ymlworkflow, which filters bytag:ci:normal,tag:ci:merged, etc. Please confirm that this sandbox is intentionally opt‑out from CI; otherwise, you likely want at least"ci:normal"here.code/sandbox/react-native-web-vite-rn-cli-ts/project.json (1)
1-28: Validate directory slug and CI tagging for new RN Web/Vite sandboxThe
name(react-native-web-vite/rn-cli-ts) anddir: "react-native-web-vite-rn-cli-ts"are consistent with thetemplate.replace('/', '-')convention used inscripts/prepare-sandbox.ts, so path resolution should work.As with
cra-default-ts,tags: []means this project won’t be included in tag‑filtered Nx runs (e.g.,-p="tag:ci:normal"innx.yml). If this sandbox should participate in standard CI, consider adding"ci:normal"(and optionally"ci:merged"/"ci:daily"); if it’s deliberately experimental/manual‑only, leaving it untagged is fine..nx/workflows/agents.yaml (1)
1-95: Align agent Node version and finalize Windows template before useThe agent templates look solid and the caching paths match the Yarn 4 monorepo layout. Two things to keep in mind:
- Node version parity: The Linux images are
ubuntu22.04-node20.19-v2, while the repo docs call out Node 22.16.0. If you rely on Node 22-specific behavior anywhere, consider switching to a Node 22 image for agents to avoid subtle discrepancies between local and remote runs.windows-jstemplate completeness:windows-jscurrently has noinit-steps. Until those are filled in (checkout, cache, Node install, Yarn install), it’s safer not to reference this template from any workflows, otherwise tasks will likely fail immediately.Everything else (caching keys, SANDBOX_ROOT, browser binaries cache) aligns well with the new CI model.
scripts/build-package.ts (1)
14-18: Simplify imports: consolidate path imports.Both
node:pathandpathare imported. Consider using onlynode:pathfor consistency.-import { join } from 'node:path'; +import { join, resolve } from 'node:path'; import { exec } from 'child_process'; import { program } from 'commander'; -import { resolve } from 'path';.circleci/src/jobs/chromatic-sandboxes.yml (1)
18-22: Aligncdusage with new path-returningget-sandbox-dirand optionally quote itUsing
cd $(yarn get-sandbox-dir --template $TEMPLATE)matches the new behavior ofget-sandbox-dirreturning a full directory path, so the functional change looks correct. To future‑proof against paths containing spaces, consider:- cd $(yarn get-sandbox-dir --template $TEMPLATE) && yarn + cd "$(yarn get-sandbox-dir --template "$TEMPLATE")" && yarnscripts/get-sandbox-dir.ts (1)
1-16: New SANDBOX_DIRECTORY join matches sandbox path semantics; tightentemplatehandlingUsing
join(SANDBOX_DIRECTORY, template.replace('/', '-'))is the right direction now that callers expect a full sandbox path rather than just a slug.However,
RunOptionsstill allowstemplateto be undefined whilerun()assumes it’s always present:type RunOptions = { template?: string }; // ... console.log(join(SANDBOX_DIRECTORY, template.replace('/', '-')));To avoid a
TypeErrorwhen--templateis omitted (e.g. from the CLI), consider:-type RunOptions = { - template?: string; -}; +type RunOptions = { + template: string; +}; async function run({ template }: RunOptions) { console.log(join(SANDBOX_DIRECTORY, template.replace('/', '-'))); }or defensively throwing a clear error if
templateis missing.code/project.json (1)
1-30: Nx targets look correct; consider whether globalbuilddependency is necessaryThe
lint,test, andkniptargets are wired cleanly throughnx:run-commandsand correctly scoped to{projectRoot}.Depending on
[{ "projects": ["*"], "target": "build" }]for all three can make these checks quite heavy, since every project’sbuildmust complete first. If that’s not strictly required, consider narrowingdependsOn(e.g., to dependencies ofcodeor dropping it forlint/knip) to keep CI times in check.scripts/task.ts (1)
441-442: Verify the necessity of optional chaining.The optional chaining suggests
tasksThatDepend.get(task)might returnundefined, but based on thegetTaskListfunction logic, every task added totasksThatDependshould have an array value (possibly empty, but never undefined).If
tasksThatDepend.get(task)is guaranteed to return an array, consider removing the optional chaining for clarity:- tasksThatDepend - .get(task) - ?.filter((t) => !t.service) - ?.forEach(setUnready); + tasksThatDepend + .get(task) + .filter((t) => !t.service) + .forEach(setUnready);.env (1)
1-3: Address dotenv linting issues.The static analysis tool flagged minor style issues: key ordering and missing trailing newline. While these don't affect functionality, fixing them improves consistency.
Apply this diff:
+IN_STORYBOOK_SANDBOX=true +SANDBOX_ROOT=../storybook-sandboxes STORYBOOK_DISABLE_TELEMETRY=true -IN_STORYBOOK_SANDBOX=true -SANDBOX_ROOT=../storybook-sandboxes +
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (1)
scripts/create-nx-sandbox-projects.ts (1)
76-100: Fix async forEach - promises won't be awaited.This issue was previously flagged. Using
forEachwith anasynccallback doesn't wait for promises to resolve, which means the script could exit before all files are written.Replace with a proper async iteration pattern:
-Object.entries(allTemplates).forEach(async ([key, value]) => { +for (const [key, value] of Object.entries(allTemplates)) { const p = key.replaceAll('/', '-'); const full = join(process.cwd(), '../code/sandbox', p, 'project.json'); - console.log(full); const framework = value.expected.framework; const project = framework.includes('@storybook/') ? framework.replace('@storybook/', '') : undefined; - console.log(project); - console.log(); + const tags = [ ...(normal.includes(key as any) && !value.inDevelopment ? ['ci:normal'] : []), ...(merged.includes(key as any) && !value.inDevelopment ? ['ci:merged'] : []), ...(daily.includes(key as any) && !value.inDevelopment ? ['ci:daily'] : []), ]; + ensureDirectoryExistence(full); - console.log(full); const data = await prettier.format(JSON.stringify(projectJson(key, project, tags, value)), { filepath: full, }); writeFileSync(full, data, { encoding: 'utf-8' }); -}); +}
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (47)
code/addons/a11y/project.json(1 hunks)code/addons/docs/project.json(1 hunks)code/addons/links/project.json(1 hunks)code/addons/onboarding/project.json(1 hunks)code/addons/pseudo-states/project.json(1 hunks)code/addons/themes/project.json(1 hunks)code/addons/vitest/project.json(1 hunks)code/builders/builder-vite/project.json(1 hunks)code/builders/builder-webpack5/project.json(1 hunks)code/core/project.json(1 hunks)code/frameworks/angular/project.json(1 hunks)code/frameworks/ember/project.json(1 hunks)code/frameworks/html-vite/project.json(1 hunks)code/frameworks/nextjs-vite/project.json(1 hunks)code/frameworks/nextjs/project.json(1 hunks)code/frameworks/preact-vite/project.json(1 hunks)code/frameworks/react-native-web-vite/project.json(1 hunks)code/frameworks/react-vite/project.json(1 hunks)code/frameworks/react-webpack5/project.json(1 hunks)code/frameworks/server-webpack5/project.json(1 hunks)code/frameworks/svelte-vite/project.json(1 hunks)code/frameworks/sveltekit/project.json(1 hunks)code/frameworks/vue3-vite/project.json(1 hunks)code/frameworks/web-components-vite/project.json(1 hunks)code/lib/cli-storybook/project.json(1 hunks)code/lib/codemod/project.json(1 hunks)code/lib/core-webpack/project.json(1 hunks)code/lib/create-storybook/project.json(1 hunks)code/lib/csf-plugin/project.json(1 hunks)code/lib/eslint-plugin/project.json(1 hunks)code/lib/react-dom-shim/project.json(1 hunks)code/presets/create-react-app/project.json(1 hunks)code/presets/react-webpack/project.json(1 hunks)code/presets/server-webpack/project.json(1 hunks)code/project.json(1 hunks)code/renderers/html/project.json(1 hunks)code/renderers/preact/project.json(1 hunks)code/renderers/react/project.json(1 hunks)code/renderers/server/project.json(1 hunks)code/renderers/svelte/project.json(1 hunks)code/renderers/vue3/project.json(1 hunks)code/renderers/web-components/project.json(1 hunks)nx.json(1 hunks)scripts/create-nx-sandbox-projects.ts(1 hunks)scripts/project.json(1 hunks)scripts/tasks/compile.ts(3 hunks)test-storybooks/yarn-pnp/project.json(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (19)
- code/frameworks/sveltekit/project.json
- code/project.json
- code/renderers/react/project.json
- code/renderers/web-components/project.json
- code/presets/server-webpack/project.json
- code/frameworks/react-native-web-vite/project.json
- scripts/project.json
- code/frameworks/vue3-vite/project.json
- code/addons/pseudo-states/project.json
- code/addons/a11y/project.json
- code/lib/create-storybook/project.json
- code/lib/codemod/project.json
- code/lib/core-webpack/project.json
- code/addons/links/project.json
- code/renderers/html/project.json
- code/addons/themes/project.json
- code/renderers/preact/project.json
- code/lib/csf-plugin/project.json
- code/frameworks/nextjs/project.json
🧰 Additional context used
📓 Path-based instructions (3)
**/*.{ts,tsx,js,jsx,json,html,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx,js,jsx,json,html,mjs}: Use ESLint and Prettier for code style enforcement
Run 'yarn prettier --write ' to format code after making changes
Run 'yarn lint:js:cmd ' to check for ESLint issues after making changes
Files:
code/builders/builder-webpack5/project.jsoncode/frameworks/preact-vite/project.jsoncode/addons/docs/project.jsoncode/frameworks/html-vite/project.jsonnx.jsoncode/frameworks/server-webpack5/project.jsoncode/addons/vitest/project.jsoncode/frameworks/ember/project.jsoncode/frameworks/angular/project.jsoncode/addons/onboarding/project.jsoncode/builders/builder-vite/project.jsoncode/frameworks/nextjs-vite/project.jsoncode/lib/eslint-plugin/project.jsontest-storybooks/yarn-pnp/project.jsonscripts/tasks/compile.tscode/frameworks/svelte-vite/project.jsoncode/frameworks/react-webpack5/project.jsoncode/renderers/vue3/project.jsoncode/renderers/server/project.jsoncode/lib/react-dom-shim/project.jsoncode/presets/create-react-app/project.jsoncode/frameworks/react-vite/project.jsoncode/lib/cli-storybook/project.jsoncode/frameworks/web-components-vite/project.jsoncode/presets/react-webpack/project.jsoncode/renderers/svelte/project.jsonscripts/create-nx-sandbox-projects.tscode/core/project.json
**/*.{ts,tsx,js,jsx,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use camelCase for variable and function names
Files:
scripts/tasks/compile.tsscripts/create-nx-sandbox-projects.ts
**/*.{ts,tsx}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx}: Enable TypeScript strict mode
Export functions from modules for testing purposes
Files:
scripts/tasks/compile.tsscripts/create-nx-sandbox-projects.ts
🧠 Learnings (13)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to .yarnrc.yml : Use Yarn 4.9.1 as the package manager
Learnt from: JReinhold
Repo: storybookjs/storybook PR: 32607
File: code/package.json:243-243
Timestamp: 2025-10-02T09:22:13.215Z
Learning: The Storybook repository uses Yarn v^4 (any 4.x version) as the package manager, configured via .yarnrc.yml and package.json packageManager field. Specific patch versions within v4 can be upgraded as needed.
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Compile packages with 'yarn task --task compile' after making code changes before testing
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
📚 Learning: 2025-11-24T17:49:31.838Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
Applied to files:
nx.jsoncode/addons/vitest/project.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Use ESLint and Prettier for code style enforcement
Applied to files:
code/lib/eslint-plugin/project.json
📚 Learning: 2025-10-02T09:22:13.215Z
Learnt from: JReinhold
Repo: storybookjs/storybook PR: 32607
File: code/package.json:243-243
Timestamp: 2025-10-02T09:22:13.215Z
Learning: The Storybook repository uses Yarn v^4 (any 4.x version) as the package manager, configured via .yarnrc.yml and package.json packageManager field. Specific patch versions within v4 can be upgraded as needed.
Applied to files:
test-storybooks/yarn-pnp/project.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.test.{ts,tsx} : Use 'yarn vitest run --coverage <test-file>' to run tests with coverage reports and aim for 75%+ coverage of statements/lines
Applied to files:
test-storybooks/yarn-pnp/project.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Compile packages with 'yarn task --task compile' after making code changes before testing
Applied to files:
test-storybooks/yarn-pnp/project.jsonscripts/tasks/compile.ts
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to .yarnrc.yml : Use Yarn 4.9.1 as the package manager
Applied to files:
test-storybooks/yarn-pnp/project.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Run 'yarn prettier --write <file>' to format code after making changes
Applied to files:
test-storybooks/yarn-pnp/project.jsonscripts/create-nx-sandbox-projects.ts
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Applied to files:
test-storybooks/yarn-pnp/project.jsonscripts/create-nx-sandbox-projects.ts
📚 Learning: 2025-11-24T17:49:31.838Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Tests are located in the `code/` directory with root directory for test execution at `./code/`
Applied to files:
scripts/tasks/compile.ts
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Applied to files:
code/lib/cli-storybook/project.jsonscripts/create-nx-sandbox-projects.ts
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Run 'yarn task --task check' to verify TypeScript types across all packages before submitting PRs
Applied to files:
code/renderers/svelte/project.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Run 'yarn lint:js:cmd <file>' to check for ESLint issues after making changes
Applied to files:
scripts/create-nx-sandbox-projects.ts
🧬 Code graph analysis (1)
scripts/create-nx-sandbox-projects.ts (2)
code/lib/cli-storybook/src/sandbox-templates.ts (5)
Template(30-106)allTemplates(988-992)normal(994-1013)merged(1015-1024)daily(1026-1049)scripts/utils/tools.ts (2)
process(115-115)prettier(122-122)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: normal
- GitHub Check: Core Unit Tests, windows-latest
- GitHub Check: nx
🔇 Additional comments (31)
nx.json (4)
56-62: Verify sandbox output path alignment with PR objectives.The PR objectives state sandboxes are being moved to
../storybook-sandboxes, but this target specifies output to{workspaceRoot}/sandbox/{options.dir}. Clarify whether:
- The output path should reference
../storybook-sandboxesinstead, or- The environment variable
SANDBOX_ROOTmentioned in PR objectives should control this path dynamically.
198-216: Align namedInputs paths with monorepo root refactoring.The PR objective is to refactor from
./coderoot to git root, butnamedInputsstill references{workspaceRoot}/code/tsconfig.json(line 200) and{workspaceRoot}/code/**/*(lines 213–214). Verify whether:
tsconfig.jsonshould now be at the repository root (Line 200), or- These paths should remain under
code/for the new structure.Additionally, confirm that the
sharedGlobalsinput (line 199) correctly captures all shared configuration files needed by targets at the new root level.
1-22: Configuration structure looks solid.The core Nx configuration (schema, cloud ID, parallelism, plugin settings, task runner options) is well-structured and appropriately configured for a TypeScript monorepo with inference plugins disabled and analytics disabled.
58-58: All sandbox-enabled projects consistently define{options.dir}. Verification confirms that all 47 projects incode/sandbox/*/project.jsondefine thesandboxtarget with the requiredoptions.dirconfiguration. No inconsistencies found.code/addons/onboarding/project.json (3)
8-8: Tags configuration looks good.The
"ci:normal"tag aligns with the PR's objective to centralize CI behavior and organize targets by CI tier.
1-9: Based on my verification, I found that the original review comment is based on an incorrect premise. The filecode/addons/onboarding/project.jsonis new (created in this PR), not modified. The$schemaproperty was never present in this file to begin with, so no removal occurred.However, the underlying verification points are still valid and have been confirmed:
Centralized schema validation exists: The root
nx.jsonfile contains"$schema": "./node_modules/nx/schemas/workspace-schema.json", providing workspace-level schema configuration.IDE tooling doesn't depend on per-file schema: VSCode settings are not configured to validate individual project.json files using per-file schema declarations. Validation is handled centrally or through other mechanisms (ESLint for JSON linting).
No tooling depends on per-file schema: The repository's ESLint configuration validates JSON files separately from schema validation. NX CLI validates project.json files using its own internal schema logic, independent of per-file
$schemareferences.File is newly created, not modified. No schema was removed.
Likely an incorrect or invalid review comment.
5-6: Perfect! I now have all the information needed. The verification shows that the empty target definitions are intentional and correct. Let me provide the final rewritten review comment:
No issues found with the empty target definitions.
The empty
compileandchecktargets in thisproject.jsonare correct. This follows the NxtargetDefaultspattern where individual project configurations declare targets as empty objects, with the actual implementation provided centrally innx.json. Thescripts/tasks/compile.tsandscripts/tasks/check.tsfiles invoke these viayarn nx run-many -t compileandyarn nx run-many -t check, which properly resolve the target definitions fromnx.json'stargetDefaultssection. This pattern is consistently used across all addon project files (a11y,docs,links,pseudo-states,themes,vitest) and is the intended monorepo architecture.code/lib/cli-storybook/project.json (1)
4-8: Verify centralized target configuration for compile and check targets.The compile and check targets are empty objects, suggesting they're implemented via inheritance or a centralized Nx config. Ensure that the root-level Nx configuration (or relevant parent project configs) properly defines the executors and options for these targets so the build pipeline functions as intended.
code/renderers/server/project.json (1)
4-8: Verify centralized target configuration for compile and check targets.The compile and check targets are empty objects, suggesting they're implemented via inheritance or a centralized Nx config. Ensure that the root-level Nx configuration (or relevant parent project configs) properly defines the executors and options for these targets so the build pipeline functions as intended.
code/frameworks/web-components-vite/project.json (1)
4-8: Verify centralized target configuration for compile and check targets.The compile and check targets are empty objects, suggesting they're implemented via inheritance or a centralized Nx config. Ensure that the root-level Nx configuration (or relevant parent project configs) properly defines the executors and options for these targets so the build pipeline functions as intended.
code/addons/docs/project.json (1)
4-8: Verify centralized target configuration for compile and check targets.The compile and check targets are empty objects, suggesting they're implemented via inheritance or a centralized Nx config. Ensure that the root-level Nx configuration (or relevant parent project configs) properly defines the executors and options for these targets so the build pipeline functions as intended.
code/addons/vitest/project.json (1)
4-8: Verify centralized target configuration for compile and check targets.The compile and check targets are empty objects, suggesting they're implemented via inheritance or a centralized Nx config. Ensure that the root-level Nx configuration (or relevant parent project configs) properly defines the executors and options for these targets so the build pipeline functions as intended.
test-storybooks/yarn-pnp/project.json (3)
6-6: Verify the broad dependency scope on all projects' compile targets.The test target depends on compiling all projects (
["*"]). In a large monorepo, this could be expensive and may be unnecessary for a yarn-pnp smoke test. Verify whether this dependency scope is intentional, or if it should be narrowed to only the required projects for this particular test storybook.
10-10: Verify the--no-immutableflag is appropriate for CI context.The command uses
yarn --no-immutable, which allows installing new or different dependency versions. For a CI smoke test, confirm this is the intended behavior. If reproducibility and consistency are required, consider using--immutableinstead to enforce exact version matching.
1-18: New test target structure looks good overall.The test target is well-structured, uses nx:run-commands appropriately, and includes proper working directory scoping. The configuration placeholder for production and the ci:normal tag align with the broader refactor pattern.
code/frameworks/ember/project.json (1)
4-8: Verify centralized target configuration for compile and check targets.The compile and check targets are empty objects, suggesting they're implemented via inheritance or a centralized Nx config. Ensure that the root-level Nx configuration (or relevant parent project configs) properly defines the executors and options for these targets so the build pipeline functions as intended.
code/frameworks/preact-vite/project.json (1)
4-8: Verify centralized target configuration for compile and check targets.The compile and check targets are empty objects, suggesting they're implemented via inheritance or a centralized Nx config. Ensure that the root-level Nx configuration (or relevant parent project configs) properly defines the executors and options for these targets so the build pipeline functions as intended.
code/frameworks/svelte-vite/project.json (1)
5-8: LGTM: Standardized Nx target structure.The migration from a single
buildtarget to separatecompileandchecktargets, along with the CI classification tag, aligns with the repository-wide standardization pattern.code/lib/eslint-plugin/project.json (1)
5-8: LGTM: Consistent target migration.The standardization to
compile/checktargets with CI tagging is consistent across the repository.scripts/create-nx-sandbox-projects.ts (2)
95-97: Good addition: Prettier formatting for generated files.Using Prettier to format the generated JSON files ensures consistency and readability.
35-72: Based on my verification, I've discovered a critical issue. Let me generate the final rewritten review comment:Generated project.json files are out of sync with the current code — naming mismatch detected.
The conditional target generation logic itself is correct:
typeCheck === trueproperly includes the'check-sandbox'target (verified withreact-vite-default-ts)skipTasksarray filtering works as expected across all template variantsinDevelopmentflag is intentionally not used in target generation (only for CI tags)However, the generated
project.jsonfiles incode/sandbox/contain mismatched target names:
- Code defines:
'prepare-sandbox'and'prepare-build-sandbox'- Generated files contain:
'prepared-sandbox'and'prepared-build-sandbox'The generated project.json files must be regenerated to match the current code by running
scripts/create-nx-sandbox-projects.ts.⛔ Skipped due to learnings
Learnt from: CR Repo: storybookjs/storybook PR: 0 File: .github/copilot-instructions.md:0-0 Timestamp: 2025-11-24T17:49:47.659Z Learning: Run 'yarn task --task check' to verify TypeScript types across all packages before submitting PRsLearnt from: CR Repo: storybookjs/storybook PR: 0 File: .github/copilot-instructions.md:0-0 Timestamp: 2025-11-24T17:49:47.659Z Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinationscode/frameworks/nextjs-vite/project.json (1)
5-8: LGTM: Consistent standardization.The target migration follows the established pattern across the repository.
scripts/tasks/compile.ts (3)
15-16: LGTM: Updated commands align with new target structure.The change from
buildtocompiletarget and fromnpxtoyarnis consistent with the repository-wide refactoring.
45-45: LGTM: Working directory updated for new monorepo root.Changing the working directory from
codeDirtoROOT_DIRECTORYcorrectly reflects the monorepo root migration from./codeto the git root.
5-5: ROOT_DIRECTORY correctly points to the git root.The constant is properly defined in
scripts/utils/constants.tsat line 8 asjoin(__dirname, '..', '..'), which resolves fromscripts/utils/to the repository root. It serves as the base for all other directory constants (CODE_DIRECTORY,SNIPPETS_DIRECTORY,PACKS_DIRECTORY, etc.), confirming it supports the monorepo root refactoring as intended.code/core/project.json (1)
5-8: LGTM: Core package aligned with repository standard.The standardized target structure is now consistently applied across the repository, including the core package.
code/renderers/vue3/project.json (1)
5-18: LGTM: Well-structured check target with proper dependencies.The
checktarget correctly depends on thecompiletarget and uses the standard Vue type-checking command. Verification confirmsvue-tscis available in the project dependencies (code/renderers/vue3/package.json, line 59).code/renderers/svelte/project.json (1)
5-18: LGTM: Proper check target implementation with verified dependencies.The
checktarget is correctly configured with the standard Svelte type-checking command and appropriate dependency ordering. Verification confirmssvelte-checkis available in the project as a devDependency (version ^4.3.2).code/builders/builder-vite/project.json (2)
8-8: Verify CI system recognizes theci:normaltag.The
tagsfield with["ci:normal"]appears to be a new CI classification strategy. Ensure that:
- CI/CD pipeline configuration (GitHub Actions, etc.) recognizes and uses these tags appropriately
- This aligns with NX agent configuration mentioned in the PR objectives
- Tag values are documented or consistent across all projects being refactored
5-7: No action needed—empty targets with inherited executors fromtargetDefaultsis the correct NX pattern and is consistently applied across all packages in the workspace.The
compileandchecktargets incode/builders/builder-vite/project.jsonare correctly configured as empty objects. They inherit their executor definitions, commands, caching rules, and other configuration from the rootnx.jsontargetDefaults, which define:
compile: runsyarn exec jiti ./scripts/build/build-package.ts --cwd {projectRoot}check: runsyarn exec jiti ./scripts/check/check-package.ts --cwd {projectRoot}This pattern is consistent across the entire workspace (verified in
code/addons/a11y,code/addons/docs,code/addons/links, etc.). NX can discover and execute these targets vianx run builder-vite:compileandnx run builder-vite:check, and theyarn taskcommand inpackage.json(which delegates toscripts/task.ts) works correctly with this setup.code/frameworks/html-vite/project.json (1)
5-8: Identical pattern and same verification requirements as builder-vite.This file has the same structure and changes as
code/builders/builder-vite/project.json. The verification concerns about empty target configuration and CI tag recognition apply here as well. Given this pattern is applied across multiple projects, consider:
- Creating a base/shared
project.jsonconfiguration if not already done- Documenting the executor definitions that are referenced by these empty targets
- Adding a validation check to the CI pipeline to ensure all projects have complete executor definitions
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (2)
code/sandbox/qwik-vite-default-ts/project.json (1)
4-11: Add "qwik-vite" to implicitDependencies array.This is still missing from the previous review. All similar sandbox configurations include their corresponding framework in
implicitDependencies:html-vite,react-vite,svelte-vite, andsveltekitall follow this pattern. Add"qwik-vite"as the final entry to maintain consistency and ensure Nx correctly tracks dependencies.Apply this diff:
"implicitDependencies": [ "core", "cli", "addon-a11y", "addon-docs", "addon-vitest", - "addon-onboarding" + "addon-onboarding", + "qwik-vite" ],code/sandbox/solid-vite-default-js/project.json (1)
13-22: Missingvitest-integrationtarget breaks consistency with TS variant and other JS frameworks.Line 9 specifies
addon-vitestin implicitDependencies, yet thevitest-integrationtarget is absent. This target exists in the TS counterpart (line 16) and in other JS variants (vue3-vite-default-js, react-vite-default-js), making this a breaking inconsistency.Add
"vitest-integration": {}between thedevandbuild-sandboxtargets:"dev": {}, + "vitest-integration": {}, "build-sandbox": { "options": { "dir": "solid-vite-default-js" } },
🧹 Nitpick comments (3)
code/sandbox/angular-cli-prerelease/project.json (1)
4-12: Verify that all implicit dependencies are actually used by this sandbox.The
implicitDependencieslist has been significantly expanded to includecli,addon-a11y,addon-docs, andaddon-vitest. Ensure these packages are actually imported or required by the angular-cli-prerelease sandbox; unused dependencies should be removed to keep the dependency graph accurate.scripts/create-nx-sandbox-projects.ts (2)
76-80: Async handling fixed, butprocess.cwd()-based path is brittleSwitching to
await Promise.all(Object.entries(allTemplates).map(async ...))correctly ensures all async work completes before the script exits and addresses the previousforEach(async ...)issue. 👍However, building
fullas:const full = join(process.cwd(), '../code/sandbox', p, 'project.json');means the output location depends on where the script is invoked. With the monorepo root now at
., running this from the repo root will resolve to a sibling../code/sandbox/..., whereas running it from./scriptsresolves to./code/sandbox/.... That can easily drift if someone calls the script differently or via CI.I’d recommend deriving the path relative to the repo (or script) root instead of
process.cwd()to make it location-independent, or at least double-check that all callers intentionally run it from a directory where this resolves to the correct sandbox folder.Please verify how this script is invoked in package.json / CI and confirm that
../code/sandboxresolves to the intended directory in all cases, given the monorepo root refactor.
81-87: Consider trimming or structuring debug loggingThere are multiple
console.logcalls (fulllogged twice,project, and an emptyconsole.log()), which look primarily diagnostic.If this script is going to be run frequently in CI or by contributors, consider:
- consolidating into a single, structured log per project, or
- removing the purely debug/duplicate logs (including the empty
console.log()).This will keep output noise down while still being debuggable when needed.
Also applies to: 94-94
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (51)
.github/copilot-instructions.md(5 hunks)code/frameworks/nextjs/src/next-image-loader-stub.ts(1 hunks)code/renderers/web-components/src/docs/web-components-properties.test.ts(1 hunks)code/sandbox/angular-cli-default-ts/project.json(1 hunks)code/sandbox/angular-cli-prerelease/project.json(1 hunks)code/sandbox/bench-react-vite-default-ts-nodocs/project.json(1 hunks)code/sandbox/bench-react-vite-default-ts-test-build/project.json(1 hunks)code/sandbox/bench-react-vite-default-ts/project.json(1 hunks)code/sandbox/bench-react-webpack-18-ts-test-build/project.json(1 hunks)code/sandbox/bench-react-webpack-18-ts/project.json(1 hunks)code/sandbox/cra-default-js/project.json(1 hunks)code/sandbox/cra-default-ts/project.json(1 hunks)code/sandbox/ember-3-js/project.json(1 hunks)code/sandbox/ember-default-js/project.json(1 hunks)code/sandbox/html-rsbuild-default-ts/project.json(1 hunks)code/sandbox/html-vite-default-js/project.json(1 hunks)code/sandbox/html-vite-default-ts/project.json(1 hunks)code/sandbox/internal-react16-webpack/project.json(1 hunks)code/sandbox/internal-react18-webpack-babel/project.json(1 hunks)code/sandbox/internal-server-webpack5/project.json(1 hunks)code/sandbox/lit-rsbuild-default-ts/project.json(1 hunks)code/sandbox/lit-vite-default-js/project.json(1 hunks)code/sandbox/lit-vite-default-ts/project.json(1 hunks)code/sandbox/nextjs-14-ts/project.json(1 hunks)code/sandbox/nextjs-15-ts/project.json(1 hunks)code/sandbox/nextjs-default-ts/project.json(1 hunks)code/sandbox/nextjs-prerelease/project.json(1 hunks)code/sandbox/nextjs-vite-14-ts/project.json(1 hunks)code/sandbox/nextjs-vite-15-ts/project.json(1 hunks)code/sandbox/nextjs-vite-default-ts/project.json(1 hunks)code/sandbox/preact-vite-default-js/project.json(1 hunks)code/sandbox/preact-vite-default-ts/project.json(1 hunks)code/sandbox/qwik-vite-default-ts/project.json(1 hunks)code/sandbox/react-native-web-vite-expo-ts/project.json(1 hunks)code/sandbox/react-native-web-vite-rn-cli-ts/project.json(1 hunks)code/sandbox/react-rsbuild-default-ts/project.json(1 hunks)code/sandbox/react-vite-default-js/project.json(1 hunks)code/sandbox/react-vite-default-ts/project.json(1 hunks)code/sandbox/react-vite-prerelease-ts/project.json(1 hunks)code/sandbox/react-webpack-17-ts/project.json(1 hunks)code/sandbox/react-webpack-18-ts/project.json(1 hunks)code/sandbox/react-webpack-prerelease-ts/project.json(1 hunks)code/sandbox/solid-vite-default-js/project.json(1 hunks)code/sandbox/solid-vite-default-ts/project.json(1 hunks)code/sandbox/svelte-kit-skeleton-ts/project.json(1 hunks)code/sandbox/svelte-vite-default-js/project.json(1 hunks)code/sandbox/svelte-vite-default-ts/project.json(1 hunks)code/sandbox/vue3-rsbuild-default-ts/project.json(1 hunks)code/sandbox/vue3-vite-default-js/project.json(1 hunks)code/sandbox/vue3-vite-default-ts/project.json(1 hunks)scripts/create-nx-sandbox-projects.ts(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- code/sandbox/nextjs-15-ts/project.json
🚧 Files skipped from review as they are similar to previous changes (19)
- .github/copilot-instructions.md
- code/sandbox/nextjs-14-ts/project.json
- code/frameworks/nextjs/src/next-image-loader-stub.ts
- code/sandbox/nextjs-vite-15-ts/project.json
- code/renderers/web-components/src/docs/web-components-properties.test.ts
- code/sandbox/html-vite-default-js/project.json
- code/sandbox/react-native-web-vite-expo-ts/project.json
- code/sandbox/react-rsbuild-default-ts/project.json
- code/sandbox/bench-react-vite-default-ts/project.json
- code/sandbox/html-rsbuild-default-ts/project.json
- code/sandbox/bench-react-vite-default-ts-test-build/project.json
- code/sandbox/lit-rsbuild-default-ts/project.json
- code/sandbox/nextjs-vite-default-ts/project.json
- code/sandbox/internal-react18-webpack-babel/project.json
- code/sandbox/cra-default-js/project.json
- code/sandbox/vue3-rsbuild-default-ts/project.json
- code/sandbox/react-vite-prerelease-ts/project.json
- code/sandbox/react-webpack-18-ts/project.json
- code/sandbox/nextjs-prerelease/project.json
🧰 Additional context used
📓 Path-based instructions (3)
**/*.{ts,tsx,js,jsx,json,html,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx,js,jsx,json,html,mjs}: Use ESLint and Prettier for code style enforcement
Run 'yarn prettier --write ' to format code after making changes
Run 'yarn lint:js:cmd ' to check for ESLint issues after making changes
Files:
code/sandbox/nextjs-default-ts/project.jsoncode/sandbox/svelte-vite-default-ts/project.jsoncode/sandbox/internal-react16-webpack/project.jsoncode/sandbox/angular-cli-default-ts/project.jsoncode/sandbox/cra-default-ts/project.jsoncode/sandbox/react-webpack-17-ts/project.jsoncode/sandbox/html-vite-default-ts/project.jsoncode/sandbox/react-native-web-vite-rn-cli-ts/project.jsoncode/sandbox/nextjs-vite-14-ts/project.jsoncode/sandbox/react-vite-default-ts/project.jsonscripts/create-nx-sandbox-projects.tscode/sandbox/svelte-kit-skeleton-ts/project.jsoncode/sandbox/ember-default-js/project.jsoncode/sandbox/ember-3-js/project.jsoncode/sandbox/preact-vite-default-ts/project.jsoncode/sandbox/react-vite-default-js/project.jsoncode/sandbox/react-webpack-prerelease-ts/project.jsoncode/sandbox/bench-react-webpack-18-ts/project.jsoncode/sandbox/vue3-vite-default-ts/project.jsoncode/sandbox/preact-vite-default-js/project.jsoncode/sandbox/qwik-vite-default-ts/project.jsoncode/sandbox/internal-server-webpack5/project.jsoncode/sandbox/bench-react-vite-default-ts-nodocs/project.jsoncode/sandbox/lit-vite-default-js/project.jsoncode/sandbox/solid-vite-default-js/project.jsoncode/sandbox/angular-cli-prerelease/project.jsoncode/sandbox/bench-react-webpack-18-ts-test-build/project.jsoncode/sandbox/svelte-vite-default-js/project.jsoncode/sandbox/solid-vite-default-ts/project.jsoncode/sandbox/lit-vite-default-ts/project.jsoncode/sandbox/vue3-vite-default-js/project.json
**/*.{ts,tsx,js,jsx,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use camelCase for variable and function names
Files:
scripts/create-nx-sandbox-projects.ts
**/*.{ts,tsx}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx}: Enable TypeScript strict mode
Export functions from modules for testing purposes
Files:
scripts/create-nx-sandbox-projects.ts
🧠 Learnings (14)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Learnt from: JReinhold
Repo: storybookjs/storybook PR: 32607
File: code/package.json:243-243
Timestamp: 2025-10-02T09:22:13.215Z
Learning: The Storybook repository uses Yarn v^4 (any 4.x version) as the package manager, configured via .yarnrc.yml and package.json packageManager field. Specific patch versions within v4 can be upgraded as needed.
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to .yarnrc.yml : Use Yarn 4.9.1 as the package manager
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
📚 Learning: 2025-11-24T17:49:31.838Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
Applied to files:
code/sandbox/svelte-vite-default-ts/project.jsoncode/sandbox/html-vite-default-ts/project.jsoncode/sandbox/react-native-web-vite-rn-cli-ts/project.jsoncode/sandbox/nextjs-vite-14-ts/project.jsoncode/sandbox/react-vite-default-ts/project.jsoncode/sandbox/preact-vite-default-ts/project.jsoncode/sandbox/vue3-vite-default-ts/project.jsoncode/sandbox/qwik-vite-default-ts/project.jsoncode/sandbox/solid-vite-default-ts/project.jsoncode/sandbox/lit-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid mocking only a subset of required dependencies in Vitest tests
Applied to files:
code/sandbox/svelte-vite-default-ts/project.jsoncode/sandbox/html-vite-default-ts/project.jsoncode/sandbox/react-native-web-vite-rn-cli-ts/project.jsoncode/sandbox/nextjs-vite-14-ts/project.jsoncode/sandbox/react-vite-default-ts/project.jsoncode/sandbox/preact-vite-default-ts/project.jsoncode/sandbox/react-vite-default-js/project.jsoncode/sandbox/vue3-vite-default-ts/project.jsoncode/sandbox/preact-vite-default-js/project.jsoncode/sandbox/bench-react-vite-default-ts-nodocs/project.jsoncode/sandbox/solid-vite-default-ts/project.jsoncode/sandbox/lit-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Run 'yarn prettier --write <file>' to format code after making changes
Applied to files:
code/sandbox/nextjs-vite-14-ts/project.jsonscripts/create-nx-sandbox-projects.ts
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Applied to files:
scripts/create-nx-sandbox-projects.tscode/sandbox/bench-react-webpack-18-ts/project.jsoncode/sandbox/bench-react-webpack-18-ts-test-build/project.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Run 'yarn lint:js:cmd <file>' to check for ESLint issues after making changes
Applied to files:
scripts/create-nx-sandbox-projects.ts
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Applied to files:
scripts/create-nx-sandbox-projects.ts
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid inline mock implementations within test cases in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Keep mock implementations simple and focused in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid mocking without the `spy: true` option in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid mock implementations outside of `beforeEach` blocks in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid direct function mocking without `vi.mocked()` in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Document complex mock behaviors in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Mock all required properties and methods that the test subject uses in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
🧬 Code graph analysis (1)
scripts/create-nx-sandbox-projects.ts (2)
code/lib/cli-storybook/src/sandbox-templates.ts (5)
Template(30-106)allTemplates(988-992)normal(994-1013)merged(1015-1024)daily(1026-1049)scripts/utils/tools.ts (2)
process(115-115)prettier(122-122)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: normal
- GitHub Check: nx
- GitHub Check: Core Unit Tests, windows-latest
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
nx.json (1)
56-96: Based on my verification, I have gathered sufficient information to rewrite the review comment. The analysis confirms the core concerns while revealing some additional nuance.
Clarify the sandbox target dependency optimization opportunity.
The
prepare-sandboxandprepare-build-sandboxtargets have redundant explicit dependencies onscripts/run-registry. Theprepare-sandboxtarget already depends onsandbox, which depends onrun-registry; similarly,prepare-build-sandboxdepends onbuild, which chains back torun-registry. Sincerun-registryis defined as continuous, these direct re-declarations are unnecessary and can be removed, allowing Nx to resolve the dependency once via the chain.Consider removing the explicit
{ "projects": ["scripts"], "target": "run-registry" }fromprepare-sandboxandprepare-build-sandboxto simplify the dependency graph.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
nx.json(1 hunks)scripts/prepare-sandbox.ts(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- scripts/prepare-sandbox.ts
🧰 Additional context used
📓 Path-based instructions (1)
**/*.{ts,tsx,js,jsx,json,html,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx,js,jsx,json,html,mjs}: Use ESLint and Prettier for code style enforcement
Run 'yarn prettier --write ' to format code after making changes
Run 'yarn lint:js:cmd ' to check for ESLint issues after making changes
Files:
nx.json
🧠 Learnings (2)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
📚 Learning: 2025-11-24T17:49:31.838Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
Applied to files:
nx.json
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: normal
- GitHub Check: nx
- GitHub Check: Core Unit Tests, windows-latest
🔇 Additional comments (2)
nx.json (2)
30-33: I'll help you verify the path reference in nx.json. Let me start by examining the directory structure and the nx.json file itself.
<function_calls>
#!/bin/bashVerify the code directory structure exists
ls -la code/bench/ 2>/dev/null || echo "code/bench/ not found"
ls -la code/ 2>/dev/null | head -20
</function_calls>Now let me examine the nx.json file to understand the full context:
<function_calls>
#!/bin/bashRead the nx.json file around lines 30-33 with context
wc -l nx.json
cat -n nx.json | sed -n '25,40p'
</function_calls>Let me also search for other references to this path in the codebase:
<function_calls>
#!/bin/bashSearch for references to bench/esbuild-metafiles
rg "bench/esbuild-metafiles" -n -C 3
Also search for compile target configuration
rg "compile" nx.json -n -C 5
</function_calls>
198-216: Yes, please provide the exact file paths for all tsconfig.json files that exist on the default branch of storybookjs/storybook.
kasperpeulen
force-pushed
the
kasper/yarn-monorepo-refactor
branch
from
93f088f to
24aa914
Compare
|
Please update and lookup our contribution guidelines/markdown files and watch out whether they need any update |
There was a problem hiding this comment.
Actionable comments posted: 4
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
scripts/release/version.ts (1)
80-86: Guard againstschema.releaseTypebeing undefined insuperRefine
schema.releaseTypeis optional, butschema.releaseType.startsWith('pre')is called unconditionally whenpreIdis present, which will throw a runtime error if a user passes--pre-idwithout--release-typeinstead of producing aZodError.A small defensive tweak keeps behavior while avoiding this edge-case crash:
- if (schema.preId && !schema.releaseType.startsWith('pre')) { + const isPreReleaseType = + typeof schema.releaseType === 'string' && schema.releaseType.startsWith('pre'); + if (schema.preId && !isPreReleaseType) { ctx.addIssue({ code: z.ZodIssueCode.custom, message: 'Using prerelease identifier requires one of release types: premajor, preminor, prepatch, prerelease', }); }
♻️ Duplicate comments (5)
scripts/build/utils/generate-types.ts (2)
21-25: Update concurrency comment to match the actuallimit(10)behaviorThe comment still says “we limit the number of concurrent processes to 3” and “3 seems to be the sweet spot”, but the code uses
limit(10). This can easily confuse anyone tuning build performance.Consider updating the comment to reflect the current concurrency:
- // we limit the number of concurrent processes to 3, because we don't want to overload the host machine - // by trial and error, 3 seems to be the sweet spot between perf and consistency + // we limit the number of concurrent processes to 10, because we don't want to overload the host machine + // by trial and error, 10 seems to be the sweet spot between perf and consistencyAfter editing, please run the usual formatting and lint commands for this file as per the repo guidelines.
64-70: Confirm 20‑minute timeout and consider extracting a named constant
1200000here is 20 minutes, which is a long time to leave a potentially hungdts-processrunning. If that’s intentional to accommodate slow builds, fine—but it’s worth double‑checking, and the magic number would be clearer as a named constant:-const DIR_CODE = join(import.meta.dirname, '..', '..', '..', 'code'); +const DIR_CODE = join(import.meta.dirname, '..', '..', '..', 'code'); +const DTS_TIMEOUT_MS = 20 * 60 * 1000; @@ - timer = setTimeout(() => { + timer = setTimeout(() => { @@ - dtsProcess.kill(408); // timed out - resolve(void 0); - }, 1200000); + dtsProcess.kill(408); // timed out + resolve(void 0); + }, DTS_TIMEOUT_MS);Please confirm that 20 minutes is the intended timeout; if not, this is a good place to adjust it now.
nx.json (2)
57-64: Fix{options.dir}placeholder in sandbox target outputs—unresolved from prior review.Line 62 uses the undefined
{options.dir}placeholder. This was flagged in prior reviews but remains unresolved. Nx only recognizes{projectRoot}and{workspaceRoot}. The command correctly uses{projectName}(line 59), so outputs should too.- "outputs": ["{workspaceRoot}/sandbox/{options.dir}"], + "outputs": ["{workspaceRoot}/sandbox/{projectName}"],
78-85: Fix{options.dir}placeholder in build target outputs—unresolved from prior review.Line 83 repeats the same invalid placeholder. Replace with
{projectName}to align with the command pattern and restore proper Nx caching.- "outputs": ["{workspaceRoot}/sandbox/{options.dir}/storybook-static"], + "outputs": ["{workspaceRoot}/sandbox/{projectName}/storybook-static"],scripts/prepare-sandbox.ts (1)
34-38: Still missing cache existence guard and backgroundexecerror handlingTwo minor robustness issues remain (both already noted in a prior review):
Cache directory assumptions (Lines 34–38)
IfcacheDirdoesn’t exist (e.g. first run for a template),cp(cacheDir, sandboxDir, …)will reject. A clearer behavior would be to either (a) warn and skip the copy, or (b) fail with an explicit message about how to create/populate the cache.Fire‑and‑forget
execcan reject unnoticed (Lines 51–57)
exec('yarn local-registry --open', …)is called and discarded. If that Promise rejects, the error won’t be observed, even though you later rely onwaitOnto confirm readiness. Wrapping the call in.catch(() => {})preserves the existing behavior while avoiding stray unhandled rejections and noisy logs.A possible adjustment:
- if (sandboxDir !== cacheDir) { - console.log(`🧹 copying cached ${cacheDir} to ${sandboxDir}`); - await rm(sandboxDir, { recursive: true, force: true }); - await cp(cacheDir, sandboxDir, { recursive: true, force: true }); - } + if (sandboxDir !== cacheDir) { + if (!existsSync(cacheDir)) { + console.warn(`⚠️ cache directory ${cacheDir} does not exist; skipping cache copy`); + } else { + console.log(`🧹 copying cached ${cacheDir} to ${sandboxDir}`); + await rm(sandboxDir, { recursive: true, force: true }); + await cp(cacheDir, sandboxDir, { recursive: true, force: true }); + } + } @@ - void exec('yarn local-registry --open', { cwd: CODE_DIRECTORY }); + void exec('yarn local-registry --open', { cwd: CODE_DIRECTORY }).catch(() => { + // best-effort; readiness is validated by waitOn below + });Also applies to: 51-57
🧹 Nitpick comments (8)
scripts/create-nx-sandbox-projects.ts (3)
86-99: Remove redundant logging.Lines 86 and 99 both log the same
fullpath. Remove the duplicate to reduce noise.Apply this diff:
- console.log(full); const framework = value.expected.framework; const project = framework.includes('@storybook/') ? framework.replace('@storybook/', '') : undefined; console.log(project); console.log(); const tags = [ ...(normal.includes(key as any) && !value.inDevelopment ? ['ci:normal'] : []), ...(merged.includes(key as any) && !value.inDevelopment ? ['ci:merged'] : []), ...(daily.includes(key as any) && !value.inDevelopment ? ['ci:daily'] : []), ]; ensureDirectoryExistence(full); console.log(full);
101-105: Consider adding error handling for formatting and file operations.If
prettier.formatfails (e.g., due to invalid JSON structure) orwriteFileSyncencounters a disk error, the script will crash without a clear indication of which template failed. Consider wrapping these operations in a try-catch block to provide better error messages.Apply this diff to add error handling:
+ try { const data = await prettier.format(JSON.stringify(projectJson(key, project, tags, value)), { filepath: full, }); writeFileSync(full, data, { encoding: 'utf-8' }); + } catch (error) { + console.error(`Failed to generate project.json for ${key}:`, error); + throw error; + }
109-116: Consider using Node.js built-in recursive directory creation.The
ensureDirectoryExistencefunction manually implements recursive directory creation. Node.jsmkdirSynchas a built-inrecursive: trueoption that achieves the same result more simply.Apply this diff to simplify:
-function ensureDirectoryExistence(filePath: string): void { - const dir = dirname(filePath); - if (existsSync(dir)) { - return; - } - ensureDirectoryExistence(dir); - mkdirSync(dir); -} +function ensureDirectoryExistence(filePath: string): void { + const dir = dirname(filePath); + mkdirSync(dir, { recursive: true }); +}.env (1)
1-3: Reorder environment variables alphabetically and add trailing newline.The
dotenv-lintertool flags the file for unordered keys and a missing blank line at EOF. While not functionally critical, these are standard conventions for configuration files.Apply this diff to address the linting warnings:
-STORYBOOK_DISABLE_TELEMETRY=true -IN_STORYBOOK_SANDBOX=true -SANDBOX_ROOT=../storybook-sandboxes +IN_STORYBOOK_SANDBOX=true +SANDBOX_ROOT=../storybook-sandboxes +STORYBOOK_DISABLE_TELEMETRY=truecode/sandbox/cra-default-js/project.json (1)
4-25: Confirm that dropping allci:*tags for this sandbox is intentionalThe new targets and expanded
implicitDependencieslook consistent with the other sandboxes, buttagsis now an empty array. Given the Nx workflow filters projects viaci:*tags, this meanscra/default-jswon’t run in the new Nx CI job at all and will only be exercised by the existing non‑Nx pipelines (if any).If that’s deliberate (e.g., to keep this sandbox out of the experimental Nx job), all good; otherwise, consider restoring an appropriate
ci:*tag set (for example,["ci:daily"]as before).scripts/run-registry.ts (1)
46-62: Consider extracting the duplicated port-clearing logic.The port-clearing blocks for 6001 and 6002 are nearly identical. A helper function would reduce duplication and make future maintenance easier.
+const clearPort = async (port: number) => { + if (await isPortUsed(port)) { + await killProcessOnPort(port); + let attempts = 0; + while ((await isPortUsed(port)) && attempts < 10) { + await sleep(1000); + attempts++; + } + } +}; + const startVerdaccio = async () => { - // Kill Verdaccio related processes if they are already running - if (await isPortUsed(6001)) { - await killProcessOnPort(6001); - - let attempts = 0; - while ((await isPortUsed(6001)) && attempts < 10) { - await sleep(1000); - attempts++; - } - } - if (await isPortUsed(6002)) { - await killProcessOnPort(6002); - let attempts = 0; - while ((await isPortUsed(6002)) && attempts < 10) { - await sleep(1000); - attempts++; - } - } + // Kill Verdaccio related processes if they are already running + await clearPort(6001); + await clearPort(6002);package.json (1)
3-17: Avoid hard‑coding../storybook-sandboxesin ecosystem‑CI scriptsThe new
svelte-ecosystem-ci:before-testandvite-ecosystem-ci:before-testscriptscdinto../storybook-sandboxes/...directly. Since the rest of this PR introducesSANDBOX_ROOT/SANDBOX_DIRECTORYto make the sandbox root configurable, consider routing these scripts through the same mechanism (or at least honoringSANDBOX_ROOT) so users overriding the sandbox root don’t end up with mismatched paths here.Also applies to: 23-23, 26-26, 29-32, 36-36, 43-64, 65-75
scripts/task.ts (1)
123-123: Clarify semantics ofstartFrom: "task"and non‑template runs with the newdir/sandboxDir wiringThe new pieces here generally make sense, but there are a couple of behaviors worth explicitly confirming:
startFrom: "task"now forces only the final task to run (shouldRun = finalTask === task), regardless of dependency readiness. That’s fine as an “experts only” mode, but it does mean anydependsOnchain is ignored in this mode. Please confirm that in all places you passstartFrom=task(likely from Nx), the dependencies are either already satisfied or not required for those invocations.templateSandboxDiris nowtemplateKey && join(SANDBOX_DIRECTORY, dir ?? templateKey.replace('/', '-')), andsandboxDir/builtSandboxDirare derived from it. For non‑sandbox tasks,templatecan be omitted, sotemplateKeyandtemplateSandboxDirend upundefined. That’s OK as long as non‑sandbox tasks never assumedetails.sandboxDir/builtSandboxDirare non‑empty; it’s worth double‑checking that assumption holds across the tasks in./tasks/*.- Using
SANDBOX_DIRECTORYplus the per‑projectdiroption lines up nicely with the newproject.jsontargetoptions.dirvalues and the SANDBOX_ROOT mechanism; this should keep Nx‑driven runs and directyarn taskruns in sync as long as callers provide--dirwhen needed.If you’d like, I can draft a quick
rg/ast-grepscript to scanscripts/tasks/**/*.tsfor usages ofsandboxDir/builtSandboxDirthat might break whentemplateis omitted.Also applies to: 152-157, 395-407, 441-442, 450-454, 501-512
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (11)
.yarn/patches/@testing-library-user-event-npm-14.6.1-5da7e1d4e2.patchis excluded by!**/.yarn/**.yarn/patches/@types-babel__traverse-npm-7.20.6-fac4243243.patchis excluded by!**/.yarn/**.yarn/patches/@vitest-expect-npm-3.2.4-97c526d5cc.patchis excluded by!**/.yarn/**.yarn/patches/react-aria-components-npm-1.12.2-6c5dcdafab.patchis excluded by!**/.yarn/**.yarn/plugins/@yarnpkg/plugin-interactive-tools.cjsis excluded by!**/.yarn/**.yarn/plugins/@yarnpkg/plugin-typescript.cjsis excluded by!**/.yarn/**code/yarn.lockis excluded by!**/yarn.lock,!**/*.locknode_modules/.package-lock.jsonis excluded by!**/node_modules/**node_modules/.yarn-state.ymlis excluded by!**/node_modules/**scripts/yarn.lockis excluded by!**/yarn.lock,!**/*.lockyarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (207)
.circleci/config.yml(14 hunks).circleci/src/jobs/bench-sandboxes.yml(1 hunks).circleci/src/jobs/build.yml(2 hunks).circleci/src/jobs/check-sandboxes.yml(1 hunks).circleci/src/jobs/check.yml(1 hunks).circleci/src/jobs/chromatic-sandboxes.yml(1 hunks).circleci/src/jobs/create-sandboxes.yml(1 hunks).circleci/src/jobs/e2e-dev.yml(1 hunks).circleci/src/jobs/e2e-production.yml(1 hunks).circleci/src/jobs/pretty-docs.yml(1 hunks).circleci/src/jobs/test-runner-production.yml(1 hunks).circleci/src/jobs/vitest-integration.yml(1 hunks).cursor/environment.json(1 hunks).env(1 hunks).github/actions/setup-node-and-install/action.yml(1 hunks).github/copilot-instructions.md(5 hunks).github/workflows/nx.yml(1 hunks).gitignore(2 hunks).ignore(0 hunks).nvmrc(1 hunks).nx/workflows/agents.yaml(1 hunks).nx/workflows/distribution-config.yaml(1 hunks).yarnrc.yml(1 hunks)CONTRIBUTING/RELEASING.md(1 hunks)code/.env(0 hunks)code/.nxignore(0 hunks)code/.yarnrc.yml(0 hunks)code/addons/a11y/package.json(0 hunks)code/addons/a11y/project.json(1 hunks)code/addons/docs/package.json(0 hunks)code/addons/docs/project.json(1 hunks)code/addons/links/package.json(0 hunks)code/addons/links/project.json(1 hunks)code/addons/onboarding/package.json(0 hunks)code/addons/onboarding/project.json(1 hunks)code/addons/pseudo-states/package.json(0 hunks)code/addons/pseudo-states/project.json(1 hunks)code/addons/themes/package.json(0 hunks)code/addons/themes/project.json(1 hunks)code/addons/vitest/package.json(0 hunks)code/addons/vitest/project.json(1 hunks)code/builders/builder-vite/package.json(0 hunks)code/builders/builder-vite/project.json(1 hunks)code/builders/builder-webpack5/package.json(0 hunks)code/builders/builder-webpack5/project.json(1 hunks)code/core/package.json(1 hunks)code/core/project.json(1 hunks)code/core/src/components/components/Tabs/StatelessTabsView.tsx(2 hunks)code/core/src/components/components/Tabs/TabsView.tsx(2 hunks)code/core/src/telemetry/anonymous-id.ts(1 hunks)code/e2e-tests/storybook-hooks.spec.ts(2 hunks)code/frameworks/angular/package.json(0 hunks)code/frameworks/angular/project.json(1 hunks)code/frameworks/ember/package.json(0 hunks)code/frameworks/ember/project.json(1 hunks)code/frameworks/html-vite/package.json(0 hunks)code/frameworks/html-vite/project.json(1 hunks)code/frameworks/nextjs-vite/package.json(0 hunks)code/frameworks/nextjs-vite/project.json(1 hunks)code/frameworks/nextjs/package.json(0 hunks)code/frameworks/nextjs/project.json(1 hunks)code/frameworks/nextjs/src/next-image-loader-stub.ts(1 hunks)code/frameworks/preact-vite/package.json(0 hunks)code/frameworks/preact-vite/project.json(1 hunks)code/frameworks/react-native-web-vite/package.json(0 hunks)code/frameworks/react-native-web-vite/project.json(1 hunks)code/frameworks/react-vite/package.json(0 hunks)code/frameworks/react-vite/project.json(1 hunks)code/frameworks/react-webpack5/package.json(0 hunks)code/frameworks/react-webpack5/project.json(1 hunks)code/frameworks/server-webpack5/package.json(0 hunks)code/frameworks/server-webpack5/project.json(1 hunks)code/frameworks/svelte-vite/package.json(0 hunks)code/frameworks/svelte-vite/project.json(1 hunks)code/frameworks/sveltekit/package.json(0 hunks)code/frameworks/sveltekit/project.json(1 hunks)code/frameworks/vue3-vite/package.json(0 hunks)code/frameworks/vue3-vite/project.json(1 hunks)code/frameworks/web-components-vite/package.json(0 hunks)code/frameworks/web-components-vite/project.json(1 hunks)code/lib/cli-sb/project.json(0 hunks)code/lib/cli-storybook/package.json(0 hunks)code/lib/cli-storybook/project.json(1 hunks)code/lib/cli-storybook/src/link.ts(1 hunks)code/lib/codemod/package.json(0 hunks)code/lib/codemod/project.json(1 hunks)code/lib/core-webpack/package.json(0 hunks)code/lib/core-webpack/project.json(1 hunks)code/lib/create-storybook/package.json(0 hunks)code/lib/create-storybook/project.json(1 hunks)code/lib/csf-plugin/package.json(0 hunks)code/lib/csf-plugin/project.json(1 hunks)code/lib/eslint-plugin/package.json(0 hunks)code/lib/eslint-plugin/project.json(1 hunks)code/lib/react-dom-shim/package.json(0 hunks)code/lib/react-dom-shim/project.json(1 hunks)code/nx.json(0 hunks)code/package.json(4 hunks)code/presets/create-react-app/package.json(0 hunks)code/presets/create-react-app/project.json(1 hunks)code/presets/react-webpack/package.json(0 hunks)code/presets/react-webpack/project.json(1 hunks)code/presets/server-webpack/package.json(0 hunks)code/presets/server-webpack/project.json(1 hunks)code/project.json(1 hunks)code/renderers/html/package.json(0 hunks)code/renderers/html/project.json(1 hunks)code/renderers/preact/package.json(0 hunks)code/renderers/preact/project.json(1 hunks)code/renderers/react/package.json(0 hunks)code/renderers/react/project.json(1 hunks)code/renderers/react/src/entry-preview.tsx(0 hunks)code/renderers/server/package.json(0 hunks)code/renderers/server/project.json(1 hunks)code/renderers/svelte/package.json(0 hunks)code/renderers/svelte/project.json(1 hunks)code/renderers/vue3/package.json(0 hunks)code/renderers/vue3/project.json(1 hunks)code/renderers/vue3/template/stories_vue3-vite-default-ts/ReactiveArgs.stories.ts(0 hunks)code/renderers/vue3/template/stories_vue3-vite-default-ts/ScopedSlots.stories.ts(1 hunks)code/renderers/web-components/package.json(0 hunks)code/renderers/web-components/project.json(1 hunks)code/renderers/web-components/src/docs/web-components-properties.test.ts(1 hunks)code/sandbox/angular-cli-default-ts/project.json(1 hunks)code/sandbox/angular-cli-prerelease/project.json(1 hunks)code/sandbox/bench-react-vite-default-ts-nodocs/project.json(1 hunks)code/sandbox/bench-react-vite-default-ts-test-build/project.json(1 hunks)code/sandbox/bench-react-vite-default-ts/project.json(1 hunks)code/sandbox/bench-react-webpack-18-ts-test-build/project.json(1 hunks)code/sandbox/bench-react-webpack-18-ts/project.json(1 hunks)code/sandbox/cra-default-js/project.json(1 hunks)code/sandbox/cra-default-ts/project.json(1 hunks)code/sandbox/ember-3-js/project.json(1 hunks)code/sandbox/ember-default-js/project.json(1 hunks)code/sandbox/experimental-nextjs-vite-default-ts/project.json(0 hunks)code/sandbox/html-rsbuild-default-ts/project.json(1 hunks)code/sandbox/html-vite-default-js/project.json(1 hunks)code/sandbox/html-vite-default-ts/project.json(1 hunks)code/sandbox/internal-react16-webpack/project.json(1 hunks)code/sandbox/internal-react18-webpack-babel/project.json(1 hunks)code/sandbox/internal-server-webpack5/project.json(1 hunks)code/sandbox/lit-rsbuild-default-ts/project.json(1 hunks)code/sandbox/lit-vite-default-js/project.json(1 hunks)code/sandbox/lit-vite-default-ts/project.json(1 hunks)code/sandbox/nextjs-14-ts/project.json(1 hunks)code/sandbox/nextjs-15-ts/project.json(1 hunks)code/sandbox/nextjs-default-js/project.json(0 hunks)code/sandbox/nextjs-default-ts/project.json(1 hunks)code/sandbox/nextjs-prerelease/project.json(1 hunks)code/sandbox/nextjs-vite-14-ts/project.json(1 hunks)code/sandbox/nextjs-vite-15-ts/project.json(1 hunks)code/sandbox/nextjs-vite-default-ts/project.json(1 hunks)code/sandbox/nuxt-vite-default-ts/project.json(0 hunks)code/sandbox/preact-vite-default-js/project.json(1 hunks)code/sandbox/preact-vite-default-ts/project.json(1 hunks)code/sandbox/qwik-vite-default-ts/project.json(1 hunks)code/sandbox/react-native-web-vite-expo-ts/project.json(1 hunks)code/sandbox/react-native-web-vite-rn-cli-ts/project.json(1 hunks)code/sandbox/react-rsbuild-default-ts/project.json(1 hunks)code/sandbox/react-vite-default-js/project.json(1 hunks)code/sandbox/react-vite-default-ts/project.json(1 hunks)code/sandbox/react-vite-prerelease-ts/project.json(1 hunks)code/sandbox/react-webpack-17-ts/project.json(1 hunks)code/sandbox/react-webpack-18-ts/project.json(1 hunks)code/sandbox/react-webpack-prerelease-ts/project.json(1 hunks)code/sandbox/solid-vite-default-js/project.json(1 hunks)code/sandbox/solid-vite-default-ts/project.json(1 hunks)code/sandbox/svelte-kit-default-ts/project.json(0 hunks)code/sandbox/svelte-kit-skeleton-js/project.json(0 hunks)code/sandbox/svelte-kit-skeleton-ts/project.json(1 hunks)code/sandbox/svelte-vite-default-js/project.json(1 hunks)code/sandbox/svelte-vite-default-ts/project.json(1 hunks)code/sandbox/vue3-rsbuild-default-ts/project.json(1 hunks)code/sandbox/vue3-vite-default-js/project.json(1 hunks)code/sandbox/vue3-vite-default-ts/project.json(1 hunks)docs/_snippets/addon-consume-and-update-globaltype.md(1 hunks)docs/_snippets/main-config-builder-custom-config.md(0 hunks)docs/_snippets/storybook-addon-css-example.md(1 hunks)docs/_snippets/storybook-addon-toolkit-types.md(1 hunks)docs/_snippets/storybook-addons-api-getchannel.md(1 hunks)docs/_snippets/storybook-preview-use-global-type.md(1 hunks)nx.json(1 hunks)package.json(1 hunks)scripts/.yarnrc.yml(0 hunks)scripts/build-package.ts(2 hunks)scripts/build/build-package.ts(2 hunks)scripts/build/utils/generate-bundle.ts(1 hunks)scripts/build/utils/generate-type-mappers.ts(1 hunks)scripts/build/utils/generate-types.ts(3 hunks)scripts/build/utils/modify-core-theme-types.ts(2 hunks)scripts/check-package.ts(1 hunks)scripts/check/check-package.ts(1 hunks)scripts/check/utils/typescript.ts(2 hunks)scripts/create-nx-sandbox-projects.ts(1 hunks)scripts/ecosystem-ci/before-test.js(1 hunks)scripts/event-log-checker.ts(1 hunks)scripts/get-sandbox-dir.ts(2 hunks)scripts/get-template.ts(3 hunks)scripts/knip.config.ts(1 hunks)scripts/package.json(2 hunks)scripts/prepare-sandbox.ts(1 hunks)scripts/project.json(1 hunks)scripts/release/__tests__/version.test.ts(1 hunks)scripts/release/version.ts(2 hunks)scripts/run-registry.ts(8 hunks)scripts/sandbox/utils/getPort.ts(1 hunks)scripts/task.ts(5 hunks)
⛔ Files not processed due to max files limit (32)
- scripts/tasks/build.ts
- scripts/tasks/check.ts
- scripts/tasks/chromatic.ts
- scripts/tasks/compile.ts
- scripts/tasks/dev.ts
- scripts/tasks/e2e-tests-build.ts
- scripts/tasks/run-registry.ts
- scripts/tasks/sandbox-parts.ts
- scripts/tasks/sandbox.ts
- scripts/tasks/serve.ts
- scripts/tasks/test-runner-build.ts
- scripts/upload-bench.ts
- scripts/utils/constants.ts
- scripts/utils/kill-process-on-port.ts
- scripts/utils/main-js.ts
- scripts/utils/tools.ts
- scripts/utils/workspace.ts
- scripts/utils/yarn.ts
- test-storybooks/portable-stories-kitchen-sink/nextjs/package.json
- test-storybooks/portable-stories-kitchen-sink/nextjs/project.json
- test-storybooks/portable-stories-kitchen-sink/react-vitest-3/package.json
- test-storybooks/portable-stories-kitchen-sink/react-vitest-3/playwright-e2e.config.ts
- test-storybooks/portable-stories-kitchen-sink/react-vitest-3/project.json
- test-storybooks/portable-stories-kitchen-sink/react/package.json
- test-storybooks/portable-stories-kitchen-sink/react/playwright-e2e.config.ts
- test-storybooks/portable-stories-kitchen-sink/react/project.json
- test-storybooks/portable-stories-kitchen-sink/svelte/package.json
- test-storybooks/portable-stories-kitchen-sink/svelte/project.json
- test-storybooks/portable-stories-kitchen-sink/vue3/package.json
- test-storybooks/portable-stories-kitchen-sink/vue3/project.json
- test-storybooks/yarn-pnp/package.json
- test-storybooks/yarn-pnp/project.json
💤 Files with no reviewable changes (55)
- code/addons/docs/package.json
- code/frameworks/web-components-vite/package.json
- code/frameworks/react-vite/package.json
- code/frameworks/ember/package.json
- code/addons/links/package.json
- code/builders/builder-vite/package.json
- code/builders/builder-webpack5/package.json
- code/presets/react-webpack/package.json
- code/frameworks/react-webpack5/package.json
- code/renderers/svelte/package.json
- code/frameworks/angular/package.json
- code/renderers/vue3/package.json
- code/addons/themes/package.json
- code/frameworks/nextjs-vite/package.json
- code/frameworks/react-native-web-vite/package.json
- code/renderers/server/package.json
- code/sandbox/svelte-kit-skeleton-js/project.json
- code/addons/onboarding/package.json
- code/presets/server-webpack/package.json
- .ignore
- code/sandbox/nextjs-default-js/project.json
- code/lib/csf-plugin/package.json
- code/frameworks/svelte-vite/package.json
- code/renderers/react/package.json
- code/addons/vitest/package.json
- code/renderers/html/package.json
- code/.env
- code/.nxignore
- code/renderers/react/src/entry-preview.tsx
- code/addons/a11y/package.json
- code/sandbox/svelte-kit-default-ts/project.json
- code/frameworks/html-vite/package.json
- code/lib/cli-sb/project.json
- code/lib/react-dom-shim/package.json
- code/frameworks/vue3-vite/package.json
- code/sandbox/experimental-nextjs-vite-default-ts/project.json
- code/lib/codemod/package.json
- code/presets/create-react-app/package.json
- code/frameworks/sveltekit/package.json
- docs/_snippets/main-config-builder-custom-config.md
- code/lib/core-webpack/package.json
- code/lib/eslint-plugin/package.json
- code/lib/cli-storybook/package.json
- code/frameworks/nextjs/package.json
- code/lib/create-storybook/package.json
- code/.yarnrc.yml
- code/frameworks/preact-vite/package.json
- code/sandbox/nuxt-vite-default-ts/project.json
- code/renderers/web-components/package.json
- code/nx.json
- code/renderers/vue3/template/stories_vue3-vite-default-ts/ReactiveArgs.stories.ts
- code/addons/pseudo-states/package.json
- code/frameworks/server-webpack5/package.json
- scripts/.yarnrc.yml
- code/renderers/preact/package.json
✅ Files skipped from review due to trivial changes (2)
- code/lib/codemod/project.json
- docs/_snippets/storybook-addon-toolkit-types.md
🚧 Files skipped from review as they are similar to previous changes (88)
- code/sandbox/vue3-rsbuild-default-ts/project.json
- code/lib/cli-storybook/src/link.ts
- scripts/event-log-checker.ts
- scripts/build/utils/generate-type-mappers.ts
- code/addons/vitest/project.json
- .circleci/src/jobs/vitest-integration.yml
- code/frameworks/sveltekit/project.json
- code/e2e-tests/storybook-hooks.spec.ts
- scripts/ecosystem-ci/before-test.js
- code/addons/onboarding/project.json
- code/frameworks/react-webpack5/project.json
- scripts/check/utils/typescript.ts
- .circleci/src/jobs/test-runner-production.yml
- code/presets/server-webpack/project.json
- code/core/src/telemetry/anonymous-id.ts
- code/renderers/react/project.json
- code/frameworks/preact-vite/project.json
- code/frameworks/nextjs/src/next-image-loader-stub.ts
- code/lib/csf-plugin/project.json
- .circleci/src/jobs/build.yml
- code/renderers/web-components/project.json
- code/renderers/vue3/template/stories_vue3-vite-default-ts/ScopedSlots.stories.ts
- code/sandbox/cra-default-ts/project.json
- .circleci/src/jobs/bench-sandboxes.yml
- scripts/check-package.ts
- code/lib/create-storybook/project.json
- code/sandbox/nextjs-vite-14-ts/project.json
- code/lib/core-webpack/project.json
- scripts/sandbox/utils/getPort.ts
- code/renderers/web-components/src/docs/web-components-properties.test.ts
- code/sandbox/angular-cli-prerelease/project.json
- code/sandbox/nextjs-15-ts/project.json
- code/sandbox/react-vite-default-ts/project.json
- code/sandbox/preact-vite-default-ts/project.json
- code/renderers/html/project.json
- scripts/get-sandbox-dir.ts
- code/frameworks/svelte-vite/project.json
- code/sandbox/nextjs-14-ts/project.json
- scripts/build/utils/generate-bundle.ts
- scripts/get-template.ts
- code/sandbox/html-vite-default-ts/project.json
- code/core/package.json
- code/sandbox/react-native-web-vite-expo-ts/project.json
- code/sandbox/vue3-vite-default-js/project.json
- code/presets/react-webpack/project.json
- scripts/check/check-package.ts
- code/presets/create-react-app/project.json
- code/renderers/server/project.json
- code/sandbox/react-rsbuild-default-ts/project.json
- code/frameworks/react-vite/project.json
- .github/actions/setup-node-and-install/action.yml
- scripts/project.json
- .cursor/environment.json
- code/renderers/svelte/project.json
- .circleci/src/jobs/e2e-production.yml
- code/sandbox/react-vite-prerelease-ts/project.json
- code/frameworks/html-vite/project.json
- code/sandbox/preact-vite-default-js/project.json
- CONTRIBUTING/RELEASING.md
- .circleci/src/jobs/check-sandboxes.yml
- code/sandbox/svelte-kit-skeleton-ts/project.json
- docs/_snippets/storybook-preview-use-global-type.md
- code/frameworks/web-components-vite/project.json
- code/frameworks/ember/project.json
- .circleci/src/jobs/pretty-docs.yml
- code/renderers/preact/project.json
- code/sandbox/html-rsbuild-default-ts/project.json
- code/sandbox/solid-vite-default-ts/project.json
- code/lib/eslint-plugin/project.json
- .circleci/src/jobs/create-sandboxes.yml
- .circleci/src/jobs/e2e-dev.yml
- code/sandbox/bench-react-webpack-18-ts/project.json
- .nvmrc
- scripts/build/utils/modify-core-theme-types.ts
- code/sandbox/nextjs-default-ts/project.json
- code/sandbox/internal-react16-webpack/project.json
- code/frameworks/vue3-vite/project.json
- code/sandbox/nextjs-vite-default-ts/project.json
- code/sandbox/react-webpack-prerelease-ts/project.json
- scripts/knip.config.ts
- code/addons/links/project.json
- code/core/src/components/components/Tabs/TabsView.tsx
- code/builders/builder-webpack5/project.json
- code/sandbox/nextjs-vite-15-ts/project.json
- code/sandbox/qwik-vite-default-ts/project.json
- code/frameworks/nextjs/project.json
- docs/_snippets/addon-consume-and-update-globaltype.md
- code/frameworks/nextjs-vite/project.json
🧰 Additional context used
📓 Path-based instructions (8)
**/*.{ts,tsx,js,jsx,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use camelCase for variable and function names
Files:
scripts/build/utils/generate-types.tscode/core/src/components/components/Tabs/StatelessTabsView.tsxscripts/release/__tests__/version.test.tsscripts/build/build-package.tsscripts/task.tsscripts/release/version.tsscripts/prepare-sandbox.tsscripts/create-nx-sandbox-projects.tsscripts/run-registry.tsscripts/build-package.ts
**/*.{ts,tsx}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx}: Enable TypeScript strict mode
Export functions from modules for testing purposes
Files:
scripts/build/utils/generate-types.tscode/core/src/components/components/Tabs/StatelessTabsView.tsxscripts/release/__tests__/version.test.tsscripts/build/build-package.tsscripts/task.tsscripts/release/version.tsscripts/prepare-sandbox.tsscripts/create-nx-sandbox-projects.tsscripts/run-registry.tsscripts/build-package.ts
**/*.{ts,tsx,js,jsx,json,html,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx,js,jsx,json,html,mjs}: Use ESLint and Prettier for code style enforcement
Run 'yarn prettier --write ' to format code after making changes
Run 'yarn lint:js:cmd ' to check for ESLint issues after making changes
Files:
scripts/build/utils/generate-types.tscode/core/src/components/components/Tabs/StatelessTabsView.tsxscripts/release/__tests__/version.test.tscode/package.jsonscripts/build/build-package.tscode/addons/a11y/project.jsoncode/addons/docs/project.jsoncode/project.jsoncode/sandbox/ember-3-js/project.jsoncode/addons/pseudo-states/project.jsoncode/sandbox/solid-vite-default-js/project.jsoncode/sandbox/svelte-vite-default-js/project.jsonscripts/task.tscode/sandbox/cra-default-js/project.jsoncode/sandbox/ember-default-js/project.jsoncode/frameworks/server-webpack5/project.jsoncode/addons/themes/project.jsoncode/sandbox/internal-server-webpack5/project.jsonscripts/release/version.tscode/sandbox/vue3-vite-default-ts/project.jsoncode/sandbox/react-native-web-vite-rn-cli-ts/project.jsoncode/frameworks/react-native-web-vite/project.jsoncode/sandbox/lit-vite-default-js/project.jsoncode/sandbox/react-webpack-18-ts/project.jsoncode/sandbox/angular-cli-default-ts/project.jsonscripts/prepare-sandbox.tscode/sandbox/react-webpack-17-ts/project.jsonscripts/create-nx-sandbox-projects.tsscripts/run-registry.tscode/sandbox/react-vite-default-js/project.jsoncode/lib/react-dom-shim/project.jsonpackage.jsoncode/lib/cli-storybook/project.jsoncode/sandbox/bench-react-vite-default-ts/project.jsoncode/sandbox/bench-react-webpack-18-ts-test-build/project.jsoncode/sandbox/internal-react18-webpack-babel/project.jsoncode/sandbox/lit-vite-default-ts/project.jsoncode/core/project.jsoncode/sandbox/nextjs-prerelease/project.jsoncode/sandbox/bench-react-vite-default-ts-nodocs/project.jsoncode/sandbox/lit-rsbuild-default-ts/project.jsoncode/builders/builder-vite/project.jsoncode/sandbox/html-vite-default-js/project.jsonscripts/build-package.tscode/sandbox/bench-react-vite-default-ts-test-build/project.jsoncode/sandbox/svelte-vite-default-ts/project.jsonscripts/package.jsonnx.jsoncode/frameworks/angular/project.jsoncode/renderers/vue3/project.json
code/**/!(*.test).{ts,tsx,js,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
code/**/!(*.test).{ts,tsx,js,mjs}: Use 'logger' from 'storybook/internal/node-logger' for server-side (Node.js) logging, not console.log/console.warn/console.error
Use 'logger' from 'storybook/internal/client-logger' for client-side (browser) logging, not console.log/console.warn/console.error
Do not use console.log, console.warn, or console.error directly unless in isolated files where importing loggers would significantly increase bundle size
Files:
code/core/src/components/components/Tabs/StatelessTabsView.tsx
**/*.{test,spec}.{ts,tsx}
📄 CodeRabbit inference engine (.cursorrules)
**/*.{test,spec}.{ts,tsx}: Test files should follow the naming pattern*.test.ts,*.test.tsx,*.spec.ts, or*.spec.tsx
Follow the spy mocking rules defined in.cursor/rules/spy-mocking.mdcfor consistent mocking patterns with Vitest
Files:
scripts/release/__tests__/version.test.ts
**/*.test.{ts,tsx}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.test.{ts,tsx}: Write meaningful unit tests that import and call functions being tested, not just verify syntax patterns
Use 'yarn vitest run --coverage ' to run tests with coverage reports and aim for 75%+ coverage of statements/lines
Focus test coverage on all branches, conditions, edge cases, error paths, and different input variations
Use 'vi.mock()' to mock external dependencies like file system and loggers in unit tests
Files:
scripts/release/__tests__/version.test.ts
**/*.test.{ts,tsx,js,jsx}
📄 CodeRabbit inference engine (.cursor/rules/spy-mocking.mdc)
**/*.test.{ts,tsx,js,jsx}: Usevi.mock()with thespy: trueoption for all package and file mocks in Vitest tests
Place all mocks at the top of the test file before any test cases
Usevi.mocked()to type and access the mocked functions in Vitest tests
Implement mock behaviors inbeforeEachblocks in Vitest tests
Mock all required dependencies that the test subject uses
Each mock implementation should return a Promise for async functions in Vitest
Mock implementations should match the expected return type of the original function
Mock all required properties and methods that the test subject uses in Vitest tests
Avoid direct function mocking withoutvi.mocked()in Vitest tests
Avoid mock implementations outside ofbeforeEachblocks in Vitest tests
Avoid mocking without thespy: trueoption in Vitest tests
Avoid inline mock implementations within test cases in Vitest tests
Avoid mocking only a subset of required dependencies in Vitest tests
Mock at the highest level of abstraction needed in Vitest tests
Keep mock implementations simple and focused in Vitest tests
Use type-safe mocking withvi.mocked()in Vitest tests
Document complex mock behaviors in Vitest tests
Group related mocks together in Vitest tests
Files:
scripts/release/__tests__/version.test.ts
.yarnrc.yml
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use Yarn 4.9.1 as the package manager
Files:
.yarnrc.yml
🧠 Learnings (41)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Applied to files:
.gitignorecode/package.json.circleci/config.ymlscripts/release/version.tsscripts/prepare-sandbox.tsscripts/create-nx-sandbox-projects.tspackage.json.github/copilot-instructions.mdscripts/build-package.ts.env
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.test.{ts,tsx} : Use 'yarn vitest run --coverage <test-file>' to run tests with coverage reports and aim for 75%+ coverage of statements/lines
Applied to files:
.gitignorecode/package.jsoncode/project.jsonscripts/package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Run 'yarn prettier --write <file>' to format code after making changes
Applied to files:
.gitignorecode/package.jsoncode/project.jsonscripts/create-nx-sandbox-projects.ts.github/copilot-instructions.mdscripts/build-package.tsscripts/package.json
📚 Learning: 2025-10-02T09:22:13.215Z
Learnt from: JReinhold
Repo: storybookjs/storybook PR: 32607
File: code/package.json:243-243
Timestamp: 2025-10-02T09:22:13.215Z
Learning: The Storybook repository uses Yarn v^4 (any 4.x version) as the package manager, configured via .yarnrc.yml and package.json packageManager field. Specific patch versions within v4 can be upgraded as needed.
Applied to files:
.gitignorecode/package.json.circleci/config.ymlscripts/release/version.ts.yarnrc.ymlpackage.json.github/copilot-instructions.md
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid mocking only a subset of required dependencies in Vitest tests
Applied to files:
.gitignorescripts/release/__tests__/version.test.tscode/package.jsoncode/sandbox/vue3-vite-default-ts/project.jsoncode/sandbox/react-native-web-vite-rn-cli-ts/project.jsoncode/sandbox/react-vite-default-js/project.jsoncode/sandbox/bench-react-vite-default-ts/project.jsoncode/sandbox/lit-vite-default-ts/project.jsoncode/sandbox/bench-react-vite-default-ts-nodocs/project.jsoncode/sandbox/bench-react-vite-default-ts-test-build/project.jsoncode/sandbox/svelte-vite-default-ts/project.jsonscripts/package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to code/**/!(*.test).{ts,tsx,js,mjs} : Use 'logger' from 'storybook/internal/node-logger' for server-side (Node.js) logging, not console.log/console.warn/console.error
Applied to files:
.gitignorecode/package.json.github/copilot-instructions.md
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Use ESLint and Prettier for code style enforcement
Applied to files:
.gitignore
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to code/**/!(*.test).{ts,tsx,js,mjs} : Use 'logger' from 'storybook/internal/client-logger' for client-side (browser) logging, not console.log/console.warn/console.error
Applied to files:
.gitignorecode/package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Run 'yarn lint:js:cmd <file>' to check for ESLint issues after making changes
Applied to files:
.gitignorecode/package.jsoncode/project.jsonscripts/create-nx-sandbox-projects.tspackage.json.github/copilot-instructions.mdscripts/package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to .yarnrc.yml : Use Yarn 4.9.1 as the package manager
Applied to files:
.gitignorecode/package.json.circleci/config.yml.yarnrc.yml.github/copilot-instructions.mdscripts/package.json
📚 Learning: 2025-10-01T15:24:01.060Z
Learnt from: Sidnioulz
Repo: storybookjs/storybook PR: 32594
File: code/core/src/components/components/Popover/WithPopover.tsx:7-9
Timestamp: 2025-10-01T15:24:01.060Z
Learning: In the Storybook repository, "react-aria-components/patched-dist/*" (e.g., "react-aria-components/patched-dist/Dialog", "react-aria-components/patched-dist/Popover", "react-aria-components/patched-dist/Tooltip") are valid import paths created by a patch applied to the react-aria-components package. These imports should not be flagged as broken or invalid until a maintainer explicitly states they are no longer acceptable.
Applied to files:
code/core/src/components/components/Tabs/StatelessTabsView.tsx
📚 Learning: 2025-11-05T09:38:47.712Z
Learnt from: Sidnioulz
Repo: storybookjs/storybook PR: 32458
File: code/core/src/components/components/Select/Select.tsx:200-204
Timestamp: 2025-11-05T09:38:47.712Z
Learning: Repo: storybookjs/storybook — Guidance: Until Storybook 11 is released, do not suggest using React.useId anywhere (e.g., in code/core/src/components/components/Select/Select.tsx) to maintain compatibility with React 17 runtimes. Prefer advising: accept a caller-provided props.id and, if needed, generate a client-only fallback id to minimize SSR hydration issues — but avoid useId. Resume prompting for useId after Storybook 11.
Applied to files:
code/core/src/components/components/Tabs/StatelessTabsView.tsxcode/package.json.github/copilot-instructions.md
📚 Learning: 2025-11-24T17:49:31.838Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
Applied to files:
scripts/release/__tests__/version.test.tscode/package.jsonscripts/release/version.tscode/sandbox/vue3-vite-default-ts/project.jsoncode/sandbox/react-native-web-vite-rn-cli-ts/project.jsonpackage.jsoncode/sandbox/bench-react-vite-default-ts/project.jsoncode/sandbox/lit-vite-default-ts/project.jsoncode/sandbox/lit-rsbuild-default-ts/project.jsoncode/sandbox/bench-react-vite-default-ts-test-build/project.jsoncode/sandbox/svelte-vite-default-ts/project.jsonnx.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx} : Export functions from modules for testing purposes
Applied to files:
scripts/release/__tests__/version.test.ts
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Mock implementations should match the expected return type of the original function
Applied to files:
scripts/release/__tests__/version.test.ts
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid direct function mocking without `vi.mocked()` in Vitest tests
Applied to files:
scripts/release/__tests__/version.test.ts
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.test.{ts,tsx} : Use 'vi.mock()' to mock external dependencies like file system and loggers in unit tests
Applied to files:
scripts/release/__tests__/version.test.ts
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid inline mock implementations within test cases in Vitest tests
Applied to files:
scripts/release/__tests__/version.test.ts
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Mock all required dependencies that the test subject uses
Applied to files:
scripts/release/__tests__/version.test.tscode/package.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Group related mocks together in Vitest tests
Applied to files:
scripts/release/__tests__/version.test.ts
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Keep mock implementations simple and focused in Vitest tests
Applied to files:
scripts/release/__tests__/version.test.ts
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Use `vi.mocked()` to type and access the mocked functions in Vitest tests
Applied to files:
scripts/release/__tests__/version.test.ts
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Use `vi.mock()` with the `spy: true` option for all package and file mocks in Vitest tests
Applied to files:
scripts/release/__tests__/version.test.ts
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Use type-safe mocking with `vi.mocked()` in Vitest tests
Applied to files:
scripts/release/__tests__/version.test.ts
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Mock at the highest level of abstraction needed in Vitest tests
Applied to files:
scripts/release/__tests__/version.test.ts
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Mock all required properties and methods that the test subject uses in Vitest tests
Applied to files:
scripts/release/__tests__/version.test.ts
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Each mock implementation should return a Promise for async functions in Vitest
Applied to files:
scripts/release/__tests__/version.test.ts
📚 Learning: 2025-09-17T07:31:04.432Z
Learnt from: ndelangen
Repo: storybookjs/storybook PR: 32484
File: code/core/package.json:326-326
Timestamp: 2025-09-17T07:31:04.432Z
Learning: In Storybook's core package, dependencies like `open` are bundled into the final distribution during the build process, so they should remain in devDependencies rather than being moved to dependencies. End users don't need these packages as separate runtime dependencies since they're included in the bundled code.
Applied to files:
code/package.jsonpackage.json.github/copilot-instructions.md
📚 Learning: 2025-11-05T09:37:25.920Z
Learnt from: Sidnioulz
Repo: storybookjs/storybook PR: 32458
File: code/core/src/components/components/tooltip/WithTooltip.tsx:54-96
Timestamp: 2025-11-05T09:37:25.920Z
Learning: Repo: storybookjs/storybook — In code/core/src/components/components/tooltip/WithTooltip.tsx, the legacy WithTooltip implementation is intentionally reintroduced for backward compatibility and is deprecated; maintainers (per Sidnioulz) do not want maintenance or improvements on it. Prefer WithTooltipNew/Popover; avoid suggesting changes to WithTooltip.* going forward.
Applied to files:
code/package.json.github/copilot-instructions.md
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'cd code && yarn storybook:ui' to start the development server for testing UI changes on http://localhost:6006/
Applied to files:
code/package.json.circleci/config.ymlscripts/prepare-sandbox.ts.github/copilot-instructions.md
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Applied to files:
code/package.json.circleci/config.ymlscripts/task.ts.circleci/src/jobs/chromatic-sandboxes.ymlscripts/prepare-sandbox.tsscripts/create-nx-sandbox-projects.tscode/sandbox/bench-react-webpack-18-ts-test-build/project.json.github/copilot-instructions.mdcode/sandbox/bench-react-vite-default-ts-test-build/project.jsonscripts/package.jsonnx.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Compile packages with 'yarn task --task compile' after making code changes before testing
Applied to files:
code/package.json.circleci/config.ymlcode/project.json.circleci/src/jobs/check.yml.github/copilot-instructions.mdscripts/build-package.tsscripts/package.json
📚 Learning: 2025-09-29T13:20:23.346Z
Learnt from: mrginglymus
Repo: storybookjs/storybook PR: 32556
File: code/core/package.json:309-313
Timestamp: 2025-09-29T13:20:23.346Z
Learning: The `fast-printf` dependency in Storybook's core package is bundled into the final distribution during the build process, so it should remain in devDependencies rather than being moved to dependencies, following the same pattern as other bundled dependencies like `open`.
Applied to files:
code/package.jsonpackage.json
📚 Learning: 2025-09-18T20:51:06.618Z
Learnt from: Sidnioulz
Repo: storybookjs/storybook PR: 32458
File: code/core/src/viewport/components/Tool.tsx:38-39
Timestamp: 2025-09-18T20:51:06.618Z
Learning: The useGlobals hook from storybook/manager-api returns a tuple where the third element (storyGlobals) is typed as Globals, not Globals | undefined. This means TypeScript guarantees it's always defined, making the `in` operator safe to use without additional null checks.
Applied to files:
docs/_snippets/storybook-addons-api-getchannel.md
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to README.md : Update relevant README files for significant changes
Applied to files:
.circleci/config.yml.github/copilot-instructions.md
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Run 'yarn task --task check' to verify TypeScript types across all packages before submitting PRs
Applied to files:
.circleci/config.yml.github/copilot-instructions.mdscripts/build-package.tsscripts/package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to .nvmrc : Use Node.js 22.16.0 as specified in .nvmrc
Applied to files:
.github/copilot-instructions.md
📚 Learning: 2025-11-24T17:49:31.838Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Tests are located in the `code/` directory with root directory for test execution at `./code/`
Applied to files:
.github/copilot-instructions.md
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Do not use 'yarn task --task dev' or 'yarn start' as these start permanent development servers that run indefinitely
Applied to files:
.github/copilot-instructions.md
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Document complex mock behaviors in Vitest tests
Applied to files:
scripts/package.json
🪛 dotenv-linter (4.0.0)
.env
[warning] 2-2: [UnorderedKey] The IN_STORYBOOK_SANDBOX key should go before the STORYBOOK_DISABLE_TELEMETRY key
(UnorderedKey)
[warning] 3-3: [EndingBlankLine] No blank line at the end of the file
(EndingBlankLine)
[warning] 3-3: [UnorderedKey] The SANDBOX_ROOT key should go before the STORYBOOK_DISABLE_TELEMETRY key
(UnorderedKey)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: normal
- GitHub Check: Core Unit Tests, windows-latest
- GitHub Check: nx
kasperpeulen
added
ci:daily
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (1)
package.json (1)
3-4: ESM type inheritance issue requires scripts/eslint-plugin-local-rules fix.As flagged in the previous review, root
"type": "module"causesscripts/eslint-plugin-local-rules/to inherit ESM, breaking therequire()calls in those files. Ensurescripts/eslint-plugin-local-rules/package.jsonhas"type": "commonjs"added or those files are converted to ES module syntax.
🧹 Nitpick comments (1)
.nx/workflows/agents.yaml (1)
91-95: Windows template initialization incomplete.The
windows-jslaunch template lacks theinit-stepsconfiguration (marked as TODO on line 94). This will require implementation before Windows agents can be used in distributed CI. For now, this is acceptable as a known gap, but should be prioritized when Windows support is needed.Would you like me to generate the Windows init-steps configuration based on the Linux templates?
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
yarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (4)
.github/workflows/nx.yml(1 hunks).nx/workflows/agents.yaml(1 hunks)code/frameworks/nextjs/src/next-image-loader-stub.ts(1 hunks)package.json(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- code/frameworks/nextjs/src/next-image-loader-stub.ts
- .github/workflows/nx.yml
🧰 Additional context used
📓 Path-based instructions (1)
**/*.{ts,tsx,js,jsx,json,html,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx,js,jsx,json,html,mjs}: Use ESLint and Prettier for code style enforcement
Run 'yarn prettier --write ' to format code after making changes
Run 'yarn lint:js:cmd ' to check for ESLint issues after making changes
Files:
package.json
🧠 Learnings (7)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Run 'yarn lint:js:cmd <file>' to check for ESLint issues after making changes
Applied to files:
package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Applied to files:
package.json
📚 Learning: 2025-10-02T09:22:13.215Z
Learnt from: JReinhold
Repo: storybookjs/storybook PR: 32607
File: code/package.json:243-243
Timestamp: 2025-10-02T09:22:13.215Z
Learning: The Storybook repository uses Yarn v^4 (any 4.x version) as the package manager, configured via .yarnrc.yml and package.json packageManager field. Specific patch versions within v4 can be upgraded as needed.
Applied to files:
package.json
📚 Learning: 2025-09-17T07:31:04.432Z
Learnt from: ndelangen
Repo: storybookjs/storybook PR: 32484
File: code/core/package.json:326-326
Timestamp: 2025-09-17T07:31:04.432Z
Learning: In Storybook's core package, dependencies like `open` are bundled into the final distribution during the build process, so they should remain in devDependencies rather than being moved to dependencies. End users don't need these packages as separate runtime dependencies since they're included in the bundled code.
Applied to files:
package.json
📚 Learning: 2025-11-24T17:49:31.838Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
Applied to files:
package.json
📚 Learning: 2025-09-29T13:20:23.346Z
Learnt from: mrginglymus
Repo: storybookjs/storybook PR: 32556
File: code/core/package.json:309-313
Timestamp: 2025-09-29T13:20:23.346Z
Learning: The `fast-printf` dependency in Storybook's core package is bundled into the final distribution during the build process, so it should remain in devDependencies rather than being moved to dependencies, following the same pattern as other bundled dependencies like `open`.
Applied to files:
package.json
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: daily
- GitHub Check: nx
🔇 Additional comments (6)
.nx/workflows/agents.yaml (1)
75-75: Sandbox path configuration aligns with PR objectives.The SANDBOX_ROOT environment variable correctly points to
../storybook-sandboxesfor sandbox isolation as described in the PR summary.package.json (5)
5-17: Workspaces configuration properly structured for monorepo root move.The workspace packages declaration correctly reflects the move from
./coderoot to the git repository root. All package globs (code/addons/, code/frameworks/, etc.) and the scripts workspace are properly configured relative to the new root.
23-23: Root-level CLI scripts updated appropriately.The simplified
i, newnx, and updatedtaskscripts align with the monorepo restructuring. Scripts now correctly delegate to thescripts/subdirectory from the new root context.Also applies to: 26-26, 32-32
29-29: Ecosystem CI script paths updated for sandbox relocation.The
svelte-ecosystem-ci:before-testandvite-ecosystem-ci:before-testscripts correctly reference the new sandbox path (../storybook-sandboxes/) as per the PR's sandbox isolation objective.Also applies to: 36-36
44-64: Resolutions block properly manages monorepo-wide dependency consistency.The resolutions configuration correctly pins transitive dependencies across the workspace, using a mix of version constraints, patches, and latest versions to ensure consistency. This is appropriate monorepo practice.
Note: The Playwright vulnerability flagged in lines 48, 58–59 is addressed separately in a prior review comment.
65-73: Root-level devDependencies appropriately support NX workspace integration.The new devDependencies include essential NX workspace tooling (@nx/workspace, nx) along with complementary utilities (prettier, jiti, kill-port, @playwright/test). This aligns with the PR's NX integration objectives.
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
scripts/build/utils/generate-types.ts (1)
21-25: Fix comment/code mismatch for concurrency limitThe comment still states "limit the number of concurrent processes to 3" and "3 seems to be the sweet spot", but the implementation uses
limit(10). Please either update the comment to "10" (and rationale) or change the limit back to 3 so they agree.
🧹 Nitpick comments (3)
.yarnrc.yml (2)
7-13: Review justification for suppressing Yarn warnings.The configuration discards three Yarn warning codes (YN0007, YN0005, YN0076) without inline commentary explaining why. These codes typically relate to lockfile version mismatches, resolution issues, and caching problems—which may surface legitimate issues during the monorepo refactor.
Verify that suppressing these warnings will not obscure problems:
YN0007: Lockfile version mismatch (could indicate schema drift after refactor)YN0005: Package resolution issuesYN0076: Registry or fetch problems (especially relevant givenunsafeHttpWhitelistfor local Verdaccio)If suppression is necessary, add an inline comment explaining the reason for each code.
1-1: Consider trade-off of zero compression.Setting
compressionLevel: 0disables package compression in the global cache. This improves install speed but increases disk footprint. Confirm this trade-off is intentional for the sandboxing/CI improvements in this PR.scripts/build/utils/generate-types.ts (1)
10-15: Optional: deriveDIR_CODEfromROOT_DIRECTORYfor consistencyGiven that
ROOT_DIRECTORYnow represents the monorepo root, you could reduce path fragility by defining:const DIR_CODE = join(ROOT_DIRECTORY, 'code');instead of relying on
import.meta.dirnamewith multiple..segments. This keeps all root-relative paths centralized.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
.gitignore(2 hunks).yarnrc.yml(1 hunks)docs/_snippets/storybook-addons-api-getchannel.md(1 hunks)scripts/build/utils/generate-types.ts(2 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- .gitignore
🧰 Additional context used
📓 Path-based instructions (4)
.yarnrc.yml
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use Yarn 4.9.1 as the package manager
Files:
.yarnrc.yml
**/*.{ts,tsx,js,jsx,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use camelCase for variable and function names
Files:
scripts/build/utils/generate-types.ts
**/*.{ts,tsx}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx}: Enable TypeScript strict mode
Export functions from modules for testing purposes
Files:
scripts/build/utils/generate-types.ts
**/*.{ts,tsx,js,jsx,json,html,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx,js,jsx,json,html,mjs}: Use ESLint and Prettier for code style enforcement
Run 'yarn prettier --write ' to format code after making changes
Run 'yarn lint:js:cmd ' to check for ESLint issues after making changes
Files:
scripts/build/utils/generate-types.ts
🧠 Learnings (4)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to .yarnrc.yml : Use Yarn 4.9.1 as the package manager
Applied to files:
.yarnrc.yml
📚 Learning: 2025-10-02T09:22:13.215Z
Learnt from: JReinhold
Repo: storybookjs/storybook PR: 32607
File: code/package.json:243-243
Timestamp: 2025-10-02T09:22:13.215Z
Learning: The Storybook repository uses Yarn v^4 (any 4.x version) as the package manager, configured via .yarnrc.yml and package.json packageManager field. Specific patch versions within v4 can be upgraded as needed.
Applied to files:
.yarnrc.yml
📚 Learning: 2025-09-18T20:51:06.618Z
Learnt from: Sidnioulz
Repo: storybookjs/storybook PR: 32458
File: code/core/src/viewport/components/Tool.tsx:38-39
Timestamp: 2025-09-18T20:51:06.618Z
Learning: In viewport tool code, when using the `useGlobals` hook from storybook/manager-api, the third returned value `storyGlobals` is guaranteed by TypeScript to be defined (not undefined/null), making the `in` operator safe to use without additional null checks.
Applied to files:
docs/_snippets/storybook-addons-api-getchannel.md
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: daily
- GitHub Check: nx
- GitHub Check: Core Unit Tests, windows-latest
🔇 Additional comments (3)
docs/_snippets/storybook-addons-api-getchannel.md (1)
15-21: Good fix — documentation example now demonstrates best practices.The comma operator pattern has been properly replaced with two clear, separate statements. Each call is now on its own line with proper semicolons, making the code more readable and teaching a better pattern to developers learning from this example. The execution order (updateGlobals first, then emit FORCE_RE_RENDER) is preserved, and the comment placement is appropriate.
.yarnrc.yml (1)
38-38: Verify Yarn version against coding guidelines.The yarnPath specifies
yarn-4.10.3.cjs, but the coding guidelines require Yarn 4.9.1. Confirm whether 4.10.3 is intentional (e.g., for a specific bug fix or feature in the monorepo refactor) or if this should be downgraded to 4.9.1.scripts/build/utils/generate-types.ts (1)
6-6: ROOT_DIRECTORY-based jiti path looks correct for new monorepo rootImporting
ROOT_DIRECTORYand resolving the jiti binary fromjoin(ROOT_DIRECTORY, 'node_modules', '.bin', 'jiti')makes this script robust to the workspace root change and avoids fragile deep relative paths. The quoting and spawn options remain consistent with the previous implementation.Also applies to: 33-35
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
scripts/utils/cli-utils.ts (2)
10-17: Excellent simplification of dependency installation.The refactoring from multiple parallel installs to a single root-level install is clearer and aligns perfectly with the monorepo restructuring. The spawn configuration correctly inherits stdio streams for proper logging.
Consider removing
shell: trueunless it's required for cross-platform compatibility, as it adds a small overhead:const task = spawn('yarn', ['install'], { cwd: ROOT_DIRECTORY, - shell: true, stdio: ['inherit', 'inherit', 'inherit'], });
19-30: Error handling is correct.The promise-based error handling properly rejects on non-zero exit codes and cleans up the spawned process. The logic is sound.
Consider capturing the exit code in the error message for better debugging:
await new Promise<void>((res, rej) => { task.on('exit', (code: number) => { if (code !== 0) { - rej(); + rej(code); } else { res(); } }); -}).catch(() => { +}).catch((code) => { task.kill(); - throw new Error('Failed to install dependencies'); + throw new Error(`Failed to install dependencies (exit code: ${code || 'unknown'})`); });
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
scripts/tasks/install.ts(2 hunks)scripts/utils/cli-utils.ts(1 hunks)
🧰 Additional context used
📓 Path-based instructions (3)
**/*.{ts,tsx,js,jsx,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use camelCase for variable and function names
Files:
scripts/tasks/install.tsscripts/utils/cli-utils.ts
**/*.{ts,tsx}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx}: Enable TypeScript strict mode
Export functions from modules for testing purposes
Files:
scripts/tasks/install.tsscripts/utils/cli-utils.ts
**/*.{ts,tsx,js,jsx,json,html,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx,js,jsx,json,html,mjs}: Use ESLint and Prettier for code style enforcement
Run 'yarn prettier --write ' to format code after making changes
Run 'yarn lint:js:cmd ' to check for ESLint issues after making changes
Files:
scripts/tasks/install.tsscripts/utils/cli-utils.ts
🧠 Learnings (3)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Run 'yarn task --task check' to verify TypeScript types across all packages before submitting PRs
Applied to files:
scripts/tasks/install.tsscripts/utils/cli-utils.ts
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Compile packages with 'yarn task --task compile' after making code changes before testing
Applied to files:
scripts/utils/cli-utils.ts
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: daily
- GitHub Check: Core Unit Tests, windows-latest
- GitHub Check: nx
🔇 Additional comments (3)
scripts/utils/cli-utils.ts (1)
5-6: LGTM: Centralized root directory import.The import of
ROOT_DIRECTORYaligns well with the PR's objective to refactor the monorepo root from./codeto the git root. This simplifies the dependency management logic.scripts/tasks/install.ts (2)
1-1: Import cleanup fromnode:fs/promiseslooks goodOnly
accessis imported and used, which keeps the dependency surface minimal and matches thepathExistshelper’s needs. No issues here.
7-7: UsingROOT_DIRECTORYcentralizes monorepo root resolutionImporting and using
ROOT_DIRECTORYhere is consistent with the PR’s move to the git root as the monorepo root, and helps avoid ad‑hoccodeDirhandling scattered across tasks.
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
scripts/ecosystem-ci/before-test.js(1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
**/*.{ts,tsx,js,jsx,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use camelCase for variable and function names
Files:
scripts/ecosystem-ci/before-test.js
**/*.{ts,tsx,js,jsx,json,html,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx,js,jsx,json,html,mjs}: Use ESLint and Prettier for code style enforcement
Run 'yarn prettier --write ' to format code after making changes
Run 'yarn lint:js:cmd ' to check for ESLint issues after making changes
Files:
scripts/ecosystem-ci/before-test.js
🧠 Learnings (4)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
📚 Learning: 2025-10-01T15:24:01.060Z
Learnt from: Sidnioulz
Repo: storybookjs/storybook PR: 32594
File: code/core/src/components/components/Popover/WithPopover.tsx:7-9
Timestamp: 2025-10-01T15:24:01.060Z
Learning: In the Storybook repository, "react-aria-components/patched-dist/*" (e.g., "react-aria-components/patched-dist/Dialog", "react-aria-components/patched-dist/Popover", "react-aria-components/patched-dist/Tooltip") are valid import paths created by a patch applied to the react-aria-components package. These imports should not be flagged as broken or invalid until a maintainer explicitly states they are no longer acceptable.
Applied to files:
scripts/ecosystem-ci/before-test.js
📚 Learning: 2025-10-02T09:22:13.215Z
Learnt from: JReinhold
Repo: storybookjs/storybook PR: 32607
File: code/package.json:243-243
Timestamp: 2025-10-02T09:22:13.215Z
Learning: The Storybook repository uses Yarn v^4 (any 4.x version) as the package manager, configured via .yarnrc.yml and package.json packageManager field. Specific patch versions within v4 can be upgraded as needed.
Applied to files:
scripts/ecosystem-ci/before-test.js
📚 Learning: 2025-09-17T07:31:04.432Z
Learnt from: ndelangen
Repo: storybookjs/storybook PR: 32484
File: code/core/package.json:326-326
Timestamp: 2025-09-17T07:31:04.432Z
Learning: In Storybook's core package, dependencies like `open` are bundled into the final distribution during the build process, so they should remain in devDependencies rather than being moved to dependencies. End users don't need these packages as separate runtime dependencies since they're included in the bundled code.
Applied to files:
scripts/ecosystem-ci/before-test.js
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: daily
- GitHub Check: nx
- GitHub Check: Core Unit Tests, windows-latest
🔇 Additional comments (2)
scripts/ecosystem-ci/before-test.js (2)
22-22: LGTM: Path correctly updated to new sandbox location.The path change from
../../sandbox/to../../../storybook-sandboxes/aligns with the PR objective of relocating sandboxes outside the monorepo root.
28-32: LGTM: Filtering logic correctly extracts patch resolutions.The filtering approach appropriately isolates patch protocol resolutions using
includes('patch:')and safely handles the case when no resolutions exist.
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
scripts/ecosystem-ci/before-test.js (1)
28-37: Resolutions filtering/merge logic is correct; consider tightening patch detectionThe new logic correctly:
- Derives a filtered
resolutionsobject from the root package.json, excluding entries whose value containspatch:.- Merges those into
sandboxPackageJson.resolutionsusing spread, fixing the previous nestedresolutions.resolutionsstructure and ensuring root resolutions override sandbox ones.If you want to make this a bit more robust and explicit about patch handling, you could additionally guard against non-string values and only treat real Yarn patch protocol entries as patches:
-const resolutions = rootPackageJson.resolutions - ? Object.fromEntries( - Object.entries(rootPackageJson.resolutions).filter(([_, v]) => !v.includes('patch:')) - ) - : {}; +const resolutions = rootPackageJson.resolutions + ? Object.fromEntries( + Object.entries(rootPackageJson.resolutions).filter( + ([, v]) => typeof v === 'string' && !v.startsWith('patch:') + ) + ) + : {};This is optional, but it makes the intent clearer and guards against any unexpected value shapes in
rootPackageJson.resolutions.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
scripts/ecosystem-ci/before-test.js(1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
**/*.{ts,tsx,js,jsx,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use camelCase for variable and function names
Files:
scripts/ecosystem-ci/before-test.js
**/*.{ts,tsx,js,jsx,json,html,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx,js,jsx,json,html,mjs}: Use ESLint and Prettier for code style enforcement
Run 'yarn prettier --write ' to format code after making changes
Run 'yarn lint:js:cmd ' to check for ESLint issues after making changes
Files:
scripts/ecosystem-ci/before-test.js
🧠 Learnings (2)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
📚 Learning: 2025-09-17T07:31:04.432Z
Learnt from: ndelangen
Repo: storybookjs/storybook PR: 32484
File: code/core/package.json:326-326
Timestamp: 2025-09-17T07:31:04.432Z
Learning: In Storybook's core package, dependencies like `open` are bundled into the final distribution during the build process, so they should remain in devDependencies rather than being moved to dependencies. End users don't need these packages as separate runtime dependencies since they're included in the bundled code.
Applied to files:
scripts/ecosystem-ci/before-test.js
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: daily
- GitHub Check: nx
- GitHub Check: Core Unit Tests, windows-latest
🔇 Additional comments (1)
scripts/ecosystem-ci/before-test.js (1)
20-23: Sandbox path update aligns with new SANDBOX_ROOT layoutThe updated
sandboxPackageJsonPathcorrectly climbs three levels and targets../storybook-sandboxes, matching the new default sandbox root while preserving thesandbox.replace('/', '-')naming convention. This looks consistent with the PR’s directory scheme.
kasperpeulen
force-pushed
the
kasper/yarn-monorepo-refactor
branch
from
c16da0e to
250127b
Compare
There was a problem hiding this comment.
Actionable comments posted: 3
♻️ Duplicate comments (9)
.nvmrc (1)
1-1: Verify documentation consistency across the repository after Node.js version bump.The
.nvmrcfile correctly pins Node.js to 22.21.1 (latest stable 22.x, released 2025-10-28). However, based on learnings,.github/copilot-instructions.mdstill references the outdated version 22.16.0. A past review comment flagged this same issue, and it remains unresolved.Ensure all references to Node.js version are updated consistently across documentation, CI workflows, Docker configurations, and developer guidelines to reflect the new version.
Run the following verification script to identify all references to the old version:
#!/bin/bash # Search for Node.js version references across the repository echo "=== Searching for Node.js 22.16.0 references ===" rg -n '22\.16\.0' --type yaml --type json --type md --type txt echo "" echo "=== Searching for Node.js version references in CI configs ===" rg -n 'node.*22\.' .github/ --type yaml -A 2 -B 2code/lib/create-storybook/project.json (1)
4-8: Consistent with broader project.json refactoring.This file follows the same pattern observed in other library projects (removing
$schema, using empty compile/check target blocks, addingci:normaltags). Confirm NX executor defaults support this minimal configuration.code/frameworks/nextjs/project.json (1)
4-8: Consistent framework project.json refactoring.Follows the established pattern. Ensure NX executor defaults support empty target configurations as described in prior reviews.
code/addons/vitest/project.json (1)
4-8: Consistent addon project.json refactoring.Follows the same pattern as other library and framework projects. Verify NX executor configuration as noted in prior reviews.
code/builders/builder-vite/project.json (1)
4-8: Consistent builder project.json refactoring.Follows the same pattern as frameworks, addons, and libraries. Ensure NX executor defaults support the empty target configuration.
code/frameworks/react-vite/project.json (1)
4-8: Consistent framework project.json refactoring.Follows the established pattern across all frameworks. Verify NX executor configuration as noted in prior reviews.
code/renderers/svelte/project.json (1)
12-17:dependsOnwithprojects: ["*"]is non-standard and not documented by Nx.According to Nx documentation, the
projectsfield independsOnaccepts only:
"{dependencies}"— all dependency projects"self"— the same project- An explicit array of project names (e.g.,
["proj1", "proj2"], not["*"])The wildcard syntax
projects: ["*"]is not documented and may result in undefined behavior. Use one of the documented patterns instead:
"dependsOn": ["compile"](same-project target)"dependsOn": ["^compile"](dependency projects' compile target)"dependsOn": [{ "projects": "{dependencies}", "target": "compile" }](explicit object form)code/renderers/vue3/project.json (1)
12-17:dependsOnwithprojects: ["*"]remains non-standard.This issue was previously flagged. According to Nx documentation, the wildcard
projects: ["*"]is not a valid value. Use one of the documented patterns:
"dependsOn": ["compile"](same-project target)"dependsOn": ["^compile"](dependency projects' compile target)"dependsOn": [{ "projects": "{dependencies}", "target": "compile" }](explicit object form)nx.json (1)
57-64:{options.dir}placeholder is not a valid Nx token—caching will fail.This issue was previously flagged: Nx only recognizes
{projectRoot},{workspaceRoot}, and{projectName}as built-in output placeholders. The{options.dir}token at lines 62 and 83 cannot be resolved intargetDefaultsoutputs.Replace with
{projectName}to match the command patterns:- "outputs": ["{workspaceRoot}/sandbox/{options.dir}"], + "outputs": ["{workspaceRoot}/sandbox/{projectName}"],- "outputs": ["{workspaceRoot}/sandbox/{options.dir}/storybook-static"], + "outputs": ["{workspaceRoot}/sandbox/{projectName}/storybook-static"],Note: Project names like
react-vite/default-tscontain/, but the sandbox directories use-(e.g.,react-vite-default-ts). You may need a custom approach or ensure directory naming is consistent.Also applies to: 78-85
🧹 Nitpick comments (16)
scripts/get-template.ts (1)
302-308: Consider validating environment variable conversion.While the unary plus operator converts to numbers, invalid values will produce
NaN, which could lead to undefined behavior when used as an array index (line 55) or in comparisons (line 43).Consider adding validation:
const { CIRCLE_NODE_INDEX = 0, CIRCLE_NODE_TOTAL = 1 } = process.env; + const index = Number(CIRCLE_NODE_INDEX); + const total = Number(CIRCLE_NODE_TOTAL); + if (!Number.isFinite(index) || !Number.isFinite(total) || index < 0 || total < 1) { + throw new Error(`Invalid CircleCI node configuration: index=${CIRCLE_NODE_INDEX}, total=${CIRCLE_NODE_TOTAL}`); + } console.log( await getTemplate(cadence as Cadence, task, { - // Convert to integer - index: +CIRCLE_NODE_INDEX, - total: +CIRCLE_NODE_TOTAL, + index, + total, }) );code/sandbox/internal-react16-webpack/project.json (1)
1-26: Schema removal may impact IDE support and validation.The root
$schemaproperty was removed from this file. While this may be intentional as part of the monorepo restructuring, ensure that NX project validation is still properly configured at the workspace level or via other means. IDE schema hints for this file may be lost.Please verify:
- Whether a workspace-level NX schema is configured to validate all project.json files
- Whether the schema removal was intentional or an oversight
If the schema is still needed for this file, consider restoring it or ensuring workspace-level validation covers it.
scripts/task.ts (2)
152-157: Consider validating the dir option value.The
diroption allows custom sandbox directory names but lacks validation. Consider adding validation to ensure it doesn't contain path separators (/,\) or other characters that could cause issues when constructing paths.Example validation could be added in the options definition:
dir: { type: 'string', description: 'Name of sandbox directory', required: false, promptType: false, validate: (value: string) => { if (value && (value.includes('/') || value.includes('\\'))) { throw new Error('Directory name cannot contain path separators'); } }, },
439-443: Redundant optional chaining on array method.The second optional chaining operator on
forEachis unnecessary becausefilteralways returns an array, neverundefined.Apply this diff:
tasksThatDepend .get(task) ?.filter((t) => !t.service) - ?.forEach(setUnready); + .forEach(setUnready);Additionally, the first
?.operator appears defensive. IngetTaskList(lines 257-278), every task added tosortedTasksreceives an entry intasksThatDepend. Ifget(task)returnsundefined, it indicates a bug in the task graph construction rather than an expected edge case.scripts/event-log-checker.ts (1)
115-117: Based on my verification of the code, I can now provide the rewritten review comment:Assertion could be more precise for clarity, though current implementation is functionally safe.
The assertion
assert.equal(mainEvent.context.anonymousId, undefined)works correctly becausegetAnonymousProjectId()returnsundefinedwhen the git command fails (which occurs in non-git directories). However, usingassert.ok(!('anonymousId' in mainEvent.context))would be more explicit about the intent—verifying the property is absent rather than merely checking its value.The behavioral change is correct and safe:
getAnonymousProjectId()silently catches git failures and returnsundefined, which the telemetry code handles properly. The absence of a complementary test for whenanonymousIdis present (in a git directory) appears intentional, given the commented note at line 121 that CI doesn't provide this value.Apply this diff for a more precise assertion:
test(`main event should not contain anonymousId because it is not a git directory`, () => { - assert.equal(mainEvent.context.anonymousId, undefined); + assert.ok(!('anonymousId' in mainEvent.context)); });.env (1)
1-3: Fix environment variable ordering and add trailing newline.The static analysis tool (dotenv-linter) detected two formatting issues:
- Environment variables are not in alphabetical order (dotenv convention).
- Missing blank line at end of file.
Apply this diff to fix the formatting:
-STORYBOOK_DISABLE_TELEMETRY=true -IN_STORYBOOK_SANDBOX=true -SANDBOX_ROOT=../storybook-sandboxes +IN_STORYBOOK_SANDBOX=true +SANDBOX_ROOT=../storybook-sandboxes +STORYBOOK_DISABLE_TELEMETRY=trueCONTRIBUTING.md (1)
74-97: Docs mostly align with new yarn root; consider two small tweaks
- The updated tree comment on Line 119 (“Root of the yarn monorepo”) plus the existing
code/entry together now describe the new setup well (root as Yarn workspace,code/as main source tree). That looks correct with this PR’s goals.- The new two-step snippet (
yarnthenyarn start) on Lines 151–155 is clearer and better matches Yarn 4 behavior than relying onyarn startto install dependencies implicitly. This is a good change.- Minor shell nit: on Line 177 the snippet
cd code yarn build --watch ...will fail in most shells becausecdis given extra args. Consider splitting into two lines or usingcd code && yarn build --watch ...for copy‑paste correctness.Also applies to: 119-119, 151-155, 176-178
scripts/build/build-package.ts (1)
15-41: parseArgs andcwdhandling are solid; watch for call‑site expectationsSwitching to
util.parseArgswith explicitprod/production/optimized/watchbooleans is a nice cleanup, andallowNegative: truecorrectly enables--no-watchsemantics.(nodejs.org) A couple of small follow‑ups to consider:
parseArgsdefaults tostrict: trueandallowPositionals: false, so any existing invocations that relied on positional args (e.g. a bare package name) or unknown flags will now throw. It’s worth sanity‑checking thescripts/task.ts/ Nx call sites to ensure they only pass the options defined here, or explicitly enablingallowPositionalsif you still want to tolerate them.DIR_CWD = cwd ? join(ROOT_DIRECTORY, cwd) : process.cwd();assumescwdis always relative to the repo root. If a caller ever passes an absolute path,joinwill mangle it. A small robustness tweak is to treat absolute paths as‑is (e.g., usingisAbsolute(cwd)).Also applies to: 45-51
scripts/build-package.ts (1)
14-18: Consider using consistent import style for path modules.Line 14 uses the modern
node:pathprefix while line 18 uses the legacypathimport. For consistency and clarity, consider using thenode:prefix for all Node.js built-in modules.Apply this diff to make the imports consistent:
-import { resolve } from 'path'; +import { resolve } from 'node:path';.nx/workflows/agents.yaml (2)
3-26: Consider extracting shared init steps to reduce duplication.The
linux-init-steps(lines 3-26) andlinux-browsers-init-steps(lines 28-70) share identical Checkout, Restore Node Modules Cache, Install Node, and Install Node Modules steps. This duplication increases maintenance burden—any change to cache paths or install commands must be made in both places.YAML anchors can be nested or composed to reduce this. However, this is a nice-to-have improvement and the current structure works correctly.
Also applies to: 28-70
91-95: Windows template lacks init-steps (TODO).The
windows-jslaunch template has a placeholder TODO for init-steps. Ensure this is tracked and completed before relying on Windows agents in Nx Cloud workflows.Do you want me to open an issue to track the Windows init-steps implementation?
scripts/create-nx-sandbox-projects.ts (1)
86-92: Consider removing or conditionalizing debug console.log statements.Multiple
console.logstatements (lines 86, 91-92, 99) output debugging information during execution. For production scripts, consider either removing these or gating them behind a--debugflag to reduce noise in CI output.Also applies to: 99-99
code/package.json (1)
46-48: Confirm NxincludedScriptsconfigurationAdding the
nxblock with"includedScripts": []is valid, but in Nx this typically disables auto-discovery of package.json scripts for this project. If the intent is for Nx to consider some of these scripts as targets (for invalidation, graph display, etc.), you may want to list them explicitly instead of leaving the array empty.scripts/check/utils/typescript.ts (1)
29-43: Making TS config resolution cwd-awareAdding the optional
cwdparameter and threading it through as the base path andoutDirmakes this utility much more robust when called from different directories, while the default still preserves previous behavior. One small improvement you might consider (not required for this PR) is to givegetTSFilesAndConfigan explicit return type oftypescript.ParsedCommandLineto make downstream usage clearer.scripts/run-registry.ts (2)
40-62: Deduplicate port‑cleanup logic for 6001 and 6002The kill + retry loops for ports 6001 and 6002 are identical. Functionally this is fine, but it’s an easy place to DRY up and centralize the retry behavior.
You could factor to something like:
-const isPortUsed = async (port: number) => (await detectFreePort(port)) !== port; +const isPortUsed = async (port: number) => (await detectFreePort(port)) !== port; const sleep = (ms: number) => new Promise((resolve) => setTimeout(resolve, ms)); + +const ensurePortFree = async (port: number) => { + if (!(await isPortUsed(port))) { + return; + } + + await killProcessOnPort(port); + + let attempts = 0; + while ((await isPortUsed(port)) && attempts < 10) { + await sleep(1000); + attempts += 1; + } +}; @@ - // Kill Verdaccio related processes if they are already running - if (await isPortUsed(6001)) { - await killProcessOnPort(6001); - - let attempts = 0; - while ((await isPortUsed(6001)) && attempts < 10) { - await sleep(1000); - attempts++; - } - } - if (await isPortUsed(6002)) { - await killProcessOnPort(6002); - let attempts = 0; - while ((await isPortUsed(6002)) && attempts < 10) { - await sleep(1000); - attempts++; - } - } + // Kill Verdaccio related processes if they are already running + await ensurePortFree(6001); + await ensurePortFree(6002);This keeps the behavior identical but makes future tuning of the timeout/backoff logic much simpler.
244-249: Ensure.npmrcis cleaned up on successful non‑publish runs
npmAuthalways writes a.npmrcinroot, but on the success path whereopts.publish === falseand!opts.open, the script exits without removing it. That can leave the user’s npm config pointing athttp://localhost:6002even though Verdaccio has been shut down.You can fix this by moving the
.npmrcremoval into the non‑open termination path and letting the error handler continue to handle failures:- if (opts.publish) { - try { - await publish(packages, 'http://localhost:6002'); - } finally { - await rm(join(root, '.npmrc'), { force: true }); - } - } - - if (!opts.open) { - await servers?.close(); - process.exit(0); - } + if (opts.publish) { + await publish(packages, 'http://localhost:6002'); + } + + if (!opts.open) { + await servers?.close(); + await rm(join(root, '.npmrc'), { force: true }); + process.exit(0); + } @@ run().catch(async (e) => { logger.error(e); await servers?.close(); await rm(join(root, '.npmrc'), { force: true }); process.exit(1); });This keeps behavior for the existing publish flows but also cleans up
.npmrcon a successful “start registry, don’t publish, then exit” run.Also applies to: 253-254, 258-262
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (11)
.yarn/patches/@testing-library-user-event-npm-14.6.1-5da7e1d4e2.patchis excluded by!**/.yarn/**.yarn/patches/@types-babel__traverse-npm-7.20.6-fac4243243.patchis excluded by!**/.yarn/**.yarn/patches/@vitest-expect-npm-3.2.4-97c526d5cc.patchis excluded by!**/.yarn/**.yarn/patches/react-aria-components-npm-1.12.2-6c5dcdafab.patchis excluded by!**/.yarn/**.yarn/plugins/@yarnpkg/plugin-interactive-tools.cjsis excluded by!**/.yarn/**.yarn/plugins/@yarnpkg/plugin-typescript.cjsis excluded by!**/.yarn/**code/yarn.lockis excluded by!**/yarn.lock,!**/*.locknode_modules/.package-lock.jsonis excluded by!**/node_modules/**node_modules/.yarn-state.ymlis excluded by!**/node_modules/**scripts/yarn.lockis excluded by!**/yarn.lock,!**/*.lockyarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (207)
.circleci/config.yml(14 hunks).circleci/src/jobs/bench-sandboxes.yml(1 hunks).circleci/src/jobs/build.yml(2 hunks).circleci/src/jobs/check-sandboxes.yml(1 hunks).circleci/src/jobs/check.yml(1 hunks).circleci/src/jobs/chromatic-sandboxes.yml(1 hunks).circleci/src/jobs/create-sandboxes.yml(1 hunks).circleci/src/jobs/e2e-dev.yml(1 hunks).circleci/src/jobs/e2e-production.yml(1 hunks).circleci/src/jobs/pretty-docs.yml(1 hunks).circleci/src/jobs/test-runner-production.yml(1 hunks).circleci/src/jobs/vitest-integration.yml(1 hunks).cursor/environment.json(1 hunks).env(1 hunks).github/actions/setup-node-and-install/action.yml(1 hunks).github/copilot-instructions.md(5 hunks).github/workflows/nx.yml(1 hunks).gitignore(2 hunks).ignore(0 hunks).nvmrc(1 hunks).nx/workflows/agents.yaml(1 hunks).nx/workflows/distribution-config.yaml(1 hunks).yarnrc.yml(1 hunks)CONTRIBUTING.md(3 hunks)CONTRIBUTING/RELEASING.md(1 hunks)code/.env(0 hunks)code/.nxignore(0 hunks)code/.yarnrc.yml(0 hunks)code/addons/a11y/package.json(0 hunks)code/addons/a11y/project.json(1 hunks)code/addons/docs/package.json(0 hunks)code/addons/docs/project.json(1 hunks)code/addons/links/package.json(0 hunks)code/addons/links/project.json(1 hunks)code/addons/onboarding/package.json(0 hunks)code/addons/onboarding/project.json(1 hunks)code/addons/pseudo-states/package.json(0 hunks)code/addons/pseudo-states/project.json(1 hunks)code/addons/themes/package.json(0 hunks)code/addons/themes/project.json(1 hunks)code/addons/vitest/package.json(0 hunks)code/addons/vitest/project.json(1 hunks)code/builders/builder-vite/package.json(0 hunks)code/builders/builder-vite/project.json(1 hunks)code/builders/builder-webpack5/package.json(0 hunks)code/builders/builder-webpack5/project.json(1 hunks)code/core/package.json(1 hunks)code/core/project.json(1 hunks)code/e2e-tests/storybook-hooks.spec.ts(2 hunks)code/frameworks/angular/package.json(0 hunks)code/frameworks/angular/project.json(1 hunks)code/frameworks/ember/package.json(0 hunks)code/frameworks/ember/project.json(1 hunks)code/frameworks/html-vite/package.json(0 hunks)code/frameworks/html-vite/project.json(1 hunks)code/frameworks/nextjs-vite/package.json(0 hunks)code/frameworks/nextjs-vite/project.json(1 hunks)code/frameworks/nextjs/package.json(0 hunks)code/frameworks/nextjs/project.json(1 hunks)code/frameworks/nextjs/src/next-image-loader-stub.ts(1 hunks)code/frameworks/preact-vite/package.json(0 hunks)code/frameworks/preact-vite/project.json(1 hunks)code/frameworks/react-native-web-vite/package.json(0 hunks)code/frameworks/react-native-web-vite/project.json(1 hunks)code/frameworks/react-vite/package.json(0 hunks)code/frameworks/react-vite/project.json(1 hunks)code/frameworks/react-webpack5/package.json(0 hunks)code/frameworks/react-webpack5/project.json(1 hunks)code/frameworks/server-webpack5/package.json(0 hunks)code/frameworks/server-webpack5/project.json(1 hunks)code/frameworks/svelte-vite/package.json(0 hunks)code/frameworks/svelte-vite/project.json(1 hunks)code/frameworks/sveltekit/package.json(0 hunks)code/frameworks/sveltekit/project.json(1 hunks)code/frameworks/vue3-vite/package.json(0 hunks)code/frameworks/vue3-vite/project.json(1 hunks)code/frameworks/web-components-vite/package.json(0 hunks)code/frameworks/web-components-vite/project.json(1 hunks)code/lib/cli-sb/project.json(0 hunks)code/lib/cli-storybook/package.json(0 hunks)code/lib/cli-storybook/project.json(1 hunks)code/lib/cli-storybook/src/link.ts(1 hunks)code/lib/codemod/package.json(0 hunks)code/lib/codemod/project.json(1 hunks)code/lib/core-webpack/package.json(0 hunks)code/lib/core-webpack/project.json(1 hunks)code/lib/create-storybook/package.json(0 hunks)code/lib/create-storybook/project.json(1 hunks)code/lib/csf-plugin/package.json(0 hunks)code/lib/csf-plugin/project.json(1 hunks)code/lib/eslint-plugin/package.json(0 hunks)code/lib/eslint-plugin/project.json(1 hunks)code/lib/react-dom-shim/package.json(0 hunks)code/lib/react-dom-shim/project.json(1 hunks)code/nx.json(0 hunks)code/package.json(4 hunks)code/presets/create-react-app/package.json(0 hunks)code/presets/create-react-app/project.json(1 hunks)code/presets/react-webpack/package.json(0 hunks)code/presets/react-webpack/project.json(1 hunks)code/presets/server-webpack/package.json(0 hunks)code/presets/server-webpack/project.json(1 hunks)code/project.json(1 hunks)code/renderers/html/package.json(0 hunks)code/renderers/html/project.json(1 hunks)code/renderers/preact/package.json(0 hunks)code/renderers/preact/project.json(1 hunks)code/renderers/react/package.json(0 hunks)code/renderers/react/project.json(1 hunks)code/renderers/react/src/entry-preview.tsx(0 hunks)code/renderers/server/package.json(0 hunks)code/renderers/server/project.json(1 hunks)code/renderers/svelte/package.json(0 hunks)code/renderers/svelte/project.json(1 hunks)code/renderers/vue3/package.json(0 hunks)code/renderers/vue3/project.json(1 hunks)code/renderers/vue3/template/stories_vue3-vite-default-ts/ReactiveArgs.stories.ts(0 hunks)code/renderers/vue3/template/stories_vue3-vite-default-ts/ScopedSlots.stories.ts(1 hunks)code/renderers/web-components/package.json(0 hunks)code/renderers/web-components/project.json(1 hunks)code/renderers/web-components/src/docs/web-components-properties.test.ts(1 hunks)code/sandbox/angular-cli-default-ts/project.json(1 hunks)code/sandbox/angular-cli-prerelease/project.json(1 hunks)code/sandbox/bench-react-vite-default-ts-nodocs/project.json(1 hunks)code/sandbox/bench-react-vite-default-ts-test-build/project.json(1 hunks)code/sandbox/bench-react-vite-default-ts/project.json(1 hunks)code/sandbox/bench-react-webpack-18-ts-test-build/project.json(1 hunks)code/sandbox/bench-react-webpack-18-ts/project.json(1 hunks)code/sandbox/cra-default-js/project.json(1 hunks)code/sandbox/cra-default-ts/project.json(1 hunks)code/sandbox/ember-3-js/project.json(1 hunks)code/sandbox/ember-default-js/project.json(1 hunks)code/sandbox/experimental-nextjs-vite-default-ts/project.json(0 hunks)code/sandbox/html-rsbuild-default-ts/project.json(1 hunks)code/sandbox/html-vite-default-js/project.json(1 hunks)code/sandbox/html-vite-default-ts/project.json(1 hunks)code/sandbox/internal-react16-webpack/project.json(1 hunks)code/sandbox/internal-react18-webpack-babel/project.json(1 hunks)code/sandbox/internal-server-webpack5/project.json(1 hunks)code/sandbox/lit-rsbuild-default-ts/project.json(1 hunks)code/sandbox/lit-vite-default-js/project.json(1 hunks)code/sandbox/lit-vite-default-ts/project.json(1 hunks)code/sandbox/nextjs-14-ts/project.json(1 hunks)code/sandbox/nextjs-15-ts/project.json(1 hunks)code/sandbox/nextjs-default-js/project.json(0 hunks)code/sandbox/nextjs-default-ts/project.json(1 hunks)code/sandbox/nextjs-prerelease/project.json(1 hunks)code/sandbox/nextjs-vite-14-ts/project.json(1 hunks)code/sandbox/nextjs-vite-15-ts/project.json(1 hunks)code/sandbox/nextjs-vite-default-ts/project.json(1 hunks)code/sandbox/nuxt-vite-default-ts/project.json(0 hunks)code/sandbox/preact-vite-default-js/project.json(1 hunks)code/sandbox/preact-vite-default-ts/project.json(1 hunks)code/sandbox/qwik-vite-default-ts/project.json(1 hunks)code/sandbox/react-native-web-vite-expo-ts/project.json(1 hunks)code/sandbox/react-native-web-vite-rn-cli-ts/project.json(1 hunks)code/sandbox/react-rsbuild-default-ts/project.json(1 hunks)code/sandbox/react-vite-default-js/project.json(1 hunks)code/sandbox/react-vite-default-ts/project.json(1 hunks)code/sandbox/react-vite-prerelease-ts/project.json(1 hunks)code/sandbox/react-webpack-17-ts/project.json(1 hunks)code/sandbox/react-webpack-18-ts/project.json(1 hunks)code/sandbox/react-webpack-prerelease-ts/project.json(1 hunks)code/sandbox/solid-vite-default-js/project.json(1 hunks)code/sandbox/solid-vite-default-ts/project.json(1 hunks)code/sandbox/svelte-kit-default-ts/project.json(0 hunks)code/sandbox/svelte-kit-skeleton-js/project.json(0 hunks)code/sandbox/svelte-kit-skeleton-ts/project.json(1 hunks)code/sandbox/svelte-vite-default-js/project.json(1 hunks)code/sandbox/svelte-vite-default-ts/project.json(1 hunks)code/sandbox/vue3-rsbuild-default-ts/project.json(1 hunks)code/sandbox/vue3-vite-default-js/project.json(1 hunks)code/sandbox/vue3-vite-default-ts/project.json(1 hunks)docs/_snippets/addon-consume-and-update-globaltype.md(1 hunks)docs/_snippets/main-config-builder-custom-config.md(0 hunks)docs/_snippets/storybook-addon-css-example.md(1 hunks)docs/_snippets/storybook-addon-toolkit-types.md(1 hunks)docs/_snippets/storybook-addons-api-getchannel.md(1 hunks)docs/_snippets/storybook-preview-use-global-type.md(1 hunks)nx.json(1 hunks)package.json(1 hunks)scripts/.yarnrc.yml(0 hunks)scripts/build-package.ts(2 hunks)scripts/build/build-package.ts(2 hunks)scripts/build/utils/generate-bundle.ts(1 hunks)scripts/build/utils/generate-type-mappers.ts(1 hunks)scripts/build/utils/generate-types.ts(2 hunks)scripts/build/utils/modify-core-theme-types.ts(2 hunks)scripts/check-package.ts(1 hunks)scripts/check/check-package.ts(1 hunks)scripts/check/utils/typescript.ts(2 hunks)scripts/create-nx-sandbox-projects.ts(1 hunks)scripts/ecosystem-ci/before-test.js(1 hunks)scripts/event-log-checker.ts(1 hunks)scripts/get-sandbox-dir.ts(2 hunks)scripts/get-template.ts(3 hunks)scripts/knip.config.ts(1 hunks)scripts/package.json(2 hunks)scripts/prepare-sandbox.ts(1 hunks)scripts/project.json(1 hunks)scripts/release/__tests__/version.test.ts(1 hunks)scripts/release/version.ts(2 hunks)scripts/run-registry.ts(8 hunks)scripts/sandbox/utils/getPort.ts(1 hunks)scripts/task.ts(5 hunks)scripts/tasks/build.ts(3 hunks)scripts/tasks/check.ts(2 hunks)
⛔ Files not processed due to max files limit (32)
- scripts/tasks/chromatic.ts
- scripts/tasks/compile.ts
- scripts/tasks/dev.ts
- scripts/tasks/e2e-tests-build.ts
- scripts/tasks/install.ts
- scripts/tasks/run-registry.ts
- scripts/tasks/sandbox-parts.ts
- scripts/tasks/sandbox.ts
- scripts/tasks/serve.ts
- scripts/tasks/test-runner-build.ts
- scripts/upload-bench.ts
- scripts/utils/cli-utils.ts
- scripts/utils/constants.ts
- scripts/utils/kill-process-on-port.ts
- scripts/utils/main-js.ts
- scripts/utils/tools.ts
- scripts/utils/workspace.ts
- scripts/utils/yarn.ts
- test-storybooks/portable-stories-kitchen-sink/nextjs/package.json
- test-storybooks/portable-stories-kitchen-sink/nextjs/project.json
- test-storybooks/portable-stories-kitchen-sink/react-vitest-3/package.json
- test-storybooks/portable-stories-kitchen-sink/react-vitest-3/playwright-e2e.config.ts
- test-storybooks/portable-stories-kitchen-sink/react-vitest-3/project.json
- test-storybooks/portable-stories-kitchen-sink/react/package.json
- test-storybooks/portable-stories-kitchen-sink/react/playwright-e2e.config.ts
- test-storybooks/portable-stories-kitchen-sink/react/project.json
- test-storybooks/portable-stories-kitchen-sink/svelte/package.json
- test-storybooks/portable-stories-kitchen-sink/svelte/project.json
- test-storybooks/portable-stories-kitchen-sink/vue3/package.json
- test-storybooks/portable-stories-kitchen-sink/vue3/project.json
- test-storybooks/yarn-pnp/package.json
- test-storybooks/yarn-pnp/project.json
💤 Files with no reviewable changes (55)
- code/sandbox/svelte-kit-default-ts/project.json
- code/renderers/vue3/package.json
- code/addons/docs/package.json
- code/addons/vitest/package.json
- code/renderers/html/package.json
- code/lib/codemod/package.json
- code/frameworks/nextjs-vite/package.json
- code/sandbox/svelte-kit-skeleton-js/project.json
- code/lib/react-dom-shim/package.json
- code/frameworks/html-vite/package.json
- code/lib/create-storybook/package.json
- code/frameworks/ember/package.json
- code/frameworks/nextjs/package.json
- code/frameworks/react-vite/package.json
- code/frameworks/web-components-vite/package.json
- code/frameworks/react-native-web-vite/package.json
- code/renderers/web-components/package.json
- code/renderers/react/src/entry-preview.tsx
- code/lib/cli-sb/project.json
- code/frameworks/angular/package.json
- code/frameworks/vue3-vite/package.json
- code/renderers/svelte/package.json
- code/.env
- code/builders/builder-vite/package.json
- code/.yarnrc.yml
- scripts/.yarnrc.yml
- code/frameworks/server-webpack5/package.json
- .ignore
- code/frameworks/sveltekit/package.json
- code/lib/csf-plugin/package.json
- code/addons/themes/package.json
- code/renderers/vue3/template/stories_vue3-vite-default-ts/ReactiveArgs.stories.ts
- code/lib/cli-storybook/package.json
- code/frameworks/react-webpack5/package.json
- code/presets/create-react-app/package.json
- code/frameworks/svelte-vite/package.json
- code/renderers/react/package.json
- code/addons/pseudo-states/package.json
- code/addons/a11y/package.json
- code/lib/eslint-plugin/package.json
- code/addons/links/package.json
- code/sandbox/nextjs-default-js/project.json
- code/sandbox/nuxt-vite-default-ts/project.json
- code/addons/onboarding/package.json
- code/builders/builder-webpack5/package.json
- code/renderers/server/package.json
- code/.nxignore
- code/lib/core-webpack/package.json
- code/presets/server-webpack/package.json
- code/sandbox/experimental-nextjs-vite-default-ts/project.json
- code/frameworks/preact-vite/package.json
- docs/_snippets/main-config-builder-custom-config.md
- code/renderers/preact/package.json
- code/presets/react-webpack/package.json
- code/nx.json
✅ Files skipped from review due to trivial changes (1)
- docs/_snippets/storybook-preview-use-global-type.md
🚧 Files skipped from review as they are similar to previous changes (77)
- code/frameworks/sveltekit/project.json
- code/sandbox/nextjs-vite-14-ts/project.json
- code/sandbox/cra-default-ts/project.json
- code/renderers/html/project.json
- code/sandbox/html-vite-default-ts/project.json
- code/renderers/web-components/src/docs/web-components-properties.test.ts
- code/renderers/react/project.json
- .circleci/src/jobs/e2e-production.yml
- scripts/knip.config.ts
- scripts/build/utils/generate-types.ts
- .circleci/src/jobs/build.yml
- .circleci/src/jobs/chromatic-sandboxes.yml
- scripts/prepare-sandbox.ts
- .yarnrc.yml
- scripts/sandbox/utils/getPort.ts
- code/frameworks/preact-vite/project.json
- .circleci/src/jobs/test-runner-production.yml
- code/sandbox/react-native-web-vite-rn-cli-ts/project.json
- code/frameworks/angular/project.json
- .github/copilot-instructions.md
- code/sandbox/nextjs-vite-default-ts/project.json
- code/sandbox/nextjs-14-ts/project.json
- code/sandbox/nextjs-vite-15-ts/project.json
- code/addons/themes/project.json
- code/addons/docs/project.json
- code/renderers/server/project.json
- .circleci/src/jobs/bench-sandboxes.yml
- code/renderers/preact/project.json
- scripts/build/utils/generate-type-mappers.ts
- code/lib/cli-storybook/src/link.ts
- code/frameworks/react-webpack5/project.json
- code/sandbox/bench-react-vite-default-ts-nodocs/project.json
- code/addons/a11y/project.json
- scripts/release/tests/version.test.ts
- scripts/project.json
- scripts/build/utils/modify-core-theme-types.ts
- .circleci/src/jobs/e2e-dev.yml
- code/lib/react-dom-shim/project.json
- code/sandbox/react-webpack-17-ts/project.json
- scripts/check-package.ts
- .circleci/src/jobs/pretty-docs.yml
- code/frameworks/web-components-vite/project.json
- docs/_snippets/addon-consume-and-update-globaltype.md
- scripts/build/utils/generate-bundle.ts
- code/sandbox/preact-vite-default-js/project.json
- scripts/release/version.ts
- .circleci/src/jobs/check-sandboxes.yml
- code/builders/builder-webpack5/project.json
- code/sandbox/html-rsbuild-default-ts/project.json
- scripts/package.json
- code/sandbox/lit-rsbuild-default-ts/project.json
- code/sandbox/html-vite-default-js/project.json
- code/lib/csf-plugin/project.json
- .cursor/environment.json
- code/sandbox/qwik-vite-default-ts/project.json
- code/presets/react-webpack/project.json
- code/sandbox/react-vite-default-ts/project.json
- docs/_snippets/storybook-addon-css-example.md
- .circleci/src/jobs/create-sandboxes.yml
- code/sandbox/react-webpack-18-ts/project.json
- code/sandbox/solid-vite-default-js/project.json
- code/frameworks/html-vite/project.json
- code/frameworks/vue3-vite/project.json
- code/sandbox/svelte-kit-skeleton-ts/project.json
- code/sandbox/preact-vite-default-ts/project.json
- code/sandbox/react-native-web-vite-expo-ts/project.json
- code/renderers/vue3/template/stories_vue3-vite-default-ts/ScopedSlots.stories.ts
- .circleci/src/jobs/check.yml
- .nx/workflows/distribution-config.yaml
- docs/_snippets/storybook-addon-toolkit-types.md
- code/e2e-tests/storybook-hooks.spec.ts
- .github/workflows/nx.yml
- code/frameworks/svelte-vite/project.json
- code/sandbox/react-webpack-prerelease-ts/project.json
- code/sandbox/angular-cli-default-ts/project.json
- code/frameworks/react-native-web-vite/project.json
- code/sandbox/angular-cli-prerelease/project.json
🧰 Additional context used
📓 Path-based instructions (5)
**/*.{ts,tsx,js,jsx,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use camelCase for variable and function names
Files:
code/frameworks/nextjs/src/next-image-loader-stub.tsscripts/tasks/check.tsscripts/run-registry.tsscripts/check/check-package.tsscripts/get-sandbox-dir.tsscripts/event-log-checker.tsscripts/create-nx-sandbox-projects.tsscripts/task.tsscripts/build/build-package.tsscripts/check/utils/typescript.tsscripts/tasks/build.tsscripts/get-template.tsscripts/ecosystem-ci/before-test.jsscripts/build-package.ts
**/*.{ts,tsx}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx}: Enable TypeScript strict mode
Export functions from modules for testing purposes
Files:
code/frameworks/nextjs/src/next-image-loader-stub.tsscripts/tasks/check.tsscripts/run-registry.tsscripts/check/check-package.tsscripts/get-sandbox-dir.tsscripts/event-log-checker.tsscripts/create-nx-sandbox-projects.tsscripts/task.tsscripts/build/build-package.tsscripts/check/utils/typescript.tsscripts/tasks/build.tsscripts/get-template.tsscripts/build-package.ts
**/*.{ts,tsx,js,jsx,json,html,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx,js,jsx,json,html,mjs}: Use ESLint and Prettier for code style enforcement
Run 'yarn prettier --write ' to format code after making changes
Run 'yarn lint:js:cmd ' to check for ESLint issues after making changes
Files:
code/frameworks/nextjs/src/next-image-loader-stub.tsscripts/tasks/check.tscode/sandbox/svelte-vite-default-ts/project.jsoncode/addons/pseudo-states/project.jsoncode/frameworks/react-vite/project.jsoncode/sandbox/vue3-rsbuild-default-ts/project.jsoncode/sandbox/solid-vite-default-ts/project.jsoncode/lib/core-webpack/project.jsonscripts/run-registry.tscode/frameworks/nextjs/project.jsoncode/project.jsoncode/sandbox/vue3-vite-default-ts/project.jsoncode/sandbox/svelte-vite-default-js/project.jsoncode/presets/server-webpack/project.jsonscripts/check/check-package.tscode/sandbox/react-vite-prerelease-ts/project.jsoncode/sandbox/vue3-vite-default-js/project.jsoncode/sandbox/lit-vite-default-js/project.jsoncode/addons/onboarding/project.jsoncode/addons/vitest/project.jsoncode/lib/cli-storybook/project.jsoncode/core/project.jsoncode/builders/builder-vite/project.jsonscripts/get-sandbox-dir.tscode/sandbox/lit-vite-default-ts/project.jsoncode/sandbox/ember-default-js/project.jsoncode/lib/create-storybook/project.jsoncode/sandbox/react-rsbuild-default-ts/project.jsoncode/sandbox/bench-react-webpack-18-ts-test-build/project.jsonscripts/event-log-checker.tscode/sandbox/ember-3-js/project.jsonscripts/create-nx-sandbox-projects.tscode/sandbox/bench-react-webpack-18-ts/project.jsoncode/sandbox/internal-server-webpack5/project.jsoncode/renderers/vue3/project.jsoncode/addons/links/project.jsonscripts/task.tscode/frameworks/nextjs-vite/project.jsonscripts/build/build-package.tscode/presets/create-react-app/project.jsoncode/core/package.jsoncode/renderers/svelte/project.jsonscripts/check/utils/typescript.tsscripts/tasks/build.tsscripts/get-template.tsscripts/ecosystem-ci/before-test.jscode/sandbox/internal-react16-webpack/project.jsoncode/sandbox/bench-react-vite-default-ts-test-build/project.jsoncode/sandbox/cra-default-js/project.jsoncode/sandbox/nextjs-prerelease/project.jsoncode/sandbox/nextjs-15-ts/project.jsonnx.jsoncode/frameworks/ember/project.jsoncode/sandbox/internal-react18-webpack-babel/project.jsoncode/frameworks/server-webpack5/project.jsoncode/renderers/web-components/project.jsoncode/sandbox/react-vite-default-js/project.jsoncode/lib/eslint-plugin/project.jsonpackage.jsoncode/sandbox/nextjs-default-ts/project.jsonscripts/build-package.tscode/lib/codemod/project.jsoncode/sandbox/bench-react-vite-default-ts/project.jsoncode/package.json
code/**/!(*.test).{ts,tsx,js,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
code/**/!(*.test).{ts,tsx,js,mjs}: Use 'logger' from 'storybook/internal/node-logger' for server-side (Node.js) logging, not console.log/console.warn/console.error
Use 'logger' from 'storybook/internal/client-logger' for client-side (browser) logging, not console.log/console.warn/console.error
Do not use console.log, console.warn, or console.error directly unless in isolated files where importing loggers would significantly increase bundle size
Files:
code/frameworks/nextjs/src/next-image-loader-stub.ts
.nvmrc
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use Node.js 22.16.0 as specified in .nvmrc
Files:
.nvmrc
🧠 Learnings (37)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Learnt from: JReinhold
Repo: storybookjs/storybook PR: 32607
File: code/package.json:243-243
Timestamp: 2025-10-02T09:22:13.215Z
Learning: The Storybook repository uses Yarn v^4 (any 4.x version) as the package manager, configured via .yarnrc.yml and package.json packageManager field. Specific patch versions within v4 can be upgraded as needed.
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Run 'yarn task --task check' to verify TypeScript types across all packages before submitting PRs
Applied to files:
scripts/tasks/check.tsscripts/check/check-package.tsCONTRIBUTING/RELEASING.mdcode/renderers/svelte/project.jsonscripts/check/utils/typescript.ts.circleci/config.ymlscripts/build-package.ts
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Run 'yarn lint:js:cmd <file>' to check for ESLint issues after making changes
Applied to files:
scripts/tasks/check.tscode/project.jsonscripts/check/check-package.tsscripts/create-nx-sandbox-projects.tsscripts/check/utils/typescript.tspackage.json.gitignorecode/package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Run 'yarn prettier --write <file>' to format code after making changes
Applied to files:
scripts/tasks/check.tsCONTRIBUTING.mdcode/sandbox/vue3-rsbuild-default-ts/project.jsoncode/project.jsonscripts/create-nx-sandbox-projects.tscode/sandbox/nextjs-15-ts/project.json.gitignorecode/sandbox/nextjs-default-ts/project.jsonscripts/build-package.tscode/package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Compile packages with 'yarn task --task compile' after making code changes before testing
Applied to files:
scripts/tasks/check.tsCONTRIBUTING.mdcode/project.jsonCONTRIBUTING/RELEASING.md.circleci/config.ymlscripts/build-package.tscode/package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.test.{ts,tsx} : Use 'yarn vitest run --coverage <test-file>' to run tests with coverage reports and aim for 75%+ coverage of statements/lines
Applied to files:
scripts/tasks/check.tsCONTRIBUTING.mdcode/project.jsonpackage.json.gitignorecode/package.json
📚 Learning: 2025-11-24T17:49:31.838Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
Applied to files:
code/sandbox/svelte-vite-default-ts/project.jsonCONTRIBUTING.mdcode/sandbox/solid-vite-default-ts/project.jsoncode/sandbox/vue3-vite-default-ts/project.jsonscripts/check/check-package.tscode/addons/vitest/project.jsoncode/sandbox/lit-vite-default-ts/project.jsoncode/sandbox/bench-react-vite-default-ts-test-build/project.jsonnx.jsonpackage.jsoncode/sandbox/bench-react-vite-default-ts/project.jsoncode/package.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid mocking only a subset of required dependencies in Vitest tests
Applied to files:
code/sandbox/svelte-vite-default-ts/project.jsonCONTRIBUTING.mdcode/sandbox/solid-vite-default-ts/project.jsoncode/sandbox/vue3-vite-default-ts/project.jsoncode/sandbox/react-vite-prerelease-ts/project.jsoncode/sandbox/lit-vite-default-ts/project.jsonscripts/ecosystem-ci/before-test.jscode/sandbox/bench-react-vite-default-ts-test-build/project.jsoncode/sandbox/react-vite-default-js/project.jsonpackage.json.gitignorecode/sandbox/bench-react-vite-default-ts/project.jsoncode/package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to README.md : Update relevant README files for significant changes
Applied to files:
CONTRIBUTING.mdCONTRIBUTING/RELEASING.md.circleci/config.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Applied to files:
CONTRIBUTING.md.envcode/lib/create-storybook/project.jsonscripts/event-log-checker.tsscripts/create-nx-sandbox-projects.tscode/core/package.jsonscripts/tasks/build.tsscripts/get-template.ts.circleci/config.ymlpackage.json.gitignorescripts/build-package.tscode/package.json
📚 Learning: 2025-10-02T09:22:13.215Z
Learnt from: JReinhold
Repo: storybookjs/storybook PR: 32607
File: code/package.json:243-243
Timestamp: 2025-10-02T09:22:13.215Z
Learning: The Storybook repository uses Yarn v^4 (any 4.x version) as the package manager, configured via .yarnrc.yml and package.json packageManager field. Specific patch versions within v4 can be upgraded as needed.
Applied to files:
CONTRIBUTING.mdcode/core/package.jsonpackage.json.gitignorecode/package.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Document complex mock behaviors in Vitest tests
Applied to files:
CONTRIBUTING.mdcode/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Group related mocks together in Vitest tests
Applied to files:
CONTRIBUTING.md
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Keep mock implementations simple and focused in Vitest tests
Applied to files:
CONTRIBUTING.mdcode/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid inline mock implementations within test cases in Vitest tests
Applied to files:
CONTRIBUTING.mdcode/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Mock all required properties and methods that the test subject uses in Vitest tests
Applied to files:
CONTRIBUTING.mdcode/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Mock at the highest level of abstraction needed in Vitest tests
Applied to files:
CONTRIBUTING.md
📚 Learning: 2025-09-29T13:20:23.346Z
Learnt from: mrginglymus
Repo: storybookjs/storybook PR: 32556
File: code/core/package.json:309-313
Timestamp: 2025-09-29T13:20:23.346Z
Learning: The `fast-printf` dependency in Storybook's core package is bundled into the final distribution during the build process, so it should remain in devDependencies rather than being moved to dependencies, following the same pattern as other bundled dependencies like `open`.
Applied to files:
CONTRIBUTING.mdpackage.jsoncode/package.json
📚 Learning: 2025-09-17T07:31:04.432Z
Learnt from: ndelangen
Repo: storybookjs/storybook PR: 32484
File: code/core/package.json:326-326
Timestamp: 2025-09-17T07:31:04.432Z
Learning: In Storybook's core package, dependencies like `open` are bundled into the final distribution during the build process, so they should remain in devDependencies rather than being moved to dependencies. End users don't need these packages as separate runtime dependencies since they're included in the bundled code.
Applied to files:
CONTRIBUTING.mdcode/core/package.jsonscripts/ecosystem-ci/before-test.jspackage.jsoncode/package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'cd code && yarn storybook:ui' to start the development server for testing UI changes on http://localhost:6006/
Applied to files:
CONTRIBUTING.md.circleci/config.ymlcode/package.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid mocking without the `spy: true` option in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json.gitignore
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid mock implementations outside of `beforeEach` blocks in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid direct function mocking without `vi.mocked()` in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to .nvmrc : Use Node.js 22.16.0 as specified in .nvmrc
Applied to files:
.nvmrc
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx} : Enable TypeScript strict mode
Applied to files:
scripts/check/check-package.tsscripts/check/utils/typescript.ts
📚 Learning: 2025-09-18T20:51:06.618Z
Learnt from: Sidnioulz
Repo: storybookjs/storybook PR: 32458
File: code/core/src/viewport/components/Tool.tsx:38-39
Timestamp: 2025-09-18T20:51:06.618Z
Learning: The useGlobals hook from storybook/manager-api returns a tuple where the third element (storyGlobals) is typed as Globals, not Globals | undefined. This means TypeScript guarantees it's always defined, making the `in` operator safe to use without additional null checks.
Applied to files:
docs/_snippets/storybook-addons-api-getchannel.md
📚 Learning: 2025-09-18T20:51:06.618Z
Learnt from: Sidnioulz
Repo: storybookjs/storybook PR: 32458
File: code/core/src/viewport/components/Tool.tsx:38-39
Timestamp: 2025-09-18T20:51:06.618Z
Learning: In viewport tool code, when using the `useGlobals` hook from storybook/manager-api, the third returned value `storyGlobals` is guaranteed by TypeScript to be defined (not undefined/null), making the `in` operator safe to use without additional null checks.
Applied to files:
docs/_snippets/storybook-addons-api-getchannel.md
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to .yarnrc.yml : Use Yarn 4.9.1 as the package manager
Applied to files:
CONTRIBUTING/RELEASING.md.github/actions/setup-node-and-install/action.yml.gitignorecode/package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Work on feature branches following the Git workflow
Applied to files:
CONTRIBUTING/RELEASING.md
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Applied to files:
scripts/get-sandbox-dir.tscode/sandbox/bench-react-webpack-18-ts-test-build/project.jsonscripts/create-nx-sandbox-projects.tscode/sandbox/bench-react-webpack-18-ts/project.jsonscripts/task.tsscripts/tasks/build.tsscripts/get-template.tscode/sandbox/bench-react-vite-default-ts-test-build/project.json.circleci/config.ymlnx.json.circleci/src/jobs/vitest-integration.ymlcode/package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Use ESLint and Prettier for code style enforcement
Applied to files:
code/lib/eslint-plugin/project.json
📚 Learning: 2025-11-24T17:49:31.838Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to **/*.{test,spec}.{ts,tsx} : Test files should follow the naming pattern `*.test.ts`, `*.test.tsx`, `*.spec.ts`, or `*.spec.tsx`
Applied to files:
.gitignore
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to code/**/!(*.test).{ts,tsx,js,mjs} : Use 'logger' from 'storybook/internal/node-logger' for server-side (Node.js) logging, not console.log/console.warn/console.error
Applied to files:
.gitignorecode/package.json
📚 Learning: 2025-11-05T09:38:47.712Z
Learnt from: Sidnioulz
Repo: storybookjs/storybook PR: 32458
File: code/core/src/components/components/Select/Select.tsx:200-204
Timestamp: 2025-11-05T09:38:47.712Z
Learning: Repo: storybookjs/storybook — Guidance: Until Storybook 11 is released, do not suggest using React.useId anywhere (e.g., in code/core/src/components/components/Select/Select.tsx) to maintain compatibility with React 17 runtimes. Prefer advising: accept a caller-provided props.id and, if needed, generate a client-only fallback id to minimize SSR hydration issues — but avoid useId. Resume prompting for useId after Storybook 11.
Applied to files:
code/package.json
📚 Learning: 2025-11-05T09:37:25.920Z
Learnt from: Sidnioulz
Repo: storybookjs/storybook PR: 32458
File: code/core/src/components/components/tooltip/WithTooltip.tsx:54-96
Timestamp: 2025-11-05T09:37:25.920Z
Learning: Repo: storybookjs/storybook — In code/core/src/components/components/tooltip/WithTooltip.tsx, the legacy WithTooltip implementation is intentionally reintroduced for backward compatibility and is deprecated; maintainers (per Sidnioulz) do not want maintenance or improvements on it. Prefer WithTooltipNew/Popover; avoid suggesting changes to WithTooltip.* going forward.
Applied to files:
code/package.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to code/**/!(*.test).{ts,tsx,js,mjs} : Use 'logger' from 'storybook/internal/client-logger' for client-side (browser) logging, not console.log/console.warn/console.error
Applied to files:
code/package.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Mock all required dependencies that the test subject uses
Applied to files:
code/package.json
🧬 Code graph analysis (1)
scripts/get-template.ts (1)
code/lib/cli-storybook/src/sandbox-templates.ts (2)
allTemplates(988-992)templatesByCadence(1051-1051)
🪛 dotenv-linter (4.0.0)
.env
[warning] 2-2: [UnorderedKey] The IN_STORYBOOK_SANDBOX key should go before the STORYBOOK_DISABLE_TELEMETRY key
(UnorderedKey)
[warning] 3-3: [EndingBlankLine] No blank line at the end of the file
(EndingBlankLine)
[warning] 3-3: [UnorderedKey] The SANDBOX_ROOT key should go before the STORYBOOK_DISABLE_TELEMETRY key
(UnorderedKey)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: daily
- GitHub Check: nx
- GitHub Check: Core Unit Tests, windows-latest
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
.nx/workflows/distribution-config.yaml (1)
13-21: Clarify parallelism allocation for mixed target types.Rule 2 assigns
parallelism: 6uniformly to heterogeneous targets:compile,check,lint,pretty-docs, andknip. However, targets likelintandpretty-docstypically benefit minimally from high parallelism due to their single-threaded or I/O-bound nature, making this configuration potentially wasteful of agent resources.Consider either:
- Splitting these targets into separate rules with appropriate parallelism values (e.g.,
parallelism: 6forcompileandcheck;parallelism: 1–2forlint,pretty-docs,knip).- Documenting the rationale if this allocation is intentional (e.g., if targets are I/O-heavy or have other parallelization benefits).
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
yarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (1)
.nx/workflows/distribution-config.yaml(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to .yarnrc.yml : Use Yarn 4.9.1 as the package manager
Learnt from: JReinhold
Repo: storybookjs/storybook PR: 32607
File: code/package.json:243-243
Timestamp: 2025-10-02T09:22:13.215Z
Learning: The Storybook repository uses Yarn v^4 (any 4.x version) as the package manager, configured via .yarnrc.yml and package.json packageManager field. Specific patch versions within v4 can be upgraded as needed.
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Compile packages with 'yarn task --task compile' after making code changes before testing
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: daily
- GitHub Check: nx
- GitHub Check: Core Unit Tests, windows-latest
🔇 Additional comments (1)
.nx/workflows/distribution-config.yaml (1)
6-12: Verify the intent behind the core-project-only assignment rule.The first assignment rule is narrowly scoped to the
coreproject withparallelism: 1. Other projects'compiletargets fall through to Rule 2 (withparallelism: 6). This asymmetry suggestscoremay require special handling (e.g., longer compile times, resource contention), but the rationale is not documented.Clarify whether this is intentional (and if so, document it) or if other projects should also have dedicated rules.
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
scripts/create-nx-sandbox-projects.ts (1)
82-117: I'll verify the review comment about the CWD-dependent output path in the script. Let me gather information about how this script is actually invoked and whether the concern about brittleness is valid.
<function_calls>
#!/bin/bashCheck how the create-nx-sandbox-projects script is wired up
rg -n "create-nx-sandbox-projects" -C3 --type-add 'scripts:*.json' -t scripts
</function_calls><function_calls>
#!/bin/bashFind package.json files and search for references to the script
find . -name "package.json" -type f | head -20 | xargs grep -l "create-nx-sandbox" 2>/dev/null
</function_calls><function_calls>
#!/bin/bashBroader search for create-nx-sandbox-projects references
rg "create-nx-sandbox-projects" -n --max-count=20
</function_calls><function_calls>
#!/bin/bashFind the actual script file
find . -name "create-nx-sandbox-projects.ts" -type f
</function_calls><function_calls>
#!/bin/bashCheck what's in scripts directory and find TypeScript files
fd -t f ".ts$" scripts/ | head -20
</function_calls><function_calls>
#!/bin/bashList top-level directories
ls -la | head -30
</function_calls><function_calls>
#!/bin/bashTry to find the scripts directory
git ls-files | grep -E "scripts/.*.ts$" | head -20
</function_calls><function_calls>
#!/bin/bashSearch for the file more broadly
git ls-files | grep -i sandbox
</function_calls><function_calls>
#!/bin/bashCheck directory structure
git ls-files | grep "^scripts/" | head -30
</function_calls>Now let me examine the actual file mentioned in the review:
<function_calls>
#!/bin/bashTry to read the file directly
wc -l scripts/create-nx-sandbox-projects.ts
</function_calls><function_calls>
#!/bin/bashRead the entire file
cat -n scripts/create-nx-sandbox-projects.ts
</function_calls><function_calls>
#!/bin/bashCheck TypeScript configuration
cat tsconfig.json | head -50
</function_calls><function_calls>
#!/bin/bashSearch for __dirname usage in TypeScript files
rg "__dirname" --type ts --type tsx -n | head -20
</function_calls><function_calls>
#!/bin/bashSearch for import.meta.url usage
rg "import.meta.url" --type ts --type tsx -n
</function_calls><function_calls>
#!/bin/bashCheck what's in the scripts directory
ls -la scripts/
</function_calls><function_calls>
#!/bin/bashFind root package.json and check scripts
find . -maxdepth 2 -name "package.json" -exec grep -H "sandbox" {} ;
</function_calls><function_calls>
#!/bin/bashLook for any .sh or script execution references
rg "create-nx-sandbox-projects|sandbox" package.json -A2 -B2
</function_calls><function_calls>
#!/bin/bashRead root package.json scripts section
cat package.json | grep -A 100 '"scripts"' | head -120
</function_calls><function_calls>
#!/bin/bashCheck for GitHub Actions workflows
find .github -name ".yml" -o -name ".yaml" 2>/dev/null | xargs grep -l "sandbox" 2>/dev/null | head -10
</function_calls><function_calls>
#!/bin/bashCheck the context around where this script might be run
Look for any shell script files that might call this
find scripts -name ".sh" -o -name ".json" | xargs grep -l "create-nx" 2>/dev/null
</function_calls><function_calls>
#!/bin/bashCheck the directory structure
From scripts directory, ../code/sandbox would be:
If repo is /repo, scripts is /repo/scripts, then ../code/sandbox is /repo/code/sandbox
Let's verify this exists
ls -la code/sandbox 2>/dev/null || echo "Does not exist at repo/code/sandbox"
ls -la ../code/sandbox 2>/dev/null || echo "Does not exist at ../code/sandbox from root"
</function_calls><function_calls>
#!/bin/bashCheck if code directory exists at root
ls -la | grep code
fd -t d "^code$"
</function_calls><function_calls>
#!/bin/bashList top-level items in repo
git ls-tree -r --name-only HEAD | cut -d/ -f1 | sort -u | head -30
</function_calls><function_calls>
#!/bin/bashKey insight: let's check what the current code does
Line 86 uses process.cwd() + '../code/sandbox'
If CWD is /repo, then this becomes /repo/../code/sandbox = /code/sandbox (WRONG!)
If CWD is /repo/scripts, then this becomes /repo/scripts/../code/sandbox = /repo/code/sandbox (CORRECT!)
Let's verify by checking if there's any documentation in the file about CWD expectations
head -50 scripts/create-nx-sandbox-projects.ts | grep -i "cwd|directory|run"
</function_calls><function_calls>
#!/bin/bashLet's verify the imports and see if __dirname equivalent can be easily created
sed -n '1,10p' scripts/create-nx-sandbox-projects.ts
</function_calls>Perfect. I've now verified the review comment. The analysis confirms that:
The concern is valid: The code at line 86 uses
join(process.cwd(), '../code/sandbox', ...)which is indeed brittle and CWD-dependent. It only works correctly if the script is executed from thescripts/directory.The file already has the infrastructure: Lines 1-4 show the file already imports
fileURLToPathand usesimport.meta.url, which means creating a__dirnameequivalent is straightforward.The suggested fix is sound: Using the script location (via
import.meta.urlwhich is already available) would make this much more robust.No workaround or mitigation found: There's no evidence of documentation or safeguards that would make the current approach acceptable.
Resolve CWD-dependent path construction to ensure script works regardless of invocation directory
The path
join(process.cwd(), '../code/sandbox', ...)assumes the script runs from thescripts/directory and will fail or write to the wrong location otherwise. Since the file already importsfileURLToPathand usesimport.meta.url(lines 1–4), use those to derive__dirnameand construct an anchor path relative to the script location instead of relying onprocess.cwd(). This makes the script robust across any invocation context.
♻️ Duplicate comments (1)
.github/workflows/nx.yml (1)
10-12: Add missingpull-requests: readpermission for label-based conditionals.The workflow reads PR labels in conditional steps (lines 36-44) but lacks the required
pull-requests: readpermission. Without this permission, the label checks may fail silently or behave unexpectedly, especially for external pull requests.Apply this diff to add the missing permission:
permissions: actions: read contents: read + pull-requests: read
🧹 Nitpick comments (2)
code/sandbox/ember-default-js/project.json (1)
3-28: Ember sandbox tags are now empty – confirm CI intentThis project now has
tags: []even thoughember/default-jsis still listed innormal/merged/dailytemplate arrays, so it will no longer participate in tag-based Nx CI runs. If Ember is intended to be temporarily excluded (e.g., viainDevelopmenton the template), this is fine; otherwise, consider restoring theci:*tags or removing it from those template lists to avoid confusion.scripts/create-nx-sandbox-projects.ts (1)
13-79: Generator structure and conditional targets look solid (with minor DRY opportunity)The
projectJsonhelper cleanly centralizes description, implicitDependencies, and dir-scoped targets, and thetypeCheck/skipTaskslogic matches the various sandboxproject.jsonoutputs. If you want to trim repetition, you could factor a small helper likeconst skip = (task: string) => template.skipTasks?.includes(task);and reuse it for each conditional block, but that’s purely cosmetic.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (52)
.github/workflows/nx.yml(1 hunks)code/addons/vitest/src/vitest-plugin/index.ts(2 hunks)code/renderers/svelte/project.json(1 hunks)code/renderers/vue3/project.json(1 hunks)code/sandbox/angular-cli-default-ts/project.json(1 hunks)code/sandbox/angular-cli-prerelease/project.json(1 hunks)code/sandbox/bench-react-vite-default-ts-nodocs/project.json(1 hunks)code/sandbox/bench-react-vite-default-ts-test-build/project.json(1 hunks)code/sandbox/bench-react-vite-default-ts/project.json(1 hunks)code/sandbox/bench-react-webpack-18-ts-test-build/project.json(1 hunks)code/sandbox/bench-react-webpack-18-ts/project.json(1 hunks)code/sandbox/cra-default-js/project.json(1 hunks)code/sandbox/cra-default-ts/project.json(1 hunks)code/sandbox/ember-3-js/project.json(1 hunks)code/sandbox/ember-default-js/project.json(1 hunks)code/sandbox/html-rsbuild-default-ts/project.json(1 hunks)code/sandbox/html-vite-default-js/project.json(1 hunks)code/sandbox/html-vite-default-ts/project.json(1 hunks)code/sandbox/internal-react16-webpack/project.json(1 hunks)code/sandbox/internal-react18-webpack-babel/project.json(1 hunks)code/sandbox/internal-server-webpack5/project.json(1 hunks)code/sandbox/lit-rsbuild-default-ts/project.json(1 hunks)code/sandbox/lit-vite-default-js/project.json(1 hunks)code/sandbox/lit-vite-default-ts/project.json(1 hunks)code/sandbox/nextjs-14-ts/project.json(1 hunks)code/sandbox/nextjs-15-ts/project.json(1 hunks)code/sandbox/nextjs-default-ts/project.json(1 hunks)code/sandbox/nextjs-prerelease/project.json(1 hunks)code/sandbox/nextjs-vite-14-ts/project.json(1 hunks)code/sandbox/nextjs-vite-15-ts/project.json(1 hunks)code/sandbox/nextjs-vite-default-ts/project.json(1 hunks)code/sandbox/preact-vite-default-js/project.json(1 hunks)code/sandbox/preact-vite-default-ts/project.json(1 hunks)code/sandbox/qwik-vite-default-ts/project.json(1 hunks)code/sandbox/react-native-web-vite-expo-ts/project.json(1 hunks)code/sandbox/react-native-web-vite-rn-cli-ts/project.json(1 hunks)code/sandbox/react-rsbuild-default-ts/project.json(1 hunks)code/sandbox/react-vite-default-js/project.json(1 hunks)code/sandbox/react-vite-default-ts/project.json(1 hunks)code/sandbox/react-vite-prerelease-ts/project.json(1 hunks)code/sandbox/react-webpack-17-ts/project.json(1 hunks)code/sandbox/react-webpack-18-ts/project.json(1 hunks)code/sandbox/react-webpack-prerelease-ts/project.json(1 hunks)code/sandbox/solid-vite-default-ts/project.json(1 hunks)code/sandbox/svelte-kit-skeleton-ts/project.json(1 hunks)code/sandbox/svelte-vite-default-js/project.json(1 hunks)code/sandbox/svelte-vite-default-ts/project.json(1 hunks)code/sandbox/vue3-rsbuild-default-ts/project.json(1 hunks)code/sandbox/vue3-vite-default-js/project.json(1 hunks)code/sandbox/vue3-vite-default-ts/project.json(1 hunks)docs/_snippets/addon-consume-and-update-globaltype.md(1 hunks)scripts/create-nx-sandbox-projects.ts(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- code/sandbox/nextjs-14-ts/project.json
🚧 Files skipped from review as they are similar to previous changes (24)
- code/sandbox/react-webpack-prerelease-ts/project.json
- code/sandbox/react-native-web-vite-expo-ts/project.json
- code/sandbox/nextjs-vite-15-ts/project.json
- code/sandbox/lit-rsbuild-default-ts/project.json
- code/sandbox/nextjs-vite-default-ts/project.json
- code/sandbox/vue3-rsbuild-default-ts/project.json
- code/sandbox/preact-vite-default-js/project.json
- code/sandbox/nextjs-15-ts/project.json
- code/sandbox/react-native-web-vite-rn-cli-ts/project.json
- code/sandbox/nextjs-vite-14-ts/project.json
- code/addons/vitest/src/vitest-plugin/index.ts
- code/sandbox/qwik-vite-default-ts/project.json
- code/sandbox/nextjs-default-ts/project.json
- code/sandbox/html-rsbuild-default-ts/project.json
- code/sandbox/react-rsbuild-default-ts/project.json
- code/sandbox/svelte-kit-skeleton-ts/project.json
- code/sandbox/react-webpack-17-ts/project.json
- code/renderers/svelte/project.json
- code/sandbox/react-webpack-18-ts/project.json
- code/sandbox/vue3-vite-default-js/project.json
- code/sandbox/preact-vite-default-ts/project.json
- code/sandbox/svelte-vite-default-js/project.json
- code/sandbox/internal-react16-webpack/project.json
- code/sandbox/lit-vite-default-js/project.json
🧰 Additional context used
📓 Path-based instructions (3)
**/*.{ts,tsx,js,jsx,json,html,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx,js,jsx,json,html,mjs}: Use ESLint and Prettier for code style enforcement
Run 'yarn prettier --write ' to format code after making changes
Run 'yarn lint:js:cmd ' to check for ESLint issues after making changes
Files:
code/sandbox/cra-default-ts/project.jsoncode/sandbox/bench-react-vite-default-ts-nodocs/project.jsoncode/sandbox/svelte-vite-default-ts/project.jsoncode/sandbox/angular-cli-default-ts/project.jsoncode/sandbox/internal-server-webpack5/project.jsoncode/sandbox/react-vite-default-ts/project.jsoncode/sandbox/react-vite-prerelease-ts/project.jsoncode/sandbox/bench-react-vite-default-ts/project.jsoncode/sandbox/html-vite-default-ts/project.jsoncode/sandbox/solid-vite-default-ts/project.jsoncode/sandbox/nextjs-prerelease/project.jsoncode/sandbox/angular-cli-prerelease/project.jsonscripts/create-nx-sandbox-projects.tscode/sandbox/ember-3-js/project.jsoncode/sandbox/bench-react-vite-default-ts-test-build/project.jsoncode/sandbox/cra-default-js/project.jsoncode/sandbox/bench-react-webpack-18-ts-test-build/project.jsoncode/sandbox/lit-vite-default-ts/project.jsoncode/sandbox/react-vite-default-js/project.jsoncode/sandbox/vue3-vite-default-ts/project.jsoncode/sandbox/html-vite-default-js/project.jsoncode/sandbox/bench-react-webpack-18-ts/project.jsoncode/renderers/vue3/project.jsoncode/sandbox/ember-default-js/project.jsoncode/sandbox/internal-react18-webpack-babel/project.json
**/*.{ts,tsx,js,jsx,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use camelCase for variable and function names
Files:
scripts/create-nx-sandbox-projects.ts
**/*.{ts,tsx}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx}: Enable TypeScript strict mode
Export functions from modules for testing purposes
Files:
scripts/create-nx-sandbox-projects.ts
🧠 Learnings (17)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Applied to files:
code/sandbox/cra-default-ts/project.jsoncode/sandbox/bench-react-vite-default-ts-nodocs/project.jsoncode/sandbox/svelte-vite-default-ts/project.jsoncode/sandbox/angular-cli-default-ts/project.jsoncode/sandbox/react-vite-prerelease-ts/project.jsoncode/sandbox/bench-react-vite-default-ts/project.jsoncode/sandbox/html-vite-default-ts/project.jsoncode/sandbox/solid-vite-default-ts/project.jsoncode/sandbox/nextjs-prerelease/project.jsoncode/sandbox/angular-cli-prerelease/project.jsonscripts/create-nx-sandbox-projects.tscode/sandbox/ember-3-js/project.jsoncode/sandbox/bench-react-vite-default-ts-test-build/project.jsoncode/sandbox/cra-default-js/project.jsoncode/sandbox/bench-react-webpack-18-ts-test-build/project.jsoncode/sandbox/lit-vite-default-ts/project.jsoncode/sandbox/react-vite-default-js/project.jsoncode/sandbox/html-vite-default-js/project.jsoncode/sandbox/bench-react-webpack-18-ts/project.jsoncode/sandbox/internal-react18-webpack-babel/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid mocking only a subset of required dependencies in Vitest tests
Applied to files:
code/sandbox/react-vite-prerelease-ts/project.jsoncode/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid inline mock implementations within test cases in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Keep mock implementations simple and focused in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid mocking without the `spy: true` option in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid mock implementations outside of `beforeEach` blocks in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid direct function mocking without `vi.mocked()` in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Document complex mock behaviors in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:31.838Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
Applied to files:
code/sandbox/solid-vite-default-ts/project.jsoncode/sandbox/lit-vite-default-ts/project.jsoncode/sandbox/vue3-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:59.279Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursor/rules/spy-mocking.mdc:0-0
Timestamp: 2025-11-24T17:49:59.279Z
Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Mock all required properties and methods that the test subject uses in Vitest tests
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,mjs} : Use camelCase for variable and function names
Applied to files:
code/sandbox/solid-vite-default-ts/project.json
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Run 'yarn prettier --write <file>' to format code after making changes
Applied to files:
scripts/create-nx-sandbox-projects.ts
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.{ts,tsx,js,jsx,json,html,mjs} : Run 'yarn lint:js:cmd <file>' to check for ESLint issues after making changes
Applied to files:
scripts/create-nx-sandbox-projects.ts
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Applied to files:
scripts/create-nx-sandbox-projects.ts
📚 Learning: 2025-09-18T20:51:06.618Z
Learnt from: Sidnioulz
Repo: storybookjs/storybook PR: 32458
File: code/core/src/viewport/components/Tool.tsx:38-39
Timestamp: 2025-09-18T20:51:06.618Z
Learning: In viewport tool code, when using the `useGlobals` hook from storybook/manager-api, the third returned value `storyGlobals` is guaranteed by TypeScript to be defined (not undefined/null), making the `in` operator safe to use without additional null checks.
Applied to files:
docs/_snippets/addon-consume-and-update-globaltype.md
📚 Learning: 2025-09-18T20:51:06.618Z
Learnt from: Sidnioulz
Repo: storybookjs/storybook PR: 32458
File: code/core/src/viewport/components/Tool.tsx:38-39
Timestamp: 2025-09-18T20:51:06.618Z
Learning: The useGlobals hook from storybook/manager-api returns a tuple where the third element (storyGlobals) is typed as Globals, not Globals | undefined. This means TypeScript guarantees it's always defined, making the `in` operator safe to use without additional null checks.
Applied to files:
docs/_snippets/addon-consume-and-update-globaltype.md
🧬 Code graph analysis (1)
scripts/create-nx-sandbox-projects.ts (1)
code/lib/cli-storybook/src/sandbox-templates.ts (5)
Template(30-106)allTemplates(978-982)normal(984-1003)merged(1005-1015)daily(1017-1040)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: normal
- GitHub Check: Core Unit Tests, windows-latest
- GitHub Check: nx
🔇 Additional comments (39)
docs/_snippets/addon-consume-and-update-globaltype.md (1)
14-21: Documentation formatting improvement — verify PR scope alignment.The formatting changes improve code readability by properly separating the
updateGlobalscall closure from the comment and subsequentemitcall. The example remains functionally correct and follows proper Storybook API usage patterns.However, this documentation change appears disconnected from the stated PR scope (monorepo root refactoring from
./codeto repository root). Please confirm whether this file should be included in this PR or if it was modified separately.code/sandbox/cra-default-js/project.json (2)
1-27: Verify the generation script and CI tag removal.This file is marked as auto-generated by
scripts/create-nx-sandbox-projects.ts, so changes should be verified against that script to ensure it was updated to produce this structure.Additionally, the removal of the
"ci:daily"tag (line 26) is a behavioral change—verify that this won't impact CI workflows or task scheduling.
3-13: Consistent with PR standardization.The expanded
implicitDependenciesarray now includes the CLI and multiple addons (addon-a11y,addon-docs,addon-vitest), aligning with the monorepo refactor pattern applied across other sandbox projects. Thediroptions onsandboxandbuildtargets (lines 15, 18) correctly reference the sandbox directory name.code/renderers/vue3/project.json (4)
5-5: Verify the emptycompiletarget is intentional.The
compiletarget is defined but empty ({}). Clarify whether:
- This is a placeholder/dependency marker with no actual compilation step for Vue3
- It should contain an executor and options (e.g., running a Vue3 build or type-check step)
Confirm this aligns with the migration pattern across other renderers in the monorepo.
6-9: Type-checking target configuration looks correct.The
checktarget properly delegates Vue type-checking vianx:run-commandswithvue-tsc --noEmit, which aligns with the refactored compile+check workflow.
11-11: CI tag standardization is correct.The
tags: ["ci:normal"]addition aligns with the PR objective to standardize CI tag usage across renderer project configs.
5-11: Verify the removal of thebuildtarget does not break CI or dependent projects.The refactor replaces the
buildtarget with separatecompileandchecktargets. Confirm that:
- CI workflows and other projects no longer reference the
buildtarget- The new
compileandchecktargets are properly wired into downstream tasks- Other renderers follow the same pattern for consistency
code/sandbox/angular-cli-prerelease/project.json (1)
18-18: The review comment is incorrect. The naming is intentional, not a bug.The file is correctly auto-generated. Across all sandbox projects in the codebase, the target is consistently named
"build"(not"build-sandbox"), paired with"prepare-build-sandbox". This asymmetric naming is intentional:
- The generation script (lines 51–56 in
scripts/create-nx-sandbox-projects.ts) deliberately produces"build"as the target namenx.jsonexplicitly configures"prepare-build-sandbox"to depend on the"build"target- All sandbox
project.jsonfiles follow this same pattern consistentlyThe assumption that the target should be named
"build-sandbox"for symmetry with"sandbox"/"prepare-sandbox"is unfounded. The actual design intentionally uses"build"as the primary target name.Likely an incorrect or invalid review comment.
code/sandbox/internal-server-webpack5/project.json (3)
3-3: I'll help you verify the auto-generated nature of this file and gather evidence about the script that generates it. Let me run some investigation scripts.
<function_calls>
#!/bin/bashStep 1: Verify the create-nx-sandbox-projects.ts script exists
echo "=== Checking for create-nx-sandbox-projects.ts ==="
fd -e ts 'create-nx-sandbox-projects' 2>/dev/null | head -5Step 2: Check file size before reading
echo -e "\n=== File size check ==="
fd -e ts 'create-nx-sandbox-projects' -x wc -l {} 2>/dev/nullStep 3: List all sandbox project.json files
echo -e "\n=== All sandbox project.json files ==="
fd 'project.json' 'code/sandbox' --type f 2>/dev/null | sortStep 4: Count total sandbox project.json files
echo -e "\n=== Total count ==="
fd 'project.json' 'code/sandbox' --type f 2>/dev/null | wc -l
</function_calls>
5-13: "addon-links" removal is intentional and consistent across all sandbox projects.The verification shows:
- Zero sandbox projects reference "addon-links" in their implicit dependencies
- The "addon-links" package does not exist in the monorepo
- All sandbox projects follow an identical implicit dependency pattern (core, cli, addon-a11y, addon-docs, addon-vitest, addon-onboarding, plus environment-specific package)
The
internal-server-webpack5project.json is aligned with the standard across all sandboxes. There is no selective oversight here.
15-25: All targets are properly configured in the root nx.json.Verification confirms that all 11 targets defined as empty objects
{}incode/sandbox/internal-server-webpack5/project.jsonhave their full implementations inroot/nx.jsonundertargetDefaults. This is the correct and intentional pattern in Nx—project-levelproject.jsonfiles reference task definitions from the workspacetargetDefaults, allowing centralized configuration management. The empty objects properly inherit all configuration (dependsOn, command, cache, inputs, outputs, etc.) from the root definitions.code/sandbox/nextjs-prerelease/project.json (3)
5-13: Verify the expanded implicitDependencies list is intentional.The implicitDependencies list expanded from 4 packages to 7, adding
"cli","addon-a11y","addon-docs", and"addon-vitest". Confirm these are necessary dependencies for the Next.js prerelease sandbox in the new monorepo structure after moving the root to git root.
15-24: Clarify the sparse target definitions and verify executor configuration location.Most targets (lines 16–17, 19–24) are defined as empty objects
{}, with onlysandboxandbuildspecifying adiroption. Confirm this is intentional and that the executor configurations (commands, scripts, etc.) are defined at the root level or in shared Nx configurations, not expected to be in this file.
3-3: Based on my verification, I now have enough information to provide the rewritten review comment.
This file was not regenerated in this PR; verify if regeneration was required for the migration.
The generation script at
scripts/create-nx-sandbox-projects.tswas not run as part of this PR—git diff shows no changes to either the script or this file. The file content matches what the script would produce, but the absence of a git diff indicates no regeneration occurred. Confirm whether the monorepo root migration required running this script to update sandbox project files.code/sandbox/cra-default-ts/project.json (2)
5-13: The implicitDependencies list in cra-default-ts is appropriately scoped and consistent with all other sandboxes.All 50+ sandbox projects in the codebase follow the same pattern: including the six core addons (core, cli, addon-a11y, addon-docs, addon-vitest, addon-onboarding) plus a single framework-specific dependency. The cra-default-ts sandbox correctly lists
react-webpack5as its framework-specific dependency, following the exact same pattern as other React-webpack sandboxes (react-webpack-17-ts, react-webpack-18-ts, etc.). The listed addons are all necessary and framework-relevant.
1-28: Based on my verification of the generation script and configuration:All concerns in the review comment have been verified and resolved—no issues found.
The generation script (
scripts/create-nx-sandbox-projects.ts) confirms:
Executor definitions are intentional: Empty target objects
{}inherit their executor configurations fromnx.jsontargetDefaults. This is the intended design pattern, as shown in thenx.jsonoutput which defines executors forsandbox,prepare-sandbox,dev,build, etc.Dir values are correct: The script uses
name.replaceAll('/', '-')to convert the template name (e.g.,"cra/default-ts") into a stable folder name ("cra-default-ts"). Thenx.jsonthen references{options.dir}as a template variable in output paths.Generation script is reliable: The script iterates through
allTemplates, dynamically generatesproject.jsonfor each, and writes them with Prettier formatting. The file is deterministic and properly auto-generated.Implicit dependencies are correct: The script includes base dependencies (
core,cli, addons) plus the framework (react-webpack5for CRA). This matches other sandbox projects likecra-default-jsandreact-vite-default-ts.Consistency verified: Comparing
cra-default-tswithcra-default-jsandreact-vite-default-tsconfirms the file follows the correct pattern.code/sandbox/html-vite-default-js/project.json (2)
15-15: Confirm dir option aligns with monorepo root refactoring.Lines 15 and 18 add
"dir": "html-vite-default-js"to thesandboxandbuildtargets. This appears consistent with the PR's refactoring to move the monorepo root to the git root—tasks now need explicit directory specifications. Verify that the NX executors handling these targets correctly interpret thisdiroption relative to the new repository root.Also applies to: 18-18
3-3: All implicit dependencies and targets are correctly generated and valid.Verification confirms:
- ✓ Generation script exists (
scripts/create-nx-sandbox-projects.ts) and is functional- ✓ All implicit dependencies are valid packages existing in the codebase:
core(code/core/)cli(code/lib/cli-storybook/)addon-a11y(code/addons/a11y/)addon-docs(code/addons/docs/)addon-vitest(code/addons/vitest/)addon-onboarding(code/addons/onboarding/)html-vite(code/frameworks/html-vite/)- ✓ All targets have corresponding NX executor definitions in
nx.jsontargetDefaults- ✓ Removal of
addon-linksis intentional—it is not included in the hardcoded implicit dependencies array in the generation script- ✓ File output matches script logic exactly—conditional targets are correctly filtered based on template configuration
No issues found. The auto-generated file is correct and consistent with the generation script.
code/sandbox/solid-vite-default-ts/project.json (4)
3-3: Auto-generated configuration notice is appropriate.The description field clearly marks this file as generated, which aids maintainability.
5-12: Based on my verification, I've found critical information that changes the nature of the review comment:
The dependencies are auto-generated, not manually expanded. The file header states: "Do not edit. Auto generated by scripts/create-nx-sandbox-projects.ts."
The dependency pattern is consistent across all Vite TypeScript sandboxes in the codebase. Every similar sandbox includes:
core,cli,addon-a11y,addon-docs,addon-vitest,addon-onboarding(plus framework-specific entries).addon-links is not present in any of the Vite TS sandboxes, so it was never an implicit dependency in this project—the concern about it being "intentionally dropped" is based on a false premise.
No manual verification needed for auto-generated dependencies. The implicit dependencies list follows the standard pattern across all similar Vite TypeScript sandboxes and is auto-generated by
create-nx-sandbox-projects.ts, not manually curated.The dependencies (
cli,addon-a11y,addon-docs,addon-vitest) are consistent with other Vite TypeScript sandboxes and are not subject to the verification concern raised. Note:addon-linksdoes not appear in any Vite TypeScript sandbox, so concerns about it being dropped are unfounded.
14-22: Missing test and integration targets in solid-vite-default-ts compared to similar Vite TS sandboxes.The review comment's concern is confirmed. The solid-vite-default-ts configuration is missing several targets present in comparable sandboxes:
Missing targets:
e2e-testsande2e-tests-dev(present in react-vite-default-ts, vue3-vite-default-ts, and 33 other sandboxes)vitest-integration(present in react-vite-default-ts, vue3-vite-default-ts)check-sandbox(present in react-vite-default-ts)Current targets in solid-vite-default-ts: sandbox, prepare-sandbox, dev, build, prepare-build-sandbox, chromatic, serve, test-runner, test-runner-dev (9 targets)
Current targets in react-vite-default-ts: Same 9 targets PLUS check-sandbox, e2e-tests, e2e-tests-dev, vitest-integration (13 targets total)
These are not renamed targets—they are genuinely absent from the solid-vite-default-ts configuration. Clarify whether the configuration should include these targets to maintain parity with other modern framework sandboxes or if solid-vite-default-ts intentionally has a reduced set of targets.
⛔ Skipped due to learnings
Learnt from: CR Repo: storybookjs/storybook PR: 0 File: .cursor/rules/spy-mocking.mdc:0-0 Timestamp: 2025-11-24T17:49:59.279Z Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Avoid mocking only a subset of required dependencies in Vitest testsLearnt from: CR Repo: storybookjs/storybook PR: 0 File: .cursor/rules/spy-mocking.mdc:0-0 Timestamp: 2025-11-24T17:49:59.279Z Learning: Applies to **/*.test.{ts,tsx,js,jsx} : Keep mock implementations simple and focused in Vitest testsLearnt from: CR Repo: storybookjs/storybook PR: 0 File: .github/copilot-instructions.md:0-0 Timestamp: 2025-11-24T17:49:47.659Z Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinationsLearnt from: CR Repo: storybookjs/storybook PR: 0 File: .cursorrules:0-0 Timestamp: 2025-11-24T17:49:31.838Z Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
24-24: Tags are appropriate and consistent with the monorepo's CI categorization.The
ci:mergedandci:dailytags insolid-vite-default-ts/project.jsoncorrectly align with the sandbox categorization incode/lib/cli-storybook/src/sandbox-templates.ts:
solid-vite/default-tsis explicitly listed in themergedarray (post-merge validation workflow)- It's implicitly included in the
dailyarray via...merged(comprehensive daily testing)- These tags match the CircleCI workflow definitions (
merged.ymlanddaily.yml) that use them as pull request labels to trigger specific workflowsThe tag combination is consistent with peer sandboxes of similar scope (e.g.,
react-webpack/17-ts,react-webpack/18-ts,nextjs-vite/15-ts,vue3-rsbuild/default-ts).code/sandbox/lit-vite-default-ts/project.json (3)
3-3: File is correctly auto-generated. Verification confirms the generation script has been properly updated for the monorepo refactor.The generation script (
scripts/create-nx-sandbox-projects.ts) confirms:
Description is correct: Line 18 of the script hardcodes the exact description appearing on line 3 of the project.json file.
Consistency verified: Examined
lit-vite-default-ts,lit-vite-default-js, andlit-rsbuild-default-tsproject.json files—all follow the same pattern generated by the script. Each includes:
- Base
implicitDependenciesarray (core, cli, addon-a11y, addon-docs, addon-vitest, addon-onboarding)- Framework-specific dependency appended (e.g., "web-components-vite" for lit sandboxes)
- Targets conditionally generated based on
template.skipTasksTask-based target generation: The presence/absence of tasks like
e2e-tests-devversuse2e-testsis intentional—determined by the template configuration. This is not a discrepancy but expected behavior.The script uses Prettier for formatting, ensuring consistent output across all generated files.
15-24: The review comment is based on incorrect assumptions about the target structure.The verification reveals that the target configuration is intentionally generated and properly aligned with the monorepo's NX architecture:
Target naming is intentional, not inconsistent: The generation script (
scripts/create-nx-sandbox-projects.ts) conditionally includes targets based on each template'sskipTasksconfiguration. Thelit-vite-default-tstemplate deliberately omits thee2e-teststarget while includinge2e-tests-dev—this is not an error or inconsistency.Centralized targets are properly aligned:
prepare-sandboxandprepare-build-sandboxare empty placeholder objects in sandboxproject.jsonfiles because the actual target implementations are defined in the root NX configuration (nx.json). This is standard NX monorepo architecture where sandbox projects inherit target definitions from the workspace root.Empty targets vs. those with
diroptions are intentional:
sandboxandbuildtargets includediroptions specific to each sandbox- All other targets are empty because they inherit implementations from root-level
targetDefaults- This pattern is consistent and correct across all 45+ sandbox configurations
5-13: Based on my comprehensive verification, I can now provide the rewritten review comment:
This is an auto-generated file controlled by scripts/create-nx-sandbox-projects.ts—manual edits will be overwritten.
The
implicitDependencieslist is not "significantly expanded" but rather reflects the standard dependencies applied uniformly to all sandboxes at generation time (lines 22–29 of the script). These are build/tooling dependencies used by Nx for task ordering and CI/CD, not runtime imports. The lit-vite/default-ts sandbox correctly includesweb-components-viteas its framework dependency. To modify implicit dependencies, edit the generation script, not this file.code/sandbox/internal-react18-webpack-babel/project.json (1)
1-27: Auto-generated project configuration looks good.This file follows the PR's standardized pattern for sandbox project configurations. The expanded
implicitDependenciesand directory-scoped targets align with the monorepo refactoring.Note: Verify that the target set is intentional for this sandbox. Some other sandboxes include
e2e-testsandvitest-integrationtargets; this one hase2e-tests-devbut note2e-tests. Confirm this aligns with the generation script's logic via code review ofscripts/create-nx-sandbox-projects.ts.code/sandbox/angular-cli-default-ts/project.json (1)
1-28: Auto-generated project configuration looks good.Properly configured with expanded dependencies and comprehensive target set including both
e2e-testsande2e-tests-dev.Note: Observe that this sandbox includes
e2e-tests(line 22) while some others do not. Ensure the generation script's logic for determining which targets to include per sandbox type is properly validated.code/sandbox/html-vite-default-ts/project.json (1)
1-27: Auto-generated configuration consistent with sandbox type.HTML sandbox properly configured with Vite-specific dependencies and appropriate target set.
code/sandbox/react-vite-prerelease-ts/project.json (1)
1-28: New sandbox configuration properly structured.New prerelease sandbox includes
vitest-integrationtarget, which is appropriate for Vite-based testing. Configuration follows the standardized pattern.code/sandbox/vue3-vite-default-ts/project.json (1)
1-29: Well-structured configuration with comprehensive targets.Vue3+Vite sandbox properly includes
vitest-integrationand bothe2e-testsvariants, reflecting the importance of this framework combination in the test matrix.code/sandbox/svelte-vite-default-ts/project.json (1)
1-29: Configuration consistent with Vite ecosystem standards.Svelte+Vite sandbox properly includes vitest integration and comprehensive e2e testing targets.
code/sandbox/bench-react-webpack-18-ts/project.json (1)
1-24: Benchmark sandbox appropriately configured.The reduced target set (no test-runner, vitest-integration, or e2e-tests) is appropriate for a benchmark-focused sandbox that prioritizes build and serve capabilities.
code/sandbox/ember-3-js/project.json (1)
28-28: Based on my verification, I now have complete clarity on the issue. The empty tags for Ember is intentional and by design, not an oversight.Rewritten Review Comment:
Empty tags array is intentional for Ember sandboxes marked as in-development.
Both Ember templates (
ember/3-jsandember/default-js) are configured withinDevelopment: trueincode/lib/cli-storybook/src/sandbox-templates.ts(lines 748, 759). The tag assignment logic inscripts/create-nx-sandbox-projects.ts(lines 94-98) explicitly filters out templates with this flag, resulting in empty tags[]. This prevents CI scheduling for templates under active development. Althoughember/default-jsis listed in thenormalCI schedule array (line 1001), theinDevelopmentflag takes precedence and blocks CI execution—this is the intended mechanism to exclude experimental or unstable templates from pipelines.code/sandbox/bench-react-webpack-18-ts-test-build/project.json (1)
3-21: Sandbox project config matches generator and template metadataThe description, implicitDependencies (incl.
react-webpack5), dir-scopedsandbox/buildtargets, and CI tags all line up with the new generator logic and the template’s inclusion innormal,merged, anddaily. Looks good.code/sandbox/react-vite-default-js/project.json (1)
3-25: React Vite JS sandbox targets and deps look consistentDir-scoped targets, expanded implicitDependencies, and
["ci:daily"]tags align with the generator and the template’s membership indailyonly. No issues from a config/Nx standpoint.code/sandbox/react-vite-default-ts/project.json (1)
3-27: TS React Vite sandbox config matches the generator’s intentPresence of
check-sandbox, all test/runner targets, properdirvalues, and full CI tag set are exactly what the new generator produces for a primary template likereact-vite/default-ts. Looks correct.code/sandbox/bench-react-vite-default-ts-nodocs/project.json (1)
3-21: Bench Vite TS nodocs sandbox config aligns with generator behaviorThe dir-scoped sandbox/build targets, expanded implicitDependencies, and full CI tag set line up with the bench template’s presence in
normal/merged/dailyand its likelyskipTaskssettings. No changes needed.code/sandbox/bench-react-vite-default-ts/project.json (1)
3-21: Bench Vite TS sandbox project.json is consistent and correctDescription, implicitDependencies (incl.
react-vite), dir-scopedsandbox/buildtargets, and CI tags mirror the generator logic and template classification. Looks good.code/sandbox/bench-react-vite-default-ts-test-build/project.json (1)
3-23: Bench Vite TS test‑build sandbox config matches the new generatorThe dir value, implicitDependencies, limited target set, and full CI tags are in line with the template’s role as a build‑only benchmark sandbox. No issues spotted.
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (2)
.circleci/config.yml (2)
1050-1072: Still missing Verdaccio step beforeprepare-sandboxin test-runner-productionThis job now uses
yarn workspace @storybook/scripts prepare-sandbox --template $TEMPLATE --no-link, but unlike other sandbox jobs (bench-sandboxes, check-sandboxes, e2e-dev, e2e-production, chromatic-sandboxes), it doesn’t start Verdaccio beforehand.Given that
prepare-sandboxdepends on the local registry (per the Nx targets) and the same pattern has been applied elsewhere, this job is likely to fail when it tries to resolve packages from the local registry.You should add the same Verdaccio background step used in other sandbox jobs immediately before the
prepare-sandboxrun, for example:- run: background: true command: | cd code yarn local-registry --open name: Verdaccio - run: command: | TEMPLATE=$(yarn get-template --cadence << pipeline.parameters.workflow >> --task test-runner) yarn workspace @storybook/scripts prepare-sandbox --template $TEMPLATE --no-link name: Install sandbox dependencies
1114-1144: Same Verdaccio gap for vitest-integration beforeprepare-sandboxAnalogous to
test-runner-production,vitest-integrationnow calls:TEMPLATE=$(yarn get-template --cadence << pipeline.parameters.workflow >> --task vitest-integration) yarn workspace @storybook/scripts prepare-sandbox --template $TEMPLATE --no-linkbut does not start Verdaccio, unlike other sandbox jobs. If
prepare-sandboxassumes a running local registry, this will break when it tries to install/publish packages to the local Verdaccio instance.Consider inserting the standard Verdaccio background step immediately before the
prepare-sandboxrun, mirroring the pattern used in bench-sandboxes/e2e-*:- run: background: true command: | cd code yarn local-registry --open name: Verdaccio - run: command: | TEMPLATE=$(yarn get-template --cadence << pipeline.parameters.workflow >> --task vitest-integration) yarn workspace @storybook/scripts prepare-sandbox --template $TEMPLATE --no-link name: Install sandbox dependencies
🧹 Nitpick comments (3)
.env (1)
1-3: Env defaults look consistent; newline/order nits are optionalThe variables and values match the new sandbox behavior and CI usage (telemetry off, sandbox flag on, default
../storybook-sandboxesroot). Functionally this is fine.If you care about
dotenv-linternoise, you can:
- Reorder keys as it suggests, and
- Add a trailing newline at EOF.
Both are purely cosmetic and can be deferred.
scripts/utils/constants.ts (1)
1-1: Sandbox directory resolution is correct; consider tightening imports and env handlingLogic-wise this does what you want:
- Unset env ⇒
${ROOT_DIRECTORY}/../storybook-sandboxes- Relative env ⇒
${ROOT_DIRECTORY}/${relativeValue}- Absolute env ⇒ use as-is
Two small, optional cleanups:
- Avoid mixing
node:pathand'path'imports; either usenode:pathfor both and destructurejoin, or dropnode:pathand rely on'path'everywhere, depending on your Node/TS config.- Cache
process.env.STORYBOOK_SANDBOX_ROOTin a localconstto avoid repeating the lookup and to make the branching slightly easier to read.Also applies to: 8-18, 21-22
.circleci/config.yml (1)
200-245: Build job now correctly targets repo-root Yarn and cacheUsing
{{ checksum "yarn.lock" }}for the cache key, runningyarnfrom the repo root, and persisting both root and workspacenode_modulesis consistent with moving the Yarn monorepo root to..One follow-up outside this file: docs/contributor guides that still talk about
cd code && yarn …should be updated to reflect the new root-levelyarnusage. Based on learnings, existing guidance in.github/copilot-instructions.mdstill points contributors atcode/as the primary Yarn root.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
.circleci/config.yml(14 hunks).env(1 hunks).nx/workflows/agents.yaml(1 hunks)scripts/utils/constants.ts(2 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- .nx/workflows/agents.yaml
🧰 Additional context used
📓 Path-based instructions (3)
**/*.{ts,tsx,js,jsx,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Use camelCase for variable and function names
Files:
scripts/utils/constants.ts
**/*.{ts,tsx}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx}: Enable TypeScript strict mode
Export functions from modules for testing purposes
Files:
scripts/utils/constants.ts
**/*.{ts,tsx,js,jsx,json,html,mjs}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
**/*.{ts,tsx,js,jsx,json,html,mjs}: Use ESLint and Prettier for code style enforcement
Run 'yarn prettier --write ' to format code after making changes
Run 'yarn lint:js:cmd ' to check for ESLint issues after making changes
Files:
scripts/utils/constants.ts
🧠 Learnings (6)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to .yarnrc.yml : Use Yarn 4.9.1 as the package manager
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Applied to files:
scripts/utils/constants.ts.circleci/config.yml.env
📚 Learning: 2025-10-02T09:22:13.215Z
Learnt from: JReinhold
Repo: storybookjs/storybook PR: 32607
File: code/package.json:243-243
Timestamp: 2025-10-02T09:22:13.215Z
Learning: The Storybook repository uses Yarn v^4 (any 4.x version) as the package manager, configured via .yarnrc.yml and package.json packageManager field. Specific patch versions within v4 can be upgraded as needed.
Applied to files:
.circleci/config.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Applied to files:
.circleci/config.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Compile packages with 'yarn task --task compile' after making code changes before testing
Applied to files:
.circleci/config.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'cd code && yarn storybook:ui' to start the development server for testing UI changes on http://localhost:6006/
Applied to files:
.circleci/config.yml
🪛 dotenv-linter (4.0.0)
.env
[warning] 2-2: [UnorderedKey] The IN_STORYBOOK_SANDBOX key should go before the STORYBOOK_DISABLE_TELEMETRY key
(UnorderedKey)
[warning] 3-3: [EndingBlankLine] No blank line at the end of the file
(EndingBlankLine)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: normal
- GitHub Check: nx
- GitHub Check: Core Unit Tests, windows-latest
🔇 Additional comments (8)
.circleci/config.yml (8)
168-197: Bench sandboxes Verdaccio + shared prepare-sandbox look goodStarting Verdaccio in the background and then using the shared
yarn workspace @storybook/scripts prepare-sandbox --template $TEMPLATE --no-linkaligns this job with the new sandbox flow and other Verdaccio-using jobs.
246-274: Check job changes are consistent with the new root-level Yarn workflowReusing the same root-level Yarn cache (
yarn.lock) and invokingyarn+yarn task --task compilefrom the repo root matches the build job and the new monorepo root arrangement.
275-305: check-sandboxes now follows the Verdaccio + prepare-sandbox patternAdding the Verdaccio background step and switching to
yarn workspace @storybook/scripts prepare-sandbox --template $TEMPLATE --no-linkmakes this job consistent with the other sandbox-centric jobs and should avoid registry-related flakiness.
324-352: Confirm intentional use ofSTORYBOOK_SANDBOX_ROOT=./sandboxin chromatic-sandboxesResource bump to
medium+, Verdaccio, and the move toprepare-sandboxall look good.The one thing to double-check: this job explicitly overrides
STORYBOOK_SANDBOX_ROOT=./sandbox, whereas the new default is../storybook-sandboxes. If the intent is that CI chromatic flows still use the in-repo./sandboxwhile other flows use the new default, it’d be good to add a brief comment here or in docs; otherwise you may want this job to rely on the shared default instead.
365-405: create-sandboxes: usingyarn get-sandbox-dirfor cleanup matches the new sandbox layoutUsing
$(yarn get-sandbox-dir --template $TEMPLATE)to locate each sandbox beforerm -rf node_modulesis a nice robustness upgrade now that sandboxes can live outside./code. It keeps the cleanup in sync with the central sandbox path logic.
408-443: e2e-dev and e2e-production now correctly prep sandboxes via Verdaccio + prepare-sandboxBoth jobs now:
- Start Verdaccio in the background from
code/, and- Use
yarn workspace @storybook/scripts prepare-sandbox --template $TEMPLATE --no-link.This aligns them with other sandbox jobs and should make their sandbox resolution consistent with the new root-level tooling.
562-585: pretty-docs resource/cache/install tweaks are consistent with root YarnBumping the executor class to
medium+, keying the cache off the rootyarn.lock, and runningyarn installfrom the repo root beforescripts’ Prettier check all match the monorepo-root change and should simplify cache behavior.
987-1021: test-portable-stories resource bump seems reasonableIncreasing the executor class to
medium+for this multi-run job (Jest, Vitest, Playwright CT, Cypress) is a pragmatic capacity improvement and doesn’t alter behavior.
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (1)
.circleci/config.yml (1)
1023-1027: [DUPLICATE] Add Verdaccio background service to test-runner-production and vitest-integration jobs.Both jobs use
prepare-sandbox(which depends onrun-registrytarget) but lack the Verdaccio background service step that all other sandbox-related jobs have. Without Verdaccio running,prepare-sandboxwill fail when attempting to access the local registry.Add before the
Install sandbox dependenciesstep in both jobs:- run: background: true command: | cd code yarn local-registry --open name: Verdaccio[scratchpad_end] -->
Also applies to: 1095-1099
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (29)
.circleci/config.yml(40 hunks).circleci/src/jobs/bench-packages.yml(2 hunks).circleci/src/jobs/bench-sandboxes.yml(1 hunks).circleci/src/jobs/build.yml(2 hunks).circleci/src/jobs/check-sandboxes.yml(1 hunks).circleci/src/jobs/check.yml(1 hunks).circleci/src/jobs/chromatic-internal-storybook.yml(1 hunks).circleci/src/jobs/chromatic-sandboxes.yml(1 hunks).circleci/src/jobs/coverage.yml(1 hunks).circleci/src/jobs/create-sandboxes.yml(2 hunks).circleci/src/jobs/e2e-dev.yml(2 hunks).circleci/src/jobs/e2e-production.yml(2 hunks).circleci/src/jobs/e2e-ui-vitest-3.yml(2 hunks).circleci/src/jobs/e2e-ui.yml(2 hunks).circleci/src/jobs/knip.yml(1 hunks).circleci/src/jobs/lint.yml(1 hunks).circleci/src/jobs/pretty-docs.yml(2 hunks).circleci/src/jobs/script-checks.yml(2 hunks).circleci/src/jobs/smoke-test-sandboxes.yml(1 hunks).circleci/src/jobs/stories-tests.yml(1 hunks).circleci/src/jobs/test-init-empty-windows.yml(2 hunks).circleci/src/jobs/test-init-empty.yml(1 hunks).circleci/src/jobs/test-init-features.yml(1 hunks).circleci/src/jobs/test-portable-stories.yml(2 hunks).circleci/src/jobs/test-runner-dev.yml(1 hunks).circleci/src/jobs/test-runner-production.yml(2 hunks).circleci/src/jobs/test-yarn-pnp.yml(1 hunks).circleci/src/jobs/unit-tests.yml(2 hunks).circleci/src/jobs/vitest-integration.yml(2 hunks)
🚧 Files skipped from review as they are similar to previous changes (7)
- .circleci/src/jobs/bench-sandboxes.yml
- .circleci/src/jobs/test-runner-production.yml
- .circleci/src/jobs/e2e-production.yml
- .circleci/src/jobs/check-sandboxes.yml
- .circleci/src/jobs/vitest-integration.yml
- .circleci/src/jobs/chromatic-sandboxes.yml
- .circleci/src/jobs/pretty-docs.yml
🧰 Additional context used
🧠 Learnings (13)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Applied to files:
.circleci/src/jobs/stories-tests.yml.circleci/src/jobs/script-checks.yml.circleci/config.yml.circleci/src/jobs/build.yml.circleci/src/jobs/unit-tests.yml.circleci/src/jobs/e2e-ui.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'cd code && yarn storybook:ui' to start the development server for testing UI changes on http://localhost:6006/
Applied to files:
.circleci/src/jobs/stories-tests.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Applied to files:
.circleci/src/jobs/smoke-test-sandboxes.yml.circleci/src/jobs/create-sandboxes.yml.circleci/src/jobs/e2e-dev.yml.circleci/config.yml
📚 Learning: 2025-11-24T17:49:31.838Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
Applied to files:
.circleci/src/jobs/e2e-ui-vitest-3.yml.circleci/config.yml.circleci/src/jobs/unit-tests.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Compile packages with 'yarn task --task compile' after making code changes before testing
Applied to files:
.circleci/src/jobs/check.yml.circleci/config.yml.circleci/src/jobs/build.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to .yarnrc.yml : Use Yarn 4.9.1 as the package manager
Applied to files:
.circleci/src/jobs/check.yml.circleci/config.yml.circleci/src/jobs/build.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Run 'yarn task --task check' to verify TypeScript types across all packages before submitting PRs
Applied to files:
.circleci/src/jobs/check.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.test.{ts,tsx} : Use 'yarn vitest run --coverage <test-file>' to run tests with coverage reports and aim for 75%+ coverage of statements/lines
Applied to files:
.circleci/src/jobs/script-checks.yml.circleci/config.yml.circleci/src/jobs/unit-tests.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to README.md : Update relevant README files for significant changes
Applied to files:
.circleci/config.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.test.{ts,tsx} : Focus test coverage on all branches, conditions, edge cases, error paths, and different input variations
Applied to files:
.circleci/config.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to .nvmrc : Use Node.js 22.16.0 as specified in .nvmrc
Applied to files:
.circleci/config.yml
📚 Learning: 2025-10-02T09:22:13.215Z
Learnt from: JReinhold
Repo: storybookjs/storybook PR: 32607
File: code/package.json:243-243
Timestamp: 2025-10-02T09:22:13.215Z
Learning: The Storybook repository uses Yarn v^4 (any 4.x version) as the package manager, configured via .yarnrc.yml and package.json packageManager field. Specific patch versions within v4 can be upgraded as needed.
Applied to files:
.circleci/src/jobs/build.yml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: normal
- GitHub Check: Core Unit Tests, windows-latest
- GitHub Check: nx
🔇 Additional comments (23)
.circleci/src/jobs/smoke-test-sandboxes.yml (1)
14-15: Verify test results output path after workspace relocation.The workspace is now attached to
/tmp, andstore_test_resultsis configured to read fromstorybook/test-results. However, the path is relative to the working_directory. Ensure that theyarn task --task smoke-testcommand (with the--junitflag) writes results to the expected location at runtime. The change fromtest-resultstostorybook/test-resultssuggests a new output directory structure, but this needs verification to ensure CircleCI can locate and collect the test artifacts.Also applies to: 22-22
.circleci/src/jobs/test-portable-stories.yml (2)
2-3: Executor resource class upgrade.The executor class was upgraded from
mediumtomedium+, aligning with the broader resource allocation adjustments in this PR refactor.
12-13: Workspace path update consistent with PR scope.The workspace attachment point has been updated to
/tmp, following the monorepo root refactoring pattern across all CircleCI jobs..circleci/src/jobs/script-checks.yml (2)
6-7: Workspace attachment location updated.Workspace is now attached to
/tmpinstead of the current directory, consistent with the monorepo root refactoring.
20-24: Verify test result output path alignment with store_test_results.The
store_test_resultspath is relative to the working directory. The test runs fromcd scriptscontext and likely outputs toscripts/junit.xml, butstore_test_resultsis configured to look instorybook/scripts/junit.xml. Ensure theyarn test --coveragecommand is configured to output results tostorybook/scripts/junit.xmlor adjust thestore_test_resultspath accordingly. This mismatch could cause test results to not be collected by CircleCI..circleci/src/jobs/test-init-empty-windows.yml (1)
11-12: Workspace relocation for Windows jobs.The workspace attachment has been moved to
/tmp, and subsequentcd codecommands will correctly resolve relative to the new workspace root. This maintains compatibility with the monorepo refactoring..circleci/src/jobs/test-yarn-pnp.yml (1)
8-9: Workspace path updated consistently.The workspace attachment point is now
/tmp, aligning with the monorepo root refactoring across CircleCI jobs..circleci/src/jobs/chromatic-internal-storybook.yml (1)
11-12: Verify Chromatic output is in JUnit format for test result storage.The job runs
yarn storybook:ui:chromaticfrom thecode/directory, andstore_test_resultsis configured tostorybook/test-results. However, Chromatic primarily produces snapshot diff artifacts rather than JUnit test results. Verify that this step should usestore_test_results(intended for JUnit XML test metadata) rather thanstore_artifacts(for general build artifacts). If Chromatic does generate JUnit output, confirm it writes to thestorybook/test-resultsdirectory.Also applies to: 16-16, 18-19
.circleci/src/jobs/lint.yml (1)
8-9: Workspace path updated for monorepo root refactoring.The workspace attachment is now
/tmp, consistent with the broader CircleCI configuration updates in this PR. The lint job correctly changes to thecode/directory from this new root..circleci/src/jobs/bench-packages.yml (1)
9-9: Verify artifact path and workspace layout consistency.The attachment point moved to /tmp and the artifact path now includes
storybook/prefix. Ensure that benchmark scripts generate results atstorybook/bench/packages/results.jsonrelative to the workspace root. If bench scripts still output tobench/packages/results.json, the artifact store step will fail to find the file.Also applies to: 59-59
.circleci/src/jobs/test-runner-dev.yml (1)
15-15: Verify test results output path alignment.The test results store path is now
storybook/test-results, but the test-runner-dev task is invoked with--junitflag. Ensure the test runner is configured to output JUnit results tostorybook/test-results(or that the prefix is correctly applied by the task runner). Misalignment will cause the artifact store to fail silently, losing test results.Also applies to: 22-22
.circleci/src/jobs/coverage.yml (1)
8-10: LGTM.The workspace attachment change is consistent with the broader CI refactoring. Codecov upload should work as expected once the workspace is properly attached.
.circleci/src/jobs/test-init-empty.yml (1)
15-15: Verify directory navigation logic with /tmp workspace root.The workspace is attached to
/tmp, and subsequentcd codecommands navigate to/tmp/code. However, the test steps then usecd ..(lines 35, 64, 94, 124) expecting to navigate to the parent directory. From/tmp/code,cd ..goes to/tmp, which is the workspace root—this appears correct. However, verify that:
- The
/tmp/codedirectory exists after workspace attachment (i.e., the workspace includes thecode/subdirectory)- The
cd ..commands are expected to land in/tmp, not elsewhere- Test artifacts are correctly created in
/tmp/empty-*directoriesAlso applies to: 25-40
.circleci/src/jobs/check.yml (1)
11-19: Verify yarn cache key strategy after monorepo root refactor.The Yarn cache key has been simplified from checking both
code/yarn.lockandscripts/yarn.lockto checking only the rootyarn.lock. This is a significant change. Verify:
- The root
yarn.locknow includes all dependencies for the entire workspace (including code/ and scripts/ subdirectories)- Any separate lockfiles in code/ and scripts/ are no longer used or have been migrated to the root lockfile
- The cache invalidation strategy is correct for the new monorepo structure
Additionally, the explicit
yarncommand (line 18) beforeyarn task --task compilemay be redundant if the task runner already handles dependency installation. Clarify whether this is necessary or can be removed..circleci/src/jobs/test-init-features.yml (1)
8-10: Consistent with workspace refactoring pattern.The workspace attachment to
/tmpis consistent with other jobs. Verify that thecd codeand subsequentcd ..navigation works correctly as described in test-init-empty.yml review comment..circleci/src/jobs/e2e-ui-vitest-3.yml (1)
9-9: Verify artifact paths for nested test-storybooks directory.The artifact paths use
storybook/test-storybooks/...prefix, which appears to be the new convention. Ensure that:
- The playwright-e2e task outputs test results to
storybook/test-results(line 21, relative to workspace root)- The portable-stories test directory outputs to
storybook/test-storybooks/portable-stories-kitchen-sink/react-vitest-3/test-results/(line 23)- These paths exist after task execution, or the artifact store will silently fail
The double prefixing (e.g.,
storybook/test-storybooks/...in the artifact path) is consistent with the pattern but should be verified to match actual output locations.Also applies to: 21-21, 23-23
.circleci/src/jobs/knip.yml (1)
8-10: LGTM.The workspace attachment change is consistent with the broader refactoring. The
cd codecommand should work correctly once the workspace structure at/tmpis verified (see verification requests in other files)..circleci/src/jobs/e2e-ui.yml (1)
9-9: Verify artifact paths resolve correctly with workspace root at /tmp.With
attach_workspace at: /tmp, the artifact paths (storybook/test-results, storybook/test-storybooks/...) assume files are located at /tmp/storybook/. Ensure the build job'spersist_to_workspacestep correctly places these artifacts.[scratchpad_end] -->
Also applies to: 21-21, 23-23
.circleci/src/jobs/create-sandboxes.yml (1)
34-34: Verifyyarn get-sandbox-dirutility is available and works correctly.Line 34 introduces a new utility command (
yarn get-sandbox-dir --template $TEMPLATE) that resolves sandbox paths dynamically. This replaces the hardcodedsandbox/prefix pattern. Ensure this command:
- Is properly exported/defined in the root yarn scripts
- Respects the
SANDBOX_ROOTenvironment variable (defaults to../storybook-sandboxes)- Returns the correct path for the given template
[scratchpad_end] -->
Also applies to: 41-43
.circleci/src/jobs/e2e-dev.yml (1)
15-15: Verify e2e-dev does not require Verdaccio background service.Unlike bench-sandboxes, check-sandboxes, chromatic-sandboxes, and e2e-production jobs, the e2e-dev job doesn't include a Verdaccio background service step. Confirm this is intentional and that e2e-dev tasks don't depend on a local npm registry.
[scratchpad_end] -->
Also applies to: 20-20
.circleci/src/jobs/build.yml (1)
11-11: Verify single yarn.lock is sufficient for cache invalidation.The cache keys have been simplified from multi-file checksums (code/yarn.lock + scripts/yarn.lock) to a single root
yarn.lock. This assumes all workspace dependencies are locked by the root yarn.lock file. Confirm that:
- Root yarn.lock includes entries for all code/ and scripts/ dependencies
- No separate lock files exist that could be modified without updating the root lock
[scratchpad_end] -->
Also applies to: 29-29
.circleci/config.yml (2)
320-334: Verify chromatic-sandboxes sandbox handling logic.The chromatic-sandboxes job uses a unique pattern that differs from other sandbox jobs:
- Line 324: Copies /tmp/storybook-sandboxes into /tmp/storybook/sandbox
- Lines 326, 329: Overrides STORYBOOK_SANDBOX_ROOT=./sandbox to use the local copy
This differs from the standard pattern used by bench, check, e2e-dev, e2e-production jobs which directly use
yarn get-sandbox-dir --template $TEMPLATE.Clarify:
- Why does chromatic require this copy+override pattern?
- Is this intentional to isolate chromatic from the shared sandboxes?
- Should the override be STORYBOOK_SANDBOX_ROOT=./sandbox or ../sandbox?
[scratchpad_end] -->
202-202: Verify Yarn cache is warmed before compile tasks.Cache restoration (line 202, 252) happens before compilation, but now explicit
yarninvocations are added in compile steps (lines 206, 256). Confirm this is the intended behavior:
- Is the explicit
yarnmeant to update the lock file if dependencies changed?- Or should it reinstall node_modules from cache?
- This differs from the previous pattern and may cause cache redundancy.
[scratchpad_end] -->
Also applies to: 220-220, 252-252, 256-256
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (2)
.circleci/config.yml (2)
1027-1032: Add missing Verdaccio background service to test-runner-production.The job installs sandbox dependencies (line 1031) which triggers
prepare-sandbox, a task that explicitly depends onrun-registryper nx.json. However, unlike all other sandbox-preparation jobs (bench-sandboxes, check-sandboxes, chromatic-sandboxes, e2e-dev, e2e-production), this job lacks the Verdaccio background service, causingprepare-sandboxto fail when accessing the local registry.- attach_workspace: at: /tmp + - run: + background: true + command: | + cd code + yarn local-registry --open + name: Verdaccio + - run: + command: | + cd code + yarn wait-on tcp:127.0.0.1:6001 + yarn wait-on tcp:127.0.0.1:6002 + name: Wait on Verdaccio - run: command: | sudo corepack enable
1100-1105: Add missing Verdaccio background service to vitest-integration.The job installs sandbox dependencies (line 1104) which triggers
prepare-sandbox, a task that explicitly depends onrun-registryper nx.json. However, unlike all other sandbox-preparation jobs (bench-sandboxes, check-sandboxes, chromatic-sandboxes, e2e-dev, e2e-production), this job lacks the Verdaccio background service, causingprepare-sandboxto fail when accessing the local registry.- attach_workspace: at: /tmp + - run: + background: true + command: | + cd code + yarn local-registry --open + name: Verdaccio + - run: + command: | + cd code + yarn wait-on tcp:127.0.0.1:6001 + yarn wait-on tcp:127.0.0.1:6002 + name: Wait on Verdaccio - run: command: | sudo corepack enable
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
.circleci/config.yml(40 hunks)scripts/tasks/build.ts(3 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- scripts/tasks/build.ts
🧰 Additional context used
🧠 Learnings (8)
📓 Common learnings
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Use 'yarn task --task sandbox --template <framework-template>' to generate sandbox test environments for different framework combinations
Applied to files:
.circleci/config.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Compile packages with 'yarn task --task compile' after making code changes before testing
Applied to files:
.circleci/config.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: The Storybook repository is a TypeScript/React monorepo with main codebase in 'code/' directory and tooling in 'scripts/' directory
Applied to files:
.circleci/config.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.test.{ts,tsx} : Use 'yarn vitest run --coverage <test-file>' to run tests with coverage reports and aim for 75%+ coverage of statements/lines
Applied to files:
.circleci/config.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to **/*.test.{ts,tsx} : Focus test coverage on all branches, conditions, edge cases, error paths, and different input variations
Applied to files:
.circleci/config.yml
📚 Learning: 2025-11-24T17:49:47.659Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-11-24T17:49:47.659Z
Learning: Applies to .nvmrc : Use Node.js 22.16.0 as specified in .nvmrc
Applied to files:
.circleci/config.yml
📚 Learning: 2025-11-24T17:49:31.838Z
Learnt from: CR
Repo: storybookjs/storybook PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-24T17:49:31.838Z
Learning: Applies to code/vitest.workspace.ts : Vitest configuration is centralized in `code/vitest.workspace.ts` for workspace setup
Applied to files:
.circleci/config.yml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: normal
- GitHub Check: nx
- GitHub Check: Core Unit Tests, windows-latest
🔇 Additional comments (1)
.circleci/config.yml (1)
325-328: Verify the sandbox copy and STORYBOOK_SANDBOX_ROOT override logic.The chromatic-sandboxes job copies
/tmp/storybook-sandboxesto/tmp/storybook/sandbox(line 326) and then overridesSTORYBOOK_SANDBOX_ROOT=./sandboxwhen callingget-sandbox-dir(line 328). Confirm this pattern is correct: does the copy operation actually create the expected directory structure, and why is the override necessary here when other jobs reference the default sandbox location?
3 participants
Closes #
What I did
This refactors the root of yarn monorepo from
./codeto the git root..Sandboxes still need to be tested in isolation, so by default they now live in:
../storybook-sandboxesTo get the old behavior use
SANDBOX_ROOT=./sandboxin your commands.This is not recommended.
You will still see that the
./sandboxdirectory exists.This is because NX outputs must live in the monorepo.
This directory is not used for any ci tests.
The real sandbox lives in process.env.SANDBOX_ROOT
NX improvements
Because of this new setup, we can fix a lot of issues we had with NX.
NX should now always invalidate compile and check steps correctly.
This is because NX can now take the
scriptsinto account in the project graph.Changes in scripts should invalidate every tasks, as every task depends on build tooling in scipts.
NX as the task runner experiment
We have implemented a custom task runner that can be used with
yarn task.I have copied our task runner to NX. This can potentially give great improvements.
For example, we can start using NX cloud agents.
This work is all experimental, and should not impact anyones workflow atm.
You can check the nx github action step though, and see if it is green.
It should be a lot faster than circle ci.
It doesn't have to be green to merge your PR though. But please let me know, so I can fix flake in nx agents.
Use NX locally
If you want try using nx locally, this is how it works:
All yarn task tasks, have the same name in nx.
Sandbox command can only be used with -c production.
In a feature PR, I will also allow NX to be used without verdaccio.
Checklist for Contributors
Testing
The changes in this PR are covered in the following automated tests:
Manual testing
This section is mandatory for all contributions. If you believe no manual test is necessary, please state so explicitly. Thanks!
Documentation
MIGRATION.MD
Checklist for Maintainers
When this PR is ready for testing, make sure to add
ci:normal,ci:mergedorci:dailyGH label to it to run a specific set of sandboxes. The particular set of sandboxes can be found incode/lib/cli-storybook/src/sandbox-templates.tsMake sure this PR contains one of the labels below:
Available labels
bug: Internal changes that fixes incorrect behavior.maintenance: User-facing maintenance tasks.dependencies: Upgrading (sometimes downgrading) dependencies.build: Internal-facing build tooling & test updates. Will not show up in release changelog.cleanup: Minor cleanup style change. Will not show up in release changelog.documentation: Documentation only changes. Will not show up in release changelog.feature request: Introducing a new feature.BREAKING CHANGE: Changes that break compatibility in some way with current major version.other: Changes that don't fit in the above categories.🦋 Canary release
This PR does not have a canary release associated. You can request a canary release of this pull request by mentioning the
@storybookjs/coreteam here.core team members can create a canary release here or locally with
gh workflow run --repo storybookjs/storybook publish.yml --field pr=<PR_NUMBER>Summary by CodeRabbit
Build & Infrastructure
Configuration
Sandbox Management
Projects & Tooling
Docs
Tests
✏️ Tip: You can customize this high-level summary in your review settings.