From 566c3641f260db2a699956babdd92e3b87349cc9 Mon Sep 17 00:00:00 2001
From: Aditya Mathur <57684218+MathurAditya724@users.noreply.github.com>
Date: Sun, 4 Aug 2024 13:58:42 +0530
Subject: [PATCH] feat: switched from yargs-parser to commander

---
 package.json              |  4 +--
 src/hooks/dependencies.ts |  2 +-
 src/index.ts              | 56 ++++++++++++++++++++++++++++++++-------
 yarn.lock                 | 10 +++----
 4 files changed, 54 insertions(+), 18 deletions(-)

diff --git a/package.json b/package.json
index 0b37dbd..c970f5a 100644
--- a/package.json
+++ b/package.json
@@ -34,6 +34,7 @@
     "@types/node": "^18.11.18",
     "@types/yargs-parser": "^21.0.0",
     "chalk": "^5.3.0",
+    "commander": "^12.1.0",
     "esbuild": "^0.16.17",
     "eslint": "^8.55.0",
     "execa": "^8.0.1",
@@ -43,7 +44,6 @@
     "prettier": "^3.3.3",
     "tsx": "^4.7.1",
     "typescript": "^5.3.3",
-    "vitest": "^0.34.6",
-    "yargs-parser": "^21.1.1"
+    "vitest": "^0.34.6"
   }
 }
diff --git a/src/hooks/dependencies.ts b/src/hooks/dependencies.ts
index d2cd719..be411be 100644
--- a/src/hooks/dependencies.ts
+++ b/src/hooks/dependencies.ts
@@ -16,7 +16,7 @@ const knownPackageManagers: { [key: string]: string } = {
   yarn: 'yarn',
 }
 
-const knownPackageManagerNames = Object.keys(knownPackageManagers)
+export const knownPackageManagerNames = Object.keys(knownPackageManagers)
 const currentPackageManager = getCurrentPackageManager()
 
 // Deno and Netlify need no dependency installation step
diff --git a/src/index.ts b/src/index.ts
index 951643e..6a1aaeb 100644
--- a/src/index.ts
+++ b/src/index.ts
@@ -4,13 +4,16 @@ import confirm from '@inquirer/confirm'
 import input from '@inquirer/input'
 import select from '@inquirer/select'
 import chalk from 'chalk'
+import { Option, program, type Command } from 'commander'
 import { downloadTemplate } from 'giget'
 import { createSpinner } from 'nanospinner'
-import yargsParser from 'yargs-parser'
 import { version } from '../package.json'
 import { projectDependenciesHook } from './hook'
 import { afterCreateHook } from './hooks/after-create'
-import { registerInstallationHook } from './hooks/dependencies'
+import {
+  knownPackageManagerNames,
+  registerInstallationHook,
+} from './hooks/dependencies'
 
 const directoryName = 'templates'
 const config = {
@@ -46,16 +49,49 @@ function mkdirp(dir: string) {
   }
 }
 
-async function main() {
-  console.log(chalk.gray(`create-hono version ${version}`))
+program
+  .name('create-hono')
+  .version(version)
+  .arguments('[target]')
+  .addOption(
+    new Option('-y, --install', 'Install dependencies').argParser(Boolean),
+  )
+  .addOption(
+    new Option('-p, --pm <pm>', 'Package manager to use').choices(
+      knownPackageManagerNames,
+    ),
+  )
+  .addOption(
+    new Option('-t, --template <template>', 'Template to use').choices(
+      templates,
+    ),
+  )
+  .addOption(
+    new Option('--offline', 'Use offline mode')
+      .argParser(Boolean)
+      .default(false),
+  )
+  .action(main)
+
+type ArgOptions = {
+  install: boolean
+  offline: boolean
+  pm: string
+  template: string
+}
 
-  const args = yargsParser(process.argv.slice(2))
+async function main(
+  targetDir: string | undefined,
+  options: ArgOptions,
+  command: Command,
+) {
+  console.log(chalk.gray(`${command.name()} version ${command.version()}`))
 
-  const { install, pm, template: templateArg } = args
+  const { install, pm, offline, template: templateArg } = options
 
   let target = ''
-  if (args._[0]) {
-    target = args._[0].toString()
+  if (targetDir) {
+    target = targetDir
     console.log(
       `${chalk.bold(`${chalk.green('✔')} Using target directory`)} … ${target}`,
     )
@@ -115,7 +151,7 @@ async function main() {
     `gh:${config.user}/${config.repository}/${config.directory}/${templateName}#${config.ref}`,
     {
       dir: targetDirectoryPath,
-      offline: false,
+      offline,
       force: true,
     },
   ).then(() => spinner.success())
@@ -159,4 +195,4 @@ async function main() {
   console.log(chalk.gray('Get started with:'), chalk.bold(`cd ${target}`))
 }
 
-main()
+program.parse()
diff --git a/yarn.lock b/yarn.lock
index 0fe7481..2c46e72 100644
--- a/yarn.lock
+++ b/yarn.lock
@@ -1363,6 +1363,11 @@ color-name@~1.1.4:
   resolved "https://registry.yarnpkg.com/color-name/-/color-name-1.1.4.tgz#c2a09a87acbde69543de6f63fa3995c826c536a2"
   integrity sha512-dOy+3AuW3a2wNbZHIuMZpTcgjGuLU/uBL/ubcZF9OXbDo8ff4O8yVp5Bf0efS8uEoYo5q4Fx7dY9OgQGXgAsQA==
 
+commander@^12.1.0:
+  version "12.1.0"
+  resolved "https://registry.yarnpkg.com/commander/-/commander-12.1.0.tgz#01423b36f501259fdaac4d0e4d60c96c991585d3"
+  integrity sha512-Vw8qHK3bZM9y/P10u3Vib8o/DdkvA2OtPtZvD871QKjy74Wj1WSKFILMPRPSdUSx5RFK1arlJzEtA4PkFgnbuA==
+
 concat-map@0.0.1:
   version "0.0.1"
   resolved "https://registry.yarnpkg.com/concat-map/-/concat-map-0.0.1.tgz#d8a96bd77fd68df7793a73036a3ba0d5405d477b"
@@ -4902,11 +4907,6 @@ yargs-parser@^20.2.3:
   resolved "https://registry.yarnpkg.com/yargs-parser/-/yargs-parser-20.2.9.tgz#2eb7dc3b0289718fc295f362753845c41a0c94ee"
   integrity sha512-y11nGElTIV+CT3Zv9t7VKl+Q3hTQoT9a1Qzezhhl6Rp21gJ/IVTW7Z3y9EWXhuUBC2Shnf+DX0antecpAwSP8w==
 
-yargs-parser@^21.1.1:
-  version "21.1.1"
-  resolved "https://registry.yarnpkg.com/yargs-parser/-/yargs-parser-21.1.1.tgz#9096bceebf990d21bb31fa9516e0ede294a77d35"
-  integrity sha512-tVpsJW7DdjecAiFpbIB1e3qxIQsE6NoPc5/eTdrbbIC4h0LVsWhnoa3g+m2HclBIujHzsxZ4VJVA+GUuc2/LBw==
-
 yocto-queue@^0.1.0:
   version "0.1.0"
   resolved "https://registry.yarnpkg.com/yocto-queue/-/yocto-queue-0.1.0.tgz#0294eb3dee05028d31ee1a5fa2c556a6aaf10a1b"