Address review feedback on public type aliases

- Delete duplicate FilePath/FileObjectW/PathOrFileW in endpoint.py; import
  from tableauserverclient.types (the whole point of the consolidation)
- Remove FileObject, PathOrFile, IDPAttributes, IDPProperty from __all__:
  FileObject/PathOrFile are wider than any runtime path accepts; the two
  Protocol classes are structural-only and users get HasIdpConfigurationID
  as the caller-facing type
- Add CHANGELOG entry for the newly-stable public typing surface
- Add smoke test guarding the top-level exports and AddResponse._fields

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: jacalata

CHANGELOG.md

@@ -5,6 +5,12 @@
   hierarchy path (e.g. `"Marketing/Q1 Reports"`). The walk is performed level by
   level using the REST API name filter, so a path with *n* components issues *n*
   requests. Returns the matching `ProjectItem` or `None` if no project is found.
+* Consolidated public typing surface under `tableauserverclient.types` and
+  exported it at the top level. The following names are now stable public API
+  for downstream type checkers: `HyperAction`, `HyperActionCondition`,
+  `HyperActionRow`, `HyperActionTable`, `FilePath`, `FileObjectR`,
+  `FileObjectW`, `PathOrFileR`, `PathOrFileW`, `AddResponse`,
+  `HasIdpConfigurationID`.
 
 ## 0.18.0 (6 April 2022)    
 * Switched to using defused_xml for xml attack protection

tableauserverclient/__init__.py

@@ -61,17 +61,13 @@
 from tableauserverclient.types import (
     AddResponse,
     FilePath,
-    FileObject,
     FileObjectR,
     FileObjectW,
     HasIdpConfigurationID,
     HyperAction,
     HyperActionCondition,
     HyperActionRow,
     HyperActionTable,
-    IDPAttributes,
-    IDPProperty,
-    PathOrFile,
     PathOrFileR,
     PathOrFileW,
 )
@@ -116,7 +112,6 @@
     "FavoriteItem",
     "FileuploadItem",
     "FilePath",
-    "FileObject",
     "FileObjectR",
     "FileObjectW",
     "Filter",
@@ -131,8 +126,6 @@
     "HyperActionCondition",
     "HyperActionRow",
     "HyperActionTable",
-    "IDPAttributes",
-    "IDPProperty",
     "ImageRequestOptions",
     "IntervalItem",
     "JobItem",
@@ -152,7 +145,6 @@
     "Permission",
     "PermissionsRule",
     "PersonalAccessTokenAuth",
-    "PathOrFile",
     "PathOrFileR",
     "PathOrFileW",
     "ProjectItem",

tableauserverclient/server/endpoint/endpoint.py

@@ -37,6 +37,7 @@
 from tableauserverclient import helpers, get_versions
 
 from tableauserverclient.helpers.logging import logger
+from tableauserverclient.types import PathOrFileW
 
 if TYPE_CHECKING:
     from tableauserverclient.server.server import Server
@@ -330,10 +331,6 @@ def wrapper(self: E, *args: P.args, **kwargs: P.kwargs) -> R:
 
 _io_types_w = (io.BytesIO, io.BufferedWriter)
 
-FilePath = str | os.PathLike
-FileObjectW = io.BufferedWriter | io.BytesIO
-PathOrFileW = FilePath | FileObjectW
-
 
 class DownloadableMixin:
     """Mixin for endpoints whose resources can be downloaded as binary files.

test/test_types_exports.py

@@ -0,0 +1,26 @@
+"""Guard that types.py public exports don't silently regress."""
+
+import tableauserverclient as TSC
+
+
+def test_types_are_top_level_importable():
+    for name in (
+        "HyperAction",
+        "HyperActionCondition",
+        "HyperActionRow",
+        "HyperActionTable",
+        "FilePath",
+        "FileObjectR",
+        "FileObjectW",
+        "PathOrFileR",
+        "PathOrFileW",
+        "AddResponse",
+        "HasIdpConfigurationID",
+    ):
+        assert hasattr(TSC, name), f"missing {name}"
+        assert name in TSC.__all__, f"{name} not in __all__"
+
+
+def test_add_response_shape():
+    # namedtuple contract that samples/users depend on
+    assert TSC.AddResponse._fields == ("result", "error", "warnings", "task_created")