Remote Templates (#329)

* install `gittar`

* remove flags: css,sass,less,dest,type

* purge *css flag-related stuffs

* extract yarn-vs-npm logic before printing

* replace ansi-codes with `chalk.green`

* extract `pkgFile` path

* use `argv.name` within pkg & manifest (Closes #326)

- declare `log-symbols` as a dependency
  - it is already a direct dependency of `ora`

* extract an `error` helper

- with nice symbol & formatting

* uninstall `recursive-copy`

* extract `isDir` helper

- uses *sync versions, but oh well

* replace `copy` with `gittar.extract` - with filter

* move `target` + `exists` + `force` logic first;

- will need to verify a destination before extracting/downloading
- will prompt (TODO) if incomplete
- will abort if unsafe

* use `argv.dest` for final instructions

* fix typo

* export `setup` functions directly

* remove debug-type stdout from `initGit`

* move common helpers to `util` file;

- use more partial import statements

* move `commandExists` —> `util.hasCommand`;

- uses which.sync 🙊
- update function calls
- remove `async` from `pkgScripts`

* check if `yarn` bin exists on flag parsing;

- instead of checking 5x per `setup` function
- return `async` function directly from `initialize`

* consolidate `setup.pkgScripts` function

* do not write a `gitignore` file

- template concern

* minor cleanup, warn if no `git` binary found

* oops, fix regex tester

* don’t assume exit within `util.error`

* use `warn` within watch command

* use `util.isDir` within serve command

* remove `init` command publicly

* Merge `master` into `templates`

* alias “official” preact-cli templates by name

- only `default` & `full` for now

* simplify `install` — install template deps

* move `trimLeft` —> `util.trim`

* remove `initialize` from setup;

- assumed template ships own `package.json` already

* simplify package-related `create`

- change `pkg.name` & `mani.name` if set
- set default `pkg.scripts` if none provided
   -  will dynamically install `if-env` for `start` script

* only extract `/template/` filepaths

- strip:2 to remove “template” from dest name

* add extra TODO note

* define `done` handler before export

* update `test/subjects` names

- fixes test? /shrug

* [MAJOR] tests refactor

- supports new `create` command
- refactor utility methods
- change snapshot handling
  — comparison & taking
- rename `WITH_INSTALL` —> `SKIP_INSTALL`
- remove `setup` step; not needed

* move `serve.snapshot` —>`images/serve`

* add `deprecated` notice on `build.snapshot` file

* remove unused `exists` helper

* Fix tests/image/create.js test error

* Revert "Fix tests/image/create.js test error"

This reverts commit ca3b235.

* [Travis] only build `master` branch

- PRs are still built

* remove `commands/init` file

- somehow `recursive-copy` was picked up?

* add console statement for debug

- WTF HAPPENING

* remove `dependency-install-loader`

- no need anymore!
- templates responsible own dependecies
- and style-* flags removed

* update scripts path

- OCD strikes again

* exit early if run `build` w/o installing deps;

- tell user to run `npm install` first

* always install deps for build subjects

* remove `SKIP_INSTALL` global for Travis (temp)

* assume any template without `/` is official lookup

- print `info` message about the assumption

* update tests, remove `full` alias

Author: lukeed

.travis.yml

@@ -10,6 +10,10 @@ node_js:
   - 6
   - 8
 
+branches:
+  only:
+    - master
+
 cache:
   npm: true
   directories:
@@ -18,7 +22,8 @@ cache:
 
 before_script:
 - export LIGHTHOUSE_CHROMIUM_PATH=`which google-chrome-stable`
-- if [ "$TRAVIS_BRANCH" = "master" ]; then export WITH_INSTALL=true; export WITH_LOG=true; fi
+# - if [ "$TRAVIS_BRANCH" = "master" ]; then export SKIP_INSTALL=true; export WITH_LOG=true; fi
+- if [ "$TRAVIS_BRANCH" = "master" ]; then export WITH_LOG=true; fi
 
 script:
 - npm run lint

README.md

@@ -95,12 +95,6 @@ $ preact init
 $ preact create
 
   --name        Directory and package name for the new app.
-  --dest        Directory to create the app within.                 [default: <name>]
-  --type        A project template to start from.
-                  [Options: "full", "root", "simple", "empty"]      [default: "full"]
-  --less        Pre-install LESS support.                [boolean]  [default: false]
-  --sass        Pre-install SASS/SCSS support.           [boolean]  [default: false]
-  --stylus      Pre-install STYLUS support.              [boolean]  [default: false]
   --git         Initialize version control using git.    [boolean]  [default: false]
   --no-install  Disables installing of dependencies.     [boolean]  [default: false]
   --yarn        Installs dependencies with yarn.         [boolean]  [default: false]

package.json

@@ -13,9 +13,9 @@
     "prepublish": "npm run -s build",
     "dev": "babel-node src",
     "lint": "eslint src tests",
-    "pretest": "npm run -s build && rimraf ./tests/output",
+    "pretest": "npm run -s build && rimraf tests/output",
     "test": "jest",
-    "posttest": "rimraf ./tests/output",
+    "posttest": "rimraf tests/output",
     "postinstall": "node -p 'require(\"./check.js\")()'",
     "test:build": "babel-node src build --cwd examples/root",
     "test:serve": "npm run -s test:build && babel-node src serve --port 3000 --cwd examples/root",
@@ -90,7 +90,6 @@
     "eslint-plugin-react": "^7.0.1",
     "html-looks-like": "^1.0.2",
     "jest": "^20.0.4",
-    "lodash": "^4.17.4",
     "ncp": "^2.0.0",
     "uuid": "^3.0.1"
   },
@@ -121,6 +120,7 @@
     "file-loader": "^0.11.1",
     "fs.promised": "^3.0.0",
     "get-port": "^3.1.0",
+    "gittar": "^0.1.0",
     "glob": "^7.1.2",
     "html-webpack-exclude-assets-plugin": "0.0.5",
     "html-webpack-plugin": "^2.28.0",
@@ -131,6 +131,7 @@
     "less": "^2.7.2",
     "less-loader": "^4.0.3",
     "loader-utils": "^1.1.0",
+    "log-symbols": "^2.0.0",
     "minimatch": "^3.0.3",
     "mkdirp": "^0.5.1",
     "offline-plugin": "^4.6.2",
@@ -144,7 +145,6 @@
     "progress-bar-webpack-plugin": "^1.9.3",
     "promise-polyfill": "^6.0.2",
     "raw-loader": "^0.5.1",
-    "recursive-copy": "^2.0.6",
     "require-relative": "^0.8.7",
     "rimraf": "^2.6.1",
     "script-ext-html-webpack-plugin": "^1.8.0",

src/commands/build.js

@@ -1,5 +1,6 @@
-import { resolve } from 'path';
 import rimraf from 'rimraf';
+import { resolve } from 'path';
+import { isDir, error } from '../util';
 import asyncCommand from '../lib/async-command';
 import runWebpack, { showStats, writeJsonStats } from '../lib/webpack/run-webpack';
 
@@ -48,8 +49,15 @@ export default asyncCommand({
 	},
 
 	async handler(argv) {
+		let cwd = argv.cwd ? resolve(argv.cwd) : process.cwd();
+		let modules = resolve(cwd, 'node_modules');
+
+		if (!isDir(modules)) {
+			return error('No `node_modules` found! Please run `npm install` before continuing.', 1);
+		}
+
 		if (argv.clean) {
-			let dest = resolve(argv.cwd || process.cwd(), argv.dest || 'build');
+			let dest = resolve(cwd, argv.dest || 'build');
 			await Promise.promisify(rimraf)(dest);
 		}