@aws/ml-container-creator 0.2.1 → 0.2.3

package/src/lib/manifest-cli.js

@@ -18,8 +18,8 @@
  * Validates: Requirements 9.1–9.7
  */
 
-import AssetManager, { VALID_RESOURCE_TYPES, VALID_STATUSES } from './asset-manager.js'
-import BootstrapConfig from './bootstrap-config.js'
+import AssetManager, { VALID_RESOURCE_TYPES, VALID_STATUSES } from './asset-manager.js';
+import BootstrapConfig from './bootstrap-config.js';
 
 /**
  * Parse command-line arguments into a map of flag → value pairs.
@@ -30,36 +30,36 @@ import BootstrapConfig from './bootstrap-config.js'
  */
 function parseArgs(argv) {
     // argv[0] = node, argv[1] = script path, argv[2] = subcommand, argv[3..] = flags
-    const args = argv.slice(2)
-    const subcommand = args.length > 0 && !args[0].startsWith('--') ? args[0] : null
-    const flags = {}
+    const args = argv.slice(2);
+    const subcommand = args.length > 0 && !args[0].startsWith('--') ? args[0] : null;
+    const flags = {};
 
     for (let i = subcommand ? 1 : 0; i < args.length; i++) {
         if (args[i].startsWith('--') && i + 1 < args.length) {
-            const key = args[i].slice(2)
-            flags[key] = args[i + 1]
-            i++
+            const key = args[i].slice(2);
+            flags[key] = args[i + 1];
+            i++;
         }
     }
 
-    return { subcommand, flags }
+    return { subcommand, flags };
 }
 
 /**
  * Print usage information and exit.
  */
 function printUsage() {
-    console.log('Usage:')
-    console.log('  manifest add --type <resourceType> --id <resourceId> --project <projectName> [--meta <json>]')
-    console.log('  manifest delete --id <resourceId>')
-    console.log('  manifest list [--project <project>] [--status <status>] [--type <type>]')
-    console.log('  manifest prune')
-    console.log('')
-    console.log('Valid resource types:')
-    console.log(`  ${VALID_RESOURCE_TYPES.join(', ')}`)
-    console.log('')
-    console.log('Valid statuses:')
-    console.log(`  ${VALID_STATUSES.join(', ')}`)
+    console.log('Usage:');
+    console.log('  manifest add --type <resourceType> --id <resourceId> --project <projectName> [--meta <json>]');
+    console.log('  manifest delete --id <resourceId>');
+    console.log('  manifest list [--project <project>] [--status <status>] [--type <type>]');
+    console.log('  manifest prune');
+    console.log('');
+    console.log('Valid resource types:');
+    console.log(`  ${VALID_RESOURCE_TYPES.join(', ')}`);
+    console.log('');
+    console.log('Valid statuses:');
+    console.log(`  ${VALID_STATUSES.join(', ')}`);
 }
 
 /**
@@ -69,40 +69,40 @@ function printUsage() {
  */
 function printResourceTable(resources) {
     if (resources.length === 0) {
-        console.log('No resources found.')
-        return
+        console.log('No resources found.');
+        return;
     }
 
     // Header
-    const header = ['Type', 'Resource ID', 'Project', 'Status', 'Created At']
-    const widths = header.map(h => h.length)
+    const header = ['Type', 'Resource ID', 'Project', 'Status', 'Created At'];
+    const widths = header.map(h => h.length);
 
     // Calculate column widths
     for (const r of resources) {
-        widths[0] = Math.max(widths[0], (r.resourceType || '').length)
-        widths[1] = Math.max(widths[1], Math.min((r.resourceId || '').length, 60))
-        widths[2] = Math.max(widths[2], (r.project || '').length)
-        widths[3] = Math.max(widths[3], (r.status || '').length)
-        widths[4] = Math.max(widths[4], (r.createdAt || '').length)
+        widths[0] = Math.max(widths[0], (r.resourceType || '').length);
+        widths[1] = Math.max(widths[1], Math.min((r.resourceId || '').length, 60));
+        widths[2] = Math.max(widths[2], (r.project || '').length);
+        widths[3] = Math.max(widths[3], (r.status || '').length);
+        widths[4] = Math.max(widths[4], (r.createdAt || '').length);
     }
 
-    const pad = (str, width) => String(str).padEnd(width)
-    const separator = widths.map(w => '-'.repeat(w)).join('  ')
+    const pad = (str, width) => String(str).padEnd(width);
+    const separator = widths.map(w => '-'.repeat(w)).join('  ');
 
-    console.log(header.map((h, i) => pad(h, widths[i])).join('  '))
-    console.log(separator)
+    console.log(header.map((h, i) => pad(h, widths[i])).join('  '));
+    console.log(separator);
 
     for (const r of resources) {
         const id = (r.resourceId || '').length > 60
-            ? r.resourceId.slice(0, 57) + '...'
-            : r.resourceId || ''
+            ? `${r.resourceId.slice(0, 57)  }...`
+            : r.resourceId || '';
         console.log([
             pad(r.resourceType || '', widths[0]),
             pad(id, widths[1]),
             pad(r.project || '', widths[2]),
             pad(r.status || '', widths[3]),
             pad(r.createdAt || '', widths[4])
-        ].join('  '))
+        ].join('  '));
     }
 }
 
@@ -113,35 +113,35 @@ function printResourceTable(resources) {
  * @param {AssetManager} assetManager - The AssetManager instance
  */
 function handleAdd(flags, assetManager) {
-    const { type, id, project, meta } = flags
+    const { type, id, project, meta } = flags;
 
     if (!type || !id || !project) {
-        console.error('Error: --type, --id, and --project are required for the add command.')
-        console.log('')
-        printUsage()
-        process.exitCode = 1
-        return
+        console.error('Error: --type, --id, and --project are required for the add command.');
+        console.log('');
+        printUsage();
+        process.exitCode = 1;
+        return;
     }
 
     if (!VALID_RESOURCE_TYPES.includes(type)) {
-        console.error(`Error: Invalid resource type "${type}".`)
-        console.error(`Valid types: ${VALID_RESOURCE_TYPES.join(', ')}`)
-        process.exitCode = 1
-        return
+        console.error(`Error: Invalid resource type "${type}".`);
+        console.error(`Valid types: ${VALID_RESOURCE_TYPES.join(', ')}`);
+        process.exitCode = 1;
+        return;
     }
 
-    let metadata = {}
+    let metadata = {};
     if (meta) {
         try {
-            metadata = JSON.parse(meta)
+            metadata = JSON.parse(meta);
         } catch (err) {
-            console.error(`Error: Invalid JSON for --meta: ${err.message}`)
-            process.exitCode = 1
-            return
+            console.error(`Error: Invalid JSON for --meta: ${err.message}`);
+            process.exitCode = 1;
+            return;
         }
     }
 
-    const now = new Date().toISOString()
+    const now = new Date().toISOString();
     const record = {
         resourceId: id,
         resourceType: type,
@@ -150,10 +150,10 @@ function handleAdd(flags, assetManager) {
         project,
         status: 'active',
         metadata
-    }
+    };
 
-    assetManager.addResource(record)
-    console.log(`Added ${type}: ${id}`)
+    assetManager.addResource(record);
+    console.log(`Added ${type}: ${id}`);
 }
 
 /**
@@ -163,21 +163,21 @@ function handleAdd(flags, assetManager) {
  * @param {AssetManager} assetManager - The AssetManager instance
  */
 function handleDelete(flags, assetManager) {
-    const { id } = flags
+    const { id } = flags;
 
     if (!id) {
-        console.error('Error: --id is required for the delete command.')
-        console.log('')
-        printUsage()
-        process.exitCode = 1
-        return
+        console.error('Error: --id is required for the delete command.');
+        console.log('');
+        printUsage();
+        process.exitCode = 1;
+        return;
     }
 
-    const updated = assetManager.updateStatus(id, 'deleted')
+    const updated = assetManager.updateStatus(id, 'deleted');
     if (updated) {
-        console.log(`Marked as deleted: ${id}`)
+        console.log(`Marked as deleted: ${id}`);
     } else {
-        console.log(`Resource not found in manifest: ${id}`)
+        console.log(`Resource not found in manifest: ${id}`);
     }
 }
 
@@ -188,13 +188,13 @@ function handleDelete(flags, assetManager) {
  * @param {AssetManager} assetManager - The AssetManager instance
  */
 function handleList(flags, assetManager) {
-    const filters = {}
-    if (flags.project) filters.project = flags.project
-    if (flags.status) filters.status = flags.status
-    if (flags.type) filters.resourceType = flags.type
+    const filters = {};
+    if (flags.project) filters.project = flags.project;
+    if (flags.status) filters.status = flags.status;
+    if (flags.type) filters.resourceType = flags.type;
 
-    const resources = assetManager.listResources(filters)
-    printResourceTable(resources)
+    const resources = assetManager.listResources(filters);
+    printResourceTable(resources);
 }
 
 /**
@@ -203,21 +203,21 @@ function handleList(flags, assetManager) {
  * @param {AssetManager} assetManager - The AssetManager instance
  */
 function handlePrune(assetManager) {
-    const all = assetManager.listResources()
-    const stale = all.filter(r => r.status === 'deleted' || r.status === 'unknown')
+    const all = assetManager.listResources();
+    const stale = all.filter(r => r.status === 'deleted' || r.status === 'unknown');
 
     if (stale.length === 0) {
-        console.log('Nothing to prune — no deleted or unknown records found.')
-        return
+        console.log('Nothing to prune — no deleted or unknown records found.');
+        return;
     }
 
     for (const resource of stale) {
-        assetManager.removeResource(resource.resourceId)
-        console.log(`  🗑️  [${resource.status}] ${resource.resourceType}: ${resource.resourceId}`)
+        assetManager.removeResource(resource.resourceId);
+        console.log(`  🗑️  [${resource.status}] ${resource.resourceType}: ${resource.resourceId}`);
     }
 
-    const remaining = assetManager.listResources()
-    console.log(`\nPruned ${stale.length} record${stale.length === 1 ? '' : 's'}. ${remaining.length} remaining.`)
+    const remaining = assetManager.listResources();
+    console.log(`\nPruned ${stale.length} record${stale.length === 1 ? '' : 's'}. ${remaining.length} remaining.`);
 }
 
 /**
@@ -227,45 +227,45 @@ function handlePrune(assetManager) {
  * @param {string[]} argv - The raw process.argv array
  */
 export function main(argv) {
-    const { subcommand, flags } = parseArgs(argv)
+    const { subcommand, flags } = parseArgs(argv);
 
     if (!subcommand) {
-        printUsage()
-        process.exitCode = 1
-        return
+        printUsage();
+        process.exitCode = 1;
+        return;
     }
 
     // Resolve active bootstrap profile
-    const bootstrapConfig = new BootstrapConfig()
-    const activeProfile = bootstrapConfig.getActiveProfile()
+    const bootstrapConfig = new BootstrapConfig();
+    const activeProfile = bootstrapConfig.getActiveProfile();
 
     if (!activeProfile) {
-        console.warn('Warning: No active bootstrap profile configured. Skipping manifest operation.')
-        console.warn('Run "ml-container-creator bootstrap" to configure a profile.')
-        return
+        console.warn('Warning: No active bootstrap profile configured. Skipping manifest operation.');
+        console.warn('Run "ml-container-creator bootstrap" to configure a profile.');
+        return;
     }
 
-    const assetManager = new AssetManager(activeProfile.name)
+    const assetManager = new AssetManager(activeProfile.name);
 
     switch (subcommand) {
-        case 'add':
-            handleAdd(flags, assetManager)
-            break
-        case 'delete':
-            handleDelete(flags, assetManager)
-            break
-        case 'list':
-            handleList(flags, assetManager)
-            break
-        case 'prune':
-            handlePrune(assetManager)
-            break
-        default:
-            console.error(`Unknown subcommand: ${subcommand}`)
-            console.log('')
-            printUsage()
-            process.exitCode = 1
-            break
+    case 'add':
+        handleAdd(flags, assetManager);
+        break;
+    case 'delete':
+        handleDelete(flags, assetManager);
+        break;
+    case 'list':
+        handleList(flags, assetManager);
+        break;
+    case 'prune':
+        handlePrune(assetManager);
+        break;
+    default:
+        console.error(`Unknown subcommand: ${subcommand}`);
+        console.log('');
+        printUsage();
+        process.exitCode = 1;
+        break;
     }
 }
 
@@ -273,8 +273,8 @@ export function main(argv) {
 const isDirectExecution = process.argv[1] && (
     process.argv[1].endsWith('manifest-cli.js') ||
     process.argv[1].endsWith('do/lib/manifest-cli.js')
-)
+);
 
 if (isDirectExecution) {
-    main(process.argv)
+    main(process.argv);
 }