Merge pull request #27 from diva-e/feature/move-to-typescript

Migrate to typescript

Author: anoack93

.eslintignore

@@ -2,9 +2,8 @@ name: CLI test
 
 on:
   push:
-    branches: [ main ]
+    branches: [main]
   pull_request:
-    branches: [ main ]
 
 jobs:
   test:
@@ -15,10 +14,10 @@ jobs:
         os: [ubuntu-latest, macos-latest, windows-latest]
     steps:
       - name: Checkout Repository
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
 
       - name: Set up Node.js
-        uses: actions/setup-node@v3
+        uses: actions/setup-node@v4
         with:
           node-version: 22
 

.eslintrc.json

@@ -5,9 +5,8 @@ name: Node.js CI
 
 on:
   push:
-    branches: [ main ]
+    branches: [main]
   pull_request:
-    branches: [ main ]
 
 jobs:
   build:
@@ -16,13 +15,13 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [18.x, 20.x, 22.x]
+        node-version: [20.x, 22.x]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v4
     - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v2
+      uses: actions/setup-node@v4
       with:
         node-version: ${{ matrix.node-version }}
     - run: npm ci

.github/workflows/cli.yml

@@ -5,4 +5,6 @@ playground/src
 .idea/
 
 *.iml
-*.iws
+*.iws
+
+dist/

.github/workflows/node.js.yml

@@ -18,7 +18,7 @@
 
 - Make sure to run `npm test` and `npm run lint:fix` after changes have been finished
 - Keep the project lightweight and avoid adding more dependencies
-- Make sure to update `src/serve-mocks.spec.js` with tests for new features, it is basically a kind of e2e test which covers most of logic
+- Make sure to update existing unit tests or create new ones for new features
 
 ## Scripts
 

.gitignore

@@ -0,0 +1,8 @@
+#!/usr/bin/env node
+export interface AppConfig {
+    types: string[] | null;
+    templatePath: string;
+    componentPath: string;
+    nameStyle: string;
+}
+//# sourceMappingURL=cli.d.ts.map

AGENTS.md

@@ -7,6 +7,7 @@ import {
   processInitCommand,
   processPromptCommand,
   processUpgradeCommand,
+  type CommandEnv,
 } from './src/commands.js'
 import { getDirectories } from './src/utilities.js'
 import { readFileSync, existsSync } from 'fs'
@@ -16,24 +17,27 @@ import { fileURLToPath } from 'url'
 const __filename = fileURLToPath(import.meta.url)
 const __dirname = path.dirname(__filename)
 
+interface AppConfig {
+  types: string[] | null
+  templatePath: string
+  componentPath: string
+  nameStyle: string
+}
 
 const CONFIG_DIRECTORY = '.create-frontend-component'
 const CONFIG_FILE_NAME = 'config.json'
 const PRESET_DIR = 'presets'
 const PRESET_PATH = path.join(__dirname, PRESET_DIR)
 
-const configDefaults = {
+const configDefaults: AppConfig = {
   types: ['atoms', 'molecules', 'organisms'],
-  templatePath: CONFIG_DIRECTORY + '/templates',
+  templatePath: `${CONFIG_DIRECTORY}/templates`,
   componentPath: 'src/components',
   nameStyle: 'pascalCase'
 }
 
-/**
- * @return {object}
- */
-function loadConfig() {
-  const filePath = path.resolve(process.cwd(), '.create-frontend-component', 'config.json')
+function loadConfig(): AppConfig {
+  const filePath = path.resolve(process.cwd(), CONFIG_DIRECTORY, CONFIG_FILE_NAME)
 
   try {
     if (!existsSync(filePath)) {
@@ -46,22 +50,32 @@ function loadConfig() {
     const configFromFile = JSON.parse(fileContent)
 
     return {
-      ...configDefaults,
+      types: configDefaults.types ?? null,
+      templatePath: configDefaults.templatePath,
+      componentPath: configDefaults.componentPath,
+      nameStyle: configDefaults.nameStyle,
       ...configFromFile
     }
   } catch (error) {
-    console.error(`Error loading configuration file: ${error.message}`)
+    console.error(`Error loading configuration file: ${(error as Error).message}`)
     console.error('Try running "npx create-frontend-component init" to reset the configuration.')
     process.exit(1)
   }
 }
 
+const packageJsonPath = path.join(__dirname, '..', 'package.json')
+const { version } = JSON.parse(readFileSync(packageJsonPath, { encoding: 'utf-8' }))
+
+program
+  .name('create-frontend-component')
+  .description('Frontend Component Generator')
+  .version(version)
+
 program
-  .version('2.1.0')
-  .command('create-frontend-component [component-name]') // Define the command
-  .option( '-t, --type <type>', 'Component type, default: atoms')
-  .option( '-f, --flavour <flavour>', 'Component flavour')
-  .action( async function(componentNameArg, env) {
+  .command('create-frontend-component [component-name]')
+  .option('-t, --type <type>', 'Component type, default: atoms')
+  .option('-f, --flavour <flavour>', 'Component flavour')
+  .action(async function(componentNameArg: string | undefined, env: CommandEnv) {
     const componentName = componentNameArg || ''
 
     if (componentName.toLowerCase() === 'init') {
@@ -81,6 +95,12 @@ program
     const fullTemplatePath = path.join(process.cwd(), templatePath)
     const availableFlavours = getDirectories(fullTemplatePath)
 
+    if (availableFlavours.length === 0) {
+      console.error('Error: No flavours found in template directory.')
+      console.error(`Please ensure templates exist in ${fullTemplatePath}`)
+      process.exit(1)
+    }
+
     if (componentName.toLowerCase() === 'prompt' || !componentName.trim()) {
       await processPromptCommand(allowedComponentTypes, availableFlavours, fullTemplatePath, componentPath, nameStyle)
     } else if (componentName.toLowerCase() === 'upgrade') {
@@ -89,4 +109,5 @@ program
       processCreateComponentCommand(env, allowedComponentTypes, fullTemplatePath, componentPath, componentName, availableFlavours, nameStyle)
     }
   })
-  .parse(process.argv)
+
+program.parse(process.argv)