Consolidated prs - #97
Closed
amrabed wants to merge 5 commits into
Closed
Conversation
- Added regex pattern `^[a-zA-Z0-9-_]+$` to `Entity.id` in `templates/models.py` for defense-in-depth. - Implemented proactive ID validation in Agent and GraphQL handlers. - Sanitized error messages across all handlers to prevent information leakage. - Standardized model dumping in SQS handler for consistency. - Updated unit tests to verify new validation logic and error messages.
- Enforce strict character set ^[a-zA-Z0-9-_]+$ for Entity IDs - Add proactive ID validation in GraphQL getItem resolver - Add proactive ID validation in Bedrock Agent getItem tool - Use generic error messages to avoid reflecting unvalidated input - Record learning in .jules/sentinel.md
…ation Optimized the DynamoDB Stream handler by validating the `DestinationItem` directly from the unmarshalled record dictionaries, bypassing the intermediate `SourceItem` validation. This reduces Pydantic instantiation overhead in high-throughput data processing paths. - Modified `_process` to accept a raw dictionary. - Updated `handle_record` to pass `record.dynamodb.new_image` directly to `_process`. - Refactored `REMOVE` event handling to validate the ID against `DestinationItem` without a full model lifecycle. - Leveraged the `.dump()` helper for efficient serialization. - Removed unused `SourceItem` import. Expected performance impact: ~20% reduction in record processing time based on benchmarks.
Improved documentation UX by adding a "← Back to Overview" link at the bottom of all template documentation pages. This prevents "dead-end" navigation and makes it easier for users to explore different templates. Changes: - Added `[← Back to Overview](../README.md)` to all `docs/template/*.md` files. - Updated `.template/docs/template.md` to include the link in future templates. - Verified changes using MkDocs and Playwright.
Updates the requirements on [aws-lambda-powertools](https://github.com/aws-powertools/powertools-lambda-python), [boto3](https://github.com/boto/boto3), [constructs](https://github.com/aws/constructs), [ruff](https://github.com/astral-sh/ruff), [coverage](https://github.com/coveragepy/coveragepy), [hypothesis](https://github.com/HypothesisWorks/hypothesis) and [mkdocstrings[python]](https://github.com/mkdocstrings/mkdocstrings) to permit the latest version. Updates `aws-lambda-powertools` from 3.31.0 to 3.31.1 - [Release notes](https://github.com/aws-powertools/powertools-lambda-python/releases) - [Changelog](https://github.com/aws-powertools/powertools-lambda-python/blob/develop/CHANGELOG.md) - [Commits](aws-powertools/powertools-lambda-python@v3.31.0...v3.31.1) Updates `boto3` from 1.43.45 to 1.43.50 - [Release notes](https://github.com/boto/boto3/releases) - [Commits](boto/boto3@1.43.45...1.43.50) Updates `constructs` from 10.6.0 to 10.7.0 - [Release notes](https://github.com/aws/constructs/releases) - [Commits](aws/constructs@v10.6.0...v10.7.0) Updates `ruff` from 0.15.21 to 0.15.22 - [Release notes](https://github.com/astral-sh/ruff/releases) - [Changelog](https://github.com/astral-sh/ruff/blob/main/CHANGELOG.md) - [Commits](astral-sh/ruff@0.15.21...0.15.22) Updates `coverage` from 7.15.0 to 7.15.2 - [Release notes](https://github.com/coveragepy/coveragepy/releases) - [Changelog](https://github.com/coveragepy/coveragepy/blob/main/CHANGES.rst) - [Commits](coveragepy/coveragepy@7.15.0...7.15.2) Updates `hypothesis` from 6.156.4 to 6.156.6 - [Release notes](https://github.com/HypothesisWorks/hypothesis/releases) - [Commits](HypothesisWorks/hypothesis@v6.156.4...v6.156.6) Updates `mkdocstrings[python]` to 1.0.6 - [Release notes](https://github.com/mkdocstrings/mkdocstrings/releases) - [Changelog](https://github.com/mkdocstrings/mkdocstrings/blob/main/CHANGELOG.md) - [Commits](mkdocstrings/mkdocstrings@1.0.4...1.0.6) --- updated-dependencies: - dependency-name: aws-lambda-powertools dependency-version: 3.31.1 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: dependencies - dependency-name: boto3 dependency-version: 1.43.50 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: dependencies - dependency-name: constructs dependency-version: 10.7.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: dependencies - dependency-name: ruff dependency-version: 0.15.22 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: dependencies - dependency-name: coverage dependency-version: 7.15.2 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: dependencies - dependency-name: hypothesis dependency-version: 6.156.6 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: dependencies - dependency-name: mkdocstrings[python] dependency-version: 1.0.6 dependency-type: direct:development dependency-group: dependencies ... Signed-off-by: dependabot[bot] <support@github.com>
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.
Pull Request Type
Summary
Please replace this line with a brief description of the changes you've made
Related Issues
QA Instructions
Please replace this line with instructions on how to test your changes
Submission Checklist
Before submitting a Pull Request, please ensure you've done the following: