NX runs unnecessary tasks when building app while using buildLibsFromSource=true

[mirobo]

Current Behavior

When I'm running a production build of an app with buildLibsFromSource: true, NX will still run all other "build" targets from the buildable libraries because of the target defaults. But if I understand correctly they wouldn't need to be built at all because the app and the libs are built from the source anyway.

According to https://nx.dev/nx-api/angular/executors/dev-server#buildlibsfromsource the default is buildLibsFromSource=true but also having "build" depend on "^build" is the default in nx.json. I guess changing this could improve build times for a lot of people, wouldn't it? In the end each buildable library used @nx/angular:ng-packagr-lite which produces something which is not used by default..

So finally, I removed "dependsOn": ["^build"], in the targetDefaults just to see.. and it still works..

Did I miss something fundamental here?

Edit:
In real life I have 170+ buildable libraries. I thought I'd profit from the incremental build by later setting buildLibsFromSource: false but NX will (or needs to) check if every library is updated (even checking the cache will be slow with so many projects). We have a lot of separate libraries to get more out of NX affected (i.e. by only running affected Cypress component tests).

Expected Behavior

When buildLibsFromSource is true (which it is by default) a prod build shouldn't execute all other depending libraries "build"-target. Or add a hint in the documentation on how it is supposed to work and how to work around it (in a supported manner).

GitHub Repo

https://github.com/mirobo/nx-angular-tailwind

Steps to Reproduce

• git clone https://github.com/mirobo/nx-angular-tailwind.git
• npm i
• npx nx run demo:build:production

Nx Report

Node   : 18.14.0
   OS     : win32-x64
   npm    : 9.3.1
   
   nx                 : 17.2.8
   @nx/js             : 17.2.8
   @nx/jest           : 17.2.8
   @nx/linter         : 17.2.8
   @nx/eslint         : 17.2.8
   @nx/workspace      : 17.2.8
   @nx/angular        : 17.2.8
   @nx/cypress        : 17.2.8
   @nx/devkit         : 17.2.8
   @nx/eslint-plugin  : 17.2.8
   @nrwl/tao          : 17.2.8
   @nx/web            : 17.2.8
   @nx/webpack        : 17.2.8
   typescript         : 5.2.2

Failure Logs

No response

Package Manager Version

No response

Operating System

• macOS
• Linux
• Windows
• Other (Please specify)

Additional Information

No response

[AgentEnder]

If buildLibsFromSource is true, you can override the build targets dependencies by specifying dependsOn: [] in project.json. The default build: { dependsOn: ["^build"]} indeed doesn't make sense for all workspaces, specifically in cases where using a bundler that doesn't reference prebuilt files.

Perhaps the generated angular configuration should overwrite the dependsOn config from nx.json in project.json.

[mirobo]

Thank you very much for your response! I'm not quite sure what kind of projects rely on build-dependencies being run when the default is buildLibsFromSource=true. It would come in handy if you would know how many people use the flag (and how) :D

I guess one could have a UI library which creates some files (properly referenced in "outputs") which are then required by the app (generating/downloading graphics, icons?)... but then again, the project isn't really "built from the source" but currently it would work.

Maybe it's best to update the documentation or to build in some warnings or suggestions on how NX users could potentially improve their build speed? Or announce to think about removing to flag altogether to see how many users will speak up ;-)
Or as you suggested: Override dependsOn config in project.js (or do it internally if buildLibsFromSource is true?)