feat: Multi-Profile Support (Issue #28)

CHANGELOG.md

@@ -5,6 +5,51 @@
 
 ---
 
+## [v0.24] Sprint 22 -- Multi-Profile Support (Issue #28)
+*April 3, 2026 | 415 tests*
+
+### Features
+- **Profile picker (topbar).** Purple-accented chip with SVG user icon. Click
+  to open dropdown listing all profiles with gateway status dots (green =
+  running), model info, and skill count. Click any profile to switch; "Manage
+  profiles" link opens the sidebar panel.
+- **Profiles management panel.** New sidebar tab with full CRUD UI. Profile
+  cards show name, model/provider, skill count, API key status, and gateway
+  status badge. "Use" button switches profile, delete button removes non-default
+  profiles (with confirmation).
+- **Profile creation.** "+ New profile" form with name validation (`[a-z0-9_-]`),
+  optional "clone config from active" checkbox. Wraps the CLI's
+  `hermes_cli.profiles.create_profile()`.
+- **Profile deletion.** Confirm dialog. Auto-switches to default if deleting
+  the active profile. Blocked while agent is running.
+- **Seamless profile switching.** No server restart. Profile switch updates
+  `HERMES_HOME`, patches module-level caches in hermes-agent's `skills_tool`
+  and `cron/jobs`, reloads `.env` API keys and `config.yaml`, refreshes the
+  model dropdown, skills, memory, and cron panels.
+- **Per-session profile tracking.** `profile` field on Session records which
+  profile was active at creation. Backward-compatible (`null` for old sessions).
+
+### Bug Fixes
+- **Hardcoded `~/.hermes` paths.** Memory read/write and model discovery used
+  hardcoded paths. Now resolved through `get_active_hermes_home()`.
+- **Module-level path caching.** hermes-agent modules snapshot `HERMES_HOME`
+  at import time. Profile switch now monkey-patches `SKILLS_DIR`, `CRON_DIR`,
+  `JOBS_FILE`, `OUTPUT_DIR` so they track the active profile.
+
+### Architecture
+- New `api/profiles.py`: profile state management wrapping `hermes_cli.profiles`.
+  Thread-safe (`_profile_lock`). Lazy imports avoid circular deps.
+- `api/config.py`: module-level `cfg` replaced with reloadable `get_config()`
+  / `reload_config()`. Dynamic `_get_config_path()` resolves through profile.
+- `api/streaming.py`: `HERMES_HOME` added to env save/restore block.
+- Profile switch blocked while agent streams are active.
+- 5 new API endpoints: `GET /api/profiles`, `GET /api/profile/active`,
+  `POST /api/profile/switch`, `POST /api/profile/create`,
+  `POST /api/profile/delete`.
+- Zero modifications to hermes-agent code.
+
+---
+
 ## [v0.23] Sprint 21 -- Mobile Responsive + Docker
 *April 3, 2026 | 415 tests*
 
@@ -748,4 +793,4 @@ Three-panel layout: sessions sidebar, chat area, workspace panel.
 
 ---
 
-*Last updated: v0.23, April 3, 2026 | Tests: 415*
+*Last updated: v0.24, April 3, 2026 | Tests: 415*

SPRINTS.md

@@ -1,6 +1,6 @@
 # Hermes Web UI -- Forward Sprint Plan
 
-> Current state: v0.23 | 415 tests | Daily driver ready
+> Current state: v0.24 | 415 tests | Daily driver ready
 > This document plans the path from here to two targets:
 >
 > Target A: 1:1 feature parity with the Hermes CLI (everything you can do from the
@@ -454,14 +454,60 @@ enables deployment beyond localhost. Both were achievable without new dependenci
 
 ---
 
-## Sprint 22 -- Multi-Profile Support (PLANNED, Issue #28)
+## Sprint 22 -- Multi-Profile Support (COMPLETED, Issue #28)
 
-**Theme:** Switch between Hermes agent profiles seamlessly.
+**Theme:** Switch between Hermes agent profiles seamlessly from the web UI.
+
+**Why now:** Issue #28 requested full profile management in the UI. The CLI has
+had comprehensive profile support since v0.6.0 — isolated instances with their
+own config, skills, memory, cron, and API keys. The web UI was locked to a
+single default profile, blocking multi-persona workflows.
+
+### Track A: Bugs
+- **Hardcoded `~/.hermes` paths.** Memory read/write in routes.py and model
+  discovery in config.py used hardcoded paths instead of the active profile's
+  directory. Fixed to resolve through `get_active_hermes_home()`.
+- **Module-level cached paths.** hermes-agent's `skills_tool.py` and `cron/jobs.py`
+  snapshot `HERMES_HOME` at import time. Profile switch now monkey-patches these
+  cached variables (`SKILLS_DIR`, `CRON_DIR`, `JOBS_FILE`, `OUTPUT_DIR`).
 
 ### Track B: Features
-- **Profile picker.** Sidebar or topbar dropdown to switch profiles.
-- **Per-profile config.** Each profile has its own skills, memory, config.yaml.
-- **Seamless switching.** No restart required.
+- **Profile picker (topbar).** Purple-accented chip with SVG user icon in the
+  topbar. Click opens a dropdown listing all profiles with gateway status dots,
+  model info, and skill count. Click to switch; "Manage profiles" link opens
+  the management panel.
+- **Profiles sidebar panel.** New nav tab with full management UI. Cards show
+  each profile with model, provider, skill count, API key status, and gateway
+  badge. "Use" button to switch, delete button for non-default profiles.
+- **Profile creation.** "+ New profile" form with name validation (lowercase
+  alphanumeric + hyphens), optional "clone config from active" checkbox. Wraps
+  `hermes_cli.profiles.create_profile()`.
+- **Profile deletion.** Confirm dialog, auto-switches to default if deleting
+  the active profile. Blocked while agent is running.
+- **Seamless switching.** No server restart required. Profile switch updates
+  `HERMES_HOME` env var, patches module-level caches, reloads `.env` API keys,
+  reloads `config.yaml`, and refreshes the model dropdown, skills, memory, and
+  cron panels.
+- **Per-session profile tracking.** New `profile` field on Session records which
+  profile was active when the session was created. Backward-compatible (defaults
+  to `null` for old sessions).
+
+### Track C: Architecture
+- New `api/profiles.py` module (~200 lines): profile state management wrapping
+  `hermes_cli.profiles`. Thread-safe with `_profile_lock`. Lazy imports to
+  avoid circular dependencies.
+- `api/config.py`: Replaced module-level `cfg` dict with reloadable
+  `get_config()`/`reload_config()`. Dynamic `_get_config_path()` resolves
+  through active profile.
+- `api/streaming.py`: `HERMES_HOME` added to env save/restore block around
+  agent runs (alongside `TERMINAL_CWD`, `HERMES_EXEC_ASK`).
+- Profile switch blocked while any agent stream is active (process-global
+  `HERMES_HOME` cannot be changed mid-run).
+- Zero modifications to hermes-agent code required.
+
+**Tests:** 0 new (profile management requires hermes-agent integration). Total: 415.
+**Hermes CLI parity impact:** Very High (profile support is a major CLI feature)
+**Claude parity impact:** Low (Claude has no profile concept)
 
 ---
 
@@ -510,7 +556,8 @@ enables deployment beyond localhost. Both were achievable without new dependenci
 | Slash commands | Done (Sprint 17) |
 | Thinking/reasoning display | Done (Sprint 18) |
 | Auth / login | Done (Sprint 19) |
-| Voice input | Sprint 20 |
+| Voice input | Done (Sprint 20) |
+| Multi-profile support | Done (Sprint 22) |
 | Subagent visibility | Deferred |
 | Code execution (Jupyter) | Deferred |
 | Toolset control | Deferred |
@@ -538,11 +585,11 @@ enables deployment beyond localhost. Both were achievable without new dependenci
 | Mobile layout (basic) | Done (v0.16.1) |
 | Workspace tree view | Done (Sprint 18) |
 | Slash commands | Done (Sprint 17) |
-| Voice input | Sprint 20 |
-| TTS playback | Sprint 20 |
+| Voice input | Done (Sprint 20) |
+| TTS playback | Deferred |
 | Artifacts (HTML/SVG preview) | Deferred |
 | Code execution inline | Deferred |
-| Mobile-optimized layout | Sprint 21 |
+| Mobile-optimized layout | Done (Sprint 21) |
 | Sharing / public URLs | Not planned (requires server infra) |
 | Claude-specific features | Not replicable (Projects AI, artifacts sync) |
 
@@ -560,5 +607,5 @@ enables deployment beyond localhost. Both were achievable without new dependenci
 ---
 
 *Last updated: April 3, 2026*
-*Current version: v0.23 | 415 tests*
-*Next sprint: Sprint 22 (Multi-Profile Support)*
+*Current version: v0.24 | 415 tests*
+*Next sprint: Sprint 23 (Desktop Application)*

api/config.py

@@ -134,17 +134,44 @@ def _discover_python(agent_dir: Path) -> str:
 else:
     _HERMES_FOUND = False
 
-# ── Config file (optional YAML) ──────────────────────────────────────────────
-CONFIG_PATH = Path(os.getenv(
-    'HERMES_CONFIG_PATH',
-    str(HOME / '.hermes' / 'config.yaml')
-)).expanduser()
+# ── Config file (reloadable -- supports profile switching) ──────────────────
+_cfg_cache = {}
+_cfg_lock = threading.Lock()
+
+def _get_config_path() -> Path:
+    """Return config.yaml path for the active profile."""
+    env_override = os.getenv('HERMES_CONFIG_PATH')
+    if env_override:
+        return Path(env_override).expanduser()
+    try:
+        from api.profiles import get_active_hermes_home
+        return get_active_hermes_home() / 'config.yaml'
+    except ImportError:
+        return HOME / '.hermes' / 'config.yaml'
+
+def get_config() -> dict:
+    """Return the cached config dict, loading from disk if needed."""
+    if not _cfg_cache:
+        reload_config()
+    return _cfg_cache
+
+def reload_config():
+    """Reload config.yaml from the active profile's directory."""
+    with _cfg_lock:
+        _cfg_cache.clear()
+        config_path = _get_config_path()
+        try:
+            import yaml as _yaml
+            if config_path.exists():
+                loaded = _yaml.safe_load(config_path.read_text())
+                if isinstance(loaded, dict):
+                    _cfg_cache.update(loaded)
+        except Exception:
+            pass
 
-try:
-    import yaml as _yaml
-    cfg = _yaml.safe_load(CONFIG_PATH.read_text()) if CONFIG_PATH.exists() else {}
-except Exception:
-    cfg = {}
+# Initial load
+reload_config()
+cfg = _cfg_cache  # alias for backward compat with existing references
 
 # ── Default workspace discovery ───────────────────────────────────────────────
 def _discover_default_workspace() -> Path:
@@ -183,7 +210,7 @@ def print_startup_config():
         f'  state dir   : {STATE_DIR}',
         f'  workspace   : {DEFAULT_WORKSPACE}',
         f'  host:port   : {HOST}:{PORT}',
-        f'  config file : {CONFIG_PATH}  {"(found)" if CONFIG_PATH.exists() else "(not found, using defaults)"}',
+        f'  config file : {_get_config_path()}  {"(found)" if _get_config_path().exists() else "(not found, using defaults)"}',
         '',
     ]
     print('\n'.join(lines), flush=True)
@@ -234,11 +261,12 @@ def verify_hermes_imports():
 }
 
 # ── Toolsets (from config.yaml or hardcoded default) ─────────────────────────
-CLI_TOOLSETS = cfg.get('platform_toolsets', {}).get('cli', [
+_DEFAULT_TOOLSETS = [
     'browser', 'clarify', 'code_execution', 'cronjob', 'delegation', 'file',
     'image_gen', 'memory', 'session_search', 'skills', 'terminal', 'todo',
     'web', 'webhook',
-])
+]
+CLI_TOOLSETS = get_config().get('platform_toolsets', {}).get('cli', _DEFAULT_TOOLSETS)
 
 # ── Model / provider discovery ───────────────────────────────────────────────
 
@@ -396,7 +424,11 @@ def get_available_models() -> dict:
 
     # 3. Try to read auth store for active provider (if hermes is installed)
     if not active_provider:
-        auth_store_path = HOME / '.hermes' / 'auth.json'
+        try:
+            from api.profiles import get_active_hermes_home as _gah
+            auth_store_path = _gah() / 'auth.json'
+        except ImportError:
+            auth_store_path = HOME / '.hermes' / 'auth.json'
         if auth_store_path.exists():
             try:
                 import json as _j
@@ -406,7 +438,11 @@ def get_available_models() -> dict:
                 pass
 
     # 4. Check for API keys that imply available providers
-    hermes_env_path = HOME / '.hermes' / '.env'
+    try:
+        from api.profiles import get_active_hermes_home as _gah2
+        hermes_env_path = _gah2() / '.env'
+    except ImportError:
+        hermes_env_path = HOME / '.hermes' / '.env'
     env_keys = {}
     if hermes_env_path.exists():
         try:
@@ -655,3 +691,11 @@ def save_settings(settings: dict) -> dict:
 
 # ── SESSIONS in-memory cache (LRU OrderedDict) ───────────────────────────────
 SESSIONS: collections.OrderedDict = collections.OrderedDict()
+
+# ── Profile state initialisation ────────────────────────────────────────────
+# Must run after all imports are resolved to correctly patch module-level caches
+try:
+    from api.profiles import init_profile_state
+    init_profile_state()
+except ImportError:
+    pass  # hermes_cli not available -- default profile only

api/models.py

@@ -34,8 +34,8 @@ def _write_session_index():
 
 
 class Session:
-    def __init__(self, session_id=None, title='Untitled', workspace=str(DEFAULT_WORKSPACE), model=DEFAULT_MODEL, messages=None, created_at=None, updated_at=None, tool_calls=None, pinned=False, archived=False, project_id=None, **kwargs):
-        self.session_id = session_id or uuid.uuid4().hex[:12]; self.title = title; self.workspace = str(Path(workspace).expanduser().resolve()); self.model = model; self.messages = messages or []; self.tool_calls = tool_calls or []; self.created_at = created_at or time.time(); self.updated_at = updated_at or time.time(); self.pinned = bool(pinned); self.archived = bool(archived); self.project_id = project_id or None
+    def __init__(self, session_id=None, title='Untitled', workspace=str(DEFAULT_WORKSPACE), model=DEFAULT_MODEL, messages=None, created_at=None, updated_at=None, tool_calls=None, pinned=False, archived=False, project_id=None, profile=None, **kwargs):
+        self.session_id = session_id or uuid.uuid4().hex[:12]; self.title = title; self.workspace = str(Path(workspace).expanduser().resolve()); self.model = model; self.messages = messages or []; self.tool_calls = tool_calls or []; self.created_at = created_at or time.time(); self.updated_at = updated_at or time.time(); self.pinned = bool(pinned); self.archived = bool(archived); self.project_id = project_id or None; self.profile = profile
     @property
     def path(self): return SESSION_DIR / f'{self.session_id}.json'
     def save(self): self.updated_at = time.time(); self.path.write_text(json.dumps(self.__dict__, ensure_ascii=False, indent=2), encoding='utf-8'); _write_session_index()
