@agentikos/omega-os 0.19.34 → 0.19.36

package/bootstrap/lib/common.sh

@@ -75,12 +75,45 @@ record_state_version() {
 }
 
 # run_step <name> <function>
+#
+# In NORMAL mode: prints `:: step <name>` headers + step output flows
+# to the terminal in real time (the v0.19.34 behaviour).
+#
+# In FULL mode (INSTALL_FULL=1, set by `--full`): redirects each step's
+# stdout+stderr to $LOG_FILE and updates a single progress bar line
+# instead. No `:: step xx` chatter. Operator sees:
+#
+#   Installing…  ████████████░░░░░░░░░░  14/21  66%   45-claude-plugins
+#
+# When the run finishes, the progress bar clears and the post-install
+# card prints normally.
 run_step() {
   local name="$1" fn="$2"
   if step_done "$name" && [ "${FORCE:-0}" != "1" ]; then
+    if [ "${INSTALL_FULL:-0}" = "1" ]; then
+      _full_progress "$name" "skip"
+      return 0
+    fi
     log "${C_DIM}-- skip $name (already done)${C_RST}"
     return 0
   fi
+
+  if [ "${INSTALL_FULL:-0}" = "1" ]; then
+    # Redirect step output to the log; only the progress bar stays on screen.
+    _full_progress "$name" "run"
+    if "$fn" >>"$LOG_FILE" 2>&1; then
+      mark_done "$name"
+      _full_progress "$name" "ok"
+    else
+      # Clear the bar then surface the error inline.
+      printf "\r\033[K"
+      err "step $name failed — see $LOG_FILE"
+      die "step $name failed — fix the cause and re-run install.sh"
+    fi
+    return 0
+  fi
+
+  # Normal verbose mode (default).
   info "step $name"
   if "$fn"; then
     mark_done "$name"
@@ -90,6 +123,38 @@ run_step() {
   fi
 }
 
+# _full_progress <step_name> <phase=run|ok|skip>
+#   Renders one line:  "Installing…  <bar>  N/M  %  <current step>"
+#   Uses \r to overwrite — no scrollback noise.
+_full_progress() {
+  local step="$1" phase="$2"
+  STEP_COUNT=$((${STEP_COUNT:-0} + 1))
+  local total="${STEP_TOTAL:-21}"
+  local current="$STEP_COUNT"
+  # Decrement displayed count for "skip" so the bar represents real progress
+  if [ "$phase" = "skip" ]; then
+    : # leave count as-is; skips still count as progress
+  fi
+  local pct=$((current * 100 / total))
+  local bar_width=30
+  local filled=$((current * bar_width / total))
+  local empty=$((bar_width - filled))
+  local bar=""
+  if [ "$filled" -gt 0 ]; then
+    bar="$(printf '█%.0s' $(seq 1 "$filled"))"
+  fi
+  if [ "$empty" -gt 0 ]; then
+    bar="${bar}$(printf '░%.0s' $(seq 1 "$empty"))"
+  fi
+  # \033[K clears to end of line so the previous step name doesn't bleed.
+  printf "\r  ${C_ORANGE}Installing…${C_RST}  ${bar}  ${C_BOLD}%2d/%d${C_RST}  %3d%%  ${C_DIM}%-32s${C_RST}\033[K" \
+    "$current" "$total" "$pct" "$step"
+  # Final newline only when we hit the last step.
+  if [ "$current" -ge "$total" ]; then
+    printf "\n"
+  fi
+}
+
 # --- user resolution ---------------------------------------------------------
 #
 # The installer is modular about WHICH user it runs as. Three modes, decided

package/install.sh

@@ -25,6 +25,7 @@ source "$REPO_DIR/bootstrap/lib/steps.sh"
 PROFILE="vps"; MANIFEST=""; NONINTERACTIVE=0; FORCE=0
 USER_REQUESTED=""; CREATE_USER=0; CREATE_USER_NAME=""
 PROFILE_EXPLICIT=0; NO_PATH_SETUP=0; DRY_RUN=0
+INSTALL_FULL=0; FULL_WANT_TELEGRAM=""
 [ "$(uname -s)" = "Darwin" ] && PROFILE="workstation"
 
 while [ $# -gt 0 ]; do
@@ -41,6 +42,14 @@ while [ $# -gt 0 ]; do
     --force)           FORCE=1; shift ;;
     --no-path-setup)   NO_PATH_SETUP=1; shift ;;
     --dry-run)         DRY_RUN=1; shift ;;
+    # `--full` — silent full-recommended install. One prompt
+    # (Telegram yes/no), then progress bar only. Selects every
+    # recommended LLM CLI / MCP / Claude plugin / system CLI.
+    --full)            INSTALL_FULL=1; NONINTERACTIVE=1; shift ;;
+    --full-no-telegram) INSTALL_FULL=1; NONINTERACTIVE=1
+                        FULL_WANT_TELEGRAM=no; shift ;;
+    --full-with-telegram) INSTALL_FULL=1; NONINTERACTIVE=1
+                          FULL_WANT_TELEGRAM=yes; shift ;;
     --user)            USER_REQUESTED="${2:?}"; shift 2 ;;
     --create-user)
       CREATE_USER=1
@@ -123,15 +132,41 @@ fi
 
 # ───── UX header ─────
 install_banner
-preflight_card
+# In --full mode skip the heavy preflight card; we'll show the progress
+# bar instead. Banner still prints so the user sees "Omega OS vX.Y.Z".
+if [ "${INSTALL_FULL:-0}" != "1" ]; then
+  preflight_card
+fi
 
 # Interactive profile picker if the operator did not pass --profile.
-pick_profile_interactive
+# (Skipped in --full mode — uses the default profile for the OS.)
+if [ "${INSTALL_FULL:-0}" != "1" ]; then
+  pick_profile_interactive
+fi
 
 # Final pre-run summary (one tight line so the operator knows what they
-# confirmed).
-log "${C_DIM}::${C_RST} starting install — profile=${C_BOLD}$PROFILE${C_RST}  os=$OMEGA_OS  user=$OMEGA_USER  home=$OMEGA_HOME"
-echo
+# confirmed). Skipped in --full mode.
+if [ "${INSTALL_FULL:-0}" != "1" ]; then
+  log "${C_DIM}::${C_RST} starting install — profile=${C_BOLD}$PROFILE${C_RST}  os=$OMEGA_OS  user=$OMEGA_USER  home=$OMEGA_HOME"
+  echo
+fi
+
+# In --full mode we ask EXACTLY one question before anything else:
+# "install Telegram?" If they pass --full-with-telegram or
+# --full-no-telegram on the CLI, we skip even this prompt.
+if [ "${INSTALL_FULL:-0}" = "1" ] && [ -z "${FULL_WANT_TELEGRAM:-}" ]; then
+  if [ -t 0 ] && [ "${NONINTERACTIVE:-0}" != "1" ]; then
+    if prompt_yes_no "  Install the Telegram bot?" "no"; then
+      FULL_WANT_TELEGRAM=yes
+    else
+      FULL_WANT_TELEGRAM=no
+    fi
+  else
+    # Truly headless (piped stdin) — default to NO Telegram.
+    FULL_WANT_TELEGRAM=no
+  fi
+  echo
+fi
 
 # Define the sequence ONCE so we can count it for the progress header.
 STEPS=(
@@ -157,7 +192,11 @@ STEPS=(
   "45-claude-plugins:step_claude_plugins"
 )
 if [ "$PROFILE" != "minimal" ]; then
-  STEPS+=("50-telegram:step_telegram")
+  # In --full mode the Telegram step is gated by the single yes/no asked
+  # at the very top. Skip the step entirely when the user said no.
+  if [ "${INSTALL_FULL:-0}" != "1" ] || [ "${FULL_WANT_TELEGRAM:-}" != "no" ]; then
+    STEPS+=("50-telegram:step_telegram")
+  fi
 fi
 STEPS+=(
   "55-autonomous:step_autonomous"
@@ -170,25 +209,27 @@ STEPS+=(
 export STEP_TOTAL="${#STEPS[@]}"
 export STEP_COUNT=0
 
-# ───── Plan mode ─────
-# Show every step BEFORE running anything — same shape as Claude Code's
-# plan mode: the operator sees the full sequence + can abort cleanly
-# with --dry-run, or just hit Enter to proceed.
-echo
-log "${C_DIM}─── Install plan (${STEP_TOTAL} steps, ~2-3 min total) ───${C_RST}"
-_idx=0
-for entry in "${STEPS[@]}"; do
-  _idx=$((_idx + 1))
-  _name="${entry%%:*}"
-  log "  ${C_DIM}${_idx}/${STEP_TOTAL}${C_RST}  ${_name}"
-done
-log "${C_DIM}─────────────────────────────────────────${C_RST}"
-echo
+# ───── Plan mode (verbose only) ─────
+# In NORMAL mode: show every step BEFORE running + ask for confirmation.
+# In --full mode: skip the plan — operator already opted in via the flag.
+if [ "${INSTALL_FULL:-0}" != "1" ]; then
+  echo
+  log "${C_DIM}─── Install plan (${STEP_TOTAL} steps, ~2-3 min total) ───${C_RST}"
+  _idx=0
+  for entry in "${STEPS[@]}"; do
+    _idx=$((_idx + 1))
+    _name="${entry%%:*}"
+    log "  ${C_DIM}${_idx}/${STEP_TOTAL}${C_RST}  ${_name}"
+  done
+  log "${C_DIM}─────────────────────────────────────────${C_RST}"
+  echo
+fi
 
 if [ "${DRY_RUN:-0}" = "1" ]; then
   ok "dry-run — exiting without running any step"
   exit 0
 fi
+
 if [ "${NONINTERACTIVE:-0}" != "1" ] && [ -t 0 ]; then
   if ! prompt_yes_no "Proceed with the install?" "yes"; then
     log "aborted by operator — no changes made"
@@ -196,10 +237,22 @@ if [ "${NONINTERACTIVE:-0}" != "1" ] && [ -t 0 ]; then
   fi
 fi
 
+# In --full mode, print the single header line that introduces the
+# progress bar so the user knows what's happening.
+if [ "${INSTALL_FULL:-0}" = "1" ]; then
+  echo
+  log "  ${C_DIM}${STEP_TOTAL} steps · output piped to ${LOG_FILE} · Ctrl-c to abort${C_RST}"
+  echo
+fi
+
 for entry in "${STEPS[@]}"; do
   name="${entry%%:*}"
   fn="${entry##*:}"
-  step_progress_header "$name"
+  # The verbose per-step header is suppressed in --full mode (progress
+  # bar replaces it).
+  if [ "${INSTALL_FULL:-0}" != "1" ]; then
+    step_progress_header "$name"
+  fi
   run_step "$name" "$fn"
 done
 

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.36"
 
 __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.36"
 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.36

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentikos/omega-os",
-  "version": "0.19.34",
+  "version": "0.19.36",
   "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"