chore: use nx graph instead of manually defined devDependencies in package.json files across whole workspace #35611

Hotell · 2026-01-02T13:56:01Z

Previous Behavior

New Behavior

This pull request removes manually defined devDependencies from the package.json files across multiple application packages in the workspace. The change is part of a shift to using the Nx dependency graph to manage development dependencies, simplifying package maintenance and reducing duplication. No production dependencies or application logic are affected.

NOTE: actual maintenance boost diff

with changefiles ignored

Details:

1. Removed all manually specified `devDependencies` from the `package.json` files

all devDeps are removed unless they are not 3rd party packages or packages not adhering to single version policy
generators are updated to reflect this change

2. `scripts-api-extractor` dependency resolution is now driven via `input` cache invalidation rather than hard dependency on the project.

Before:

if scripts-api-extractor is changed all its dependants were affected and triggered all targets

After:

if scripts-api-extractor is changed all its dependants were affected and will trigger only contextually related target. In our case generate-api and build

3. `jest-serializer-merge-styles` dependency resolution

is now explicitly being loaded via node require.resolve so Nx can infer it as project dependency instead of manually specifing in package.json / implicitDependencies

4. v9 stories lint update

removed invalid 'import/no-extraneous-dependencies' as dep graph is now created purely by Nx

5. removing `"@types/react-addons-test-utils`

after cleanup, yarn started to fail on CI which was caused by v1 and aliasing issue, this is resolved as part of this PR

Related Issue(s)

Fixes #

Hotell · 2026-01-02T15:48:24Z

