Skip to content

Conversation

@ncipollina
Copy link
Contributor

Summary

This PR introduces a new SiteDomain property to the StaticSiteConstruct and includes routine dependency updates for version 2.0.1.

  • New Feature: Added public SiteDomain property to expose the fully qualified domain name after construct instantiation
  • Dependency Updates: Updated AWS CDK from 2.209.1 to 2.213.0 and xUnit test dependencies
  • Documentation: Added comprehensive XML documentation and updated all relevant documentation files
  • Version Bump: Incremented version to 2.0.1

Changes Made

Core Functionality

  • Added SiteDomain property to StaticSiteConstruct class with full XML documentation
  • Refactored internal domain name usage to use the new public property

Dependencies

  • AWS CDK: 2.209.12.213.0
  • xUnit v3: 3.0.03.0.1
  • xUnit runner: 3.1.33.1.4

Documentation

  • Updated CLAUDE.md with new property information and CDK version
  • Enhanced docs/constructs/static-site.md with property documentation and usage examples
  • Added comprehensive XML documentation for the new property

API Impact

This is a non-breaking change that only adds new functionality. Existing code will continue to work unchanged, but consumers can now access the computed site domain:

var site = new StaticSiteConstruct(this, "MySite", props);
var domain = site.SiteDomain; // "www.example.com"

Test Plan

  • Build solution successfully
  • All existing tests pass
  • Documentation updated and reviewed
  • XML documentation added for new property
  • No breaking changes to existing API

🤖 Generated with Claude Code

…dencies

- Add public SiteDomain property to expose fully qualified domain name
- Update AWS CDK from 2.209.1 to 2.213.0
- Update xUnit dependencies (v3.0.0 -> v3.0.1, runner v3.1.3 -> v3.1.4)
- Version bump to 2.0.1
- Add comprehensive XML documentation for new property
- Update CLAUDE.md and static-site.md documentation

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@ncipollina ncipollina requested a review from Copilot September 2, 2025 11:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a new SiteDomain property to the StaticSiteConstruct class and performs routine dependency updates for version 2.0.1.

  • Adds a public SiteDomain property to expose the fully qualified domain name after construct instantiation
  • Updates AWS CDK from 2.209.1 to 2.213.0 and xUnit test dependencies
  • Increments version to 2.0.1 with updated documentation

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
StaticSiteConstruct.cs Adds new public SiteDomain property and refactors existing code to use it
LayeredCraft.Cdk.Constructs.csproj Updates AWS CDK dependency from 2.209.1 to 2.213.0
LayeredCraft.Cdk.Constructs.Tests.csproj Updates xUnit test dependencies to latest versions
static-site.md Adds documentation for the new SiteDomain property with usage examples
Directory.Build.props Increments version from 2.0.0 to 2.0.1
CLAUDE.md Updates CDK version reference and adds note about new property

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

@ncipollina ncipollina merged commit 06bdc5a into main Sep 2, 2025
3 checks passed
@ncipollina ncipollina deleted the feature/static-site-domain branch September 2, 2025 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants