Home

Awesome

npm version Downloads Build Status code style: prettier Conventional Commits Gitter Chat

<p align="center"> <img src="https://raw.githubusercontent.com/mgechev/codelyzer/master/assets/logo.png" alt="" width="200"> </p>

codelyzer

A set of tslint rules for static code analysis of Angular TypeScript projects.
(If you are using ESLint check out the new angular-eslint repository.)

You can run the static code analyzer over web apps, NativeScript, Ionic, etc.

Vote for your favorite feature here. For more details about the feature request process see this document

How to use?

Angular CLI

Angular CLI has support for codelyzer. In order to validate your code with CLI and the custom Angular specific rules just use:

ng new codelyzer
ng lint

Note that by default all components are aligned with the style guide so you won't see any errors in the console.

Angular Seed

Another project which has out of the box integration with codelyzer is angular-seed. In order to run the linter you should:

# Skip if you've already cloned Angular Seed
git clone https://github.com/mgechev/angular-seed

# Skip if you've already installed all the dependencies of Angular Seed
cd angular-seed && npm i

# Run all the tslint and codelyzer rules
npm run lint

Note that by default all components are aligned with the style guide so you won't see any errors in the console.

Custom Setup

Preset

You can use the tslint-angular preset. All you need is:

npm i tslint-angular

After that create a tslint.json file with the following configuration:

{
  "extends": ["tslint-angular"]
}

Run the linter with:

./node_modules/.bin/tslint -c tslint.json

TSLint will now complain that there are rules which require type checking. In order to fix this, use the -p config option:

./node_modules/.bin/tslint -p tsconfig.json -c tslint.json

Custom Installation

You can easily use codelyzer with your custom setup:

npm i codelyzer tslint @angular/compiler @angular/core

A. Using codelyzer package in PATH

Create the following tslint.json file like:

{
  "extends": ["codelyzer"],
  "rules": {
    "component-class-suffix": true,
    "component-max-inline-declarations": true,
    "component-selector": [true, "element", "sg", "kebab-case"],
    "contextual-lifecycle": true,
    "directive-class-suffix": true,
    "directive-selector": [true, "attribute", "sg", "camelCase"],
    "no-attribute-decorator": true,
    "no-conflicting-lifecycle": true,
    "no-forward-ref": true,
    "no-host-metadata-property": true,
    "no-input-rename": true,
    "no-inputs-metadata-property": true,
    "no-lifecycle-call": true,
    "no-output-native": true,
    "no-output-on-prefix": true,
    "no-output-rename": true,
    "no-outputs-metadata-property": true,
    "no-pipe-impure": true,
    "no-queries-metadata-property": true,
    "no-unused-css": true,
    "prefer-inline-decorator": true,
    "prefer-output-readonly": true,
    "template-banana-in-box": true,
    "template-conditional-complexity": [true, 4],
    "template-cyclomatic-complexity": [true, 5],
    "template-i18n": [true, "check-id", "check-text"],
    "template-no-negated-async": true,
    "template-use-track-by-function": true,
    "use-component-selector": true,
    "use-component-view-encapsulation": true,
    "use-lifecycle-interface": true,
    "use-pipe-transform-interface": true
  }
}

To run TSLint with this setup you can use one of the following alternatives:

  1. Install codelyzer globally npm install -g codelyzer

  2. Run TSLint from a package.json script by adding a script like tslint . to your package.json, similar to:

"scripts": {
  ...
  "lint": "tslint .",
  ...
},

Then run npm run lint

B. Using codelyzer from node_modules directory

Now create the following tslint.json file where your node_modules directory is:

{
  "rulesDirectory": ["node_modules/codelyzer"],
  "rules": {
    "component-class-suffix": true,
    "component-max-inline-declarations": true,
    "component-selector": [true, "element", "sg", "kebab-case"],
    "contextual-lifecycle": true,
    "directive-class-suffix": true,
    "directive-selector": [true, "attribute", "sg", "camelCase"],
    "no-attribute-decorator": true,
    "no-conflicting-lifecycle": true,
    "no-forward-ref": true,
    "no-host-metadata-property": true,
    "no-input-rename": true,
    "no-inputs-metadata-property": true,
    "no-lifecycle-call": true,
    "no-output-native": true,
    "no-output-on-prefix": true,
    "no-output-rename": true,
    "no-outputs-metadata-property": true,
    "no-pipe-impure": true,
    "no-queries-metadata-property": true,
    "no-unused-css": true,
    "prefer-inline-decorator": true,
    "prefer-output-readonly": true,
    "template-banana-in-box": true,
    "template-conditional-complexity": [true, 4],
    "template-cyclomatic-complexity": [true, 5],
    "template-i18n": [true, "check-id", "check-text"],
    "template-no-negated-async": true,
    "template-use-track-by-function": true,
    "use-component-selector": true,
    "use-component-view-encapsulation": true,
    "use-lifecycle-interface": true,
    "use-pipe-transform-interface": true
  }
}

