The easiest fix here is probably to just return an interstitial.

Entering MFA credentials during an OAuth server redirect workflow can currently be blocked by the CSP.

PHI410 and PHI348 are both related to Differential's TYPE_UPDATE transaction. This is essentially the only Differential transaction which has not moved to ModularTransactions.

Just to briefly touch this:

This is effectively paused until I'm more convinced that the stabilizations changes really stabilized things -- I'm hoping to stabilize first, then work on improvements from there.

The corresponding support ticket was likely addressed by these changes. The remaining stuff here is still planned, but probably not moving forward any time soon.

The actual support issues were likely addressed or obsoleted by the above changes. The rest of this is still planned, probably won't move forward any time soon.

• Phage is now an ArcanistToolset.
• Every toolset gets <toolset> help/--help, <toolset> version/--version.
• Every toolset gets <toolset> alias, although this is probably broken for everything for now.
• Every toolset gets <toolset> shell-complete, but good luck.
• Aliases can now resolve to other aliases.
• getCommandSynopses() is now getWorkflowSynopses().
  • Return a string or list of strings.
  • Don't format/indent.
• getCommandHelp() is now getWorkflowHelp().
  • Return a string.
  • Don't format/indent.
  • Don't include "Supports: svn, git, hg".
• ArcanistConfiguration no longer exists. Its hooks no longer exist, either:
  • willRunWorkflow() no longer exists.
  • didRunWorkflow() no longer exists.
  • didAbortWorkflow() no longer exists.
  • getCustomArgumentsForCommand() no longer exists.
• The Workflow->didParseArguments() callback no longer exists. Just do the parsing in executeWorkflow().
• ArcanistWorkflow->passedArguments no longer exists (no callers).
• ArcanistWorkflow->getPassedArguments() no longer exists (no callers).

When any configuration source specifies an invalid library, we now prompt you to continue without it if stdin is a TTY. This allows you to fix config with arc set-config --local and similar. If stdin is not a TTY, we fatal.

• Removed arcanist_configuration configuration setting. This is a legacy/FB-only option which doesn't fit into the roadmap.
• Removed --conduit-version. We do capability testing, now.
• Removed --conduit-timeout. Not clear this ever really had any callers.
• Removed --skip-arcconfig. Unclear what this was for. FB only option because someone created an invalid file at /home/.arcconfig?
• Removed --arcrc-file. Unclear that this had any real uses. Potentially open to restoring this or similar if there's an automation/bot use for it.
• --config and --load-phutil-library must now appear before other arguments.

Current state of the world here is:

That's likely everything for this week.

Add line numbers and links to the log page with proper copy/paste behavior.

Add line numbers and links to the log page ... This is actually legitimately hard without scanning the entire file. Maybe we can emit a map as part of finalization which shows which has line markers every megabyte or so.

Maybe summarization and highlighting aren't the same object?

Very rough version of the new log: