refactor(cli): rename outdated --system to --all

The scan never leaves the user account, so --system overpromised. --all
matches the behavior: both scopes and every Agent target directory.

Author: harlan-zw

GLOSSARY.md

@@ -33,6 +33,7 @@ Every public export, command, error, route, and document uses these terms.
 | `skilld update --check --json` | update relation check |
 | `skilld verify` | source verification |
 | `skilld outdated` | outdated Skill report |
+| `skilld outdated --all` | system-wide outdated Skill report |
 | `skilld install skilld --global` | global skilld Skill install |
 | `skilld auth login` | account login |
 | `skilld auth status` | account authentication status |

crates/skilld-command/src/lib.rs

@@ -133,9 +133,9 @@ enum Command {
     Verify { skill: Option<String> },
     /// Report outdated and unmanaged Skills.
     Outdated {
-        /// Scan every Agent target on this system.
+        /// Check both scopes and every Agent target directory.
         #[arg(long)]
-        system: bool,
+        all: bool,
     },
     /// Manage account authentication.
     Auth {
@@ -229,7 +229,7 @@ pub trait Host {
         ))
     }
 
-    fn outdated(&self, _system: bool) -> Result<Vec<String>, CommandError> {
+    fn outdated(&self, _all: bool) -> Result<Vec<String>, CommandError> {
         Err(CommandError::unsupported_host(
             "Outdated Skill reports are unavailable on this host",
         ))
@@ -794,7 +794,7 @@ fn dispatch<H: Host>(command: Command, host: &H) -> Result<CommandOutput, Comman
             }
         }
         Command::Verify { skill } => host.verify(skill.as_deref()).map(CommandOutput::Lines),
-        Command::Outdated { system } => host.outdated(system).map(CommandOutput::Lines),
+        Command::Outdated { all } => host.outdated(all).map(CommandOutput::Lines),
     }
 }
 
@@ -1734,8 +1734,8 @@ impl Host for LocalHost {
         Ok(UpdatePlanV1::new(plan))
     }
 
-    fn outdated(&self, system: bool) -> Result<Vec<String>, CommandError> {
-        let scopes = if system {
+    fn outdated(&self, all: bool) -> Result<Vec<String>, CommandError> {
+        let scopes = if all {
             vec![InstallScope::Project, InstallScope::Global]
         } else {
             vec![InstallScope::Project]
@@ -1759,7 +1759,7 @@ impl Host for LocalHost {
                         scope.as_str(),
                         CommandError::store(error).message
                     ));
-                    if system {
+                    if all {
                         // The ancestor scan must not report Skills this scope cannot verify.
                         suppressed_roots.extend(known.iter().map(|target| target.root.clone()));
                     }
@@ -1792,12 +1792,12 @@ impl Host for LocalHost {
                 progress.found(&format!("{name} ({} scope)", scope.as_str()));
                 views.push((view, scope));
             }
-            if system {
+            if all {
                 store_roots.push(store.root().to_path_buf());
                 scan.push((scope, known));
             }
         }
-        if system {
+        if all {
             // Global Agent directories keep the global scope, so ancestor roots
             // scan after them and skip roots another scope already claimed.
             let mut claimed = scan
@@ -1816,7 +1816,7 @@ impl Host for LocalHost {
                 }
             }
         }
-        let unmanaged = if system {
+        let unmanaged = if all {
             outdated::scan_unmanaged(&scan, &store_roots, &managed)
         } else {
             Vec::new()
@@ -1828,7 +1828,7 @@ impl Host for LocalHost {
             progress.checking(&view.name);
             lines.extend(self.report_outdated_view(view, *scope));
         }
-        if system {
+        if all {
             #[cfg(not(target_os = "wasi"))]
             let results = search_candidates_parallel(self, &unmanaged);
             #[cfg(target_os = "wasi")]

crates/skilld-command/tests/outdated.rs

@@ -313,7 +313,7 @@ fn outdated_system_reports_a_stale_global_skill_with_the_global_update() {
     let mut stderr = Vec::new();
 
     let result = run(
-        ["skilld", "outdated", "--system"],
+        ["skilld", "outdated", "--all"],
         &host,
         &mut stdout,
         &mut stderr,
@@ -340,7 +340,7 @@ fn outdated_system_links_unmanaged_skills_to_a_repository() {
     let mut stderr = Vec::new();
 
     let result = run(
-        ["skilld", "outdated", "--system"],
+        ["skilld", "outdated", "--all"],
         &host,
         &mut stdout,
         &mut stderr,
@@ -369,7 +369,7 @@ fn outdated_system_reports_unmanaged_skills_without_a_match() {
     let mut stderr = Vec::new();
 
     let result = run(
-        ["skilld", "outdated", "--system"],
+        ["skilld", "outdated", "--all"],
         &host,
         &mut stdout,
         &mut stderr,
@@ -398,7 +398,7 @@ fn outdated_system_surfaces_a_search_failure_and_keeps_scanning() {
     let mut stderr = Vec::new();
 
     let result = run(
-        ["skilld", "outdated", "--system"],
+        ["skilld", "outdated", "--all"],
         &host,
         &mut stdout,
         &mut stderr,
@@ -426,7 +426,7 @@ fn outdated_system_reports_a_managed_skill_once() {
     let mut stderr = Vec::new();
 
     let result = run(
-        ["skilld", "outdated", "--system"],
+        ["skilld", "outdated", "--all"],
         &host,
         &mut stdout,
         &mut stderr,
@@ -533,7 +533,7 @@ fn outdated_system_survives_a_corrupt_global_store() {
     let mut stderr = Vec::new();
 
     let result = run(
-        ["skilld", "outdated", "--system"],
+        ["skilld", "outdated", "--all"],
         &host,
         &mut stdout,
         &mut stderr,
@@ -565,7 +565,7 @@ fn outdated_system_groups_agents_sharing_one_directory() {
     let mut stderr = Vec::new();
 
     let result = run(
-        ["skilld", "outdated", "--system"],
+        ["skilld", "outdated", "--all"],
         &host,
         &mut stdout,
         &mut stderr,
@@ -637,7 +637,7 @@ fn outdated_reports_the_bundled_skill_by_its_source() {
     let mut stderr = Vec::new();
 
     let result = run(
-        ["skilld", "outdated", "--system"],
+        ["skilld", "outdated", "--all"],
         &host,
         &mut stdout,
         &mut stderr,
@@ -671,7 +671,7 @@ fn outdated_system_runs_candidate_searches_in_parallel_with_a_bound() {
 
     let started = Instant::now();
     let result = run(
-        ["skilld", "outdated", "--system"],
+        ["skilld", "outdated", "--all"],
         &host,
         &mut stdout,
         &mut stderr,
@@ -726,7 +726,7 @@ fn outdated_system_finds_skills_in_parent_directories() {
     let mut stderr = Vec::new();
 
     let result = run(
-        ["skilld", "outdated", "--system"],
+        ["skilld", "outdated", "--all"],
         &host,
         &mut stdout,
         &mut stderr,
@@ -782,7 +782,7 @@ fn outdated_reports_found_skills_before_remote_checks() {
     let mut stderr = Vec::new();
 
     let result = run(
-        ["skilld", "outdated", "--system"],
+        ["skilld", "outdated", "--all"],
         &host,
         &mut stdout,
         &mut stderr,