← Back to PRs

#21481: ClarityOS: restore/stabilize tab + add update/smoke scripts

by omniclaw open 2026-02-20 00:42 View on GitHub →
app: web-ui gateway scripts size: XL
Focused PR for ClarityOS restoration and maintenance workflow.\n\nIncludes:\n- Restore ClarityOS tab wiring (UI + gateway handlers)\n- Stabilize polling/rendering behavior\n- Resolve gateway handler integration cleanly\n- Add scripts/local/update-custom.sh\n- Add scripts/local/smoke-clarityos.sh <!-- greptile_comment --> <h3>Greptile Summary</h3> This PR restores the ClarityOS tab to the Control UI and adds maintenance scripts for local development workflows. **Key changes:** - Added new ClarityOS tab with comprehensive dashboard showing status, timeline, proposals, and nightly improvements - Integrated 16 new gateway RPC handlers (`clarityos.*`) that invoke Python scripts for orchestration data - Added polling (60s interval) with visibility/connection checks to prevent aggressive requests - Added two bash maintenance scripts: `update-custom.sh` (branch syncing) and `smoke-clarityos.sh` (validation checks) - Removed i18n dependency from UI rendering (hardcoded English strings) **Critical issues:** - `ui/src/ui/app.ts` imports and calls `resolveInjectedAssistantIdentity()` which does not exist in `assistant-identity.ts`. This will cause a runtime error on app initialization. - Sessions tab behavior changed: clicking delete no longer auto-refreshes the list, requiring manual refresh **Minor concerns:** - The ClarityOS handlers spawn Python subprocesses synchronously via `spawnSync` which blocks the event loop during execution - Type definitions for ClarityOS responses use optional properties extensively (`status?.extra?.watchdog?.state`) making the view logic brittle <h3>Confidence Score: 1/5</h3> - This PR has critical runtime errors that will break the application on startup - The missing `resolveInjectedAssistantIdentity` function will cause immediate runtime failure when the app initializes. This is a blocking issue that must be fixed before merge. Additionally, the session deletion behavior change removes auto-refresh functionality. - Pay close attention to `ui/src/ui/app.ts` (missing function will cause crash) and `ui/src/ui/assistant-identity.ts` (needs to export the missing function) <sub>Last reviewed commit: 90736c7</sub> <!-- greptile_other_comments_section --> <sub>(3/5) Reply to the agent's comments like "Can you suggest a fix for this @greptileai?" or ask follow-up questions!</sub> <!-- /greptile_comment -->

Most Similar PRs