Skip to content

fix(server): compact the startup banner - #293

Merged
nachiketb-nvidia merged 1 commit into
mainfrom
fix/compact-server-banner
Aug 4, 2026
Merged

fix(server): compact the startup banner#293
nachiketb-nvidia merged 1 commit into
mainfrom
fix/compact-server-banner

Conversation

@nachiketb-nvidia

@nachiketb-nvidia nachiketb-nvidia commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

What

Reduce the Rust switchyard-server startup wordmark from 120 columns to 60 columns.

Why

The existing banner wraps in a standard 80-column terminal, including the default-sized macOS Terminal window.

How

Preserve the existing six-line block-and-shade artwork while halving its horizontal pixel scale.

What to review

  • The compact banner retains the existing visual style.
  • The widest line is 60 columns.
  • No server behavior or startup content changes.

Validation

  • wc -L crates/switchyard-server/assets/startup_banner.txt reports 60.
  • git diff --check passes.
  • No tests run; this changes only a static text asset.

Summary by CodeRabbit

  • Style
    • Updated the startup banner with a refreshed six-line ASCII design and new visual layout.

Signed-off-by: nachiketb <nachiketb@nvidia.com>
@nachiketb-nvidia
nachiketb-nvidia requested a review from a team as a code owner August 4, 2026 21:38
@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: ce6f914a-8b46-443f-b55b-c5faeb29cc98

📥 Commits

Reviewing files that changed from the base of the PR and between eea4e01 and 9be5905.

📒 Files selected for processing (1)
  • crates/switchyard-server/assets/startup_banner.txt

Walkthrough

The startup banner asset now uses a new six-line block-character design.

Changes

Startup banner redesign

Layer / File(s) Summary
Replace startup banner artwork
crates/switchyard-server/assets/startup_banner.txt
The existing six-line artwork was replaced with a new six-line block-character layout.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Poem

I’m a rabbit hopping near,
A fresh banner now appears.
Six neat lines in blocks align,
Startup art looks bright and fine.
Hop, hop—change complete! 🐇

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: reducing the startup banner width.
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.

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

@nachiketb-nvidia
nachiketb-nvidia enabled auto-merge (squash) August 4, 2026 21:41
@nachiketb-nvidia
nachiketb-nvidia merged commit a3283c9 into main Aug 4, 2026
17 of 18 checks passed
@nachiketb-nvidia
nachiketb-nvidia deleted the fix/compact-server-banner branch August 4, 2026 21:44
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