fix(ui): replace [dev] debug label with skill icon in Automation Advisor (#5934) - #5993
Conversation
…sor (#5934) `loadAutomationAdvisorData` used the bare text `[dev]` as the icon for skill-type suggestions — a debug placeholder left from development. Replace it with 🧬, consistent with the `node-skills` icon at line 25480. Add `test_no_debug_label_text_in_app_js` to guard against this class of regression across all functions in app.js, not just the rendered cost-optimizer body. Closes #5934 Co-Authored-By: ClawMetry Autofix Bot <bot-autofix@clawmetry.dev> Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NTMYY4MukDVJ4haD8Lo2SL
Visual diffComparing 41 of 70 comparison(s) flagged (>1% pixel diff).
Folder: 09fcba3a2729. Full PNGs also attached as a workflow artefact. Generated by visual-diff bot. Pixel diffs >1% flagged; eyeball the table before merging. This check is non-blocking — fail = bot bug, not a code problem. |
|
E2E Gate (required) — runner queue timeout, not a code regression The E2E Gate failed after its 3600s (60-minute) timeout. The gate aggregates sub-checks and times out when any of them don't start. In this run, 7 sub-checks never received a GitHub Actions runner within that window:
All pending jobs were then cancelled at 22:18:59 UTC by the This is a GitHub Actions runner availability issue. The change in this PR — replacing Generated by Claude Code |
✅ Drift Bot (ClawMetry): no drift detectedDrift Bot analyzed the changed files against this project's blueprints and requirements and found no drift. |
✅ Drift Bot (ClawMetry): no drift detectedDrift Bot analyzed the changed files against this project's blueprints and requirements and found no drift. |
docs/ATLAS_OPENCLAW_SCORECARD.md was stale after the main merge: framework_map.py moved from mapping version 2026-09-14.1 to 2026-09-14.2 (landed in main via #5944/#5952). One line in the generated preamble was out of date; re-running gen_atlas_openclaw_scorecard.py fixes it. No-PRD: generated file sync; no product logic changed Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NTMYY4MukDVJ4haD8Lo2SL
✅ Drift Bot (ClawMetry): no drift detectedDrift Bot analyzed the changed files against this project's blueprints and requirements and found no drift. |
1 participant
Summary
The Automation Advisor's suggestion renderer used the bare text
[dev]as the icon for skill-type suggestions — a debug placeholder that slipped into the shipped bundle. This replaces it with🧬, consistent with the existingnode-skillsicon inCOMP_MAP. A companion static-analysis test prevents the same class of label from re-entering unnoticed.No-PRD: cosmetic debug-label fix; already covered by REQ-OBS-CEA-023 acceptance criteria tracked in #5934
Changes
clawmetry/static/js/app.js:27082—'[dev]'→'🧬'inloadAutomationAdvisorData'stypeIconexpressiontests/test_cost_optimizer_demo_honesty.py— newtest_no_debug_label_text_in_app_jsthat regex-scans all ofapp.jsfor bracket-style debug labels ([dev],[prod],[debug],[test]), so the guard covers every function, not just the rendered cost-optimizer bodyTest plan
python3 -m pytest tests/test_cost_optimizer_demo_honesty.py -q— both the new test andtest_modal_renders_basis_and_no_debug_labelshould pass[dev]Bot meta
Draft PR opened autonomously based on the plan in #5934. Marked draft for human review — mark Ready for Review once happy.
The 33s cold-load spinner is tracked separately in #5935 (open PR). The provenance/figure labelling, 402 upgrade-prompt and time-travel message are already fixed in the current code; only this debug label remained.
Closes #5934
🤖 Generated with Claude Code
https://claude.ai/code/session_01NTMYY4MukDVJ4haD8Lo2SL