fix: address Copilot review feedback on public type aliases

- Parameterize os.PathLike as os.PathLike[str] in FilePath so
  type checkers get precise element-type info instead of the bare
  generic form (Copilot review, types.py:7).
- Add @runtime_checkable to IDPAttributes and IDPProperty so that
  isinstance() checks against the re-exported public Protocol types
  do not raise TypeError. These Protocols are exported from the
  package root; making them runtime-checkable is required for them
  to be safe to use as documented public types.
- Widen FileObject from `io.BufferedReader | io.BytesIO` to the more
  idiomatic `typing.BinaryIO`. FileObject is not used as a parameter
  or return annotation anywhere in the codebase (only re-exported via
  __init__ / PathOrFile), so the widening is source-compatible for
  in-repo callers. Public consumers passing a wider binary-file-like
  object now type-check cleanly.

Skipped items: none.

Author: jacalata

tableauserverclient/types.py

@@ -1,11 +1,11 @@
 import io
 import os
 from collections import namedtuple
-from typing import Literal, Protocol, TypedDict
+from typing import BinaryIO, Literal, Protocol, TypedDict, runtime_checkable
 
 # File path and object type aliases used across publish/download methods
-FilePath = str | os.PathLike
-FileObject = io.BufferedReader | io.BytesIO
+FilePath = str | os.PathLike[str]
+FileObject = BinaryIO
 FileObjectR = io.BufferedReader | io.BytesIO
 FileObjectW = io.BufferedWriter | io.BytesIO
 PathOrFile = FilePath | FileObject
@@ -59,10 +59,12 @@
 # HasIdpConfigurationID unions both so downstream callers can implement either
 # style without hitting mypy invariance errors. Callers accept
 # `str | HasIdpConfigurationID` -- passing a raw id string or an object.
+@runtime_checkable
 class IDPAttributes(Protocol):
     idp_configuration_id: str
 
 
+@runtime_checkable
 class IDPProperty(Protocol):
     @property
     def idp_configuration_id(self) -> str: ...