Sk csp break return type by shmuel44 · Pull Request #45610 · demisto/content

shmuel44 · 2026-08-23T10:49:50Z

Contributing to Cortex XSOAR Content

Make sure to register your contribution by filling the contribution registration form

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

In Progress
Ready
In Hold - (Reason for hold)

Related Issues

Description

Must have

Tests
Documentation

Deliberate breaking change for CIAC-17274 AI-adjudication testing. The function now returns a list wrapping the war room entry dict instead of the dict itself, breaking callers that index or return it directly. Disposable test branch, MUST NOT be merged.

content-bot · 2026-08-23T10:50:29Z

CommonServerPython dependency impact

Impacted content items were found and need manual verification.

Changed symbols: 1
Public entrypoints: 1
Affected files: 9
Affected packs: 5

CommonScripts

Packs/CommonScripts/Scripts/ExportToXLSX/ExportToXLSX.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/PDFUnlocker/PDFUnlocker.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/ReadPDFFileV2/ReadPDFFileV2.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/UnPackFile/UnPackFile.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/script-PCAPMiner.yml - matched: file_result_existing_file

GoogleCloudStorage

Packs/GoogleCloudStorage/Integrations/GoogleCloudStorage/GoogleCloudStorage.py - matched: file_result_existing_file

ReversingLabs_A1000

Packs/ReversingLabs_A1000/Integrations/ReversingLabsA1000/ReversingLabsA1000.py - matched: file_result_existing_file

ThreatZone

Packs/ThreatZone/Integrations/ThreatZone/ThreatZone.py - matched: file_result_existing_file

Zoom

Packs/Zoom/Integrations/Zoom/Zoom.py - matched: file_result_existing_file

Open the HTML impact checklist

To accept this change without verifying each item, apply one of these PR labels: I'm okay with it, not-relevant-for-csp-validation.

_{Analyzed commit b4a6aaf7 · generated 2026-08-23T20:34:41Z · CI job}

content-bot · 2026-08-23T10:51:05Z

🤖 AI-Powered Code Review Available

You can leverage AI-powered code review to assist with this PR!

Available Commands:

@marketplace-ai-reviewer start review - Initiate a full AI code review
@marketplace-ai-reviewer re-review - Incremental review for new commits

github-actions · 2026-08-23T10:51:07Z

⚠️ `CommonServerPython` change detected

This PR modifies one of the runtime-injected helper scripts (CommonServerPython.py / CommonServerPowerShell.ps1 / CommonServer.js).
A change here can impact every integration and script in the repository, so the standard PR pipeline is not enough.

Required before merge:

Trigger the content nightly pipeline against this branch.
Make sure the nightly run is green (all jobs passed).
Add the nightly-run-passed label to this PR so this check turns green.

If you believe nightly is not required for this change, please justify it in a PR comment and add the label to unblock the check.

github-actions · 2026-08-23T10:56:12Z

Tests	Skipped	Failures	Errors	Time
7287	269 💤	0 ❌	0 🔥	5m 9s ⏱️

content-bot · 2026-08-23T10:56:55Z

CommonServerPython dependency impact

The AI adjudication stage confirmed 1 breaking file.

Adjudicated: 1 breaking, 0 warning, 8 safe.

Impacted content items were found and need manual verification.

Changed symbols: 1
Public entrypoints: 1
Affected files: 9
Affected packs: 5

The AI adjudication stage flagged these files as breaking:

File	Confidence	Why
`Packs/ThreatZone/Integrations/ThreatZone/ThreatZone.py`	high	The call at line 221 (inside Client._download_submission_asset, declared as returning dict) bubbles up through threatzone_get_sanitized_file/threatzone_get_html...

CommonScripts

Packs/CommonScripts/Scripts/ExportToXLSX/ExportToXLSX.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/PDFUnlocker/PDFUnlocker.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/ReadPDFFileV2/ReadPDFFileV2.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/UnPackFile/UnPackFile.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/script-PCAPMiner.yml - matched: file_result_existing_file

GoogleCloudStorage

Packs/GoogleCloudStorage/Integrations/GoogleCloudStorage/GoogleCloudStorage.py - matched: file_result_existing_file

ReversingLabs_A1000

Packs/ReversingLabs_A1000/Integrations/ReversingLabsA1000/ReversingLabsA1000.py - matched: file_result_existing_file

ThreatZone

Packs/ThreatZone/Integrations/ThreatZone/ThreatZone.py - matched: file_result_existing_file

Zoom

Packs/Zoom/Integrations/Zoom/Zoom.py - matched: file_result_existing_file

Open the HTML impact checklist

To accept this change without verifying each item, apply one of these PR labels: I'm okay with it, not-relevant-for-csp-validation.

_{Analyzed commit 05af24f8 · generated 2026-08-23T10:50:27Z · CI job}

content-bot · 2026-08-23T16:54:29Z

CommonServerPython dependency impact

The AI adjudication stage confirmed 1 breaking file.

Adjudicated: 1 breaking, 2 warning, 6 safe.

Impacted content items were found and need manual verification.

Changed symbols: 1
Public entrypoints: 1
Affected files: 9
Affected packs: 5

The AI adjudication stage flagged these files as breaking:

File	Confidence	Why
`Packs/ThreatZone/Integrations/ThreatZone/ThreatZone.py`	high	Line 221 (`return file_result_existing_file(filename)`) is the return statement of Client._download_submission_asset, whose return type is annotated `-> dict` a...

CommonScripts

Packs/CommonScripts/Scripts/ExportToXLSX/ExportToXLSX.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/PDFUnlocker/PDFUnlocker.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/ReadPDFFileV2/ReadPDFFileV2.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/UnPackFile/UnPackFile.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/script-PCAPMiner.yml - matched: file_result_existing_file

