close
Skip to content

feat(status): surface major service incidents - #6987

Merged
TheodoreSpeaks merged 2 commits into
stagingfrom
feat/sim-status-banner
Aug 22, 2026
Merged

feat(status): surface major service incidents#6987
TheodoreSpeaks merged 2 commits into
stagingfrom
feat/sim-status-banner

Conversation

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator

Summary

  • show a sidebar status alert for major and critical incidents
  • poll the public Sim status endpoint while preserving workflow-list scrolling
  • add a public env override for forcing the critical preview during testing

Type of Change

  • New feature

Testing

  • 43 focused tests passing
  • lint, env-flag validation, block-registry validation, and all 32 audits passing

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Aug 22, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
Image docs Skipped Skipped Aug 22, 2026 10:05pm

Request Review

@cursor

cursor Bot commented Aug 22, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
UI-only status polling of a public endpoint with validated responses; no auth or data-handling changes. CSP is widened only for status.sim.ai.

Overview
Hosted workspaces now show a sidebar alert when Sim’s public status page reports a major or critical incident, with a link to status.sim.ai. Minor/operational states and fetch failures stay hidden.

The notice polls the public status JSON every 60s (CSP connect-src updated). Self-hosted builds skip it unless NEXT_PUBLIC_STATUS_NOTICE_PREVIEW forces the critical preview for testing. The sidebar scroll area gets min-h-0 so the list still scrolls when the notice is visible.

Reviewed by Cursor Bugbot for commit 6fccacd. Bugbot is set up for automated code reviews on this repo. Configure here.

@greptile-apps

greptile-apps Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR adds a hosted-workspace sidebar notice for major and critical Sim service incidents while keeping the notice hidden when status data is unavailable.

  • Polls and validates the public status endpoint through React Query.
  • Adds an environment-controlled critical-incident preview.
  • Updates sidebar layout and CSP connectivity for the new integration.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/status-notice/status-notice.tsx Renders the incident notice only for major or critical status data and safely remains hidden when the optional query fails.
apps/sim/hooks/queries/status-page.ts Defines non-retrying status polling with cancellation, stale-time, and refresh behavior.
apps/sim/lib/status-page.ts Fetches the public status summary and strictly validates its response.
apps/sim/app/workspace/[workspaceId]/w/components/sidebar/sidebar.tsx Mounts the notice below the scrollable workflow list for hosted or preview environments.
apps/sim/lib/core/security/csp.ts Allows browser connections to the Sim status origin.

Reviews (2): Last reviewed commit: "fix(status): isolate status endpoint fai..." | Re-trigger Greptile

<div className='flex-shrink-0 px-2 py-2'>
<StatusNotice preview={isStatusNoticePreviewEnabled} />
</div>
) : null}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Empty status spacer always reserved

Low Severity

The hosted sidebar always mounts a px-2 py-2 wrapper around StatusNotice. When the notice returns null for loading, operational, or minor status, that empty padded div still occupies vertical space above the footer on every expanded hosted session.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 165db31. Configure here.

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator Author

@greptile

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

1 issue from previous review remains unresolved.

Fix All in Cursor

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 6fccacd. Configure here.

@TheodoreSpeaks
TheodoreSpeaks merged commit 7173a3f into staging Aug 22, 2026
30 checks passed
@TheodoreSpeaks
TheodoreSpeaks deleted the feat/sim-status-banner branch August 22, 2026 22:11
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