feat: improve the result table display (#217)

Author: art049

AGENTS.md

@@ -9,6 +9,7 @@ CodSpeed Runner is a Rust CLI application for gathering performance data and upl
 ## Common Development Commands
 
 ### Building and Testing
+
 ```bash
 # Build the project
 cargo build
@@ -32,6 +33,7 @@ cargo test -- --nocapture      # with cargo test
 **Note**: Always check if `cargo nextest` is available first (with `cargo nextest --version` or `which cargo-nextest`). If available, use it instead of `cargo test` as it provides faster and more reliable test execution.
 
 ### Running the Application
+
 ```bash
 # Build and run
 cargo run -- <subcommand> <args>
@@ -43,6 +45,7 @@ cargo run -- setup
 ```
 
 ### Code Quality
+
 ```bash
 # Check code without building
 cargo check
@@ -59,26 +62,31 @@ cargo clippy
 The application follows a modular structure:
 
 ### Core Modules
+
 - **`main.rs`**: Entry point with error handling and logging setup
 - **`app.rs`**: CLI definition using clap with subcommands (Run, Auth, Setup)
 - **`api_client.rs`**: CodSpeed GraphQL API client
 - **`auth.rs`**: Authentication management
 - **`config.rs`**: Configuration loading and management
 
 ### Run Module (`src/run/`)
+
 The core functionality for running benchmarks:
+
 - **`run_environment/`**: CI provider implementations (GitHub Actions, GitLab CI, Buildkite, local)
 - **`runner/`**: Execution modes:
   - **`valgrind/`**: Instrumentation mode using custom Valgrind
   - **`wall_time/perf/`**: Walltime mode with perf integration
 - **`uploader/`**: Results upload to CodSpeed
 
 ### Key Dependencies
+
 - `clap`: CLI framework with derive macros
 - `tokio`: Async runtime (current_thread flavor)
 - `reqwest`: HTTP client with middleware/retry
 - `serde`/`serde_json`: Serialization
 - `gql_client`: Custom GraphQL client
+- `tabled`: Table formatting for CLI output (https://docs.rs/tabled/latest/tabled/index.html)
 - Platform-specific: `procfs` (Linux), `linux-perf-data`
 
 ## Environment Variables
@@ -90,6 +98,7 @@ The core functionality for running benchmarks:
 ## Testing
 
 The project uses:
+
 - `cargo nextest` (preferred) or standard Rust `cargo test`
 - `insta` for snapshot testing
 - `rstest` for parameterized tests
@@ -98,5 +107,6 @@ The project uses:
 Test files include snapshots in `snapshots/` directories for various run environment providers.
 
 **Important**:
+
 - Always prefer `cargo nextest run` over `cargo test` when running tests, as it provides better performance and reliability.
 - Some walltime executor tests require `sudo` access and will fail in non-interactive environments (e.g., `test_walltime_executor::*`). These failures are expected if sudo is not available.

CLAUDE.md

@@ -0,0 +1 @@
+AGENTS.md

Cargo.lock

@@ -69,17 +69,17 @@ shellexpand = { version = "3.1.1", features = ["tilde"] }
 addr2line = "0.25"
 gimli = "0.32"
 open = "5.3.2"
-tabled = "0.17"
+tabled = { version = "0.20.0", features = ["ansi"] }
 shell-words = "1.1.0"
 rmp-serde = "1.3.0"
-test-log = "0.2.19"
 
 [target.'cfg(target_os = "linux")'.dependencies]
 procfs = "0.17.0"
 
 [dev-dependencies]
 temp-env = { version = "0.3.6", features = ["async_closure"] }
 insta = { version = "1.29.0", features = ["json", "redactions"] }
+test-log = "0.2.19"
 test-with = { version = "0.15", default-features = false, features = [] }
 rstest = { version = "0.25.0", default-features = false }
 rstest_reuse = "0.7.0"

Cargo.toml

@@ -110,16 +110,6 @@ impl Display for ReportConclusion {
     }
 }
 
-#[derive(Debug, Deserialize, Serialize)]
-#[serde(rename_all = "camelCase")]
-pub struct FetchLocalRunReportRun {
-    pub id: String,
-    pub status: RunStatus,
-    pub url: String,
-    pub head_reports: Vec<FetchLocalRunReportHeadReport>,
-    pub results: Vec<FetchLocalRunBenchmarkResult>,
-}
-
 #[derive(Deserialize, Serialize, Debug, Clone, PartialEq)]
 #[serde(rename_all = "SCREAMING_SNAKE_CASE")]
 pub enum RunStatus {
@@ -129,24 +119,44 @@ pub enum RunStatus {
     Processing,
 }
 
-#[derive(Debug, Deserialize, Serialize)]
-#[serde(rename_all = "camelCase")]
-pub struct FetchLocalRunReportHeadReport {
-    pub id: String,
-    pub impact: Option<f64>,
-    pub conclusion: ReportConclusion,
-}
-
-#[derive(Debug, Deserialize, Serialize)]
-pub struct FetchLocalRunBenchmarkResult {
-    pub value: f64,
-    pub benchmark: FetchLocalRunBenchmark,
-}
-
-#[derive(Debug, Deserialize, Serialize)]
-pub struct FetchLocalRunBenchmark {
-    pub name: String,
-    pub executor: ExecutorName,
+nest! {
+    #[derive(Debug, Deserialize, Serialize)]*
+    #[serde(rename_all = "camelCase")]*
+    pub struct FetchLocalRunReportRun {
+        pub id: String,
+        pub status: RunStatus,
+        pub url: String,
+        pub head_reports: Vec<pub struct FetchLocalRunReportHeadReport {
+            pub id: String,
+            pub impact: Option<f64>,
+            pub conclusion: ReportConclusion,
+        }>,
+        pub results: Vec<pub struct FetchLocalRunBenchmarkResult {
+            pub value: f64,
+            pub benchmark: pub struct FetchLocalRunBenchmark {
+                pub name: String,
+                pub executor: ExecutorName,
+            },
+            pub valgrind: Option<pub struct ValgrindResult {
+                pub time_distribution: Option<pub struct TimeDistribution {
+                    pub ir: f64,
+                    pub l1m: f64,
+                    pub llm: f64,
+                    pub sys: f64,
+                }>,
+            }>,
+            pub walltime: Option<pub struct WallTimeResult {
+                pub iterations: f64,
+                pub stdev: f64,
+                pub total_time: f64,
+            }>,
+            pub memory: Option<pub struct MemoryResult {
+                pub peak_memory: i64,
+                pub total_allocated: i64,
+                pub alloc_calls: i64,
+            }>,
+        }>,
+    }
 }
 
 nest! {

src/api_client.rs

@@ -5,7 +5,7 @@ use crate::api_client::{
     CodSpeedAPIClient, FetchLocalRunReportResponse, FetchLocalRunReportVars, RunStatus,
 };
 use crate::prelude::*;
-use crate::run::helpers::poll_results::{
+use crate::run::helpers::benchmark_display::{
     POLLING_INTERVAL, RUN_PROCESSING_MAX_DURATION, build_benchmark_table, build_detailed_summary,
 };
 use crate::run::uploader::UploadResult;
@@ -62,7 +62,7 @@ pub async fn poll_results(
         }
 
         info!(
-            "\nTo see the full report, visit: {}",
+            "\nFull report: {}",
             style(response.run.url).blue().bold().underlined()
         );
     }

src/exec/poll_results.rs

@@ -13,11 +13,29 @@ query FetchLocalRunReport($owner: String!, $name: String!, $runId: String!) {
         conclusion
       }
       results {
-        value
         benchmark {
           name
           executor
         }
+        value
+        valgrind {
+          timeDistribution {
+            ir
+            l1m
+            llm
+            sys
+          }
+        }
+        walltime {
+          iterations
+          stdev
+          totalTime
+        }
+        memory {
+          peakMemory
+          totalAllocated
+          allocCalls
+        }
       }
     }
   }