Ts/gatsby core utils

packages/gatsby-core-utils/package.json

@@ -16,9 +16,9 @@
     "directory": "packages/gatsby-core-utils"
   },
   "scripts": {
-    "build": "babel src --out-dir dist/ --ignore **/__tests__",
+    "build": "babel src --out-dir dist/ --ignore **/__tests__ --ignore **/__mocks__ --extensions \".ts\"",
     "prepare": "cross-env NODE_ENV=production npm run build",
-    "watch": "babel -w src --out-dir dist/ --ignore **/__tests__"
+    "watch": "babel -w src --out-dir dist/ --ignore **/__tests__ --extensions \".ts,.js\""
   },
   "bugs": {
     "url": "https://github.com/gatsbyjs/gatsby/issues"
@@ -36,6 +36,7 @@
   "devDependencies": {
     "@babel/cli": "^7.7.5",
     "@babel/core": "^7.7.5",
+    "@types/ci-info": "2.0.0",
     "babel-preset-gatsby-package": "^0.2.17",
     "cross-env": "^5.2.1"
   }

packages/gatsby-core-utils/src/__mocks__/ci-info.ts

@@ -0,0 +1,24 @@
+import ci from "ci-info"
+const ciInfo: typeof ci = jest.genMockFromModule(`ci-info`)
+
+// @ts-ignore
+ciInfo.isCI = false
+// @ts-ignore
+ciInfo.name = `bad default`
+
+function setIsCI(value: boolean): void {
+  // @ts-ignore
+  ciInfo.isCI = value
+}
+
+function setName(name: string): void {
+  // @ts-ignore
+  ciInfo.name = name
+}
+
+// @ts-ignore
+ciInfo.setIsCI = setIsCI
+// @ts-ignore
+ciInfo.setName = setName
+
+export default ciInfo

packages/gatsby-core-utils/src/__tests__/ci.js → packages/gatsby-core-utils/src/__tests__/ci.ts

@@ -30,7 +30,7 @@ describe(`CI detection`, () => {
   })
 
   it(`Detects Now v2`, () => {
-    process.env.NOW_BUILDER_ANNOTATE = 1
+    process.env.NOW_BUILDER_ANNOTATE = `1`
     const { isCI, getCIName } = require(`../ci`)
 
     expect(isCI()).toBeTruthy()
@@ -53,15 +53,15 @@ describe(`CI detection`, () => {
     expect(getCIName()).toEqual(`Heroku`)
   })
   it(`Detects CI and CI_NAME`, () => {
-    process.env.CI = true
+    process.env.CI = `true`
     process.env.CI_NAME = `test CI`
     const { isCI, getCIName } = require(`../ci`)
 
     expect(isCI()).toBeTruthy()
     expect(getCIName()).toEqual(`test CI`)
   })
   it(`Detects CI and no CI_NAME`, () => {
-    process.env.CI = true
+    process.env.CI = `true`
     delete process.env.CI_NAME
     const { isCI, getCIName } = require(`../ci`)
 

packages/gatsby-core-utils/src/__tests__/cpu-core-count.js → packages/gatsby-core-utils/src/__tests__/cpu-core-count.ts

@@ -1,22 +1,22 @@
 jest.mock(`../physical-cpu-count`, () => 1)
-const getCPUCoreCount = require(`../cpu-core-count`)
+import { cpuCoreCount } from "../cpu-core-count"
 
 beforeEach(() => {
   delete process.env.GATSBY_CPU_COUNT
 })
 
 test(`it defaults to physical CPU count, if override not detected`, () => {
-  expect(getCPUCoreCount()).toBe(1)
+  expect(cpuCoreCount(false)).toBe(1)
 })
 
 test(`it does not use env far override, if ignoreEnvVar is true`, () => {
-  process.env.GATSBY_CPU_COUNT = 9001
+  process.env.GATSBY_CPU_COUNT = `9001`
 
-  expect(getCPUCoreCount(true)).not.toBe(Number(process.env.GATSBY_CPU_COUNT))
+  expect(cpuCoreCount(true)).not.toBe(Number(process.env.GATSBY_CPU_COUNT))
 })
 
 test(`uses env var override, if exists`, () => {
-  process.env.GATSBY_CPU_COUNT = 9001
+  process.env.GATSBY_CPU_COUNT = `9001`
 
-  expect(getCPUCoreCount()).toBe(Number(process.env.GATSBY_CPU_COUNT))
+  expect(cpuCoreCount(false)).toBe(Number(process.env.GATSBY_CPU_COUNT))
 })

packages/gatsby-core-utils/src/__tests__/create-content-digest.js → packages/gatsby-core-utils/src/__tests__/create-content-digest.ts

@@ -1,4 +1,4 @@
-const createContentDigest = require(`../create-content-digest`)
+import { createContentDigest } from "../create-content-digest"
 
 describe(`Create content digest`, () => {
   it(`returns the content digest when the input is a string`, () => {

packages/gatsby-core-utils/src/__tests__/path.js → packages/gatsby-core-utils/src/__tests__/path.ts

@@ -1,5 +1,5 @@
-const { joinPath, isNodeInternalModulePath, slash } = require(`../path`)
-const os = require(`os`)
+import { joinPath, isNodeInternalModulePath, slash } from "../path"
+import os from "os"
 
 describe(`paths`, () => {
   describe(`joinPath`, () => {

packages/gatsby-core-utils/src/__tests__/physical-cpu-count.js → packages/gatsby-core-utils/src/__tests__/physical-cpu-count.ts

@@ -18,8 +18,8 @@ describe(`physical-cpu-count`, () => {
       const cProc = require(`child_process`)
       cProc.execSync.mockImplementation(() => `4`)
       mockPlatform(platform)
-      const pcc = require(`../physical-cpu-count`)
-      expect(pcc).toBe(4)
+      const { getPhysicalCpuCount } = require(`../physical-cpu-count`)
+      expect(getPhysicalCpuCount()).toBe(4)
     }
   )
 
@@ -29,8 +29,8 @@ describe(`physical-cpu-count`, () => {
       const cProc = require(`child_process`)
       cProc.execSync.mockImplementation(() => `4`)
       mockPlatform(platform)
-      const pcc = require(`../physical-cpu-count`)
-      expect(pcc).toBe(4)
+      const { getPhysicalCpuCount } = require(`../physical-cpu-count`)
+      expect(getPhysicalCpuCount()).toBe(4)
     }
   )
 
@@ -43,8 +43,8 @@ describe(`physical-cpu-count`, () => {
     `
     )
     mockPlatform(`win32`)
-    const pcc = require(`../physical-cpu-count`)
-    expect(pcc).toBe(4)
+    const { getPhysicalCpuCount } = require(`../physical-cpu-count`)
+    expect(getPhysicalCpuCount()).toBe(4)
   })
 
   it(`should return fallback CPU count on Windows when childProcess fails`, () => {
@@ -53,8 +53,8 @@ describe(`physical-cpu-count`, () => {
       throw new Error(`Fail!`)
     })
     mockPlatform(`win32`)
-    const pcc = require(`../physical-cpu-count`)
-    expect(pcc).toBe(1)
+    const { getPhysicalCpuCount } = require(`../physical-cpu-count`)
+    expect(getPhysicalCpuCount()).toBe(1)
   })
 
   it(`should check for hyperthreading when intel is the processor`, () => {
@@ -65,7 +65,7 @@ describe(`physical-cpu-count`, () => {
 
     os.cpus.mockImplementation(() => [{ model: `Intel` }, { model: `Intel` }])
     mockPlatform(`linux`)
-    const pcc = require(`../physical-cpu-count`)
-    expect(pcc).toBe(1)
+    const { getPhysicalCpuCount } = require(`../physical-cpu-count`)
+    expect(getPhysicalCpuCount()).toBe(1)
   })
 })

packages/gatsby-core-utils/src/__tests__/url.js → packages/gatsby-core-utils/src/__tests__/url.ts

@@ -1,4 +1,4 @@
-const { resolve } = require(`../url`)
+import { resolve } from "../url"
 
 describe(`url`, () => {
   describe(`resolve`, () => {

packages/gatsby-core-utils/src/ci.js → packages/gatsby-core-utils/src/ci.ts

@@ -1,4 +1,4 @@
-const ci = require(`ci-info`)
+import ci from "ci-info"
 
 const CI_DEFINITIONS = [
   envFromCIandCIName,
@@ -9,7 +9,7 @@ const CI_DEFINITIONS = [
   envFromCIWithNoName,
 ]
 
-function lookupCI() {
+function lookupCI(): string | null {
   for (const fn of CI_DEFINITIONS) {
     try {
       const res = fn()
@@ -24,45 +24,53 @@ function lookupCI() {
 }
 const CIName = lookupCI()
 
-function isCI() {
+export function isCI(): boolean {
   return !!CIName
 }
 
-function getCIName() {
+export function getCIName(): string | null {
   if (!isCI()) {
     return null
   }
   return CIName
 }
 
-module.exports = { isCI, getCIName }
-
-function getEnvFromCIInfo() {
+function getEnvFromCIInfo(): string | null {
   if (ci.isCI) return ci.name || `ci-info detected w/o name`
   return null
 }
 
-function getEnvDetect({ key, name }) {
-  return function() {
+interface IGetEnvDetect {
+  // @prettier-ignore
+  key: string
+  name: string
+}
+
+function getEnvDetect({ key, name }: IGetEnvDetect): () => string | null {
+  return function(): string | null {
     if (process.env[key]) {
       return name
     }
     return null
   }
 }
 
-function herokuDetect() {
-  return /\.heroku\/node\/bin\/node/.test(process.env.NODE) && `Heroku`
+function herokuDetect(): false | "Heroku" {
+  return (
+    typeof process.env.NODE === `string` &&
+    /\.heroku\/node\/bin\/node/.test(process.env.NODE) &&
+    `Heroku`
+  )
 }
 
-function envFromCIandCIName() {
+function envFromCIandCIName(): string | null {
   if (process.env.CI_NAME && process.env.CI) {
     return process.env.CI_NAME
   }
   return null
 }
 
-function envFromCIWithNoName() {
+function envFromCIWithNoName(): "CI detected without name" | null {
   if (process.env.CI) {
     return `CI detected without name`
   }

packages/gatsby-core-utils/src/cpu-core-count.js → packages/gatsby-core-utils/src/cpu-core-count.ts

@@ -1,9 +1,9 @@
-/**
- * @type {import('../index').cpuCoreCount}
- */
-const cpuCoreCount = ignoreEnvVar => {
+import getPhysicalCpuCount from "./physical-cpu-count"
+console.log(getPhysicalCpuCount)
+
+export const cpuCoreCount = (ignoreEnvVar: boolean): number => {
   try {
-    let coreCount = require(`./physical-cpu-count`) || 1
+    let coreCount = getPhysicalCpuCount() || 1
 
     if (ignoreEnvVar) {
       // Return the physical CPU count,
@@ -47,5 +47,3 @@ const cpuCoreCount = ignoreEnvVar => {
     throw new Error(`There has been a problem counting the number of CPU cores`)
   }
 }
-
-module.exports = cpuCoreCount

packages/gatsby-core-utils/src/create-content-digest.js → packages/gatsby-core-utils/src/create-content-digest.ts

@@ -1,5 +1,5 @@
-const crypto = require(`crypto`)
-const objectHash = require(`node-object-hash`)
+import crypto, { BinaryLike } from "crypto"
+import objectHash from "node-object-hash"
 
 const hasher = objectHash({
   coerce: false,
@@ -13,21 +13,18 @@ const hasher = objectHash({
   },
 })
 
-const hashPrimitive = input =>
+const hashPrimitive = (input: BinaryLike | string): string =>
   crypto
     .createHash(`md5`)
     .update(input)
     .digest(`hex`)
 
-/**
- * @type {import('../index').createContentDigest}
- */
-const createContentDigest = input => {
+export const createContentDigest = (
+  input: BinaryLike | string | any
+): string => {
   if (typeof input === `object`) {
     return hasher.hash(input)
   }
 
   return hashPrimitive(input)
 }
-
-module.exports = createContentDigest

packages/gatsby-core-utils/src/create-require-from-path.ts

@@ -0,0 +1,18 @@
+import Module from "module"
+import path from "path"
+
+const fallback = (filename: string): NodeRequire => {
+  // @ts-ignore
+  const mod = new Module(filename, null)
+
+  mod.filename = filename
+  // @ts-ignore
+  mod.paths = Module._nodeModulePaths(path.dirname(filename))
+  mod._compile(`module.exports = require;`, filename)
+
+  return mod.exports
+}
+
+// Polyfill Node's `Module.createRequireFromPath` if not present (added in Node v10.12.0)
+export const createRequireFromPath =
+  Module.createRequire || Module.createRequireFromPath || fallback

packages/gatsby-core-utils/src/get-config-store.js → packages/gatsby-core-utils/src/get-config-store.ts

@@ -1,8 +1,8 @@
 import Configstore from "configstore"
 
-let config
+let config: Configstore
 
-module.exports = () => {
+export const getConfigStore = (): Configstore => {
   if (!config) {
     config = new Configstore(
       `gatsby`,

packages/gatsby-core-utils/src/get-gatsby-version.js → packages/gatsby-core-utils/src/get-gatsby-version.ts

@@ -1,6 +1,6 @@
 import path from "path"
 
-module.exports = () => {
+export const getGatsbyVersion = (): string => {
   try {
     return require(path.join(
       process.cwd(),