Skip to content

fix: use identityPool provider directly in auth directive output#682

Open
soberm wants to merge 1 commit intomainfrom
fix/use-identitypool-provider-in-auth-directive
Open

fix: use identityPool provider directly in auth directive output#682
soberm wants to merge 1 commit intomainfrom
fix/use-identitypool-provider-in-auth-directive

Conversation

@soberm
Copy link
Copy Markdown
Contributor

@soberm soberm commented Mar 16, 2026

Problem

The SchemaProcessor was mapping identityPool to iam when generating auth directive output. This mapping is unnecessary at the schema level since the transform layer already handles it, and it obscures the original provider name in the generated schema.

Issue number, if available:

Changes

  • Removed the identityPooliam provider mapping in SchemaProcessor.calculateAuth()
  • Auth directives now preserve identityPool as the provider name in the schema output
  • Updated snapshots and integration tests to reflect the corrected provider name

Corresponding docs PR, if applicable:

Validation

  • All 36 existing test suites pass (548 tests, 267 snapshots)
  • Integration test assertions updated to verify identityPool is preserved in auth directives
  • Snapshot files updated to match new output

Checklist

  • If this PR includes a functional change to the runtime or type-level behavior of the code, I have added or updated automated test coverage for this change. (see Testing Strategy README)
  • If this PR requires a docs update, I have linked to that docs PR above.

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

Stop mapping identityPool to iam in SchemaProcessor auth rule
generation. The transform layer handles this mapping, so the schema
output should preserve the original provider name.
@soberm soberm requested review from a team as code owners March 16, 2026 10:20
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 16, 2026

🦋 Changeset detected

Latest commit: f0eb3f0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@aws-amplify/data-schema Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

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