Replace flow + babel with tsc

muan · muan · commit a60d4f6c9d2d · 2020-02-18T18:37:38.000-05:00
diff --git a/.eslintignore b/.eslintignore
@@ -1,2 +1,4 @@
 dist
 prettier.config.js
+karma.config.js
+rollup.config.js
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -1,8 +1,15 @@
 {
   "extends": [
-    "plugin:flowtype/recommended",
+    "plugin:github/browser",
     "plugin:github/es6",
-    "plugin:github/browser"
+    "plugin:github/typescript"
   ],
-  "parser": "babel-eslint"
+  "overrides": [
+    {
+      "files": "test/*.js",
+      "rules": {
+        "github/unescaped-html-literal": "off"
+      }
+    }
+  ]
 }
diff --git a/.flowconfig b/.flowconfig
diff --git a/index.d.ts b/index.d.ts
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -5,12 +5,12 @@
   "repository": "github/paste-markdown",
   "main": "dist/index.umd.js",
   "module": "dist/index.esm.js",
-  "types": "index.d.ts",
+  "types": "dist/index.d.ts",
   "scripts": {
     "clean": "rm -rf dist",
     "lint": "github-lint",
     "prebuild": "npm run clean && npm run lint && mkdir dist",
-    "build": "rollup -c && cp src/index.js.flow dist/index.umd.js.flow && cp src/index.js.flow dist/index.esm.js.flow",
+    "build": "rollup -c",
     "pretest": "npm run build",
     "test": "karma start karma.config.js",
     "prepublishOnly": "npm run build",
@@ -22,24 +22,23 @@
   ],
   "license": "MIT",
   "files": [
-    "dist",
-    "index.d.ts"
+    "dist/index.d.ts",
+    "dist/index.esm.js",
+    "dist/index.umd.js"
   ],
   "devDependencies": {
-    "@babel/core": "^7.7.0",
-    "babel-eslint": "^10.0.3",
-    "babel-preset-github": "^3.2.1",
+    "@rollup/plugin-typescript": "^3.0.0",
     "chai": "^4.2.0",
-    "eslint": "^6.6.0",
-    "eslint-plugin-github": "^3.2.1",
-    "flow-bin": "^0.111.1",
+    "eslint": "^6.8.0",
+    "eslint-plugin-github": "^3.4.0",
     "karma": "^4.4.1",
     "karma-chai": "^0.1.0",
     "karma-chrome-launcher": "^3.1.0",
     "karma-mocha": "^1.3.0",
     "karma-mocha-reporter": "^2.2.5",
     "mocha": "^7.0.1",
-    "rollup": "^1.26.3",
-    "rollup-plugin-babel": "^4.3.3"
+    "rollup": "^1.31.1",
+    "rollup-plugin-typescript2": "^0.26.0",
+    "typescript": "^3.7.5"
   }
 }
