Sync main with dev.

[lukaszgryglicki]

cc @ahmedomosanya @mlehotskylf rollback if anything goes wrong...

Signed-off-by: Lukasz Gryglicki lgryglicki@cncf.io

Assisted by OpenAI

Assisted by GitHub Copilot

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

✨ Finishing touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch dev

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull Request Overview

This PR syncs the main branch with the dev branch, incorporating multiple improvements to functional testing infrastructure and bug fixes across the EasyCLA project.

• Enhanced functional test framework with better error handling, authentication, and debugging capabilities
• Updated API specifications and validation patterns for Salesforce IDs
• Added new utility scripts for AWS operations and testing
• Fixed deployment workflow bugs and added defensive null checks in backend services

Reviewed Changes

Copilot reviewed 84 out of 93 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
tests/functional/	Complete overhaul of Cypress test suite with improved API validation, error handling, and test utilities
utils/	New shell scripts for AWS SSM, Lambda functions, and API testing operations
cla-backend-go/	Updated Swagger specifications and added null safety checks in service layers
cla-backend/	Added PyNaCl dependency and defensive null checks for model attributes
.github/workflows/	Fixed typo in deployment workflows affecting exit code validation

Files not reviewed (1)

• tests/functional/package-lock.json: Language not supported

Comments suppressed due to low confidence (1)

cla-backend-go/v2/cla_manager/service.go:1180

• The error message is identical to the one above (line 1177), making it difficult to distinguish between 'error occurred' vs 'user not found' scenarios in logs. Consider making the messages more specific.

	if userModel == nil {
		msg := fmt.Sprintf("Problem getting user by ID: %s", notifyCLAManagers.UserID)
		log.WithFields(f).Warn(msg)
		return ErrCLAUserNotFound
	}

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}