The addition of `spawnSync` in `auto-review-guardrails.ts` should be reviewed for potential blocking behavior #207

New issue

Closed

opened 2026-05-15 11:10:25 -07:00 by jwilger · 1 comment

jwilger commented

2026-05-15 11:10:25 -07:00

Owner

🟡 Warning: The addition of spawnSync in auto-review-guardrails.ts should be reviewed for potential blocking behavior, as it can block the event loop. Consider using asynchronous alternatives if possible.

Originally posted by @auto-review in jwilger/auto_review#203 (comment)

🟡 **Warning:** The addition of `spawnSync` in `auto-review-guardrails.ts` should be reviewed for potential blocking behavior, as it can block the event loop. Consider using asynchronous alternatives if possible. _Originally posted by @auto-review in https://git.johnwilger.com/jwilger/auto_review/pulls/203#issuecomment-8176_

jwilger added the

bug

label

2026-05-15 11:10:34 -07:00

jwilger added this to the 1.0 milestone

2026-05-15 11:10:37 -07:00

~~jwilger referenced this issue 2026-05-15 15:32:34 -07:00~~

feat(pi): add focused guardrail workflow tools #208

jwilger referenced this issue from a commit

2026-05-15 16:11:54 -07:00

chore(pi): add safe branch workflow tools (#208)

jwilger commented