Error in circular dependency detection ?

Prerequisites

Please answer the following questions for yourself before submitting an issue. YOU MAY DELETE THE PREREQUISITES SECTION.

• I am running the latest version
• I checked the documentation (nx.dev) and found no answer
• I checked to make sure that this issue has not already been filed
• I’m reporting the issue to the correct repository (not related to React, Angular or any dependency)

Expected Behavior

External packages with names that starts like an library name in the nx workspace should not be treated like the this lib.

Current Behavior

I have four libs in my monorepo: common, desktop, concept and basiskomponenten. In this monorepo I’m consuming a library “@inova/desktop-api-service” (from a private npm registry). Since I updated from @nrwl/workspace@8.12.11 to the latest I have lint errors where I’m importing objects from “@inova/desktop-api-service”.

Failure Information (for bugs)

ERROR: C:/.../InovaFrontend/libs/common/src/lib/app-context/culture-context/culture-context.service.ts[10, 1]: Circular dependency between "common" and "desktop" detected

nx.json:

lint.json:

tsconfig.json:

Context

Please provide any relevant information about your setup:

>  NX  Report complete - copy this into the issue template

  @nrwl/angular : 9.2.4
  @nrwl/cli : 9.2.4
  @nrwl/cypress : 9.2.4
  @nrwl/eslint-plugin-nx : Not Found
  @nrwl/express : Not Found
  @nrwl/jest : 9.2.4
  @nrwl/linter : Not Found
  @nrwl/nest : Not Found
  @nrwl/next : Not Found
  @nrwl/node : Not Found
  @nrwl/react : Not Found
  @nrwl/schematics : Not Found
  @nrwl/tao : 9.2.4
  @nrwl/web : Not Found
  @nrwl/workspace : 9.2.4
  typescript : 3.8.3

[FrozenPandaz]

Thanks for the unit test case, that was super for fixing the issue!

[admir86]

@FrozenPandaz is the unit test as repro not sufficient? do you need something more? or can you remove the “blocked: repro needed” label? 😃