msggen: Extend support range to v0.10.1 and mark address, added in v23.02

.msggen.json

@@ -2874,7 +2874,7 @@
             "deprecated": null
         },
         "Getinfo.address[]": {
-            "added": "pre-v0.10.1",
+            "added": "v23.02",
             "deprecated": false
         },
         "Getinfo.address[].address": {

cln-grpc/proto/node.proto

@@ -70,7 +70,7 @@ message GetinfoRequest {
 
 message GetinfoResponse {
 	bytes id = 1;
-	string alias = 2;
+	optional string alias = 2;
 	bytes color = 3;
 	uint32 num_peers = 4;
 	uint32 num_pending_channels = 5;

cln-grpc/src/convert.rs

@@ -52,7 +52,7 @@ impl From<responses::GetinfoResponse> for pb::GetinfoResponse {
     fn from(c: responses::GetinfoResponse) -> Self {
         Self {
             id: c.id.serialize().to_vec(), // Rule #2 for type pubkey
-            alias: c.alias, // Rule #2 for type string
+            alias: c.alias, // Rule #2 for type string?
             color: hex::decode(&c.color).unwrap(), // Rule #2 for type hex
             num_peers: c.num_peers, // Rule #2 for type u32
             num_pending_channels: c.num_pending_channels, // Rule #2 for type u32
@@ -65,7 +65,7 @@ impl From<responses::GetinfoResponse> for pb::GetinfoResponse {
             network: c.network, // Rule #2 for type string
             fees_collected_msat: Some(c.fees_collected_msat.into()), // Rule #2 for type msat
             // Field: Getinfo.address[]
-            address: c.address.into_iter().map(|i| i.into()).collect(), // Rule #3 for type GetinfoAddress
+            address: c.address.map(|arr| arr.into_iter().map(|i| i.into()).collect()).unwrap_or(vec![]), // Rule #3
             // Field: Getinfo.binding[]
             binding: c.binding.map(|arr| arr.into_iter().map(|i| i.into()).collect()).unwrap_or(vec![]), // Rule #3
             warning_bitcoind_sync: c.warning_bitcoind_sync, // Rule #2 for type string?

cln-rpc/src/model.rs

@@ -1518,7 +1518,8 @@ pub mod responses {
 	#[derive(Clone, Debug, Deserialize, Serialize)]
 	pub struct GetinfoResponse {
 	    pub id: PublicKey,
-	    pub alias: String,
+	    #[serde(skip_serializing_if = "Option::is_none")]
+	    pub alias: Option<String>,
 	    pub color: String,
 	    pub num_peers: u32,
 	    pub num_pending_channels: u32,
@@ -1532,7 +1533,8 @@ pub mod responses {
 	    pub blockheight: u32,
 	    pub network: String,
 	    pub fees_collected_msat: Amount,
-	    pub address: Vec<GetinfoAddress>,
+	    #[serde(skip_serializing_if = "crate::is_none_or_empty")]
+	    pub address: Option<Vec<GetinfoAddress>>,
 	    #[serde(skip_serializing_if = "crate::is_none_or_empty")]
 	    pub binding: Option<Vec<GetinfoBinding>>,
 	    #[serde(skip_serializing_if = "Option::is_none")]

contrib/msggen/msggen/patch.py

@@ -111,7 +111,7 @@ class OptionalPatch(Patch):
     # Oldest supported versions. Bump this if you no longer want to
     # support older versions, and you want to make required fields
     # more stringent.
-    supported = 'v0.12.0'
+    supported = 'v0.10.1'
 
     def visit(self, f: model.Field) -> None:
         if f.added not in self.versions: