@aws/ml-container-creator 0.2.1 → 0.2.2

package/src/lib/bootstrap-config.js

@@ -31,9 +31,9 @@
  *   - ciTableName (string): Name of the DynamoDB CI table. Defaults to "mlcc-ci-table".
  */
 
-import { readFileSync, writeFileSync, mkdirSync, existsSync } from 'node:fs'
-import { dirname, join } from 'node:path'
-import { homedir } from 'node:os'
+import { readFileSync, writeFileSync, mkdirSync, existsSync } from 'node:fs';
+import { dirname, join } from 'node:path';
+import { homedir } from 'node:os';
 
 export default class BootstrapConfig {
     /**
@@ -41,7 +41,7 @@ export default class BootstrapConfig {
      *   Defaults to ~/.ml-container-creator/config.json
      */
     constructor(configPath) {
-        this.configPath = configPath || join(homedir(), '.ml-container-creator', 'config.json')
+        this.configPath = configPath || join(homedir(), '.ml-container-creator', 'config.json');
     }
 
     /**
@@ -51,11 +51,11 @@ export default class BootstrapConfig {
      */
     read() {
         if (!existsSync(this.configPath)) {
-            return null
+            return null;
         }
 
-        const raw = readFileSync(this.configPath, 'utf8')
-        return JSON.parse(raw)
+        const raw = readFileSync(this.configPath, 'utf8');
+        return JSON.parse(raw);
     }
 
     /**
@@ -66,12 +66,12 @@ export default class BootstrapConfig {
      * @param {Object} config - The config object to write
      */
     write(config) {
-        const dir = dirname(this.configPath)
+        const dir = dirname(this.configPath);
         if (!existsSync(dir)) {
-            mkdirSync(dir, { recursive: true })
+            mkdirSync(dir, { recursive: true });
         }
 
-        writeFileSync(this.configPath, JSON.stringify(config, null, 2) + '\n')
+        writeFileSync(this.configPath, `${JSON.stringify(config, null, 2)  }\n`);
     }
 
     /**
@@ -80,7 +80,7 @@ export default class BootstrapConfig {
      * @returns {boolean} true if the config file exists
      */
     exists() {
-        return existsSync(this.configPath)
+        return existsSync(this.configPath);
     }
 
     /**
@@ -89,17 +89,17 @@ export default class BootstrapConfig {
      * @returns {{ name: string, config: Object }|null} The active profile, or null if no active profile
      */
     getActiveProfile() {
-        const config = this.read()
+        const config = this.read();
         if (!config || !config.activeProfile || !config.profiles) {
-            return null
+            return null;
         }
 
-        const profileConfig = config.profiles[config.activeProfile]
+        const profileConfig = config.profiles[config.activeProfile];
         if (!profileConfig) {
-            return null
+            return null;
         }
 
-        return { name: config.activeProfile, config: profileConfig }
+        return { name: config.activeProfile, config: profileConfig };
     }
 
     /**
@@ -109,12 +109,12 @@ export default class BootstrapConfig {
      * @returns {Object|null} The profile config object, or null if not found
      */
     getProfile(name) {
-        const config = this.read()
+        const config = this.read();
         if (!config || !config.profiles) {
-            return null
+            return null;
         }
 
-        return config.profiles[name] || null
+        return config.profiles[name] || null;
     }
 
     /**
@@ -126,16 +126,16 @@ export default class BootstrapConfig {
      * @returns {Object|null} The profile config with CI defaults, or null if not found
      */
     getProfileWithDefaults(name) {
-        const profile = this.getProfile(name)
+        const profile = this.getProfile(name);
         if (!profile) {
-            return null
+            return null;
         }
 
         return {
             ciInfraProvisioned: false,
             ciTableName: 'mlcc-ci-table',
             ...profile
-        }
+        };
     }
 
     /**
@@ -146,9 +146,9 @@ export default class BootstrapConfig {
      * @returns {{ name: string, config: Object }|null} The active profile with CI defaults, or null
      */
     getActiveProfileWithDefaults() {
-        const active = this.getActiveProfile()
+        const active = this.getActiveProfile();
         if (!active) {
-            return null
+            return null;
         }
 
         return {
@@ -158,7 +158,7 @@ export default class BootstrapConfig {
                 ciTableName: 'mlcc-ci-table',
                 ...active.config
             }
-        }
+        };
     }
 
     /**
@@ -169,17 +169,17 @@ export default class BootstrapConfig {
      * @param {Object} profileData - The profile configuration data
      */
     setProfile(name, profileData) {
-        let config = this.read()
+        let config = this.read();
         if (!config) {
-            config = { activeProfile: null, profiles: {} }
+            config = { activeProfile: null, profiles: {} };
         }
         if (!config.profiles) {
-            config.profiles = {}
+            config.profiles = {};
         }
 
-        config.profiles[name] = profileData
-        config.activeProfile = name
-        this.write(config)
+        config.profiles[name] = profileData;
+        config.activeProfile = name;
+        this.write(config);
     }
 
     /**
@@ -191,20 +191,20 @@ export default class BootstrapConfig {
      * @returns {boolean} true if the profile was removed, false if not found
      */
     removeProfile(name) {
-        const config = this.read()
+        const config = this.read();
         if (!config || !config.profiles || !config.profiles[name]) {
-            return false
+            return false;
         }
 
-        delete config.profiles[name]
+        delete config.profiles[name];
 
         if (config.activeProfile === name) {
-            const remaining = Object.keys(config.profiles)
-            config.activeProfile = remaining.length > 0 ? remaining[0] : null
+            const remaining = Object.keys(config.profiles);
+            config.activeProfile = remaining.length > 0 ? remaining[0] : null;
         }
 
-        this.write(config)
-        return true
+        this.write(config);
+        return true;
     }
 
     /**
@@ -213,12 +213,12 @@ export default class BootstrapConfig {
      * @returns {string[]} Array of profile name strings
      */
     listProfiles() {
-        const config = this.read()
+        const config = this.read();
         if (!config || !config.profiles) {
-            return []
+            return [];
         }
 
-        return Object.keys(config.profiles)
+        return Object.keys(config.profiles);
     }
 
     /**
@@ -227,12 +227,12 @@ export default class BootstrapConfig {
      * @param {string} name - The profile name to set as active
      */
     setActiveProfile(name) {
-        const config = this.read()
+        const config = this.read();
         if (!config) {
-            return
+            return;
         }
 
-        config.activeProfile = name
-        this.write(config)
+        config.activeProfile = name;
+        this.write(config);
     }
 }

