eslint-plugin-ember-template-lint 0.17.0 → 0.18.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +2 -2
- package/lib/index.js +1 -1
- package/lib/rules/lint.js +1 -1
- package/package.json +2 -2
- package/lib/config/base.js +0 -20
- package/lib/config/index.js +0 -20
- package/lib/ember-teplate-lint/config.js +0 -18
- package/lib/ember-teplate-lint/info.js +0 -106
- package/lib/ember-teplate-lint/worker.js +0 -12
- package/lib/parser/hbs-parser.js +0 -57
- package/lib/processor.js +0 -11
- package/lib/utils/document.js +0 -100
package/README.md
CHANGED
@@ -42,7 +42,7 @@ to just use the `.template-lintrc.js` config file just use
|
|
42
42
|
|
43
43
|
to load ember-template-lint plugins in eslinrc you need to manually register them:
|
44
44
|
```js
|
45
|
-
require('eslint-plugin-ember-template-lint/lib/ember-
|
45
|
+
require('eslint-plugin-ember-template-lint/lib/ember-template-lint/config').registerPlugin('ember-template-lint-plugin-prettier');
|
46
46
|
```
|
47
47
|
|
48
48
|
## IMPORTANT
|
@@ -53,7 +53,7 @@ for typescript parser you can extend its base config, then it will work correctl
|
|
53
53
|
```js
|
54
54
|
// .eslintrc.js
|
55
55
|
// optional:
|
56
|
-
require('eslint-plugin-ember-template-lint/lib/ember-
|
56
|
+
require('eslint-plugin-ember-template-lint/lib/ember-template-lint/config').registerPlugin('ember-template-lint-plugin-prettier');
|
57
57
|
|
58
58
|
|
59
59
|
module.exports = {
|
package/lib/index.js
CHANGED
@@ -10,7 +10,7 @@
|
|
10
10
|
|
11
11
|
const configs = require('./config');
|
12
12
|
const base = require('./config/base');
|
13
|
-
const templateRules = require('./ember-
|
13
|
+
const templateRules = require('./ember-template-lint/info');
|
14
14
|
const processor = require('./processor');
|
15
15
|
|
16
16
|
//------------------------------------------------------------------------------
|
package/lib/rules/lint.js
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
'use strict';
|
2
2
|
const synckit = require('synckit');
|
3
3
|
const DocumentLines = require('../utils/document');
|
4
|
-
const { templateLintConfig } = require('../ember-
|
4
|
+
const { templateLintConfig } = require('../ember-template-lint/config');
|
5
5
|
|
6
6
|
function runTemplateLint(node, context) {
|
7
7
|
const sourceCode = context.getSourceCode();
|
package/package.json
CHANGED
@@ -1,6 +1,6 @@
|
|
1
1
|
{
|
2
2
|
"name": "eslint-plugin-ember-template-lint",
|
3
|
-
"version": "0.
|
3
|
+
"version": "0.18.0",
|
4
4
|
"description": "Provide linting for ember template",
|
5
5
|
"keywords": [
|
6
6
|
"eslint",
|
@@ -46,6 +46,6 @@
|
|
46
46
|
},
|
47
47
|
"license": "ISC",
|
48
48
|
"files": [
|
49
|
-
"lib
|
49
|
+
"lib/rules/*.js"
|
50
50
|
]
|
51
51
|
}
|
package/lib/config/base.js
DELETED
@@ -1,20 +0,0 @@
|
|
1
|
-
module.exports = {
|
2
|
-
root: true,
|
3
|
-
|
4
|
-
plugins: ['ember-template-lint'],
|
5
|
-
|
6
|
-
rules: {},
|
7
|
-
|
8
|
-
overrides: [
|
9
|
-
{
|
10
|
-
files: ['**/*.hbs'],
|
11
|
-
parser: require.resolve('../parser/hbs-parser'),
|
12
|
-
processor: 'ember-template-lint/noop'
|
13
|
-
},
|
14
|
-
{
|
15
|
-
files: ['**/*.gts', '**/*.gjs'],
|
16
|
-
parser: 'ember-eslint-parser',
|
17
|
-
processor: 'ember-template-lint/noop'
|
18
|
-
},
|
19
|
-
],
|
20
|
-
};
|
package/lib/config/index.js
DELETED
@@ -1,20 +0,0 @@
|
|
1
|
-
const info = require('../ember-teplate-lint/info');
|
2
|
-
const base = require('./base');
|
3
|
-
|
4
|
-
const configs = {};
|
5
|
-
|
6
|
-
Object.entries(info.configs).forEach(([name, config]) => {
|
7
|
-
Object.entries(config.rules).forEach(([name, conf]) => {
|
8
|
-
if (typeof conf == 'boolean') {
|
9
|
-
config.rules[name] = [conf ? 'error' : 'off'];
|
10
|
-
}
|
11
|
-
});
|
12
|
-
configs[name] = {...base, rules: config.rules};
|
13
|
-
});
|
14
|
-
configs['config'] = {
|
15
|
-
...base,
|
16
|
-
rules: info.configuredRules
|
17
|
-
};
|
18
|
-
|
19
|
-
module.exports = configs;
|
20
|
-
|
@@ -1,18 +0,0 @@
|
|
1
|
-
const templateLintConfig = {
|
2
|
-
rules: {},
|
3
|
-
plugins: [],
|
4
|
-
};
|
5
|
-
|
6
|
-
function registerRule(name, config) {
|
7
|
-
templateLintConfig.rules[name] = Array.isArray(config) ? config[0] : config;
|
8
|
-
}
|
9
|
-
|
10
|
-
function registerPlugin(name) {
|
11
|
-
templateLintConfig.plugins.push(name);
|
12
|
-
}
|
13
|
-
|
14
|
-
module.exports = {
|
15
|
-
registerRule,
|
16
|
-
registerPlugin,
|
17
|
-
templateLintConfig
|
18
|
-
};
|
@@ -1,106 +0,0 @@
|
|
1
|
-
const synckit = require('synckit');
|
2
|
-
const syncFn = synckit.createSyncFn(require.resolve('./worker'));
|
3
|
-
const { runTemplateLint } = require('../rules/lint');
|
4
|
-
const { registerRule, templateLintConfig } = require('../ember-teplate-lint/config');
|
5
|
-
|
6
|
-
const lintWithEslintConfigs = syncFn({ config: templateLintConfig });
|
7
|
-
const lintConfigs = syncFn();
|
8
|
-
|
9
|
-
const activeRules = new Map();
|
10
|
-
const allMessages = {};
|
11
|
-
|
12
|
-
const reporter = {
|
13
|
-
setup(context) {
|
14
|
-
this.getSourceCode = context.getSourceCode;
|
15
|
-
this.getPhysicalFilename = context.getPhysicalFilename;
|
16
|
-
},
|
17
|
-
report(message) {
|
18
|
-
message.meta = {
|
19
|
-
fixable: 'code'
|
20
|
-
};
|
21
|
-
allMessages[message.rule] = allMessages[message.rule] || [];
|
22
|
-
allMessages[message.rule].push(message);
|
23
|
-
}
|
24
|
-
};
|
25
|
-
|
26
|
-
class Rule {
|
27
|
-
|
28
|
-
constructor(name) {
|
29
|
-
this.create = this.create.bind(this);
|
30
|
-
this.name = name;
|
31
|
-
this.meta = {
|
32
|
-
fixable: 'code'
|
33
|
-
};
|
34
|
-
}
|
35
|
-
create(context) {
|
36
|
-
const rule = this;
|
37
|
-
let options = context.options;
|
38
|
-
if (options.length === 0) {
|
39
|
-
options = true;
|
40
|
-
}
|
41
|
-
registerRule(this.name, options);
|
42
|
-
const visitor = {
|
43
|
-
enter(node) {
|
44
|
-
if (!activeRules.get(node)) {
|
45
|
-
activeRules.set(node, 0);
|
46
|
-
reporter.setup(context);
|
47
|
-
runTemplateLint(node, reporter);
|
48
|
-
}
|
49
|
-
activeRules.set(node, activeRules.get(node) + 1);
|
50
|
-
},
|
51
|
-
exit(node) {
|
52
|
-
const messages = allMessages[rule.name] || [];
|
53
|
-
messages.forEach(m => context.report(m));
|
54
|
-
activeRules.set(node, activeRules.get(node) - 1);
|
55
|
-
allMessages[rule.name] = [];
|
56
|
-
},
|
57
|
-
};
|
58
|
-
return {
|
59
|
-
'Program': (node) => node.isHbs && visitor.enter(node),
|
60
|
-
'Program:exit': (node) => node.isHbs && visitor.exit(node),
|
61
|
-
'GlimmerTemplate': visitor.enter,
|
62
|
-
'GlimmerTemplate:exit': visitor.exit,
|
63
|
-
};
|
64
|
-
}
|
65
|
-
}
|
66
|
-
|
67
|
-
function createRules(rules) {
|
68
|
-
const created = rules.map(r => new Rule(r));
|
69
|
-
const map = {};
|
70
|
-
created.forEach(r => {
|
71
|
-
map[r.name] = r;
|
72
|
-
});
|
73
|
-
return map;
|
74
|
-
}
|
75
|
-
|
76
|
-
const configs = {
|
77
|
-
...lintWithEslintConfigs.configs,
|
78
|
-
...lintConfigs.configs
|
79
|
-
};
|
80
|
-
|
81
|
-
const rules = [...new Set([...lintConfigs.rules, ...lintWithEslintConfigs.rules])];
|
82
|
-
|
83
|
-
delete configs.recommended.overrides;
|
84
|
-
|
85
|
-
Object.values(configs).forEach((config) => {
|
86
|
-
const rules = {};
|
87
|
-
Object.entries(config.rules).forEach(([rule, conf]) => {
|
88
|
-
rules['ember-template-lint/' + rule] = conf;
|
89
|
-
});
|
90
|
-
config.rules = rules;
|
91
|
-
});
|
92
|
-
|
93
|
-
|
94
|
-
const configuredRules = {};
|
95
|
-
Object.entries(lintConfigs.configuredRules).forEach(([name, conf]) => {
|
96
|
-
configuredRules['ember-template-lint/' + name] = conf.config && [conf.severity];
|
97
|
-
if (typeof conf.config !== 'boolean') {
|
98
|
-
configuredRules['ember-template-lint/' + name].push(conf.config);
|
99
|
-
}
|
100
|
-
});
|
101
|
-
|
102
|
-
module.exports = {
|
103
|
-
configs: configs,
|
104
|
-
rules: createRules(rules),
|
105
|
-
configuredRules: configuredRules
|
106
|
-
};
|
@@ -1,12 +0,0 @@
|
|
1
|
-
const { runAsWorker } = require('synckit');
|
2
|
-
|
3
|
-
runAsWorker(async (options) => {
|
4
|
-
const Lint = await import('ember-template-lint');
|
5
|
-
const lint = new Lint.default(options);
|
6
|
-
await lint.loadConfig();
|
7
|
-
return {
|
8
|
-
configs: lint.config.loadedConfigurations,
|
9
|
-
rules: Object.keys(lint.config.loadedRules),
|
10
|
-
configuredRules: lint.config.rules,
|
11
|
-
};
|
12
|
-
});
|
package/lib/parser/hbs-parser.js
DELETED
@@ -1,57 +0,0 @@
|
|
1
|
-
|
2
|
-
class Scope {
|
3
|
-
type = 'global';
|
4
|
-
variables = [];
|
5
|
-
through= [];
|
6
|
-
set = new Map();
|
7
|
-
upper = null;
|
8
|
-
childScopes = [];
|
9
|
-
references = [];
|
10
|
-
block = null;
|
11
|
-
}
|
12
|
-
|
13
|
-
class ScopeManager {
|
14
|
-
globalScope = new Scope();
|
15
|
-
scopes = [this.globalScope];
|
16
|
-
|
17
|
-
acquire() {
|
18
|
-
return;
|
19
|
-
}
|
20
|
-
|
21
|
-
getDeclaredVariables() {
|
22
|
-
return [];
|
23
|
-
}
|
24
|
-
}
|
25
|
-
|
26
|
-
module.exports = {
|
27
|
-
parseForESLint(code) {
|
28
|
-
const comments = [];
|
29
|
-
const types = new Set(['Program']);
|
30
|
-
const ast = {};
|
31
|
-
ast.body = [];
|
32
|
-
ast.tokens = [ast];
|
33
|
-
ast.range = [0, code.length];
|
34
|
-
const lines = code.split('\n');
|
35
|
-
ast.loc = {
|
36
|
-
start: {
|
37
|
-
line: 1,
|
38
|
-
column: 1
|
39
|
-
},
|
40
|
-
end: {
|
41
|
-
line: lines.length,
|
42
|
-
column: lines[lines.length - 1].length
|
43
|
-
}
|
44
|
-
};
|
45
|
-
ast.comments = comments;
|
46
|
-
const visitorKeys = {};
|
47
|
-
types.forEach((t) => {
|
48
|
-
visitorKeys[t] = [];
|
49
|
-
});
|
50
|
-
ast.type = 'Program';
|
51
|
-
ast.isHbs = true;
|
52
|
-
ast.contents = code;
|
53
|
-
const scope = new ScopeManager();
|
54
|
-
scope.globalScope.block = ast;
|
55
|
-
return { ast, scopeManager: scope, visitorKeys };
|
56
|
-
}
|
57
|
-
};
|
package/lib/processor.js
DELETED
package/lib/utils/document.js
DELETED
@@ -1,100 +0,0 @@
|
|
1
|
-
/**
|
2
|
-
* @typedef {{ line: number; character: number }} Position
|
3
|
-
*/
|
4
|
-
|
5
|
-
// Helper class to convert line/column from and to offset
|
6
|
-
// taken and adapt from https://github.com/typed-ember/glint/blob/main/packages/core/src/language-server/util/position.ts
|
7
|
-
class DocumentLines {
|
8
|
-
/**
|
9
|
-
* @param {string} contents
|
10
|
-
*/
|
11
|
-
constructor(contents) {
|
12
|
-
this.lineStarts = computeLineStarts(contents);
|
13
|
-
}
|
14
|
-
|
15
|
-
/**
|
16
|
-
* @param {Position} position
|
17
|
-
* @return {number}
|
18
|
-
*/
|
19
|
-
positionToOffset(position) {
|
20
|
-
const { line, character } = position;
|
21
|
-
return this.lineStarts[line] + character;
|
22
|
-
}
|
23
|
-
|
24
|
-
/**
|
25
|
-
*
|
26
|
-
* @param {number} position
|
27
|
-
* @return {Position}
|
28
|
-
*/
|
29
|
-
offsetToPosition(position) {
|
30
|
-
const lineStarts = this.lineStarts;
|
31
|
-
let line = 0;
|
32
|
-
while (line + 1 < lineStarts.length && lineStarts[line + 1] <= position) {
|
33
|
-
line++;
|
34
|
-
}
|
35
|
-
const character = position - lineStarts[line];
|
36
|
-
return { line, character };
|
37
|
-
}
|
38
|
-
}
|
39
|
-
|
40
|
-
/**
|
41
|
-
* @returns {number[]}
|
42
|
-
* @param {string} text
|
43
|
-
*/
|
44
|
-
function computeLineStarts(text) {
|
45
|
-
const result = [];
|
46
|
-
let pos = 0;
|
47
|
-
let lineStart = 0;
|
48
|
-
while (pos < text.length) {
|
49
|
-
const ch = text.codePointAt(pos);
|
50
|
-
pos++;
|
51
|
-
switch (ch) {
|
52
|
-
case 13 /* carriageReturn */: {
|
53
|
-
if (text.codePointAt(pos) === 10 /* lineFeed */) {
|
54
|
-
pos++;
|
55
|
-
}
|
56
|
-
}
|
57
|
-
// falls through
|
58
|
-
case 10 /* lineFeed */: {
|
59
|
-
result.push(lineStart);
|
60
|
-
lineStart = pos;
|
61
|
-
break;
|
62
|
-
}
|
63
|
-
default: {
|
64
|
-
if (ch > 127 /* maxAsciiCharacter */ && isLineBreak(ch)) {
|
65
|
-
result.push(lineStart);
|
66
|
-
lineStart = pos;
|
67
|
-
}
|
68
|
-
break;
|
69
|
-
}
|
70
|
-
}
|
71
|
-
}
|
72
|
-
result.push(lineStart);
|
73
|
-
return result;
|
74
|
-
}
|
75
|
-
|
76
|
-
/* istanbul ignore next */
|
77
|
-
/**
|
78
|
-
* @param {number} ch
|
79
|
-
* @return {boolean}
|
80
|
-
*/
|
81
|
-
function isLineBreak(ch) {
|
82
|
-
// ES5 7.3:
|
83
|
-
// The ECMAScript line terminator characters are listed in Table 3.
|
84
|
-
// Table 3: Line Terminator Characters
|
85
|
-
// Code Unit Value Name Formal Name
|
86
|
-
// \u000A Line Feed <LF>
|
87
|
-
// \u000D Carriage Return <CR>
|
88
|
-
// \u2028 Line separator <LS>
|
89
|
-
// \u2029 Paragraph separator <PS>
|
90
|
-
// Only the characters in Table 3 are treated as line terminators. Other new line or line
|
91
|
-
// breaking characters are treated as white space but not as line terminators.
|
92
|
-
return (
|
93
|
-
ch === 10 /* lineFeed */ ||
|
94
|
-
ch === 13 /* carriageReturn */ ||
|
95
|
-
ch === 8232 /* lineSeparator */ ||
|
96
|
-
ch === 8233 /* paragraphSeparator */
|
97
|
-
);
|
98
|
-
}
|
99
|
-
|
100
|
-
module.exports = DocumentLines;
|