Merge pull request 'chore: change docker build context to repo root' (#2) from stevanfreeborn/chore/change-docker-image-build-process into main

fix: address reconnection logic not working

.editorconfig

@@ -30,6 +30,7 @@ dotnet_diagnostic.IDE0058.severity = none
 dotnet_diagnostic.IDE0290.severity = none
 dotnet_diagnostic.CA1031.severity = none
 dotnet_diagnostic.CA1873.severity = none
+dotnet_diagnostic.CS1591.severity = none
 
 # Organize usings
 dotnet_separate_import_directive_groups = true

.gitea/workflows/deploy.yml

@@ -6,6 +6,7 @@ on:
       - main
     paths:
       - 'src/**'
+      - '.editorconfig'
 jobs:
   build:
     name: Build Docker Images
@@ -26,10 +27,10 @@ jobs:
       - name: Build Docker image
         run: |
           WORKER_TAG=${{ secrets.DOCKERHUB_USERNAME }}/steves-bot:worker-${{ steps.version.outputs.version }}
-          docker build -f src/StevesBot.Worker.Dockerfile -t $WORKER_TAG src
+          docker build -f src/StevesBot.Worker.Dockerfile -t $WORKER_TAG .
           docker push $WORKER_TAG
           WEBHOOK_TAG=${{ secrets.DOCKERHUB_USERNAME }}/steves-bot:webhook-${{ steps.version.outputs.version }}
-          docker build -f src/StevesBot.Webhook.Dockerfile -t $WEBHOOK_TAG src
+          docker build -f src/StevesBot.Webhook.Dockerfile -t $WEBHOOK_TAG .
           docker push $WEBHOOK_TAG
   deploy:
     name: Deploy Docker Images

.gitea/workflows/pull_request.yml

@@ -30,13 +30,15 @@ jobs:
         with:
           dotnet-version: 10.0.x
       - name: Install report generator
-        run: dotnet tool install -g dotnet-reportgenerator-globaltool
+        run: |
+          dotnet tool install --tool-path ./.tools dotnet-reportgenerator-globaltool
+          echo "$PWD/.tools" >> "$GITHUB_PATH"
       - name: Run tests
         run: dotnet test src
       - name: Merge coverage reports
         run: reportgenerator "-reports:src/tests/*/TestResults/*/coverage.cobertura.xml" "-reporttypes:Html;Cobertura" "-targetdir:src/tests/TestResults/Coverage"
       - name: Upload code coverage
-        uses: actions/upload-artifact@v6
+        uses: actions/upload-artifact@v3
         with:
           name: coverage-reports
           path: src/tests/TestResults/Coverage

README.md

@@ -234,6 +234,13 @@ FROM mcr.microsoft.com/dotnet/aspnet:10.0
 # ... runtime setup ...
 ```
 
+Images are built with the **repository root** as Docker build context so solution-wide configuration (including root-level analyzer/editorconfig rules) is available during build:
+
+```bash
+docker build -f src/StevesBot.Worker.Dockerfile -t steves-bot:worker .
+docker build -f src/StevesBot.Webhook.Dockerfile -t steves-bot:webhook .
+```
+
 ### Configuration for Production
 
 Use environment variables or configuration providers for production secrets.

src/Directory.Build.props

@@ -9,6 +9,7 @@
     <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
     <CodeAnalysisTreatWarningsAsErrors>true</CodeAnalysisTreatWarningsAsErrors>
     <EnforceCodeStyleInBuild>true</EnforceCodeStyleInBuild>
+    <GenerateDocumentationFile>true</GenerateDocumentationFile>
   </PropertyGroup>
 
 </Project>

src/StevesBot.Webhook.Dockerfile

@@ -2,32 +2,29 @@ FROM mcr.microsoft.com/dotnet/sdk:10.0 AS base
 WORKDIR /app
 
 # solution level
-COPY *.sln ./
+COPY src/StevesBot.sln src/
-COPY Directory.Build.props ./
+COPY src/Directory.Build.props src/
 
 # src level
-COPY src/StevesBot.Library/*.csproj src/StevesBot.Library/
+COPY src/src/StevesBot.Library/*.csproj src/src/StevesBot.Library/
-COPY src/StevesBot.Worker/*.csproj src/StevesBot.Worker/
+COPY src/src/StevesBot.Worker/*.csproj src/src/StevesBot.Worker/
-COPY src/StevesBot.Webhook/*.csproj src/StevesBot.Webhook/
+COPY src/src/StevesBot.Webhook/*.csproj src/src/StevesBot.Webhook/
-COPY src/Directory.Build.props src/
+COPY src/src/Directory.Build.props src/src/
-COPY src/Directory.Packages.props src/
+COPY src/src/Directory.Packages.props src/src/
 
 # test level
-COPY tests/StevesBot.Library.Tests/*.csproj tests/StevesBot.Library.Tests/
+COPY src/tests/StevesBot.Library.Tests/*.csproj src/tests/StevesBot.Library.Tests/
-COPY tests/StevesBot.Worker.Tests/*.csproj tests/StevesBot.Worker.Tests/
+COPY src/tests/StevesBot.Worker.Tests/*.csproj src/tests/StevesBot.Worker.Tests/
-COPY tests/StevesBot.Webhook.Tests/*.csproj tests/StevesBot.Webhook.Tests/
+COPY src/tests/StevesBot.Webhook.Tests/*.csproj src/tests/StevesBot.Webhook.Tests/
-COPY tests/Directory.Build.props tests/
+COPY src/tests/Directory.Build.props src/tests/
-COPY tests/Directory.Packages.props tests/
+COPY src/tests/Directory.Packages.props src/tests/
 
-COPY ./src/StevesBot.Worker/StevesBot.Worker.csproj ./
+RUN dotnet restore src/StevesBot.sln
-COPY ./Directory.Build.props ./
-
-RUN dotnet restore StevesBot.sln
 
 COPY . .
 
 FROM base AS publish-stage
-RUN dotnet publish -c Release -o dist src/StevesBot.Webhook/StevesBot.Webhook.csproj
+RUN dotnet publish -c Release -o dist src/src/StevesBot.Webhook/StevesBot.Webhook.csproj
 
 FROM mcr.microsoft.com/dotnet/aspnet:10.0
 WORKDIR /app

src/StevesBot.Worker.Dockerfile

@@ -2,32 +2,29 @@ FROM mcr.microsoft.com/dotnet/sdk:10.0 AS base
 WORKDIR /app
 
 # solution level
-COPY *.sln ./
+COPY src/StevesBot.sln src/
-COPY Directory.Build.props ./
+COPY src/Directory.Build.props src/
 
 # src level
-COPY src/StevesBot.Library/*.csproj src/StevesBot.Library/
+COPY src/src/StevesBot.Library/*.csproj src/src/StevesBot.Library/
-COPY src/StevesBot.Worker/*.csproj src/StevesBot.Worker/
+COPY src/src/StevesBot.Worker/*.csproj src/src/StevesBot.Worker/
-COPY src/StevesBot.Webhook/*.csproj src/StevesBot.Webhook/
+COPY src/src/StevesBot.Webhook/*.csproj src/src/StevesBot.Webhook/
-COPY src/Directory.Build.props src/
+COPY src/src/Directory.Build.props src/src/
-COPY src/Directory.Packages.props src/
+COPY src/src/Directory.Packages.props src/src/
 
 # test level
-COPY tests/StevesBot.Library.Tests/*.csproj tests/StevesBot.Library.Tests/
+COPY src/tests/StevesBot.Library.Tests/*.csproj src/tests/StevesBot.Library.Tests/
-COPY tests/StevesBot.Worker.Tests/*.csproj tests/StevesBot.Worker.Tests/
+COPY src/tests/StevesBot.Worker.Tests/*.csproj src/tests/StevesBot.Worker.Tests/
-COPY tests/StevesBot.Webhook.Tests/*.csproj tests/StevesBot.Webhook.Tests/
+COPY src/tests/StevesBot.Webhook.Tests/*.csproj src/tests/StevesBot.Webhook.Tests/
-COPY tests/Directory.Build.props tests/
+COPY src/tests/Directory.Build.props src/tests/
-COPY tests/Directory.Packages.props tests/
+COPY src/tests/Directory.Packages.props src/tests/
 
-COPY ./src/StevesBot.Worker/StevesBot.Worker.csproj ./
+RUN dotnet restore src/StevesBot.sln
-COPY ./Directory.Build.props ./
-
-RUN dotnet restore StevesBot.sln
 
 COPY . .
 
 FROM base AS publish-stage
-RUN dotnet publish -c Release -o dist src/StevesBot.Worker/StevesBot.Worker.csproj
+RUN dotnet publish -c Release -o dist src/src/StevesBot.Worker/StevesBot.Worker.csproj
 
 FROM mcr.microsoft.com/dotnet/aspnet:10.0
 WORKDIR /app

src/src/Directory.Packages.props

@@ -5,15 +5,15 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageVersion Include="Microsoft.AspNetCore.OpenApi" Version="9.0.5" />
+    <PackageVersion Include="Microsoft.AspNetCore.OpenApi" Version="10.0.7" />
-    <PackageVersion Include="Microsoft.Extensions.Hosting" Version="10.0.1" />
+    <PackageVersion Include="Microsoft.Extensions.Hosting" Version="10.0.7" />
-    <PackageVersion Include="Microsoft.Extensions.Http" Version="10.0.1" />
+    <PackageVersion Include="Microsoft.Extensions.Http" Version="10.0.7" />
-    <PackageVersion Include="Microsoft.Extensions.Http.Resilience" Version="10.1.0" />
+    <PackageVersion Include="Microsoft.Extensions.Http.Resilience" Version="10.5.0" />
-    <PackageVersion Include="OpenTelemetry.Exporter.Console" Version="1.14.0" />
+    <PackageVersion Include="OpenTelemetry.Exporter.Console" Version="1.15.3" />
-    <PackageVersion Include="OpenTelemetry.Exporter.OpenTelemetryProtocol" Version="1.14.0" />
+    <PackageVersion Include="OpenTelemetry.Exporter.OpenTelemetryProtocol" Version="1.15.3" />
-    <PackageVersion Include="OpenTelemetry.Extensions.Hosting" Version="1.14.0" />
+    <PackageVersion Include="OpenTelemetry.Extensions.Hosting" Version="1.15.3" />
-    <PackageVersion Include="OpenTelemetry.Instrumentation.AspNetCore" Version="1.14.0" />
+    <PackageVersion Include="OpenTelemetry.Instrumentation.AspNetCore" Version="1.15.2" />
-    <PackageVersion Include="OpenTelemetry.Instrumentation.Http" Version="1.14.0" />
+    <PackageVersion Include="OpenTelemetry.Instrumentation.Http" Version="1.15.1" />
   </ItemGroup>
 
 </Project>

src/src/StevesBot.Worker/Discord/Gateway/DiscordGatewayClient.cs

@@ -70,6 +70,8 @@ internal sealed class DiscordGatewayClient : IDiscordGatewayClient
       await SetGatewayUrlAsync(gatewayUrl, cancellationToken);
     }
 
+    _webSocket?.Dispose();
+
     await SetWebSocketAsync(_webSocketFactory.Create(), cancellationToken);
     await ConnectWithGatewayUrlAsync(cancellationToken);
 
@@ -685,9 +687,10 @@ internal sealed class DiscordGatewayClient : IDiscordGatewayClient
         await _webSocket.ConnectAsync(uri, cancellationToken);
         connected = true;
       }
-      catch (Exception e) when (e is WebSocketException)
+      catch (Exception e)
       {
-        var currentDelay = (int)(delay.TotalMilliseconds * Math.Pow(2, attempts));
+        var calculatedDelay = delay.TotalMilliseconds * Math.Pow(2, attempts);
+        var currentDelay = (int)Math.Min(calculatedDelay, int.MaxValue);
 
         _logger.LogWarning(
           e,
@@ -700,6 +703,9 @@ internal sealed class DiscordGatewayClient : IDiscordGatewayClient
         await Task.Delay(currentDelay, cancellationToken);
 
         attempts++;
+
+        _webSocket.Dispose();
+        await SetWebSocketAsync(_webSocketFactory.Create(), cancellationToken);
       }
     }
   }

src/tests/Directory.Packages.props

@@ -5,11 +5,11 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageVersion Include="coverlet.collector" Version="6.0.4" />
+    <PackageVersion Include="coverlet.collector" Version="10.0.0" />
-    <PackageVersion Include="coverlet.msbuild" Version="6.0.4" />
+    <PackageVersion Include="coverlet.msbuild" Version="10.0.0" />
     <PackageVersion Include="FluentAssertions" Version="[7.2.0]" />
-    <PackageVersion Include="Microsoft.AspNetCore.Mvc.Testing" Version="10.0.1" />
+    <PackageVersion Include="Microsoft.AspNetCore.Mvc.Testing" Version="10.0.7" />
-    <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="18.0.1" />
+    <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="18.5.1" />
     <PackageVersion Include="moq" Version="4.20.72" />
     <PackageVersion Include="RichardSzalay.MockHttp" Version="7.0.0" />
     <PackageVersion Include="xunit" Version="2.9.3" />

src/tests/StevesBot.Worker.Tests/Unit/AsyncLockTests.cs

@@ -43,7 +43,9 @@ public class AsyncLockTests
     var taskOne = FirstLockAction();
     var taskTwo = SecondLockAction();
 
+#pragma warning disable CA2027 // Cancel Task.Delay after Task.WhenAny completes
     await Task.WhenAll(taskOne, Task.WhenAny(taskTwo, Task.Delay(500)));
+#pragma warning restore CA2027 // Cancel Task.Delay after Task.WhenAny completes
 
     lockOneAcquired.Task.IsCompleted.Should().BeTrue();
     lockTwoAttempted.Task.IsCompleted.Should().BeTrue();

src/tests/StevesBot.Worker.Tests/Unit/IdentifyDataTests.cs

@@ -1,5 +1,3 @@
-using StevesBot.Worker.Discord.Gateway.Events.Data;
-
 namespace StevesBot.Worker.Tests.Unit;
 
 public class IdentifyDataTests