package/src/lib/ci-register-helpers.js

@@ -12,7 +12,7 @@
  * behavior without executing the bash template directly.
  */
 
-import { createHash } from 'node:crypto'
+import { createHash } from 'node:crypto';
 
 /**
  * Compute a deterministic configId from canonical deployment fields.
@@ -29,9 +29,9 @@ import { createHash } from 'node:crypto'
  * @returns {string} 16-character lowercase hex string
  */
 export function computeConfigId(deploymentConfig, modelName, instanceType, region, deploymentTarget) {
-    const input = `${deploymentConfig}:${modelName || 'none'}:${instanceType}:${region}:${deploymentTarget}`
-    const hash = createHash('sha256').update(input).digest('hex')
-    return hash.slice(0, 16)
+    const input = `${deploymentConfig}:${modelName || 'none'}:${instanceType}:${region}:${deploymentTarget}`;
+    const hash = createHash('sha256').update(input).digest('hex');
+    return hash.slice(0, 16);
 }
 
 /**
@@ -49,7 +49,7 @@ export function computeConfigId(deploymentConfig, modelName, instanceType, regio
  * @returns {object} DynamoDB item structure (plain JS object, not DynamoDB JSON)
  */
 export function buildCiRecord(configId, configJson, promotedAttrs) {
-    const createdAt = new Date().toISOString().replace(/\.\d{3}Z$/, 'Z')
+    const createdAt = new Date().toISOString().replace(/\.\d{3}Z$/, 'Z');
     return {
         configId,
         schemaVersion: 1,
@@ -61,7 +61,7 @@ export function buildCiRecord(configId, configJson, promotedAttrs) {
         baseImageVersion: promotedAttrs.baseImageVersion || '',
         projectName: promotedAttrs.projectName || '',
         createdAt
-    }
+    };
 }
 
 /**
@@ -75,36 +75,36 @@ export function buildCiRecord(configId, configJson, promotedAttrs) {
  */
 export function applyRecordDefaults(record) {
     if (record.schemaVersion === undefined || record.schemaVersion === null) {
-        record.schemaVersion = 1
+        record.schemaVersion = 1;
     }
     if (!record.testStatus) {
-        record.testStatus = 'untested'
+        record.testStatus = 'untested';
     }
     if (!record.lastTestTimestamp) {
-        record.lastTestTimestamp = '1970-01-01T00:00:00Z'
+        record.lastTestTimestamp = '1970-01-01T00:00:00Z';
     }
     if (!record.buildStrategy) {
-        record.buildStrategy = 'codebuild-submit'
+        record.buildStrategy = 'codebuild-submit';
     }
     if (!record.stageResults) {
-        record.stageResults = {}
+        record.stageResults = {};
     }
     if (!record.errorMessage && record.errorMessage !== '') {
-        record.errorMessage = ''
+        record.errorMessage = '';
     }
     if (!record.deploymentConfig) {
-        record.deploymentConfig = ''
+        record.deploymentConfig = '';
     }
     if (!record.baseImage) {
-        record.baseImage = ''
+        record.baseImage = '';
     }
     if (!record.baseImageVersion) {
-        record.baseImageVersion = ''
+        record.baseImageVersion = '';
     }
     if (!record.projectName) {
-        record.projectName = ''
+        record.projectName = '';
     }
-    return record
+    return record;
 }
 
 /**
@@ -118,7 +118,7 @@ export function applyRecordDefaults(record) {
  */
 export function extractBaseImageVersion(baseImage) {
     if (!baseImage || !baseImage.includes(':')) {
-        return ''
+        return '';
     }
-    return baseImage.split(':').pop()
+    return baseImage.split(':').pop();
 }

