Update repository URLs to new locations (#53)

* Update Crucible repository URLs to canonical names

Updates repository URLs in scripts/repos.json to use the canonical GitHub repository names with proper casing. This eliminates redirect warnings during git operations and ensures consistency with actual repository names on GitHub.

Updated repositories (19 total):
- Alloy: alloy.api → Alloy.Api, alloy.ui → Alloy.Ui
- Caster: caster.api → Caster.Api, caster.ui → Caster.Ui
- Player group: console.ui → Console.Ui, player.api → Player.Api, player.ui → Player.Ui, vm.api → Vm.Api, vm.ui → Vm.Ui
- Steamfitter: steamfitter.api → Steamfitter.Api, steamfitter.ui → Steamfitter.Ui
- CITE: cite.api → CITE.Api, cite.ui → CITE.Ui
- Gallery: gallery.api → Gallery.Api, gallery.ui → Gallery.Ui
- Blueprint: blueprint.api → Blueprint.Api, blueprint.ui → Blueprint.Ui
- Gameboard: gameboard → Gameboard
- TopoMojo: TopoMojo-ui → topomojo-ui (corrected to actual lowercase repo name)

* updates sync scritp with same login as clone script regarding repo locations

* removes xdebug_filter.php since it is generated by generate-xdebug-filter.sh

* Updates AppHost to run npm install for prod mode apps, updates CITE UI path to dist/browser

* Make AWS credentials optional for Moodle and fix Steamfitter UI path

- Update ReadAwsCredentials() to return null when credentials file doesn't exist
- Conditionally set AWS Bedrock environment variables on Moodle container
- Add console logging for AWS credential status (found/not found/error)
- Fix Steamfitter UI production mode build path (dist/browser)

* Skip AWS Bedrock configuration in Moodle when credentials are missing

- post_configure.sh: Check for AWS credentials before calling configure_ai_bedrock
- setup_environment.php: Fix syntax error (missing closing parenthesis on line 53)

This prevents the error:
"Failed to configure AWS Bedrock AI provider: Missing required parameters"
when AWS credentials are not available.

The configure_ai_bedrock section is now only executed if:
- AWS_ACCESS_KEY_ID is set
- AWS_SECRET_ACCESS_KEY is set
- AWS_REGION is set

This completes the AWS credentials optional changes started in commit 45ed60b.

* fix sync-repos exiting on any repo failure

---------

Co-authored-by: Andrew Schlackman <72105194+sei-aschlackman@users.noreply.github.com>

Author: sei-awelle

Crucible.AppHost/AppHost.cs

@@ -95,7 +95,7 @@ private static IResourceBuilder<ExecutableResource> AddAngularUI(
         else
         {
             var buildCommand = string.IsNullOrEmpty(buildArgs) ? "npm run build" : $"npm run build {buildArgs}";
-            var serveProd = $"if [ ! -d {distPath} ] || [ -z \"$(ls -A {distPath} 2>/dev/null)\" ] || [ -n \"$(find src -newer {distPath} -print -quit)\" ]; then {buildCommand}; fi; npx serve -s {distPath} -l {port}";
+            var serveProd = $"if [ ! -d {distPath} ] || [ -z \"$(ls -A {distPath} 2>/dev/null)\" ] || [ -n \"$(find src -newer {distPath} -print -quit)\" ]; then npm install && {buildCommand}; fi; npx serve -s {distPath} -l {port}";
             ui = builder.AddExecutable(name, "bash", appRoot, "-c", serveProd)
                 .WithHttpEndpoint(port: port, isProxied: false);
         }
@@ -322,7 +322,7 @@ public static void AddCaster(this IDistributedApplicationBuilder builder, IResou
 
         File.Copy($"{builder.AppHostDirectory}/resources/caster.ui.json", $"{casterUiRoot}/src/assets/config/settings.env.json", overwrite: true);
 
-        var casterUi = builder.AddAngularUI("caster-ui", casterUiRoot, port: 4310, casterMode);
+        var casterUi = builder.AddAngularUI("caster-ui", casterUiRoot, port: 4310, casterMode, distPath: "dist/browser");
 
         if (!IsEnabled(casterMode))
         {
@@ -433,7 +433,7 @@ public static void AddTopoMojo(this IDistributedApplicationBuilder builder, IRes
             // Prod/off mode with fixup-wmks check
             var distPath = "dist/topomojo-work/browser";
             var fixupCheck = "[ -e node_modules/vmware-wmks/css/css/wmks-all.css ] || [ -d node_modules/vmware-wmks/img/img ]";
-            var serveProd = $"if {fixupCheck}; then bash tools/fixup-wmks.sh; fi; if [ ! -d {distPath} ] || [ -z \"$(ls -A {distPath} 2>/dev/null)\" ] || [ -n \"$(find src -newer {distPath} -print -quit)\" ]; then npm run build topomojo-work; fi; npx serve -s {distPath} -l 4201";
+            var serveProd = $"if {fixupCheck}; then bash tools/fixup-wmks.sh; fi; if [ ! -d {distPath} ] || [ -z \"$(ls -A {distPath} 2>/dev/null)\" ] || [ -n \"$(find src -newer {distPath} -print -quit)\" ]; then npm install && npm run build topomojo-work; fi; npx serve -s {distPath} -l 4201";
             topoUi = builder.AddExecutable("topomojo-ui", "bash", topoUiRoot, "-c", serveProd)
                 .WithHttpEndpoint(port: 4201, isProxied: false)
                 .WithHttpHealthCheck();
@@ -510,7 +510,7 @@ public static void AddSteamfitter(this IDistributedApplicationBuilder builder, I
 
         File.Copy($"{builder.AppHostDirectory}/resources/steamfitter.ui.json", $"{steamfitterUiRoot}/src/assets/config/settings.env.json", overwrite: true);
 
-        var steamfitterUi = builder.AddAngularUI("steamfitter-ui", steamfitterUiRoot, port: 4401, steamfitterMode);
+        var steamfitterUi = builder.AddAngularUI("steamfitter-ui", steamfitterUiRoot, port: 4401, steamfitterMode, distPath: "dist/browser");
 
         if (!IsEnabled(steamfitterMode))
         {
@@ -773,11 +773,19 @@ public static void AddMoodle(this IDistributedApplicationBuilder builder, IResou
             .WithEnvironment("DB_USER", postgres.Resource.UserNameReference)
             .WithEnvironment("DB_PASS", postgres.Resource.PasswordParameter)
             .WithEnvironment("DB_HOST", postgres.Resource.PrimaryEndpoint.Property(EndpointProperty.Host))
-            .WithEnvironment("DB_NAME", moodleDb.Resource.DatabaseName)
-            .WithEnvironment("AWS_ACCESS_KEY_ID", awsCreds["aws_access_key_id"])
-            .WithEnvironment("AWS_SECRET_ACCESS_KEY", awsCreds["aws_secret_access_key"])
-            .WithEnvironment("AWS_SESSION_TOKEN", awsCreds["aws_session_token"])
-            .WithEnvironment("AWS_REGION", awsCreds["region"])
+            .WithEnvironment("DB_NAME", moodleDb.Resource.DatabaseName);
+
+        // Only set AWS credentials if the credentials file exists
+        if (awsCreds != null)
+        {
+            moodle
+                .WithEnvironment("AWS_ACCESS_KEY_ID", awsCreds["aws_access_key_id"])
+                .WithEnvironment("AWS_SECRET_ACCESS_KEY", awsCreds["aws_secret_access_key"])
+                .WithEnvironment("AWS_SESSION_TOKEN", awsCreds["aws_session_token"])
+                .WithEnvironment("AWS_REGION", awsCreds["region"]);
+        }
+
+        moodle
             // Pass which Crucible services are enabled
             .WithEnvironment("CRUCIBLE_PLAYER_ENABLED", IsEnabled(playerMode) ? "1" : "0")
             .WithEnvironment("CRUCIBLE_CASTER_ENABLED", IsEnabled(casterMode) ? "1" : "0")
@@ -1024,8 +1032,17 @@ private static void ConfigureXApi<T>(IResourceBuilder<T> resource, string platfo
             .WithEnvironment("XApiOptions__Platform", platform);
     }
 
-    private static Dictionary<string, string> ReadAwsCredentials()
+    private static Dictionary<string, string>? ReadAwsCredentials()
     {
+        var homeDir = Environment.GetEnvironmentVariable("HOME") ?? "";
+        var credentialsPath = Path.Combine(homeDir, ".aws", "sso-credentials");
+
+        if (!File.Exists(credentialsPath))
+        {
+            Console.WriteLine($"AWS credentials file not found at {credentialsPath}. AWS environment variables will not be set.");
+            return null;
+        }
+
         var creds = new Dictionary<string, string>
         {
             ["aws_access_key_id"] = "",
@@ -1034,11 +1051,6 @@ private static Dictionary<string, string> ReadAwsCredentials()
             ["region"] = "us-east-1"
         };
 
-        var homeDir = Environment.GetEnvironmentVariable("HOME") ?? "";
-        var credentialsPath = Path.Combine(homeDir, ".aws", "sso-credentials");
-
-        if (!File.Exists(credentialsPath)) return creds;
-
         try
         {
             var json = File.ReadAllText(credentialsPath);
@@ -1056,10 +1068,13 @@ private static Dictionary<string, string> ReadAwsCredentials()
 
             if (root.TryGetProperty("Region", out var region))
                 creds["region"] = region.GetString() ?? "us-east-1";
+
+            Console.WriteLine($"AWS credentials loaded from {credentialsPath}");
         }
         catch (Exception ex)
         {
             Console.WriteLine($"Warning: Failed to parse AWS credentials from {credentialsPath}: {ex.Message}");
+            return null;
         }
 
         return creds;

Crucible.AppHost/resources/moodle/post_configure.sh

@@ -390,7 +390,14 @@ execute_section "xAPI Configuration" configure_xapi
 execute_section "Crucible Configuration" configure_crucible
 execute_section "TopoMojo Configuration" configure_topomojo
 execute_section "Course Creation" create_course
-execute_section "Configure AWS Bedrock AI Provider" configure_ai_bedrock
+
+# Only configure AWS Bedrock if credentials are available
+if [ -n "$AWS_ACCESS_KEY_ID" ] && [ -n "$AWS_SECRET_ACCESS_KEY" ] && [ -n "$AWS_REGION" ]; then
+    log "AWS credentials found, configuring Bedrock AI provider..."
+    execute_section "Configure AWS Bedrock AI Provider" configure_ai_bedrock
+else
+    log "AWS credentials not found, skipping Bedrock AI provider configuration"
+fi
 
 # On subsequent runs add admin user to the list of site admins
 ADMINUSERID=$(moosh user-list | grep admin@localhost | sed -e "s/admin.*(\([0-9]\)),.*/\1/")

Crucible.AppHost/resources/moodle/setup_environment.php

@@ -50,8 +50,8 @@
         enable_auth_oauth2();
         break;
     case 'configure_ai_bedrock':
-        if (empty($options['accesskeyid']) || empty($options['secretaccesskey']) || empty($options['sessiontoken'] ||
-            empty($options['region']) || empty($options['modelid']))) {
+        if (empty($options['accesskeyid']) || empty($options['secretaccesskey']) || empty($options['sessiontoken']) ||
+            empty($options['region']) || empty($options['modelid'])) {
             cli_error("Missing required parameters. Current values:\n" .
                       "  --accesskeyid={$options['accesskeyid']}\n" .
                       "  --secretaccesskey={$options['secretaccesskey']}\n" .

Crucible.AppHost/resources/moodle/xdebug_filter.php

@@ -5,11 +5,11 @@
       "repos": [
         {
           "name": "alloy.api",
-          "url": "https://github.com/cmu-sei/alloy.api"
+          "url": "https://github.com/cmu-sei/Alloy.Api"
         },
         {
           "name": "alloy.ui",
-          "url": "https://github.com/cmu-sei/alloy.ui"
+          "url": "https://github.com/cmu-sei/Alloy.Ui"
         }
       ]
     },
@@ -18,11 +18,11 @@
       "repos": [
         {
           "name": "caster.api",
-          "url": "https://github.com/cmu-sei/caster.api"
+          "url": "https://github.com/cmu-sei/Caster.Api"
         },
         {
           "name": "caster.ui",
-          "url": "https://github.com/cmu-sei/caster.ui"
+          "url": "https://github.com/cmu-sei/Caster.Ui"
         }
       ]
     },
@@ -44,23 +44,23 @@
       "repos": [
         {
           "name": "console.ui",
-          "url": "https://github.com/cmu-sei/console.ui"
+          "url": "https://github.com/cmu-sei/Console.Ui"
         },
         {
           "name": "player.api",
-          "url": "https://github.com/cmu-sei/player.api"
+          "url": "https://github.com/cmu-sei/Player.Api"
         },
         {
           "name": "player.ui",
-          "url": "https://github.com/cmu-sei/player.ui"
+          "url": "https://github.com/cmu-sei/Player.Ui"
         },
         {
           "name": "vm.api",
-          "url": "https://github.com/cmu-sei/vm.api"
+          "url": "https://github.com/cmu-sei/Vm.Api"
         },
         {
           "name": "vm.ui",
-          "url": "https://github.com/cmu-sei/vm.ui"
+          "url": "https://github.com/cmu-sei/Vm.Ui"
         }
       ]
     },
@@ -73,7 +73,7 @@
         },
         {
           "name": "topomojo-ui",
-          "url": "https://github.com/cmu-sei/TopoMojo-ui"
+          "url": "https://github.com/cmu-sei/topomojo-ui"
         }
       ]
     },
@@ -82,11 +82,11 @@
       "repos": [
         {
           "name": "steamfitter.api",
-          "url": "https://github.com/cmu-sei/steamfitter.api"
+          "url": "https://github.com/cmu-sei/Steamfitter.Api"
         },
         {
           "name": "steamfitter.ui",
-          "url": "https://github.com/cmu-sei/steamfitter.ui"
+          "url": "https://github.com/cmu-sei/Steamfitter.Ui"
         }
       ]
     },
@@ -95,11 +95,11 @@
       "repos": [
         {
           "name": "cite.api",
-          "url": "https://github.com/cmu-sei/cite.api"
+          "url": "https://github.com/cmu-sei/CITE.Api"
         },
         {
           "name": "cite.ui",
-          "url": "https://github.com/cmu-sei/cite.ui"
+          "url": "https://github.com/cmu-sei/CITE.Ui"
         }
       ]
     },
@@ -108,11 +108,11 @@
       "repos": [
         {
           "name": "gallery.api",
-          "url": "https://github.com/cmu-sei/gallery.api"
+          "url": "https://github.com/cmu-sei/Gallery.Api"
         },
         {
           "name": "gallery.ui",
-          "url": "https://github.com/cmu-sei/gallery.ui"
+          "url": "https://github.com/cmu-sei/Gallery.Ui"
         }
       ]
     },
@@ -121,11 +121,11 @@
       "repos": [
         {
           "name": "blueprint.api",
-          "url": "https://github.com/cmu-sei/blueprint.api"
+          "url": "https://github.com/cmu-sei/Blueprint.Api"
         },
         {
           "name": "blueprint.ui",
-          "url": "https://github.com/cmu-sei/blueprint.ui"
+          "url": "https://github.com/cmu-sei/Blueprint.Ui"
         }
       ]
     },
@@ -134,7 +134,7 @@
       "repos": [
         {
           "name": "gameboard",
-          "url": "https://github.com/cmu-sei/gameboard"
+          "url": "https://github.com/cmu-sei/Gameboard"
         },
         {
           "name": "gameboard-ui",