mirror of
https://github.com/Microsoft/vscode
synced 2024-10-28 14:18:54 +00:00
e483059807
Clean up some assertions Mostly just removing extra type assertions or adding them to a better spot Also addresses some missing semicolons that snuck in somehow
40 lines
1.5 KiB
TypeScript
40 lines
1.5 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 } from '@typescript-eslint/experimental-utils';
|
|
|
|
export = new class NoDangerousTypeAssertions implements eslint.Rule.RuleModule {
|
|
|
|
create(context: eslint.Rule.RuleContext): eslint.Rule.RuleListener {
|
|
// Disable in tests for now
|
|
if (context.getFilename().includes('.test')) {
|
|
return {};
|
|
}
|
|
|
|
return {
|
|
// Disallow type assertions on object literals: <T>{ ... } or {} as T
|
|
['TSTypeAssertion > ObjectExpression, TSAsExpression > ObjectExpression']: (node: any) => {
|
|
const objectNode = node as TSESTree.Node;
|
|
|
|
const parent = objectNode.parent as TSESTree.TSTypeAssertion | TSESTree.TSAsExpression;
|
|
if (
|
|
// Allow `as const` assertions
|
|
(parent.typeAnnotation.type === 'TSTypeReference' && parent.typeAnnotation.typeName.type === 'Identifier' && parent.typeAnnotation.typeName.name === 'const')
|
|
|
|
// For also now still allow `any` casts
|
|
|| (parent.typeAnnotation.type === 'TSAnyKeyword')
|
|
) {
|
|
return;
|
|
}
|
|
|
|
context.report({
|
|
node,
|
|
message: "Don't use type assertions for creating objects as this can hide type errors."
|
|
});
|
|
},
|
|
};
|
|
}
|
|
};
|