1.6.0 - Include chainId in JWT audience field and fix StepOutputVar

[chrisli30]

No description provided.

[Copilot]

Pull Request Overview

This PR introduces enhancements for improved error handling and chainId JWT audience verification while refining various CI/CD workflows for release automation. Key changes include:

• Error handling improvements in the task engine, RPC server initialization, and JWT verification in the auth service.
• Addition of chainId validation in JWT audience fields.
• Updates to pre-commit hooks and GitHub workflows covering testing, release, and Docker image publishing.

Reviewed Changes

Copilot reviewed 61 out of 61 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
aggregator/task_engine.go	Enhanced error handling for task processor registration and engine startup.
aggregator/rpc_server.go	Added chainId field and corresponding initialization from smart wallet RPC.
aggregator/pool.go	Introduced error checking for JSON unmarshalling with a silent continue strategy.
aggregator/auth_test.go	Fixed error message typos and added cross-chain JWT validation tests.
aggregator/auth.go	Improved chainId validation in GetKey and verifyAuth along with caching error logging.
aggregator/aggregator.go	Updated RPC server startup to include error logging.
RELEASE.md	Detailed the updated release process and branching strategy.
.pre-commit-config.yaml	Modified go fmt command and added a conventional commit check hook.
Various GitHub workflow YAML files	Updated workflows and removed deprecated ones for better CI/CD integration.

[Copilot]

For consistency with other parts of the code that use r.chainID.Int64(), consider formatting chainIdStr using r.chainID.Int64() instead of r.chainID.

Suggested change

	chainIdStr := fmt.Sprintf("%d", r.chainID)
	chainIdStr := fmt.Sprintf("%d", r.chainID.Int64())

[Copilot]

[nitpick] Consider logging or handling the error from json.Unmarshal instead of silently continuing to help with debugging potential issues with data unmarshalling.

Suggested change

	if err := json.Unmarshal(rawValue.Value, node); err != nil {
	if err := json.Unmarshal(rawValue.Value, node); err != nil {
	fmt.Printf("Error unmarshalling operator node: %v\n", err)

[Copilot]

Pull Request Overview

This PR enhances JWT authentication by including chainId in the JWT audience field and refines signature verification and environment configuration. It also updates various workflows, deployment scripts, documentation, and build configurations to support these changes.

• Updates the API claims in JWT to include the chainId value.
• Refactors authentication logic to use improved message parsing and variable naming.
• Adjusts CI/CD workflows and Makefile to align with the new requirements.

Reviewed Changes

Copilot reviewed 104 out of 104 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
aggregator/key.go	Renames fields for better clarity and correctly assigns roles.
aggregator/chain_id.go	Introduces global chainID management.
aggregator/auth_test.go	Updates tests to use the new message format and chainId validation.
aggregator/auth.go	Implements enhanced message parsing, JWT audience inclusion, and updated error handling.
aggregator/aggregator.go	Configures global chainID and updates RPC server startup error handling.
RELEASE.md, README.md, Makefile	Updates documentation and build instructions for the new flows.
.github/workflows/*	Updates and adds CI/CD workflows for testing, release, and Docker image publishing.