Skip to content

Commit 4fcdc4a

Browse files
ZviBaratzclaude
andcommitted
fix: suppress blank lines for filtered extensions in field test output
Use return code 2 for silently filtered extensions (--extension filter) vs return code 1 for skipped extensions that produce output. Only print separator blank lines for extensions that had visible output. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent a16a5e0 commit 4fcdc4a

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

scripts/field-test-runner.sh

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ print(json.dumps(exts[$idx]))
214214

215215
# Filter by --extension if specified
216216
if [[ -n "$OPT_SINGLE_EXT" && "$name" != "$OPT_SINGLE_EXT" ]]; then
217-
return 1 # signal: skipped by filter
217+
return 2 # signal: silently filtered out
218218
fi
219219

220220
echo "--- $name ($uuid) ---"
@@ -329,10 +329,15 @@ print(json.dumps(entry))
329329

330330
# Main loop
331331
for idx in $(seq 0 $((EXT_COUNT - 1))); do
332-
if ! process_extension "$idx"; then
332+
rc=0
333+
process_extension "$idx" || rc=$?
334+
if [[ $rc -eq 1 ]]; then
333335
SKIPPED=$((SKIPPED + 1))
336+
echo ""
337+
elif [[ $rc -eq 0 ]]; then
338+
echo ""
334339
fi
335-
echo ""
340+
# rc==2: silently filtered, no output
336341
done
337342

338343
# ── Review Phase ──────────────────────────────────────────────────────

0 commit comments

Comments
 (0)