fix(gitlab[_warn_truncation]) Qualify truncation warning when client filters active

why: _warn_truncation shows "Showing N of M" where M comes from the x-total
header (unfiltered server count). When --skip-group/--language/--topics are
active, M includes excluded projects, misleading users about how many repos
they actually care about.  The GitLab API provides no server-side equivalent
for skip_groups; client-side filtering is unavoidable.
what:
- Add has_client_filters kwarg to _warn_truncation
- When True: warn "N total on server (note: server total includes filtered)"
- When False: existing "Showing N of M" message unchanged
- Pass has_client_filters=bool(options.skip_groups or options.language or options.topics)
  from both _fetch_search and _paginate_repos
- Remove TODO comment
- Add test_gitlab_truncation_warning_with_client_filter in test_gitlab.py

Author: tony

src/vcspull/_internal/remotes/gitlab.py

@@ -241,7 +241,15 @@ def _fetch_search(self, options: ImportOptions) -> t.Iterator[RemoteRepo]:
             page += 1
 
         # Warn if results were truncated by --limit
-        self._warn_truncation(count, options.limit, total_available, last_x_next_page)
+        self._warn_truncation(
+            count,
+            options.limit,
+            total_available,
+            last_x_next_page,
+            has_client_filters=bool(
+                options.skip_groups or options.language or options.topics
+            ),
+        )
 
     def _paginate_repos(
         self,
@@ -325,43 +333,63 @@ def _paginate_repos(
             page += 1
 
         # Warn if results were truncated by --limit
-        self._warn_truncation(count, options.limit, total_available, last_x_next_page)
+        self._warn_truncation(
+            count,
+            options.limit,
+            total_available,
+            last_x_next_page,
+            has_client_filters=bool(
+                options.skip_groups or options.language or options.topics
+            ),
+        )
 
     def _warn_truncation(
         self,
         count: int,
         limit: int,
         total_available: int | None,
         x_next_page: str | None,
+        *,
+        has_client_filters: bool = False,
     ) -> None:
         """Warn if results were truncated by the --limit option.
 
         Parameters
         ----------
         count : int
-            Number of repositories actually returned
+            Number of repositories returned after client-side filtering
         limit : int
             The configured limit
         total_available : int | None
-            Value of x-total header (None if absent)
+            Value of x-total header (None if absent). Reflects the unfiltered
+            server-side total; may overstate the count of repos matching
+            client-side filters (skip_groups, language, topics).
         x_next_page : str | None
             Value of x-next-page header (None if absent/empty)
+        has_client_filters : bool
+            True when any client-side filter (skip_groups, language, topics) is
+            active; used to qualify the warning message so the server-side
+            total is not presented as the filtered total.
         """
         if count < limit:
             return
 
-        # TODO: total_available comes from the x-total header which reflects
-        # the unfiltered server-side project count.  After client-side
-        # filter_repo filtering (e.g. skip_groups, language, topics), count
-        # may be less than total_available for reasons unrelated to --limit.
-        # A more accurate message would require tracking pre-filter vs
-        # post-filter counts separately.
         if total_available is not None and total_available > count:
-            log.warning(
-                "Showing %d of %d repositories (use --limit 0 to fetch all)",
-                count,
-                total_available,
-            )
+            if has_client_filters:
+                log.warning(
+                    "Showing %d repositories; %d total on server "
+                    "(note: server total includes projects excluded by "
+                    "--skip-group / --language / --topics; "
+                    "use --limit 0 to fetch all matching your filters)",
+                    count,
+                    total_available,
+                )
+            else:
+                log.warning(
+                    "Showing %d of %d repositories (use --limit 0 to fetch all)",
+                    count,
+                    total_available,
+                )
         elif x_next_page is not None:
             log.warning(
                 "Showing %d repositories; more are available "

tests/_internal/remotes/test_gitlab.py

@@ -858,6 +858,36 @@ def urlopen_side_effect(
         assert "--limit" not in caplog.text.lower()
 
 
+def test_gitlab_truncation_warning_with_client_filter(
+    monkeypatch: pytest.MonkeyPatch,
+    caplog: pytest.LogCaptureFixture,
+) -> None:
+    """Truncation warning with client filters active uses qualified message."""
+    import logging
+
+    caplog.set_level(logging.WARNING)
+
+    repos = [_make_gitlab_project(i) for i in range(5)]
+    headers = {"x-total": "5", "x-next-page": "2"}
+
+    def urlopen_side_effect(
+        request: urllib.request.Request,
+        timeout: int | None = None,
+    ) -> MockHTTPResponse:
+        return MockHTTPResponse(json.dumps(repos).encode(), headers, 200)
+
+    # Mock urlopen: return all 5 repos with x-total=5 to trigger truncation path
+    monkeypatch.setattr("urllib.request.urlopen", urlopen_side_effect)
+
+    importer = GitLabImporter()
+    options = ImportOptions(
+        mode=ImportMode.USER, target="user", limit=2, skip_groups=["bots"]
+    )
+    list(importer.fetch_repos(options))
+
+    assert "server total includes projects excluded" in caplog.text
+
+
 # ---------------------------------------------------------------------------
 # with_shared URL parameter tests
 # ---------------------------------------------------------------------------