From 9b14d4881ddabc0411c90f51439af59b004c9cd5 Mon Sep 17 00:00:00 2001
From: 魏曹先生 <1992414357@qq.com>
Date: Mon, 3 Aug 2026 01:35:20 +0800
Subject: feat(ci): build workspace with all documented features

Use `--workspace` for build, clippy, and test tasks to cover
feature-gated code instead of relying on default features. Skip
workspace members in individual crate tasks to avoid redundant runs.
---
 .run/src/bin/ci.rs | 90 +++++++++++++++++++++++++++++++++++++++++++++++++++---
 1 file changed, 86 insertions(+), 4 deletions(-)

diff --git a/.run/src/bin/ci.rs b/.run/src/bin/ci.rs
index 4b6f973..d8941d4 100644
--- a/.run/src/bin/ci.rs
+++ b/.run/src/bin/ci.rs
@@ -1,4 +1,5 @@
 use std::io::Write as _;
+use std::path::{Path, PathBuf};
 use std::process::exit;
 
 use arg_picker::{Picker, macros::arg};
@@ -167,16 +168,56 @@ fn test_docs_code_blocks() -> Result<(), i32> {
     )
 }
 
+/// Returns the manifest paths of all workspace members (via `cargo metadata --no-deps`).
+///
+/// These crates are tested/built/clipped together with `--workspace` so that
+/// feature-gated code is covered, instead of relying on each crate's default features.
+fn workspace_manifests() -> Vec<PathBuf> {
+    let Ok(output) = tools::run_cmd_capture("cargo metadata --no-deps --format-version 1") else {
+        return Vec::new();
+    };
+    let Ok(json) = serde_json::from_str::<serde_json::Value>(&output) else {
+        return Vec::new();
+    };
+    json["packages"]
+        .as_array()
+        .into_iter()
+        .flatten()
+        .filter_map(|p| p["manifest_path"].as_str().map(PathBuf::from))
+        .collect()
+}
+
+fn same_path(a: &Path, b: &Path) -> bool {
+    let norm = |p: &Path| std::fs::canonicalize(p).unwrap_or_else(|_| p.to_path_buf());
+    norm(a) == norm(b)
+}
+
 fn build_all() -> Result<(), i32> {
     let ignore_dirs = get_ignore_dirs();
     let cargo_tomls = cargo_tomls();
+    let workspace_manifests = workspace_manifests();
     let mut tasks = Vec::new();
+
+    // Workspace members: build with all documented features (same set used by cov-test)
+    let features_arg = doc_features_arg();
+    tasks.push((
+        "Build: workspace".to_string(),
+        "workspace".to_string(),
+        format!("cargo build --workspace{features_arg} --color always"),
+    ));
+
     for cargo_toml in cargo_tomls {
-        let path = cargo_toml.parent().unwrap_or(std::path::Path::new(""));
+        let path = cargo_toml.parent().unwrap_or(Path::new(""));
         let path_str = path.to_string_lossy();
         if ignore_dirs.iter().any(|d| path_str.contains(d.as_str())) {
             continue;
         }
+        if workspace_manifests
+            .iter()
+            .any(|m| same_path(m, &cargo_toml))
+        {
+            continue;
+        }
         let label = format!("Build: {}", cargo_toml.to_string_lossy());
         let crate_name = crate_name_from(&cargo_toml);
         let cmd = format!(
@@ -191,13 +232,29 @@ fn build_all() -> Result<(), i32> {
 fn clippy_all() -> Result<(), i32> {
     let ignore_dirs = get_ignore_dirs();
     let cargo_tomls = cargo_tomls();
+    let workspace_manifests = workspace_manifests();
     let mut tasks = Vec::new();
+
+    // Workspace members: clippy with all documented features
+    let features_arg = doc_features_arg();
+    tasks.push((
+        "Clippy: workspace".to_string(),
+        "workspace".to_string(),
+        format!("cargo clippy --workspace{features_arg} --color always -- -D warnings"),
+    ));
+
     for cargo_toml in cargo_tomls {
-        let path = cargo_toml.parent().unwrap_or(std::path::Path::new(""));
+        let path = cargo_toml.parent().unwrap_or(Path::new(""));
         let path_str = path.to_string_lossy();
         if ignore_dirs.iter().any(|d| path_str.contains(d.as_str())) {
             continue;
         }
+        if workspace_manifests
+            .iter()
+            .any(|m| same_path(m, &cargo_toml))
+        {
+            continue;
+        }
         let label = format!("Clippy: {}", cargo_toml.to_string_lossy());
         let crate_name = crate_name_from(&cargo_toml);
         let cmd = format!(
@@ -209,17 +266,42 @@ fn clippy_all() -> Result<(), i32> {
     run_parallel("Clippy", tasks)
 }
 
+/// ` --features "<docs.rs features>"` (empty string when unavailable)
+fn doc_features_arg() -> String {
+    match tools::read_features() {
+        Ok(features) if !features.is_empty() => format!(" --features \"{}\"", features.join(",")),
+        _ => String::new(),
+    }
+}
+
 fn test_all() -> Result<(), i32> {
     let ignore_dirs = get_ignore_dirs();
     let cargo_tomls = cargo_tomls();
+    let workspace_manifests = workspace_manifests();
     let mut tasks = Vec::new();
+
+    // Workspace members: test with all documented features so that feature-gated
+    // tests (comp/repl/picker/structural_renderer/...) are actually executed.
+    let features_arg = doc_features_arg();
+    tasks.push((
+        "Test: workspace".to_string(),
+        "workspace".to_string(),
+        format!("cargo test --workspace{features_arg} --color always"),
+    ));
+
     for cargo_toml in cargo_tomls {
-        let path = cargo_toml.parent().unwrap_or(std::path::Path::new(""));
+        let path = cargo_toml.parent().unwrap_or(Path::new(""));
         let path_str = path.to_string_lossy();
         if ignore_dirs.iter().any(|d| path_str.contains(d.as_str())) {
             continue;
         }
-        let label = format!("Testing: {}", cargo_toml.to_string_lossy());
+        if workspace_manifests
+            .iter()
+            .any(|m| same_path(m, &cargo_toml))
+        {
+            continue;
+        }
+        let label = format!("Test: {}", cargo_toml.to_string_lossy());
         let crate_name = crate_name_from(&cargo_toml);
         let cmd = format!(
             "cargo test --manifest-path {} --color always",
-- 
cgit