🎨 Simplify things; no command field

Author: mcncl

README.md

@@ -10,7 +10,7 @@ A Buildkite plugin that creates rich annotations from Bazel Event Protocol (BEP)
 - ✅ Displays build status with success/failure counts
 - ⏱️ Shows test durations and highlights slow tests
 - ❌ Provides detailed failure information with error logs
-- 🔄 Automatically detects BEP files based on Bazel commands
+- 🔄 Automatically detects BEP files at common locations
 - 💡 Includes random developer wisdom quotes for inspiration
 
 ## Prerequisites
@@ -24,13 +24,7 @@ This plugin requires:
 
 ### `bep_file` (optional)
 
-Path to the Bazel Event Protocol JSON file to parse. If not provided, the plugin will attempt to:
-1. Extract it from the `bazel_command` if provided
-2. Look for files at common locations (bazel-events.json, bazel-bep.json, bep.json)
-
-### `bazel_command` (optional)
-
-The original Bazel command that was run. This is used for display purposes and can also be used to extract the BEP file path if it contains a `--build_event_json_file` flag.
+Path to the Bazel Event Protocol JSON file to parse. If not provided, the plugin will look for files at common locations (bazel-events.json, bazel-bep.json, bep.json).
 
 ### `skip_if_no_bep` (optional, boolean)
 
@@ -51,17 +45,6 @@ steps:
           bep_file: bazel-events.json
 ```
 
-### Automatic BEP detection from Bazel command
-
-```yaml
-steps:
-  - label: "🔨 Build with Bazel"
-    command: |
-      bazel build //... --build_event_json_file=bazel-events.json
-    plugins:
-      - bazel-bep-annotate#v1.0.0:
-          bazel_command: "bazel build //... --build_event_json_file=bazel-events.json"
-```
 
 ### Skip annotations if no BEP file found
 

hooks/post-command

@@ -10,28 +10,10 @@ DIR="$(cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd)"
 
 # Get plugin configuration
 BEP_FILE=$(plugin_read_config BEP_FILE "")
-BAZEL_COMMAND=$(plugin_read_config BAZEL_COMMAND "")
 
 # Check if we should skip if no BEP file found
 SKIP_IF_NO_BEP=$(plugin_read_config SKIP_IF_NO_BEP "false")
 
-# Check for automatic BEP file detection based on Bazel command
-if [[ -z "${BEP_FILE}" && -n "${BAZEL_COMMAND}" ]]; then
-  # Try to extract the BEP file from the command if it was specified
-  if [[ "${BAZEL_COMMAND}" =~ --build_event_json_file=([^ ]+) ]]; then
-    BEP_FILE="${BASH_REMATCH[1]}"
-    echo "Detected BEP file from Bazel command: ${BEP_FILE}"
-  else
-    # No BEP file specified in command, check if skip is enabled
-    if [[ "${SKIP_IF_NO_BEP}" == "true" ]]; then
-      echo "No BEP file found in command and skip_if_no_bep is true, skipping annotation"
-      exit 0
-    fi
-    echo "Error: No BEP file found in command"
-    exit 1
-  fi
-fi
-
 # If we still don't have a BEP file, check if it exists at common locations
 if [[ -z "${BEP_FILE}" ]]; then
   # Try some common locations
@@ -73,7 +55,7 @@ if [[ ! -f "${BEP_FILE}" ]]; then
 fi
 
 # Process the BEP file and create the annotation
-process_bep "${BEP_FILE}" "${BAZEL_COMMAND}" || {
+process_bep "${BEP_FILE}" || {
   status=$?
   echo "Warning: BEP processing returned non-zero status ($status)"
   if [[ "${SKIP_IF_NO_BEP}" == "true" ]]; then

lib/bazel-bep.bash

@@ -43,7 +43,6 @@ create_annotation() {
 # Process the BEP file to extract useful information
 process_bep() {
   local BEP_FILE="$1"
-  local BAZEL_COMMAND="${2:-}"
 
   # Ensure the file exists
   if [[ ! -f "$BEP_FILE" ]]; then
@@ -225,9 +224,9 @@ process_bep() {
   # Clean header for the output
   local summary="## 🚀 Bazel Results\n\n"
 
-  # Add command used
-  if [ -n "$BAZEL_COMMAND" ]; then
-    summary+="**🏃 Command:** \`$BAZEL_COMMAND\`\n\n"
+  # Add command used if running in Buildkite
+  if [ -n "${BUILDKITE_COMMAND:-}" ]; then
+    summary+="**🏃 Command:** \`${BUILDKITE_COMMAND}\`\n\n"
   fi
 
   if [ $total_build_time -gt 0 ]; then

plugin.yml

@@ -9,8 +9,6 @@ configuration:
   properties:
     bep_file:
       type: string
-    bazel_command:
-      type: string
     skip_if_no_bep:
       type: boolean
   additionalProperties: false

tests/post-command.bats

@@ -17,7 +17,6 @@ setup() {
 
 process_bep() {
   local BEP_FILE="$1"
-  local BAZEL_COMMAND="${2:-}"
 
   echo "Mock processing BEP file: $BEP_FILE"
   if [[ -f "$BEP_FILE" ]]; then
@@ -60,28 +59,10 @@ LIB_DIR="$(cd "$DIR/../lib" && pwd)"
 
 # Get plugin configuration
 BEP_FILE=$(plugin_read_config BEP_FILE "")
-BAZEL_COMMAND=$(plugin_read_config BAZEL_COMMAND "")
 
 # Check if we should skip if no BEP file found
 SKIP_IF_NO_BEP=$(plugin_read_config SKIP_IF_NO_BEP "false")
 
-# Check for automatic BEP file detection based on Bazel command
-if [[ -z "${BEP_FILE}" && -n "${BAZEL_COMMAND}" ]]; then
-  # Try to extract the BEP file from the command if it was specified
-  if [[ "${BAZEL_COMMAND}" =~ --build_event_json_file=([^ ]+) ]]; then
-    BEP_FILE="${BASH_REMATCH[1]}"
-    echo "Detected BEP file from Bazel command: ${BEP_FILE}"
-  else
-    # No BEP file specified in command, check if skip is enabled
-    if [[ "${SKIP_IF_NO_BEP}" == "true" ]]; then
-      echo "No BEP file found in command and skip_if_no_bep is true, skipping annotation"
-      exit 0
-    fi
-    echo "Error: No BEP file found in command"
-    exit 1
-  fi
-fi
-
 # If we still don't have a BEP file, check if it exists at common locations
 if [[ -z "${BEP_FILE}" ]]; then
   # Try some common locations
@@ -123,7 +104,7 @@ if [[ ! -f "${BEP_FILE}" ]]; then
 fi
 
 # Process the BEP file and create the annotation
-process_bep "${BEP_FILE}" "${BAZEL_COMMAND}"
+process_bep "${BEP_FILE}"
 EOF
   chmod +x "$TEMP_DIR/hooks/post-command"
 }
@@ -173,17 +154,7 @@ teardown() {
   assert_output --partial "Mock processing BEP file: $TEMP_DIR/sample.bep"
 }
 
-@test "Detect BEP file from Bazel command" {
-  # Create a sample BEP file
-  touch "$TEMP_DIR/detected.bep"
-  export BUILDKITE_PLUGIN_BAZEL_BEP_ANNOTATE_BAZEL_COMMAND="bazel build //... --build_event_json_file=$TEMP_DIR/detected.bep"
-
-  run "$TEMP_DIR/hooks/post-command"
-
-  assert_success
-  assert_output --partial "Detected BEP file from Bazel command: $TEMP_DIR/detected.bep"
-  assert_output --partial "Mock processing BEP file: $TEMP_DIR/detected.bep"
-}
+# Removed test for bazel_command since we removed that functionality
 
 @test "Find BEP file at common location" {
   # Create sample BEP file at a common location