Circular dependencies with TypeScript

[jwrosewell]

1.4.0

Core

When transpiling in TypeScript the following warning is generated.

(!) Circular dependencies
..\node_modules@iabtcf\core\lib\encoder\index.js -> ..\node_modules@iabtcf\core\lib\encoder\SegmentEncoder.js -> ..\node_modules@iabtcf\core\lib\encoder\field\index.js -> ..\node_modules@iabtcf\core\lib\encoder\field\FieldEncoderMap.js -> ..\node_modules@iabtcf\core\lib\encoder\field\VendorVectorEncoder.js -> ..\node_modules@iabtcf\core\lib\encoder\index.js
..\node_modules@iabtcf\core\lib\encoder\index.js -> ..\node_modules@iabtcf\core\lib\encoder\SegmentEncoder.js -> ..\node_modules@iabtcf\core\lib\encoder\field\index.js -> ..\node_modules@iabtcf\core\lib\encoder\field\FieldEncoderMap.js -> ..\node_modules@iabtcf\core\lib\encoder\field\VendorVectorEncoder.js -> D:\Workspace\paf-mvp-implementation\node_modules@iabtcf\core\lib\encoder\index.js?commonjs-proxy -> ..\node_modules@iabtcf\core\lib\encoder\index.js

The module is included with the following import.

import { TCString, TCModel, SegmentEncoder, Segment } from '@iabtcf/core/lib';

Do others have the same issue?

[apfritts]

yea, having the same issue. Did you have a chance to figure out how to fix it @jwrosewell?

[jwrosewell]

Thanks for confirming. Not yet looked at solving it. I'm concerned it's interfering with treeshaking and resulting in larger than required modules.

[jwrosewell]

Issues looks like it's related to this line.

iabtcf-es/modules/core/src/encoder/field/VendorVectorEncoder.ts

Line 2 in 49972aa

import {BitLength} from '../';

I don't have the time to get a build environment working to make a PR. Others?

[marco-prontera]

Hi @jwrosewell try the latest version 1.5.3 and tell me if there is still this problem.
Read CONTRIBUTING.md to know how to start with the project ;)

[krasnoperov]

In my case it is still there:

Imports:

import type { GVL as GVLType, TCModel as TCModelType } from '@iabtcf/core'
import { TCModel, TCString, GVL } from '@iabtcf/core'
import { CmpApi } from '@iabtcf/cmpapi'
import cmpstub from '@iabtcf/stub'

Build output:

(!) Circular dependencies
node_modules/@iabtcf/core/lib/mjs/encoder/index.js -> node_modules/@iabtcf/core/lib/mjs/encoder/SegmentEncoder.js -> node_modules/@iabtcf/core/lib/mjs/encoder/field/index.js -> node_modules/@iabtcf/core/lib/mjs/encoder/field/FieldEncoderMap.js -> node_modules/@iabtcf/core/lib/mjs/encoder/field/VendorVectorEncoder.js -> node_modules/@iabtcf/core/lib/mjs/encoder/index.js
node_modules/@iabtcf/cmpapi/lib/mjs/response/index.js -> node_modules/@iabtcf/cmpapi/lib/mjs/response/Disabled.js -> node_modules/@iabtcf/cmpapi/lib/mjs/response/Response.js -> node_modules/@iabtcf/cmpapi/lib/mjs/CmpApiModel.js -> node_modules/@iabtcf/cmpapi/lib/mjs/EventListenerQueue.js -> node_modules/@iabtcf/cmpapi/lib/mjs/command/GetTCDataCommand.js -> node_modules/@iabtcf/cmpapi/lib/mjs/response/index.js

[NevilleS]

Yup, we see this too!

@iabtechlabtcf/core: version 1.5.10
@iabtechlabtcf/cmpapi: version 1.5.8

[HeinzBaumann]

-> @lamrowena

[elliot-nelson]

It looks like this is still an issue in 2024 -- with @iabtcf/core@1.5.6 (latest version I see in NPM).

[HeinzBaumann]

Latest library is here: "@iabtechlabtcf/core": "^1.5.20". Is this still reproducible?

[sewolf]

Yes, Rollup still shows this warnings

(!) Circular dependencies node_modules/.pnpm/@iabtechlabtcf+cmpapi@1.5.21_@iabtechlabtcf+core@1.5.21/node_modules/@iabtechlabtcf/cmpapi/lib/mjs/response/index.js -> node_modules/.pnpm/@iabtechlabtcf+cmpapi@1.5.21_@iabtechlabtcf+core@1.5.21/node_modules/@iabtechlabtcf/cmpapi/lib/mjs/response/Disabled.js -> node_modules/.pnpm/@iabtechlabtcf+cmpapi@1.5.21_@iabtechlabtcf+core@1.5.21/node_modules/@iabtechlabtcf/cmpapi/lib/mjs/response/Response.js -> node_modules/.pnpm/@iabtechlabtcf+cmpapi@1.5.21_@iabtechlabtcf+core@1.5.21/node_modules/@iabtechlabtcf/cmpapi/lib/mjs/CmpApiModel.js -> node_modules/.pnpm/@iabtechlabtcf+cmpapi@1.5.21_@iabtechlabtcf+core@1.5.21/node_modules/@iabtechlabtcf/cmpapi/lib/mjs/EventListenerQueue.js -> node_modules/.pnpm/@iabtechlabtcf+cmpapi@1.5.21_@iabtechlabtcf+core@1.5.21/node_modules/@iabtechlabtcf/cmpapi/lib/mjs/command/GetTCDataCommand.js -> node_modules/.pnpm/@iabtechlabtcf+cmpapi@1.5.21_@iabtechlabtcf+core@1.5.21/node_modules/@iabtechlabtcf/cmpapi/lib/mjs/response/index.js node_modules/.pnpm/@iabtechlabtcf+core@1.5.21/node_modules/@iabtechlabtcf/core/lib/mjs/encoder/index.js -> node_modules/.pnpm/@iabtechlabtcf+core@1.5.21/node_modules/@iabtechlabtcf/core/lib/mjs/encoder/SegmentEncoder.js -> node_modules/.pnpm/@iabtechlabtcf+core@1.5.21/node_modules/@iabtechlabtcf/core/lib/mjs/encoder/field/index.js -> node_modules/.pnpm/@iabtechlabtcf+core@1.5.21/node_modules/@iabtechlabtcf/core/lib/mjs/encoder/field/FieldEncoderMap.js -> node_modules/.pnpm/@iabtechlabtcf+core@1.5.21/node_modules/@iabtechlabtcf/core/lib/mjs/encoder/field/VendorVectorEncoder.js -> node_modules/.pnpm/@iabtechlabtcf+core@1.5.21/node_modules/@iabtechlabtcf/core/lib/mjs/encoder/index.js