Skip to content

Update Sucuri extension docs for current scan flow and metadata - #91

Merged
bkatusic merged 1 commit into
mainfrom
docs-review/sucuri-extension
Jul 13, 2026
Merged

bkatusic merged 1 commit into
mainfrom
docs-review/sucuri-extension

Conversation

@bkatusic

@bkatusic bkatusic commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Refresh the Sucuri extension page with the current add-on version and updated integration wording
  • Clarify that the extension connects to Sucuri SiteCheck, supports manual single-site scans, and does not require the Sucuri plugin on child sites
  • Update the step-by-step instructions and report/settings notes to match the current MainWP Dashboard navigation and behavior

Testing

  • Not run (not requested)

Summary by CodeRabbit

  • Documentation
    • Updated the Sucuri add-on documentation for version 5.0.3.
    • Clarified that a Sucuri plugin or subscription is not required.
    • Documented manual, one-site-at-a-time scanning limitations.
    • Updated scan instructions, navigation labels, report settings, reminders, and SSL verification guidance.

@coderabbitai

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

The Sucuri add-on page now advertises version 5.0.3, clarifies plugin and subscription requirements, documents manual one-site scans, and updates report navigation and site-wide SSL verification guidance.

Changes

Sucuri documentation

Layer / File(s) Summary
Metadata and scan guidance
add-ons/security/sucuri-extension.mdx
Updates the advertised version, Sucuri requirements, scan limitations, subscription notes, and scan navigation instructions.
Reports and settings guidance
add-ons/security/sucuri-extension.mdx
Revises previous-report navigation and states that SSL verification applies to all sites while retaining report and reminder options.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • mainwp/docs#88: Updates related security add-on MDX metadata and instructional text.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the doc updates to the Sucuri extension’s scan flow, metadata, and related wording.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs-review/sucuri-extension

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
add-ons/security/sucuri-extension.mdx (1)

92-94: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

SSL bullet says "applies to all sites" but is reached from a per-site page.

The step just above (Line 81) says to locate one specific site and click its "Reports & Settings" button, yet the resulting toggle is described as global. Consider a brief clarifying note (e.g., "this is a global setting shown here for convenience") to avoid confusing readers about per-site vs. global scope.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@add-ons/security/sucuri-extension.mdx` around lines 92 - 94, Clarify the
“Verify SSL Certificate” bullet in the Sucuri settings instructions by stating
that it is a global setting displayed on the per-site page for convenience. Keep
the existing per-site report and reminder instructions unchanged.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@add-ons/security/sucuri-extension.mdx`:
- Around line 92-94: Clarify the “Verify SSL Certificate” bullet in the Sucuri
settings instructions by stating that it is a global setting displayed on the
per-site page for convenience. Keep the existing per-site report and reminder
instructions unchanged.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: bf62acd9-f77d-48e3-82f6-eb40718812bd

📥 Commits

Reviewing files that changed from the base of the PR and between 15ae762 and 58449f6.

📒 Files selected for processing (1)
  • add-ons/security/sucuri-extension.mdx

@bkatusic
bkatusic merged commit 4b0073f into main Jul 13, 2026
4 checks passed
@bkatusic
bkatusic deleted the docs-review/sucuri-extension branch July 13, 2026 15:21
@mintlify

mintlify Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
mainwp 🔴 Failed Jul 13, 2026, 3:44 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant