So, from what I understand, the subcommand name is set by the name of the function in the plugin, when the swh.command() decorator is called.

The PostgresHandler has been dead code for two years, I think we can just remove it rather than shuffling it around.

The fix has been deployed to moma.

I'm not familiar with this code at all, so I apologize in advance for asking dumb questions :)

Just to be clear, the "history truncated" flag stuff can happen later, the new behavior after this minimal change is still better than just crashing

Please clarify the code and the commit message to say that this is handling truncated/shallow histories where a revision's parent is referenced but doesn't exist in the archive.

re-re-re your base, gently down the stream

rebase

This change looks sensible in the context of what's already here, but I'm not sure if favoring the first author over other authors is the best choice here.

This seems to work for me, thanks!

This is missing the addition of swh-loader-npm to the Dockerfile