[tomrittervg]

arc diff exits when it needs an answer, because stdin isn't a TTY, and it asks the same question every time. PDF.js's update to 7a4022e2 (bug 2092381) changed binary cmap files that arc doesn't recognise as binary, so arc asked whether to mark them as binary; we retried it four times over nine minutes, then commented only that we couldn't submit to Phabricator. Give up on the first attempt, and quote the question in the comment.

[Copilot]

🔵 Needs a closer look

The stdout-only mock leaves the terminal-failure classification unverified because real arc output has not confirmed the prompt’s stream placement or trailing [Y/n] or [y/N] markers.

No open issues 🎉

What changed in this PR

This PR updates automated Phabricator submissions to stop retrying when arc diff asks an interactive question and include that question in the Bugzilla failure comment.

Changes:

• Detect unanswered yes/no prompts and raise an error that bypasses retries.
• Include the extracted question in the existing submission-failure notification.
• Add a regression test covering a single attempt, the terminal job outcome, and the reported question.

File	Description
tests/​functionality_all_platforms.py	Tests prompt handling, retry suppression, and Bugzilla reporting.
tasktypes/​vendoring.py	Includes prompt details in submission-failure comments.
apis/​phabricator.py	Detects unanswered prompts and stops retries with a dedicated exception.

_{Review effort: Balanced}