Incorrect behavior of @ngrx/avoid-mapping-selectors lint rule

[yurakhomitsky]

Minimal reproduction of the bug/regression with instructions

Minimal reproduction of the bug/regression with instructions

1. install ngrx-eslint-plugin: "@ngrx/eslint-plugin": "^14.0.2",
2. add to eslint config : extends: ['plugin:@ngrx/recommended'],
3. Use this.store.select(selector).pipe(switchMap, map)
4. Note warning

Expected behavior there should not be a warning if there is an operator that changes the stream before the map

Versions of NgRx, Angular, Node, affected browser(s) and operating system(s)

Ngrx: 13.1.0
Angular: 13.3.2
Node: 16.15.0

Other information

No response

I would be willing to submit a PR to fix this issue

• Yes
• No

[timdeschryver]

Could you elaborate this use case a bit more please?
In the snippet shown, the map that's giving the error isn't doing anything with the selector's result, so I would move it to the map logic of the switchMap.

[yurakhomitsky]

@timdeschryver, sorry for the lousy screenshot, here is another one.
In my case, the map logic could be moved to switchMap's map.

However what if the logic in the switchMap's map would be hard to read and I would want to move it to a separate map

I was thinking about whether it is possible to not show a warning if there is a switchMap/mergeMap before the map operator in the store selector pipe chain.