GoogleCloudStorage

Packs/GoogleCloudStorage/Integrations/GoogleCloudStorage/GoogleCloudStorage.py - matched: file_result_existing_file

ReversingLabs_A1000

Packs/ReversingLabs_A1000/Integrations/ReversingLabsA1000/ReversingLabsA1000.py - matched: file_result_existing_file

ThreatZone

Packs/ThreatZone/Integrations/ThreatZone/ThreatZone.py - matched: file_result_existing_file

Zoom

Packs/Zoom/Integrations/Zoom/Zoom.py - matched: file_result_existing_file

Open the HTML impact checklist

To accept this change without verifying each item, apply one of these PR labels: I'm okay with it, not-relevant-for-csp-validation.

_{Analyzed commit 212030d5 · generated 2026-08-23T16:48:47Z · CI job}

content-bot · 2026-08-23T20:30:44Z

⚠️ The PR is missing the ready-for-pipeline-running label. Please add the label when the PR is ready in order to proceed.

content-bot · 2026-08-23T20:40:47Z

CommonServerPython dependency impact

The AI adjudication stage confirmed 1 breaking file.

Adjudicated: 1 breaking, 2 warning, 6 safe.

Impacted content items were found and need manual verification.

Changed symbols: 1
Public entrypoints: 1
Affected files: 9
Affected packs: 5

The AI adjudication stage flagged these files as breaking:

File	Confidence	Why
`Packs/ThreatZone/Integrations/ThreatZone/ThreatZone.py`	high	At line 221, `_download_submission_asset` returns `file_result_existing_file(filename)`, whose result now flows (via `threatzone_get_sanitized_file`) into the `...

CommonScripts

Packs/CommonScripts/Scripts/ExportToXLSX/ExportToXLSX.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/PDFUnlocker/PDFUnlocker.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/ReadPDFFileV2/ReadPDFFileV2.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/UnPackFile/UnPackFile.py - matched: file_result_existing_file
Packs/CommonScripts/Scripts/script-PCAPMiner.yml - matched: file_result_existing_file

GoogleCloudStorage

Packs/GoogleCloudStorage/Integrations/GoogleCloudStorage/GoogleCloudStorage.py - matched: file_result_existing_file

ReversingLabs_A1000

Packs/ReversingLabs_A1000/Integrations/ReversingLabsA1000/ReversingLabsA1000.py - matched: file_result_existing_file

ThreatZone

Packs/ThreatZone/Integrations/ThreatZone/ThreatZone.py - matched: file_result_existing_file

Zoom

Packs/Zoom/Integrations/Zoom/Zoom.py - matched: file_result_existing_file

Open the HTML impact checklist

To accept this change without verifying each item, apply one of these PR labels: I'm okay with it, not-relevant-for-csp-validation.

_{Analyzed commit b4a6aaf7 · generated 2026-08-23T20:34:41Z · CI job}

content-bot · 2026-08-23T21:00:16Z

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

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

content-bot · 2026-08-23T21:52:21Z

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: ee67ba968240189e

📋 Triage Report
💡 Resolutions are available in the full report.

⚠️ AI-generated triage. Validate before acting.

shmuel44 added 2 commits August 23, 2026 05:04

ci: pin infra CI ref to sk-csp-validation for CSP validator testing

891a1a8

This comment has been minimized.

Sign in to view

chore: re-run CSP adjudication against updated infra

106f9e2

This comment has been minimized.

Sign in to view

chore: re-run CSP adjudication against updated prompt

f499312

Navigation Menu

Uh oh!

Sk csp break return type - #45610

Sk csp break return type#45610
shmuel44 wants to merge 4 commits into
masterfrom
sk_csp_break_return_type

shmuel44 commented Aug 23, 2026

Uh oh!

content-bot commented Aug 23, 2026 •

edited

Loading

Uh oh!

content-bot commented Aug 23, 2026

Uh oh!

github-actions Bot commented Aug 23, 2026

Uh oh!

github-actions Bot commented Aug 23, 2026 •

edited

Loading

Uh oh!

content-bot commented Aug 23, 2026

Uh oh!

This comment has been minimized.

content-bot commented Aug 23, 2026

Uh oh!

This comment has been minimized.

content-bot commented Aug 23, 2026

Uh oh!

content-bot commented Aug 23, 2026

Uh oh!

content-bot commented Aug 23, 2026

Uh oh!

content-bot commented Aug 23, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

shmuel44 commented Aug 23, 2026

Contributing to Cortex XSOAR Content

Status

Related Issues

Description

Must have

Uh oh!

content-bot commented Aug 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CommonServerPython dependency impact

Uh oh!

content-bot commented Aug 23, 2026

🤖 AI-Powered Code Review Available

Uh oh!

github-actions Bot commented Aug 23, 2026

⚠️ CommonServerPython change detected

Uh oh!

github-actions Bot commented Aug 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

content-bot commented Aug 23, 2026

CommonServerPython dependency impact

Uh oh!

This comment has been minimized.

content-bot commented Aug 23, 2026

CommonServerPython dependency impact

Uh oh!

This comment has been minimized.

content-bot commented Aug 23, 2026

Uh oh!

content-bot commented Aug 23, 2026

CommonServerPython dependency impact

Uh oh!

content-bot commented Aug 23, 2026

Uh oh!

content-bot commented Aug 23, 2026

🔍 AI Triage Report Available

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

content-bot commented Aug 23, 2026 •

edited

Loading

⚠️ `CommonServerPython` change detected

github-actions Bot commented Aug 23, 2026 •

edited

Loading