Merge pull request #84 from ianwremmel/issue/79-in-src-dir

fix(cli): only load typescript files if we can interpret typescript

package.json

@@ -25,7 +25,7 @@
     "precommit": "npm run --silent lint:staged",
     "presemantic-release": "npm run build",
     "semantic-release": "semantic-release",
-    "test": "jest --runInBand"
+    "test": "CLARK_ENV=development jest --runInBand"
   },
   "engines": {
     "node": ">=8"
@@ -101,20 +101,8 @@
     "transform": {
       "^.+\\.tsx?$": "ts-jest"
     },
-    "moduleFileExtensions": [
-      "ts",
-      "tsx",
-      "js",
-      "jsx",
-      "json",
-      "node"
-    ],
-    "setupFiles": [
-      "<rootDir>/test/initialize.ts"
-    ],
-    "testMatch": [
-      "<rootDir>/src/**/*.spec.[tj]s?(x)",
-      "<rootDir>/test/*/spec/**/*.[tj]s?(x)"
-    ]
+    "moduleFileExtensions": ["ts", "tsx", "js", "jsx", "json", "node"],
+    "setupFiles": ["<rootDir>/test/initialize.ts"],
+    "testMatch": ["<rootDir>/src/**/*.spec.[tj]s?(x)", "<rootDir>/test/*/spec/**/*.[tj]s?(x)"]
   }
 }

src/cli.ts

@@ -7,10 +7,11 @@ const pkg = JSON.parse(readFileSync(pkgUp(__dirname), 'utf-8'));
 // so, yargs's d.ts is...weird. This is an unfortunate set of castings to
 // convince typescript that all is well.
 const yargs = (y as any) as Argv;
+
 yargs
   // support ts extensions during development
   .commandDir('./commands', {
-    extensions: __dirname.includes('src') ? ['js', 'ts'] : ['js'],
+    extensions: process.env.CLARK_ENV === 'development' ? ['js', 'ts'] : ['js'],
   })
   .options({
     silent: {

test/integration/spec/exec.ts

@@ -55,7 +55,7 @@ describe('exec', () => {
 
   it('injects useful environment variables', async () => {
     const result = await run(
-      'clark exec --silent --package @example/scoped-package-the-first  "env | grep CLARK"',
+      'clark exec --silent --package @example/scoped-package-the-first  "env | grep CLARK | grep -v CLARK_ENV"',
     );
     // strip out this project's path so we can write consistent assertions
     const modified = result.replace(