package.json

@@ -19,7 +19,7 @@
     "author": "Marc Cataford <c.marcandre@gmail.com>",
     "license": "MIT",
     "files": [
-        "dist/*.ts"
+        "dist/*.js"
     ],
     "bin": "./dist/cli.js",
     "repository": {
@@ -29,7 +29,7 @@
     "scripts": {
         "prepack": "yarn build",
         "prebuild": "rm -rf dist",
-        "build": "babel src/index.ts src/cli.ts -d dist --extensions .ts",
+        "build": "tsc --project .",
         "lint": "eslint src/**/*.ts",
         "lint:fix": "yarn lint --fix",
         "test": "jest src",
@@ -39,15 +39,11 @@
         "types:watch": "yarn types --watch"
     },
     "devDependencies": {
-        "@babel/cli": "^7.8.4",
-        "@babel/core": "^7.8.6",
-        "@babel/preset-env": "^7.8.7",
-        "@babel/preset-typescript": "^7.12.7",
         "@tophat/eslint-config": "^0.7.0",
         "@types/jest": "^26.0.19",
         "@types/node": "^14.14.13",
-        "@typescript-eslint/eslint-plugin": "^4.9.1",
+        "@typescript-eslint/eslint-plugin": "^4.26.0",
-        "@typescript-eslint/parser": "^4.9.1",
+        "@typescript-eslint/parser": "^4.26.0",
         "eslint": "^7.0.0",
         "eslint-config-prettier": "^7.2.0",
         "eslint-import-resolver-node": "^0.3.4",
@@ -57,6 +53,6 @@
         "jest": "^26.0.1",
         "prettier": "^2.0.5",
         "ts-jest": "^26",
-        "typescript": "^4.1.3"
+        "typescript": "^4.3.2"
     }
 }

src/__tests__/index.test.ts

@@ -1,3 +1,4 @@
+// @ts-nocheck
 import { promises as fs } from 'fs'
 import { tmpdir } from 'os'
 import { join, resolve } from 'path'
