relax exact gateway session resume count match

Requring an exact match causes unexpected invalidations. At least 25%
more guilds are supported in the worst case.

Author: vilgotf

src/main.rs

@@ -38,19 +38,6 @@ async fn main() -> anyhow::Result<()> {
     let info = async { Ok::<_, anyhow::Error>(http.gateway().authed().await?.model().await?) }
         .await
         .context("getting info")?;
-    async {
-        http.interaction(APPLICATION_ID)
-            .set_global_commands(&command::global_commands())
-            .await?;
-        http.interaction(APPLICATION_ID)
-            .set_guild_commands(ADMIN_GUILD_ID, &command::admin_commands(info.shards))
-            .await?;
-        Ok::<_, anyhow::Error>(())
-    }
-    .await
-    .context("putting commands")?;
-    let shard_handles = DashMap::new();
-    context::initialize(http, shard_handles);
 
     // The queue defaults are static and may be incorrect for large or newly
     // restarted bots.
@@ -61,9 +48,25 @@ async fn main() -> anyhow::Result<()> {
         info.session_start_limit.total,
     );
     let config = ConfigBuilder::new(token, INTENTS).queue(queue).build();
-
     let shards = resume::restore(config, info.shards).await;
 
+    async {
+        http.interaction(APPLICATION_ID)
+            .set_global_commands(&command::global_commands())
+            .await?;
+        http.interaction(APPLICATION_ID)
+            .set_guild_commands(
+                ADMIN_GUILD_ID,
+                &command::admin_commands(shards.len() as u32),
+            )
+            .await?;
+        Ok::<_, anyhow::Error>(())
+    }
+    .await
+    .context("putting commands")?;
+    let shard_handles = DashMap::with_capacity(shards.len());
+    context::initialize(http, shard_handles);
+
     let tasks = shards
         .into_iter()
         .map(|shard| tokio::spawn(dispatch::run(shard, event_handler)))

src/resume.rs

@@ -1,4 +1,5 @@
 use serde::{Deserialize, Serialize};
+use std::iter;
 use tokio::fs;
 use twilight_gateway::{Config, ConfigBuilder, Session, Shard, ShardId};
 
@@ -54,37 +55,37 @@ pub async fn save(info: &[Info]) -> anyhow::Result<()> {
 }
 
 /// Restores shard resumption information from the file system.
-pub async fn restore(config: Config, shards: u32) -> Vec<Shard> {
+pub async fn restore(config: Config, recommended_shards: u32) -> Vec<Shard> {
     let info = async {
         let contents = fs::read(INFO_FILE).await?;
         Ok::<_, anyhow::Error>(serde_json::from_slice::<Vec<Info>>(&contents)?)
     }
     .await;
 
-    let shard_ids = (0..shards).map(|shard| ShardId::new(shard, shards));
-
-    // A session may only be successfully resumed if it retains its shard ID, but
-    // Discord may have recommend a different shard count (producing different shard
-    // IDs).
-    let shards: Vec<_> = if let Ok(info) = info
-        && info.len() == shards as usize
+    // The recommended shard count targets 1000 guilds per shard (out of a maximum
+    // of 2500), so it might be different from the previous shard count.
+    let shards = if let Ok(info) = info
+        && recommended_shards / 2 <= info.len() as u32
     {
         tracing::info!("resuming previous gateway sessions");
-        shard_ids
+        let configs = iter::repeat_n(config, info.len())
             .zip(info)
-            .map(|(shard_id, info)| {
-                let builder = ConfigBuilder::from(config.clone()).resume_info(info);
-                Shard::with_config(shard_id, builder.build())
-            })
-            .collect()
+            .map(|(config, info)| ConfigBuilder::from(config).resume_info(info).build());
+        shards(configs).collect()
     } else {
-        shard_ids
-            .map(|shard_id| Shard::with_config(shard_id, config.clone()))
-            .collect()
+        shards(iter::repeat_n(config, recommended_shards as usize)).collect()
     };
 
     // Resumed or not, the saved resume info is now stale.
     _ = fs::remove_file(INFO_FILE).await;
 
     shards
 }
+
+fn shards(configs: impl ExactSizeIterator<Item = Config>) -> impl ExactSizeIterator<Item = Shard> {
+    let total = configs.len().try_into().unwrap();
+    (0..total)
+        .map(move |shard| ShardId::new(shard, total))
+        .zip(configs)
+        .map(|(shard_id, config)| Shard::with_config(shard_id, config))
+}