@agentikos/omega-os 0.19.34 → 0.19.35

package/omega/Agentik_Engine/omega_engine/__init__.py

@@ -188,7 +188,7 @@ from omega_engine.genesis import (
 )
 from omega_engine import plan as plan_v7
 
-__version__ = "0.19.34"
+__version__ = "0.19.35"
 
 __all__ = [
     "__version__",

package/omega/Agentik_Engine/omega_engine/cli.py

@@ -3147,28 +3147,14 @@ def cmd_menu_whiptail(_args: argparse.Namespace) -> int:
     return run_menu()
 
 
-# Default LLM provider for chat sessions — recorded in a tiny state file
-# so spawn_*_chat picks the right one. Hot-swap via `omega switch <id>`.
-_PROVIDER_MARKER = "active-llm-provider"
-
-
-def _provider_state_path() -> "Path":
-    from pathlib import Path
-    home = _omega_home()
-    return home / "Agentik_Extra" / "var" / _PROVIDER_MARKER
-
-
-def _active_provider() -> str:
-    p = _provider_state_path()
-    if p.exists():
-        return p.read_text().strip() or "claude_code"
-    return "claude_code"
-
-
-def _set_active_provider(provider_id: str) -> None:
-    p = _provider_state_path()
-    p.parent.mkdir(parents=True, exist_ok=True)
-    p.write_text(provider_id.strip() + "\n")
+# Active LLM provider — now lives in omega_engine.provider_state so the
+# Textual TUI can read it without a circular cli↔tui import (v0.19.35 fix).
+from omega_engine.provider_state import (
+    active_provider as _active_provider,
+    set_active_provider as _set_active_provider,
+    provider_state_path as _provider_state_path,
+    PROVIDER_MARKER as _PROVIDER_MARKER,
+)
 
 
 def cmd_paperclip(args: argparse.Namespace) -> int:

package/omega/Agentik_Engine/omega_engine/provider_state.py

@@ -0,0 +1,42 @@
+"""Active-LLM-provider marker — shared between cli.py and tui.py.
+
+The provider state lives in a tiny file at
+``$OMEGA_HOME/Agentik_Extra/var/active-llm-provider``. `omega switch
+<id>` writes here; the TUI and chat-spawn helpers read here.
+
+Previously these helpers lived only in cli.py — but tui.py also needs
+to read the active provider (to render `Switch LLM [current: claude_code]`
+in the menu). Importing from cli would be circular (cli already imports
+tui), so the helpers live here in their own neutral module.
+"""
+from __future__ import annotations
+
+import os
+from pathlib import Path
+
+
+PROVIDER_MARKER = "active-llm-provider"
+DEFAULT_PROVIDER = "claude_code"
+
+
+def _omega_home() -> Path:
+    return Path(os.environ.get("OMEGA_HOME", str(Path.home() / "Omega")))
+
+
+def provider_state_path() -> Path:
+    return _omega_home() / "Agentik_Extra" / "var" / PROVIDER_MARKER
+
+
+def active_provider() -> str:
+    """Read the active provider id, falling back to claude_code."""
+    p = provider_state_path()
+    if p.exists():
+        return p.read_text().strip() or DEFAULT_PROVIDER
+    return DEFAULT_PROVIDER
+
+
+def set_active_provider(provider_id: str) -> None:
+    """Persist the active provider id atomically."""
+    p = provider_state_path()
+    p.parent.mkdir(parents=True, exist_ok=True)
+    p.write_text(provider_id.strip() + "\n")

package/omega/Agentik_Engine/omega_engine/tui.py

@@ -456,6 +456,12 @@ def _arrow_menu() -> int:
     def _section(name: str) -> str:
         return f"{ORANGE}{BOLD}── {name} ──{RST}"
 
+    # `_active_provider` lived in cli.py until v0.19.34 — calling it from
+    # here NameError'd at runtime because cli isn't imported here (would
+    # be circular: cli already imports tui). Now lives in a neutral
+    # `omega_engine.provider_state` module both can read.
+    from omega_engine.provider_state import active_provider as _active_provider
+
     def _build_items() -> list[tuple[str, str]]:
         """Return (display, action_key) — section headers have key '__sep__'."""
         provider = _active_provider()

package/omega/Agentik_Engine/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "omega-engine"
-version = "0.19.34"
+version = "0.19.35"
 description = "The Omega OS orchestration engine — event-sourced, verified-completion agent graphs."
 readme = "README.md"
 requires-python = ">=3.11"

package/omega/Agentik_Engine/tests/test_tui_runtime.py

@@ -0,0 +1,104 @@
+"""Regression tests for omega_engine.tui — specifically the NameError
+that escaped pure-import tests in v0.19.34.
+
+v0.19.34's `_arrow_menu` referenced `_active_provider` (defined only in
+cli.py) which failed with NameError when actually invoked, even though
+`import omega_engine.tui` succeeded. Lesson: import-only smoke tests
+don't catch undefined-name errors inside function bodies. These tests
+exercise the function bodies enough to catch the same class of bug.
+"""
+from __future__ import annotations
+
+import inspect
+import os
+import tempfile
+import unittest
+from pathlib import Path
+
+
+class TestTuiBodyResolves(unittest.TestCase):
+    """Compile + exec the bodies of TUI functions to surface undefined
+    names. Pytest's normal collection only imports modules — it never
+    runs `_arrow_menu` itself."""
+
+    def test_arrow_menu_body_compiles(self):
+        from omega_engine.tui import _arrow_menu
+        src = inspect.getsource(_arrow_menu)
+        # If a referenced helper isn't importable from inside the
+        # function, this raises (catches v0.19.34's NameError class).
+        compile(src, "<test>", "exec")
+
+    def test_plain_repl_body_compiles(self):
+        from omega_engine.tui import _plain_repl
+        src = inspect.getsource(_plain_repl)
+        compile(src, "<test>", "exec")
+
+    def test_dispatch_slash_for_parse_only_commands(self):
+        """Every documented slash command that doesn't shell out to the
+        omega binary must produce a CommandResult (no NameError, no
+        AttributeError). Commands that shell out (`/audit`, `/switch`,
+        `/doctor`, etc.) need the omega binary on disk — those are
+        smoke-tested separately on real installs."""
+        from omega_engine.tui import dispatch_slash
+        # These never shell out — pure dispatch.
+        cmds = [
+            "/help", "/quit", "/detach", "/unknown_xyz_command",
+            "/audit",       # no args → static text only, no shell-out
+        ]
+        for c in cmds:
+            r = dispatch_slash(c)
+            self.assertIsNotNone(r, f"dispatch_slash({c!r}) returned None")
+            self.assertTrue(
+                r.text or r.exit_tui or r.detach,
+                f"dispatch_slash({c!r}) returned empty result",
+            )
+
+
+class TestProviderStateSharedModule(unittest.TestCase):
+    """The fix for v0.19.34's NameError was to extract the provider
+    state helpers into omega_engine.provider_state. Lock that in."""
+
+    def test_provider_state_module_exposes_helpers(self):
+        from omega_engine import provider_state as PS
+        for name in (
+            "active_provider", "set_active_provider",
+            "provider_state_path", "PROVIDER_MARKER", "DEFAULT_PROVIDER",
+        ):
+            self.assertTrue(hasattr(PS, name),
+                f"provider_state.{name} missing — TUI + CLI rely on it")
+
+    def test_get_set_roundtrip(self):
+        from omega_engine.provider_state import (
+            active_provider, set_active_provider,
+        )
+        with tempfile.TemporaryDirectory() as tmp:
+            old_home = os.environ.get("OMEGA_HOME")
+            os.environ["OMEGA_HOME"] = tmp
+            try:
+                # Default before any set
+                self.assertEqual(active_provider(), "claude_code")
+                # Set + read
+                set_active_provider("gemini_cli")
+                self.assertEqual(active_provider(), "gemini_cli")
+                # Whitespace stripped
+                set_active_provider("  codex  ")
+                self.assertEqual(active_provider(), "codex")
+            finally:
+                if old_home is None:
+                    os.environ.pop("OMEGA_HOME", None)
+                else:
+                    os.environ["OMEGA_HOME"] = old_home
+
+    def test_cli_imports_from_provider_state(self):
+        """The cli.py local `_active_provider` must be re-exported from
+        provider_state — not a divergent definition (the bug was that
+        cli.py owned the definition and tui.py couldn't see it)."""
+        from omega_engine import cli, provider_state
+        self.assertIs(cli._active_provider, provider_state.active_provider,
+            "cli._active_provider must alias provider_state.active_provider")
+        self.assertIs(cli._set_active_provider,
+                      provider_state.set_active_provider)
+
+
+if __name__ == "__main__":
+    unittest.main()

package/omega/Agentik_SSOT/VERSION

@@ -1 +1 @@
-0.19.34
+0.19.35

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentikos/omega-os",
-  "version": "0.19.34",
+  "version": "0.19.35",
   "description": "Omega OS — installable agentic operating system with verified-completion orchestration. Event-sourced engine, 8-block rack, autonomous agents, MCP.",
   "bin": {
     "omega-os": "bin/omega-os.js"