Skip to content
This repository was archived by the owner on Dec 5, 2024. It is now read-only.

Commit 8c1f016

Browse files
committed
fix(cli): remove -f from start and add start-prod command
1 parent ad171f8 commit 8c1f016

File tree

7 files changed

+26
-13
lines changed

7 files changed

+26
-13
lines changed

packages/fusuma/src/cli/index.js

+15-2
Original file line numberDiff line numberDiff line change
@@ -28,15 +28,28 @@ async function cli() {
2828
.option('-b <directory>', 'A base path', prog.STRING, process.cwd())
2929
.option('-i <directory>', 'A directory to load', prog.STRING, 'slides')
3030
.option('-p <port>', 'Dev server port', prog.INT, '8080')
31-
.option('-f <fileMode>', 'A serve for a bundled directory', prog.BOOLEAN, false)
3231
.action((args, options) => {
3332
resolve({
3433
type: 'start',
3534
options: {
3635
basePath: options.b,
3736
inputDir: options.i,
3837
port: options.p,
39-
isFileServer: options.f,
38+
},
39+
});
40+
})
41+
42+
.command('start-prod', 'Serve bundle directory')
43+
.option('-b <directory>', 'A base path', prog.STRING, process.cwd())
44+
.option('-i <directory>', 'A directory to load', prog.STRING, 'dist')
45+
.option('-p <port>', 'Dev server port', prog.INT, '8080')
46+
.action((args, options) => {
47+
resolve({
48+
type: 'startProd',
49+
options: {
50+
basePath: options.b,
51+
inputDir: options.i,
52+
port: options.p,
4053
},
4154
});
4255
})

packages/fusuma/src/tasks/index.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -51,15 +51,13 @@ async function tasks({ type, options }) {
5151
}
5252
}
5353

54-
if (options.isFileServer) {
55-
return require('./serverFilesForProd')(config);
56-
}
57-
5854
switch (type) {
5955
case 'init':
6056
return require('./init')(config);
6157
case 'start':
6258
return require('./start')(config);
59+
case 'startProd':
60+
return require('./startProd')(config);
6361
case 'build':
6462
return require('./build')(config);
6563
case 'deploy':
@@ -68,6 +66,8 @@ async function tasks({ type, options }) {
6866
return require('./pdf')(config);
6967
case 'live':
7068
return require('./live')(config);
69+
default:
70+
error('cli', `the command not found, please see "--help"`);
7171
}
7272
}
7373

packages/fusuma/src/tasks/serverFilesForProd.js packages/fusuma/src/tasks/startProd.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,9 @@
33
const server = require('../server/fileServer');
44
const { info } = require('../cli/log');
55

6-
async function serverFilesForProd(config) {
6+
async function startProd(config) {
77
await server(config.internal.outputDirPath, config.build.publicPath, config.internal.port);
8-
info('start', `Serving on http://localhost:${config.internal.port}`);
8+
info('start-prod', `Serving on http://localhost:${config.internal.port}`);
99
}
1010

11-
module.exports = serverFilesForProd;
11+
module.exports = startProd;

samples/advanced/package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
"main": "index.js",
66
"scripts": {
77
"start": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli start",
8-
"start:prod": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli start -f true",
8+
"start:prod": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli start-prod",
99
"build": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli build",
1010
"pdf": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli pdf",
1111
"live": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli live -w '@nodejs'",

samples/debug/package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
"main": "index.js",
66
"scripts": {
77
"start": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli start",
8-
"start:prod": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli start -f true",
8+
"start:prod": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli start-prod",
99
"build": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli build",
1010
"pdf": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli pdf",
1111
"live": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli live -w '@nodejs'",

samples/intro/package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
"scripts": {
77
"init": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli init",
88
"start": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli start",
9-
"start:prod": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli start -f true",
9+
"start:prod": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli start-prod",
1010
"build": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli build",
1111
"pdf": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli pdf",
1212
"live": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli live -w '@nodejs'",

samples/plain/package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
"main": "index.js",
66
"scripts": {
77
"start": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli start",
8-
"start:prod": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli start -f true",
8+
"start:prod": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli start-prod",
99
"build": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli build",
1010
"pdf": "FUSUMA_DEBUG=true node ../../packages/fusuma/src/cli pdf"
1111
},

0 commit comments

Comments
 (0)