@@ -44,7 +44,7 @@ def load(cls, sid):
         p = SESSION_DIR / f'{sid}.json'
         if not p.exists(): return None
         return cls(**json.loads(p.read_text(encoding='utf-8')))
-    def compact(self): return {'session_id': self.session_id, 'title': self.title, 'workspace': self.workspace, 'model': self.model, 'message_count': len(self.messages), 'created_at': self.created_at, 'updated_at': self.updated_at, 'pinned': self.pinned, 'archived': self.archived, 'project_id': self.project_id}
+    def compact(self): return {'session_id': self.session_id, 'title': self.title, 'workspace': self.workspace, 'model': self.model, 'message_count': len(self.messages), 'created_at': self.created_at, 'updated_at': self.updated_at, 'pinned': self.pinned, 'archived': self.archived, 'project_id': self.project_id, 'profile': self.profile}
 
 def get_session(sid):
     with LOCK:
@@ -63,7 +63,12 @@ def get_session(sid):
 
 def new_session(workspace=None, model=None):
     # Use _cfg.DEFAULT_MODEL (not the import-time snapshot) so save_settings() changes take effect
-    s = Session(workspace=workspace or get_last_workspace(), model=model or _cfg.DEFAULT_MODEL)
+    try:
+        from api.profiles import get_active_profile_name
+        _profile = get_active_profile_name()
+    except ImportError:
+        _profile = None
+    s = Session(workspace=workspace or get_last_workspace(), model=model or _cfg.DEFAULT_MODEL, profile=_profile)
     with LOCK:
         SESSIONS[s.session_id] = s
         SESSIONS.move_to_end(s.session_id)