feat: remove is-commit - will be determined from payload (#282)

mkysel · web-flow · commit a2b24d55883d · 2025-07-17T09:34:09.000-07:00
### Remove `is_commit` field from `AuthenticatedData` message in protocol buffer definition as it will be determined from payload The `is_commit` boolean field (tag 4) is removed from the `AuthenticatedData` message definition in [proto/xmtpv4/envelopes/envelopes.proto](https://github.com/xmtp/proto/pull/282/files#diff-f812a33b13a82e881082c50bb3818e6aad1d114d1141279972acbc496a645fda). A comment is added indicating that tag 4 should not be reused, following the same pattern as the existing comment for tag 1 which was previously used by `target_originator`.
diff --git a/proto/xmtpv4/envelopes/envelopes.proto b/proto/xmtpv4/envelopes/envelopes.proto
@@ -20,7 +20,7 @@ message AuthenticatedData {
   // Do NOT reuse tag 1 — previously used by target_originator
   bytes target_topic = 2;
   Cursor depends_on = 3;
-  bool is_commit = 4;
+  // Do NOT reuse tag 4 — previously used by is_commit
 }
 
 message ClientEnvelope {

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ message AuthenticatedData {`
`20`	`20`	`// Do NOT reuse tag 1 — previously used by target_originator`
`21`	`21`	`bytes target_topic = 2;`
`22`	`22`	`Cursor depends_on = 3;`
`23`		`- bool is_commit = 4;`
	`23`	`+ // Do NOT reuse tag 4 — previously used by is_commit`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`message ClientEnvelope {`