Skip to content

fix(web): publish dialog's failure banner uses the shared error alert - #33893

Merged
shivamhwp merged 1 commit into
pingdotgg:maria/redesign/4-settings-setupfrom
shivamhwp:t3/redesign-publish-dialog-alert
Nov 12, 2026
Merged

shivamhwp merged 1 commit into
pingdotgg:maria/redesign/4-settings-setupfrom
shivamhwp:t3/redesign-publish-dialog-alert

Conversation

@shivamhwp

@shivamhwp shivamhwp commented Nov 10, 2026 •

Copy link
Copy Markdown
Collaborator

Note

🤖 Claude Opus 5.5 on behalf of Shivam

On layer 4, the Publish repository dialog draws its "Publish failed" banner by hand: its own border, fill, padding and red text. That duplicates the shared alert surface. Macroscope's UI Consistency review pointed this out on #28842.

The banner now uses Alert variant="error" with AlertTitle and AlertDescription from ui/alert, the same way AddEnvironmentDialog and ThemeImportDialog show errors in a dialog body. Alert still sets role="alert", and the title and message are unchanged.

Stacks on maria/redesign/4-settings-setup.

Same dev server, same dialog: a project with no remote, GitHub made unreachable from the server (isolated gh config, dummy token, outbound HTTPS sent to a dead proxy), owner demo-owner, repository publish-demo, Publish clicked.

Before (layer 4): the hand-built banner.
Before: hand-built Publish failed banner

After: the shared error alert.
After: shared error Alert in the dialog body

vp run check:changed against layer 4 at 105d89f: format, lint, knip, typecheck and the React compiler check passed. No tests changed; this is a visual change, shown above.

The Agent context budget check fails on this PR because of layer 4, not this diff, which leaves agentContextBudget.json alone: main's budget shrank in #26570 after layer 4's last run, and layer 4's file is 2, 2 and 16 bytes over it now.

Claude Opus 5.5, Claude Code via T3 Code.

🤖 Generated with Claude Code

@github-actions github-actions Bot added vouch:trusted Author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Nov 10, 2026
@macroscopeapp

macroscopeapp Bot commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 105d89f

Macroscope's review found this PR approvable — The publish failure banner now uses the shared error alert styling while retaining its existing message, display condition, and accessibility role. Publishing behavior and repository data are unchanged.

No code changes detected at ce27509. Prior analysis still applies.

You can add or adjust custom eligibility rules. Learn more.

@maria-rcks
maria-rcks force-pushed the maria/redesign/4-settings-setup branch from 3851bcf to 51a4542 Compare November 11, 2026 07:29
@shivamhwp
shivamhwp force-pushed the t3/redesign-publish-dialog-alert branch from 105d89f to 7683547 Compare November 11, 2026 07:41
@maria-rcks
maria-rcks force-pushed the maria/redesign/4-settings-setup branch 2 times, most recently from 1f0f37f to b0c2282 Compare November 11, 2026 08:38
@maria-rcks

maria-rcks commented Nov 11, 2026 •

Copy link
Copy Markdown
Collaborator

Note

Written by claude-opus-5-5 on behalf of Maria

Update: merged the rebased base

maria/redesign/4-settings-setup was rebased, so this branch conflicted with it. I merged its new tip ae02238b1b into the head with a merge commit (4b8b08ebd7, no rebase or force-push). All 27 conflicts were old stack commits against their rebased copies, so the merge takes the base's version of each, and the PR diff is still the same one-file change to PublishRepositoryDialog.tsx (+5/-7). That file and ui/alert.tsx are unchanged on the new base, so the evidence below still applies. vp run check:changed against the base passes (format, lint, knip, typecheck, react compiler). The commit list still shows the pre-rebase stack commits, but they add nothing to the diff.

Agent context budget fails on this head (mcpToolListJsonBytes 48131 -> 48217). This PR touches no server or MCP text: the stack base maria/redesign/4-settings-setup already carries that budget number versus main, so the fix belongs in the stack, not here. Every other check is green.

Verified in a real dev environment

Head 7683547c19, web client against vp run dev, driven with Playwright and Chrome. The dialog only exists on this stack, so "before" is the PR's parent 51a4542678 (layer 4), not main. Both runs used the same copied state.

  • Added ~/publish-demo (a local git repo with no remote) through Add project → Local folder, and ran a real Claude Haiku 5.5 turn on it ("Read math.ts…", finished in 6s).
  • Git action options → Publish repository..., GitHub, owner demo-owner, repository publish-demo, Publish. The server had a dummy GH_TOKEN, an empty gh config, and GitHub HTTPS sent to a dead proxy, so publishing really failed and nothing was created.
  • PR: the failure shows in the shared error Alert. It still has role="alert" (1 element), and the title "Publish failed" and the server message are unchanged. Checked in light and dark themes.
  • Nearby behavior: after the failure, Cancel and then reopening Publish repository gives a clean form (0 alerts, empty owner and repository). The Git actions menu, the provider tiles and the Haiku thread are unchanged.
  • CI on this head is green.

Before (layer 4), light:
Before: hand-built Publish failed banner, light

After (this PR), light:
After: shared error Alert in the Publish dialog, light

Before (layer 4), dark:
Before: hand-built Publish failed banner, dark

After (this PR), dark:
After: shared error Alert in the Publish dialog, dark

Recording on this PR: open the menu, fill the fields, Publish, see the failure alert, Cancel, then reopen to a clean form.
https://uploads-production-47e4.up.railway.app/files/8a71c12d-29fd-4718-9fc8-62f96da0429b/publish-failed-after.mp4

Not verified

  • Desktop (Electron) shell. It renders the same web component, but I didn't run it on this Linux VM.
  • A successful publish to a real GitHub account. I blocked it on purpose so no repository got created. This diff only touches the failure banner.
  • Mobile doesn't have this dialog.

@maria-rcks
maria-rcks force-pushed the maria/redesign/4-settings-setup branch 2 times, most recently from ae02238 to 34d194a Compare November 11, 2026 20:02
@shivamhwp
shivamhwp force-pushed the t3/redesign-publish-dialog-alert branch from 4b8b08e to 41b823c Compare November 11, 2026 20:06
@maria-rcks
maria-rcks force-pushed the maria/redesign/4-settings-setup branch from 34d194a to d36d9a3 Compare November 12, 2026 01:52
@shivamhwp
shivamhwp force-pushed the t3/redesign-publish-dialog-alert branch from 41b823c to ce27509 Compare November 12, 2026 02:09
@shivamhwp
shivamhwp merged commit e5616fc into pingdotgg:maria/redesign/4-settings-setup Nov 12, 2026
35 checks passed
flamboh pushed a commit to flamboh/t3code that referenced this pull request Nov 12, 2026
screen-gd pushed a commit to screen-gd/t3code that referenced this pull request Nov 12, 2026
screen-gd pushed a commit to screen-gd/t3code that referenced this pull request Nov 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:trusted Author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants