rust-lang · nikomatsakis · Feb 28, 2025 · Feb 28, 2025 · Feb 28, 2025 · Feb 28, 2025
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/book.toml b/book.toml
@@ -23,6 +23,10 @@ command = "cargo run -p mdbook-goals --"
 [preprocessor.goals.users]
 "@Nadrieril" = "@Nadrieril"
 
+preprocessor.goals.ignore_users = [
+    "@triagebot",
+]
+
 
 [output.html]
 git-repository-url = "https://github.com/rust-lang/rust-project-goals"

diff --git a/crates/mdbook-goals/src/mdbook_preprocessor.rs b/crates/mdbook-goals/src/mdbook_preprocessor.rs
@@ -20,6 +20,7 @@ use rust_project_goals::{
 const LINKS: &str = "links";
 const LINKIFIERS: &str = "linkifiers";
 const USERS: &str = "users";
+const IGNORE_USERS: &str = "ignore_users";
 
 pub struct GoalPreprocessor;
 
@@ -45,6 +46,7 @@ pub struct GoalPreprocessorWithContext<'c> {
     links: Vec<(String, String)>,
     linkifiers: Vec<(Regex, String)>,
     display_names: BTreeMap<String, Rc<String>>,
+    ignore_users: Vec<String>,
     goal_document_map: BTreeMap<PathBuf, Arc<Vec<GoalDocument>>>,
 }
 
