Add targeted build rendering

Add build-only render selection from --only and workspace
[render].only, deduping proposal numbers into ResolvedExecution.

Build OnlyRenderPlan during prepared runtime setup, rewrite omitted
proposal links and requires entries to public URLs, and prune unselected
proposal content before Zola runs.

Restrict targeted rendering to local dirty build mode for now, leaving
targeted serve sync to the next PR. Remove the eipw lint step from the
prepared runtime pipeline so linting is reached only through editorial
commands.

Author: ritorhymes

src/cli.rs

@@ -11,7 +11,7 @@ use std::path::{Path, PathBuf};
 use clap::{Parser, Subcommand};
 use url::Url;
 
-use crate::{lint, print};
+use crate::{lint, print, proposal::ProposalNumber};
 
 /// Build script for Ethereum EIPs and ERCs.
 #[derive(Parser, Debug)]
@@ -66,6 +66,13 @@ pub(crate) struct CleanCliArgs {
     pub(crate) clean: bool,
 }
 
+#[derive(Debug, Clone, Default, PartialEq, Eq, clap::Args)]
+pub(crate) struct OnlyCliArgs {
+    /// Render only the selected proposal number(s)
+    #[arg(long, value_name = "NUMBER", value_parser = ProposalNumber::parse_cli_selector, num_args = 1..)]
+    pub(crate) only: Vec<ProposalNumber>,
+}
+
 #[derive(Debug, Clone, Subcommand)]
 pub(crate) enum Operation {
     /// Print linter schema metadata and lint configuration
@@ -81,6 +88,9 @@ pub(crate) enum Operation {
 
         #[command(flatten)]
         clean: CleanCliArgs,
+
+        #[command(flatten)]
+        only: OnlyCliArgs,
     },
 
     /// Serve the existing built output without rebuilding it
@@ -153,7 +163,7 @@ pub(crate) enum ProfiledOperation {
         base_url: BaseUrlCliArgs,
     },
 
-    /// Build the project and launch a web server to preview it
+    /// Build a fresh temporary site, serve it locally, and watch tracked edits
     Serve {
         #[command(flatten)]
         server: ServerCliArgs,
@@ -272,6 +282,22 @@ impl Operation {
         }
     }
 
+    pub(crate) fn only_cli_args(&self) -> Option<&OnlyCliArgs> {
+        match self {
+            Self::Build { only, .. } => Some(only),
+            Self::Print { .. }
+            | Self::Preview { .. }
+            | Self::Serve { .. }
+            | Self::Clean
+            | Self::Check { .. }
+            | Self::Changed { .. }
+            | Self::Editorial { .. }
+            | Self::Init { .. }
+            | Self::Doctor
+            | Self::Parity { .. } => None,
+        }
+    }
+
     pub(crate) fn is_plain_site_command(&self) -> bool {
         matches!(
             self,
@@ -315,14 +341,14 @@ impl ProfiledOperation {
     fn server_cli_args(&self) -> ServerCliArgs {
         match self {
             Self::Serve { server, .. } => server.clone(),
-            Self::Build { .. } | Self::Check { .. } => ServerCliArgs::default(),
+            Self::Build { .. } | Self::Check => ServerCliArgs::default(),
         }
     }
 
     fn base_url_cli_args(&self) -> BaseUrlCliArgs {
         match self {
             Self::Build { base_url, .. } | Self::Serve { base_url, .. } => base_url.clone(),
-            Self::Check { .. } => BaseUrlCliArgs::default(),
+            Self::Check => BaseUrlCliArgs::default(),
         }
     }
 
@@ -382,6 +408,8 @@ impl EditorialSelectorArgs {
 mod tests {
     use clap::Parser;
 
+    use crate::proposal::ProposalNumber;
+
     use super::{Args, EditorialCommand, Operation, ProfiledOperation, RuntimeOperation};
 
     fn parse_args(arguments: &[&str]) -> Args {
@@ -420,6 +448,54 @@ mod tests {
             .contains(&format!("unexpected argument '{removed_flag}'")));
     }
 
+    #[test]
+    fn only_flag_parses_one_or_more_proposal_numbers_on_build() {
+        let one = parse_args(&["build-eips", "build", "--only", "00555"]);
+        let many = parse_args(&["build-eips", "build", "--only", "555", "678", "897"]);
+
+        match one.operation {
+            Operation::Build { only, .. } => {
+                assert_eq!(only.only, vec![ProposalNumber::from_u32(555).unwrap()]);
+            }
+            other => panic!("unexpected operation: {other:?}"),
+        }
+        match many.operation {
+            Operation::Build { only, .. } => {
+                assert_eq!(
+                    only.only,
+                    vec![
+                        ProposalNumber::from_u32(555).unwrap(),
+                        ProposalNumber::from_u32(678).unwrap(),
+                        ProposalNumber::from_u32(897).unwrap(),
+                    ]
+                );
+            }
+            other => panic!("unexpected operation: {other:?}"),
+        }
+    }
+
+    #[test]
+    fn only_flag_rejects_invalid_selectors_and_non_build_commands() {
+        for selector in [
+            "+555",
+            "0",
+            "-555",
+            "abc",
+            "555,678",
+            "content/00555.md",
+            "4294967296",
+        ] {
+            assert!(
+                Args::try_parse_from(["build-eips", "build", "--only", selector]).is_err(),
+                "expected `{selector}` to be rejected"
+            );
+        }
+
+        assert!(Args::try_parse_from(["build-eips", "serve", "--only", "555"]).is_err());
+        assert!(Args::try_parse_from(["build-eips", "check", "--only", "555"]).is_err());
+        assert!(Args::try_parse_from(["build-eips", "parity", "build", "--only", "555"]).is_err());
+    }
+
     #[test]
     fn base_url_flags_parse_on_build_and_serve_forms() {
         let cases: &[(&[&str], &str)] = &[

src/config.rs

@@ -14,6 +14,8 @@ use serde::{Deserialize, Deserializer, Serialize, Serializer};
 use snafu::{Backtrace, IntoError, OptionExt, ResultExt, Snafu};
 use url::{Position, Url};
 
+use crate::proposal::ProposalNumber;
+
 pub const LOCAL_CONFIG_FILE: &str = ".build-eips.toml";
 pub const REPO_MANIFEST_FILE: &str = ".build-eips.repo.toml";
 pub const DEFAULT_BUILD_ROOT_BASE: &str = ".local-build";
@@ -468,17 +470,31 @@ pub struct WorkspaceConfig {
     /// Local rendered-site URL defaults for build and serve commands.
     #[serde(default)]
     pub site: SiteSettings,
+
+    /// Local render filtering defaults.
+    #[serde(default)]
+    pub render: RenderSettings,
 }
 
 impl WorkspaceConfig {
     fn starter() -> Self {
         Self {
             server: ServerSettings::default(),
             site: SiteSettings::starter(),
+            render: RenderSettings::default(),
         }
     }
 }
 
+/// Local render filtering settings.
+#[derive(Debug, Clone, Default, PartialEq, Eq, Serialize, Deserialize)]
+#[serde(default, deny_unknown_fields)]
+pub struct RenderSettings {
+    /// Proposal numbers to render for applicable local build commands.
+    #[serde(default)]
+    pub only: Vec<ProposalNumber>,
+}
+
 /// Workspace-local bind address defaults for local server commands.
 #[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize)]
 #[serde(default, deny_unknown_fields)]
@@ -642,6 +658,10 @@ impl LoadedWorkspaceConfig {
         &self.config.site
     }
 
+    pub fn render_settings(&self) -> &RenderSettings {
+        &self.config.render
+    }
+
     pub fn local_theme_path(&self) -> PathBuf {
         self.workspace_root.join(DEFAULT_THEME_DIR)
     }
@@ -689,6 +709,7 @@ mod tests {
         RepoManifestError, ServerBinding, ServerSettings, WorkspaceError, DEFAULT_SERVER_HOST,
         DEFAULT_SERVER_PORT, DEFAULT_SITE_BASE_URL, LOCAL_CONFIG_FILE, REPO_MANIFEST_FILE,
     };
+    use crate::proposal::ProposalNumber;
 
     struct TestRepo {
         tempdir: TempDir,
@@ -941,9 +962,10 @@ base_url = "https://staging.example.test/ERCs/"
         assert!(original.contains("port = 1111"));
         assert!(original.contains("[site]"));
         assert!(original.contains(&format!("base_url = \"{DEFAULT_SITE_BASE_URL}\"")));
+        assert!(original.contains("[render]"));
+        assert!(original.contains("only = []"));
         assert!(!original.contains("default_profile"));
         assert!(!original.contains("[profiles"));
-        assert!(!original.contains("[render]"));
     }
 
     #[test]
@@ -1074,6 +1096,72 @@ base_url = "http://127.0.0.1:1111"
 
         assert_eq!(config.server_settings(), &ServerSettings::default());
         assert!(config.site_settings().base_url.is_none());
+        assert!(config.render_settings().only.is_empty());
+    }
+
+    #[test]
+    fn parses_workspace_config_render_only_settings() {
+        let repo = TestRepo::new();
+        let config_path = repo.write_file(
+            LOCAL_CONFIG_FILE,
+            r#"
+[render]
+only = [555, 678, 555]
+"#,
+        );
+
+        let config = LoadedWorkspaceConfig::from_path(&config_path).unwrap();
+
+        assert_eq!(
+            config.render_settings().only,
+            vec![
+                ProposalNumber::from_u32(555).unwrap(),
+                ProposalNumber::from_u32(678).unwrap(),
+                ProposalNumber::from_u32(555).unwrap(),
+            ]
+        );
+    }
+
+    #[test]
+    fn missing_render_missing_only_and_empty_only_disable_filtering() {
+        let cases = [
+            ("missing render", ""),
+            ("missing only", "[render]\n"),
+            ("empty only", "[render]\nonly = []\n"),
+        ];
+
+        for (name, contents) in cases {
+            let repo = TestRepo::new();
+            let config_path = repo.write_file(LOCAL_CONFIG_FILE, contents);
+            let config = LoadedWorkspaceConfig::from_path(&config_path).unwrap();
+
+            assert!(
+                config.render_settings().only.is_empty(),
+                "expected `{name}` to disable render filtering"
+            );
+        }
+    }
+
+    #[test]
+    fn workspace_config_render_only_rejects_non_positive_and_non_integer_values() {
+        let cases = [
+            ("zero", "only = [0]"),
+            ("negative", "only = [-555]"),
+            ("quoted", "only = [\"555\"]"),
+            ("overflow", "only = [4294967296]"),
+        ];
+
+        for (name, contents) in cases {
+            let repo = TestRepo::new();
+            let config_path =
+                repo.write_file(LOCAL_CONFIG_FILE, &format!("[render]\n{contents}\n"));
+            let error = LoadedWorkspaceConfig::from_path(&config_path).unwrap_err();
+
+            assert!(
+                matches!(error, WorkspaceError::Parse { .. }),
+                "expected `{name}` render only config to fail, got {error:?}"
+            );
+        }
     }
 
     #[test]

src/editorial.rs

@@ -573,6 +573,7 @@ base_url = "https://staging.example.test/{sibling_id}/"
                 other_repos: Default::default(),
             },
             theme_path: Some(PathBuf::from("/workspace/theme")),
+            only: None,
             source_materialization: crate::git::SourceMaterialization::Clean,
             server_binding: ServerBinding::default(),
             base_url_override: None,
@@ -756,6 +757,7 @@ base_url = "https://staging.example.test/{sibling_id}/"
                 other_repos: Default::default(),
             },
             theme_path: Some(PathBuf::from("/workspace/theme")),
+            only: None,
             source_materialization: crate::git::SourceMaterialization::Clean,
             server_binding: ServerBinding::default(),
             base_url_override: None,