Skip to content

Conversation

@iut62elec
Copy link
Contributor

This PR enhances the Text-to-SQL Bedrock Agent CDK Enhanced with improved cross-environment compatibility and better dependency management.

Changes Made

  • requirements.txt: Added version pinning for all packages to ensure consistency across environments
  • package.json: Updated CDK versions from 2.101.1 to 2.200.0+ and added proper package metadata
  • setup.sh: Enhanced with error handling, Python version validation, and Node.js 20 LTS support
  • .nvmrc: Added for consistent Node.js version across environments
  • runtime.txt: Added for Python version specification

Key Improvements

  • ✅ Better version consistency across different deployment environments
  • ✅ Enhanced error handling in setup script
  • ✅ Python version validation before proceeding with setup
  • ✅ Updated to Node.js 20 LTS for better stability
  • ✅ Clear logging and user guidance throughout setup process

Testing

  • Tested successfully in the original development environment
  • All CDK deployment functionality preserved
  • Enhanced compatibility for new environments

Author

iut62elec added 3 commits June 13, 2025 11:19
…ibility

- Updated requirements.txt with version pinning for better dependency management
- Enhanced package.json with proper metadata and updated CDK versions
- Improved setup.sh with error handling, Python version validation, and Node.js 20 LTS
- Added .nvmrc for consistent Node.js version across environments
- Added runtime.txt for Python version specification
- Better logging and user guidance in setup script
- Ensures compatibility across different deployment environments
Copy link
Contributor

Choose a reason for hiding this comment

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

we should remove the cdk out folder

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done! Thanks

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