bug: dispatcher grep -oP fails in Alpine — BusyBox doesn't support Perl regex #150

New issue

Closed

opened 2026-04-02 15:57:40 +00:00 by dev-bot · 0 comments

dev-bot commented

2026-04-02 15:57:40 +00:00

Collaborator

Problem

The dispatcher's get_pr_for_file() function uses grep -oP '#\d+' to extract PR numbers from merge commit messages. BusyBox grep in the Alpine-based edge container does not support -P (Perl regex). The command fails silently, returning empty, causing every vault action to fail with "No PR found".

Fix

Replace grep -oP with grep -oE (extended regex, supported by BusyBox):

# Before (broken in Alpine):
pr_num=$(echo "$merge_line" | grep -oP '#\d+' | head -1 | tr -d '#')

# After (works everywhere):
pr_num=$(echo "$merge_line" | grep -oE '#[0-9]+' | head -1 | tr -d '#')

Also audit the entire dispatcher for other grep -P usage and replace with -E.

Affected files

docker/edge/dispatcher.sh — get_pr_for_file() function

Acceptance criteria

No grep -P or grep -oP in dispatcher.sh
get_pr_for_file() correctly extracts PR number in Alpine/BusyBox
End-to-end: vault TOML via merged PR is detected and runner launched
CI green

## Problem The dispatcher's `get_pr_for_file()` function uses `grep -oP '#\d+'` to extract PR numbers from merge commit messages. BusyBox grep in the Alpine-based edge container does not support `-P` (Perl regex). The command fails silently, returning empty, causing every vault action to fail with "No PR found". ## Fix Replace `grep -oP` with `grep -oE` (extended regex, supported by BusyBox): ```bash # Before (broken in Alpine): pr_num=$(echo "$merge_line" | grep -oP '#\d+' | head -1 | tr -d '#') # After (works everywhere): pr_num=$(echo "$merge_line" | grep -oE '#[0-9]+' | head -1 | tr -d '#') ``` Also audit the entire dispatcher for other `grep -P` usage and replace with `-E`. ## Affected files - `docker/edge/dispatcher.sh` — `get_pr_for_file()` function ## Acceptance criteria - [ ] No `grep -P` or `grep -oP` in dispatcher.sh - [ ] `get_pr_for_file()` correctly extracts PR number in Alpine/BusyBox - [ ] End-to-end: vault TOML via merged PR is detected and runner launched - [ ] CI green