Skip to content

Latest commit

 

History

History
212 lines (160 loc) · 5.95 KB

README.md

File metadata and controls

212 lines (160 loc) · 5.95 KB

Detect Circular Dependencies

detect-circular-deps detect-circular-deps David David

Detect Circular Dependencies problems that occurs while requiring a module

Circular Dependencies maybe necessary in some applications and it's not easy to avoid it.

This tools helps you to find Circular Dependencies that cause problems in your application.

Installation

npm i detect-circular-deps -g

What the problems this tool could detect?

Empty exports

Suppose you have two modules

a.js

const b = require('./b');

module.exports = {
  name: 'a',
};

b.js

const a = require('./a'); // a is empty {}

module.exports = {
  name: 'b',
};

And your entrypoint trying to

require("./a.js");

As you can see, when you require a.js, the a.js file is required again in b.js file and it will return empty object and that definitely will cause a problem in your application.

We can name this situation: alwaysEmptyExports

You can solve this issue by moving module.exports above the require statement

a.solved.js

module.exports = {
  name: 'a',
};

const b = require('./b.solved');

b.solved.js

module.exports = {
  name: 'b',
};

const a = require('./a.solved');

Missing Property

Suppose

a.js

const b = require('./b');

exports.name = 'a';

b.js

const a = require('./a');

// console.log(a.name); // = undefined
process.nextTick(() => {
  console.log(a.name); // = a
});

exports.name = 'b';

When you use exports. instead of module.exports, the exports are updated to the dependants but in the next tick.

If you need to detect that situation, we name that emptySyncAccess

If your file tried to access a property of another module in a circular dependency situation and could not get it (uncomment the third line of b.js), we name that: missingProperty

CLI

  Usage: detect-circular-deps [options] <file...>

  Options:

    -V, --version               output the version number
    -p, --problems              Report CD. that causing problems (Default)
    -c, --circular              Report all Circular Dependencies.
    -e, --always-empty-exports  Report CD. which its exports are always empty even when it's async-accessed after requiring (Causes Problems)
    -s, --empty-sync-access     Report CD. which its exports are empty only when it is sync-accessed after requiring. (May causes problems in future)
    -m, --missing-properties    Report CD. which some of the properties of its exports was sync-accessed after requiring but not found (Causes Problems)
    -d, --debug                 Print debugging messages
    -h, --help                  output usage information

Examples

Detect problems

detect-circular-deps examples/always-empty/a.js
✗  The exports of examples/always-empty/a is empty when it is required at examples/always-empty/b (It causes problems)
    Circular Path: examples/always-empty/a > examples/always-empty/b
detect-circular-deps examples/always-empty/a.solved.js
✓ No Problems for Circular Dependencies found! [filtered]

Detect any circular dependencies even it doesn't cause any problems

detect-circular-deps examples/always-empty/a.solved.js -c
✗  Circular requiring of examples/always-empty/a.solved
    Circular Path: examples/always-empty/a.solved > examples/always-empty/b.solved

Support es6 import/export

Now you can run the tool on es6 files

You need to add the flag --es6 to enable es6 modules.

The package babel-register is used to support es6 at runtime but you need to add .babelrc file to your working directory or any upper directory.

Example:

detect-circular-deps --es6 examples/es6/always-empty/a.js
Start detecting entrypoint: examples/es6/always-empty/a.js
✗  The exports of examples/es6/always-empty/a is empty when it is required at examples/es6/always-empty/b (It causes problems)
    Circular Path: examples/es6/always-empty/a > examples/es6/always-empty/b

Wildcard matching

This tool uses glob to match files using the patterns the shell uses but u need to qoute the pattern so that your terminal don't override it An example for running the tool on all files in this directory and sub directories

detect-circular-deps '*/**/*.js'

API

Functions

.problems({callback})

.circular({callback})

.alwaysEmptyExports({callback})

.emptySyncAccess({callback})

.missingProperties({callback})

All methods return a promise, also you can pass callback in the args

See examples/api for more info

Add to your tests

You can use the API in your tests to ensure that there is no circular dependencies in your application.

const detectCircularDeps = require('../index'); // replace with require("detect-circular-deps")

describe('Circular Dependencies Issues', () => {
  it('Should not cause a problem', async () => {
    const promise = detectCircularDeps.problems();
    require('../always-empty/a'); // you must require your module before the promise being resolved
    const results = await promise;
    if (results[0]) {
      throw new Error(results[0].message);
    }
  });
  it('Should not cause a problem', async () => {
    const promise = detectCircularDeps.problems();
    require('../always-empty/a.solved');
    const results = await promise;
    if (results[0]) {
      throw new Error(results[0].message);
    }
    process.exit(0);
  });
});

Limitations

This tool works only with node.js environment as it depends on the built-in node.js module package

I aim to make it works with webpack and browser applications.