refactor(rules): port rules to typescript (conventional-changelog#785)

* refactor(ensure): expose target case type for rules

* refactor(rules): rewrite all rules and tests to typescript

* refactor(rules): refactor indentation and readability for case rules

* test(rules): ignore types file and use ts extension

* test(rules): import non-typed preset with require

* chore: remove javascript rules pattern from jest

* fix(rules): export rule types for implementing packages

Author: byCedric

@commitlint/ensure/src/case.ts

@@ -2,7 +2,7 @@ import * as _ from 'lodash';
 
 export default ensureCase;
 
-type TargetCaseType =
+export type TargetCaseType =
 	| 'camel-case'
 	| 'kebab-case'
 	| 'snake-case'

@commitlint/ensure/src/index.ts

@@ -8,3 +8,4 @@ import notEmpty from './not-empty';
 export {ensureCase as case};
 export {ensureEnum as enum};
 export {maxLength, maxLineLength, minLength, notEmpty};
+export {TargetCaseType} from './case';

@commitlint/rules/package.json

@@ -3,20 +3,13 @@
   "version": "8.3.4",
   "description": "Lint your commit messages",
   "main": "lib/index.js",
+  "types": "lib/index.d.ts",
   "files": [
     "lib/"
   ],
   "scripts": {
-    "build": "cross-env NODE_ENV=production babel src --out-dir lib  --source-maps",
     "deps": "dep-check",
-    "pkg": "pkg-check --skip-import",
-    "start": "concurrently \"ava -c 4 --verbose --watch\" \"yarn run watch\"",
-    "watch": "babel src --out-dir lib --watch --source-maps"
-  },
-  "babel": {
-    "presets": [
-      "babel-preset-commitlint"
-    ]
+    "pkg": "pkg-check"
   },
   "engines": {
     "node": ">=4"

@commitlint/rules/src/body-case.test.js renamed to @commitlint/rules/src/body-case.test.ts

@@ -1,5 +1,5 @@
 import parse from '@commitlint/parse';
-import bodyCase from './body-case';
+import {bodyCase} from './body-case';
 
 const messages = {
 	empty: 'test: subject',

@commitlint/rules/src/body-case.js renamed to @commitlint/rules/src/body-case.ts

@@ -1,7 +1,12 @@
 import * as ensure from '@commitlint/ensure';
 import message from '@commitlint/message';
+import {Rule} from './types';
 
-export default (parsed, when, value) => {
+export const bodyCase: Rule<ensure.TargetCaseType> = (
+	parsed,
+	when = 'always',
+	value = undefined
+) => {
 	const {body} = parsed;
 
 	if (!body) {

@commitlint/rules/src/body-empty.test.js renamed to @commitlint/rules/src/body-empty.test.ts

@@ -1,5 +1,5 @@
 import parse from '@commitlint/parse';
-import bodyEmpty from './body-empty';
+import {bodyEmpty} from './body-empty';
 
 const messages = {
 	empty: 'test: subject',

@commitlint/rules/src/body-empty.js renamed to @commitlint/rules/src/body-empty.ts

@@ -1,9 +1,10 @@
 import * as ensure from '@commitlint/ensure';
 import message from '@commitlint/message';
+import {Rule} from './types';
 
-export default (parsed, when) => {
+export const bodyEmpty: Rule = (parsed, when = 'always') => {
 	const negated = when === 'never';
-	const notEmpty = ensure.notEmpty(parsed.body);
+	const notEmpty = ensure.notEmpty(parsed.body || '');
 
 	return [
 		negated ? notEmpty : !notEmpty,

@commitlint/rules/src/body-leading-blank.test.js renamed to @commitlint/rules/src/body-leading-blank.test.ts

@@ -1,5 +1,5 @@
 import parse from '@commitlint/parse';
-import bodyLeadingBlank from './body-leading-blank';
+import {bodyLeadingBlank} from './body-leading-blank';
 
 const messages = {
 	simple: 'test: subject',

@commitlint/rules/src/body-leading-blank.js renamed to @commitlint/rules/src/body-leading-blank.ts

@@ -1,7 +1,8 @@
 import toLines from '@commitlint/to-lines';
 import message from '@commitlint/message';
+import {Rule} from './types';
 
-export default (parsed, when) => {
+export const bodyLeadingBlank: Rule = (parsed, when) => {
 	// Flunk if no body is found
 	if (!parsed.body) {
 		return [true];

@commitlint/rules/src/body-max-length.test.js renamed to @commitlint/rules/src/body-max-length.test.ts

@@ -1,5 +1,5 @@
 import parse from '@commitlint/parse';
-import check from './body-max-length';
+import {bodyMaxLength} from './body-max-length';
 
 const short = 'a';
 const long = 'ab';
@@ -19,19 +19,19 @@ const parsed = {
 };
 
 test('with empty should succeed', async () => {
-	const [actual] = check(await parsed.empty, '', value);
+	const [actual] = bodyMaxLength(await parsed.empty, undefined, value);
 	const expected = true;
 	expect(actual).toEqual(expected);
 });
 
 test('with short should succeed', async () => {
-	const [actual] = check(await parsed.short, '', value);
+	const [actual] = bodyMaxLength(await parsed.short, undefined, value);
 	const expected = true;
 	expect(actual).toEqual(expected);
 });
 
 test('with long should fail', async () => {
-	const [actual] = check(await parsed.long, '', value);
+	const [actual] = bodyMaxLength(await parsed.long, undefined, value);
 	const expected = false;
 	expect(actual).toEqual(expected);
 });