Checks JS code for pure functions
Purecheck is inspired by this article.
For a function to be pure, it must:
- Have no side effects, i.e.:
- Only local variables should be modified. A function that modifies parameters,
this or variables from other scopes has side effects.
- Only invoke functions that have no side effects.
- Have no side causes, i.e.:
- Depend exclusively on the input parameters. Accessing variables from other scopes depends on side causes.
- Only invoke functions that have no side causes.
throw any error: pure functions do not break the execution flow
- Should always return some value: given that pure functions have no side effects, if they don't return any value, they are totally useless anyway.
Purecheck generates a report listing all scanned functions along with their associated side causes and side effects. It uses Esprima to do the parsing.
npm run build
- If you have installed it via
npm install purecheck:
- If you have cloned the repository:
npm start file_to_check.js
If no file is specified, purecheck will read from the standard input. This allows piping the output of another command, e.g. a transpile step.
-t: purecheck reports errors specifying the line and column of the offending code. When the JS file has tabs, the column position may not be reported correctly, so this parameter can be used to specify the number of spaces used by a tab. The default value is 4 spaces per tab.
purecheck --tabsize 3
Purecheck is still under development. Check the ToDo section below for details.
Rules for pure functions
This list provides more detail about the rules mentioned above:
- Should not have side causes, i.e., should not:
- Read a non-local variable
- Read from
- Should not have side effects, i.e., should not:
- Write to a non-local variable
- Write to a parameter
- Write to
- Should not invoke impure functions
- Should not invoke a function from a blacklist of non-pure functions (pending)
- Alternatively, should only invoke its own pure functions and functions in a whitelist of safe functions (pending)
throw statement is not allowed within pure functions
- Pure functions should explicitly return some value. Otherwise, if they don't have side effects and return nothing, they are useless and their invocation can be replaced by
- Improve CLI
- Read configuration from purecheck.json
- Check for:
- All branches should return some value
- Function blacklist / whitelist
- Test it with big, real-world JS files to ensure it does not crash
- Catch invocation of nested impure functions
- Support ES6-style params: defaults, rest and destructuring
- Check for source maps, and if present, use them to translate error locations. This would provide support for TypeScript (and potentially other similar languages)
- ESLint plugin and rule, so it can be used from the ESLint tool, especially when integrated in editors/IDEs such as Visual Studio Code.
- Improve tab expansion implementation so it adds only the spaces remaining to the next tab position.