Require props alias rule

package.json

@@ -12,7 +12,9 @@
     "prettier-check": "diffs=$(prettier --list-different '{src,tests,types,examples,docs}/**/*.ts?(x)'); if [ ! -z $diffs ]; then echo \"Run 'npm run prettier'\nThe following files need formatting:\n$diffs\" && exit 1; fi;",
     "tests": "jest",
     "test": "npm run lint && npm run tests -- --runInBand --coverage",
-    "prettier": "prettier --write '{src,tests,types,examples,docs}/**/*.ts?(x)'"
+    "prettier": "prettier --write '{src,tests,types,examples,docs}/**/*.ts?(x)'",
+    "test-files": "npm run dist && tslint --project tsconfig.json 'test-files/**/*.@(ts|tsx)'",
+    "prepublishOnly": "npm run dist"
   },
   "repository": {
     "type": "git",
@@ -37,8 +39,13 @@
   },
   "devDependencies": {
     "@types/jest": "^23.3.2",
+    "@types/node": "^10.10.3",
+    "@types/react": "^16.4.14",
+    "@types/react-redux": "^6.0.9",
     "jest": "^23.6.0",
     "prettier": "^1.14.2",
+    "react": "^16.5.2",
+    "react-redux": "^5.0.7",
     "ts-jest": "^23.10.1",
     "tslint": "^5.11.0",
     "tslint-config-dabapps": "github:dabapps/tslint-config-dabapps#v0.5.1"

src/__reference-rule__.ts

@@ -0,0 +1,21 @@
+import * as Lint from 'tslint';
+import * as ts from 'typescript';
+
+const FAILURE_STRING = 'Import statements are disallowed';
+
+// The walker takes care of all the work.
+class NoImportsWalker extends Lint.RuleWalker {
+  public visitImportDeclaration (node: ts.ImportDeclaration) {
+    // create a failure at the current position
+    this.addFailure(this.createFailure(node.getStart(), node.getWidth(), FAILURE_STRING));
+
+    // call the base version of this visitor to actually parse this node
+    super.visitImportDeclaration(node);
+  }
+}
+
+export class Rule extends Lint.Rules.AbstractRule {
+  public apply(sourceFile: ts.SourceFile): Lint.RuleFailure[] {
+    return this.applyWithWalker(new NoImportsWalker(sourceFile, this.getOptions()));
+  }
+}

src/index.ts

@@ -1,3 +1 @@
-export const message = 'Hello, World!';
-
-export default message;
+export = { rulesDirectory: '.' };

src/noPropIntersectionRule.ts

@@ -0,0 +1,55 @@
+import * as Lint from 'tslint';
+import * as ts from 'typescript';
+
+const FAILURE_STRING = 'Props must not be an intersection in Component or PureComponent params';
+
+// The walker takes care of all the work.
+class NoPropIntersectionWalker extends Lint.RuleWalker {
+  public visitClassDeclaration (node: ts.ClassDeclaration) {
+    if (node.heritageClauses && node.heritageClauses.length) {
+      node.heritageClauses.forEach((heritage) => {
+        heritage.forEachChild((child) => {
+          if (child.kind === ts.SyntaxKind.ExpressionWithTypeArguments) {
+            this.checkExpression(child as ts.ExpressionWithTypeArguments);
+          }
+        });
+      });
+    }
+
+    // call the base version of this visitor to actually parse this node
+    super.visitClassDeclaration(node);
+  }
+
+  private getExpressionName (node: ts.ExpressionWithTypeArguments) {
+    if (ts.isIdentifier(node.expression)) {
+      return (node.expression as ts.Identifier).escapedText;
+    } else if (node.expression.kind === ts.SyntaxKind.PropertyAccessExpression) {
+      return (node.expression as ts.PropertyAccessExpression).name.escapedText;
+    }
+
+    return '';
+  }
+
+  private checkExpression (node: ts.ExpressionWithTypeArguments) {
+    const name = this.getExpressionName(node);
+
+    if (name === 'PureComponent' || name === 'Component') {
+      const { typeArguments } = node;
+
+      if (typeArguments) {
+        typeArguments.forEach((typeArgument) => {
+          if (typeArgument.kind === ts.SyntaxKind.IntersectionType) {
+            // create a failure at the current position
+            this.addFailure(this.createFailure(node.getStart(), node.getWidth(), FAILURE_STRING));
+          }
+        });
+      }
+    }
+  }
+}
+
+export class Rule extends Lint.Rules.AbstractRule {
+  public apply(sourceFile: ts.SourceFile): Lint.RuleFailure[] {
+    return this.applyWithWalker(new NoPropIntersectionWalker(sourceFile, this.getOptions()));
+  }
+}

src/noPropUnionRule.ts

@@ -0,0 +1,55 @@
+import * as Lint from 'tslint';
+import * as ts from 'typescript';
+
+const FAILURE_STRING = 'Props must not be an union in Component or PureComponent params';
+
+// The walker takes care of all the work.
+class NoPropUnionWalker extends Lint.RuleWalker {
+  public visitClassDeclaration (node: ts.ClassDeclaration) {
+    if (node.heritageClauses && node.heritageClauses.length) {
+      node.heritageClauses.forEach((heritage) => {
+        heritage.forEachChild((child) => {
+          if (child.kind === ts.SyntaxKind.ExpressionWithTypeArguments) {
+            this.checkExpression(child as ts.ExpressionWithTypeArguments);
+          }
+        });
+      });
+    }
+
+    // call the base version of this visitor to actually parse this node
+    super.visitClassDeclaration(node);
+  }
+
+  private getExpressionName (node: ts.ExpressionWithTypeArguments) {
+    if (ts.isIdentifier(node.expression)) {
+      return (node.expression as ts.Identifier).escapedText;
+    } else if (node.expression.kind === ts.SyntaxKind.PropertyAccessExpression) {
+      return (node.expression as ts.PropertyAccessExpression).name.escapedText;
+    }
+
+    return '';
+  }
+
+  private checkExpression (node: ts.ExpressionWithTypeArguments) {
+    const name = this.getExpressionName(node);
+
+    if (name === 'PureComponent' || name === 'Component') {
+      const { typeArguments } = node;
+
+      if (typeArguments) {
+        typeArguments.forEach((typeArgument) => {
+          if (typeArgument.kind === ts.SyntaxKind.UnionType) {
+            // create a failure at the current position
+            this.addFailure(this.createFailure(node.getStart(), node.getWidth(), FAILURE_STRING));
+          }
+        });
+      }
+    }
+  }
+}
+
+export class Rule extends Lint.Rules.AbstractRule {
+  public apply(sourceFile: ts.SourceFile): Lint.RuleFailure[] {
+    return this.applyWithWalker(new NoPropUnionWalker(sourceFile, this.getOptions()));
+  }
+}

src/requirePropAliasRule.ts

@@ -0,0 +1,21 @@
+import * as Lint from 'tslint';
+import * as ts from 'typescript';
+
+const FAILURE_STRING = 'Props must alias individual prop types (State, Dispatch, Own, etc)';
+
+// The walker takes care of all the work.
+class RequirePropAliasWalker extends Lint.RuleWalker {
+  public visitImportDeclaration (node: ts.ImportDeclaration) {
+    // create a failure at the current position
+    this.addFailure(this.createFailure(node.getStart(), node.getWidth(), FAILURE_STRING));
+
+    // call the base version of this visitor to actually parse this node
+    super.visitImportDeclaration(node);
+  }
+}
+
+export class Rule extends Lint.Rules.AbstractRule {
+  public apply(sourceFile: ts.SourceFile): Lint.RuleFailure[] {
+    return this.applyWithWalker(new RequirePropAliasWalker(sourceFile, this.getOptions()));
+  }
+}