fileflows 1.0.1 → 1.0.3

package/cli.js

@@ -3,11 +3,9 @@
  * CLI tool for generating FILE_FLOWS.md
  * Analyzes project files and creates documentation showing data flow relationships
  * Following NPM architecture guidelines with modular structure
+ * Refactored: Global const declarations moved to config/localVars.js
  */
 
-const generateFileFlows = require(`./lib/fileFlowsGenerator`);
-const localVars = require(`./config/localVars`);
-
 /**
  * CLI usage display functionality
  * Shows help information for the CLI tool
@@ -31,6 +29,9 @@ function showUsage() {
  * Parses arguments and executes file flows generation
  */
 async function main() {
+  const localVars = require(`./config/localVars`);
+  const { generateFileFlows } = localVars.getDependencies();
+  
   const args = process.argv.slice(2);
   let rootDir = localVars.DEFAULT_ROOT_DIR;
   let outputFile = null;

package/config/localVars.js

@@ -35,6 +35,64 @@ const MAX_YAML_KEYS = 5;
 const MAX_SHELL_COMMANDS = 5;
 const MAX_HTML_TAGS = 5;
 
+// === CENTRALIZED DEPENDENCIES ===
+// All require statements moved from global scope to centralized configuration
+function getDependencies() {
+  return {
+    // Core library modules (from lib/index.js)
+    generateFileFlows: require(`../lib/fileFlowsGenerator`),
+    groupByDataFlow: require(`../lib/dataFlowGrouper`),
+    classifyFile: require(`../lib/fileClassifier`),
+    parseJSFile: require(`../lib/jsParser`),
+    parseOtherFile: require(`../lib/otherFileParser`),
+    extractDependencies: require(`../lib/dependencyExtractor`),
+    fileIO: require(`../lib/fileIO`),
+    graphUtils: require(`../lib/graphUtils`),
+    
+    // System dependencies (from qtests-runner.js)
+    fs: require(`fs`),
+    path: require(`path`),
+    childProcess: require(`child_process`),
+    os: require(`os`)
+  };
+}
+
+// === DEVELOPMENT DEPENDENCIES ===
+// Async loading for ES module dependencies (qtests is an ES module)
+async function getDevDependencies() {
+  // Check if this is a development environment and qtests is available
+  if (process.env.NODE_ENV === 'development' || process.env.NODE_ENV === 'test') {
+    try {
+      // Use dynamic import for ES modules
+      const qtests = await import('qtests');
+      return {
+        qtests: qtests
+      };
+    } catch (error) {
+      // qtests not available 
+      return {
+        qtests: null
+      };
+    }
+  }
+  
+  // Production environment - don't load qtests at all
+  return {
+    qtests: null
+  };
+}
+
+// === ENVIRONMENT VARIABLES ===
+// Centralized environment variable access using qerrors utilities
+const { getEnv } = require(`qerrors`);
+
+function getEnvironmentConfig() {
+  return {
+    NODE_ENV: getEnv(`NODE_ENV`, `development`),
+    DEBUG_TESTS: getEnv(`DEBUG_TESTS`, `false`) === `true`
+  };
+}
+
 module.exports = {
   CODE_EXTENSIONS,
   ALL_EXTENSIONS,
@@ -44,5 +102,8 @@ module.exports = {
   MAX_JSON_KEYS,
   MAX_YAML_KEYS,
   MAX_SHELL_COMMANDS,
-  MAX_HTML_TAGS
+  MAX_HTML_TAGS,
+  getDependencies,
+  getDevDependencies,
+  getEnvironmentConfig
 };

package/index.js

@@ -1,8 +1,13 @@
 /**
  * Main entry point for the file-flows-cli NPM module
  * Exports public functionality following NPM architecture guidelines
+ * Refactored: Global const declarations moved to config/localVars.js
  */
 
-const { generateFileFlows } = require(`./lib/index`);
+function getMainExport() {
+  const localVars = require(`./config/localVars`);
+  const { generateFileFlows } = localVars.getDependencies();
+  return generateFileFlows;
+}
 
-module.exports = generateFileFlows;
+module.exports = getMainExport();

package/lib/fileFlowsGenerator.js

@@ -71,13 +71,21 @@ async function generateFileFlows(rootDir = `.`, outputFile = null) {
     
     files = findFiles('.');
   } catch (error) {
-    console.error(`⚠️ File flow generation failed:`, error.message);
+    require('qerrors').logError(error, 'File flow generation failed during file discovery', { 
+      context: 'FILE_DISCOVERY_ERROR',
+      rootDir: ROOT,
+      outputFile: FILE_OUTPUT 
+    });
     return { filesAnalyzed: 0, flowGroups: 0, outputFile: null };
   }
 
   // Check if any files were found
   if (files.length === 0) {
-    console.log(`⚠️ No files found in directory: ${ROOT}`);
+    require('qerrors').logWarn(`No files found in directory: ${ROOT}`, 'generateFileFlows', { 
+      context: 'NO_FILES_FOUND',
+      rootDir: ROOT,
+      searchExtensions: localVars.ALL_EXTENSIONS 
+    });
     return { filesAnalyzed: 0, flowGroups: 0, outputFile: null };
   }
 
@@ -136,7 +144,12 @@ async function generateFileFlows(rootDir = `.`, outputFile = null) {
     outputFile: FILE_OUTPUT
   };
   
-  console.log(`✅ FILE_FLOWS.md written with ${result.filesAnalyzed} files.`);
+  require('qerrors').logInfo(`FILE_FLOWS.md generation completed`, 'generateFileFlows', { 
+    context: 'GENERATION_SUCCESS',
+    filesAnalyzed: result.filesAnalyzed,
+    flowGroups: result.flowGroups,
+    outputFile: result.outputFile
+  });
   return result;
 }
 

package/lib/fileIO.js

@@ -39,7 +39,12 @@ function safeResolvePath(rootDir, filePath) {
     
     // Prevent directory traversal attacks
     if (relative.startsWith(`..`)) {
-      throw new Error(`Path traversal attempt detected: ${filePath}`);
+      const { createTypedError, ErrorTypes } = require('qerrors');
+      throw createTypedError(
+        `Path traversal attempt detected: ${filePath}`,
+        ErrorTypes.VALIDATION,
+        'PATH_TRAVERSAL_DETECTED'
+      );
     }
     
     return { success: true, resolvedPath: resolved };

package/lib/index.js

@@ -1,37 +1,29 @@
 /**
  * Library aggregation module
  * Exports all library functionality following NPM architecture guidelines
+ * Refactored: Global const declarations moved to config/localVars.js
  */
 
-const generateFileFlows = require(`./fileFlowsGenerator`);
-const groupByDataFlow = require(`./dataFlowGrouper`);
-const classifyFile = require(`./fileClassifier`);
-const parseJSFile = require(`./jsParser`);
-const parseOtherFile = require(`./otherFileParser`);
-const extractDependencies = require(`./dependencyExtractor`);
+// 🚩AI: MAIN_LIBRARY_EXPORTS_REGISTRY - Dependencies centralized in config/localVars.js
+function getLibraryModules() {
+  const localVars = require(`../config/localVars`);
+  const deps = localVars.getDependencies();
+  
+  return {
+    generateFileFlows: deps.generateFileFlows,
+    groupByDataFlow: deps.groupByDataFlow,
+    classifyFile: deps.classifyFile,
+    parseJSFile: deps.parseJSFile,
+    parseOtherFile: deps.parseOtherFile,
+    extractDependencies: deps.extractDependencies,
+    safeReadFileSync: deps.fileIO.safeReadFileSync,
+    safeResolvePath: deps.fileIO.safeResolvePath,
+    buildDependencyGraph: deps.graphUtils.buildDependencyGraph,
+    findConnectedComponents: deps.graphUtils.findConnectedComponents,
+    depthFirstSearch: deps.graphUtils.depthFirstSearch,
+    groupBySimilarity: deps.graphUtils.groupBySimilarity,
+    calculateRoleScore: deps.graphUtils.calculateRoleScore
+  };
+}
 
-// 🚩AI: MAIN_LIBRARY_EXPORTS_REGISTRY
-const { safeReadFileSync, safeResolvePath } = require(`./fileIO`);
-const {
-  buildDependencyGraph,
-  findConnectedComponents,
-  depthFirstSearch,
-  groupBySimilarity,
-  calculateRoleScore
-} = require(`./graphUtils`);
-
-module.exports = {
-  generateFileFlows,
-  groupByDataFlow,
-  classifyFile,
-  parseJSFile,
-  parseOtherFile,
-  extractDependencies,
-  safeReadFileSync,
-  safeResolvePath,
-  buildDependencyGraph,
-  findConnectedComponents,
-  depthFirstSearch,
-  groupBySimilarity,
-  calculateRoleScore
-};
+module.exports = getLibraryModules();

package/lib/jsParser.js

@@ -110,7 +110,12 @@ function parseJSFile(content, filePath) {
     
     visitNode(ast);
   } catch (err) {
-    // Suppress parse errors for cleaner console output
+    // Log parse errors for debugging but don't crash
+    require('qerrors').logWarn(`JavaScript parsing failed for ${filePath}`, 'parseJSFile', {
+      context: 'JS_PARSE_ERROR',
+      filePath: filePath,
+      errorMessage: err.message
+    });
   }
 
   return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fileflows",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "description": "CLI tool for deploying data workflow analysis documentation",
   "main": "cli.js",
   "bin": {
@@ -54,9 +54,6 @@
   "dependencies": {
     "@babel/parser": "^7.28.3",
     "@babel/traverse": "^7.28.3",
-    "commoncontext": "^1.0.0",
-    "globby": "^14.1.0",
-    "npmcontext": "^1.0.0",
     "qerrors": "^1.2.6",
     "winston": "^3.17.0",
     "winston-daily-rotate-file": "^5.0.0"
@@ -64,11 +61,13 @@
   "devDependencies": {
     "agentsqripts": "^1.0.8",
     "arqitect": "^1.0.7",
+    "commoncontext": "^1.0.1",
     "loqatevars": "^1.0.6",
     "madge": "^8.0.0",
-    "qtests": "^1.1.8",
+    "npmcontext": "^1.0.1",
+    "qtests": "^1.1.9",
     "quantumagent": "^1.0.5",
     "repomix": "^1.2.0",
     "unqommented": "^1.1.0"
   }
-}
+}