From 9dca8880ccb3af36ca3a49788b85c4ecf352bcfa Mon Sep 17 00:00:00 2001 From: Marcus Quinn <6428977+marcusquinn@users.noreply.github.com> Date: Wed, 18 Mar 2026 21:27:22 +0000 Subject: [PATCH] fix: use word boundaries in grep patterns to prevent false positives (#100) Address PR #86 review feedback from Gemini: - Line 145: replace 'grep -i error|fail|exception' with grep -E -i '\b(error|fail|exception)' - Line 224: replace 'grep -i ERROR|WARNING' with grep -E -i '\b(ERROR|WARNING)\b' Using \b word boundaries prevents substring matches (e.g. 'exception' in 'unexceptional') and removes the unnecessary cat pipe (UUOC). Closes #94 --- .agents/error-checking-feedback-loops.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.agents/error-checking-feedback-loops.md b/.agents/error-checking-feedback-loops.md index 90cb3f2..5df7858 100644 --- a/.agents/error-checking-feedback-loops.md +++ b/.agents/error-checking-feedback-loops.md @@ -142,7 +142,7 @@ npm run test:playground:multisite 2. **Analyze Output for Errors**: ```bash - grep -i 'error\|fail\|exception' test-output.log + grep -E -i '\b(error|fail|exception)' test-output.log ``` 3. **Parse Structured Test Results** (if available): @@ -221,7 +221,7 @@ npm run lint:css 2. **Analyze Output for Errors**: ```bash - grep -i 'ERROR\|WARNING' phpcs-output.log + grep -E -i '\b(ERROR|WARNING)\b' phpcs-output.log ``` 3. **Automatically Fix Issues** (when possible):