@git.zone/cli 1.16.8 → 1.17.1

package/ts/gitzone.cli.ts

@@ -131,6 +131,14 @@ export let run = async () => {
     modHelpers.run(argvArg);
   });
 
+  /**
+   * manage development services (MongoDB, S3/MinIO)
+   */
+  gitzoneSmartcli.addCommand('services').subscribe(async (argvArg) => {
+    const modServices = await import('./mod_services/index.js');
+    await modServices.run(argvArg);
+  });
+
   // start parsing of the cli
   gitzoneSmartcli.startParse();
   return await done.promise;

package/ts/mod_format/classes.baseformatter.ts

@@ -53,7 +53,19 @@ export abstract class BaseFormatter {
   }
 
   protected async modifyFile(filepath: string, content: string): Promise<void> {
-    await plugins.smartfile.memory.toFs(content, filepath);
+    // Validate filepath before writing
+    if (!filepath || filepath.trim() === '') {
+      throw new Error(`Invalid empty filepath in modifyFile`);
+    }
+
+    // Ensure we have a proper path with directory component
+    // If the path has no directory component (e.g., "package.json"), prepend "./"
+    let normalizedPath = filepath;
+    if (!plugins.path.parse(filepath).dir) {
+      normalizedPath = './' + filepath;
+    }
+
+    await plugins.smartfile.memory.toFs(content, normalizedPath);
   }
 
   protected async createFile(filepath: string, content: string): Promise<void> {

package/ts/mod_format/format.gitignore.ts

@@ -8,15 +8,40 @@ const gitignorePath = plugins.path.join(paths.cwd, './.gitignore');
 
 export const run = async (projectArg: Project) => {
   const gitignoreExists = await plugins.smartfile.fs.fileExists(gitignorePath);
-  const templateModule = await import('../mod_template/index.js');
-  const ciTemplate = await templateModule.getTemplate('gitignore');
+  let customContent = '';
+
   if (gitignoreExists) {
     // lets get the existing gitignore file
     const existingGitIgnoreString =
       plugins.smartfile.fs.toStringSync(gitignorePath);
-    let customPart = existingGitIgnoreString.split('# custom\n')[1];
-    customPart ? null : (customPart = '');
+
+    // Check for different custom section markers
+    const customMarkers = ['#------# custom', '# custom'];
+    for (const marker of customMarkers) {
+      const splitResult = existingGitIgnoreString.split(marker);
+      if (splitResult.length > 1) {
+        // Get everything after the marker (excluding the marker itself)
+        customContent = splitResult[1].trim();
+        break;
+      }
+    }
+  }
+
+  // Write the template
+  const templateModule = await import('../mod_template/index.js');
+  const ciTemplate = await templateModule.getTemplate('gitignore');
+  await ciTemplate.writeToDisk(paths.cwd);
+
+  // Append the custom content if it exists
+  if (customContent) {
+    const newGitignoreContent =
+      plugins.smartfile.fs.toStringSync(gitignorePath);
+    // The template already ends with "#------# custom", so just append the content
+    const finalContent =
+      newGitignoreContent.trimEnd() + '\n' + customContent + '\n';
+    await plugins.smartfile.fs.toFs(finalContent, gitignorePath);
+    logger.log('info', 'Updated .gitignore while preserving custom section!');
+  } else {
+    logger.log('info', 'Added a .gitignore!');
   }
-  ciTemplate.writeToDisk(paths.cwd);
-  logger.log('info', 'Added a .gitignore!');
 };

package/ts/mod_format/format.packagejson.ts

@@ -83,10 +83,10 @@ export const run = async (projectArg: Project) => {
         type: 'git',
         url: `https://${gitzoneData.module.githost}/${gitzoneData.module.gitscope}/${gitzoneData.module.gitrepo}.git`,
       };
-      (packageJson.bugs = {
+      ((packageJson.bugs = {
         url: `https://${gitzoneData.module.githost}/${gitzoneData.module.gitscope}/${gitzoneData.module.gitrepo}/issues`,
       }),
-        (packageJson.homepage = `https://${gitzoneData.module.githost}/${gitzoneData.module.gitscope}/${gitzoneData.module.gitrepo}#readme`);
+        (packageJson.homepage = `https://${gitzoneData.module.githost}/${gitzoneData.module.gitscope}/${gitzoneData.module.gitrepo}#readme`));
 
       // Check for module type
       if (!packageJson.type) {

package/ts/mod_format/formatters/prettier.formatter.ts

@@ -29,8 +29,9 @@ export class PrettierFormatter extends BaseFormatter {
       'README.md',
       'changelog.md',
       'CHANGELOG.md',
-      'license',
-      'LICENSE',
+      // Skip files without extensions as prettier can't infer parser
+      // 'license',
+      // 'LICENSE',
       '*.md',
     ];
 
@@ -102,40 +103,26 @@ export class PrettierFormatter extends BaseFormatter {
     try {
       await this.preExecute();
 
-      // Batch process files
-      const batchSize = 10; // Process 10 files at a time
-      const batches: IPlannedChange[][] = [];
+      logVerbose(`Processing ${changes.length} files sequentially`);
 
-      for (let i = 0; i < changes.length; i += batchSize) {
-        batches.push(changes.slice(i, i + batchSize));
-      }
-
-      logVerbose(
-        `Processing ${changes.length} files in ${batches.length} batches`,
-      );
-
-      for (let i = 0; i < batches.length; i++) {
-        const batch = batches[i];
+      // Process files sequentially to avoid prettier cache/state issues
+      for (let i = 0; i < changes.length; i++) {
+        const change = changes[i];
         logVerbose(
-          `Processing batch ${i + 1}/${batches.length} (${batch.length} files)`,
+          `Processing file ${i + 1}/${changes.length}: ${change.path}`,
         );
 
-        // Process batch in parallel
-        const promises = batch.map(async (change) => {
-          try {
-            await this.applyChange(change);
-            this.stats.recordFileOperation(this.name, change.type, true);
-          } catch (error) {
-            this.stats.recordFileOperation(this.name, change.type, false);
-            logger.log(
-              'error',
-              `Failed to format ${change.path}: ${error.message}`,
-            );
-            // Don't throw - continue with other files
-          }
-        });
-
-        await Promise.all(promises);
+        try {
+          await this.applyChange(change);
+          this.stats.recordFileOperation(this.name, change.type, true);
+        } catch (error) {
+          this.stats.recordFileOperation(this.name, change.type, false);
+          logger.log(
+            'error',
+            `Failed to format ${change.path}: ${error.message}`,
+          );
+          // Don't throw - continue with other files
+        }
       }
 
       await this.postExecute();
@@ -151,25 +138,71 @@ export class PrettierFormatter extends BaseFormatter {
     if (change.type !== 'modify') return;
 
     try {
+      // Validate the path before processing
+      if (!change.path || change.path.trim() === '') {
+        logger.log(
+          'error',
+          `Invalid empty path in change: ${JSON.stringify(change)}`,
+        );
+        throw new Error('Invalid empty path');
+      }
+
       // Read current content
       const content = plugins.smartfile.fs.toStringSync(change.path);
 
       // Format with prettier
       const prettier = await import('prettier');
-      const formatted = await prettier.format(content, {
-        filepath: change.path,
-        ...(await this.getPrettierConfig()),
-      });
 
-      // Only write if content actually changed
-      if (formatted !== content) {
-        await this.modifyFile(change.path, formatted);
-        logVerbose(`Formatted ${change.path}`);
-      } else {
-        logVerbose(`No formatting changes for ${change.path}`);
+      // Skip files that prettier can't parse without explicit parser
+      const fileExt = plugins.path.extname(change.path).toLowerCase();
+      if (!fileExt || fileExt === '') {
+        // Files without extensions need explicit parser
+        logVerbose(
+          `Skipping ${change.path} - no file extension for parser inference`,
+        );
+        return;
+      }
+
+      try {
+        const formatted = await prettier.format(content, {
+          filepath: change.path,
+          ...(await this.getPrettierConfig()),
+        });
+
+        // Only write if content actually changed
+        if (formatted !== content) {
+          // Debug: log the path being written
+          logVerbose(`Writing formatted content to: ${change.path}`);
+          await this.modifyFile(change.path, formatted);
+          logVerbose(`Formatted ${change.path}`);
+        } else {
+          logVerbose(`No formatting changes for ${change.path}`);
+        }
+      } catch (prettierError) {
+        // Check if it's a parser error
+        if (
+          prettierError.message &&
+          prettierError.message.includes('No parser could be inferred')
+        ) {
+          logVerbose(`Skipping ${change.path} - ${prettierError.message}`);
+          return; // Skip this file silently
+        }
+        throw prettierError;
       }
     } catch (error) {
-      logger.log('error', `Failed to format ${change.path}: ${error.message}`);
+      // Log the full error stack for debugging mkdir issues
+      if (error.message && error.message.includes('mkdir')) {
+        logger.log(
+          'error',
+          `Failed to format ${change.path}: ${error.message}`,
+        );
+        logger.log('error', `Error stack: ${error.stack}`);
+      } else {
+        logger.log(
+          'error',
+          `Failed to format ${change.path}: ${error.message}`,
+        );
+      }
       throw error;
     }
   }

package/ts/mod_format/index.ts

@@ -185,5 +185,8 @@ export const handleListBackups = async (): Promise<void> => {
 };
 
 export const handleCleanBackups = async (): Promise<void> => {
-  logger.log('info', 'Backup cleaning has been disabled - backup system removed');
+  logger.log(
+    'info',
+    'Backup cleaning has been disabled - backup system removed',
+  );
 };

package/ts/mod_services/classes.dockercontainer.ts

@@ -0,0 +1,227 @@
+import * as plugins from './mod.plugins.js';
+import * as helpers from './helpers.js';
+import { logger } from '../gitzone.logging.js';
+
+export type ContainerStatus = 'running' | 'stopped' | 'not_exists';
+
+export interface IDockerRunOptions {
+  name: string;
+  image: string;
+  ports?: { [key: string]: string };
+  volumes?: { [key: string]: string };
+  environment?: { [key: string]: string };
+  restart?: string;
+  command?: string;
+}
+
+export class DockerContainer {
+  private smartshell: plugins.smartshell.Smartshell;
+  
+  constructor() {
+    this.smartshell = new plugins.smartshell.Smartshell({
+      executor: 'bash',
+    });
+  }
+  
+  /**
+   * Check if Docker is installed and available
+   */
+  public async checkDocker(): Promise<boolean> {
+    try {
+      const result = await this.smartshell.exec('docker --version');
+      return result.exitCode === 0;
+    } catch (error) {
+      return false;
+    }
+  }
+  
+  /**
+   * Get container status
+   */
+  public async getStatus(containerName: string): Promise<ContainerStatus> {
+    try {
+      // Check if running
+      const runningResult = await this.smartshell.exec(
+        `docker ps --format '{{.Names}}' | grep -q "^${containerName}$"`
+      );
+      
+      if (runningResult.exitCode === 0) {
+        return 'running';
+      }
+      
+      // Check if exists but stopped
+      const existsResult = await this.smartshell.exec(
+        `docker ps -a --format '{{.Names}}' | grep -q "^${containerName}$"`
+      );
+      
+      if (existsResult.exitCode === 0) {
+        return 'stopped';
+      }
+      
+      return 'not_exists';
+    } catch (error) {
+      return 'not_exists';
+    }
+  }
+  
+  /**
+   * Start a container
+   */
+  public async start(containerName: string): Promise<boolean> {
+    try {
+      const result = await this.smartshell.exec(`docker start ${containerName}`);
+      return result.exitCode === 0;
+    } catch (error) {
+      return false;
+    }
+  }
+  
+  /**
+   * Stop a container
+   */
+  public async stop(containerName: string): Promise<boolean> {
+    try {
+      const result = await this.smartshell.exec(`docker stop ${containerName}`);
+      return result.exitCode === 0;
+    } catch (error) {
+      return false;
+    }
+  }
+  
+  /**
+   * Remove a container
+   */
+  public async remove(containerName: string, force: boolean = false): Promise<boolean> {
+    try {
+      const forceFlag = force ? '-f' : '';
+      const result = await this.smartshell.exec(`docker rm ${forceFlag} ${containerName}`);
+      return result.exitCode === 0;
+    } catch (error) {
+      return false;
+    }
+  }
+  
+  /**
+   * Run a new container
+   */
+  public async run(options: IDockerRunOptions): Promise<boolean> {
+    let command = 'docker run -d';
+    
+    // Add name
+    command += ` --name ${options.name}`;
+    
+    // Add ports
+    if (options.ports) {
+      for (const [hostPort, containerPort] of Object.entries(options.ports)) {
+        command += ` -p ${hostPort}:${containerPort}`;
+      }
+    }
+    
+    // Add volumes
+    if (options.volumes) {
+      for (const [hostPath, containerPath] of Object.entries(options.volumes)) {
+        command += ` -v "${hostPath}:${containerPath}"`;
+      }
+    }
+    
+    // Add environment variables
+    if (options.environment) {
+      for (const [key, value] of Object.entries(options.environment)) {
+        command += ` -e ${key}="${value}"`;
+      }
+    }
+    
+    // Add restart policy
+    if (options.restart) {
+      command += ` --restart ${options.restart}`;
+    }
+    
+    // Add image
+    command += ` ${options.image}`;
+    
+    // Add command if provided
+    if (options.command) {
+      command += ` ${options.command}`;
+    }
+    
+    try {
+      const result = await this.smartshell.exec(command);
+      return result.exitCode === 0;
+    } catch (error) {
+      logger.log('error', `Failed to run container: ${error.message}`);
+      return false;
+    }
+  }
+  
+  /**
+   * Execute a command in a running container
+   */
+  public async exec(containerName: string, command: string): Promise<string> {
+    try {
+      const result = await this.smartshell.exec(`docker exec ${containerName} ${command}`);
+      if (result.exitCode === 0) {
+        return result.stdout;
+      }
+      return '';
+    } catch (error) {
+      return '';
+    }
+  }
+  
+  /**
+   * Get container logs
+   */
+  public async logs(containerName: string, lines?: number): Promise<string> {
+    try {
+      const tailFlag = lines ? `--tail ${lines}` : '';
+      const result = await this.smartshell.exec(`docker logs ${tailFlag} ${containerName}`);
+      return result.stdout;
+    } catch (error) {
+      return `Error getting logs: ${error.message}`;
+    }
+  }
+  
+  /**
+   * Check if a container exists
+   */
+  public async exists(containerName: string): Promise<boolean> {
+    const status = await this.getStatus(containerName);
+    return status !== 'not_exists';
+  }
+  
+  /**
+   * Check if a container is running
+   */
+  public async isRunning(containerName: string): Promise<boolean> {
+    const status = await this.getStatus(containerName);
+    return status === 'running';
+  }
+  
+  /**
+   * Wait for a container to be ready
+   */
+  public async waitForReady(containerName: string, maxAttempts: number = 30): Promise<boolean> {
+    for (let i = 0; i < maxAttempts; i++) {
+      if (await this.isRunning(containerName)) {
+        return true;
+      }
+      await plugins.smartdelay.delayFor(1000);
+    }
+    return false;
+  }
+  
+  /**
+   * Get container information
+   */
+  public async inspect(containerName: string): Promise<any> {
+    try {
+      const result = await this.smartshell.exec(`docker inspect ${containerName}`);
+      if (result.exitCode === 0) {
+        return JSON.parse(result.stdout);
+      }
+      return null;
+    } catch (error) {
+      return null;
+    }
+  }
+}