package/src/lib/ci-report-helpers.js

@@ -32,7 +32,7 @@ export const KNOWN_DEPLOYMENT_CONFIGS = [
     'diffusors-vllm',
     'diffusors-sglang',
     'diffusors-comfyui'
-]
+];
 
 /**
  * Group an array of CI records by their deploymentConfig field.
@@ -41,15 +41,15 @@ export const KNOWN_DEPLOYMENT_CONFIGS = [
  * @returns {Map<string, object[]>} Map from deploymentConfig to array of records
  */
 export function groupByDeploymentConfig(records) {
-    const groups = new Map()
+    const groups = new Map();
     for (const record of records) {
-        const key = record.deploymentConfig || ''
+        const key = record.deploymentConfig || '';
         if (!groups.has(key)) {
-            groups.set(key, [])
+            groups.set(key, []);
         }
-        groups.get(key).push(record)
+        groups.get(key).push(record);
     }
-    return groups
+    return groups;
 }
 
 /**
@@ -64,10 +64,10 @@ export function groupByDeploymentConfig(records) {
  */
 export function detectRegressions(records) {
     return records.filter(record => {
-        const current = record.testStatus || ''
-        const previous = record.previousTestStatus || ''
-        return current.startsWith('fail-') && previous === 'pass'
-    })
+        const current = record.testStatus || '';
+        const previous = record.previousTestStatus || '';
+        return current.startsWith('fail-') && previous === 'pass';
+    });
 }
 
 /**
@@ -87,41 +87,41 @@ export function detectRegressions(records) {
  * @returns {string[]} return.untestedConfigs - Known configs with no CI_Record
  */
 export function computeCoverageReport(records, knownConfigs) {
-    const grouped = groupByDeploymentConfig(records)
+    const grouped = groupByDeploymentConfig(records);
 
     // Determine which known configs have been tested
-    const testedConfigSet = new Set()
-    const passingSet = new Set()
-    const failingSet = new Set()
+    const testedConfigSet = new Set();
+    const passingSet = new Set();
+    const failingSet = new Set();
 
-    const configurations = []
+    const configurations = [];
 
     for (const config of knownConfigs) {
-        const configRecords = grouped.get(config) || []
+        const configRecords = grouped.get(config) || [];
         if (configRecords.length === 0) {
             configurations.push({
                 deploymentConfig: config,
                 status: 'untested',
                 recordCount: 0
-            })
-            continue
+            });
+            continue;
         }
 
-        testedConfigSet.add(config)
+        testedConfigSet.add(config);
 
         // Use the most recent record (by lastTestTimestamp) to determine status
         const sorted = [...configRecords].sort((a, b) => {
-            const tsA = a.lastTestTimestamp || '1970-01-01T00:00:00Z'
-            const tsB = b.lastTestTimestamp || '1970-01-01T00:00:00Z'
-            return tsB.localeCompare(tsA)
-        })
-        const latest = sorted[0]
-        const status = latest.testStatus || 'untested'
+            const tsA = a.lastTestTimestamp || '1970-01-01T00:00:00Z';
+            const tsB = b.lastTestTimestamp || '1970-01-01T00:00:00Z';
+            return tsB.localeCompare(tsA);
+        });
+        const latest = sorted[0];
+        const status = latest.testStatus || 'untested';
 
         if (status === 'pass') {
-            passingSet.add(config)
+            passingSet.add(config);
         } else if (status.startsWith('fail-')) {
-            failingSet.add(config)
+            failingSet.add(config);
         }
 
         configurations.push({
@@ -129,20 +129,20 @@ export function computeCoverageReport(records, knownConfigs) {
             status,
             recordCount: configRecords.length,
             latestRecord: latest
-        })
+        });
     }
 
-    const total = knownConfigs.length
-    const tested = testedConfigSet.size
-    const untested = total - tested
-    const passing = passingSet.size
-    const failing = failingSet.size
+    const total = knownConfigs.length;
+    const tested = testedConfigSet.size;
+    const untested = total - tested;
+    const passing = passingSet.size;
+    const failing = failingSet.size;
     const coveragePercent = total > 0
         ? Math.round((tested / total) * 1000) / 10
-        : 0
+        : 0;
 
-    const untestedConfigs = knownConfigs.filter(c => !testedConfigSet.has(c))
-    const regressions = detectRegressions(records)
+    const untestedConfigs = knownConfigs.filter(c => !testedConfigSet.has(c));
+    const regressions = detectRegressions(records);
 
     return {
         total,
@@ -154,5 +154,5 @@ export function computeCoverageReport(records, knownConfigs) {
         configurations,
         regressions,
         untestedConfigs
-    }
+    };
 }