MohamedGamil / MapX / 26710072042

88%

Build Type

push

github

Committed by MohamedGamil

Commit Message

fix: correct Flutter detector content guards, auto_route path inference, and role classifier precedence

- FlutterDetector.extractRoutes/extractHooks: changed content guards
  from checking file content for 'dart'/'.dart' to checking filePath
  ends with '.dart'. The old guards caused false negatives for Dart
  files without imports or without the word 'dart' in their content.

- auto_route path inference: fixed replace callback parameter order.
  The (c, i) parameters mapped to (match, captureGroup1) not
  (match, offset). Since captureGroup1 is always a string,
  'i > 0' evaluated to NaN > 0 = false, so no hyphens were ever
  inserted between PascalCase words (e.g. UserProfileScreen produced
  '/userprofile' instead of '/user-profile').

- role-classifier getNamingSignal: removed dead 'Controller' branch
  from the state-management else-if chain. Due to operator precedence
  (&& binds tighter than ||), the PascalCase check only applied to
  Controller — which was unreachable anyway since Controller already
  matches on line 277 for the 'api' role.

- Updated flutter-detector tests with stronger assertions and 3 new
  test cases covering the fixed behavior.

Coverage Stats

2869 of 3935 branches covered (72.91%)

Branch coverage included in aggregate %.

1 of 1 new or added line in 1 file covered. (100.0%)

8 existing lines in 2 files now uncovered.

4862 of 5682 relevant lines covered (85.57%)

11.8 hits per line