updating wrapper app.py

app.py

@@ -1,8 +1,7 @@
 """
 Root app.py – HuggingFace Spaces entry point.
 
-Presents a landing page that lets users pick between the two dashboards,
-then dynamically loads the chosen sub-app without touching its source code.
+Delegates directly to visualization/app.py.
 
 Run locally :  streamlit run app.py
 HF Spaces   :  set "app_file: app.py" in README.md front-matter (default)
@@ -14,136 +13,37 @@ from pathlib import Path
 
 import streamlit as st
 
-# ── Paths ──────────────────────────────────────────────────────────────────────
 ROOT = Path(__file__).resolve().parent
-
-_SUB_APPS: dict[str, Path] = {
-    "sentiment": ROOT / "visualization",
-    "brand":     ROOT / "visualization_brand_sentiment",
-}
-
-# Module-name prefixes that belong to the sub-apps and must be evicted from
-# sys.modules when switching dashboards (both apps share identical package names:
-# data, utils, components, agents, visualizations).
-_SUB_APP_MODULE_PREFIXES = (
-    "data", "utils", "components", "agents", "visualizations", "_subapp",
-)
+APP_DIR = ROOT / "visualization"
 
 # ── Page config (must be the very first Streamlit call) ────────────────────────
 st.set_page_config(
-    page_title="Musora Analytics Suite",
+    page_title="Musora Sentiment Analysis",
     page_icon="📊",
     layout="wide",
     initial_sidebar_state="expanded",
 )
 
-# After we've set the page config, prevent sub-apps from overriding it.
-# Both sub-apps call st.set_page_config() at module level; patching the
-# function on the streamlit module object makes those calls no-ops.
-import streamlit as _st_mod          # same object as `st`
+# Prevent visualization/app.py's module-level st.set_page_config() call from
+# raising an error – patch it to a no-op on the shared streamlit module object.
+import streamlit as _st_mod
 _st_mod.set_page_config = lambda *a, **kw: None
 
-
-# ── Helpers ────────────────────────────────────────────────────────────────────
-
-def _activate_app_path(app_key: str) -> None:
-    """
-    Swap out the active sub-app path in sys.path and clear any stale
-    module cache entries so both apps' identically-named packages resolve
-    to the correct source tree.
-    """
-    # Remove every sub-app directory that may already be on the path.
-    for path in _SUB_APPS.values():
-        s = str(path)
-        while s in sys.path:
-            sys.path.remove(s)
-
-    # Insert the target sub-app directory at position 0 (highest priority).
-    sys.path.insert(0, str(_SUB_APPS[app_key]))
-
-    # Evict cached modules that originated from any sub-app so that the
-    # next import resolves from the newly active path.
-    for mod_name in list(sys.modules.keys()):
-        if any(
-            mod_name == prefix or mod_name.startswith(prefix + ".")
-            for prefix in _SUB_APP_MODULE_PREFIXES
-        ):
-            del sys.modules[mod_name]
-
-
-def _load_and_run(app_key: str) -> None:
-    """
-    Dynamically load the selected sub-app module and invoke its main().
-
-    Notes
-    -----
-    * exec_module() runs the module's top-level code (config loading,
-      auth check, CSS injection, etc.) exactly as Streamlit would.
-    * main() is called explicitly because the sub-apps guard it with
-      ``if __name__ == "__main__"``, which is False under importlib.
-    * StopException (raised by st.stop() inside the auth gate) is
-      re-raised so Streamlit's runner can handle it correctly.
-    """
-    _activate_app_path(app_key)
-
-    app_path = _SUB_APPS[app_key] / "app.py"
-    spec = importlib.util.spec_from_file_location("_subapp", app_path)
-    mod  = importlib.util.module_from_spec(spec)
-
-    try:
-        spec.loader.exec_module(mod)        # runs module-level code (auth, CSS …)
-        if hasattr(mod, "main"):
-            mod.main()                      # renders the full dashboard
-    except Exception as exc:               # noqa: BLE001
-        # Re-raise Streamlit's internal StopException (triggered by st.stop())
-        # so the runner knows to halt rendering cleanly.
-        if "Stop" in type(exc).__name__:
-            raise
-        st.error(f"Dashboard error: {exc}")
-        st.exception(exc)
-
-
-# ── Routing ────────────────────────────────────────────────────────────────────
-
-choice: str | None = st.session_state.get("app_choice")
-
-if choice is None:
-    # ── Landing page ───────────────────────────────────────────────────────────
-    st.title("📊 Musora Analytics Suite")
-    st.markdown("Select a dashboard to get started.")
-    st.markdown("---")
-
-    col1, col2 = st.columns(2, gap="large")
-
-    with col1:
-        st.subheader("Musora Sentiment Dashboard")
-        st.markdown(
-            "Analyse social-media comment sentiment across YouTube, "
-            "Facebook, and other platforms. Includes reply-required "
-            "triage and global date/platform filters."
-        )
-        if st.button("Open →", key="btn_sentiment", use_container_width=True):
-            st.session_state.app_choice = "sentiment"
-            st.rerun()
-
-    with col2:
-        st.subheader("Sabian Brand Sentiment")
-        st.markdown(
-            "Track brand sentiment for Sabian products across Musora "
-            "forums and YouTube. Covers competitor mentions, purchase "
-            "intent, and demographic breakdowns."
-        )
-        if st.button("Open →", key="btn_brand", use_container_width=True):
-            st.session_state.app_choice = "brand"
-            st.rerun()
-
-else:
-    # ── Sub-app: inject a "back" button at the top of the sidebar ─────────────
-    with st.sidebar:
-        if st.button("← Home", key="back_home", use_container_width=True):
-            st.session_state.app_choice = None
-            st.rerun()
-        st.markdown("---")
-
-    # ── Load and render the chosen dashboard ───────────────────────────────────
-    _load_and_run(choice)
+# ── Ensure sub-app packages are importable ─────────────────────────────────────
+app_dir_str = str(APP_DIR)
+if app_dir_str not in sys.path:
+    sys.path.insert(0, app_dir_str)
+
+# ── Load and run visualization/app.py ─────────────────────────────────────────
+spec = importlib.util.spec_from_file_location("_subapp", APP_DIR / "app.py")
+mod  = importlib.util.module_from_spec(spec)
+
+try:
+    spec.loader.exec_module(mod)   # runs module-level code (config, auth, etc.)
+    if hasattr(mod, "main"):
+        mod.main()                 # renders the full dashboard
+except Exception as exc:
+    if "Stop" in type(exc).__name__:   # re-raise st.stop() for Streamlit's runner
+        raise
+    st.error(f"Dashboard error: {exc}")
+    st.exception(exc)