Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ratify bot-mode #495

Merged
merged 2 commits into from
Apr 26, 2022
Merged

Ratify bot-mode #495

merged 2 commits into from
Apr 26, 2022

Conversation

progval
Copy link
Contributor

@progval progval commented Apr 12, 2022

Known implementations

Unchecked means incomplete or an intent to implement has been expressed. The numbers indicate minimum requirements. Any others?

Server (3/2)

  • InspIRCd: (@SadieCat ) ircs://testnet.inspircd.org:6697
  • Ergo (@slingamn) ircs://testnet.oragono.io:6697
  • UnrealIRCd (@syzop) ircs://irc.unrealircd.org:6697

Client (2/1)

Bouncer

Bots

Networks

@dgw
Copy link
Contributor

dgw commented Apr 12, 2022

Sopel implements both the mode (sopel-irc/sopel#2088) and tag (sopel-irc/sopel#2089).

Yes, it accepts the unprefixed tag. Figured this spec would be ratified long before we made it to release, and it seems I'm about to be proven correct.

@SadieCat
Copy link
Contributor

I can push my support for this the moment its ratified. 👍🏻

@SadieCat
Copy link
Contributor

Is there any objections to ratifying this? I'd like it if I could get our implementation into InspIRCd v3.13.0 which is due to be released next Friday.

@jwheare
Copy link
Member

jwheare commented Apr 25, 2022

There was an objection raised in IRC from solanum/libera staff. As I understand it, they felt that IRCv3 shouldn't be pursuing specs that feel like a stop gap for a true metadata implementation.

This spec doesn't define a mechanism for keeping up to date with a change in a user's bot status other than periodic WHOIS or WHO polling. True metadata notifications on status changes would allow clients to keep member lists in sync.

In reality, a real bot is unlikely to ever change status, so I feel like running an initial WHO check on channel join and checking for the tag on new user JOINs should be sufficient for the vast majority of use cases.

There are outstanding issues with PMs when no shared channel exists, but those issues extend to many other things, and a future METADATA spec won't necessarily cater to that situation either.

I acknowledge and agree that effort should be spent on reviving a true metadata spec, so that we can focus in that direction for future cosmetic user status things, . But I think for this specific spec, where there is prior art for non-standard mode/whois numerics indicating bot status, and we have a broad consensus of implementation, we should ratify this.

It might be worth mentioning the prior art in the spec, similar to the Motivation section in the SETNAME spec: https://ircv3.net/specs/extensions/setname.html#motivation

See initial discussion here: ircv3/ircv3-ideas#43

Copy link
Member

@jwheare jwheare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How's this?

extensions/bot-mode.md Outdated Show resolved Hide resolved
Co-authored-by: James Wheare <james@wheare.org>
@jwheare jwheare merged commit 446235c into ircv3:master Apr 26, 2022
@jwheare
Copy link
Member

jwheare commented Apr 26, 2022

Thanks for feedback and implementations all.

dgw added a commit to sopel-irc/sopel that referenced this pull request Apr 26, 2022
Bot Mode spec has been ratified (ircv3/ircv3-specifications#495). Since
we haven't published a release with support for this message tag yet,
it's a very easy decision to take *right now* that we'll only support
the finalized spec and not the WIP version.
@progval progval deleted the ratify-bot-mode branch June 25, 2022 17:08
progval added a commit to progval/unrealircd that referenced this pull request Jul 2, 2022
https://ircv3.net/specs/extensions/bot-mode was ratified a few months ago
(ircv3/ircv3-specifications#495)

This commit keeps the draft mtag, for now.
progval added a commit to progval/unrealircd that referenced this pull request Jul 2, 2022
https://ircv3.net/specs/extensions/bot-mode was ratified a few months ago
(ircv3/ircv3-specifications#495)

This commit keeps the draft mtag in addition to the stable one, for now.
syzop pushed a commit to unrealircd/unrealircd that referenced this pull request Nov 14, 2022
https://ircv3.net/specs/extensions/bot-mode was ratified a few months ago
(ircv3/ircv3-specifications#495)

This commit keeps the draft mtag in addition to the stable one, for now.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants