fix: remove todos and add throw error on edge cases

ipetinate · May 8, 2024 · 0ff6a5b · 0ff6a5b
1 parent 9919f9c
commit 0ff6a5b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 11 deletions.
diff --git a/src/actions/create.js b/src/actions/create.js
@@ -35,16 +35,8 @@ export async function createAction(resourceName, options) {
    */
   let preset = options.preset
 
-  if (!options.type) {
-    // TODO: prompt type
-  }
-
-  if (!options.framework) {
-    // TODO: prompt framework
-  }
-
-  if (!options.path) {
-    // TODO: prompt framework
+  if (!options.type || !options.framework || !options.path) {
+    throw new Error('You did not pass any of the mandatory parameters for Options Mode')
   }
 
   if (!preset && options.framework) {

diff --git a/src/generators/components.js b/src/generators/components.js
@@ -135,7 +135,7 @@ export function makeFolderWrapperOrBypass(data) {
   const folderWrapperPath = join(data.resourcePath, data.name)
   const folderWrapperExists = checkDirectoriesTree(splitPathString(folderWrapperPath))
 
-  if (!folderWrapperExists) {
+  if (data.folderWrapper && !folderWrapperExists) {
     const created = createDir(folderWrapperPath)
 
     if (!created) console.error('Error: cannot create folder wrapper.')
@@ -238,5 +238,11 @@ export function generateComponentFile(data) {
   const path = data.pathWithFileName
   const success = createFileWithContent(data.pathWithFileName, data.fileContent)
 
+  if (data.folderWrapper) {
+    const filePath = join(data.resourcePath, data.name, 'index.' + data.extension)
+
+    createFileWithContent(filePath, `export * from './${data.name}'`)
+  }
+
   return { success, path }
 }