Skip to content

fix(dashboard): fall back to latest release viewer on version-pinned 404 - #633

Open
fengleli wants to merge 1 commit into
Egonex-AI:mainfrom
fengleli:fix/dashboard-release-fallback
Open

fix(dashboard): fall back to latest release viewer on version-pinned 404#633
fengleli wants to merge 1 commit into
Egonex-AI:mainfrom
fengleli:fix/dashboard-release-fallback

Conversation

@fengleli

@fengleli fengleli commented Aug 9, 2026

Copy link
Copy Markdown

Problem

The dashboard fast path pins the prebuilt viewer to the plugin's package.json version. That version can be ahead of the latest published viewer (e.g. 2.9.4 has no release asset while the viewer ships under 2.9.0). When the pinned asset 404s, the launch fails and falls through to the heavy local build (which needs pnpm install + core build + Node >= 22), so the dashboard effectively never starts.

Fix

  • In step 4, after the version-pinned attempt fails (process exits without printing Dashboard URL), automatically resolve the latest published release via gh release list (or the GitHub API as a fallback) and retry the viewer there.
  • Surface the tokenized URL as a DASHBOARD_URL= line and log everything to /tmp/ua-dashboard.log so the agent can report it without scraping the terminal.

Verification

Ran the updated fast path locally: pinned 2.9.4 failed → resolved v2.9.0 → launched the viewer and printed the tokenized DASHBOARD_URL. Dashboard started successfully.

This is a behavioral fix only in understand-dashboard/SKILL.md; no code changes.

…d asset 404s

The fast path pinned the prebuilt viewer to the plugin's package.json
version, but that version can be ahead of the latest published viewer
(e.g. 2.9.4 with no release while the viewer ships under 2.9.0). When
the pinned asset is missing the launch failed and fell through to the
heavy local build.

Resolve the latest published release via 'gh release list' (or the
GitHub API as a fallback) and retry the viewer there. Also surface the
tokenized Dashboard URL as DASHBOARD_URL and log to /tmp/ua-dashboard.log.
@Lum1104
Lum1104 self-requested a review August 11, 2026 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant