Fix command plugin when run on targets with dependencies

.licenseignore

@@ -43,3 +43,4 @@ Examples/streaming-chatgpt-proxy/players.txt
 **/Makefile
 **/*.html
 .editorconfig
+IntegrationTest/**/Empty.swift

IntegrationTest/Package.swift

@@ -51,5 +51,12 @@ let package = Package(
             name: "MockTransportServer",
             dependencies: ["Server", .product(name: "OpenAPIRuntime", package: "swift-openapi-runtime")]
         ),
+        // Targets to integration test the command plugin
+        .target(name: "Empty"),
+        .target(name: "TypesAOT", dependencies: [.product(name: "OpenAPIRuntime", package: "swift-openapi-runtime")]),
+        .target(
+            name: "TypesAOTWithDependency",
+            dependencies: ["Empty", .product(name: "OpenAPIRuntime", package: "swift-openapi-runtime")]
+        ),
     ]
 )

IntegrationTest/Sources/TypesAOT/openapi-generator-config.yaml

@@ -0,0 +1,4 @@
+generate:
+  - types
+accessModifier: package
+namingStrategy: idiomatic

IntegrationTest/Sources/TypesAOT/openapi.yaml

@@ -0,0 +1 @@
+../openapi.yaml

IntegrationTest/Sources/TypesAOTWithDependency/openapi-generator-config.yaml

@@ -0,0 +1,4 @@
+generate:
+  - types
+accessModifier: package
+namingStrategy: idiomatic

IntegrationTest/Sources/TypesAOTWithDependency/openapi.yaml

@@ -0,0 +1 @@
+../openapi.yaml

Plugins/OpenAPIGeneratorCommand/plugin.swift

@@ -81,16 +81,23 @@ extension SwiftOpenAPIGeneratorPlugin: CommandPlugin {
                 log("- ✅ OpenAPI code generation for target '\(target.name)' successfully completed.")
                 hadASuccessfulRun = true
             } catch let error as PluginError {
-                if targetNameArguments.isEmpty, case .fileErrors(let errors) = error,
-                    Set(errors.map(\.fileKind)) == Set(FileError.Kind.allCases),
+                if case .fileErrors(let errors) = error, Set(errors.map(\.fileKind)) == Set(FileError.Kind.allCases),
                     errors.map(\.issue).allSatisfy({ $0 == FileError.Issue.noFilesFound })
                 {
-                    // The command plugin was run with no --target argument so its looping over all targets.
-                    // If a target does not have any of the required files, this should only be considered an error
-                    // if the plugin is being explicitly run on a target, either using the build plugin, or using the
-                    // command plugin with a --target argument.
-                    log("- Skipping because target isn't configured for OpenAPI code generation.")
-                    continue
+                    // The error is that neither of the required files are present for code generation for this target.
+                    // This should only be considered an error if this target was explicitly provided as a target for
+                    // code generation with --target.
+                    // We may get this error for other targets if:
+                    //
+                    // 1. The command plugin was run with no --target arguments, in which case the plugin loops over
+                    //    all targets; or
+                    // 2. This target is a dependency of a target that was requested using --target.
+                    //
+                    // In either of these cases, we should not consider this an error and skip the target.
+                    if !targetNameArguments.contains(target.name) {
+                        log("- Skipping because target isn't configured for OpenAPI code generation.")
+                        continue
+                    }
                 }
 
                 if error.isMisconfigurationError {

scripts/run-integration-test.sh

@@ -49,4 +49,22 @@ log "Running command plugin on integration test package: ${INTEGRATION_TEST_PACK
 swift package --package-path "${INTEGRATION_TEST_PACKAGE_PATH}" \
  --allow-writing-to-package-directory generate-code-from-openapi
 
+log "Running command plugin on integration test package AOT target: ${INTEGRATION_TEST_PACKAGE_PATH}"
+swift package --package-path "${INTEGRATION_TEST_PACKAGE_PATH}" \
+ --allow-writing-to-package-directory generate-code-from-openapi \
+ --target TypesAOT
+
+log "Building integration test package AOT target: ${INTEGRATION_TEST_PACKAGE_PATH}"
+swift build --package-path "${INTEGRATION_TEST_PACKAGE_PATH}" \
+ --target TypesAOT
+
+log "Running command plugin on integration test package AOT target with dependency: ${INTEGRATION_TEST_PACKAGE_PATH}"
+swift package --package-path "${INTEGRATION_TEST_PACKAGE_PATH}" \
+ --allow-writing-to-package-directory generate-code-from-openapi \
+ --target TypesAOTWithDependency
+
+log "Building integration test package AOT target with dependency: ${INTEGRATION_TEST_PACKAGE_PATH}"
+swift build --package-path "${INTEGRATION_TEST_PACKAGE_PATH}" \
+ --target TypesAOTWithDependency
+
 log "✅ Successfully built integration test package."