Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat(eslint-plugin): updated rule to cover more reference directives
- Loading branch information
Jesse Trinity
committed
Jun 22, 2019
1 parent
ac3cd5d
commit ecc65e1
Showing
10 changed files
with
316 additions
and
171 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
58 changes: 58 additions & 0 deletions
58
packages/eslint-plugin/docs/rules/triple-slash-reference.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,58 @@ | ||
| # Sets preference level for triple slash directives versus ES6-style import declarations. (triple-slash-reference) | ||
|
|
||
| Use of triple-slash reference type directives is discouraged in favor of the newer `import` style. This rule allows you to ban use of `/// <reference path="" />`, `/// <reference types="" />`, or `/// <reference lib="" />` directives. | ||
|
|
||
| If you use the `no-triple-slash-reference` rule, consider using this rule instead. | ||
|
|
||
| ## Rule Details | ||
|
|
||
| With `{ "path": "never", "types": "never", "lib": "never" }` options set, the following will all be **incorrect** usage: | ||
|
|
||
| ```ts | ||
| /// <reference path="foo" /> | ||
| /// <reference types="bar" /> | ||
| /// <reference lib="baz" /> | ||
| ``` | ||
|
|
||
| Examples of **incorrect** code for the `{ "types": "prefer-import" }` option. Note that these are only errors when **both** stlyes are used for the **same** module: | ||
|
|
||
| ```ts | ||
| /// <reference types="foo" /> | ||
| import * as foo from 'foo'; | ||
| ``` | ||
|
|
||
| ```ts | ||
| /// <reference types="foo" /> | ||
| import foo = require('foo'); | ||
| ``` | ||
|
|
||
| With `{ "path": "always", "types": "always", "lib": "always" }` options set, the following will all be **correct** usage: | ||
|
|
||
| ```ts | ||
| /// <reference path="foo" /> | ||
| /// <reference types="bar" /> | ||
| /// <reference lib="baz" /> | ||
| ``` | ||
|
|
||
| Examples of **correct** code for the `{ "types": "prefer-import" }` option: | ||
|
|
||
| ```ts | ||
| import * as foo from 'foo'; | ||
| ``` | ||
|
|
||
| ```ts | ||
| import foo = require('foo'); | ||
| ``` | ||
|
|
||
| ## When To Use It | ||
|
|
||
| If you want to ban use of one or all of the triple slash reference directives, or any time you might use triple-slash type reference directives and ES6 import declarations in the same file. | ||
|
|
||
| ## When Not To Use It | ||
|
|
||
| If you want to use all flavors of triple slash reference directives. | ||
|
|
||
| ## Compatibility | ||
|
|
||
| - TSLint: [no-reference](http://palantir.github.io/tslint/rules/no-reference/) | ||
| - TSLint: [no-reference-import](https://palantir.github.io/tslint/rules/no-reference-import/) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
129 changes: 129 additions & 0 deletions
129
packages/eslint-plugin/src/rules/triple-slash-reference.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,129 @@ | ||
| import * as util from '../util'; | ||
| import { | ||
| Literal, | ||
| Node, | ||
| TSExternalModuleReference, | ||
| } from '@typescript-eslint/typescript-estree/dist/ts-estree/ts-estree'; | ||
| import { TSESTree } from '@typescript-eslint/typescript-estree'; | ||
|
|
||
| type Options = [ | ||
| { | ||
| lib?: 'always' | 'never'; | ||
| path?: 'always' | 'never'; | ||
| types?: 'always' | 'never' | 'prefer-import'; | ||
| } | ||
| ]; | ||
| type MessageIds = 'tripleSlashReference'; | ||
|
|
||
| export default util.createRule<Options, MessageIds>({ | ||
| name: 'triple-slash-reference', | ||
| meta: { | ||
| type: 'suggestion', | ||
| docs: { | ||
| description: | ||
| 'Sets preference level for triple slash directives versus ES6-style import declarations', | ||
| category: 'Best Practices', | ||
| recommended: false, | ||
| }, | ||
| messages: { | ||
| tripleSlashReference: | ||
| 'Do not use a triple slash reference for {{module}}, use `import` style instead.', | ||
| }, | ||
| schema: [ | ||
| { | ||
| type: 'object', | ||
| properties: { | ||
| lib: { | ||
| enum: ['always', 'never'], | ||
| }, | ||
| path: { | ||
| enum: ['always', 'never'], | ||
| }, | ||
| types: { | ||
| enum: ['always', 'never', 'prefer-import'], | ||
| }, | ||
| }, | ||
| additionalProperties: false, | ||
| }, | ||
| ], | ||
| }, | ||
| defaultOptions: [ | ||
| { | ||
| lib: 'always', | ||
| path: 'never', | ||
| types: 'prefer-import', | ||
| }, | ||
| ], | ||
| create(context, [{ lib, path, types }]) { | ||
| let programNode: Node; | ||
| const sourceCode = context.getSourceCode(); | ||
| const references: ({ | ||
| comment: TSESTree.Comment; | ||
| importName: string; | ||
| })[] = []; | ||
|
|
||
| function hasMatchingReference(source: Literal) { | ||
| references.forEach(reference => { | ||
| if (reference.importName === source.value) { | ||
| context.report({ | ||
| node: reference.comment, | ||
| messageId: 'tripleSlashReference', | ||
| data: { | ||
| module: reference.importName, | ||
| }, | ||
| }); | ||
| } | ||
| }); | ||
| } | ||
| return { | ||
| ImportDeclaration(node) { | ||
| if (programNode) { | ||
| const source = node.source as Literal; | ||
| hasMatchingReference(source); | ||
| } | ||
| }, | ||
| TSImportEqualsDeclaration(node) { | ||
| if (programNode) { | ||
| const source = (node.moduleReference as TSExternalModuleReference) | ||
| .expression as Literal; | ||
| hasMatchingReference(source); | ||
| } | ||
| }, | ||
| Program(node) { | ||
| if (lib === 'always' && path === 'always' && types == 'always') { | ||
| return; | ||
| } | ||
| programNode = node; | ||
| const referenceRegExp = /^\/\s*<reference\s*(types|path|lib)\s*=\s*["|'](.*)["|']/; | ||
| const commentsBefore = sourceCode.getCommentsBefore(programNode); | ||
|
|
||
| commentsBefore.forEach(comment => { | ||
| if (comment.type !== 'Line') { | ||
| return; | ||
| } | ||
| const referenceResult = referenceRegExp.exec(comment.value); | ||
|
|
||
| if (referenceResult) { | ||
| if ( | ||
| (referenceResult[1] === 'types' && types === 'never') || | ||
| (referenceResult[1] === 'path' && path === 'never') || | ||
| (referenceResult[1] === 'lib' && lib === 'never') | ||
| ) { | ||
| context.report({ | ||
| node: comment, | ||
| messageId: 'tripleSlashReference', | ||
| data: { | ||
| module: referenceResult[2], | ||
| }, | ||
| }); | ||
| return; | ||
| } | ||
| if (referenceResult[1] === 'types' && types === 'prefer-import') { | ||
| references.push({ comment, importName: referenceResult[2] }); | ||
| } | ||
| } | ||
| }); | ||
| }, | ||
| }; | ||
| }, | ||
| }); |
Oops, something went wrong.