Skip to content

chore(release): staging to production - 2026.06.03 - #1091

Merged
ct3685 merged 2 commits into
productionfrom
staging
Jun 3, 2026
Merged

chore(release): staging to production - 2026.06.03#1091
ct3685 merged 2 commits into
productionfrom
staging

Conversation

@github-actions

@github-actions github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown

🚀 Release: Staging to Production

Release Date: 2026-06-03

Changes in this release


This PR is automatically created/updated when commits are pushed to staging.
Merging this PR will trigger the release workflow to create a new GitHub release.

…1090)

## Summary
- `MainLayout`'s main content area offset itself by `drawerWidth`
(260px) whenever the menu state was "closed" (the default below the `lg`
breakpoint), via `margin-left: -260px` and `width: calc(100% - 260px)`.
- The in-layout `<Sidebar />` drawer it was compensating for is
commented out (the studio renders its own `AppDrawer`), so this offset
was phantom: it shifted content off-screen and shrank its width by
260px, breaking responsiveness on sub-`lg` widths.
- The `!open` branch now uses `margin-left: 0`, `width: 100%`, keeping
only the responsive `padding`. Removed the now-unused `drawerWidth`
import.

This is the permanent equivalent of "unchecking" `margin-left: -260px`
(and reclaiming the lost 260px of width) that was confirmed to fix the
layout in devtools.

## Test plan
- [ ] Load a `sidekick-studio` page (e.g.
`/sidekick-studio/admin/guardrails`)
- [ ] Resize the viewport below ~1200px (lg) and confirm content is no
longer shifted off-screen or horizontally overflowing
- [ ] Confirm layout is unchanged at >= lg widths
@vercel

vercel Bot commented Jun 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
answerai-docs Building Building Preview Jun 3, 2026 7:31pm
the-answerai Building Building Preview Jun 3, 2026 7:31pm

Request Review

## Summary
- Harden the shared axios response interceptor in
`packages/ui/src/api/client.js` to use optional chaining
(`error.response?.status`) instead of `error.response.status`.
- On network failures, CORS-blocked responses, and gateway timeouts
(e.g. a `504 Gateway Timeout` returned by the ALB with no
`Access-Control-Allow-Origin` header), `error.response` is `undefined`.
The previous code threw `Cannot read properties of undefined (reading
'status')`, which aborted the interceptor and masked the real failure
across the whole app.
- With this guard, such errors now fall through to the existing `return
Promise.reject(error)`, so callers' `.catch` handlers run and the actual
error message surfaces.

### Context
Surfaced while diagnosing the Guardrails Settings page: `GET
/api/v1/guardrails/selftest` was returning a `504` (backend can't reach
the Fiddler VPC endpoint outside the VPN), and the resulting
CORS/`ERR_FAILED` rejection had no `error.response`, triggering the
`reading 'status'` crash shown in the Plan capabilities panel. This
change does not fix the underlying Fiddler reachability (infra/VPC
endpoint work); it only makes the client fail gracefully.

### Risk
Behavior is unchanged for any real HTTP response (including the 401 /
token-refresh flow, which still has `error.response`). The optional
chaining only adds graceful handling for the previously-crashing
no-response case.

## Test plan
- [ ] Trigger an API call that fails with no response (e.g. network
offline / 504 from gateway) and confirm the UI shows a real error
instead of `Cannot read properties of undefined (reading 'status')`.
- [ ] Confirm a genuine 401 still triggers token refresh / redirect to
login as before.
@ct3685
ct3685 merged commit 1058fca into production Jun 3, 2026
8 of 11 checks passed
@maxtechera
maxtechera temporarily deployed to staging - aai-unified2-flowise-moonstruck June 3, 2026 19:43 — with Render Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants