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

Merged

dev-qwen merged 1 commit from fix/issue-150 into main

2026-04-02 16:14:23 +00:00

dev-qwen commented

2026-04-02 16:00:09 +00:00

Collaborator

Fixes #150

Changes

Fixes #150 ## Changes

dev-qwen added 1 commit 2026-04-02 16:00:10 +00:00

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

ci/woodpecker/push/ci Pipeline was successful

Details

ci/woodpecker/pr/ci Pipeline was successful

Details

843440428e

review-bot commented

2026-04-02 16:13:04 +00:00

Collaborator

AI Review

Summary

Replaces grep -oP '#\d+' with grep -oE '#[0-9]+' to fix Alpine/BusyBox compatibility. BusyBox grep does not support Perl regex (-P), causing the dispatcher to silently fail when extracting PR numbers from merge commit messages.

Assessment

Correctness: grep -oE '#[0-9]+' is functionally identical to grep -oP '#\d+' for this use case. Both extract the first #NNN token from a string.
Compatibility: -oE with bracket expressions is POSIX ERE, supported universally including BusyBox.
No regressions: The rest of the pipeline (head -1 | tr -d '#') is unchanged.
Scope: Single-line, targeted fix with no side effects.

Change is correct and complete.

Verdict

APPROVE — Correct BusyBox-compatible fix: -oE with [0-9]+ is functionally equivalent to -oP with \d+

Reviewed at 8434404 | AGENTS.md

## AI Review  ### Summary Replaces `grep -oP '#\d+'` with `grep -oE '#[0-9]+'` to fix Alpine/BusyBox compatibility. BusyBox grep does not support Perl regex (`-P`), causing the dispatcher to silently fail when extracting PR numbers from merge commit messages. ### Assessment - **Correctness**: `grep -oE '#[0-9]+'` is functionally identical to `grep -oP '#\d+'` for this use case. Both extract the first `#NNN` token from a string. - **Compatibility**: `-oE` with bracket expressions is POSIX ERE, supported universally including BusyBox. - **No regressions**: The rest of the pipeline (`head -1 | tr -d '#'`) is unchanged. - **Scope**: Single-line, targeted fix with no side effects. Change is correct and complete. ### Verdict **APPROVE** — Correct BusyBox-compatible fix: -oE with [0-9]+ is functionally equivalent to -oP with \d+ --- *Reviewed at `8434404` | [AGENTS.md](AGENTS.md)*

review-bot approved these changes 2026-04-02 16:13:04 +00:00

review-bot left a comment

Collaborator

AI Review: APPROVE — Correct BusyBox-compatible fix: -oE with [0-9]+ is functionally equivalent to -oP with \d+

AI Review: **APPROVE** — Correct BusyBox-compatible fix: -oE with [0-9]+ is functionally equivalent to -oP with \d+