Skip to content

demisto-sdk-release 1.39.13 - #46335

Merged
BEAdi merged 3 commits into
masterfrom
1.39.13
Oct 21, 2026
Merged

demisto-sdk-release 1.39.13#46335
BEAdi merged 3 commits into
masterfrom
1.39.13

Conversation

@content-bot

@content-bot content-bot commented Oct 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Breaking

  • Removed the defaultDataSource field from the pack metadata schema, along with its validators PA131 (IsDefaultDataSourceProvidedValidator) and PA132 (IsValidDefaultDataSourceNameValidator), as all integrations are now shown in the Data Sources screen. #5513

Fix

  • Enforced supportedModules against PlatformSupportedModules in pack metadata, and unified the field's declaration across the strict schemas. #5536
  • Fixed the integration supportlevelheader structure validation (ST110) rejecting valid community, partner and developer values. #5536
  • Fixed an issue where validate in git mode crashed with a parse error when an Agentix agent or skill was renamed. Markdown side files (system instructions, skill markdown) are no longer parsed as content items, and the BA114 validation no longer fails when the previous version of an item cannot be parsed. #5539

Internal

  • Renamed the Collection content item type to Knowledge. #5494

@content-bot

content-bot commented Oct 20, 2026

Copy link
Copy Markdown
Contributor Author

🤖 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

github-actions Bot commented Oct 20, 2026

Copy link
Copy Markdown

Your contributed AbnormalSecurity pack has been modified on files:

Packs/AbnormalSecurity/ReleaseNotes/2_4_10.md
Packs/AbnormalSecurity/pack_metadata.json
Packs/AbnormalSecurity/Integrations/AbnormalSecurityEventCollector/README.md
Please review the changes here

@github-actions

github-actions Bot commented Oct 20, 2026

Copy link
Copy Markdown

Your contributed CiscoETDConnector pack has been modified on files:

Packs/CiscoETDConnector/ReleaseNotes/1_0_1.md
Please review the changes here

@yedidyacohenpalo yedidyacohenpalo added the ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines label Oct 20, 2026
@content-bot

This comment has been minimized.

Sign in to view
@content-bot

content-bot commented Oct 20, 2026

Copy link
Copy Markdown
Contributor Author

Validate summary
The following errors were thrown as a part of this pr: BA133, ST111, DS104, DO106, PA131.
The following errors cannot be ignored: BA133, ST111, DS104, DO106, PA131.
The following errors don't run as part of the nightly flow and therefore can be force merged: BA133, ST111, DS104, DO106.
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

content-bot commented Oct 21, 2026

Copy link
Copy Markdown
Contributor Author

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: a8c581d192c586f5

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

⚠️ AI-generated triage. Validate before acting.

@yedidyacohenpalo yedidyacohenpalo added the ForceMerge Forcing the merge of the PR despite the build status label Oct 21, 2026
@yedidyacohenpalo

yedidyacohenpalo commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

force merge:
PA131 - will be fix in this release
BA133, ST111, DS104, DO106 - not relevant to this pr

@yedidyacohenpalo yedidyacohenpalo mentioned this pull request Oct 21, 2026
Merged
@BEAdi
BEAdi merged commit 9987cbb into master Oct 21, 2026
36 of 37 checks passed
@BEAdi
BEAdi deleted the 1.39.13 branch October 21, 2026 06:34
o101010 pushed a commit to o101010/demisto_content that referenced this pull request Oct 22, 2026
@content-bot Content Bot
@yedidyacohenpalo @o101010
demisto-sdk-release 1.39.13 (demisto#46335)
a76f1d9 
* poetry files

* Apply remove-all-data-source changes onto 1.39.13

* update integration name in release note

---------

Co-authored-by: Content Bot <bot@demisto.com>
Co-authored-by: yedidyacohenpalo <yecohen@paloaltonetworks.com>
@amshamah419 amshamah419 mentioned this pull request Oct 28, 2026
Merged
2 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yedidyacohenpalo yedidyacohenpalo Awaiting requested review from yedidyacohenpalo
@JasBeilin JasBeilin Awaiting requested review from JasBeilin JasBeilin is a code owner
@Moish-Gilboa Moish-Gilboa Awaiting requested review from Moish-Gilboa Moish-Gilboa is a code owner
@RotemAmit RotemAmit Awaiting requested review from RotemAmit RotemAmit is a code owner
@Shellyber Shellyber Awaiting requested review from Shellyber Shellyber is a code owner
@noydavidi noydavidi Awaiting requested review from noydavidi noydavidi is a code owner
@omerKarkKatz omerKarkKatz Awaiting requested review from omerKarkKatz omerKarkKatz is a code owner
@ilappe ilappe Awaiting requested review from ilappe ilappe is a code owner

Assignees

No one assigned

Labels

docs-approved ForceMerge Forcing the merge of the PR despite the build status 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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@content-bot @yedidyacohenpalo @BEAdi @tcarmeli1