streamlit 1.53.0__py3-none-any.whl → 1.54.0__py3-none-any.whl

streamlit/runtime/caching/cache_utils.py

@@ -592,10 +592,10 @@ def _get_positional_arg_name(func: Callable[..., Any], arg_index: int) -> str |
     if arg_index >= len(params):
         return None
 
-    if params[arg_index].kind in (
+    if params[arg_index].kind in {
         inspect.Parameter.POSITIONAL_OR_KEYWORD,
         inspect.Parameter.POSITIONAL_ONLY,
-    ):
+    }:
         return params[arg_index].name
 
     return None

streamlit/runtime/caching/hashing.py

@@ -25,7 +25,7 @@ import hashlib
 import inspect
 import io
 import os
-import pickle
+import pickle  # noqa: S403
 import sys
 import tempfile
 import threading
@@ -653,5 +653,3 @@ class _CacheFuncHasher:
 
 class NoResult:
     """Placeholder class for return values when None is meaningful."""
-
-    pass

streamlit/runtime/caching/storage/cache_storage_protocol.py

@@ -156,7 +156,6 @@ class CacheStorage(Protocol):
         to close open resources, before we delete the storage instance.
         e.g. close the database connection etc.
         """
-        pass
 
 
 class CacheStorageManager(Protocol):
@@ -235,5 +234,3 @@ class CacheStorageManager(Protocol):
         -----
         Threading: Should be safe to call from any thread.
         """
-
-        pass

streamlit/runtime/connection_factory.py

@@ -99,7 +99,7 @@ def _create_connection(
     )
 
     scope = connection_class.scope()
