[mastodon] add "instance_remote" field

[allendema]

Example usage:

If the url is mastodon:https://social.linux.pizza/@[email protected]
the "remote_instance" will be "botsin.space".

"directory": ["mastodon", "{instance_remote|instance}", "{account[username]!l}"]

[mikf]

Suggested change

	if "@" in status["account"]["acct"]:
	status["remote_instance"] = status["account"]["acct"].split('@')[1]
	else:
	status["remote_instance"] = None
	acct = status["account"]["acct"]
	status["remote_instance"] = \
	acct.rpartition("@")[2] if "@" in acct else None

• Accessing status["account"]["acct"] multiple times is slower than storing it in a local variable
• I prefer (r)partition over split. It's usually even faster.

[allendema]

Thanks for you suggestions. I had messed up flake8 with GH GUI.

[mikf]

Do you think it'd be better to call this field instance_remote instead?
(better when metadata field names get listed alphabetically)

And what about only defining it when a post/status is from a remote instance?

[allendema]

call this field instance_remote instead?

That is also fine.

And what about only defining it when a post/status is from a remote instance?

Would that hide the field from the -K option if its a local instance - so nobody will use this field?

[mikf]

Would that hide the field from the -K option if its a local instance - so nobody will use this field?

It would ... OK, lets forget about that idea then.
I'd still like to call it instance_remote.