kalo-cli 0.1.2 → 0.1.3

package/bin/kalo.ts

@@ -5,36 +5,13 @@ import minimist from "minimist";
 import { Plop, run } from "plop";
 import { fileURLToPath } from "node:url";
 import { loadConfig } from "../generators/utils/config";
+import { findProjectRoot } from "../generators/utils/filesystem";
 
 const args = process.argv.slice(2);
 const argv = minimist(args);
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
-/**
- * Recherche le répertoire racine du projet en remontant à partir d'un dossier donné.
- */
-function findProjectRoot(startDir: string): string {
-  let currentDir = startDir;
-  while (currentDir !== path.parse(currentDir).root) {
-    const packageJsonPath = path.join(currentDir, 'package.json');
-    if (fs.existsSync(packageJsonPath)) {
-      try {
-        const pkg = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-        if (pkg.name !== 'kalo-cli' || currentDir.includes('node_modules')) {
-          if (!currentDir.includes('node_modules')) {
-            return currentDir;
-          }
-        } else {
-          return currentDir;
-        }
-      } catch (e) {}
-    }
-    currentDir = path.dirname(currentDir);
-  }
-  return startDir;
-}
-
 const projectRoot = findProjectRoot(process.cwd());
 const config = await loadConfig();
 

package/generators/ai-enhancer/index.ts

@@ -4,6 +4,7 @@ import { generateWithModel } from '../utils/ai/common';
 import { injectMarkers, applyGeneratedCode } from '../utils/code-manipulation';
 import { Glob } from "bun";
 import { searchChoices } from '../utils/search';
+import { findProjectRoot } from '../utils/filesystem';
 import { AI_PROVIDER_TYPES, AI_TEMP_TYPES, BACK_VALUE, EXIT_VALUE } from '../constants';
 import keywordConfig from './keywords.json';
 
@@ -108,8 +109,9 @@ export const aiEnhancerGenerator: PlopGeneratorConfig = {
             source: async (answers, input) => {
                 const glob = new Glob("**/*.py");
                 const files: string[] = [];
-                // Scans the current directory
-                for await (const file of glob.scan(".")) {
+                const rootDir = findProjectRoot();
+                // Scans the project root
+                for await (const file of glob.scan(rootDir)) {
                     // Filter for likely python model files or snippets
                     if (file.endsWith('.py') && (file.includes("models") || file.includes("snippet"))) {
                          files.push(file);

package/generators/django-app/index.ts

@@ -1,4 +1,6 @@
 import { PlopGeneratorConfig } from 'plop';
+import * as path from 'path';
+import { findProjectRoot } from '../utils/filesystem';
 
 /**
  * Generator configuration for creating a new Django Application.
@@ -22,7 +24,9 @@ export const djangoAppGenerator: PlopGeneratorConfig = {
     },
   ],
   actions: (data: any) => {
-    const appDir = data.config?.appDir || 'app';
+    const rootDir = findProjectRoot();
+    const appDirName = data.config?.appDir || 'app';
+    const appDir = path.join(rootDir, appDirName);
     return [
       {
         type: 'add',

package/generators/django-channel/index.ts

@@ -1,5 +1,7 @@
 import { PlopGeneratorConfig } from 'plop';
+import * as path from 'path';
 import { ensureSuffix, createAppendActions } from '../utils/plop-actions';
+import { findProjectRoot } from '../utils/filesystem';
 import { CHANNEL_TYPES } from '../constants';
 import { searchChoices } from '../utils/search';
 
@@ -41,7 +43,9 @@ export const djangoChannelGenerator: PlopGeneratorConfig = {
   ],
   actions: (data: any) => {
     const actions = [];
-    const appDir = data.config?.appDir || 'app';
+    const rootDir = findProjectRoot();
+    const appDirName = data.config?.appDir || 'app';
+    const appDir = path.join(rootDir, appDirName);
     const appPath = `${appDir}/${data.app}`;
     const isAsync = data.type === 'async';
     data.isAsync = isAsync;

package/generators/django-form/index.ts

@@ -1,5 +1,7 @@
 import { PlopGeneratorConfig } from 'plop';
+import * as path from 'path';
 import { ensureSuffix, createAppendActions } from '../utils/plop-actions';
+import { findProjectRoot } from '../utils/filesystem';
 import { FORM_TYPES } from '../constants';
 import { searchChoices } from '../utils/search';
 
@@ -40,7 +42,9 @@ export const djangoFormGenerator: PlopGeneratorConfig = {
   ],
   actions: (data: any) => {
     const actions = [];
-    const appDir = data.config?.appDir || 'app';
+    const rootDir = findProjectRoot();
+    const appDirName = data.config?.appDir || 'app';
+    const appDir = path.join(rootDir, appDirName);
     const appPath = `${appDir}/${data.app}`;
     const isModelForm = data.type === 'model_form';
 

package/generators/django-view/index.ts

@@ -36,7 +36,9 @@ export const djangoViewGenerator: PlopGeneratorConfig = {
   ],
   actions: (data: any) => {
     const actions = [];
-    const appDir = data.config?.appDir || 'app';
+    const rootDir = findProjectRoot();
+    const appDirName = data.config?.appDir || 'app';
+    const appDir = path.join(rootDir, appDirName);
     const appPath = `${appDir}/${data.app}`;
     const isCBV = data.type !== 'fbv';
 

package/generators/utils/filesystem.ts

@@ -3,7 +3,8 @@ import * as path from 'path';
 
 /**
  * Recherche le répertoire racine du projet en remontant à partir d'un dossier donné.
- * Cherche la présence d'un fichier package.json qui n'est pas celui du package kalo-cli lui-même.
+ * Cherche la présence d'un fichier package.json valide (avec name et version).
+ * Vérifie également les permissions d'écriture.
  *
  * @param startDir - Le dossier de départ pour la recherche.
  * @returns Le chemin absolu vers la racine du projet.
@@ -16,18 +17,50 @@ export const findProjectRoot = (startDir: string = process.cwd()): string => {
     if (fs.existsSync(packageJsonPath)) {
       try {
         const pkg = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
+        
+        // Validation basique du package.json
+        if (!pkg.name || !pkg.version) {
+            // Package.json invalide, on continue
+            currentDir = path.dirname(currentDir);
+            continue;
+        }
+
         // Si on trouve un package.json qui n'est pas celui de kalo-cli, c'est probablement la racine du projet utilisateur
         // Ou si on est dans un projet qui n'a pas encore de kalo-cli installé mais qui a un package.json
         if (pkg.name !== 'kalo-cli' || currentDir.includes('node_modules')) {
             // Si on est dans node_modules, on doit continuer à monter car on a trouvé le package.json de kalo-cli
+            // SAUF si le dossier courant n'est PAS dans node_modules (ex: projet utilisateur nommé 'node_modules' - improbable mais possible)
+            // La logique ici : Si le chemin CONTIENT node_modules, c'est suspect, on veut sortir de node_modules.
+            // MAIS si on a trouvé un package.json valide à un niveau, il faut vérifier si c'est celui de notre dépendance ou du projet racine.
+            
+            // Cas simple : On est dans .../mon-projet/node_modules/kalo-cli
+            // currentDir = .../mon-projet/node_modules/kalo-cli -> pkg.name = kalo-cli.
+            // La condition (pkg.name !== 'kalo-cli') est fausse. -> else -> return currentDir (Mode dev local)
+            // ATTENTION: La logique originale retournait currentDir si pkg.name === 'kalo-cli' (mode dev).
+            
+            // Nouvelle logique plus stricte :
             if (currentDir.includes('node_modules')) {
-                // On continue de monter
+                // On est à l'intérieur d'un node_modules, ce n'est probablement pas la racine du projet utilisateur
+                // On continue de remonter
             } else {
-                return currentDir;
+                 // Vérification des permissions d'écriture
+                 try {
+                     fs.accessSync(currentDir, fs.constants.W_OK);
+                     return currentDir;
+                 } catch (e) {
+                     console.warn(`[Kalo] Attention: Le répertoire détecté ${currentDir} n'est pas accessible en écriture.`);
+                     // On continue quand même ? Ou on retourne avec un warning ? 
+                     // Pour l'instant on retourne, l'erreur surviendra à l'écriture.
+                     return currentDir;
+                 }
             }
         } else {
-            // C'est le package.json de kalo-cli en mode développement local
-            return currentDir;
+            // C'est le package.json de kalo-cli (pkg.name === 'kalo-cli')
+            // Si on n'est pas dans node_modules, c'est qu'on développe kalo-cli lui-même
+             if (!currentDir.includes('node_modules')) {
+                return currentDir;
+             }
+             // Sinon, c'est qu'on est dans node_modules/kalo-cli, donc on continue
         }
       } catch (e) {
         // En cas d'erreur de lecture, on continue de monter
@@ -36,7 +69,12 @@ export const findProjectRoot = (startDir: string = process.cwd()): string => {
     currentDir = path.dirname(currentDir);
   }
 
-  return startDir; // Fallback sur le dossier de départ
+  // Fallback: Si on n'a rien trouvé, on retourne le dossier de départ
+  // Mais on prévient l'utilisateur
+  if (startDir.includes('node_modules')) {
+      console.warn("[Kalo] Attention: Impossible de trouver la racine du projet hors de node_modules.");
+  }
+  return startDir; 
 };
 
 /**

package/generators/wagtail-admin/index.ts

@@ -1,5 +1,7 @@
 import { PlopGeneratorConfig } from 'plop';
+import * as path from 'path';
 import { ensureSuffix } from '../utils/plop-actions';
+import { findProjectRoot } from '../utils/filesystem';
 import { ADMIN_EXT_TYPES } from '../constants';
 import { searchChoices } from '../utils/search';
 
@@ -35,7 +37,9 @@ export const wagtailAdminGenerator: PlopGeneratorConfig = {
   ],
   actions: (data: any) => {
     const actions = [];
-    const appDir = data.config?.appDir || 'app';
+    const rootDir = findProjectRoot();
+    const appDirName = data.config?.appDir || 'app';
+    const appDir = path.join(rootDir, appDirName);
     const appPath = `${appDir}/${data.app}`;
     const isView = data.type === 'view';
 

package/generators/wagtail-block/index.ts

@@ -2,6 +2,7 @@ import { PlopGeneratorConfig } from 'plop';
 import * as fs from 'fs';
 import * as path from 'path';
 import { ensureSuffix, createAppendActions } from '../utils/plop-actions';
+import { findProjectRoot } from '../utils/filesystem';
 
 /**
  * Generator configuration for Wagtail StreamField Blocks.
@@ -29,7 +30,9 @@ export const wagtailBlockGenerator: PlopGeneratorConfig = {
   ],
   actions: (data: any) => {
     const actions = [];
-    const appDir = data.config?.appDir || 'app';
+    const rootDir = findProjectRoot();
+    const appDirName = data.config?.appDir || 'app';
+    const appDir = path.join(rootDir, appDirName);
 
     // Ensure suffix
     data.name = ensureSuffix(data.name, 'Block');

package/generators/wagtail-page/actions/model.ts

@@ -1,6 +1,8 @@
 
 import { ActionType } from 'plop';
+import * as path from 'path';
 import { createAppendActions } from '../../utils/plop-actions';
+import { findProjectRoot } from '../../utils/filesystem';
 
 /**
  * Generates Plop actions for creating a standard Django model.
@@ -9,7 +11,9 @@ import { createAppendActions } from '../../utils/plop-actions';
  * @returns {ActionType[]} An array of Plop actions to create/update model files.
  */
 export const getModelActions = (data: any): ActionType[] => {
-  const appDir = data.config?.appDir || 'app';
+  const rootDir = findProjectRoot();
+  const appDirName = data.config?.appDir || 'app';
+  const appDir = path.join(rootDir, appDirName);
   const appPath = `${appDir}/${data.app}`;
 
   return createAppendActions({

package/generators/wagtail-page/actions/orderable.ts

@@ -1,6 +1,8 @@
 
 import { ActionType } from 'plop';
+import * as path from 'path';
 import { ensureSuffix, createAppendActions } from '../../utils/plop-actions';
+import { findProjectRoot } from '../../utils/filesystem';
 
 /**
  * Generates Plop actions for creating a Wagtail Orderable model.
@@ -10,7 +12,9 @@ import { ensureSuffix, createAppendActions } from '../../utils/plop-actions';
  * @returns {ActionType[]} An array of Plop actions to create/update orderable files.
  */
 export const getOrderableActions = (data: any): ActionType[] => {
-  const appDir = data.config?.appDir || 'app';
+  const rootDir = findProjectRoot();
+  const appDirName = data.config?.appDir || 'app';
+  const appDir = path.join(rootDir, appDirName);
   const appPath = `${appDir}/${data.app}`;
 
   data.name = ensureSuffix(data.name, 'Orderable');

package/generators/wagtail-page/actions/page.ts

@@ -1,6 +1,8 @@
 
 import { ActionType } from 'plop';
+import * as path from 'path';
 import { createAppendActions } from '../../utils/plop-actions';
+import { findProjectRoot } from '../../utils/filesystem';
 
 /**
  * Generates Plop actions for creating a pair of Wagtail Pages (Index + Detail).
@@ -11,7 +13,9 @@ import { createAppendActions } from '../../utils/plop-actions';
  */
 export const getPageActions = (data: any): ActionType[] => {
   const actions: ActionType[] = [];
-  const appDir = data.config?.appDir || 'app';
+  const rootDir = findProjectRoot();
+  const appDirName = data.config?.appDir || 'app';
+  const appDir = path.join(rootDir, appDirName);
   const appPath = `${appDir}/${data.app}`;
   
   // Clean base name: Remove 'Page' if present to avoid BlogPagePage

package/generators/wagtail-page/actions/snippet.ts

@@ -8,7 +8,9 @@ import { createAppendActions, ensureSuffix } from '../../utils/plop-actions';
  * @returns {ActionType[]} An array of Plop actions to create snippet files and tests.
  */
 export const getSnippetActions = (data: any): ActionType[] => {
-  const appDir = data.config?.appDir || 'app';
+  const rootDir = findProjectRoot();
+  const appDirName = data.config?.appDir || 'app';
+  const appDir = path.join(rootDir, appDirName);
   const appPath = `${appDir}/${data.app}`;
   
   data.name = ensureSuffix(data.name, 'Snippet');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kalo-cli",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Générateur de code structuré et uniforme pour Django et Wagtail",
   "bin": {
     "kalo": "./bin/kalo.ts"