Skip to content

Commit 2b03532

Browse files
authoredDec 20, 2022
Make home directory lookup lazy (#32)
1 parent fcc8b3a commit 2b03532

File tree

4 files changed

+13
-5
lines changed

4 files changed

+13
-5
lines changed
 

Diff for: ‎home-directory-browser.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
const homeDirectory = '';
1+
const getHomeDirectory = () => '';
22

3-
export default homeDirectory;
3+
export default getHomeDirectory;

Diff for: ‎home-directory.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import os from 'node:os';
22

3-
const homeDirectory = os.homedir().replace(/\\/g, '/');
3+
const getHomeDirectory = () => os.homedir().replace(/\\/g, '/');
44

5-
export default homeDirectory;
5+
export default getHomeDirectory;

Diff for: ‎index.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
import escapeStringRegexp from 'escape-string-regexp';
2-
import homeDirectory from '#home-directory';
2+
import getHomeDirectory from '#home-directory';
33

44
const extractPathRegex = /\s+at.*[(\s](.*)\)?/;
55
const pathRegex = /^(?:(?:(?:node|node:[\w/]+|(?:(?:node:)?internal\/[\w/]*|.*node_modules\/(?:babel-polyfill|pirates)\/.*)?\w+)(?:\.js)?:\d+:\d+)|native)/;
66

77
export default function cleanStack(stack, {pretty = false, basePath} = {}) {
88
const basePathRegex = basePath && new RegExp(`(file://)?${escapeStringRegexp(basePath.replace(/\\/g, '/'))}/?`, 'g');
9+
const homeDirectory = pretty ? getHomeDirectory() : '';
910

1011
if (typeof stack !== 'string') {
1112
return undefined;

Diff for: ‎test.js

+7
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
import os from 'node:os';
22
import test from 'ava';
3+
import getHomeDirectoryNode from './home-directory.js';
4+
import getHomeDirectoryBrowser from './home-directory-browser.js';
35
import cleanStack from './index.js';
46

57
test('default', t => {
@@ -270,3 +272,8 @@ test('handle undefined', t => {
270272
const expected = undefined;
271273
t.is(cleanStack(stack, {pretty: true}), expected);
272274
});
275+
276+
test('exports for home-directory files match', t => {
277+
t.is(typeof getHomeDirectoryNode, 'function');
278+
t.is(typeof getHomeDirectoryBrowser, 'function');
279+
});

0 commit comments

Comments
 (0)
Please sign in to comment.