fix: make dependencies optional

package.json

@@ -6,28 +6,20 @@
   "bugs": "https://github.com/anycli/config/issues",
   "dependencies": {
     "debug": "^3.1.0",
-    "fs-extra": "^5.0.0",
-    "fs-extra-debug": "^1.0.4",
-    "globby": "^7.1.1",
-    "load-json-file": "^4.0.0",
-    "read-pkg": "^3.0.0"
+    "globby": "^7.1.1"
   },
   "devDependencies": {
     "@anycli/parser": "^3.2.2",
     "@anycli/tslint": "^0.2.5",
     "@types/chai": "^4.1.2",
-    "@types/fs-extra": "^5.0.0",
     "@types/globby": "^6.1.0",
-    "@types/load-json-file": "^2.0.7",
     "@types/mocha": "^2.2.48",
-    "@types/nock": "^9.1.2",
     "@types/node": "^9.4.0",
-    "@types/read-pkg": "^3.0.0",
     "chai": "^4.1.2",
     "concurrently": "^3.5.1",
     "eslint": "^4.17.0",
     "eslint-config-anycli": "^1.3.2",
-    "fancy-test": "^0.6.10",
+    "fancy-test": "^1.0.1",
     "mocha": "^5.0.0",
     "ts-node": "^4.1.0",
     "typescript": "^2.7.1"

src/config.ts

@@ -1,10 +1,10 @@
 import * as os from 'os'
 import * as path from 'path'
-import * as readPkg from 'read-pkg'
 
 import {Hooks} from './hooks'
 import {PJSON} from './pjson'
 import * as Plugin from './plugin'
+import {loadJSONSync} from './util'
 
 export type PlatformTypes = 'darwin' | 'linux' | 'win32' | 'aix' | 'freebsd' | 'openbsd' | 'sunos'
 export type ArchTypes = 'arm' | 'arm64' | 'mips' | 'mipsel' | 'ppc' | 'ppc64' | 's390' | 's390x' | 'x32' | 'x64' | 'x86'
@@ -137,7 +137,7 @@ export class Config extends Plugin.Plugin implements IConfig {
 
     try {
       const userPJSONPath = path.join(this.dataDir, 'package.json')
-      const pjson = this.userPJSON = readPkg.sync(userPJSONPath) as any
+      const pjson = this.userPJSON = loadJSONSync(userPJSONPath)
       if (!pjson.anycli) pjson.anycli = {schema: 1}
       this.loadPlugins(userPJSONPath, pjson.anycli.plugins)
     } catch (err) {

src/index.ts

@@ -1,4 +1,5 @@
-import 'fs-extra-debug'
+// tslint:disable no-implicit-dependencies
+try { require('fs-extra-debug') } catch {}
 
 export {IConfig, Config, Options, load} from './config'
 export {Command} from './command'

src/pjson.ts

@@ -1,14 +1,13 @@
-import {Package} from 'read-pkg'
-
 export interface PJSON {
+  [k: string]: any
   dependencies?: {[name: string]: string}
   anycli: {
     schema?: number
   }
 }
 
 export namespace PJSON {
-  export interface Plugin extends PJSON, Package {
+  export interface Plugin extends PJSON {
     name: string
     version: string
     anycli: PJSON['anycli'] & {

src/plugin.ts

@@ -1,7 +1,5 @@
-import * as fs from 'fs-extra'
-import * as loadJSON from 'load-json-file'
+import * as fs from 'fs'
 import * as path from 'path'
-import * as readPkg from 'read-pkg'
 import {inspect} from 'util'
 
 import {Command} from './command'
@@ -10,7 +8,7 @@ import {Manifest} from './manifest'
 import {PJSON} from './pjson'
 import {Topic} from './topic'
 import {tsPath} from './ts_node'
-import {flatMap, mapValues} from './util'
+import {flatMap, loadJSONSync, mapValues} from './util'
 
 export interface Options {
   root: string
@@ -108,7 +106,7 @@ export class Plugin implements IPlugin {
     Plugin.loadedPlugins[root] = this
     this.root = root
     debug('reading plugin %s', root)
-    this.pjson = readPkg.sync(path.join(root, 'package.json')) as any
+    this.pjson = loadJSONSync(path.join(root, 'package.json')) as any
     this.name = this.pjson.name
     this.version = this.pjson.version
     if (!this.pjson.anycli) {
@@ -222,7 +220,7 @@ export class Plugin implements IPlugin {
     const readManifest = () => {
       try {
         const p = path.join(this.root, '.anycli.manifest.json')
-        const manifest: Manifest = loadJSON.sync(p)
+        const manifest: Manifest = loadJSONSync(p)
         if (manifest.version !== this.version) {
           process.emitWarning(`Mismatched version in ${this.name} plugin manifest. Expected: ${this.version} Received: ${manifest.version}`)
         } else {
@@ -299,6 +297,6 @@ function findRoot(name: string | undefined, root: string) {
     } else {
       cur = path.join(next, 'package.json')
     }
-    if (fs.pathExistsSync(cur)) return path.dirname(cur)
+    if (fs.existsSync(cur)) return path.dirname(cur)
   }
 }

src/ts_node.ts

@@ -1,8 +1,9 @@
-import * as fs from 'fs-extra'
-import * as loadJSON from 'load-json-file'
+import * as fs from 'fs'
 import * as path from 'path'
 import * as TSNode from 'ts-node'
 
+import {loadJSONSync} from './util'
+
 const tsconfigs: {[root: string]: TSConfig} = {}
 const rootDirs: string[] = []
 const typeRoots = [`${__dirname}/../node_modules/@types`]
@@ -49,7 +50,7 @@ function loadTSConfig(root: string): TSConfig | undefined {
     // if (!await fs.pathExists(path.join(this.root, '.git'))) return
 
     const tsconfigPath = path.join(root, 'tsconfig.json')
-    const tsconfig = loadJSON.sync(tsconfigPath)
+    const tsconfig = loadJSONSync(tsconfigPath)
     if (!tsconfig || !tsconfig.compilerOptions) return
     return tsconfig
   } catch (err) {
@@ -84,7 +85,7 @@ export function tsPath(root: string, orig: string | undefined): string | undefin
     // For hooks, it might point to a module, not a file. Something like "./hooks/myhook"
     // That file doesn't exist, and the real file is "./hooks/myhook.ts"
     // In that case we attempt to resolve to the filename. If it fails it will revert back to the lib path
-    if (fs.pathExistsSync(out) || fs.pathExistsSync(out + '.ts')) return out
+    if (fs.existsSync(out) || fs.existsSync(out + '.ts')) return out
     else return orig
   } catch (err) {
     debug(err)

src/util.ts

@@ -1,3 +1,6 @@
+// tslint:disable no-implicit-dependencies
+import * as fs from 'fs'
+
 export function flatMap<T, U>(arr: T[], fn: (i: T) => U[]): U[] {
   return arr.reduce((arr, i) => arr.concat(fn(i)), [] as U[])
 }
@@ -9,3 +12,10 @@ export function mapValues<T extends object, TResult>(obj: {[P in keyof T]: T[P]}
     return o
   }, {} as any)
 }
+
+export function loadJSONSync(path: string): any {
+  let loadJSON
+  try { loadJSON = require('load-json-file') } catch {}
+  if (loadJSON) return loadJSON.sync(path)
+  return JSON.parse(fs.readFileSync(path, 'utf8'))
+}

test/test.ts

@@ -1,4 +1,4 @@
-import {expect, fancy as base, FancyTypes, NockScope} from 'fancy-test'
+import {expect, fancy as base, FancyTypes} from 'fancy-test'
 
 import * as Config from '../src'
 
@@ -13,5 +13,4 @@ export const fancy = base
 export {
   expect,
   FancyTypes,
-  NockScope,
 }