mirror of
https://github.com/Microsoft/vscode
synced 2024-08-27 13:00:58 +00:00
![Matt Bierner](/assets/img/avatar_default.png)
Resubmission of #157532 with the following changes: - Use `eslint-plugin-local` instead of `yarn` link to run our plugins - Move our plugins to a top level `.eslintplugin` dir (as required by `eslint-plugin-local`) - Update all names to `local/`
98 lines
2.9 KiB
TypeScript
98 lines
2.9 KiB
TypeScript
/*---------------------------------------------------------------------------------------------
|
|
* Copyright (c) Microsoft Corporation. All rights reserved.
|
|
* Licensed under the MIT License. See License.txt in the project root for license information.
|
|
*--------------------------------------------------------------------------------------------*/
|
|
|
|
import * as eslint from 'eslint';
|
|
import { TSESTree, AST_NODE_TYPES } from '@typescript-eslint/experimental-utils';
|
|
|
|
export = new class ApiEventNaming implements eslint.Rule.RuleModule {
|
|
|
|
private static _nameRegExp = /on(Did|Will)([A-Z][a-z]+)([A-Z][a-z]+)?/;
|
|
|
|
readonly meta: eslint.Rule.RuleMetaData = {
|
|
docs: {
|
|
url: 'https://github.com/microsoft/vscode/wiki/Extension-API-guidelines#event-naming'
|
|
},
|
|
messages: {
|
|
naming: 'Event names must follow this patten: `on[Did|Will]<Verb><Subject>`',
|
|
verb: 'Unknown verb \'{{verb}}\' - is this really a verb? Iff so, then add this verb to the configuration',
|
|
subject: 'Unknown subject \'{{subject}}\' - This subject has not been used before but it should refer to something in the API',
|
|
unknown: 'UNKNOWN event declaration, lint-rule needs tweaking'
|
|
}
|
|
};
|
|
|
|
create(context: eslint.Rule.RuleContext): eslint.Rule.RuleListener {
|
|
|
|
const config = <{ allowed: string[]; verbs: string[] }>context.options[0];
|
|
const allowed = new Set(config.allowed);
|
|
const verbs = new Set(config.verbs);
|
|
|
|
return {
|
|
['TSTypeAnnotation TSTypeReference Identifier[name="Event"]']: (node: any) => {
|
|
|
|
const def = (<TSESTree.Identifier>node).parent?.parent?.parent;
|
|
const ident = this.getIdent(def);
|
|
|
|
if (!ident) {
|
|
// event on unknown structure...
|
|
return context.report({
|
|
node,
|
|
message: 'unknown'
|
|
});
|
|
}
|
|
|
|
if (allowed.has(ident.name)) {
|
|
// configured exception
|
|
return;
|
|
}
|
|
|
|
const match = ApiEventNaming._nameRegExp.exec(ident.name);
|
|
if (!match) {
|
|
context.report({
|
|
node: ident,
|
|
messageId: 'naming'
|
|
});
|
|
return;
|
|
}
|
|
|
|
// check that <verb> is spelled out (configured) as verb
|
|
if (!verbs.has(match[2].toLowerCase())) {
|
|
context.report({
|
|
node: ident,
|
|
messageId: 'verb',
|
|
data: { verb: match[2] }
|
|
});
|
|
}
|
|
|
|
// check that a subject (if present) has occurred
|
|
if (match[3]) {
|
|
const regex = new RegExp(match[3], 'ig');
|
|
const parts = context.getSourceCode().getText().split(regex);
|
|
if (parts.length < 3) {
|
|
context.report({
|
|
node: ident,
|
|
messageId: 'subject',
|
|
data: { subject: match[3] }
|
|
});
|
|
}
|
|
}
|
|
}
|
|
};
|
|
}
|
|
|
|
private getIdent(def: TSESTree.Node | undefined): TSESTree.Identifier | undefined {
|
|
if (!def) {
|
|
return;
|
|
}
|
|
|
|
if (def.type === AST_NODE_TYPES.Identifier) {
|
|
return def;
|
|
} else if ((def.type === AST_NODE_TYPES.TSPropertySignature || def.type === AST_NODE_TYPES.PropertyDefinition) && def.key.type === AST_NODE_TYPES.Identifier) {
|
|
return def.key;
|
|
}
|
|
|
|
return this.getIdent(def.parent);
|
|
}
|
|
};
|