Skip to content

docs: explain pentest retests and scan limits - #89

Merged
zeyu2001 merged 6 commits into
mainfrom
devin/1792578590-pentest-retest-docs
Oct 21, 2026
Merged

zeyu2001 merged 6 commits into
mainfrom
devin/1792578590-pentest-retest-docs

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Add a concise White-box Pentest retest guide covering requests, results, fix notes, and one free retest per scan, with navigation and related links.
  • Put the intended 500-file limit at the end of the Code Review overview. Link the recommendation to review all PRs and MRs, without split advice or extra billing explanations.
  • Use the humanizer skill and keep request limits to the numeric caps.

Rollout note: Zayne chose 500 as the intended limit. The application code inspected still enforces 200 (FILE_CHANGE_CAP in apps/iva/src/seat/dev/dev-seat-pr-usage.service.ts in HacktronAI/web). No application code changed; coordinate rollout before publishing the new limit.

Targeting main with Zayne's approval because docs has no staging branch.

Validation: pnpm lint and pnpm exec mintlify broken-links pass. Browser-verified the latest copy changes on local Mintlify at b338fac; the unchanged overview placement was checked at 30a0a4b. The full guide was checked earlier; product behavior was not tested. Includes a formatter-only fix to the existing changelog entry required by Prettier.

Link to Devin session: https://app.devin.ai/sessions/543f63808f494de7995f5b0a894a79f8
Open in Devin Desktop: https://app.devin.ai/desktop/session/543f63808f494de7995f5b0a894a79f8?variant=devin
Requested by: @zeyu2001

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-21T10:31:30.450567Z af0e2b6 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: af0e2b66a7

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread code-review/overview.mdx Outdated
Comment on lines +12 to +14
Code Review scans pull requests and merge requests with **up to 500 changed
files**. Larger changes are skipped, not partially scanned. Split them into
separate PRs or MRs with no more than 500 changed files each.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Publish the currently enforced 200-file scan limit

For PRs or MRs containing 201–500 changed files, this promises that Code Review will scan them, but the rollout note confirms that production still enforces a 200-file FILE_CHANGE_CAP and skips them. Users following the suggested 500-file split could therefore receive no security review; keep the documentation at 200 until the implementation rollout is complete, or deploy the new limit before publishing these pages.

Useful? React with 👍 / 👎.

@devin-ai-integration

devin-ai-integration Bot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor Author

Browser delta checks passed on local Mintlify at b338fac: shortened troubleshooting text/link, numeric retest limits, and the new report/fix-notes ending. Overview Scan limits is the final section (verified at 30a0a4b, unchanged since).

pnpm lint and pnpm exec mintlify broken-links pass. The full guide was checked earlier; unchanged steps were not rerun, and product behavior was not tested. The rollout caveat remains in the PR body.

Scan limits below Next steps

Retest limits and report ending

Latest browser walkthrough (5x speed)

Latest docs walkthrough

Written by Devin

zeyu2001 and others added 4 commits October 21, 2026 10:42
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@zeyu2001
zeyu2001 merged commit 813fe81 into main Oct 21, 2026
5 checks passed
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