Conversation
There was a problem hiding this comment.
🟢 Approval recommended
The rename is consistently applied across code, tooling, documentation, and metadata with no unresolved review issues.
Pull request overview
Renames the package, projects, namespaces, configuration keys, documentation, and release metadata to Csag.WorkloadIdentity.
Changes:
- Updated .NET projects, namespaces, references, and lock files.
- Updated documentation, demos, Docker files, CI/CD workflows, and package metadata.
- Updated Changesets configuration and removed obsolete legacy paths.
File summaries
| File | Summary |
|---|---|
SECURITY.md |
Updated library scope |
README.md |
Updated project and package references |
package.json |
Renamed workspace metadata |
package-lock.json |
Updated workspace lock metadata |
docs/cloud-identity-setup.md |
Updated configuration documentation |
Csag.WorkloadIdentity/README.md |
Updated package usage documentation |
Csag.WorkloadIdentity/packages.lock.json |
Updated dependency lock |
Csag.WorkloadIdentity/package.json |
Updated package metadata |
Csag.WorkloadIdentity/Options/GoogleOptions.cs |
Updated namespace |
Csag.WorkloadIdentity/Options/AzureSqlFederatedIdentityOptionsValidator.cs |
Updated namespace |
Csag.WorkloadIdentity/Options/AzureSqlFederatedIdentityOptions.cs |
Updated namespace and configuration |
Csag.WorkloadIdentity/Internal/Services/AzureSqlTokenRefreshService.cs |
Updated namespace |
Csag.WorkloadIdentity/Internal/IIamCredentialsClientFactory.cs |
Updated namespace |
Csag.WorkloadIdentity/Internal/IClientAssertionCredentialFactory.cs |
Updated namespace |
Csag.WorkloadIdentity/Internal/IAzureSqlTokenRefresher.cs |
Updated namespace |
Csag.WorkloadIdentity/Internal/IamCredentialsClientFactory.cs |
Updated namespace |
Csag.WorkloadIdentity/Internal/GoogleIdTokenProvider.cs |
Updated namespace |
Csag.WorkloadIdentity/Internal/ClientAssertionCredentialFactory.cs |
Updated namespace |
Csag.WorkloadIdentity/Internal/AzureSqlTokenExchanger.cs |
Updated namespace |
Csag.WorkloadIdentity/icon.png |
Added renamed package icon |
Csag.WorkloadIdentity/FederatedIdentityServiceCollectionExtensions.cs |
Updated namespace imports |
Csag.WorkloadIdentity/Csag.WorkloadIdentity.csproj |
Renamed project metadata |
Csag.WorkloadIdentity/CHANGELOG.md |
Updated package name |
Csag.WorkloadIdentity/AzureSqlTokenProvider.cs |
Updated namespace imports |
Csag.WorkloadIdentity/Abstractions/IGoogleIdTokenProvider.cs |
Updated namespace |
Csag.WorkloadIdentity/Abstractions/IAzureSqlTokenProvider.cs |
Updated namespace |
Csag.WorkloadIdentity/Abstractions/IAzureSqlTokenExchanger.cs |
Updated namespace |
Csag.WorkloadIdentity.UnitTests/packages.lock.json |
Updated dependency lock |
Csag.WorkloadIdentity.UnitTests/GoogleIdTokenProviderTests.cs |
Updated test namespace |
Csag.WorkloadIdentity.UnitTests/FederatedIdentityServiceCollectionExtensionsTests.cs |
Updated test namespace |
Csag.WorkloadIdentity.UnitTests/Csag.WorkloadIdentity.UnitTests.csproj |
Updated project reference |
Csag.WorkloadIdentity.UnitTests/AzureSqlTokenRefreshServiceTests.cs |
Updated test namespace |
Csag.WorkloadIdentity.UnitTests/AzureSqlTokenProviderTests.cs |
Updated test namespace |
Csag.WorkloadIdentity.UnitTests/AzureSqlTokenExchangerTests.cs |
Updated test namespace |
Csag.WorkloadIdentity.UnitTests/AzureSqlFederatedIdentityOptionsValidatorTests.cs |
Updated test namespace |
Csag.WorkloadIdentity.sln |
Updated solution paths |
Csag.WorkloadIdentity.Demo/wwwroot/favicon.ico |
Updated demo asset |
Csag.WorkloadIdentity.Demo/README.md |
Updated demo documentation |
Csag.WorkloadIdentity.Demo/Properties/launchSettings.json |
Updated launch settings |
Csag.WorkloadIdentity.Demo/Program.cs |
Updated imports |
Csag.WorkloadIdentity.Demo/packages.lock.json |
Updated dependency lock |
Csag.WorkloadIdentity.Demo/Entities/TestEntity.cs |
Updated namespace |
Csag.WorkloadIdentity.Demo/Dockerfile |
Updated demo container |
Csag.WorkloadIdentity.Demo/Database/IAppDbContextFactory.cs |
Updated namespace |
Csag.WorkloadIdentity.Demo/Database/AppDbContextFactory.cs |
Updated namespace |
Csag.WorkloadIdentity.Demo/Database/AppDbContext.cs |
Updated namespace |
Csag.WorkloadIdentity.Demo/Csag.WorkloadIdentity.Demo.csproj |
Updated project reference |
Csag.WorkloadIdentity.Demo/appsettings.json |
Updated configuration |
Csag.WorkloadIdentity.Demo/appsettings.Development.json |
Updated development configuration |
Csag.AzureSqlFederatedIdentity/package.json |
Removed obsolete metadata |
Csag.AzureSqlFederatedIdentity.Demo/Dockerfile |
Removed obsolete Dockerfile |
CONTRIBUTING.md |
Updated contributor references |
.github/workflows/nuget-publish.yml |
Updated publishing paths |
.github/workflows/dependabot-changeset.yml |
Updated package paths |
.github/workflows/ci.yml |
Updated CI paths |
.dockerignore |
Updated Docker paths |
.changeset/token-provider-hardening.md |
Updated package identifier |
.changeset/rename-to-csag.md |
Documented rename |
.changeset/docs-usage-and-guides.md |
Updated package identifier |
.changeset/config.json |
Updated package configuration |
.changeset/build-hygiene.md |
Updated package identifier |
Review details
- Files reviewed: 55/61 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
neotrow
added this pull request to stack #13
September 18, 2026 15:30
neotrow
force-pushed
the
rename/workload-identity
branch
from
September 18, 2026 16:02
02d4a34 to
bd6bb33
Compare
There was a problem hiding this comment.
🟢 Approval recommended
The rename is consistently applied; only a minor release-note correction remains.
Review details
Suppressed comments (1)
.changeset/rename-to-csag.md:5
- This release note identifies the previous package as
Neolution.AzureSqlFederatedIdentity, but the package being renamed in this PR isCsag.AzureSqlFederatedIdentity(and the repository explicitly says the intermediate name was never published). Please name the actual predecessor here so the generated changelog does not describe a rename that did not occur.
Rename the package to `Csag.WorkloadIdentity` (previously `Neolution.AzureSqlFederatedIdentity`).
- Files reviewed: 55/61 changed files
- Comments generated: 0 new
- Review effort level: Lite
neotrow
force-pushed
the
rename/workload-identity
branch
from
September 18, 2026 16:37
bd6bb33 to
92c386a
Compare
neotrow
force-pushed
the
rename/workload-identity
branch
from
September 18, 2026 17:03
92c386a to
e96c638
Compare
The library is about to grow beyond Azure SQL (Blob Storage, Azure Managed Identity as an alternative to Google federation), so the package, root namespace, project, solution and folder names move from Csag.AzureSqlFederatedIdentity to Csag.WorkloadIdentity, and the changesets package becomes @neolution-ch/csag-workload-identity. The configuration section key changes accordingly. Nothing has been published under the intermediate name. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
neotrow
force-pushed
the
rename/workload-identity
branch
from
September 18, 2026 17:27
e96c638 to
422f75e
Compare
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
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
Mechanical rename
Csag.AzureSqlFederatedIdentity→Csag.WorkloadIdentity(package ID, root namespace, project/solution/folder names, changesets package@neolution-ch/csag-workload-identity, configuration section key), in preparation for the multi-resource / multi-provider generalisation that follows in the next PRs (rebuilding #2 on the hardened base).Nothing has been published under the intermediate name, so nuget.org will only ever see
Csag.WorkloadIdentity. The GitHub repository name is unchanged. Type names (AzureSqlFederatedIdentityOptions,AddAzureSqlFederatedIdentity, …) are untouched here and are restructured by the generalisation PR.Stacked on #8.
Validation
dotnet build -c Releaseclean;dotnet test40/40 on net8.0 and net10.0<id>Csag.WorkloadIdentity</id>git grepfor the old identifiers → none (repository URLs intentionally kept)🤖 Generated with Claude Code