@@ -55,6 +57,7 @@ impl<'c> GoalPreprocessorWithContext<'c> {
         let mut links: Vec<(String, String)> = Default::default();
         let mut linkifiers = Default::default();
         let mut display_names: BTreeMap<String, Rc<String>> = Default::default();
+        let mut ignore_users: Vec<String> = Default::default();
         if let Some(config) = ctx.config.get_preprocessor(GoalPreprocessor.name()) {
             if let Some(value) = config.get(LINKS) {
                 links = value
@@ -112,6 +115,24 @@ impl<'c> GoalPreprocessorWithContext<'c> {
                     display_names.insert(user, Rc::new(display_name));
                 }
             }
+
+            if let Some(value) = config.get(IGNORE_USERS) {
+                ignore_users = value
+                    .as_array()
+                    .with_context(|| format!("`{}` must be an array", IGNORE_USERS))?
+                    .iter()
+                    .map(|v| {
+                        if let Some(v) = v.as_str() {
+                            Ok(v.to_string())
+                        } else {
+                            Err(anyhow::anyhow!(
+                                "ignore user value `{}` must be a string",
+                                v
+                            ))
+                        }
+                    })
+                    .collect::<anyhow::Result<Vec<_>>>()?;
+            }
         }
 
         Ok(GoalPreprocessorWithContext {
@@ -122,6 +143,7 @@ impl<'c> GoalPreprocessorWithContext<'c> {
             links,
             linkifiers,
             display_names,
+            ignore_users,
             goal_document_map: Default::default(),
         })
     }
@@ -312,6 +334,7 @@ impl<'c> GoalPreprocessorWithContext<'c> {
             .username
             .find_iter(&chapter.content)
             .map(|m| m.as_str().to_string())
+            .filter(|username| !self.ignore_users.contains(username))
             .collect();
 
         for username in &usernames {

diff --git a/crates/rust-project-goals-cli-llm/src/updates.rs b/crates/rust-project-goals-cli-llm/src/updates.rs
@@ -123,9 +123,9 @@ async fn prepare_goals(
         let why_this_goal = why_this_goal(&issue_id, issue)?;
 
         let details_summary = match comments.len() {
-            0 => String::from("No updates posted."),
-            1 => String::from("1 update posted."),
-            len => format!("{len} updates posted."),
+            0 => String::from("No detailed updates available."),
+            1 => String::from("1 detailed update available."),
+            len => format!("{len} detailed updates availabled."),
         };
         result.push(UpdatesGoal {
             title: title.clone(),

diff --git a/crates/rust-project-goals/Cargo.toml b/crates/rust-project-goals/Cargo.toml
@@ -6,7 +6,6 @@ edition = "2021"
 [dependencies]
 anyhow = "1.0.94"
 chrono = "0.4.39"
-disk-persist = "0.1.0"
 lazy_static = "1.5.0"
 regex = "1.11.1"
 reqwest = { version = "0.12.9", features = ["blocking", "json"] }

diff --git a/crates/rust-project-goals/src/util.rs b/crates/rust-project-goals/src/util.rs
@@ -4,7 +4,6 @@ use std::{
     path::{Path, PathBuf},
 };
 
-use disk_persist::DiskPersist;
 use walkdir::WalkDir;
 
 pub const ARROW: &str = "↳";
@@ -59,20 +58,7 @@ pub struct GithubUserInfo {
 
 impl GithubUserInfo {
     pub fn load(login: &str) -> anyhow::Result<Self> {
-        let path = PathBuf::from("gh-cache").join(format!("{}.bincode", login));
-        let persist = DiskPersist::init_with_path(&path)?;
-        if let Some(info) = persist.read()? {
-            Ok(info)
-        } else {
-            let info = Self::github_request(login)?;
-            persist.write(&info)?;
-            eprintln!(
-                "cached info for `{}` from github in `{}`",
-                login,
-                path.display()
-            );
-            Ok(info)
-        }
+        Self::github_request(login)
     }
 
     fn github_request(login: &str) -> anyhow::Result<Self> {

diff --git a/gh-cache/@1c3t3a.bincode b/gh-cache/@1c3t3a.bincode
diff --git a/gh-cache/@BoxyUwU.bincode b/gh-cache/@BoxyUwU.bincode
diff --git a/gh-cache/@Darksonn.bincode b/gh-cache/@Darksonn.bincode
diff --git a/gh-cache/@Deleted.bincode b/gh-cache/@Deleted.bincode
diff --git a/gh-cache/@GuillaumeGomez.bincode b/gh-cache/@GuillaumeGomez.bincode
diff --git a/gh-cache/@Jamesbarford.bincode b/gh-cache/@Jamesbarford.bincode
diff --git a/gh-cache/@Kobzol.bincode b/gh-cache/@Kobzol.bincode
diff --git a/gh-cache/@Muscraft.bincode b/gh-cache/@Muscraft.bincode
diff --git a/gh-cache/@SparrowLii.bincode b/gh-cache/@SparrowLii.bincode
diff --git a/gh-cache/@ZuseZ4.bincode b/gh-cache/@ZuseZ4.bincode
diff --git a/gh-cache/@adetaylor.bincode b/gh-cache/@adetaylor.bincode
diff --git a/gh-cache/@amandasystems.bincode b/gh-cache/@amandasystems.bincode
diff --git a/gh-cache/@baumanj.bincode b/gh-cache/@baumanj.bincode
diff --git a/gh-cache/@blyxyas.bincode b/gh-cache/@blyxyas.bincode
diff --git a/gh-cache/@camelid.bincode b/gh-cache/@camelid.bincode
diff --git a/gh-cache/@compiler-errors.bincode b/gh-cache/@compiler-errors.bincode
diff --git a/gh-cache/@dingxiangfei2009.bincode b/gh-cache/@dingxiangfei2009.bincode
diff --git a/gh-cache/@eh2406.bincode b/gh-cache/@eh2406.bincode
diff --git a/gh-cache/@eholk.bincode b/gh-cache/@eholk.bincode
diff --git a/gh-cache/@epage.bincode b/gh-cache/@epage.bincode
diff --git a/gh-cache/@estebank.bincode b/gh-cache/@estebank.bincode
diff --git a/gh-cache/@fee1-dead.bincode b/gh-cache/@fee1-dead.bincode
diff --git a/gh-cache/@folkertdev.bincode b/gh-cache/@folkertdev.bincode
diff --git a/gh-cache/@ghost.bincode b/gh-cache/@ghost.bincode
diff --git a/gh-cache/@harmou01.bincode b/gh-cache/@harmou01.bincode
diff --git a/gh-cache/@hi-rustin.bincode b/gh-cache/@hi-rustin.bincode
diff --git a/gh-cache/@jaisnan.bincode b/gh-cache/@jaisnan.bincode
diff --git a/gh-cache/@jdonszelmann.bincode b/gh-cache/@jdonszelmann.bincode
diff --git a/gh-cache/@jkelleyrtp.bincode b/gh-cache/@jkelleyrtp.bincode
diff --git a/gh-cache/@joshtriplett.bincode b/gh-cache/@joshtriplett.bincode
diff --git a/gh-cache/@kernel.bincode b/gh-cache/@kernel.bincode
diff --git a/gh-cache/@lcnr.bincode b/gh-cache/@lcnr.bincode
diff --git a/gh-cache/@lqd.bincode b/gh-cache/@lqd.bincode
diff --git a/gh-cache/@matthewjasper.bincode b/gh-cache/@matthewjasper.bincode
diff --git a/gh-cache/@mrkajetanp.bincode b/gh-cache/@mrkajetanp.bincode
diff --git a/gh-cache/@nbdd0121.bincode b/gh-cache/@nbdd0121.bincode
diff --git a/gh-cache/@nikomatsakis.bincode b/gh-cache/@nikomatsakis.bincode
diff --git a/gh-cache/@notriddle.bincode b/gh-cache/@notriddle.bincode
diff --git a/gh-cache/@obi1kenobi.bincode b/gh-cache/@obi1kenobi.bincode
diff --git a/gh-cache/@oli-obk.bincode b/gh-cache/@oli-obk.bincode
diff --git a/gh-cache/@petrochenkov.bincode b/gh-cache/@petrochenkov.bincode
diff --git a/gh-cache/@pnkfelix.bincode b/gh-cache/@pnkfelix.bincode
diff --git a/gh-cache/@spastorino.bincode b/gh-cache/@spastorino.bincode
diff --git a/gh-cache/@tautschnig.bincode b/gh-cache/@tautschnig.bincode
diff --git a/gh-cache/@tmandry.bincode b/gh-cache/@tmandry.bincode
diff --git a/gh-cache/@traviscross.bincode b/gh-cache/@traviscross.bincode
diff --git a/gh-cache/@triagebot.bincode b/gh-cache/@triagebot.bincode
diff --git a/gh-cache/@vabr-g.bincode b/gh-cache/@vabr-g.bincode
diff --git a/gh-cache/@veluca93.bincode b/gh-cache/@veluca93.bincode
diff --git a/gh-cache/@walterhpearce.bincode b/gh-cache/@walterhpearce.bincode
diff --git a/gh-cache/@weihanglo.bincode b/gh-cache/@weihanglo.bincode
diff --git a/src/admin/author_updates.md b/src/admin/author_updates.md
@@ -5,7 +5,7 @@
 Triagebot can ping project-goal owners for updates. To use it, go to Zulip and execute a command like this (you need to use an `@` in front of triagebot).
 
 ```
-triagebot ping-goals 14 Oct-21
+@triagebot ping-goals 14 Oct-21
 ```
 
 The first number (14) is a threshold, it is typically set to the current day of the month (e.g., the above command assumes it is Oct 14). It means "if they have posted a comment in the last 14 days, don't bug them".