Address remaining review items: rebase, polish, cleanup

L1:  Remove GFM trailing whitespace from sample markdown files
L6:  Add comment about approximate alignment in waterfall markers
L8:  Clarify --time-period help text in latency_report.py
L9:  Scope .gitignore *.log to scripts/ and examples/ only
L10: Remove duplicate env validation from latency_report.sh
     (Python script now handles validation with structured logging)
L11: Rebased onto origin/main (17 commits ahead, 0 behind)

79 tests pass, autoformat clean.

Author: evekhm

.gitignore

@@ -21,7 +21,8 @@ scripts/reports/
 examples/*/reports/
 examples/*/reports_*/
 examples/*/trials_*/
-*.log
+scripts/**/*.log
+examples/**/*.log
 
 # Local workspace metadata
 .code*/

scripts/latency_report.py

@@ -377,7 +377,7 @@ def render_waterfall(trace):
     lines.append(f"  {label:<{label_w}} {bar} {format_ms(span.latency_ms)}")
 
   lines.append(f"  {'':{label_w}} {'─' * BAR_W}")
-  # Time axis labels
+  # Time axis labels (approximate alignment; format_ms width varies)
   markers = f"  {'':{label_w}} 0{'':>{BAR_W//2-1}}{format_ms(total_ms/2)}{'':>{BAR_W//2-1}}{format_ms(total_ms)}"
   lines.append(markers)
 
@@ -557,8 +557,9 @@ def main():
   parser.add_argument(
       "--time-period",
       type=str,
-      help="Time range filter (e.g. 1h, 30m, 7d). "
-      "If omitted, fetches the latest traces regardless of age",
+      default=None,
+      help="Time range filter: 1h, 30m, 7d (default: no time filter, "
+      "fetches the latest traces by --limit)",
   )
   parser.add_argument(
       "--app-name",

scripts/latency_report.sh

@@ -46,14 +46,7 @@ elif [ -f "${SCRIPT_DIR}/../.env" ]; then
     set +a
 fi
 
-# Validate required env vars
-for var in PROJECT_ID DATASET_ID TABLE_ID DATASET_LOCATION; do
-    if [ -z "${!var}" ]; then
-        echo "ERROR: Required environment variable ${var} is not set."
-        echo "Set it in your shell or create a .env file with these variables,"
-        echo "or pass --env path/to/.env. See scripts/README.md."
-        exit 1
-    fi
-done
+# Env validation is handled by the Python script with structured logging;
+# the shell wrapper only needs to load the .env file.
 
 python3 "${SCRIPT_DIR}/latency_report.py" "$@"

scripts/sample_latency_report.md

@@ -2,9 +2,9 @@
 
 Output of `./scripts/latency_report.sh --limit 3 --time-period 1h`:
 
-**Project:** my-gcp-project  
-**Dataset:** agent_logs.agent_events  
-**Location:** us-central1  
+**Project:** my-gcp-project
+**Dataset:** agent_logs.agent_events
+**Location:** us-central1
 
 ```
 Fetching traces from my-gcp-project.agent_logs.agent_events...

scripts/sample_quality_report.md

@@ -2,12 +2,12 @@
 
 Markdown report generated by `./scripts/quality_report.sh --report --config agent_context.json`.
 
-**Generated:** 2026-05-10 04:53:12  
-**Project:** my-gcp-project  
-**Dataset:** agent_logs.agent_events  
-**Location:** us-central1  
-**Eval model:** gemini-2.5-flash  
-**Sessions:** 100  
+**Generated:** 2026-05-10 04:53:12
+**Project:** my-gcp-project
+**Dataset:** agent_logs.agent_events
+**Location:** us-central1
+**Eval model:** gemini-2.5-flash
+**Sessions:** 100
 
 ## Summary