fix: Revert - fix(session): Always have calling session be the parent session when making scoped session (#1924)

Author: schloerke

CHANGELOG.md

@@ -25,9 +25,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 * Fixed an issue with the `.click()` method on InputActionButton controllers in `shiny.playwright.controllers` where the method would not work as expected. (#1886)
 
-* Fixed an issue where the parent session of a nested module's session was not being set to the calling session. Now, a session who's namespace is `A-B` has a parent of `A` and grandparent of `(root)` (whereas before it would have skipped `A` and the parent would have been `(root)`). (#1923)
-
-
 ## [1.3.0] - 2025-03-03
 
 ### New features

shiny/session/_session.py

@@ -1233,9 +1233,8 @@ def is_stub_session(self) -> bool:
     async def close(self, code: int = 1001) -> None:
         await self._parent.close(code)
 
-    def make_scope(self, id: Id) -> Session:
-        ns = self.ns(id)
-        return SessionProxy(parent=self, ns=ns)
+    def make_scope(self, id: str) -> Session:
+        return self._parent.make_scope(self.ns(id))
 
     def root_scope(self) -> Session:
         res = self

tests/playwright/shiny/bookmark/modules/test_bookmark_modules.py

@@ -28,9 +28,6 @@ def test_bookmark_modules(
     mod1_key: str,
 ) -> None:
 
-    if "recursive" in app_name:
-        pytest.skip("Skip broken recursive test")
-
     # Set environment variable before the app starts
     os.environ["SHINY_BOOKMARK_STORE"] = bookmark_store