Skip to content

Base: CommonServerPython documentation and metadata updates#44158

Merged
amshamah419 merged 3 commits intomasterfrom
base-commonserverpython-fileresult-update
May 3, 2026
Merged

Base: CommonServerPython documentation and metadata updates#44158
amshamah419 merged 3 commits intomasterfrom
base-commonserverpython-fileresult-update

Conversation

@amshamah419
Copy link
Copy Markdown
Contributor

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

fixes: https://jira-dc.paloaltonetworks.com/browse/CIAC-16543

Description

reverts change introduced in version 1.41.77

Must have

  • Tests
  • Documentation

- Updated CommonServerPython with documentation and metadata updates.
- Bumped Base to 1.41.78.
@amshamah419 amshamah419 self-assigned this May 3, 2026
Copilot AI review requested due to automatic review settings May 3, 2026 10:39
@amshamah419 amshamah419 added docs-approved ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines labels May 3, 2026
@content-bot
Copy link
Copy Markdown
Contributor

🤖 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

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Reverts a previously introduced change in CommonServerPython.fileResult filename handling (per CIAC-16543) and updates the Base pack version/release notes accordingly.

Changes:

  • Bumps Base pack currentVersion to 1.41.78.
  • Updates CommonServerPython.fileResult filename sanitization behavior and related inline comments.
  • Removes a fileResult sanitization test class and adds a 1.41.78 release notes entry.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
Packs/Base/pack_metadata.json Version bump to 1.41.78.
Packs/Base/Scripts/CommonServerPython/CommonServerPython.py Changes fileResult filename sanitization and debug logging.
Packs/Base/Scripts/CommonServerPython/CommonServerPython_test.py Removes dedicated fileResult sanitization tests.
Packs/Base/ReleaseNotes/1_41_78.md Adds release notes for 1.41.78 (currently generic).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Packs/Base/Scripts/CommonServerPython/CommonServerPython.py
Comment thread Packs/Base/Scripts/CommonServerPython/CommonServerPython_test.py
Comment thread Packs/Base/ReleaseNotes/1_41_78.md
Comment thread Packs/Base/Scripts/CommonServerPython/CommonServerPython.py
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

Coverage

Tests Skipped Failures Errors Time
6671 226 💤 0 ❌ 0 🔥 4m 31s ⏱️

@content-bot
Copy link
Copy Markdown
Contributor

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: 4eb65658235c25ba

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

⚠️ AI-generated triage. Validate before acting.

@content-bot
Copy link
Copy Markdown
Contributor

Validate summary
The following errors were thrown as a part of this pr: .
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? ✅

@amshamah419 amshamah419 merged commit f9e6bf6 into master May 3, 2026
21 of 22 checks passed
@amshamah419 amshamah419 deleted the base-commonserverpython-fileresult-update branch May 3, 2026 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-approved ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines skip-ai-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants