feat(infra): prevent invariant globalization in server web because of client web publish settings #11367 (#11368)

Author: ysmoradi

.github/workflows/admin-sample.cd.yml

@@ -127,6 +127,11 @@ jobs:
     - name: Configure bswup # The adminpanel.bitplatform.dev lacks pre-rendering functionality, but the configuration below enables lazy loading of files when feasible. True offline support is not expected for this demo.
       run: |
           sed -i "s/\/\/ self.prerenderMode = 'initial'/self.prerenderMode = 'initial'/g" AdminPanel/src/Client/AdminPanel.Client.Web/wwwroot/service-worker.published.js
+
+    - name: Enable WebInteropApp route
+      run: |
+          # Uncomment the Route attribute for WebInteropApp component as noted in the WebInteropApp.razor's comments
+          sed -i 's/@\* @attribute \[Route("web-interop-app")\] \*@/@attribute [Route("web-interop-app")]/' AdminPanel/src/Client/AdminPanel.Client.Core/Components/WebInteropApp.razor
       
     - name: Generate CSS/JS files
       run: dotnet build AdminPanel/src/Client/AdminPanel.Client.Core/AdminPanel.Client.Core.csproj -t:BeforeBuildTasks -p:Version="${{ vars.APP_VERSION}}" --no-restore -c Release

.github/workflows/todo-sample.cd.yml

@@ -150,6 +150,11 @@ jobs:
 
     - name: Install wasm
       run:  cd src && dotnet workload install wasm-tools
+
+    - name: Enable WebInteropApp route
+      run: |
+          # Uncomment the Route attribute for WebInteropApp component as noted in the WebInteropApp.razor's comments
+          sed -i 's/@\* @attribute \[Route("web-interop-app")\] \*@/@attribute [Route("web-interop-app")]/' TodoSample/src/Client/TodoSample.Client.Core/Components/WebInteropApp.razor
       
     - name: Generate CSS/JS files
       run: dotnet build TodoSample/src/Client/TodoSample.Client.Core/TodoSample.Client.Core.csproj -t:BeforeBuildTasks -p:Version="${{ vars.APP_VERSION}}" --no-restore -c Release
@@ -197,6 +202,11 @@ jobs:
 
     - name: Install wasm
       run:  cd src && dotnet workload install wasm-tools
+
+    - name: Enable WebInteropApp route
+      run: |
+          # Uncomment the Route attribute for WebInteropApp component as noted in the WebInteropApp.razor's comments
+          sed -i 's/@\* @attribute \[Route("web-interop-app")\] \*@/@attribute [Route("web-interop-app")]/' TodoSample/src/Client/TodoSample.Client.Core/Components/WebInteropApp.razor
       
     - name: Generate CSS/JS files
       run: dotnet build TodoSample/src/Client/TodoSample.Client.Core/TodoSample.Client.Core.csproj -t:BeforeBuildTasks -p:Version="${{ vars.APP_VERSION}}" -p:InvariantGlobalization=true --no-restore -c Release
@@ -244,6 +254,11 @@ jobs:
 
     - name: Install wasm
       run:  cd src && dotnet workload install wasm-tools
+
+    - name: Enable WebInteropApp route
+      run: |
+          # Uncomment the Route attribute for WebInteropApp component as noted in the WebInteropApp.razor's comments
+          sed -i 's/@\* @attribute \[Route("web-interop-app")\] \*@/@attribute [Route("web-interop-app")]/' TodoSample/src/Client/TodoSample.Client.Core/Components/WebInteropApp.razor
       
     - name: Generate CSS/JS files
       run: dotnet build TodoSample/src/Client/TodoSample.Client.Core/TodoSample.Client.Core.csproj -t:BeforeBuildTasks -p:Version="${{ vars.APP_VERSION}}" --no-restore -c Release

src/Templates/Boilerplate/Bit.Boilerplate/src/Client/Boilerplate.Client.Core/Components/WebInteropApp.razor

@@ -11,7 +11,11 @@ Additionally, in a Web Browser, if Social Sign-in is performed in a Popup or a n
 state and avoid restarting Blazor upon returning from Social Sign-in, the Popup should redirect back to WebInteropApp
 after authentication. Using app.js, it can notify the main Window via window.opener.postMessage({ key: 'PUBLISH_MESSAGE', ... }),
 allowing the main Window to resume its operations.
+
+When publishing Client.Web (BlazorWebAssembly Standalone) and Server.Api, there won't be any WebInteropAppEndpoint,
+in this case, uncomment the Route attribute below to make this component accessible via /web-interop-app route.
 *@
+@* @attribute [Route("web-interop-app")] *@
 
 @code {
     [Inject] public IStringLocalizer<AppStrings> Localizer { get; set; } = default!;

src/Templates/Boilerplate/Bit.Boilerplate/src/Server/Boilerplate.Server.Api/Controllers/AttachmentController.cs

@@ -228,7 +228,7 @@ private async Task<IActionResult> UploadAttachment(Guid attachmentId, Attachment
 
                     var response = await chatClient.GetResponseAsync<AIImageReviewResponse>(
                         messages: [
-                            new ChatMessage(ChatRole.System, "Return a JSON object with two properties: 'isCar' (boolean, true if the image contains a car, false otherwise) and 'alt' (string, describing the image content). Do not include any other properties or text."),
+                            new ChatMessage(ChatRole.System, "Return a JSON object with two properties: 'isCar' (boolean, true if the image contains a car, false otherwise) and 'alt' (string, describing the image content, focusing on the car). Do not include any other properties or text."),
                             new ChatMessage(ChatRole.User, "Analyze this image.")
                             {
                                 Contents = [new DataContent(imageBytes, "image/webp")]

src/Templates/Boilerplate/Bit.Boilerplate/src/Server/Boilerplate.Server.Api/SignalR/AppHub.Chatbot.cs

@@ -168,7 +168,10 @@ async Task HandleIncomingMessage(string incomingMessage, CancellationToken messa
                         new(ChatRole.System, supportSystemPrompt),
                         new(ChatRole.User, incomingMessage),
                         new(ChatRole.Assistant, assistantResponse.ToString()),
-                        new(ChatRole.User, @"Return up to 3 relevant follow-up suggestions that help users discover related topics and continue the conversation naturally based on user's query in JSON object containing string[] named FollowUpSuggestions."),],
+                        new(ChatRole.User, @"Return up to 3 relevant follow-up suggestions that help users discover related topics and continue the conversation naturally based on user's query in JSON object containing string[] named FollowUpSuggestions.
+Only suggest follow-up questions that are within the assistant's scope and knowledge.
+Do not suggest questions that require access to data or functionality that is unavailable or out of scope for this assistant.
+Avoid suggesting questions that the assistant would not be able to answer."),],
                         chatOptions, cancellationToken: cancellationToken);
 
                     await channel.Writer.WriteAsync(JsonSerializer.Serialize(followUpItems.Result), cancellationToken);

src/Templates/Boilerplate/Bit.Boilerplate/src/Server/Boilerplate.Server.Web/Program.cs

@@ -11,6 +11,8 @@ public static partial class Program
 {
     public static async Task Main(string[] args)
     {
+        ConfigureGlobalization();
+
         var builder = WebApplication.CreateBuilder(options: new()
         {
             Args = args,
@@ -69,4 +71,16 @@ private static void LogException(object? error, string reportedBy, WebApplicatio
             _ = Console.Error.WriteLineAsync(error?.ToString() ?? "Unknown error");
         }
     }
+
+    /// <summary>
+    /// You might consider setting `InvariantGlobalization` to `true` when publishing Server.Web and Blazor WebAssembly simultaneously,
+    /// as this can reduce the website's size. However, doing so would also make the server project culture-invariant, which offers minimal benefit
+    /// and could potentially cause issues.The following environment variable allows you to maintain server culture support
+    /// while reducing the client's size through invariant culture.
+    /// https://learn.microsoft.com/en-us/dotnet/core/runtime-config/globalization#invariant-mode
+    /// </summary>
+    private static void ConfigureGlobalization()
+    {
+        Environment.SetEnvironmentVariable("DOTNET_SYSTEM_GLOBALIZATION_INVARIANT", "false");
+    }
 }