Next you can create a component file in the same directory with name component.ts and the following content:

import { Component } from '@angular/core';

@Component({
  selector: 'codelyzer',
  template: ` <h1>Hello {{ name }}!</h1> `,
})
class Codelyzer {
  name: string = 'World';

  ngOnInit() {
    console.log('Initialized');
  }
}

As last step you can execute all the rules against your code with tslint:

./node_modules/.bin/tslint -c tslint.json component.ts

You should see the following output:

component.ts[4, 13]: The selector of the component "Codelyzer" should have prefix "sg" (https://goo.gl/cix8BY)
component.ts[12, 3]: Implement lifecycle hook interface OnInit for method ngOnInit in class Codelyzer (https://goo.gl/w1Nwk3)
component.ts[9, 7]: The name of the class Codelyzer should end with the suffix Component (https://goo.gl/5X1TE7)

Editor Configuration

Note that you need to have tslint plugin install on your editor.

Codelyzer should work out of the box with Atom but for VSCode you will have to open Code > Preferences > User Settings, and enter the following config:

{
  "tslint.rulesDirectory": "./node_modules/codelyzer",
  "typescript.tsdk": "node_modules/typescript/lib"
}

Now you should have the following result:

VSCode Codelyzer

Enjoy!

Changelog

You can find it here.

Recommended configuration

Below you can find a recommended configuration which is based on the Angular Style Guide.

{
  // The rules component-selector and directive-selector have the following arguments:
  // [ENABLED, "attribute" | "element", "prefix" | ["listOfPrefixes"], "camelCase" | "kebab-case"]
  "component-selector": [true, "element", ["cmp-prefix1", "cmp-prefix2"], "kebab-case"],
  "directive-selector": [true, "attribute", ["dir-prefix1", "dir-prefix2"], "camelCase"],

  "component-max-inline-declarations": true,
  "contextual-lifecycle": true,
  "no-conflicting-lifecycle": true,
  "no-host-metadata-property": true,
  "no-input-rename": true,
  "no-inputs-metadata-property": true,
  "no-output-native": true,
  "no-output-on-prefix": true,
  "no-output-rename": true,
  "no-outputs-metadata-property": true,
  "no-queries-metadata-property": true,
  "prefer-inline-decorator": true,
  "template-banana-in-box": true,
  "template-no-negated-async": true,
  "use-lifecycle-interface": true,
  "use-pipe-transform-interface": true,

  // The rules component-class-suffix and directive-class-suffix have the following arguments:
  // [ENABLED, "suffix" | ["listOfSuffixes"]]
  // Where "suffix" is/are your custom(s) suffix(es), for instance "Page" for Ionic components.
  "component-class-suffix": [true, "Component"],
  "directive-class-suffix": [true, "Directive"]
}

Rules Status

RuleStatus
component-class-suffixStable
component-max-inline-declarationsStable
component-selectorStable
contextual-decoratorStable
contextual-lifecycleStable
directive-class-suffixStable
directive-selectorStable
import-destructuring-spacingStable
no-attribute-decoratorStable
no-forward-refStable
no-host-metadata-propertyStable
no-input-prefixStable
no-input-renameStable
no-inputs-metadata-propertyStable
no-lifecycle-callStable
no-output-nativeStable
no-output-on-prefixStable
no-output-renameStable
no-outputs-metadata-propertyStable
no-pipe-impureStable
no-queries-metadata-propertyStable
prefer-inline-decoratorStable
prefer-output-readonlyStable
template-banana-in-boxStable
template-cyclomatic-complexityStable
template-no-call-expressionStable
template-no-negated-asyncStable
template-use-track-by-functionStable
use-component-selectorStable
use-component-view-encapsulationStable
use-lifecycle-interfaceStable
use-pipe-decoratorStable
use-pipe-transform-interfaceStable
prefer-on-push-component-change-detectionExperimental
no-conflicting-lifecycleExperimental
no-unused-cssExperimental
pipe-prefixExperimental
relative-url-prefixExperimental
template-accessibility-alt-textExperimental
template-accessibility-elements-contentExperimental
template-accessibility-label-forExperimental
template-accessibility-tabindex-no-positiveExperimental
template-accessibility-table-scopeExperimental
template-accessibility-valid-ariaExperimental
template-click-events-have-key-eventsExperimental
template-conditional-complexityExperimental
template-i18nExperimental
template-mouse-events-have-key-eventsExperimental
template-no-anyExperimental
template-no-autofocusExperimental
template-no-distracting-elementsExperimental
angular-whitespaceDeprecated

Disable a rule that validates Template or Styles

Lint rules can be disabled by adding a marker in TypeScript files. More information here.

To disable rules that validate templates or styles you'd need to add a marker in the TypeScript file referencing them.

import { Component } from '@angular/core';

/* tslint:disable:template-use-track-by-function */
@Component({
  selector: 'codelyzer',
  templateUrl: './codelyzer.component.html',
})
class Codelyzer {}

Advanced configuration

Codelyzer supports any template and style language by custom hooks. If you're using Sass for instance, you can allow codelyzer to analyze your styles by creating a file .codelyzer.js in the root of your project (where the node_modules directory is). In the configuration file can implement custom pre-processing and template resolution logic:

// Demo of transforming Sass styles
var sass = require('node-sass');

module.exports = {
  // Definition of custom interpolation strings
  interpolation: ['{{', '}}'],

  // You can transform the urls of your external styles and templates
  resolveUrl(url, decorator) {
    return url;
  },

  // Transformation of the templates. This hooks is quite useful
  // if you're using any other templating language, for instance
  // jade, markdown, haml, etc.
  //
  // NOTE that this method WILL NOT throw an error in case of invalid template.
  //
  transformTemplate(code, url, decorator) {
    return { code: code, url: url };
  },

  // Transformation of styles. This hook is useful is you're using
  // any other style language, for instance Sass, Less, etc.
  //
  // NOTE that this method WILL NOT throw an error in case of invalid style.
  //
  transformStyle(code, url, decorator) {
    var result = { code: code, url: url };
    if (url && /\.scss$/.test(url)) {
      var transformed = sass.renderSync({ data: code, sourceMap: true, outFile: '/dev/null' });
      result.source = code;
      result.code = transformed.css.toString();
      result.map = transformed.map.toString();
    }
    return result;
  },

  // Custom predefined directives in case you get error for
  // missing property and you are using a template reference
  predefinedDirectives: [{ selector: 'form', exportAs: 'ngForm' }],

  // None = 0b000, Error = 0b001, Info = 0b011, Debug = 0b111
  logLevel: 0b111,
};

Contributors

<img alt="mgechev" src="https://avatars1.githubusercontent.com/u/455023?v=4&s=117" width="117"><img alt="wKoza" src="https://avatars2.githubusercontent.com/u/11403260?v=4&s=117" width="117"><img alt="rafaelss95" src="https://avatars0.githubusercontent.com/u/11965907?v=4&s=117" width="117"><img alt="preslavsh" src="https://avatars2.githubusercontent.com/u/6237138?v=4&s=117" width="117"><img alt="mohammedzamakhan" src="https://avatars3.githubusercontent.com/u/2327532?v=4&s=117" width="117"><img alt="rokerkony" src="https://avatars3.githubusercontent.com/u/156132?v=4&s=117" width="117">
mgechevwKozarafaelss95preslavshmohammedzamakhanrokerkony
<img alt="GregOnNet" src="https://avatars3.githubusercontent.com/u/444278?v=4&s=117" width="117"><img alt="alan-agius4" src="https://avatars3.githubusercontent.com/u/17563226?v=4&s=117" width="117"><img alt="kevinphelps" src="https://avatars1.githubusercontent.com/u/7399499?v=4&s=117" width="117"><img alt="eppsilon" src="https://avatars1.githubusercontent.com/u/5643?v=4&s=117" width="117"><img alt="csvn" src="https://avatars2.githubusercontent.com/u/8770194?v=4&s=117" width="117"><img alt="ghsyeung" src="https://avatars0.githubusercontent.com/u/1243185?v=4&s=117" width="117">
GregOnNetalan-agius4kevinphelpseppsiloncsvnghsyeung
<img alt="Kobzol" src="https://avatars0.githubusercontent.com/u/4539057?v=4&s=117" width="117"><img alt="mattlewis92" src="https://avatars1.githubusercontent.com/u/6425649?v=4&s=117" width="117"><img alt="lazarljubenovic" src="https://avatars3.githubusercontent.com/u/7661457?v=4&s=117" width="117"><img alt="sagittarius-rev" src="https://avatars0.githubusercontent.com/u/23564517?v=4&s=117" width="117"><img alt="connor4312" src="https://avatars0.githubusercontent.com/u/2230985?v=4&s=117" width="117"><img alt="Foxandxss" src="https://avatars2.githubusercontent.com/u/1087957?v=4&s=117" width="117">
Kobzolmattlewis92lazarljubenovicsagittarius-revconnor4312Foxandxss
<img alt="gbilodeau" src="https://avatars2.githubusercontent.com/u/532543?v=4&s=117" width="117"><img alt="NagRock" src="https://avatars2.githubusercontent.com/u/5803314?v=4&s=117" width="117"><img alt="Hotell" src="https://avatars0.githubusercontent.com/u/1223799?v=4&s=117" width="117"><img alt="Martin-Wegner" src="https://avatars1.githubusercontent.com/u/8995517?v=4&s=117" width="117"><img alt="comfroels" src="https://avatars2.githubusercontent.com/u/4616177?v=4&s=117" width="117"><img alt="plantain-00" src="https://avatars0.githubusercontent.com/u/7639395?v=4&s=117" width="117">
gbilodeauNagRockHotellMartin-Wegnercomfroelsplantain-00
<img alt="nexus-uw" src="https://avatars2.githubusercontent.com/u/3188890?v=4&s=117" width="117"><img alt="alexkpek" src="https://avatars0.githubusercontent.com/u/8692873?v=4&s=117" width="117"><img alt="loktionov129" src="https://avatars1.githubusercontent.com/u/20480552?v=4&s=117" width="117"><img alt="alisd23" src="https://avatars2.githubusercontent.com/u/5804010?v=4&s=117" width="117"><img alt="aboyton" src="https://avatars0.githubusercontent.com/u/7865563?v=4&s=117" width="117"><img alt="bmvantunes" src="https://avatars1.githubusercontent.com/u/9042965?v=4&s=117" width="117">
nexus-uwalexkpekloktionov129alisd23aboytonbmvantunes
<img alt="Moeriki" src="https://avatars2.githubusercontent.com/u/38758?v=4&s=117" width="117"><img alt="sneas" src="https://avatars2.githubusercontent.com/u/144651?v=4&s=117" width="117"><img alt="EmmanuelDemey" src="https://avatars2.githubusercontent.com/u/555768?v=4&s=117" width="117"><img alt="eromano" src="https://avatars1.githubusercontent.com/u/1030050?v=4&s=117" width="117"><img alt="Manduro" src="https://avatars0.githubusercontent.com/u/2545042?v=4&s=117" width="117"><img alt="karol-depka" src="https://avatars1.githubusercontent.com/u/958486?v=4&s=117" width="117">
MoerikisneasEmmanuelDemeyeromanoMandurokarol-depka
<img alt="leosvelperez" src="https://avatars3.githubusercontent.com/u/12051310?v=4&s=117" width="117"><img alt="muhammadghazali" src="https://avatars3.githubusercontent.com/u/863947?v=4&s=117" width="117"><img alt="PapsOu" src="https://avatars3.githubusercontent.com/u/5792207?v=4&s=117" width="117"><img alt="rwlogel" src="https://avatars2.githubusercontent.com/u/3373178?v=4&s=117" width="117"><img alt="robzenn92" src="https://avatars0.githubusercontent.com/u/3225625?v=4&s=117" width="117"><img alt="rtfpessoa" src="https://avatars0.githubusercontent.com/u/902384?v=4&s=117" width="117">
leosvelperezmuhammadghazaliPapsOurwlogelrobzenn92rtfpessoa
<img alt="santoshyadav198613" src="https://avatars3.githubusercontent.com/u/11923975?v=4&s=117" width="117"><img alt="scttcper" src="https://avatars3.githubusercontent.com/u/1400464?v=4&s=117" width="117"><img alt="stschake" src="https://avatars1.githubusercontent.com/u/104132?v=4&s=117" width="117"><img alt="tmair" src="https://avatars2.githubusercontent.com/u/1596276?v=4&s=117" width="117"><img alt="YogliB" src="https://avatars2.githubusercontent.com/u/10498929?v=4&s=117" width="117"><img alt="cexbrayat" src="https://avatars3.githubusercontent.com/u/411874?v=4&s=117" width="117">
santoshyadav198613scttcperstschaketmairYogliBcexbrayat
<img alt="clydin" src="https://avatars2.githubusercontent.com/u/19598772?v=4&s=117" width="117"><img alt="reduckted" src="https://avatars0.githubusercontent.com/u/10321525?v=4&s=117" width="117"><img alt="someblue" src="https://avatars1.githubusercontent.com/u/5562124?v=4&s=117" width="117">
clydinreducktedsomeblue

License

MIT