Skip to content

feat(openaev): compatibility with new JSON:API format (#317)#358

Draft
guzmud wants to merge 13 commits into
mainfrom
fix/317-collector-openaev-jsonapi
Draft

feat(openaev): compatibility with new JSON:API format (#317)#358
guzmud wants to merge 13 commits into
mainfrom
fix/317-collector-openaev-jsonapi

Conversation

@guzmud

@guzmud guzmud commented May 12, 2026

Copy link
Copy Markdown
Member

Proposed changes

  • Add compatibility with JSON:API format for payloads to the openaev collector

Testing Instructions

  1. Try to import with JSON:API formatted payload available in the source repository

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality
  • For bug fix -> I implemented a test that covers the bug

@guzmud guzmud added the bug Type: something isn't working (fix:). label May 12, 2026
@github-actions github-actions Bot added the filigran team Item from the Filigran team. label May 12, 2026
@guzmud guzmud force-pushed the fix/317-collector-openaev-jsonapi branch 2 times, most recently from ef208ea to 343bc64 Compare May 13, 2026 10:24
@codecov

codecov Bot commented May 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.44554% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.87%. Comparing base (4068464) to head (1744927).

Files with missing lines Patch % Lines
openaev/openaev/openaev_openaev.py 85.41% 21 Missing ⚠️
...penaev/tests/functional/test_payload_processing.py 96.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #358      +/-   ##
==========================================
+ Coverage   71.06%   72.87%   +1.81%     
==========================================
  Files         128      136       +8     
  Lines        6790     7336     +546     
==========================================
+ Hits         4825     5346     +521     
- Misses       1965     1990      +25     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@guzmud guzmud added the do not merge Do not merge until this label is removed. label May 13, 2026
@guzmud

guzmud commented May 13, 2026

Copy link
Copy Markdown
Member Author

Adding do not merge since the product orientation of the overall feature in unclear in its expectations regarding implementation (same goes for OpenAEV-Platform/payloads#8). PR in standby in order to have groundwork already done.

@guzmud guzmud force-pushed the fix/317-collector-openaev-jsonapi branch 3 times, most recently from 84c27ee to aeba730 Compare May 19, 2026 08:28
@guzmud guzmud added feature Type: new feature or capability (feat:). and removed do not merge Do not merge until this label is removed. bug Type: something isn't working (fix:). labels May 19, 2026
@guzmud

guzmud commented May 19, 2026

Copy link
Copy Markdown
Member Author

Note: changing the tag from bug to feature due to the redefinition of the issue / redefinition of the target (more details in the attached issue #317 )

@guzmud guzmud force-pushed the fix/317-collector-openaev-jsonapi branch 7 times, most recently from 837fcd5 to 966251e Compare May 22, 2026 06:54
@guzmud guzmud force-pushed the fix/317-collector-openaev-jsonapi branch from 966251e to 9976f89 Compare May 27, 2026 07:05
@SamuelHassine SamuelHassine removed the feature Type: new feature or capability (feat:). label Jun 7, 2026
@SamuelHassine SamuelHassine changed the title [openaev] feat(collector): compatibility with new JSON:API format (#317) feat(openaev): compatibility with new JSON:API format (#317) Jun 7, 2026
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(payload): support e2e workflow in the OpenAEV payload collector

3 participants