Skip to content

Commit e76d173

Browse files
committed
feat: coloured console log in nodejs env
1 parent e99fb19 commit e76d173

9 files changed

Lines changed: 35 additions & 19 deletions

File tree

package.json

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,9 @@
44
"description": "A dumb JavasScript bundler for Single Page Application, dumber than you and me",
55
"main": "dist/index.js",
66
"browser": {
7-
"./cache/default.js": "./cache/local-storage.js",
8-
"./package-locators/default.js": "./package-locators/jsDelivr.js"
7+
"./dist/cache/default.js": "./dist/cache/local-storage.js",
8+
"./dist/package-locators/default.js": "./dist/package-locators/jsDelivr.js",
9+
"./dist/log.js": "./dist/log-browser.js"
910
},
1011
"files": [
1112
"dist"

src/index.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,8 @@ import defaultPackageLocator from './package-locators/default';
55
import PackageReader from './package-reader';
66
import Package from './package';
77
import stubModule from './stub-module';
8-
import {info, error, warn, stripJsExtension, resolvePackagePath, contentOrFile} from './shared';
8+
import {info, error, warn} from './log';
9+
import {stripJsExtension, resolvePackagePath, contentOrFile} from './shared';
910
import * as cache from './cache/default';
1011
import path from 'path';
1112
import mergeTransformed from './transformers/merge';

src/log-browser.js

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
export function info(message) {
2+
console.info('[dumber] INFO: ' + message);
3+
}
4+
5+
export function warn(message) {
6+
console.warn('[dumber] WARN: ' + message);
7+
}
8+
9+
export function error(message) {
10+
console.error('[dumber] ERROR: ' + message);
11+
}

src/log.js

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
export function info(message) {
2+
console.info('[dumber] INFO: ' + message);
3+
}
4+
5+
export function warn(message) {
6+
console.warn(`\x1b[43m\x1b[30m[dumber] WARN: ${message}\x1b[0m`);
7+
}
8+
9+
export function error(message) {
10+
console.error(`\x1b[41m\x1b[37m[dumber] ERROR: ${message}\x1b[0m`);
11+
}

src/package-locators/default.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import path from 'path';
2-
import {info, warn, resolvePackagePath, fsReadFile} from '../shared';
2+
import {resolvePackagePath, fsReadFile} from '../shared';
3+
import {info, warn} from '../log';
34
import {ext} from 'dumber-module-loader/dist/id-utils';
45

56
// default locator using nodejs to resolve package

src/package-reader.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import {ext, parse, resolveModuleId, relativeModuleId} from 'dumber-module-loader/dist/id-utils';
22
import {stripJsExtension, isPackageName, stripSourceMappingUrl} from './shared';
33
import replace from './transformers/replace';
4+
import {error} from './log';
45
import path from 'path';
56

67
export default class PackageReader {
@@ -168,7 +169,8 @@ export default class PackageReader {
168169
try {
169170
metadata = JSON.parse(file.contents);
170171
} catch (err) {
171-
console.error(err);
172+
error('Failed to parse ' + packageJsonPath);
173+
error(err);
172174
return;
173175
}
174176
let metaMain;

src/shared.js

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -4,18 +4,6 @@ import crypto from 'crypto';
44
import {ensureParsed} from 'ast-matcher';
55
import './ensure-parser-set';
66

7-
export function info(message) {
8-
console.info('[dumber] INFO: ' + message);
9-
}
10-
11-
export function warn(message) {
12-
console.warn('[dumber] WARN: ' + message);
13-
}
14-
15-
export function error(message) {
16-
console.error('[dumber] ERROR: ' + message);
17-
}
18-
197
export function stripJsExtension(d) {
208
return d && d.endsWith('.js') ? d.slice(0, -3) : d;
219
}

src/stub-module.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
import {resolvePackagePath, warn} from './shared';
1+
import {resolvePackagePath} from './shared';
2+
import {warn} from './log';
23

34
// stub core Node.js modules based on https://github.com/webpack/node-libs-browser/blob/master/index.js
45
// no need stub for following modules, they got same name on npm package

src/transformers/defines.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import {globalIndentifiers} from '../parser';
2-
import {warn} from '../shared';
2+
import {warn} from '../log';
33
import astMatcher from 'ast-matcher';
44
const ensureParsed = astMatcher.ensureParsed;
55

0 commit comments

Comments
 (0)