diff --git a/rollup.config.js b/rollup.config.js
@@ -1,11 +1,7 @@
-/* @flow strict */
-
-import babel from 'rollup-plugin-babel'
-
-const pkg = require('./package.json')
-
+import typescript from 'rollup-plugin-typescript2'
+import pkg from './package.json'
 export default {
-  input: 'src/index.js',
+  input: 'src/index.ts',
   output: [
     {
       file: pkg['module'],
@@ -18,8 +14,6 @@ export default {
     }
   ],
   plugins: [
-    babel({
-      presets: ['github']
-    })
+    typescript()
   ]
 }
diff --git a/src/index.js.flow b/src/index.js.flow
diff --git a/src/index.ts b/src/index.ts
@@ -1,14 +1,12 @@
-/* @flow strict */
-
 import {install as installLink, uninstall as uninstallLink} from './paste-markdown-image-link'
 import {install as installTable, uninstall as uninstallTable} from './paste-markdown-table'
 import {install as installText, uninstall as uninstallText} from './paste-markdown-text'
 
-type Subscription = {|
+interface Subscription {
   unsubscribe: () => void
-|}
+}
 
-export default function subscribe(el: Element): Subscription {
+export default function subscribe(el: HTMLElement): Subscription {
   installTable(el)
   installLink(el)
   installText(el)
diff --git a/src/paste-markdown-image-link.ts b/src/paste-markdown-image-link.ts
@@ -2,13 +2,13 @@
 
 import {insertText} from './text'
 
-export function install(el: Element) {
+export function install(el: HTMLElement) {
   el.addEventListener('dragover', onDragover)
   el.addEventListener('drop', onDrop)
   el.addEventListener('paste', onPaste)
 }
 
-export function uninstall(el: Element) {
+export function uninstall(el: HTMLElement) {
   el.removeEventListener('dragover', onDragover)
   el.removeEventListener('drop', onDrop)
   el.removeEventListener('paste', onPaste)
@@ -64,7 +64,7 @@ function hasLink(transfer: DataTransfer): boolean {
   return Array.from(transfer.types).indexOf('text/uri-list') >= 0
 }
 
-function extractLinks(transfer: DataTransfer): Array<string> {
+function extractLinks(transfer: DataTransfer): string[] {
   return (transfer.getData('text/uri-list') || '').split('\r\n')
 }
 
diff --git a/src/paste-markdown-table.ts b/src/paste-markdown-table.ts
@@ -2,13 +2,13 @@
 
 import {insertText} from './text'
 
-export function install(el: Element) {
+export function install(el: HTMLElement) {
   el.addEventListener('dragover', onDragover)
   el.addEventListener('drop', onDrop)
   el.addEventListener('paste', onPaste)
 }
 
-export function uninstall(el: Element) {
+export function uninstall(el: HTMLElement) {
   el.removeEventListener('dragover', onDragover)
   el.removeEventListener('drop', onDrop)
   el.removeEventListener('paste', onPaste)
@@ -58,21 +58,23 @@ function hasFile(transfer: DataTransfer): boolean {
 
 function columnText(column: Element): string {
   const noBreakSpace = '\u00A0'
-  const text = column.textContent
+  const text = (column.textContent || '')
     .trim()
     .replace(/\|/g, '\\|')
     .replace(/\n/g, ' ')
   return text || noBreakSpace
 }
 
-function tableHeaders(row: Element): Array<string> {
+function tableHeaders(row: Element): string[] {
   return Array.from(row.querySelectorAll('td, th')).map(columnText)
 }
 
 function tableMarkdown(node: Element): string {
   const rows = Array.from(node.querySelectorAll('tr'))
 
-  const headers = tableHeaders(rows.shift())
+  const firstRow = rows.shift()
+  if (!firstRow) return ''
+  const headers = tableHeaders(firstRow)
   const spacers = headers.map(() => '--')
   const header = `${headers.join(' | ')}\n${spacers.join(' | ')}\n`
 
@@ -87,13 +89,13 @@ function tableMarkdown(node: Element): string {
   return `\n${header}${body}\n\n`
 }
 
-function parseTable(html: string): ?Element {
+function parseTable(html: string): HTMLElement | null {
   const el = document.createElement('div')
   el.innerHTML = html
   return el.querySelector('table')
 }
 
-function hasTable(transfer: DataTransfer): ?Element {
+function hasTable(transfer: DataTransfer): HTMLElement | null | void {
   if (Array.from(transfer.types).indexOf('text/html') === -1) return
 
   const html = transfer.getData('text/html')
diff --git a/src/paste-markdown-text.ts b/src/paste-markdown-text.ts
@@ -1,12 +1,10 @@
-/* @flow strict */
-
 import {insertText} from './text'
 
-export function install(el: Element) {
+export function install(el: HTMLElement) {
   el.addEventListener('paste', onPaste)
 }
 
-export function uninstall(el: Element) {
+export function uninstall(el: HTMLElement) {
   el.removeEventListener('paste', onPaste)
 }
 
diff --git a/src/text.ts b/src/text.ts
@@ -1,6 +1,4 @@
-/* @flow strict */
-
-export function insertText(textarea: HTMLInputElement | HTMLTextAreaElement, text: string): void {
+export function insertText(textarea: HTMLTextAreaElement, text: string): void {
   const beginning = textarea.value.substring(0, textarea.selectionStart)
   const remaining = textarea.value.substring(textarea.selectionEnd)
 
diff --git a/test/.eslintrc.json b/test/.eslintrc.json
@@ -1,12 +1,8 @@
 {
-  "rules": {
-    "flowtype/require-valid-file-annotation": "off"
-  },
   "env": {
     "mocha": true
   },
   "globals": {
     "assert": true
-  },
-  "extends": "../.eslintrc.json"
+  }
 }
diff --git a/tsconfig.json b/tsconfig.json
@@ -0,0 +1,15 @@
+{
+  "compilerOptions": {
+    "module": "esnext",
+    "target": "es2017",
+    "lib": [
+      "es2018",
+      "dom"
+    ],
+    "strict": true,
+    "declaration": true,
+    "outDir": "dist",
+    "removeComments": true,
+    "preserveConstEnums": true
+  }
+}

Original file line number	Diff line number	Diff line change
`@@ -1,11 +1,7 @@`
`1`		`-/* @flow strict */`
`2`		`-`
`3`		`-import babel from 'rollup-plugin-babel'`
`4`		`-`
`5`		`-const pkg = require('./package.json')`
`6`		`-`
	`1`	`+import typescript from 'rollup-plugin-typescript2'`
	`2`	`+import pkg from './package.json'`
`7`	`3`	`export default {`
`8`		`- input: 'src/index.js',`
	`4`	`+ input: 'src/index.ts',`
`9`	`5`	`output: [`
`10`	`6`	`{`
`11`	`7`	`file: pkg['module'],`
`@@ -18,8 +14,6 @@ export default {`
`18`	`14`	`}`
`19`	`15`	`],`
`20`	`16`	`plugins: [`
`21`		`- babel({`
`22`		`- presets: ['github']`
`23`		`- })`
	`17`	`+ typescript()`
`24`	`18`	`]`
`25`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -2,13 +2,13 @@`
`2`	`2`
`3`	`3`	`import {insertText} from './text'`
`4`	`4`
`5`		`-export function install(el: Element) {`
	`5`	`+export function install(el: HTMLElement) {`
`6`	`6`	`el.addEventListener('dragover', onDragover)`
`7`	`7`	`el.addEventListener('drop', onDrop)`
`8`	`8`	`el.addEventListener('paste', onPaste)`
`9`	`9`	`}`
`10`	`10`
`11`		`-export function uninstall(el: Element) {`
	`11`	`+export function uninstall(el: HTMLElement) {`
`12`	`12`	`el.removeEventListener('dragover', onDragover)`
`13`	`13`	`el.removeEventListener('drop', onDrop)`
`14`	`14`	`el.removeEventListener('paste', onPaste)`
`@@ -64,7 +64,7 @@ function hasLink(transfer: DataTransfer): boolean {`
`64`	`64`	`return Array.from(transfer.types).indexOf('text/uri-list') >= 0`
`65`	`65`	`}`
`66`	`66`
`67`		`-function extractLinks(transfer: DataTransfer): Array<string> {`
	`67`	`+function extractLinks(transfer: DataTransfer): string[] {`
`68`	`68`	`return (transfer.getData('text/uri-list') \|\| '').split('\r\n')`
`69`	`69`	`}`
`70`	`70`