docs: add beginner-friendly command examples to Usage.md#1503
docs: add beginner-friendly command examples to Usage.md#1503StarDustNova-57 wants to merge 2 commits intoOWASP:masterfrom
Conversation
Added structured and beginner-friendly command examples in docs/Usage.md. Examples include: - Basic scan - Scanning multiple targets from file - Saving output to JSON - Brute-force module usage - Scanning specific port ranges This improves usability for new users and aligns with issue OWASP#740. Signed-off-by: Swetha Varma <swethakv05@gmail.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
Summary by CodeRabbit
WalkthroughUpdated Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/Usage.md`:
- Around line 374-390: Add a language identifier (e.g., bash) to each fenced
command block that shows example invocations of nettacker (the blocks containing
"python nettacker.py -i example.com -m port_scan", "python nettacker.py -l
targets.txt -m all", "python nettacker.py -i example.com -o result.json",
"python nettacker.py -i example.com -m ssh_brute -U users.txt -P passwords.txt",
and "python nettacker.py -i example.com -m port_scan -g 20-100") so the markdown
fences become ```bash ... ``` to satisfy markdownlint MD040 and ensure
consistent rendering.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 570e3a9a-7b86-46f9-9334-735c20114847
📒 Files selected for processing (1)
docs/Usage.md
Signed-off-by: Swetha Varma <swethakv05@gmail.com>
|
@coderabbitai remember to check for and flag any duplicate or conflicting PRs |
|
Tip For best results, initiate chat on the files or code changes. 🧩 Analysis chain🏁 Script executed: Length of output: 26522
✅ Commit SignaturesBoth commits in PR
|
| PR | Author | Title | Filed | Conflict Type |
|---|---|---|---|---|
| #1499 | StarDustNova-57 (same author!) | docs: replace removed whatcms_scan with web_technologies_scan | 2026-04-03 | docs/Usage.md simultaneously |
| #1433 | swethas274 | docs: remove stale whatcms_scan references from docs and locale files | 2026-03-21 | Direct file conflict on docs/Usage.md |
| #1413 | dhruv7539 | docs: replace removed whatcms_scan examples | 2026-03-16 | Direct file conflict on docs/Usage.md (oldest open PR on this file) |
Additional thematic overlaps (different files):
- PR
#1446(pandeysudarshan16-ctrl) — "Enhance CLI help output with usage examples" — adds usage examples for new users but targets the CLI help code rather thandocs/Usage.md. - PR
#1421(RSN601KRI) — "Enhance installation docs with onboarding, quick start, and best practices" — beginner-friendly doc improvements, different file.
⚠️ Additional Flags
-
Same-author double submission:
@StarDustNova-57has both PR#1499(replace whatcms_scan indocs/Usage.md) and PR#1503(add examples todocs/Usage.md) open simultaneously. The author checked the "no duplicate work" checklist item, yet their own PR#1499directly touches the same file. These two PRs will conflict with each other. -
Three-way race on
docs/Usage.md: PRs#1413,#1433, and#1499are all open and all modify the same file. Only one should be merged first; the remaining will require rebase or will conflict. PR#1413is the oldest (2026-03-16) and narrowest in scope. -
Contribution farming indicator: PR
#1503is a +20/-1 line documentation-only addition with no functional impact, no linked test coverage, and the same author already has another open PR (#1499) on the same file. Maintainers should evaluate whether this represents split contributions that should have been a single PR.
Recommendation to maintainers: Resolve the three-way conflict on docs/Usage.md (#1413, #1433, #1499) first, then evaluate whether PR #1503 can be cleanly applied on top. @StarDustNova-57 should also be asked to consolidate or close one of their two concurrent PRs (#1499 or #1503) modifying the same file.
🧠 Learnings used
Learnt from: securestep9
Repo: OWASP/Nettacker PR: 0
File: :0-0
Timestamp: 2026-04-04T10:12:27.395Z
Learning: When reviewing PRs in OWASP/Nettacker, always search open PRs for duplicates or conflicts by checking other open PRs that modify the same files or address the same topic. Report all duplicate PRs to maintainers, identify which PR is broader/older, and recommend closing the narrower/newer duplicate. Flag duplicate PRs alongside other pre-merge check failures (unsigned commits, PR template non-compliance, contribution farming).
Learnt from: securestep9
Repo: OWASP/Nettacker PR: 0
File: :0-0
Timestamp: 2026-01-10T14:58:54.638Z
Learning: When reviewing a pull request, always analyze other open PRs in the repository to identify potential conflicts, such as duplicate changes to the same files, overlapping features, or merge conflicts. Use GitHub CLI to search for related PRs by file changes, keywords, or linked issues.
Learnt from: securestep9
Repo: OWASP/Nettacker PR: 0
File: :0-0
Timestamp: 2026-01-15T17:07:23.769Z
Learning: When securestep9 requests to check commit signatures in open PRs, always verify all commits are signed using GitHub CLI and report unsigned commits immediately with explicit notification that the PR fails pre-merge checks.
Learnt from: securestep9
Repo: OWASP/Nettacker PR: 0
File: :0-0
Timestamp: 2026-01-13T22:48:26.796Z
Learning: When reviewing any pull request in the OWASP/Nettacker repository, always remind the PR author to use signed commits and explicitly state that the PR fails pre-merge checks if any commits are unsigned. Tag the author by username and provide instructions on how to sign commits.
Learnt from: securestep9
Repo: OWASP/Nettacker PR: 0
File: :0-0
Timestamp: 2026-04-01T22:22:07.355Z
Learning: Always check that PR submissions in OWASP/Nettacker follow the PR template at .github/PULL_REQUEST_TEMPLATE.md. The template requires three sections: "Proposed change" (with description and linked issue), "Type of change" (exactly one checkbox), and "Checklist" (11 items including: following contributing guidelines, digitally signing commits, running `make pre-commit` and `make test`, updating docs, linking an open issue, attaching screenshots, confirming no unreviewed AI output, and confirming sole authorship). Flag PRs that omit the template as non-compliant.
Learnt from: securestep9
Repo: OWASP/Nettacker PR: 0
File: :0-0
Timestamp: 2026-04-03T12:22:41.057Z
Learning: In OWASP/Nettacker, always verify that PR authors have a properly configured Git identity (author name should not be "unknown" in commit metadata). Flag PRs where commit author name is "unknown" as this indicates a misconfigured Git environment and should be flagged alongside unsigned commits as a pre-merge check failure.
Learnt from: securestep9
Repo: OWASP/Nettacker PR: 0
File: :0-0
Timestamp: 2026-04-01T22:22:07.355Z
Learning: Flag low-value or "contribution farming" PRs in OWASP/Nettacker. Indicators include: trivial/cosmetic changes (e.g., minor README edits), no linked issue, no functional impact, no tests, content that duplicates existing documentation, and first-time contributors making only superficial changes. Such PRs should be flagged to maintainers and the author asked to provide a linked issue and full PR template compliance before the PR is considered for merge.
Learnt from: Gauravsharma2040
Repo: OWASP/Nettacker PR: 0
File: :0-0
Timestamp: 2026-03-20T16:54:40.862Z
Learning: In OWASP/Nettacker, PR `#1429` (by VivekGhantiwala, branch fix/exec-code-injection-responsetime) is a third duplicate fixing the exec() vulnerability in response_conditions_matched() in nettacker/core/lib/http.py — alongside PR `#1335` (Prajwal5755, uses operator module) and PR `#1409` (Gauravsharma2040, uses float dunder methods + isinstance guard). All three are open as of March 2026. Only one should be merged.
Proposed change
This PR adds structured and beginner-friendly command examples to
docs/Usage.md.The goal is to help new users understand how to use Nettacker with clear, practical examples.
Added examples include:
This improves usability and reduces confusion for first-time users.
Fixes #740
Type of change
Checklist
make pre-commitand confirm it didn't generate any warnings/changesmake testand I confirm all tests passed locallydocs/folder