-    if scope not in ("global", "session"):
+    if scope not in {"global", "session"}:
         raise StreamlitAPIException(
             f"Connection class {connection_class} has scope '{scope}'. Valid values "
             "are 'global' or 'session'."

streamlit/runtime/credentials.py

@@ -148,7 +148,7 @@ class Credentials:
         import toml
 
         try:
-            with open(self._conf_file) as f:
+            with open(self._conf_file, encoding="utf-8") as f:
                 data = toml.load(f).get("general")
             if data is None:
                 raise RuntimeError  # noqa: TRY301
@@ -218,7 +218,7 @@ class Credentials:
 
         import toml
 
-        with open(self._conf_file, "w") as f:
+        with open(self._conf_file, "w", encoding="utf-8") as f:
             toml.dump({"general": data}, f)
 
         try:

streamlit/runtime/metrics_util.py

@@ -251,11 +251,11 @@ def _get_machine_id_v3() -> str:
     """
 
     if os.path.isfile(_ETC_MACHINE_ID_PATH):
-        with open(_ETC_MACHINE_ID_PATH) as f:
+        with open(_ETC_MACHINE_ID_PATH, encoding="utf-8") as f:
             machine_id = f.read()
 
     elif os.path.isfile(_DBUS_MACHINE_ID_PATH):
-        with open(_DBUS_MACHINE_ID_PATH) as f:
+        with open(_DBUS_MACHINE_ID_PATH, encoding="utf-8") as f:
             machine_id = f.read()
 
     else:
@@ -380,7 +380,7 @@ def _get_command_telemetry(
         pos = i
         if is_method:
             # If func is a method, ignore the first argument (self)
-            i = i + 1  # noqa: PLW2901
+            i += 1  # noqa: PLW2901
 
         keyword = arg_keywords[i] if len(arg_keywords) > i else f"{i}"
         if keyword == "self":

streamlit/runtime/runtime.py

@@ -345,7 +345,7 @@ class Runtime:
         async_objs = self._get_async_objs()
 
         def stop_on_eventloop() -> None:
-            if self._state in (RuntimeState.STOPPING, RuntimeState.STOPPED):
+            if self._state in {RuntimeState.STOPPING, RuntimeState.STOPPED}:
                 return
 
             _LOGGER.debug("Runtime stopping...")
@@ -411,7 +411,7 @@ class Runtime:
                 "This should never happen."
             )
 
-        if self._state in (RuntimeState.STOPPING, RuntimeState.STOPPED):
+        if self._state in {RuntimeState.STOPPING, RuntimeState.STOPPED}:
             raise RuntimeStoppedError(f"Can't connect_session (state={self._state})")
 
         session_id = self._session_mgr.connect_session(
@@ -510,7 +510,7 @@ class Runtime:
         -----
         Threading: UNSAFE. Must be called on the eventloop thread.
         """
-        if self._state in (RuntimeState.STOPPING, RuntimeState.STOPPED):
+        if self._state in {RuntimeState.STOPPING, RuntimeState.STOPPED}:
             raise RuntimeStoppedError(f"Can't handle_backmsg (state={self._state})")
 
         session_info = self._session_mgr.get_active_session_info(session_id)
@@ -538,7 +538,7 @@ class Runtime:
         -----
         Threading: UNSAFE. Must be called on the eventloop thread.
         """
-        if self._state in (RuntimeState.STOPPING, RuntimeState.STOPPED):
+        if self._state in {RuntimeState.STOPPING, RuntimeState.STOPPED}:
             raise RuntimeStoppedError(
                 f"Can't handle_backmsg_deserialization_exception (state={self._state})"
             )
@@ -555,11 +555,11 @@ class Runtime:
 
     @property
     async def is_ready_for_browser_connection(self) -> tuple[bool, str]:
-        if self._state not in (
+        if self._state not in {
             RuntimeState.INITIAL,
             RuntimeState.STOPPING,
             RuntimeState.STOPPED,
-        ):
+        }:
             return True, "ok"
 
         return False, "unavailable"

streamlit/runtime/scriptrunner/script_runner.py

@@ -528,6 +528,23 @@ class ScriptRunner:
                     and rerun_data.widget_states.widgets is not None
                 ):
                     widget_ids = {w.id for w in rerun_data.widget_states.widgets}
+
+                # For MPA page transitions: filter query params BEFORE cleanup.
+                # This uses existing bindings to remove params from other pages,
+                # ensuring st.query_params is accurate when the new page runs.
+                # (st.query_params in user code will reflect the correct params for the new page)
+                main_script_hash = self._pages_manager.main_script_hash
+                valid_script_hashes = {main_script_hash, page_script_hash}
+                with self._session_state.query_params() as qp:
+                    qp.populate_from_query_string(
+                        rerun_data.query_string, valid_script_hashes
+                    )
+                    # Set initial params from FILTERED state for widget seeding.
+                    # This prevents stale params from previous pages from seeding
+                    # widgets on the new page if keys collide.
+                    qp.set_initial_query_params_from_current()
+
+                # Now safe to do normal cleanup - filtering already done
                 self._session_state.on_script_finished(widget_ids)
 
             fragment_ids_this_run: list[str] | None = (

streamlit/runtime/scriptrunner_utils/exceptions.py

@@ -22,14 +22,10 @@ from streamlit.util import repr_
 class ScriptControlException(BaseException):  # NOSONAR
     """Base exception for ScriptRunner."""
 
-    pass
-
 
 class StopException(ScriptControlException):
     """Silently stop the execution of the user's script."""
 
-    pass
-
 
 class RerunException(ScriptControlException):
     """Silently stop and rerun the user's script."""

streamlit/runtime/scriptrunner_utils/script_run_context.py

@@ -24,11 +24,9 @@ from typing import (
     Final,
     TypeAlias,
 )
-from urllib import parse
 
 from streamlit.errors import (
     NoSessionContext,
-    StreamlitAPIException,
 )
 from streamlit.logger import get_logger
 from streamlit.runtime.forward_msg_cache import (
@@ -109,10 +107,6 @@ class ScriptRunContext:
     # we allow only one dialog to be open at the same time
     has_dialog_opened: bool = False
 
-    # TODO(willhuang1997): Remove this variable when experimental query params are removed
-    _experimental_query_params_used = False
-    _production_query_params_used = False
-
     @property
     def page_script_hash(self) -> str:
         return self.pages_manager.current_page_script_hash
@@ -147,6 +141,9 @@ class ScriptRunContext:
         cached_message_hashes: set[str] | None = None,
         context_info: ContextInfo | None = None,
     ) -> None:
+        # Check if this is a same-page rerun BEFORE updating page_script_hash
+        is_same_page = self.page_script_hash == page_script_hash
+
         self.cursors = {}
         self.widget_ids_this_run = set()
         self.widget_user_keys_this_run = set()
@@ -168,16 +165,17 @@ class ScriptRunContext:
 
         in_cached_function.set(False)
 
-        parsed_query_params = parse.parse_qs(query_string, keep_blank_values=True)
         with self.session_state.query_params() as qp:
-            qp.clear_with_no_forward_msg()
-            for key, val in parsed_query_params.items():
-                if len(val) == 0:
-                    qp.set_with_no_forward_msg(key, val="")
-                elif len(val) == 1:
-                    qp.set_with_no_forward_msg(key, val=val[-1])
-                else:
-                    qp.set_with_no_forward_msg(key, val)
+            # For same-page reruns (widget interactions), populate _query_params from URL
+            # and set initial params for widget seeding.
+            # For page transitions, both populate_from_query_string() AND
+            # set_initial_query_params_from_current() are called in script_runner.py
+            # BEFORE reset() to ensure filtering is applied to both _query_params
+            # AND _initial_query_params, preventing stale params from previous pages
+            # from seeding widgets on the new page.
+            if is_same_page:
+                qp.set_initial_query_params(query_string)
+                qp.populate_from_query_string(query_string)
 
     def on_script_start(self) -> None:
         self._has_script_started = True
@@ -202,22 +200,6 @@ class ScriptRunContext:
         # Pass the message up to our associated ScriptRunner.
         self._enqueue(msg_to_send)
 
-    def ensure_single_query_api_used(self) -> None:
-        if self._experimental_query_params_used and self._production_query_params_used:
-            raise StreamlitAPIException(
-                "Using `st.query_params` together with either `st.experimental_get_query_params` "
-                "or `st.experimental_set_query_params` is not supported. Please "
-                " convert your app to only use `st.query_params`"
-            )
-
-    def mark_experimental_query_params_used(self) -> None:
-        self._experimental_query_params_used = True
-        self.ensure_single_query_api_used()
-
-    def mark_production_query_params_used(self) -> None:
-        self._production_query_params_used = True
-        self.ensure_single_query_api_used()
-
 
 SCRIPT_RUN_CONTEXT_ATTR_NAME: Final = "streamlit_script_run_ctx"
 

streamlit/runtime/secrets.py

@@ -230,7 +230,6 @@ class Secrets(Mapping[str, Any]):
         """Left in place for compatibility with integrations until integration
         code can be updated.
         """
-        pass
 
     def _reset(self) -> None:
         """Clear the secrets dictionary and remove any secrets that were
@@ -308,7 +307,7 @@ class Secrets(Mapping[str, Any]):
                 if os.path.isdir(file_path):
                     continue
 
-                with open(file_path) as f:
+                with open(file_path, encoding="utf-8") as f:
                     sub_secrets[filename] = f.read().strip()
                     found_secrets_file = True
 
@@ -396,7 +395,7 @@ class Secrets(Mapping[str, Any]):
         is a string, int, or float.
         """
         value_type = type(v)
-        if value_type in (str, int, float):
+        if value_type in {str, int, float}:
             os.environ[k] = str(v)
 
     @staticmethod
@@ -405,7 +404,7 @@ class Secrets(Mapping[str, Any]):
         is a string, int, or float.
         """
         value_type = type(v)
-        if value_type in (str, int, float) and os.environ.get(k) == v:
+        if value_type in {str, int, float} and os.environ.get(k) == v:
             del os.environ[k]
 
     def _maybe_install_file_watchers(self) -> None:

streamlit/runtime/state/__init__.py

@@ -12,7 +12,12 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-from streamlit.runtime.state.common import WidgetArgs, WidgetCallback, WidgetKwargs
+from streamlit.runtime.state.common import (
+    BindOption,
+    WidgetArgs,
+    WidgetCallback,
+    WidgetKwargs,
+)
 from streamlit.runtime.state.query_params_proxy import QueryParamsProxy
 from streamlit.runtime.state.safe_session_state import SafeSessionState
 from streamlit.runtime.state.session_state import (
@@ -28,6 +33,7 @@ from streamlit.runtime.state.widgets import register_widget
 
 __all__ = [
     "SCRIPT_RUN_WITHOUT_ERRORS_KEY",
+    "BindOption",
     "QueryParamsProxy",
     "SafeSessionState",
     "SessionState",

streamlit/runtime/state/common.py

@@ -51,6 +51,10 @@ WidgetArgs: TypeAlias = tuple[Any, ...] | list[Any]
 WidgetKwargs: TypeAlias = dict[str, Any]
 WidgetCallback: TypeAlias = Callable[..., None]
 
+# Type for the bind parameter on widgets
+# Currently only supports binding to query params
+BindOption: TypeAlias = Literal["query-params"] | None
+
 # A deserializer receives the value from whatever field is set on the
 # WidgetState proto, and returns a regular python value. A serializer
 # receives a regular python value, and returns something suitable for
@@ -147,6 +151,15 @@ class WidgetMetadata(Generic[T]):
     # multiple internal widget states.
     presenter: WidgetValuePresenter | None = None
 
+    # Optional binding for the widget's value to external state (e.g. query params)
+    bind: BindOption = None
+
+    # TODO(query-params): Remove formatted_options once all selection widgets use
+    # string-based wire formats (string_value/string_array_value).
+    # Currently used for query param binding to convert indices back to human-readable
+    # option strings in URLs when auto-correcting filtered values.
+    formatted_options: list[str] | None = None
+
     def __repr__(self) -> str:
         return util.repr_(self)