feat: update pkg.files that if file exists (#37)

Avoid missing files when publish to npm

.autod.conf

@@ -13,6 +13,7 @@ module.exports = {
     'co-mocha',
     'intelli-espower-loader',
     'power-assert',
+    'ypkgfiles',
   ],
   devdep: [
     'autod',

README.md

@@ -125,6 +125,14 @@ You can set `COV_EXCLUDES` env to add dir ignore coverage.
 $ COV_EXCLUDES="app/plugins/c*,app/autocreate/**" egg-bin cov
 ```
 
+### pkgfiles
+
+Generate `pkg.files` automatically before npm publish, see [ypkgfiles] for detail
+
+```bash
+$ egg-bin pkgfiles
+```
+
 ### auto require `test/.setup.js`
 
 If `test/.setup.js` file exists, it will be auto require on `test` and `cov` command.
@@ -221,3 +229,4 @@ run nsp check at /foo/bar with []
 [iron-node]: https://github.com/s-a/iron-node
 [intelli-espower-loader]: https://github.com/power-assert-js/intelli-espower-loader
 [power-assert]: https://github.com/power-assert-js/power-assert
+[ypkgfiles]: https://github.com/popomore/ypkgfiles

README.zh-CN.md

@@ -115,6 +115,14 @@ TEST_TIMEOUT=2000 egg-bin test
 $ COV_EXCLUDES="app/plugins/c*,app/autocreate/**" egg-bin cov
 ```
 
+### pkgfiles
+
+在 npm publish 之前使用 pkgfiles 自动生成 `pkg.files`， 更多查看 [ypkgfiles]
+
+```bash
+$ egg-bin pkgfiles
+```
+
 ## 定制属于你团队的 egg-bin
 
 如果你的团队已经基于 egg 开发了属于自己的框架，那么很可能你会需要在 egg-bin 上做更多自定义功能。
@@ -195,3 +203,4 @@ run nsp check at /foo/bar with []
 [nsp]: https://npmjs.com/nsp
 [intelli-espower-loader]: https://github.com/power-assert-js/intelli-espower-loader
 [power-assert]: https://github.com/power-assert-js/power-assert
+[ypkgfiles]: https://github.com/popomore/ypkgfiles

lib/helper.js

@@ -1,6 +1,6 @@
 'use strict';
 
-const fs = require('fs');
+const existsSync = require('fs').existsSync;
 const path = require('path');
 const glob = require('glob');
 const detect = require('detect-port');
@@ -23,7 +23,7 @@ exports.getTestFiles = () => {
 
 exports.getTestSetupFile = () => {
   const setupFile = path.join(process.cwd(), 'test/.setup.js');
-  if (fs.existsSync(setupFile)) {
+  if (existsSync(setupFile)) {
     return setupFile;
   }
   return null;

lib/pkgfiles_command.js

@@ -0,0 +1,21 @@
+'use strict';
+
+const Command = require('./command');
+
+class PkgfilesCommand extends Command {
+  * run(cwd) {
+    const args = [
+      '--entry', 'app',
+      '--entry', 'config',
+      '--entry', '*.js',
+    ];
+    const pkgfiles = require.resolve('ypkgfiles/bin/pkgfiles.js');
+    yield this.helper.forkNode(pkgfiles, args, { cwd });
+  }
+
+  help() {
+    return 'Generate pkg.files automatically';
+  }
+}
+
+module.exports = PkgfilesCommand;

lib/program.js

@@ -15,6 +15,7 @@ class Program extends BaseProgram {
     } else {
       this.addCommand('cov', path.join(__dirname, 'cov_command.js'));
     }
+    this.addCommand('pkgfiles', path.join(__dirname, 'pkgfiles_command.js'));
   }
 }
 

lib/test_command.js

@@ -3,7 +3,7 @@
 const Command = require('./command');
 
 class TestCommand extends Command {
-  * run(_, args) {
+  * run(cwd, args) {
     yield this.helper.checkDeps();
 
     const newArgs = this.helper.formatTestArgs(args);

package.json

@@ -20,6 +20,7 @@
     "mocha": "^3.2.0",
     "power-assert": "^1.4.2",
     "rimraf": "^2.6.1",
+    "ypkgfiles": "^1.1.0",
     "yargs": "^6.6.0"
   },
   "devDependencies": {

test/egg-pkgfiles.test.js

@@ -0,0 +1,31 @@
+'use strict';
+
+const path = require('path');
+const assert = require('assert');
+const coffee = require('coffee');
+const fs = require('mz/fs');
+
+describe('egg-bin pkgfiles', () => {
+  const eggBin = require.resolve('../bin/egg-bin.js');
+
+  let cwd;
+  afterEach(() => fs.writeFile(path.join(cwd, 'package.json'), '{}'));
+
+  it('should update pkg.files', function* () {
+    cwd = path.join(__dirname, 'fixtures/pkgfiles');
+    yield fs.writeFile(path.join(cwd, 'package.json'), '{}');
+
+    yield coffee.fork(eggBin, [ 'pkgfiles' ], { cwd })
+    // .debug()
+    .expect('code', 0)
+    .end();
+
+    const body = yield fs.readFile(path.join(cwd, 'package.json'), 'utf8');
+    assert.deepEqual(JSON.parse(body).files, [
+      'app',
+      'config',
+      'app.js',
+    ]);
+  });
+
+});

test/fixtures/enzyme-example-mocha/package.json

@@ -27,5 +27,6 @@
   "dependencies": {
     "react": "^0.14.7",
     "react-dom": "^0.14.7"
-  }
-}
+  },
+  "files": []
+}

test/fixtures/pkgfiles/app.js

@@ -0,0 +1,3 @@
+'use strict';
+
+module.exports = () => {};

test/fixtures/pkgfiles/app/extend/context.js

@@ -0,0 +1,3 @@
+'use strict';
+
+module.exports = {};

test/fixtures/pkgfiles/config/config.default.js

@@ -0,0 +1,3 @@
+'use strict';
+
+module.exports = {};

test/fixtures/pkgfiles/package.json

@@ -0,0 +1 @@
+{}

test/fixtures/test-files/package.json

@@ -1,3 +1,6 @@
 {
-  "name": "test-files"
-}
+  "name": "test-files",
+  "files": [
+    "lib"
+  ]
+}