Merge branch 'robertknight-read-files-from-tsconfig'

Author: vvakame

README.md

@@ -24,6 +24,8 @@ $ tsfmt
 
 ## Usage
 
+### Format or verify specific TypeScript files
+
 ```bash
 $ cat sample.ts
 class Sample {hello(word="world"){return "Hello, "+word;}}
@@ -61,6 +63,17 @@ $ echo $?
 1
 ```
 
+### Reformat all files in a TypeScript project
+
+If no files are specified on the command line but
+a TypeScript project file (tsconfig.json) exists,
+the list of files will be read from the project file.
+
+```bash
+# reads list of files to format from tsconfig.json
+tsfmt -r
+```
+
 ## Note
 
 now `indentSize` parameter is ignored. it is TypeScript compiler matters.

lib/cli.ts

@@ -2,6 +2,7 @@ require("es6-promise").polyfill();
 
 import fs = require("fs");
 import commandpost = require("commandpost");
+import path = require("path");
 
 import lib = require("./index");
 
@@ -39,7 +40,12 @@ var root = commandpost
         var editorconfig = !!opts.editorconfig;
         var tsfmt = !!opts.tsfmt;
 
-        if (args.files.length === 0 && !opts.stdin) {
+        var files = args.files;
+        if (files.length === 0 && fs.existsSync("tsconfig.json")) {
+            files = readFilesFromTsconfig("tsconfig.json");
+        }
+
+        if (files.length === 0 && !opts.stdin) {
             process.stdout.write(root.helpText() + '\n');
             return;
         }
@@ -59,7 +65,7 @@ var root = commandpost
                 return;
             }
             lib
-                .processStream(args.files[0] || "temp.ts", process.stdin, {
+                .processStream(files[0] || "temp.ts", process.stdin, {
                     replace: replace,
                     verify: verify,
                     tslint: tslint,
@@ -75,7 +81,7 @@ var root = commandpost
                 .catch(errorHandler);
         } else {
             lib
-                .processFiles(args.files, {
+                .processFiles(files, {
                     replace: replace,
                     verify: verify,
                     tslint: tslint,
@@ -126,3 +132,16 @@ function errorHandler(err: any): Promise<any> {
         return null;
     });
 }
+
+function readFilesFromTsconfig(configPath: string) {
+    "use strict";
+
+    var tsconfigDir = path.dirname(configPath);
+    var tsconfig = JSON.parse(fs.readFileSync(configPath, "utf-8"));
+    if (tsconfig.files) {
+        var files: string[] = tsconfig.files;
+        return files.map(filePath => path.resolve(tsconfigDir, filePath));
+    } else {
+        throw new Error(`No "files" section present in tsconfig.json`);
+    }
+}

test/cli/main.ts

@@ -0,0 +1,5 @@
+class TestCLI {
+    method() {
+
+    }
+}

test/cli/tsconfig.json

@@ -0,0 +1,13 @@
+{
+    "compilerOptions": {
+        "target": "es5",
+        "module": "commonjs",
+        "declaration": false,
+        "noImplicitAny": true,
+        "removeComments": false,
+        "noLib": false
+    },
+    "files": [
+        "./main.ts"
+    ]
+}