Skip to content

fix(webapp): render error page full screen and use Enter shortcut#4121

Merged
samejr merged 3 commits into
mainfrom
samejr/fullscreen-error-page
Jul 2, 2026
Merged

fix(webapp): render error page full screen and use Enter shortcut#4121
samejr merged 3 commits into
mainfrom
samejr/fullscreen-error-page

Conversation

@samejr

@samejr samejr commented Jul 2, 2026

Copy link
Copy Markdown
Member

Makes the app error page always render full screen — it previously inherited the width/offset of whatever container the error boundary was mounted in (e.g. the centered max-w-xs column in the root boundary), so min-h-screen alone couldn't fill the viewport. The root container now uses fixed inset-0 z-50 to break out and cover the full screen regardless of nesting.

Also changes the "Go to homepage" shortcut from Cmd/Ctrl+G (which collides with the browser's native "Find Again") to Enter.

@changeset-bot

changeset-bot Bot commented Jul 2, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 2bc3f11

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

devin-ai-integration[bot]

This comment was marked as resolved.

@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: b59d58e7-9511-46dd-bf22-35c4940b7751

📥 Commits

Reviewing files that changed from the base of the PR and between ee0c55d and 2bc3f11.

📒 Files selected for processing (1)
  • apps/webapp/app/components/ErrorDisplay.tsx
🚧 Files skipped from review as they are similar to previous changes (1)
  • apps/webapp/app/components/ErrorDisplay.tsx
📜 Recent review details
⚠️ CI failures not shown inline (24)

GitHub Actions: 🔎 REVIEW.md Drift Audit / audit: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run anthropics/claude-code-action@428971d2ecd6e3a7cb0ee0da2a3a8b33fdb3678d
 with:
   anthropic_***REDACTED***
   use_sticky_comment: true
   allowed_bots: devin-ai-integration[bot]
   claude_args: --max-turns 30
--allowedTools "Read,Glob,Grep,Bash(git diff:*)"
   prompt: You are auditing this PR for drift against `.claude/REVIEW.md`.
## Context
`.claude/REVIEW.md` is the repo's source of truth for what AI / agent code reviewers should treat as critical findings (rolling-deploy safety, hot-table indexes, recovery-path queries, testcontainers usage, Lua versioning, etc.). It is consumed by review agents to calibrate severity. If REVIEW.md goes stale, every future agent review degrades.
## Strategy — read this first
You have a hard turn budget. Spend it on signal, not coverage. The audit is allowed to miss things; it is NOT allowed to time out.
1. Read `.claude/REVIEW.md` once, in full.
2. Run `git diff origin/main...HEAD --name-only` to get the list of changed files. Do NOT read the diff content yet.
3. Scan the file-list for relevance to REVIEW.md scope. Relevance signals: changes to Prisma schema, Redis / queue / Lua code, hot tables, recovery / restart loops, new packages, deletions of paths REVIEW.md cites. Skim everything else.
4. Open at most **5 files** total — only the ones most likely to surface a real signal. If nothing in the file-list looks relevant to any REVIEW.md rule, do NOT read any files; go straight to the verdict.
5. Form a verdict and stop. Do not exhaust the turn budget exploring.
Large PRs (>50 files changed) are a strong signal to be MORE selective, not more thorough. Pick 3-5 files at most.
## What to look for
- **Stale references** — does any REVIEW.md rule cite a file, directory, function, table, Prisma model, or package name that has been removed or renamed in this PR (or is already gone from `main`)?
- **Contradictions** — does code in this PR clearly violate a current REVIEW.md rule? (Don't re-review the PR. Only flag if REVIE...

GitHub Actions: 🔎 REVIEW.md Drift Audit / 0_audit.txt: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

egacy-run-engine.fix2
  * [new tag]             build-legacy-run-engine.fix3 -> build-legacy-run-engine.fix3
  * [new tag]             build-manual-checkpoints.rc1 -> build-manual-checkpoints.rc1
  * [new tag]             build-metadata-upgrade-logging.rc1 -> build-metadata-upgrade-logging.rc1
  * [new tag]             build-metadata-upgrade-logging.rc2 -> build-metadata-upgrade-logging.rc2
  * [new tag]             build-metadata-upgrade-logging.rc3 -> build-metadata-upgrade-logging.rc3
  * [new tag]             build-new-build-system.rc.1 -> build-new-build-system.rc.1
  * [new tag]             build-otel-upgrade-rc.0     -> build-otel-upgrade-rc.0
  * [new tag]             build-otel-upgrade-rc.1     -> build-otel-upgrade-rc.1
  * [new tag]             build-pre-pull-deployments-rc.1 -> build-pre-pull-deployments-rc.1
  * [new tag]             build-prod-rescue-rc.1      -> build-prod-rescue-rc.1
  * [new tag]             build-rate-limiter-fix-rc.1 -> build-rate-limiter-fix-rc.1
  * [new tag]             build-re2.rc0               -> build-re2.rc0
  * [new tag]             build-realtime-v2-stream-fix -> build-realtime-v2-stream-fix
  * [new tag]             build-realtime-v2-stream-fix-2 -> build-realtime-v2-stream-fix-2
  * [new tag]             build-realtime-v2-stream-fix-3 -> build-realtime-v2-stream-fix-3
  * [new tag]             build-realtime-v2-stream-fix-4 -> build-realtime-v2-stream-fix-4
  * [new tag]             build-realtime-v2-stream-fix-5 -> build-realtime-v2-stream-fix-5
  * [new tag]             build-realtimestreams-dedupe -> build-realtimestreams-dedupe
  * [new tag]             build-registry-maintenance-rc.1 -> build-registry-maintenance-rc.1
  * [new tag]             build-registry-maintenance-rc.2 -> build-registry-maintenance-rc.2
  * [new tag]             build-remote-ecr-rc.0       -> build-remote-ecr-rc.0
  * [new tag]             build-reschedule-hotfix.rc1 -> build-reschedule-hotfix.rc1
  * [new tag]             build-resume-f...

GitHub Actions: 📝 CLAUDE.md Audit / audit: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run anthropics/claude-code-action@428971d2ecd6e3a7cb0ee0da2a3a8b33fdb3678d
 with:
   anthropic_***REDACTED***
   use_sticky_comment: true
   allowed_bots: devin-ai-integration[bot]
   claude_args: --max-turns 25
--model claude-opus-4-8
--allowedTools "Read,Glob,Grep,Bash(git diff:*)"
   prompt: You are reviewing a PR to check whether any CLAUDE.md files or .claude/rules/ files need updating.
## Your task
1. Run `git diff origin/main...HEAD --name-only` to see which files changed in this PR.
2. For each changed directory, check if there's a CLAUDE.md in that directory or a parent directory.
3. Determine if any CLAUDE.md or .claude/rules/ file should be updated based on the changes. Consider:
   - New files/directories that aren't covered by existing documentation
   - Changed architecture or patterns that contradict current CLAUDE.md guidance
   - New dependencies, services, or infrastructure that Claude should know about
   - Renamed or moved files that are referenced in CLAUDE.md
   - Changes to build commands, test patterns, or development workflows
## Response format
If NO updates are needed, respond with exactly:
✅ CLAUDE.md files look current for this PR.
If updates ARE needed, respond with a short list:
📝 **CLAUDE.md updates suggested:**
- `path/to/CLAUDE.md`: [what should be added/changed]
- `.claude/rules/file.md`: [what should be added/changed]
Keep suggestions specific and brief. Only flag things that would actually mislead Claude in future sessions.
Do NOT suggest updates for trivial changes (bug fixes, small refactors within existing patterns).
Do NOT suggest creating new CLAUDE.md files - only updates to existing ones.
   trigger_phrase: `@claude`
   label_trigger: claude
   branch_prefix: claude/
   use_bedrock: false
   use_vertex: false
   use_foundry: false
   classify_inline_comments: true
   use_commit_signing: false
   bot_id: 41898282
   bot_name: claude[bot]
   track_progress: false
   include_fix_links: true
   display_report: false...

GitHub Actions: 📝 CLAUDE.md Audit / 0_audit.txt: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

trigger.dev/yalt@3.0.0-beta.45
  * [new tag]             `@trigger.dev/yalt`@3.0.0-beta.46 -> `@trigger.dev/yalt`@3.0.0-beta.46
  * [new tag]             `@trigger.dev/yalt`@3.0.0-beta.47 -> `@trigger.dev/yalt`@3.0.0-beta.47
  * [new tag]             `@trigger.dev/yalt`@3.0.0-beta.48 -> `@trigger.dev/yalt`@3.0.0-beta.48
  * [new tag]             `@trigger.dev/yalt`@3.0.0-beta.49 -> `@trigger.dev/yalt`@3.0.0-beta.49
  * [new tag]             `@trigger.dev/yalt`@3.0.0-beta.5 -> `@trigger.dev/yalt`@3.0.0-beta.5
  * [new tag]             `@trigger.dev/yalt`@3.0.0-beta.50 -> `@trigger.dev/yalt`@3.0.0-beta.50
  * [new tag]             `@trigger.dev/yalt`@3.0.0-beta.51 -> `@trigger.dev/yalt`@3.0.0-beta.51
  * [new tag]             `@trigger.dev/yalt`@3.0.0-beta.52 -> `@trigger.dev/yalt`@3.0.0-beta.52
  * [new tag]             `@trigger.dev/yalt`@3.0.0-beta.53 -> `@trigger.dev/yalt`@3.0.0-beta.53
  * [new tag]             `@trigger.dev/yalt`@3.0.0-beta.55 -> `@trigger.dev/yalt`@3.0.0-beta.55
  * [new tag]             `@trigger.dev/yalt`@3.0.0-beta.6 -> `@trigger.dev/yalt`@3.0.0-beta.6
  * [new tag]             `@trigger.dev/yalt`@3.0.0-beta.7 -> `@trigger.dev/yalt`@3.0.0-beta.7
  * [new tag]             build-alert-hotfix.rc1      -> build-alert-hotfix.rc1
  * [new tag]             build-alert-hotfix.rc2      -> build-alert-hotfix.rc2
  * [new tag]             build-arm-builds-rc.1       -> build-arm-builds-rc.1
  * [new tag]             build-arm-builds-rc.2       -> build-arm-builds-rc.2
  * [new tag]             build-arm-builds-rc.3       -> build-arm-builds-rc.3
  * [new tag]             build-batchid-carryover-rc.0 -> build-batchid-carryover-rc.0
  * [new tag]             build-batching-rc.1         -> build-batching-rc.1
  * [new tag]             build-batching-rc.2         -> build-batching-rc.2
  * [new tag]             build-billing-0.0.1         -> build-billing-0.0.1
  * [new tag]             build-billing-0.0.2         -> build-billing-0.0.2
  * [new tag]             build-billing-0.0.3         -> build-b...

GitHub Actions: 🤖 PR Checks / webapp _ 🧪 Unit Tests Webapp (3, 10): fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / webapp _ 🧪 Unit Tests Webapp (2, 10): fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / webapp _ 🧪 Unit Tests Webapp (8, 10): fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / webapp _ 🧪 Unit Tests Webapp (6, 10): fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / webapp _ 🧪 Unit Tests Webapp (5, 10): fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / webapp _ 🧪 Unit Tests Webapp (10, 10): fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / webapp _ 🧪 Unit Tests Webapp (9, 10): fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / webapp _ 🧪 Unit Tests Webapp (7, 10): fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / webapp _ 🧪 Unit Tests Webapp (4, 10): fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / 7_webapp _ 🧪 Unit Tests Webapp (6, 10).txt: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / 9_webapp _ 🧪 Unit Tests Webapp (10, 10).txt: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / 11_webapp _ 🧪 Unit Tests Webapp (9, 10).txt: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / 4_webapp _ 🧪 Unit Tests Webapp (3, 10).txt: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / webapp _ 🧪 Unit Tests Webapp (1, 10): fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / 10_webapp _ 🧪 Unit Tests Webapp (7, 10).txt: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / 13_webapp _ 🧪 Unit Tests Webapp (1, 10).txt: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / 8_webapp _ 🧪 Unit Tests Webapp (5, 10).txt: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / 5_webapp _ 🧪 Unit Tests Webapp (8, 10).txt: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / 12_webapp _ 🧪 Unit Tests Webapp (4, 10).txt: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

GitHub Actions: 🤖 PR Checks / 6_webapp _ 🧪 Unit Tests Webapp (2, 10).txt: fix(webapp): render error page full screen and use Enter shortcut

Conclusion: failure

View job details

##[group]Run # Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.
 �[36;1m# Retry each pull - DockerHub registry timeouts are a recurring transient CI flake.�[0m
 �[36;1mpull() {�[0m
 �[36;1m  for attempt in 1 2 3; do�[0m
 �[36;1m    docker pull "$1" && return 0�[0m
 �[36;1m    echo "::warning::docker pull $1 failed (attempt ${attempt}/3); retrying in 10s"�[0m
 �[36;1m    sleep 10�[0m
 �[36;1m  done�[0m
 �[36;1m  echo "::error::docker pull $1 failed after 3 attempts"�[0m

Walkthrough

The PR updates ErrorDisplay to render as a fixed, full-viewport overlay and changes the homepage shortcut from Cmd/Ctrl+G to Enter. It also adds a changelog entry in .server-changes/error-display-fullscreen.md describing the webapp fix.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description covers the change but misses most required template sections like issue link, checklist, testing, changelog, and screenshots. Add the required template sections: Closes #issue, checklist items, testing steps, changelog, and screenshots or mark them not applicable.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main changes: fullscreen error page rendering and the Enter shortcut.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch samejr/fullscreen-error-page

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

coderabbitai[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@samejr samejr merged commit 22a7e33 into main Jul 2, 2026
30 checks passed
@samejr samejr deleted the samejr/fullscreen-error-page branch July 2, 2026 18:09
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.

2 participants