refactor: convert ConnectorResourceReference from namedtuple to datac…

[akshatsinha0]

Issue #, if available

N/A - Addresses TODO comment in project codebase

Description of changes

Refactored ConnectorResourceReference from namedtuple to @dataclass(frozen=True) (Python 3.8+)

Description of how you validated changes

Ran connector test suite:267 test cases
Verified no breaking changes to the public API+immutability

Checklist

• Adheres to the development guidelines
• Add/update transform tests
  • Using correct values
  • Using wrong values
• Add/update integration tests - Not required for internal refactor with no behavior changes

Examples?

N/A - Internal refactor with no user-facing changes

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[reedham-aws]

When looking at all places that reference this class, it seems like there's an instance where logical_id would be None:

serverless-application-model/samtranslator/model/connector/connector.py

Lines 129 to 138 in e7f8b96

	return ConnectorResourceReference(
	logical_id=None,
	resource_type=resource_type,
	arn=obj.get("Arn"),
	role_name=obj.get("RoleName"),
	queue_url=obj.get("QueueUrl"),
	resource_id=obj.get("ResourceId"),
	name=obj.get("Name"),
	qualifier=obj.get("Qualifier"),
	)

Should this be Optional[str]?

[akshatsinha0]

ya ya, because id is not provided in the resource reference