refactor!: rename all namespaces to LayeredCraft.DynamoMapper.*#86
Merged
ncipollina merged 4 commits intomainfrom Mar 30, 2026
Merged
refactor!: rename all namespaces to LayeredCraft.DynamoMapper.*#86ncipollina merged 4 commits intomainfrom
ncipollina merged 4 commits intomainfrom
Conversation
Downgrades Microsoft.CodeAnalysis.Common and Microsoft.CodeAnalysis.CSharp from 5.3.0 to 5.0.0 to work around a bug in the newer version that breaks source generator interception. Also updates AWSSDK.DynamoDBv2 (4.0.15 → 4.0.17), Scriban (6.6.0 → 7.0.0), and bumps VersionPrefix to 1.0.6. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Renames every C# namespace, using statement, and WellKnownType enum member from the old DynamoMapper.* prefix to LayeredCraft.DynamoMapper.* to align with the new NuGet package IDs (LayeredCraft.DynamoMapper and LayeredCraft.DynamoMapper.Runtime). Also sets AssemblyName and RootNamespace in both .csproj files and updates the Verify snapshot baselines to reflect the new runtime using directive in generated code. BREAKING CHANGE: All public namespaces have changed. Consumers must update their using statements from DynamoMapper.Runtime to LayeredCraft.DynamoMapper.Runtime. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Updates README.md and all docs/ pages to reflect the package rename from DynamoMapper to LayeredCraft.DynamoMapper: - Headings, titles, and NuGet badges updated to LayeredCraft.DynamoMapper - All using statements in code blocks changed to LayeredCraft.DynamoMapper.Runtime - Package install commands changed to LayeredCraft.DynamoMapper - Runtime/generator package name references updated in requirements.md - Migration callout added to README, docs/index.md, and installation.md explicitly noting this supersedes the DynamoMapper package - Changelog documents the breaking namespace/package rename Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
ncipollina
had a problem deploying
to
github-pages
GitHub Actions
Failure
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Renames every C# namespace,
usingstatement, Scriban template directive, andWellKnownTypeenum member from the oldDynamoMapper.*prefix toLayeredCraft.DynamoMapper.*. This aligns the codebase with the new NuGet package IDs (LayeredCraft.DynamoMapperandLayeredCraft.DynamoMapper.Runtime) already set in the.csprojfiles. Because this is a pre-release package with no prior consumers, no backwards-compatibility shim is provided.Changes
Build / project files
Directory.Build.props— updated version prefixDirectory.Packages.props— pinnedMicrosoft.CodeAnalysisto 5.0.0, bumped remaining deps.csprojfiles —AssemblyNameandRootNamespacenow derive from the newPackageIdvaluesSource (src/)
namespaceandusingdeclarations inLayeredCraft.DynamoMapper.Generatorsupdated fromDynamoMapper.Generator.*→LayeredCraft.DynamoMapper.Generator.*namespaceandusingdeclarations inLayeredCraft.DynamoMapper.Runtimeupdated fromDynamoMapper.Runtime→LayeredCraft.DynamoMapper.RuntimeWellKnownTypeDataenum members renamed fromDynamoMapper_Runtime_*→LayeredCraft_DynamoMapper_Runtime_*to satisfy the compile-time enum/type-name assertionMapper.scribantemplate updated:using DynamoMapper.Runtime→using LayeredCraft.DynamoMapper.RuntimeTests (test/)
usingstatements updatedVerify.SourceGeneratorssnapshot baselines regenerated to reflect the newusing LayeredCraft.DynamoMapper.Runtimedirective in generated codeExamples (examples/)
usingandnamespacedeclarations updatedValidation
dotnet build— 0 errors, 11 warnings (pre-existing RS2001 analyzer warnings)dotnet test(net10.0) — 94/94 generator snapshot tests passed, 245/245 runtime tests passedusingdirective line changedBreaking Changes
using DynamoMapper.Runtime;using LayeredCraft.DynamoMapper.Runtime;using DynamoMapper.*statements tousing LayeredCraft.DynamoMapper.*. Since this package is pre-release (0.1.0-alpha) with no published consumers, no migration guide is required.Release Notes
NuGet package IDs are now
LayeredCraft.DynamoMapperandLayeredCraft.DynamoMapper.Runtime. The root namespace for all public types isLayeredCraft.DynamoMapper.Runtime.🤖 Generated with Claude Code