@@ -37,9 +38,9 @@ async function createManifest(
     await createFile(path, JSON.stringify(configuration))
 }
 describe('Packwatch', () => {
-    let mockLogger
+    let mockLogger = jest.fn()
-    let mockWarn
+    let mockWarn = jest.fn()
-    let mockError
+    let mockError = jest.fn()
     let workspacePath
     beforeEach(() => {
         mockLogger = jest.spyOn(console, 'log').mockImplementation()

src/__tests__/utils.test.ts

@@ -1,3 +1,4 @@
+// @ts-nocheck
 import { convertSizeToBytes } from '../utils'
 
 describe('utils', () => {

src/index.d.ts

@@ -1,13 +1,13 @@
 export type PackwatchArguments = {
-    cwd?: string
+    cwd: string
-    isUpdatingManifest?: boolean
+    isUpdatingManifest: boolean
 }
 
 export type Report = {
     packageSize: string
-    unpackedSize?: string
+    unpackedSize: string
-    packageSizeBytes?: number
+    packageSizeBytes: number
-    unpackedSizeBytes?: number
+    unpackedSizeBytes: number
     limit?: string
     limitBytes?: number
 }

src/index.ts

@@ -5,6 +5,7 @@ import {
     createOrUpdateManifest,
     getCurrentPackageStats,
     getPreviousPackageStats,
+    mergeDefaultArguments,
 } from './utils'
 import type { PackwatchArguments } from './index.d'
 import { assertInPackageRoot } from './invariants'
@@ -12,10 +13,11 @@ import logger from './logger'
 
 const MANIFEST_FILENAME = '.packwatch.json'
 
-export default async function packwatch({
+export default async function packwatch(
-    cwd,
+    args: Partial<PackwatchArguments>,
-    isUpdatingManifest,
+): Promise<void> {
-}: PackwatchArguments): Promise<void> {
+    const { cwd, isUpdatingManifest } = mergeDefaultArguments(args)
+
     const manifestPath = resolve(join(cwd, MANIFEST_FILENAME))
 
     assertInPackageRoot(cwd)
@@ -50,7 +52,8 @@ export default async function packwatch({
         limit,
         limitBytes,
     } = previousStats
-    const hasExceededLimit = packageSizeBytes > limitBytes
+    const hasExceededLimit =
+        packageSizeBytes > (limitBytes ?? Number.MAX_SAFE_INTEGER)
 
     /*
      * If we are updating the manifest, we can write right away and terminate.

src/utils.ts

@@ -2,7 +2,7 @@ import { spawnSync } from 'child_process'
 import { readFileSync, writeFileSync } from 'fs'
 import { join, resolve } from 'path'
 
-import { Report } from './index.d'
+import { PackwatchArguments, Report } from './index.d'
 
 const PACKAGE_SIZE_PATT = /package size:\s*([0-9]+\.?[0-9]*\s+[A-Za-z]{1,2})/
 const UNPACKED_SIZE_PATT = /unpacked size:\s*([0-9]+\.?[0-9]*\s+[A-Za-z]{1,2})/
@@ -11,9 +11,18 @@ const SIZE_MAGNITUDE_PATT = /([0-9]+\.?[0-9]*)/
 
 const MANIFEST_FILENAME = '.packwatch.json'
 
+export function mergeDefaultArguments(
+    args: Partial<PackwatchArguments>,
+): PackwatchArguments {
+    return {
+        cwd: args.cwd ?? '.',
+        isUpdatingManifest: args.isUpdatingManifest ?? false,
+    }
+}
+
 export function convertSizeToBytes(sizeString: string): number {
-    const sizeSuffix = SIZE_SUFFIX_PATT.exec(sizeString)[1]
+    const sizeSuffix = SIZE_SUFFIX_PATT.exec(sizeString)?.[1] ?? ''
-    const sizeMagnitude = SIZE_MAGNITUDE_PATT.exec(sizeString)[1]
+    const sizeMagnitude = SIZE_MAGNITUDE_PATT.exec(sizeString)?.[1] ?? '0.0'
 
     let multiplier = 1
 
@@ -31,8 +40,8 @@ export function getCurrentPackageStats(cwd: string): Report {
         cwd,
     })
     const stderrString = String(stderr)
-    const packageSize = PACKAGE_SIZE_PATT.exec(stderrString)[1]
+    const packageSize = PACKAGE_SIZE_PATT.exec(stderrString)?.[1] ?? '0'
-    const unpackedSize = UNPACKED_SIZE_PATT.exec(stderrString)[1]
+    const unpackedSize = UNPACKED_SIZE_PATT.exec(stderrString)?.[1] ?? '0'
 
     return {
         packageSize,
@@ -42,7 +51,7 @@ export function getCurrentPackageStats(cwd: string): Report {
     }
 }
 
-export function getPreviousPackageStats(cwd: string): Report | null {
+export function getPreviousPackageStats(cwd: string): Report {
     const manifestPath = resolve(join(cwd, MANIFEST_FILENAME))
     try {
         const currentManifest = readFileSync(manifestPath, {
@@ -57,6 +66,13 @@ export function getPreviousPackageStats(cwd: string): Report | null {
         }
     } catch {
         /* No manifest */
+        return {
+            packageSize: '0',
+            packageSizeBytes: 0,
+            unpackedSizeBytes: 0,
+            unpackedSize: '0',
+            limitBytes: 0,
+        }
     }
 }
 

tsconfig.json

@@ -0,0 +1,73 @@
+{
+  "compilerOptions": {
+    /* Visit https://aka.ms/tsconfig.json to read more about this file */
+
+    /* Basic Options */
+    // "incremental": true,                         /* Enable incremental compilation */
+    "target": "es5",                                /* Specify ECMAScript target version: 'ES3' (default), 'ES5', 'ES2015', 'ES2016', 'ES2017', 'ES2018', 'ES2019', 'ES2020', 'ES2021', or 'ESNEXT'. */
+    "module": "commonjs",                           /* Specify module code generation: 'none', 'commonjs', 'amd', 'system', 'umd', 'es2015', 'es2020', or 'ESNext'. */
+    // "lib": [],                                   /* Specify library files to be included in the compilation. */
+    "allowJs": false,                             /* Allow javascript files to be compiled. */
+    // "checkJs": true,                             /* Report errors in .js files. */
+    // "jsx": "preserve",                           /* Specify JSX code generation: 'preserve', 'react-native', 'react', 'react-jsx' or 'react-jsxdev'. */
+    "declaration": true,                         /* Generates corresponding '.d.ts' file. */
+    "declarationMap": true,                      /* Generates a sourcemap for each corresponding '.d.ts' file. */
+    "sourceMap": true,                           /* Generates corresponding '.map' file. */
+    // "outFile": "./",                             /* Concatenate and emit output to single file. */
+    "outDir": "./dist",                              /* Redirect output structure to the directory. */
+    "rootDir": "./src",                             /* Specify the root directory of input files. Use to control the output directory structure with --outDir. */
+        // "composite": true,                           /* Enable project compilation */
+    // "tsBuildInfoFile": "./",                     /* Specify file to store incremental compilation information */
+    "removeComments": true,                      /* Do not emit comments to output. */
+    // "noEmit": true,                              /* Do not emit outputs. */
+    // "importHelpers": true,                       /* Import emit helpers from 'tslib'. */
+    // "downlevelIteration": true,                  /* Provide full support for iterables in 'for-of', spread, and destructuring when targeting 'ES5' or 'ES3'. */
+    // "isolatedModules": true,                     /* Transpile each file as a separate module (similar to 'ts.transpileModule'). */
+
+    /* Strict Type-Checking Options */
+    "strict": true,                                 /* Enable all strict type-checking options. */
+    // "noImplicitAny": true,                       /* Raise error on expressions and declarations with an implied 'any' type. */
+    // "strictNullChecks": true,                    /* Enable strict null checks. */
+    // "strictFunctionTypes": true,                 /* Enable strict checking of function types. */
+    // "strictBindCallApply": true,                 /* Enable strict 'bind', 'call', and 'apply' methods on functions. */
+    // "strictPropertyInitialization": true,        /* Enable strict checking of property initialization in classes. */
+    // "noImplicitThis": true,                      /* Raise error on 'this' expressions with an implied 'any' type. */
+    // "alwaysStrict": true,                        /* Parse in strict mode and emit "use strict" for each source file. */
+
+    /* Additional Checks */
+    // "noUnusedLocals": true,                      /* Report errors on unused locals. */
+    // "noUnusedParameters": true,                  /* Report errors on unused parameters. */
+    // "noImplicitReturns": true,                   /* Report error when not all code paths in function return a value. */
+    // "noFallthroughCasesInSwitch": true,          /* Report errors for fallthrough cases in switch statement. */
+    // "noUncheckedIndexedAccess": true,            /* Include 'undefined' in index signature results */
+    // "noImplicitOverride": true,                  /* Ensure overriding members in derived classes are marked with an 'override' modifier. */
+    // "noPropertyAccessFromIndexSignature": true,  /* Require undeclared properties from index signatures to use element accesses. */
+
+    /* Module Resolution Options */
+    // "moduleResolution": "node",                  /* Specify module resolution strategy: 'node' (Node.js) or 'classic' (TypeScript pre-1.6). */
+    // "baseUrl": "./",                             /* Base directory to resolve non-absolute module names. */
+    // "paths": {},                                 /* A series of entries which re-map imports to lookup locations relative to the 'baseUrl'. */
+    // "rootDirs": [],                              /* List of root folders whose combined content represents the structure of the project at runtime. */
+    // "typeRoots": [],                             /* List of folders to include type definitions from. */
+    // "types": [],                                 /* Type declaration files to be included in compilation. */
+    // "allowSyntheticDefaultImports": true,        /* Allow default imports from modules with no default export. This does not affect code emit, just typechecking. */
+    "esModuleInterop": true,                        /* Enables emit interoperability between CommonJS and ES Modules via creation of namespace objects for all imports. Implies 'allowSyntheticDefaultImports'. */
+    // "preserveSymlinks": true,                    /* Do not resolve the real path of symlinks. */
+    // "allowUmdGlobalAccess": true,                /* Allow accessing UMD globals from modules. */
+
+    /* Source Map Options */
+    // "sourceRoot": "",                            /* Specify the location where debugger should locate TypeScript files instead of source locations. */
+    // "mapRoot": "",                               /* Specify the location where debugger should locate map files instead of generated locations. */
+    // "inlineSourceMap": true,                     /* Emit a single file with source maps instead of having a separate file. */
+    // "inlineSources": true,                       /* Emit the source alongside the sourcemaps within a single file; requires '--inlineSourceMap' or '--sourceMap' to be set. */
+
+    /* Experimental Options */
+    // "experimentalDecorators": true,              /* Enables experimental support for ES7 decorators. */
+    // "emitDecoratorMetadata": true,               /* Enables experimental support for emitting type metadata for decorators. */
+
+    /* Advanced Options */
+    "skipLibCheck": true,                           /* Skip type checking of declaration files. */
+    "forceConsistentCasingInFileNames": true        /* Disallow inconsistently-cased references to the same file. */
+    },
+    "exclude": ["dist/**/*", "src/__tests__/**/*"]
+}