Add an operationId strategy to reject or ignore missing operations

cargo-progenitor/src/main.rs

@@ -9,7 +9,7 @@ use std::{
 use anyhow::{bail, Result};
 use clap::{Parser, ValueEnum};
 use openapiv3::OpenAPI;
-use progenitor::{GenerationSettings, Generator, InterfaceStyle, TagStyle};
+use progenitor::{GenerationSettings, Generator, InterfaceStyle, OperationIdStrategy, TagStyle};
 use progenitor_impl::space_out_items;
 
 fn is_non_release() -> bool {
@@ -51,6 +51,9 @@ struct Args {
     /// SDK tag style
     #[clap(value_enum, long, default_value_t = TagArg::Merged)]
     tags: TagArg,
+    /// SDK operation id style
+    #[clap(value_enum, long, default_value_t = OperationIdStrategyArg::RejectMissing)]
+    operation_id_strategy: OperationIdStrategyArg,
     /// Include client code rather than depending on progenitor-client
     #[clap(default_value = match is_non_release() { true => "true", false => "false" }, long, action = clap::ArgAction::Set)]
     include_client: bool,
@@ -86,6 +89,23 @@ impl From<TagArg> for TagStyle {
     }
 }
 
+#[derive(Copy, Clone, ValueEnum)]
+enum OperationIdStrategyArg {
+    RejectMissing,
+    OmitMissing,
+    GenerateMissing,
+}
+
+impl From<OperationIdStrategyArg> for OperationIdStrategy {
+    fn from(arg: OperationIdStrategyArg) -> Self {
+        match arg {
+            OperationIdStrategyArg::RejectMissing => OperationIdStrategy::RejectMissing,
+            OperationIdStrategyArg::OmitMissing => OperationIdStrategy::OmitMissing,
+            OperationIdStrategyArg::GenerateMissing => OperationIdStrategy::GenerateMissing,
+        }
+    }
+}
+
 fn reformat_code(input: String) -> String {
     let config = rustfmt_wrapper::config::Config {
         normalize_doc_attributes: Some(true),
@@ -119,7 +139,8 @@ fn main() -> Result<()> {
     let mut builder = Generator::new(
         GenerationSettings::default()
             .with_interface(args.interface.into())
-            .with_tag(args.tags.into()),
+            .with_tag(args.tags.into())
+            .with_operation_id_strategy(args.operation_id_strategy.into()),
     );
 
     match builder.generate_tokens(&api) {

cargo-progenitor/tests/test_cmd.rs

@@ -1,11 +1,10 @@
 // Copyright 2025 Oxide Computer Company
 
-use assert_cmd::Command;
+use assert_cmd::cargo::cargo_bin_cmd;
 
 #[test]
 fn test_help() {
-    Command::cargo_bin("cargo-progenitor")
-        .unwrap()
+    cargo_bin_cmd!("cargo-progenitor")
         .arg("progenitor")
         .arg("--help")
         .assert()

progenitor-impl/src/cli.rs

@@ -24,7 +24,7 @@ struct CliOperation {
 impl Generator {
     /// Generate a `clap`-based CLI.
     pub fn cli(&mut self, spec: &OpenAPI, crate_name: &str) -> Result<TokenStream> {
-        validate_openapi(spec)?;
+        validate_openapi(spec, self.settings.operation_id_strategy)?;
 
         // Convert our components dictionary to schemars
         let schemas = spec.components.iter().flat_map(|components| {
@@ -46,8 +46,16 @@ impl Generator {
                     (path.as_str(), method, operation, &item.parameters)
                 })
             })
-            .map(|(path, method, operation, path_parameters)| {
-                self.process_operation(operation, &spec.components, path, method, path_parameters)
+            .filter_map(|(path, method, operation, path_parameters)| {
+                self.process_operation(
+                    operation,
+                    &spec.components,
+                    path,
+                    method,
+                    path_parameters,
+                    self.settings.operation_id_strategy,
+                )
+                .transpose()
             })
             .collect::<Result<Vec<_>>>()?;
 

progenitor-impl/src/httpmock.rs

@@ -30,7 +30,7 @@ impl Generator {
     /// the SDK. This can include `::` and instances of `-` in the crate name
     /// should be converted to `_`.
     pub fn httpmock(&mut self, spec: &OpenAPI, crate_path: &str) -> Result<TokenStream> {
-        validate_openapi(spec)?;
+        validate_openapi(spec, self.settings.operation_id_strategy)?;
 
         // Convert our components dictionary to schemars
         let schemas = spec.components.iter().flat_map(|components| {
@@ -52,8 +52,16 @@ impl Generator {
                     (path.as_str(), method, operation, &item.parameters)
                 })
             })
-            .map(|(path, method, operation, path_parameters)| {
-                self.process_operation(operation, &spec.components, path, method, path_parameters)
+            .filter_map(|(path, method, operation, path_parameters)| {
+                self.process_operation(
+                    operation,
+                    &spec.components,
+                    path,
+                    method,
+                    path_parameters,
+                    self.settings.operation_id_strategy,
+                )
+                .transpose()
             })
             .collect::<Result<Vec<_>>>()?;