Skip to content

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

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

Christophe Melchior (Kakudou) wants to merge 1 commit 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                                         84.82%   84.84%   +0.02%     
================================================================================
  Files                                               186      187       +1     
  Lines                                              8058     8079      +21     
================================================================================
+ Hits                                               6835     6855      +20     
- Misses                                             1223     1224       +1     
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%> (ø)

📢 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.

@Kakudou
Christophe Melchior (Kakudou) force-pushed the feat/422-prowler-injector-chk7-base-aws branch from 133745e to 5603b0d Compare September 16, 2026 09:13
@Kakudou
Christophe Melchior (Kakudou) force-pushed the feat/422-prowler-injector-chk7-base-aws branch from 5603b0d to 133745e Compare September 16, 2026 09:21
@Kakudou
Christophe Melchior (Kakudou) force-pushed the feat/422-prowler-injector-chk7-base-aws branch from 133745e to 46e7c8e Compare September 17, 2026 18:40
@Kakudou Christophe Melchior (Kakudou) linked an issue Sep 18, 2026 that may be closed by this pull request
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.

feat(prowler): Create the Prowler injector

3 participants