fix: run tb fmt on collections widget pipes (IN-1195)

CI's Datafiles checks job runs `tb fmt --diff` on every changed pipe; these
5 files had formatting drift. pull_requests_filtered.pipe needed a small
WHERE-clause reorder (AND is commutative, no behavior change) because
tb fmt oscillates indefinitely on the original `{% end %} AND pra.channel
IN (...)` line. Confirmed with the exact CI-pinned tinybird-cli==5.22.2
binary that re-running `tb fmt --yes` on that line flips between a
collapsed and expanded form forever, so no direct reformat converges.

Signed-off-by: Gašper Grom <gasper.grom@gmail.com>

Author: gaspergrom

services/libs/tinybird/pipes/active_contributors.pipe

@@ -59,8 +59,7 @@ SQL >
         {% if defined(collectionSlug) %}
             insightsProjectId IN (SELECT insightsProjectId FROM segments_filtered_by_collection)
         {% else %} insightsProjectId = (SELECT insightsProjectId FROM segments_filtered)
-        {% end %}
-        AND role = 'maintainer'
+        {% end %} AND role = 'maintainer'
         {% if defined(repos) %}
             AND repoUrl
             IN {{ Array(repos, 'String', description="Filter maintainer repo list", required=False) }}

services/libs/tinybird/pipes/collection_insights_aggregate.pipe

@@ -10,10 +10,10 @@ DESCRIPTION >
     - `projectCount`: count of distinct projects in the collection.
     - `uniqueContributorCount`: total unique contributors across all projects in the collection, deduplicated (a contributor active on multiple projects in the collection is counted once).
     - `avgHealthScore`: average health score across the collection's onboarded projects, rounded. A project
-      is "onboarded" when it has at least one contributor or organization attributed to it (same definition
-      used by insightsProjects_filtered.pipe's `onboarded` filter and the frontend's project-row `isOnboarded`
-      check) — projects with no attributed contributors/organizations have no meaningful health signal yet
-      and would otherwise pull the average down artificially.
+    is "onboarded" when it has at least one contributor or organization attributed to it (same definition
+    used by insightsProjects_filtered.pipe's `onboarded` filter and the frontend's project-row `isOnboarded`
+    check) — projects with no attributed contributors/organizations have no meaningful health signal yet
+    and would otherwise pull the average down artificially.
 
 TAGS "Insights, Widget", "Collection"
 

services/libs/tinybird/pipes/contributors_leaderboard.pipe

@@ -40,8 +40,7 @@ SQL >
 NODE is_request_from_non_lf_project
 SQL >
     %
-    {% if defined(collectionSlug) %}
-        SELECT false AS result
+    {% if defined(collectionSlug) %} SELECT false AS result
     {% else %}
         SELECT not isLF AS result
         FROM insightsProjects final

services/libs/tinybird/pipes/pull_requests_filtered.pipe

@@ -19,8 +19,8 @@ SQL >
     SELECT *
     FROM pull_requests_analyzed pra
     WHERE
+        pra.channel IN (SELECT channel FROM repos_to_channels)
         {% if defined(collectionSlug) %}
-            pra.segmentId IN (SELECT segmentId FROM segments_filtered_by_collection)
-        {% else %} pra.segmentId = (SELECT segmentId FROM segments_filtered)
+            AND pra.segmentId IN (SELECT segmentId FROM segments_filtered_by_collection)
+        {% else %} AND pra.segmentId = (SELECT segmentId FROM segments_filtered)
         {% end %}
-        AND pra.channel IN (SELECT channel FROM repos_to_channels)

services/libs/tinybird/pipes/search_volume.pipe

@@ -23,8 +23,7 @@ SQL >
     FROM searchVolume FINAL
     WHERE
         {% if defined(collectionSlug) %}
-            insightsProjectId
-            IN (select toUUID(insightsProjectId) from segments_filtered_by_collection)
+            insightsProjectId IN (select toUUID(insightsProjectId) from segments_filtered_by_collection)
         {% else %} insightsProjectId = toUUID((select insightsProjectId from segments_filtered))
         {% end %}
         {% if defined(startDate) %}