Skip to content

Commit 9df586a

Browse files
committed
Use a here-string for the MIXED-only check in sync-project
`printf | grep -q` under pipefail can SIGPIPE printf and flip the result; test-pipefail-grep-q.sh flagged sync-project.sh:1702. Same pattern as the rest of the file. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018LeNWZV2sVH62vkgfYh5u5
1 parent 54cd032 commit 9df586a

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

‎bin/sync-project.sh‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1699,7 +1699,7 @@ if [ "$WIRED" -eq 1 ] && [ -x "$SCRIPT_DIR/doctor.sh" ] \
16991699
3) echo " (the project's mxcli predates 'fix property-sets', v0.26.0 — upgrade it before a Team Server merge)" ;;
17001700
4) _ps_why="$(printf '%s\n' "$_ps_out" | sed -n 's/^ WARN property sets: //p' | head -1)"
17011701
echo " could not check property sets: ${_ps_why:-see above} — not recorded; the next sync re-checks" ;;
1702-
1) if printf '%s\n' "$_ps_out" | grep -q ' NEEDS '; then
1702+
1) if grep -q -- ' NEEDS ' <<<"$_ps_out"; then
17031703
warn "property sets: Studio Pro's merge/diff can throw on this model (see above). Repair, gated:" \
17041704
"bin/doctor.sh --fix-property-sets '$PROJECT_DIR'"
17051705
else # only FAILED / MIXED rows (mxcli v0.29+): nothing the guarded repair can clear

0 commit comments

Comments
 (0)