cleanup and version check (#47)

relative fix

Author: lulusuki

.editorconfig

@@ -175,7 +175,7 @@ csharp_style_prefer_index_operator = false:silent
 csharp_style_prefer_range_operator = false:silent
 csharp_style_prefer_switch_expression = false:none
 
-csharp_style_namespace_declarations = block_scoped:warning
+csharp_style_namespace_declarations = file_scoped:warning
 
 #Style - C# 12 features
 csharp_style_prefer_primary_constructors = false

.github/godot_version.txt

@@ -0,0 +1 @@
+4.6.1

.github/workflows/ci.yml

@@ -2,17 +2,68 @@ name: CI
 
 on:
   pull_request:
-    types: [opened, reopened]
+    types:
+      - opened
   workflow_dispatch:
 
 jobs:
-  dotnet-format:
+  godot-project-check:
     runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
+
+      - name: Verify xllint
+        shell: bash
+        run: |
+          command -v xmllint >/dev/null 2>&1 || {
+          sudo apt-get update
+          sudo apt-get install -y --no-install-recommends libxml2-utils
+          }
+
+      - name: Verify project
+        shell: bash
+        env:
+          CSPROJ_PATH: "Rhythia.csproj"
+          VERSION_FILE: ".github/godot_version.txt"
+        run: |
+          if compgen -G "${CSPROJ_PATH}.old*" > /dev/null; then
+            echo "::error file=${CSPROJ_PATH}.old::${CSPROJ_PATH}.old migration exists"
+            exit 1
+          fi
+
+          if [ ! -f "$CSPROJ_PATH" ]; then
+            echo "::error::$CSPROJ_PATH not found"
+            exit 1
+          fi
+
+          if [ ! -f "$VERSION_FILE" ]; then
+            echo "::error::$VERSION_FILE not found"
+            exit 1
+          fi
+
+          EXPECTED_VERSION=$(tr -d '[:space:]' < "$VERSION_FILE")
+          EXPECTED_SDK="Godot.NET.Sdk/${EXPECTED_VERSION}"
 
+          ACTUAL_SDK=$(xmllint --xpath 'string(/Project/@Sdk)' "$CSPROJ_PATH")
+
+          if [ -z "$ACTUAL_SDK" ]; then
+            echo "::error file=$CSPROJ_PATH::no <Project Sdk=\"...\"> attribute found"
+            exit 1
+          fi
+
+          if [ "$ACTUAL_SDK" != "$EXPECTED_SDK" ]; then
+            echo "::error file=$CSPROJ_PATH::expected Sdk=\"$EXPECTED_SDK\" but found Sdk=\"$ACTUAL_SDK\""
+            exit 1
+          fi
+
+          echo "Godot SDK OK: $ACTUAL_SDK"
+
+  dotnet-format:
+    runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5
+    - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # Checkout v4
     - name: Setup
-      uses: actions/setup-dotnet@67a3573c9a986a3f9c594539f4ab511d57bb3ce9
+      uses: actions/setup-dotnet@67a3573c9a986a3f9c594539f4ab511d57bb3ce9 # Dotnet 4.0.2
       with:
         dotnet-version: 10.0.x
 

.gitignore

@@ -21,6 +21,7 @@ export_presets.cfg
 *.translation
 
 # Mono-specific ignores
+*.csproj.user
 .mono/
 data_*/
 mono_crash.*.json

Rhythia.csproj.old

@@ -35,7 +35,7 @@ public override void _Process(double delta)
         }
 
         noteOpacity = Math.Clamp(noteOpacity, 0, 1);
-        
+
         float noteSize = (float)(LegacyRunner.CurrentAttempt.IsReplay ? LegacyRunner.CurrentAttempt.Replays[0].NoteSize : settings.NoteSize.Value) / 4;
         Transform3D transform = new(Vector3.Right * noteSize, Vector3.Up * noteSize, Vector3.Back * noteSize, Vector3.Zero);
 

Rhythia.csproj.old.1

@@ -113,10 +113,10 @@ public static void Delete(Map map)
             Maps.RemoveAll(x => x.Id == map.Id);
 
 
-                Callable.From(() =>
-                {
-                    _ = ToastNotification.Notify($"Deleted {map.PrettyTitle}!");
-                }).CallDeferred();
+            Callable.From(() =>
+            {
+                _ = ToastNotification.Notify($"Deleted {map.PrettyTitle}!");
+            }).CallDeferred();
             Callable.From(() => MapDeleted?.Invoke(map)).CallDeferred();
         }
         catch (Exception e)

Rhythia.csproj.user

@@ -930,7 +930,7 @@ public override void _Process(double delta)
         else
         {
             double audioDelay = CurrentAttempt.Progress - 1000 * (SoundManager.Song.GetPlaybackPosition() + AudioServer.GetTimeSinceLastMix());
-        
+
             if (Math.Abs(audioDelay) > 25 && CurrentAttempt.Progress > 0)
             {
                 SoundManager.Song.PitchScale = Math.Max(Mathf.Epsilon, (float)CurrentAttempt.Speed + (float)audioDelay / 1000);
@@ -1041,7 +1041,7 @@ public override void _Process(double delta)
         CurrentAttempt.Skippable = false;
 
         startGameplayMediaAtExpected(isPauseRampActive() ? SoundManager.Song.VolumeDb : getTargetMusicVolumeDb());
-            
+
         int nextNoteMillisecond = CurrentAttempt.PassedNotes >= CurrentAttempt.Map.Notes.Length ? (int)MapLength + 5000 : CurrentAttempt.Map.Notes[CurrentAttempt.PassedNotes].Millisecond;
         int lastNoteMillisecond = CurrentAttempt.PassedNotes > 0 ? CurrentAttempt.Map.Notes[CurrentAttempt.PassedNotes - 1].Millisecond : 0;
 
@@ -1638,7 +1638,7 @@ private static void updatePauseStateEachFrame(double delta)
         pauseHoldTime += (float)delta;
         pauseState = Math.Max(0, pauseState - (float)(delta / pauseHoldDuration));
         pauseHudControl.SetProgress(Math.Clamp(1f - pauseState, 0f, 1f));
-        
+
         if (CurrentAttempt.Map.AudioBuffer != null && musicStarted && SoundManager.Song.Playing)
         {
             SoundManager.Song.VolumeDb = Mathf.Lerp(getTargetMusicVolumeDb() - 60, getTargetMusicVolumeDb(), 1 - pauseState);