Chore: fix definition parse logic and allow if/for comprehension & use op.#Suspend for deploy

[Somefive]

…e op.#Suspend for deploy

1. Fix definition parse logic to support if/for comprehension under template field.
2. Use op.#Suspend for deploy workflowstep.

Now deploy.cue is like

import (
	"vela/op"
)

"deploy": {
	type: "workflow-step"
	annotations: {
		"category": "Application Delivery"
	}
	labels: {
		"scope": "Application"
	}
	description: "A powerful and unified deploy step for components multi-cluster delivery with policies."
}
template: {
	if parameter.auto == false {
		suspend: op.#Suspend & {message: "Waiting approval to the deploy step \"\(context.stepName)\""}
	}
	deploy: op.#Deploy & {
		policies:                 parameter.policies
		parallelism:              parameter.parallelism
		ignoreTerraformComponent: parameter.ignoreTerraformComponent
	}
	parameter: {
		//+usage=If set to false, the workflow will suspend automatically before this step, default to be true.
		auto: *true | bool
		//+usage=Declare the policies that used for this deployment. If not specified, the components will be deployed to the hub cluster.
		policies: *[] | [...string]
		//+usage=Maximum number of concurrent delivered components.
		parallelism: *5 | int
		//+usage=If set false, this step will apply the components with the terraform workload.
		ignoreTerraformComponent: *true | bool
	}
}

Description of your changes

Fixes #

I have:

• Read and followed KubeVela's contribution process.
• Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
• Run make reviewable to ensure this PR is ready for review.
• Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Special notes for your reviewer

[github-advanced-security]

You have successfully added a new Trivy configuration .github/workflows/trivy-scan.yml:images. As part of the setup process, we have scanned this repository and found 5 existing alerts. Please check the repository Security tab to see all alerts.

[codecov]

Codecov Report

Patch coverage: 50.00% and project coverage change: -0.14 ⚠️

Comparison is base (8619f1c) 60.07% compared to head (923c9f4) 59.94%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #5743      +/-   ##
==========================================
- Coverage   60.07%   59.94%   -0.14%     
==========================================
  Files         217      217              
  Lines       30717    30694      -23     
==========================================
- Hits        18453    18398      -55     
- Misses      10521    10548      +27     
- Partials     1743     1748       +5     

Flag	Coverage Δ
core-unittests	55.83% <50.00%> (-0.08%)	⬇️
e2e-multicluster-test	25.64% <0.00%> (-0.13%)	⬇️
e2e-rollout-tests	22.27% <ø> (-0.09%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
pkg/appfile/parser.go	61.76% <ø> (-0.07%)	⬇️
pkg/workflow/step/generator.go	100.00% <ø> (ø)
pkg/definition/definition.go	72.39% <50.00%> (-0.20%)	⬇️

... and 10 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.