Skip to content

feat(prowler): chunk7 add the AWS base contract (#422) - #437

Draft
Christophe Melchior (Kakudou) wants to merge 2 commits into
feat/422-prowler-injector-chk6-contract-basefrom
feat/422-prowler-injector-chk7-base-aws
Draft

feat(prowler): chunk7 add the AWS base contract (#422)#437
Christophe Melchior (Kakudou) wants to merge 2 commits into
feat/422-prowler-injector-chk6-contract-basefrom
feat/422-prowler-injector-chk7-base-aws

Conversation

@Kakudou

Copy link
Copy Markdown
Member

Part of #422.

Chunk 7 of the Prowler injector stack: add the AWS base contract — the CHK007 base AWS provider: the BDD suite (behaviour, Unicode-digit account-ID rejection, OCSF status / AWS endpoint / lifecycle / verbose-diagnostic fixtures), the contracts/aws.py AWS contract module, its registration in the registry / __init__ / injector (AWS becomes the sole default executable contract), the quality-gate pass, and the final OCSF capture alignment (chk001 BDD + chk006 unit included).

Re-landed from feat/422-prowler-chk007-base-aws-provider (3d06af3..5603b0d): 9 source commits folded into 2 (G1: cba1e76 + 5 test-alignment folds; G2: fc54715 + a1cde93 + 5603b0d — every cherry-pick applied clean, zero conflicts). Gate V (whole-branch tree vs source tip 5603b0d): byte-identical except (a) the 14 pyproject.toml version differences of the other injectors (base move to 3.260821.0; prowler/ untouched), (b) the 4 trap-analysis.md files excluded by standing call (3 carried from earlier chunks + the new chk007 one), and (c) one approved deviation — the pinned pyoaev version assert removed from prowler/tests/unit/chk006_executable_base/test_outputs_registry_runtime.py.

@Filigran-Automation Filigran Automation (Filigran-Automation) added the filigran team Item from the Filigran team. label Aug 27, 2026
@codecov

codecov Bot commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.45455% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
prowler/prowler/contracts/aws.py 94.73% 1 Missing ⚠️
Additional details and impacted files
@@                               Coverage Diff                                @@
##           feat/422-prowler-injector-chk6-contract-base     #437      +/-   ##
================================================================================
- Coverage                                         85.62%   84.84%   -0.78%     
================================================================================
  Files                                               167      187      +20     
  Lines                                              6989     8079    +1090     
================================================================================
+ Hits                                               5984     6855     +871     
- Misses                                             1005     1224     +219     
Files with missing lines Coverage Δ
prowler/prowler/contracts/__init__.py 100.00% <100.00%> (ø)
prowler/prowler/contracts/registry.py 93.47% <100.00%> (+0.14%) ⬆️
prowler/prowler/injector/openaev_prowler.py 93.97% <ø> (ø)
prowler/prowler/contracts/aws.py 94.73% <94.73%> (ø)

... and 20 files with indirect coverage changes

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team Item from the Filigran team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants