Skip to content

feat(catalog): reload catalog when a new manifest is fetched (#18280) - #18332

Draft
esrevi wants to merge 45 commits into
issue/16059-a-runtime-behaviorfrom
issue/18280-a
Draft

esrevi wants to merge 45 commits into
issue/16059-a-runtime-behaviorfrom
issue/18280-a

Conversation

@esrevi

@esrevi esrevi commented Sep 17, 2026

Copy link
Copy Markdown
Member

Proposed changes

  • Add query catalogsRevisions to compare current revision with backend catalog revision,
  • On new revision, query catalog and update ingestion catalog

Related issues

How to test this PR

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant use cases (coverage and e2e)
  • I added/updated the relevant documentation (either on GitHub or on Notion)
  • Where necessary, I refactored code to improve the overall quality

Further comments

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
… (#0)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@esrevi esrevi self-assigned this Sep 17, 2026
@esrevi esrevi added the filigran team Item from the Filigran team. label Sep 17, 2026
@github-actions github-actions Bot added the migration For pull request that includes a migration script. label Sep 17, 2026
@codecov

codecov Bot commented Sep 17, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 11.11111% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 32.65%. Comparing base (8b9e064) to head (ca82fa2).

Files with missing lines Patch % Lines
...ncti-graphql/src/modules/catalog/catalog-domain.ts 11.11% 8 Missing ⚠️
...-graphql/src/modules/catalog/catalog-repository.ts 14.28% 6 Missing ⚠️
...ti-graphql/src/modules/catalog/catalog-resolver.ts 0.00% 2 Missing ⚠️
Additional details and impacted files
@@                         Coverage Diff                         @@
##           issue/16059-a-runtime-behavior   #18332       +/-   ##
===================================================================
+ Coverage                           21.41%   32.65%   +11.24%     
===================================================================
  Files                                3479     1050     -2429     
  Lines                              141783    61978    -79805     
  Branches                            38285    12095    -26190     
===================================================================
- Hits                                30367    20242    -10125     
+ Misses                             111416    41736    -69680     
Flag Coverage Δ
opencti-client-python 49.85% <ø> (ø)
opencti-front ?
opencti-graphql 28.63% <11.11%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 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.

@esrevi
esrevi changed the base branch from master to issue/16059-a-runtime-behavior September 17, 2026 13:29
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. migration For pull request that includes a migration script.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(xtm-integration): load catalog when new manifest is fetched

2 participants