fix(bun): lowercase -D paremeter if the agent is bun (antfu-collectiv…

…e#90)

Co-authored-by: Anthony Fu <[email protected]>

README.md

@@ -38,15 +38,13 @@ ni vite
 # bun add vite
 ```
 
-```
-
 ```bash
 ni @types/node -D
 
 # npm i @types/node -D
 # yarn add @types/node -D
 # pnpm add -D @types/node
-# bun add -D @types/node
+# bun add -d @types/node
 ```
 
 ```bash
@@ -55,6 +53,7 @@ ni --frozen
 # npm ci
 # yarn install --frozen-lockfile
 # pnpm install --frozen-lockfile
+# bun install --no-save
 ```
 
 ```bash
@@ -101,6 +100,7 @@ nr -
 ```bash
 nx vitest
 
+# (not available for bun)
 # npx vitest
 # yarn dlx vitest
 # pnpm dlx vitest
@@ -113,6 +113,7 @@ nx vitest
 ```bash
 nu
 
+# (not available for bun)
 # npm upgrade
 # yarn upgrade
 # pnpm update
@@ -121,7 +122,7 @@ nu
 ```bash
 nu -i
 
-# (not available for npm)
+# (not available for npm & bun)
 # yarn upgrade-interactive
 # pnpm update -i
 ```
@@ -158,6 +159,7 @@ nci
 # npm ci
 # yarn install --frozen-lockfile
 # pnpm install --frozen-lockfile
+# bun install --no-save
 ```
 
 if the corresponding node manager is not present, this command will install it globally along the way.
@@ -172,6 +174,7 @@ na
 # npm
 # yarn
 # pnpm
+# bun
 ```
 
 ```bash
@@ -180,6 +183,7 @@ na run foo
 # npm run foo
 # yarn run foo
 # pnpm run foo
+# bun run foo
 ```
 
 <br>

src/agents.ts

@@ -37,10 +37,9 @@ const bun = {
   'frozen': 'bun install --no-save',
   'global': 'bun add -g {0}',
   'add': 'bun add {0}',
-  // bun doesn't include an upgrade and execute command as of now
-  'upgrade': 'bun add {0}',
-  'upgrade-interactive': 'bun add {0}',
-  'execute': 'echo Bun does not support execute command, yet.',
+  'upgrade': null,
+  'upgrade-interactive': null,
+  'execute': null,
   'uninstall': 'bun remove {0}',
   'global_uninstall': 'bun remove -g {0}',
 }

src/parse.ts

@@ -29,6 +29,10 @@ export const parseNi = <Runner>((agent, args, ctx) => {
     process.exit(0)
   }
 
+  // bun use `-d` instead of `-D`, #90
+  if (agent === 'bun')
+    args = args.map(i => i === '-D' ? '-d' : i)
+
   if (args.includes('-g'))
     return getCommand(agent, 'global', exclude(args, '-g'))
 

test/ni/bun.spec.ts

@@ -14,6 +14,8 @@ test('empty', _('', 'bun install'))
 
 test('single add', _('axios', 'bun add axios'))
 
+test('add dev', _('vite -D', 'bun add vite -d'))
+
 test('multiple', _('eslint @types/node', 'bun add eslint @types/node'))
 
 test('global', _('eslint -g', 'bun add -g eslint'))

test/nu/bun.spec.ts

@@ -2,14 +2,14 @@ import { expect, test } from 'vitest'
 import { parseNu } from '../../src/commands'
 
 const agent = 'bun'
-const _ = (arg: string, expected: string) => () => {
+const _ = (arg: string, expected: string | null) => () => {
   expect(
     parseNu(agent, arg.split(' ').filter(Boolean)),
   ).toBe(
     expected,
   )
 }
 
-test('empty', _('', 'bun add'))
-
-test('interactive', _('-i', 'bun add'))
+test.fails('empty', _('', null))
+test.fails('interactive', _('-i', null))
+test.fails('interactive latest', _('-i --latest', null))