feat: Configurable OTEL layer with architecture and version support (BREAKING)

[ncipollina]

Summary

• Add configurable OpenTelemetry layer with architecture and version support
• BREAKING CHANGE: OTEL layer now disabled by default (was enabled by default in v1.x)
• Bump version to 2.0.0-beta for breaking changes

Breaking Changes

• IncludeOtelLayer default changed from true to false
• Added new properties: Architecture (default: "amd64") and OtelLayerVersion (default: "0-117-0")

New Features

• Configurable OTEL layer architecture (amd64/arm64)
• Configurable OTEL layer version (future-proof for AWS updates)
• Dynamic OTEL layer ARN format: arn:aws:lambda:{region}:901920570463:layer:aws-otel-collector-{architecture}-ver-{version}:1

Migration Guide

For users upgrading from v1.x who want to keep OTEL enabled:

var props = new LambdaFunctionConstructProps
{
    // ... existing properties
    IncludeOtelLayer = true,           // Explicitly enable (was default in v1.x)
    Architecture = "arm64",            // Optional: specify architecture
    OtelLayerVersion = "0-117-0"       // Optional: specify version
};

Documentation Updates

• Updated CLAUDE.md with new OTEL configuration section
• Updated README.md examples
• Updated docs/constructs/lambda-function.md with breaking change warnings
• Added migration examples

Test Updates

• Updated all test attributes and customizations to use new defaults
• Updated test patterns to expect new OTEL layer ARN format
• All 151 tests passing

Benefits

• Future-proof: No more package updates when AWS releases new OTEL versions
• Architecture flexibility: Support for both amd64 and arm64
• Cost control: OTEL layer now opt-in, reducing unnecessary costs
• Backward compatibility: Existing code works with explicit configuration

Test plan

• All existing tests pass (151/151)
• Build succeeds without warnings
• New properties work correctly in tests
• Documentation is comprehensive and accurate
• Manual testing of new OTEL configuration
• Verify ARM64 architecture support works

🤖 Generated with Claude Code

[Copilot]

Pull Request Overview

This PR introduces configurable OpenTelemetry (OTEL) layer support with architecture and version flexibility, while implementing breaking changes for version 2.0.0-beta. The OTEL layer is now disabled by default instead of enabled, providing better cost control and making observability features opt-in.

• BREAKING: OTEL layer default changed from enabled to disabled
• Added configurable OTEL layer version and architecture support (amd64/arm64)
• Updated version to 2.0.0-beta and bumped AWS CDK dependency

Reviewed Changes

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

Show a summary per file

File	Description
Directory.Build.props	Version bump to 2.0.0-beta for breaking changes
src/LayeredCraft.Cdk.Constructs/LayeredCraft.Cdk.Constructs.csproj	AWS CDK dependency update to 2.209.1
src/LayeredCraft.Cdk.Constructs/Models/LambdaFunctionConstructProps.cs	Added new OTEL configuration properties with updated defaults
src/LayeredCraft.Cdk.Constructs/LambdaFunctionConstruct.cs	Dynamic OTEL layer ARN generation using new configuration properties
src/LayeredCraft.Cdk.Constructs/Testing/LambdaFunctionConstructPropsBuilder.cs	Builder pattern updates for new OTEL configuration options
test files	Test framework updates to use new default values and validate new OTEL ARN format
documentation	Updated docs with breaking change warnings and migration examples

[Copilot]

[nitpick] The default value assignment has been removed from these properties. Consider maintaining explicit default values for consistency with other properties in the class.

Suggested change

	public bool EnableSnapStart { get; set; }
	public bool GenerateUrl { get; set; }
	public bool EnableSnapStart { get; set; } = false;
	public bool GenerateUrl { get; set; } = false;

[Copilot]

[nitpick] The default value assignment has been removed from these properties. Consider maintaining explicit default values for consistency with other properties in the class.

Suggested change

	public bool EnableSnapStart { get; set; }
	public bool GenerateUrl { get; set; }
	public bool EnableSnapStart { get; set; } = false;
	public bool GenerateUrl { get; set; } = false;