Skip to content

removed cloud_appsec from agentixagent supported modules#43723

Closed
Shir2611 wants to merge 7 commits intomasterfrom
cleanup-invalid-license-agentixagent-cloud_appsec
Closed

removed cloud_appsec from agentixagent supported modules#43723
Shir2611 wants to merge 7 commits intomasterfrom
cleanup-invalid-license-agentixagent-cloud_appsec

Conversation

@Shir2611
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: CIAC-15886

Description

removed cloud_appsec from agentixagent supported modules
MATCH (a:AgentixAgent) WHERE 'cloud_appsec' IN a.supportedModules RETURN a.name AS agent_name, a.supportedModules AS supported_modules ORDER BY a.name

Must have

  • Tests
  • Documentation

@Shir2611 Shir2611 requested a review from BEAdi March 29, 2026 11:09
@Shir2611 Shir2611 self-assigned this Mar 29, 2026
@Shir2611 Shir2611 added docs-approved bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines supported-modules-approved ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. labels Mar 29, 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

@marketplace-ai-reviewer marketplace-ai-reviewer removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Mar 29, 2026
@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

🤖 Analysis started. Please wait for results...

@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

🤖 AI Review Disclaimer

This review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause.

Copy link
Copy Markdown
Contributor

@marketplace-ai-reviewer marketplace-ai-reviewer left a comment

Choose a reason for hiding this comment

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

Hi! Thanks for contributing the AIAgents pack. I've taken a look and just have a few quick updates needed for the pack metadata. Please expand the description to better explain the pack's purpose, add the vendor name to the keywords, and include the missing created date. Thanks again for your work on this!

Additionally, please address the following file-level notes:

  • Packs/AIAgents/pack_metadata.json: - The description field is too vague and just repeats the pack name. Please provide a concise, high-level summary of the pack's purpose (e.g., "Integrate with..." or "Helps manage...").
  • The vendor name (e.g., "Cortex" or "XSOAR") must be included in the keywords list.
  • The mandatory created key is missing from the pack metadata. Please add it with the pack's creation date.

@BEAdi please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

🤖 Analysis started. Please wait for results...

@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

🤖 AI Review Disclaimer

This review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause.

Copy link
Copy Markdown
Contributor

@marketplace-ai-reviewer marketplace-ai-reviewer left a comment

Choose a reason for hiding this comment

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

🔄 Re-reviewing updates...

Important

To maintain review integrity, threads marked as Resolved without accompanying changes or explanations will be automatically unresolved for further verification.

Thank you for your contribution! Re-reviewed your PR, No new issues found. Great work! 👍
Please check the previous comments to ensure all feedback has been resolved.

@marketplace-ai-reviewer, @BEAdi please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

@content-bot
Copy link
Copy Markdown
Contributor

Validate summary
The following errors were thrown as a part of this pr: BC115, GR110, BC116.
The following errors cannot be ignored: BC115, GR110, BC116.
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: BC115, GR110, BC116.

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

@Shir2611 Shir2611 closed this Apr 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url docs-approved ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines supported-modules-approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants