MSC3958: Suppress notifications from message edits

proposals/3958-suppress-notifications-of-message-edits.md

@@ -0,0 +1,108 @@
+# MSC3958: Suppress notifications from message edits
+
+[Event replacement](https://spec.matrix.org/v1.5/client-server-api/#event-replacements)
+(more commonly known as message edits) signals that a message is intended to
+be replaced with new content.
+
+This works well for fixing typos or other minor correction, but can cause
+spurious notifications if the event mentions a user's display name / localpart or
+if it includes `@room` (which is particularly bad in large rooms as every user
+is re-notified). This contributes to notification fatigue as the additional
+notifications contain no new information.
+
+## Proposal
+
+A new default push rule is added to suppress notifications due to [edits](https://spec.matrix.org/v1.5/client-server-api/#event-replacements).
+
+```json
+{
+    "rule_id": ".m.rule.suppress_edits",
+    "default": true,
+    "enabled": true,
+    "conditions": [
+        {
+            "kind": "event_match",
+            "key": "content.m.relates_to.rel_type",
+            "pattern": "m.replace"
+        }
+    ],
+    "actions": []
+}
+```
+
+This rule should be placed before the [`.m.rule.suppress_notices` rule](https://spec.matrix.org/v1.5/client-server-api/#default-override-rules)
+as the first non-master, non-user added override rule.
+
+It would match events such as those given in [event replacements](https://spec.matrix.org/v1.5/client-server-api/#event-replacements)
+portion of the spec:
+
+```json5
+{
+    "type": "m.room.message",
+    "content": {
+        "body": "* Hello! My name is bar",
+        "msgtype": "m.text",
+        "m.new_content": {
+            "body": "Hello! My name is bar",
+            "msgtype": "m.text"
+        },
+        "m.relates_to": {
+            "rel_type": "m.replace",
+            "event_id": "$some_event_id"
+        }
+    },
+    // ... other fields required by events
+}
+```
+
+## Potential issues
+
+Some users may be depending on notifications of edits. If a user would like to
+revert to the old behavior they can disable the `.m.rule.suppress_edits` push rule.
+
+If the message edits were allowed by other senders than it would be useful to
+know that your own message was edited, but this
+[is not currently allowed](https://spec.matrix.org/v1.5/client-server-api/#validity-of-replacement-events).
+
+The rule is ambiguous (see [MSC3873](https://github.com/matrix-org/matrix-spec-proposals/pull/3873))
+due to the `.` in `m.relates_to` and could also match other, unrelated, events:
+
+```json5
+{
+    "type": "m.room.message",
+    "content": {
+        "body": "* Hello! My name is bar",
+        "msgtype": "m.text",
+        "m.new_content": {
+            "body": "Hello! My name is bar",
+            "msgtype": "m.text"
+        },
+        "m": {
+            "relates_to": {
+                "rel_type": "m.replace",
+                "event_id": "$some_event_id"
+            }
+        }
+    },
+    // ... other fields required by events
+}
+```
+
+(Note that `relates_to` being embedded inside of the `m`.)
+
+## Alternatives
+
+None explored.
+
+## Security considerations
+
+None forseen.
+
+## Unstable prefix
+
+The unstable prefix of `.com.beeper.suppress_edits` should be used in place of
+`.m.rule.suppress_edits`.
+
+## Dependencies
+
+N/A