Skip to content

Comments

feat(appsync): integrate plugin functionality#13217

Merged
czubocha merged 5 commits intomainfrom
sc-3611
Jan 8, 2026
Merged

feat(appsync): integrate plugin functionality#13217
czubocha merged 5 commits intomainfrom
sc-3611

Conversation

@czubocha
Copy link
Contributor

@czubocha czubocha commented Jan 7, 2026

Summary

This Pull Request integrates the previously standalone AWS AppSync plugin functionality directly into the Serverless Framework core. By migrating this support into the main repository, AWS AppSync becomes a native feature, providing a more seamless experience with built-in command support, consolidated documentation, and simplified dependency management.

Main Changes:

  • Core Plugin Migration: Integrated the serverless-appsync-plugin source code and logic into the core Framework as a native AWS provider plugin.
  • Documentation Standardization: Reorganized and updated the AppSync documentation into 12 specialized pages (index + 11 sub-topics), adding standardized metadata.
  • Native Command Support: Ensured all AppSync-specific CLI commands are correctly exposed and scoped within the core serverless CLI.
  • Test Suite Integration: Migrated and validated the full suite of AppSync unit tests to ensure functional parity and maintain long-term stability within the core codebase.
  • Schema Optimization: Fixed regressions in command and schema resolution and removed the legacy ajv-merge-patch dependency to simplify internal validation logic.
  • Improved User Experience: Standardized the metadata and organizational structure of the AppSync guide to match other documentation sections.

@czubocha czubocha requested a review from eahefnawy January 7, 2026 17:56
@Mmarzex
Copy link
Contributor

Mmarzex commented Jan 7, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

# Conflicts:
#	package-lock.json
#	packages/serverless/package.json
Copy link
Contributor

@eahefnawy eahefnawy left a comment

Choose a reason for hiding this comment

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

@czubocha Looks good. Glad we finally did this. Huge thanks to the original AppSync contributors! My only question is, how does the "migration" look like for users to move out of the plugin? Do they just have to comment it out and remove it from package.json? Maybe this should be in the docs.

@czubocha
Copy link
Contributor Author

czubocha commented Jan 7, 2026

@czubocha Looks good. Glad we finally did this. Huge thanks to the original AppSync contributors! My only question is, how does the "migration" look like for users to move out of the plugin? Do they just have to comment it out and remove it from package.json? Maybe this should be in the docs.

Thanks! Yes, exactly - the migration steps are covered in the docs. Users just need to remove the plugin from serverless.yml and package.json.

There's a dedicated “Migration from the community plugin” section here:
https://github.com/serverless/serverless/pull/13217/changes#diff-dfebf77c34b13457ca7cfb4452507a4519617805f8667a600aa4f98c9a0affb5R27

@czubocha czubocha merged commit 697b66b into main Jan 8, 2026
10 checks passed
@czubocha czubocha deleted the sc-3611 branch January 8, 2026 18:39
@github-actions github-actions bot locked and limited conversation to collaborators Jan 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants