Skip to content

Conversation

@ricekot
Copy link
Member

@ricekot ricekot commented Nov 29, 2025

Ensure that templates and help are up to date with supported fields.

@psiinon
Copy link
Member

psiinon commented Nov 29, 2025

Logo
Checkmarx One – Scan Summary & Details4089811f-239a-4a6b-b73f-bb314f78291f

Great job! No new security vulnerabilities introduced in this pull request


Use @Checkmarx to reach out to us for assistance.

Just send a PR comment with @Checkmarx followed by a natural language request.

Examples: @Checkmarx how are you able to help me? @Checkmarx rescan this PR

Copilot AI review requested due to automatic review settings December 5, 2025 03:12
Copilot finished reviewing on behalf of ricekot December 5, 2025 03:16
Copy link

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

This pull request updates Automation Framework templates and help documentation across multiple ZAP add-ons to ensure they accurately reflect all currently supported fields. The changes primarily add missing field documentation to YAML templates and correct minor inconsistencies in help files.

Key changes include:

  • Adding comprehensive automation documentation for the wappalyzer add-on
  • Updating template files to include previously undocumented fields
  • Correcting parameter references in help documentation

Reviewed changes

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

Show a summary per file
File Description
addOns/wappalyzer/src/main/javahelp/org/zaproxy/zap/extension/wappalyzer/resources/help/toc.xml Adds automation help entry to table of contents
addOns/wappalyzer/src/main/javahelp/org/zaproxy/zap/extension/wappalyzer/resources/help/map.jhm Maps automation help target to HTML file
addOns/wappalyzer/src/main/javahelp/org/zaproxy/zap/extension/wappalyzer/resources/help/index.xml Adds automation help to index
addOns/wappalyzer/src/main/javahelp/org/zaproxy/zap/extension/wappalyzer/resources/help/contents/automation.html New help page documenting the wappalyzer automation job
addOns/wappalyzer/CHANGELOG.md Documents the addition of automation help page
addOns/scripts/src/main/resources/org/zaproxy/zap/extension/scripts/resources/script-max.yaml Adds missing inline field to template
addOns/scripts/src/main/javahelp/org/zaproxy/zap/extension/scripts/resources/help/contents/automation.html Corrects parameter reference from 'file' to 'source'
addOns/scripts/CHANGELOG.md Documents template update
addOns/reports/src/main/resources/org/zaproxy/addon/reports/resources/outputSummary-max.yaml Adds missing rules configuration section
addOns/reports/CHANGELOG.md Documents template update
addOns/client/src/main/resources/org/zaproxy/addon/client/resources/spiderClient-max.yaml Adds missing scopeCheck field
addOns/client/CHANGELOG.md Documents template update
addOns/alertFilters/src/main/resources/org/zaproxy/zap/extension/alertFilters/resources/alertFilter-max.yaml Adds missing ruleName and methods fields
addOns/alertFilters/src/main/javahelp/org/zaproxy/zap/extension/alertFilters/resources/help/contents/automation.html Adds missing ruleName and methods fields to help documentation
addOns/alertFilters/CHANGELOG.md Documents template and help updates

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

@thc202
Copy link
Member

thc202 commented Dec 8, 2025

Needs rebase (at least one changelog update is out of the unreleased section).

Signed-off-by: ricekot <git@ricekot.com>
@thc202
Copy link
Member

thc202 commented Dec 8, 2025

Thank you!

@psiinon psiinon enabled auto-merge December 8, 2025 14:09
@psiinon psiinon merged commit 6a4346a into zaproxy:main Dec 8, 2025
13 of 16 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants