fix(test): Use unsigned literal in count() comparisons to silence -Wtype-limits

Author: libraz

.github/workflows/ci.yml

@@ -98,8 +98,8 @@ jobs:
       - name: Generate coverage report
         run: |
           mkdir -p build/coverage
-          lcov --directory build --capture --output-file build/coverage/coverage.info --ignore-errors source,gcov
-          lcov --extract build/coverage/coverage.info "${PWD}/src/*" --output-file build/coverage/coverage_filtered.info --ignore-errors unused
+          lcov --directory build --capture --output-file build/coverage/coverage.info --ignore-errors source,gcov,negative
+          lcov --extract build/coverage/coverage.info "${PWD}/src/*" --output-file build/coverage/coverage_filtered.info --ignore-errors unused,negative
 
       - name: Upload coverage to Codecov
         uses: codecov/codecov-action@v4

tests/analysis/music_analyzer_test.cpp

@@ -321,9 +321,10 @@ TEST_CASE("MusicAnalyzer precompute then lazy access", "[music_analyzer]") {
   REQUIRE(analyzer.key_analyzer().key().confidence >= 0.0f);
   REQUIRE(analyzer.timbre_analyzer().brightness() >= 0.0f);
   REQUIRE(analyzer.dynamics_analyzer().dynamics().dynamic_range_db >= 0.0f);
-  REQUIRE(analyzer.beat_analyzer().count() >= 0);
-  REQUIRE(analyzer.chord_analyzer().count() >= 0);
+  // count() returns size_t; just verify the calls succeed without throwing
+  (void)analyzer.beat_analyzer().count();
+  (void)analyzer.chord_analyzer().count();
   REQUIRE(!analyzer.rhythm_analyzer().groove_type().empty());
-  REQUIRE(analyzer.section_analyzer().count() >= 0);
-  REQUIRE(analyzer.melody_analyzer().count() >= 0);
+  (void)analyzer.section_analyzer().count();
+  (void)analyzer.melody_analyzer().count();
 }

tsconfig.json

@@ -2,7 +2,7 @@
   "compilerOptions": {
     "target": "ES2020",
     "module": "ESNext",
-    "moduleResolution": "node",
+    "moduleResolution": "bundler",
     "declaration": true,
     "declarationMap": true,
     "sourceMap": true,