Skip to content

Commit f607e96

Browse files
vibeguiGuilherme Rodrigues
andauthored
fix(deno): Fix deno argv handling (#70)
Co-authored-by: Guilherme Rodrigues <guilherme@vtex.com>
1 parent a3cb4e5 commit f607e96

File tree

3 files changed

+15
-3
lines changed

3 files changed

+15
-3
lines changed

dist/index.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -281,7 +281,10 @@ class Option {
281281
}
282282

283283
const deno = typeof window !== 'undefined' && window.Deno;
284-
const processArgs = deno ? ['deno'].concat(Deno.args) : process.argv;
284+
const denoScriptPath = deno && typeof window !== 'undefined' && window.location.pathname;
285+
// Adds deno executable and script path to processArgs as "compatibility" layer for node
286+
// See https://github.com/cacjs/cac/issues/69
287+
const processArgs = deno ? ['deno', denoScriptPath].concat(Deno.args) : process.argv;
285288
const platformInfo = deno
286289
? `${Deno.build.os}-${Deno.build.arch} deno-${Deno.version.deno}`
287290
: `${process.platform}-${process.arch} node-${process.version}`;

mod.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -704,7 +704,10 @@ class Option {
704704
}
705705

706706
const deno = typeof window !== 'undefined' && window.Deno;
707-
const processArgs = deno ? ['deno'].concat(Deno.args) : process.argv;
707+
const denoScriptPath = deno && typeof window !== 'undefined' && window.location.pathname;
708+
// Adds deno executable and script path to processArgs as "compatibility" layer for node
709+
// See https://github.com/cacjs/cac/issues/69
710+
const processArgs = deno ? ['deno', denoScriptPath].concat(Deno.args) : process.argv;
708711
const platformInfo = deno
709712
? `${Deno.build.os}-${Deno.build.arch} deno-${Deno.version.deno}`
710713
: `${process.platform}-${process.arch} node-${process.version}`;

src/node.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,14 @@ declare let window: any
22
declare let Deno: any
33

44
const deno = typeof window !== 'undefined' && window.Deno
5+
const denoScriptPath =
6+
deno && typeof window !== 'undefined' && window.location.pathname
57

6-
export const processArgs = deno ? ['deno'].concat(Deno.args) : process.argv
8+
// Adds deno executable and script path to processArgs as "compatibility" layer for node
9+
// See https://github.com/cacjs/cac/issues/69
10+
export const processArgs = deno
11+
? ['deno', denoScriptPath].concat(Deno.args)
12+
: process.argv
713

814
export const platformInfo = deno
915
? `${Deno.build.os}-${Deno.build.arch} deno-${Deno.version.deno}`

0 commit comments

Comments
 (0)