@microsoft/sp-tslint-rules 1.21.0-beta.0 → 1.21.0-beta.2

package/lib/pairReactDomRenderUnmountRule.js

@@ -1,108 +0,0 @@
-"use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.Rule = void 0;
-const tsUtils = __importStar(require("tsutils"));
-const ts = __importStar(require("typescript"));
-const Lint = __importStar(require("tslint"));
-class Rule extends Lint.Rules.AbstractRule {
-    apply(sourceFile) {
-        return this.applyWithFunction(sourceFile, this._walk.bind(this));
-    }
-    _walk(ctx) {
-        const renderCalls = [];
-        const unmountCalls = [];
-        // Do not convert `callback` to a function. It references `this` keyword inside it.
-        const callback = (node) => {
-            const reactDOMImportNamespaceName = this._getReactDOMImportNamespaceName(node);
-            if (reactDOMImportNamespaceName) {
-                this._reactDOMImportNamespaceName = reactDOMImportNamespaceName;
-                return;
-            }
-            const isRenderCallExpression = this._isReactDOMCallExpression(node, 'render');
-            if (isRenderCallExpression) {
-                renderCalls.push(node);
-                return;
-            }
-            const isUnmountCallExpression = this._isReactDOMCallExpression(node, 'unmountComponentAtNode');
-            if (isUnmountCallExpression) {
-                unmountCalls.push(node);
-                return;
-            }
-            ts.forEachChild(node, callback);
-        };
-        ts.forEachChild(ctx.sourceFile, callback);
-        // Verify the count of render calls and unmount calls are the same in the file.
-        if (renderCalls.length !== unmountCalls.length) {
-            renderCalls.concat(unmountCalls).forEach((renderCallExpression) => {
-                ctx.addFailureAtNode(renderCallExpression, 'Unmatched calls between `ReactDOM.render` and `ReactDOM.unmountComponentAtNode`.');
-            });
-        }
-    }
-    _getReactDOMImportNamespaceName(node) {
-        if (
-        // import * as ReactDOM from 'react-dom'
-        tsUtils.isImportDeclaration(node) &&
-            // * as ReactDOM
-            node.importClause &&
-            tsUtils.isImportClause(node.importClause) &&
-            // ReactDOM
-            node.importClause.namedBindings &&
-            tsUtils.isNamespaceImport(node.importClause.namedBindings) &&
-            // 'react-dom'
-            tsUtils.isStringLiteral(node.moduleSpecifier) &&
-            node.moduleSpecifier.text === 'react-dom') {
-            return node.importClause.namedBindings.name.text;
-        }
-        else {
-            return undefined;
-        }
-    }
-    _isReactDOMCallExpression(node, methodName) {
-        return (
-        // ReactDOM.method(param1, param2)
-        tsUtils.isCallExpression(node) &&
-            // ReactDOM.method
-            tsUtils.isPropertyAccessExpression(node.expression) &&
-            // ReactDOM
-            tsUtils.isIdentifier(node.expression.expression) &&
-            node.expression.expression.text === this._reactDOMImportNamespaceName &&
-            // method
-            tsUtils.isIdentifier(node.expression.name) &&
-            node.expression.name.text === methodName);
-    }
-}
-exports.Rule = Rule;
-Rule.metadata = {
-    ruleName: 'pair-react-dom-render-unmount',
-    description: 'Pair ReactDOM render and unmount calls in one file.' +
-        ' If a ReactDOM render tree is not unmounted when disposed, it will cause a memory leak.',
-    rationale: 'Pair the render and unmount calls to avoid memory leak.',
-    optionsDescription: 'No options available.',
-    options: {},
-    type: 'maintainability',
-    typescriptOnly: false
-};
-//# sourceMappingURL=pairReactDomRenderUnmountRule.js.map

package/lib/validKillSwitchGuidRule.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"validKillSwitchGuidRule.d.ts","sourceRoot":"","sources":["../src/validKillSwitchGuidRule.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,IAAI,MAAM,QAAQ,CAAC;AAE/B,OAAO,KAAK,EAAE,MAAM,YAAY,CAAC;AAEjC;;;;GAIG;AACH,qBAAa,IAAK,SAAQ,IAAI,CAAC,KAAK,CAAC,YAAY;IAC/C,OAAc,QAAQ,EAAE,IAAI,CAAC,aAAa,CAQxC;IAEK,KAAK,CAAC,MAAM,EAAE,EAAE,CAAC,UAAU,GAAG,IAAI,CAAC,WAAW,EAAE;CAGxD"}

package/lib/validKillSwitchGuidRule.js

@@ -1,72 +0,0 @@
-"use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || function (mod) {
-    if (mod && mod.__esModule) return mod;
-    var result = {};
-    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
-    __setModuleDefault(result, mod);
-    return result;
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.Rule = void 0;
-const Lint = __importStar(require("tslint"));
-const tsutils_1 = require("tsutils");
-const ts = __importStar(require("typescript"));
-/**
- * Validate valid GUID strings are being passed as an argument to SPKillSwitch.isActivated calls.
- * JavaScript does not have a native GUID type. We removed expensive runtime parsing during production
- * builds when checking kill switches. In order to ensure type safety, we are validating at build time with tslint.
- */
-class Rule extends Lint.Rules.AbstractRule {
-    apply(source) {
-        return this.applyWithFunction(source, validKillSwitchGuid);
-    }
-}
-exports.Rule = Rule;
-Rule.metadata = {
-    ruleName: 'valid-kill-switch-guid',
-    description: 'Warn if _SPKillSwitch.isActivated is being called with an incorrect argument',
-    rationale: 'Compile time type safety',
-    optionsDescription: 'No options available.',
-    options: undefined,
-    type: 'functionality',
-    typescriptOnly: false
-};
-function validKillSwitchGuid(context, node) {
-    if (!node) {
-        ts.forEachChild(context.sourceFile, (n) => validKillSwitchGuid(context, n));
-    }
-    else {
-        if ((0, tsutils_1.isCallExpression)(node) && isSPKillSwitch(node) && !isValidGuid(node.arguments[0].getText())) {
-            context.addFailureAtNode(node, '_SPKillSwitch.isActivated argument must be a valid, lower-case GUID');
-        }
-        ts.forEachChild(node, (n) => validKillSwitchGuid(context, n));
-    }
-}
-function isSPKillSwitch(node) {
-    return ((0, tsutils_1.isPropertyAccessExpression)(node.expression) &&
-        node.expression.name.getText() === 'isActivated' &&
-        (0, tsutils_1.isIdentifier)(node.expression.expression) &&
-        (node.expression.expression.getText() === '_SPKillSwitch' ||
-            node.expression.expression.getText() === 'SPKillSwitch') &&
-        node.arguments.length === 1);
-}
-function isValidGuid(guidString) {
-    return /^'[0-9a-f]{8}-?[0-9a-f]{4}-?[0-9a-f]{4}-?[0-9a-f]{4}-?[0-9a-f]{12}'$/.test(guidString);
-}
-//# sourceMappingURL=validKillSwitchGuidRule.js.map