packages/charts/react-charting/package.json

  "devDependencies": {
-    "@fluentui/eslint-plugin": "*",
-    "@fluentui/react": "*",
-    "@types/react-addons-test-utils": "0.14.18",


unused 3rd party package. removing

github-actions · 2026-01-02T16:15:31Z

📊 Bundle size report

✅ No changes found

apps/chart-docsite/package.json

…ache to trigger affected tree rather than hard dependency on that package

…kage.json files across whole workspace

…r test and define it as implicit dep

…elines

…rather than nested dictionary which doesnt work on ci?

… order to properly resolve dependency graph

…ndenies helper

…of using nx implicitDeps to build graph resolution

…d for build/bundle targets

github-actions · 2026-01-05T14:49:51Z

Pull request demo site: URL

… rule from lint configs within stories projects

…from stories project

Hotell · 2026-01-05T17:18:16Z

apps/public-docsite-resources/package.json

  "description": "Fluent UI React local demo app",
  "main": "lib-commonjs/index.js",
  "module": "lib/index.js",
+  "exports": {


these nested path imports are used across ssr-tests and public-docsite ( v8 ), in order to be able to resovle the dep graph correctly by nx we added proper API export maps

Hotell · 2026-01-05T17:19:32Z

nx.json

    "build": {
      "dependsOn": ["^build"],
-      "inputs": ["production", "^production"],
+      "inputs": ["production", "^production", "{workspaceRoot}/scripts/api-extractor/api-extractor.*.json"],


if we change any api-extractor config within our scoped library it will invalidate caches and trigger affected build of all packages that use its definitions without need of providing implicit Dependency

dmytrokirpa

Awesome to see so much of the boilerplate gone, even if implicit staff isn’t really my preference.

Everything looks good to me! Just wanted to double-check - are we dropping import/no-extraneous-dependencies, and is it okay to import transitive dependencies now?

dmytrokirpa · 2026-01-06T11:16:38Z

packages/charts/react-charts/stories/eslint.config.js

  {
    rules: {
      '@typescript-eslint/no-explicit-any': 'warn',
-      'import/no-extraneous-dependencies': [


Should we remove this rule? What would happen if a developer imports a transitive dependency that isn't listed in the package/monorepo's package.json?

Should we remove this rule?

yes ! good catch ty! didn't caught this one as it wasn't failing pipeline. lemme update

What would happen if a developer imports a transitive dependency that isn't listed in the package/monorepo's package.json?

for devDeps or non publishable packages in general we don't care (exempt are applications that can be deployed - aka production deps necessary for docker etc), that's unnecessary overhead as nx will build the graph for us.

we still need to enforce SVP via a workspace sync generator feat: adopt single version policy for monorepo dependencies #21395

packages/react-experiments/package.json

Hotell · 2026-01-06T12:59:15Z

are we dropping import/no-extraneous-dependencies, and is it okay to import transitive dependencies now?

@dmytrokirpa not for publishable packages yet, but we will replace it with nx dep check microsoft/fluentui-contrib#316, this will also enable us to remove lint->build task dependency which will speed up pipeline

github-actions bot added the NX: core label Jan 2, 2026

Hotell force-pushed the nx/use-nx-graph-for-devdeps branch from 3bc38e3 to 5659e41 Compare January 2, 2026 15:23

Hotell changed the title ~~Nx/use nx graph for devdeps~~ chore: use nx graph instead of manually defined devDependencies in package.json files across whole workspace Jan 2, 2026

Hotell commented Jan 2, 2026

View reviewed changes

github-actions bot reviewed Jan 2, 2026

View reviewed changes

apps/chart-docsite/package.json Show resolved Hide resolved

github-actions bot added the CI label Jan 2, 2026

Hotell added 12 commits January 5, 2026 14:44

feat(nx): use scripts-api-extractor package as dependency via input c…

02929a3

…ache to trigger affected tree rather than hard dependency on that package

chore:use nx graph instead of manually defined devDependencies in pac…

171f24b

…kage.json files across whole workspace

chore(workspace-plugin): remove generating devDeps when creating library

c6220c3

chore: remove unused 3rd party deps

f296be0

change files

3d15ce5

chore: fix yarn.lock aliases merge preventing frozen install

a33772c

fix: revert using jest-serializer-merge-styles as build dependency fo…

6ce9950

…r test and define it as implicit dep

ci: remove invalid --exclude proejct definition from main and rit pip…

fbe4de2

…elines

chore(nx): revert api-extractor invalidation to physicall input glob …

1ebc61f

…rather than nested dictionary which doesnt work on ci?

chore(ssr-tests): fix nx graph dependency resolution

00f4180

fix(public-docsite-resource): add deep import used within ssr-test in…

2d8e0d8

… order to properly resolve dependency graph

fix(monorepo): accomodate nx graph resolution for devDeps for getDepe…

5fd1b11

…ndenies helper

Hotell force-pushed the nx/use-nx-graph-for-devdeps branch from 6d68ef2 to 5fd1b11 Compare January 5, 2026 13:44

Hotell added 2 commits January 5, 2026 15:38

chore: use explicit require for jest-serializer-merge-styles instead …

66d9d4e

…of using nx implicitDeps to build graph resolution

fix(public-docsite-resource): add additional export maps that are use…

240cba7

…d for build/bundle targets

Hotell added 3 commits January 5, 2026 17:19

style: apply formatting

af5ca1a

style(stories): remove not needed 'import/no-extraneous-dependencies'…

d25699d

… rule from lint configs within stories projects

fix(workspace-plugin): remove import/no-extraneous-dependencies rule …

b29d417

…from stories project

microsoft-github-policy-service bot added Area: Build System Area: Website labels Jan 5, 2026

Hotell commented Jan 5, 2026

View reviewed changes

Hotell requested review from a team, Jacqueline-ms, Jahnp, Mitch-At-Work, ValentinaKozlova, bigbadcapers, chpalac, dmytrokirpa, dzearing, khmakoto, mainframev, marcosmoura, sopranopillow and spmonahan as code owners January 6, 2026 09:19

AtishayMsft approved these changes Jan 6, 2026

View reviewed changes

dmytrokirpa reviewed Jan 6, 2026

View reviewed changes

dmytrokirpa approved these changes Jan 6, 2026

View reviewed changes

Hotell merged commit 5e24597 into microsoft:master Jan 7, 2026
12 checks passed

Hotell deleted the nx/use-nx-graph-for-devdeps branch January 7, 2026 13:58

Hotell mentioned this pull request Jan 7, 2026

test(scripts-monorepo): udpate snapshot diff that changed re-ordering after switching to nx graph resolution for devDeps #35626

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

chore: use nx graph instead of manually defined devDependencies in package.json files across whole workspace #35611

chore: use nx graph instead of manually defined devDependencies in package.json files across whole workspace #35611

Hotell commented Jan 2, 2026 •

edited

Loading

Uh oh!

Hotell Jan 2, 2026

Uh oh!

github-actions bot commented Jan 2, 2026 •

edited

Loading

Uh oh!

Uh oh!

github-actions bot commented Jan 5, 2026

Uh oh!

Hotell Jan 5, 2026

Uh oh!

Hotell Jan 5, 2026

Uh oh!

dmytrokirpa left a comment

Uh oh!

dmytrokirpa Jan 6, 2026

Uh oh!

Hotell Jan 6, 2026

Uh oh!

Uh oh!

Hotell commented Jan 6, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

chore: use nx graph instead of manually defined devDependencies in package.json files across whole workspace #35611

chore: use nx graph instead of manually defined devDependencies in package.json files across whole workspace #35611

Conversation

Hotell commented Jan 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Previous Behavior

New Behavior

1. Removed all manually specified devDependencies from the package.json files

2. scripts-api-extractor dependency resolution is now driven via input cache invalidation rather than hard dependency on the project.

3. jest-serializer-merge-styles dependency resolution

4. v9 stories lint update

5. removing "@types/react-addons-test-utils

Related Issue(s)

Uh oh!

Hotell Jan 2, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Jan 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📊 Bundle size report

Uh oh!

Uh oh!

github-actions bot commented Jan 5, 2026

Uh oh!

Hotell Jan 5, 2026

Choose a reason for hiding this comment

Uh oh!

Hotell Jan 5, 2026

Choose a reason for hiding this comment

Uh oh!

dmytrokirpa left a comment

Choose a reason for hiding this comment

Uh oh!

dmytrokirpa Jan 6, 2026

Choose a reason for hiding this comment

Uh oh!

Hotell Jan 6, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Hotell commented Jan 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Hotell commented Jan 2, 2026 •

edited

Loading

1. Removed all manually specified `devDependencies` from the `package.json` files

2. `scripts-api-extractor` dependency resolution is now driven via `input` cache invalidation rather than hard dependency on the project.

3. `jest-serializer-merge-styles` dependency resolution

5. removing `"@types/react-addons-test-utils`

github-actions bot commented Jan 2, 2026 •

edited

Loading

Hotell commented Jan 6, 2026 •

edited

Loading