content icon indicating copy to clipboard operation
content copied to clipboard
verified publisher icon demisto

File Enrichment Refactor

Open almog2296 opened this issue 2 months ago • 5 comments

refactoring file-enrichment to use AggregatedCommandApiModule.

Status

  • [x] In Progress
  • [ ] Ready
  • [ ] In Hold - (Reason for hold)

Related Issues

fixes: link to the issue

almog2296 avatar Nov 03 '25 16:11 almog2296

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/AggregatedScripts/Scripts/CVEEnrichment
   CVEEnrichment.py130100% 
Packs/AggregatedScripts/Scripts/DomainEnrichment
   DomainEnrichment.py270100% 
Packs/AggregatedScripts/Scripts/FileEnrichment
   FileEnrichment.py210100% 
Packs/AggregatedScripts/Scripts/IPEnrichment
   IPEnrichment.py411270%131–138, 140–143
Packs/AggregatedScripts/Scripts/URLEnrichment
   URLEnrichment.py210100% 
Packs/ApiModules/Scripts/AggregatedCommandApiModule
   AggregatedCommandApiModule.py53411079%127–130, 211–214, 216–217, 255–258, 261–262, 265–266, 290–295, 327–335, 530, 578–579, 676–680, 682–685, 688–691, 693, 695–696, 698–702, 704–705, 804–806, 808–812, 814, 868, 931–933, 935–942, 944, 946–949, 951, 953, 1056–1058, 1061, 1063–1073, 1075, 1118–1119, 1152, 1174, 1186, 1218, 1258
TOTAL65712281% 

Tests Skipped Failures Errors Time
122 0 :zzz: 0 :x: 0 :fire: 6.671s :stopwatch:

github-actions[bot] avatar Nov 03 '25 16:11 github-actions[bot]

@almog2296 docs approved

talihaff avatar Nov 23 '25 13:11 talihaff

This PR was automatically updated by a GitHub Action

  • AggregatedScripts pack version was bumped to 1.3.6.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

content-bot avatar Nov 25 '25 11:11 content-bot

This PR was automatically updated by a GitHub Action

  • AggregatedScripts pack version was bumped to 1.3.7.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

content-bot avatar Nov 27 '25 12:11 content-bot

Validate summary The following errors were thrown as a part of this pr: BC101. The following errors cannot be ignored: BC101. If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged. The following errors don't run as part of the nightly flow and therefore can be force merged: BC101.

Verdict: PR can be force merged from validate perspective? ✅

content-bot avatar Dec 04 '25 09:12 content-bot

This PR requires a ForceMerge. We refactored the file-enrichment script, which involved changing the context structure and command flow to align with the other enrichment scripts. These updates introduce breaking changes to the previous context structure and replace it with a new one.

almog2296 avatar Dec 04 '25 12:12 almog2296