Remove /limits after moving to /status (#4055)

Moved to /status #4053
2025-09-22 11:23:05 -07:00
parent 76a9b11678
commit d2940bd4c3
11 changed files with 2 additions and 797 deletions
--- a/codex-rs/tui/src/chatwidget/snapshots/codex_tuichatwidgettests__limits_placeholder.snap
+++ b/codex-rs/tui/src/chatwidget/snapshots/codex_tuichatwidgettests__limits_placeholder.snap
@@ -1,10 +0,0 @@
---
-source: tui/src/chatwidget/tests.rs
-expression: visual
---
-[magenta]/limits[/]
-
-[bold]Rate limit usage snapshot[/]
-[dim]  Tip: run `/limits` right after Codex replies for freshest numbers.[/]
-  Real usage data is not available yet.
-[dim]  Send a message to Codex, then run /limits again.[/]
--- a/codex-rs/tui/src/chatwidget/snapshots/codex_tuichatwidgettests__limits_snapshot_basic.snap
+++ b/codex-rs/tui/src/chatwidget/snapshots/codex_tuichatwidgettests__limits_snapshot_basic.snap
@@ -1,29 +0,0 @@
---
-source: tui/src/chatwidget/tests.rs
-expression: visual
---
-[magenta]/limits[/]
-
-[bold]Rate limit usage snapshot[/]
-[dim]  Tip: run `/limits` right after Codex replies for freshest numbers.[/]
-  • Hourly limit[dim] (≈5 hours window)[/]: [dark-gray+bold]30.0% used[/]
-  • Weekly limit[dim] (≈1 week window)[/]: [dark-gray+bold]60.0% used[/]
-[green]  Within current limits[/]
-
-[dim]  ╭─────────────────────────────────────────────────╮[/]
-  [dim]│[/][dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/][dim]│[/]
-  [dim]│[/][dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/][dim]│[/]
-  [dim]│[/][dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/][dim]│[/]
-  [dim]│[/][dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/][dim]│[/]
-  [dim]│[/][dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/][dim]│[/]
-  [dim]│[/][dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/][dim]│[/]
-  [dim]│[/][green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/][dim]│[/]
-  [dim]│[/][green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/][dim]│[/]
-  [dim]│[/][green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-[dim]  ╰─────────────────────────────────────────────────╯[/]
-
-[bold]Legend[/]
-  • [dark-gray+bold]Dark gray[/] = weekly usage so far
-  • [green+bold]Green[/] = hourly capacity still available
-  • [bold]Default[/] = weekly capacity beyond the hourly window
--- a/codex-rs/tui/src/chatwidget/snapshots/codex_tuichatwidgettests__limits_snapshot_hourly_remaining.snap
+++ b/codex-rs/tui/src/chatwidget/snapshots/codex_tuichatwidgettests__limits_snapshot_hourly_remaining.snap
@@ -1,29 +0,0 @@
---
-source: tui/src/chatwidget/tests.rs
-expression: visual
---
-[magenta]/limits[/]
-
-[bold]Rate limit usage snapshot[/]
-[dim]  Tip: run `/limits` right after Codex replies for freshest numbers.[/]
-  • Hourly limit[dim] (≈5 hours window)[/]: [dark-gray+bold]0.0% used[/]
-  • Weekly limit[dim] (≈1 week window)[/]: [dark-gray+bold]20.0% used[/]
-[green]  Within current limits[/]
-
-[dim]  ╭─────────────────────────────────────────────────╮[/]
-  [dim]│[/][dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/][dim]│[/]
-  [dim]│[/][dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/][dim]│[/]
-  [dim]│[/][green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/][dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-[dim]  ╰─────────────────────────────────────────────────╯[/]
-
-[bold]Legend[/]
-  • [dark-gray+bold]Dark gray[/] = weekly usage so far
-  • [green+bold]Green[/] = hourly capacity still available
-  • [bold]Default[/] = weekly capacity beyond the hourly window
--- a/codex-rs/tui/src/chatwidget/snapshots/codex_tuichatwidgettests__limits_snapshot_mixed_usage.snap
+++ b/codex-rs/tui/src/chatwidget/snapshots/codex_tuichatwidgettests__limits_snapshot_mixed_usage.snap
@@ -1,29 +0,0 @@
---
-source: tui/src/chatwidget/tests.rs
-expression: visual
---
-[magenta]/limits[/]
-
-[bold]Rate limit usage snapshot[/]
-[dim]  Tip: run `/limits` right after Codex replies for freshest numbers.[/]
-  • Hourly limit[dim] (≈5 hours window)[/]: [dark-gray+bold]20.0% used[/]
-  • Weekly limit[dim] (≈1 week window)[/]: [dark-gray+bold]20.0% used[/]
-[green]  Within current limits[/]
-
-[dim]  ╭─────────────────────────────────────────────────╮[/]
-  [dim]│[/][dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/][dim]│[/]
-  [dim]│[/][dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/] [dark-gray](>_)[/][dim]│[/]
-  [dim]│[/][green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] [green](>_)[/] (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-[dim]  ╰─────────────────────────────────────────────────╯[/]
-
-[bold]Legend[/]
-  • [dark-gray+bold]Dark gray[/] = weekly usage so far
-  • [green+bold]Green[/] = hourly capacity still available
-  • [bold]Default[/] = weekly capacity beyond the hourly window
--- a/codex-rs/tui/src/chatwidget/snapshots/codex_tuichatwidgettests__limits_snapshot_weekly_heavy.snap
+++ b/codex-rs/tui/src/chatwidget/snapshots/codex_tuichatwidgettests__limits_snapshot_weekly_heavy.snap
@@ -1,29 +0,0 @@
---
-source: tui/src/chatwidget/tests.rs
-expression: visual
---
-[magenta]/limits[/]
-
-[bold]Rate limit usage snapshot[/]
-[dim]  Tip: run `/limits` right after Codex replies for freshest numbers.[/]
-  • Hourly limit[dim] (≈5 hours window)[/]: [dark-gray+bold]98.0% used[/]
-  • Weekly limit[dim] (≈1 week window)[/]: [dark-gray+bold]0.0% used[/]
-[green]  Within current limits[/]
-
-[dim]  ╭─────────────────────────────────────────────────╮[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-  [dim]│[/](>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_) (>_)[dim]│[/]
-[dim]  ╰─────────────────────────────────────────────────╯[/]
-
-[bold]Legend[/]
-  • [dark-gray+bold]Dark gray[/] = weekly usage so far
-  • [green+bold]Green[/] = hourly capacity still available
-  • [bold]Default[/] = weekly capacity beyond the hourly window
--- a/codex-rs/tui/src/chatwidget/tests.rs
+++ b/codex-rs/tui/src/chatwidget/tests.rs
@@ -25,7 +25,6 @@ use codex_core::protocol::InputMessageKind;
 use codex_core::protocol::Op;
 use codex_core::protocol::PatchApplyBeginEvent;
 use codex_core::protocol::PatchApplyEndEvent;
-use codex_core::protocol::RateLimitSnapshotEvent;
 use codex_core::protocol::ReviewCodeLocation;
 use codex_core::protocol::ReviewFinding;
 use codex_core::protocol::ReviewLineRange;
@@ -40,8 +39,6 @@ use crossterm::event::KeyEvent;
 use crossterm::event::KeyModifiers;
 use insta::assert_snapshot;
 use pretty_assertions::assert_eq;
-use ratatui::style::Color;
-use ratatui::style::Modifier;
 use std::fs::File;
 use std::io::BufRead;
 use std::io::BufReader;
@@ -380,115 +377,6 @@ fn lines_to_single_string(lines: &[ratatui::text::Line<'static>]) -> String {
    s
 }

-fn styled_lines_to_string(lines: &[ratatui::text::Line<'static>]) -> String {
-    let mut out = String::new();
-    for line in lines {
-        for span in &line.spans {
-            let mut tags: Vec<&str> = Vec::new();
-            if let Some(color) = span.style.fg {
-                let name = match color {
-                    Color::Black => "black",
-                    Color::Blue => "blue",
-                    Color::Cyan => "cyan",
-                    Color::DarkGray => "dark-gray",
-                    Color::Gray => "gray",
-                    Color::Green => "green",
-                    Color::LightBlue => "light-blue",
-                    Color::LightCyan => "light-cyan",
-                    Color::LightGreen => "light-green",
-                    Color::LightMagenta => "light-magenta",
-                    Color::LightRed => "light-red",
-                    Color::LightYellow => "light-yellow",
-                    Color::Magenta => "magenta",
-                    Color::Red => "red",
-                    Color::Rgb(_, _, _) => "rgb",
-                    Color::Indexed(_) => "indexed",
-                    Color::Reset => "reset",
-                    Color::Yellow => "yellow",
-                    Color::White => "white",
-                };
-                tags.push(name);
-            }
-            let modifiers = span.style.add_modifier;
-            if modifiers.contains(Modifier::BOLD) {
-                tags.push("bold");
-            }
-            if modifiers.contains(Modifier::DIM) {
-                tags.push("dim");
-            }
-            if modifiers.contains(Modifier::ITALIC) {
-                tags.push("italic");
-            }
-            if modifiers.contains(Modifier::UNDERLINED) {
-                tags.push("underlined");
-            }
-            if !tags.is_empty() {
-                out.push('[');
-                out.push_str(&tags.join("+"));
-                out.push(']');
-            }
-            out.push_str(&span.content);
-            if !tags.is_empty() {
-                out.push_str("[/]");
-            }
-        }
-        out.push('\n');
-    }
-    out
-}
-
-fn sample_rate_limit_snapshot(
-    primary_used_percent: f64,
-    weekly_used_percent: f64,
-    ratio_percent: f64,
-) -> RateLimitSnapshotEvent {
-    RateLimitSnapshotEvent {
-        primary_used_percent,
-        weekly_used_percent,
-        primary_to_weekly_ratio_percent: ratio_percent,
-        primary_window_minutes: 300,
-        weekly_window_minutes: 10_080,
-    }
-}
-
-fn capture_limits_snapshot(snapshot: Option<RateLimitSnapshotEvent>) -> String {
-    let lines = match snapshot {
-        Some(ref snapshot) => history_cell::new_limits_output(snapshot).display_lines(80),
-        None => history_cell::new_limits_unavailable().display_lines(80),
-    };
-    styled_lines_to_string(&lines)
-}
-
-#[test]
-fn limits_placeholder() {
-    let visual = capture_limits_snapshot(None);
-    assert_snapshot!(visual);
-}
-
-#[test]
-fn limits_snapshot_basic() {
-    let visual = capture_limits_snapshot(Some(sample_rate_limit_snapshot(30.0, 60.0, 40.0)));
-    assert_snapshot!(visual);
-}
-
-#[test]
-fn limits_snapshot_hourly_remaining() {
-    let visual = capture_limits_snapshot(Some(sample_rate_limit_snapshot(0.0, 20.0, 10.0)));
-    assert_snapshot!(visual);
-}
-
-#[test]
-fn limits_snapshot_mixed_usage() {
-    let visual = capture_limits_snapshot(Some(sample_rate_limit_snapshot(20.0, 20.0, 10.0)));
-    assert_snapshot!(visual);
-}
-
-#[test]
-fn limits_snapshot_weekly_heavy() {
-    let visual = capture_limits_snapshot(Some(sample_rate_limit_snapshot(98.0, 0.0, 10.0)));
-    assert_snapshot!(visual);
-}
-
 #[test]
 fn rate_limit_warnings_emit_thresholds() {
    let mut state = RateLimitWarningState::default();