@sun-asterisk/sunlint 1.3.14 → 1.3.16

package/core/output-service.js

@@ -14,6 +14,24 @@ class OutputService {
     this.scoringService = new ScoringService();
     this.summaryReportService = new SummaryReportService();
     this.uploadService = new UploadService();
+    
+    // Load version from package.json
+    this.version = this._loadVersion();
+  }
+  
+  /**
+   * Load version from package.json
+   * @returns {string} Package version
+   * @private
+   */
+  _loadVersion() {
+    try {
+      const packageJsonPath = path.join(__dirname, '..', 'package.json');
+      const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
+      return packageJson.version || '1.3.16';
+    } catch (error) {
+      return '1.3.16'; // Fallback version
+    }
   }
 
   async outputResults(results, options, metadata = {}) {
@@ -111,7 +129,7 @@ class OutputService {
         cwd: process.cwd(),
         filesAnalyzed: totalFiles,
         duration: metadata.duration,
-        version: metadata.version || '1.3.13'
+        version: metadata.version || this.version
       }
     );
 

package/core/summary-report-service.js

@@ -9,7 +9,25 @@ const path = require('path');
 const { execSync } = require('child_process');
 
 class SummaryReportService {
-  constructor() {}
+  constructor() {
+    // Load version from package.json
+    this.version = this._loadVersion();
+  }
+  
+  /**
+   * Load version from package.json
+   * @returns {string} Package version
+   * @private
+   */
+  _loadVersion() {
+    try {
+      const packageJsonPath = path.join(__dirname, '..', 'package.json');
+      const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
+      return packageJson.version || '1.3.16';
+    } catch (error) {
+      return '1.3.16'; // Fallback version
+    }
+  }
 
   /**
    * Get Git repository information
@@ -214,7 +232,7 @@ class SummaryReportService {
       info_count: 0, // Reserved for future use
       lines_of_code: scoringSummary.metrics.linesOfCode,
       files_analyzed: options.filesAnalyzed || 0,
-      sunlint_version: options.version || '1.3.13',
+      sunlint_version: options.version || this.version,
       analysis_duration_ms: options.duration || 0,
       violations: violationsSummary,
       
@@ -222,7 +240,7 @@ class SummaryReportService {
       metadata: {
         generated_at: new Date().toISOString(),
         tool: 'SunLint',
-        version: options.version || '1.3.13',
+        version: options.version || this.version,
         analysis_duration_ms: options.duration || 0
       },
       quality: {

package/core/upload-service.js

@@ -89,25 +89,45 @@ class UploadService {
     const fileName = path.basename(filePath);
     const timeout = options.timeout || 30; // 30 seconds default timeout
     
-    // Build curl command
+    // Build curl command with status code output
     const curlCommand = this.buildCurlCommand(filePath, apiUrl, fileName, timeout);
     
     try {
-      // Execute curl command synchronously for simplicity
+      // Execute curl command with write-out to get status code
       const output = execSync(curlCommand, { 
         encoding: 'utf8',
         maxBuffer: 1024 * 1024, // 1MB buffer
         timeout: timeout * 1000 // Convert to milliseconds
       });
       
-      // Parse response if possible
-      let response = output.trim();
+      // Parse response with headers included (-i flag)
+      // Output format: "HTTP/1.1 200 OK\nHeaders...\n\nResponse Body"
+      const lines = output.trim().split('\n');
+      
+      // Find the status line (first line starting with HTTP)
       let statusCode = null;
+      const statusLine = lines.find(line => line.startsWith('HTTP/'));
+      if (statusLine) {
+        const statusMatch = statusLine.match(/HTTP\/[\d.]+\s+(\d{3})/);
+        if (statusMatch) {
+          statusCode = parseInt(statusMatch[1]);
+        }
+      }
+      
+      // Find response body (after empty line separating headers from body)
+      let response = '';
+      let foundEmptyLine = false;
+      for (const line of lines) {
+        if (foundEmptyLine) {
+          response += (response ? '\n' : '') + line;
+        } else if (line.trim() === '') {
+          foundEmptyLine = true;
+        }
+      }
       
-      // Try to extract status code from curl verbose output
-      const statusMatch = response.match(/HTTP\/[\d.]+\s+(\d{3})/);
-      if (statusMatch) {
-        statusCode = parseInt(statusMatch[1]);
+      // If no empty line found, assume entire output is response
+      if (!foundEmptyLine) {
+        response = output.trim();
       }
       
       return {
@@ -144,7 +164,9 @@ class UploadService {
     }
 
     curlOptions.push(
-      `--data @"${filePath}"`,
+      `--data-binary @"${filePath}"`,
+      '-i', // Include response headers in output
+      '-s', // Silent mode (no progress meter)
       `"${apiUrl}"`
     );
     

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sun-asterisk/sunlint",
-  "version": "1.3.14",
+  "version": "1.3.16",
   "description": "☀️ SunLint - Multi-language static analysis tool for code quality and security | Sun* Engineering Standards",
   "main": "cli.js",
   "bin": {