Sabo/webui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: two issues found in post-merge review of PRs #82 #83 (#84)

Browse Source 
- routes.py /api/git-info: get_session raises KeyError on miss, does not
  return None -- wrap in try/except KeyError to correctly return 404
  (PR #82, api/routes.py line 222)

- style.css ctx-bar used undefined --teal CSS variable -- replaced with
  --blue which is defined in :root and fits the existing color palette
  (PR #83, static/style.css)

Co-authored-by: Nathan Esquenazi <nesquena@gmail.com>
This commit is contained in:
nesquena-hermes
2026-04-04 14:29:24 -07:00
committed by GitHub
parent fffdc34fdb
commit 74dd613b1d
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
api/routes.py
View File

@@ -219,8 +219,9 @@ def handle_get(handler, parsed):
sid = qs.get('session_id', [''])[0] sid = qs.get('session_id', [''])[0]
if not sid: if not sid:
return bad(handler, 'session_id required') return bad(handler, 'session_id required')
s = get_session(sid) try:
if not s: s = get_session(sid)
except KeyError:
return bad(handler, 'Session not found', 404) return bad(handler, 'Session not found', 404)
from api.workspace import git_info_for_workspace from api.workspace import git_info_for_workspace
info = git_info_for_workspace(Path(s.workspace)) info = git_info_for_workspace(Path(s.workspace))

2
static/style.css
View File

@@ -192,7 +192,7 @@
/* Context usage indicator */ /* Context usage indicator */
.ctx-indicator{display:flex;align-items:center;gap:6px;padding:2px 4px;flex-shrink:1;min-width:0;} .ctx-indicator{display:flex;align-items:center;gap:6px;padding:2px 4px;flex-shrink:1;min-width:0;}
.ctx-bar-wrap{width:70px;height:5px;border-radius:3px;background:rgba(255,255,255,.08);overflow:hidden;flex-shrink:0;} .ctx-bar-wrap{width:70px;height:5px;border-radius:3px;background:rgba(255,255,255,.08);overflow:hidden;flex-shrink:0;}
.ctx-bar{display:block;height:100%;border-radius:3px;transition:width .4s ease,background .4s ease;min-width:2px;background:var(--teal);} .ctx-bar{display:block;height:100%;border-radius:3px;transition:width .4s ease,background .4s ease;min-width:2px;background:var(--blue);}
.ctx-bar.ctx-mid{background:#e6a817;} .ctx-bar.ctx-mid{background:#e6a817;}
.ctx-bar.ctx-high{background:#e05252;} .ctx-bar.ctx-high{background:#e05252;}
.ctx-label{font-size:9px;color:var(--muted);white-space:nowrap;font-variant-numeric:tabular-nums;} .ctx-label{font-size:9px;color:var(--muted);white-space:nowrap;font-variant-numeric:tabular-nums;}