From 296d49f95c243aa6f2d20329d87c1e4f41b1c73d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E5=AD=99=E6=8C=AF=E5=AE=87?= <>
Date: Tue, 4 Feb 2025 15:16:36 +0800
Subject: [PATCH] refactor(executeFreeleapsPipeline): restructure
 generateComponentStages to improve error handling and encapsulate steps
 within a script block
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: 孙振宇 <>
---
 .../vars/executeFreeleapsPipeline.groovy      | 36 ++++++++++---------
 1 file changed, 20 insertions(+), 16 deletions(-)

diff --git a/first-class-pipeline/vars/executeFreeleapsPipeline.groovy b/first-class-pipeline/vars/executeFreeleapsPipeline.groovy
index b8cfd61a..11ba9556 100644
--- a/first-class-pipeline/vars/executeFreeleapsPipeline.groovy
+++ b/first-class-pipeline/vars/executeFreeleapsPipeline.groovy
@@ -48,25 +48,29 @@ def generateComponentStages(component, configurations) {
         ]
       ) {
         node("dep-resolver-${component.name}") {
-          catchError(buildResult: 'FAILURE', stageResult: 'FAILURE') {
-            if (env.executeMode == "fully" || env.changedComponents.contains(component.name)) {
-              log.info("Pipeline", "Using ${env.buildAgentImage} as build agent image for dependencies resolving")
-              def sourceFetcher = new SourceFetcher(this)
-              sourceFetcher.fetch(configurations)
+          steps {
+            script {
+              catchError(buildResult: 'FAILURE', stageResult: 'FAILURE') {
+                if (env.executeMode == "fully" || env.changedComponents.contains(component.name)) {
+                  log.info("Pipeline", "Using ${env.buildAgentImage} as build agent image for dependencies resolving")
+                  def sourceFetcher = new SourceFetcher(this)
+                  sourceFetcher.fetch(configurations)
 
-              def language = ServiceLanguage.parse(component.language)
-              def depManager = DependenciesManager.parse(component.dependenciesManager)
+                  def language = ServiceLanguage.parse(component.language)
+                  def depManager = DependenciesManager.parse(component.dependenciesManager)
 
-              def dependenciesResolver = new DependenciesResolver(this, language, env.workspace + "/" + component.root + "/")
-              dependenciesResolver.useManager(depManager)
-              
-              if (component.buildCacheEnabled) {
-                  dependenciesResolver.enableCachingSupport()
-              } else {
-                  dependenciesResolver.disableCachingSupport()
+                  def dependenciesResolver = new DependenciesResolver(this, language, env.workspace + "/" + component.root + "/")
+                  dependenciesResolver.useManager(depManager)
+                  
+                  if (component.buildCacheEnabled) {
+                      dependenciesResolver.enableCachingSupport()
+                  } else {
+                      dependenciesResolver.disableCachingSupport()
+                  }
+
+                  dependenciesResolver.resolve(component)
+                }
               }
-
-              dependenciesResolver.resolve(component)
             }
           }
         }