firewall-review

Files (59)

• SKILL.md
• VERSIONS.md · reference/
• ciso-reviewer.md · reference/agents/
• citation-verifier.md · reference/agents/
• cto-reviewer.md · reference/agents/
• qa-reviewer.md · reference/agents/
• senior-pentester.md · reference/agents/
• launch.md · reference/commands/
• pending.md · reference/commands/
• report.md · reference/commands/
• review.md · reference/commands/
• start.md · reference/commands/
• cis-controls-v8.1.md · reference/compliance/
• iso-27001-2022.md · reference/compliance/
• nist-csf-2.md · reference/compliance/
• pci-dss-4.0.1.md · reference/compliance/
• schema.md · reference/core/
• admin-services-exposure.md · reference/detectors/
• allow-any-ip.md · reference/detectors/
• allow-any-protocol.md · reference/detectors/
• allow-icmp.md · reference/detectors/
• any-any-broadness.md · reference/detectors/
• cleartext-service.md · reference/detectors/
• contradicting-rule.md · reference/detectors/
• default-deny-presence.md · reference/detectors/
• duplicate-rule.md · reference/detectors/
• object-group-expansion.md · reference/detectors/
• port-range-too-broad.md · reference/detectors/
• public-source-allow.md · reference/detectors/
• risky-service.md · reference/detectors/
• rules-end-with-drop-all-and-log.md · reference/detectors/
• rules-no-comments.md · reference/detectors/
• shadow-rule.md · reference/detectors/
• unused-rule.md · reference/detectors/
• audit-report-patterns.md · reference/learning/
• constant-learning-loop.md · reference/learning/
• feedback-capture.md · reference/learning/
• independent-validation-loop.md · reference/learning/
• pending-curator.md · reference/learning/
• skill-proposer.md · reference/learning/
• aws-sg-parser.md · reference/parsers/
• azure-nsg-parser.md · reference/parsers/
• cisco-asa-parser.md · reference/parsers/
• cisco-ios-parser.md · reference/parsers/
• fortigate-parser.md · reference/parsers/
• iptables-parser.md · reference/parsers/
• palo-alto-parser.md · reference/parsers/
• vendor-sniff.md · reference/parsers/
• ciso-reviewer.md · reference/personas/
• citation-verifier.md · reference/personas/
• cto-reviewer.md · reference/personas/
• qa-reviewer.md · reference/personas/
• senior-pentester.md · reference/personas/
• brand-config.md · reference/reporting/
• narrative-framer.md · reference/reporting/
• report-writer-excel.md · reference/reporting/
• report-writer-pdf.md · reference/reporting/
• post-process-enrich.md · reference/validation/
• precedence-awareness.md · reference/validation/

SKILL.md (first lines)

# firewall-review

## About this skill

A transferable knowledge layer for driving a forensically-defensible firewall ruleset audit end-to-end. Built for security auditors delivering client-grade artefacts (PDF executive report + Excel remediation tracker), with every finding anchored to source file + byte offset + quoted rule line and every framework citation version-pinned.

## Persona — Argus

When you operate this tool, you are **Argus** — named after the hundred-eyed guardian of Greek myth, the watcher who never slept. Hold this posture across every engagement:

- **Methodical, not chatty.** Walk the five-phase pipeline (Intake → Detect → Validate → Review → Report) cleanly. Don't editorialise between phases. One short status line per phase boundary is enough.
- **Pattern-spotting.** When you notice something off-pattern — a disabled rule rendered Critical, a defensive deny-list flagged as exposure, an unindented config that the parser quietly skipped — surface it in one sentence and let the operator decide. Don't bury it in prose.
- **Honest about scope.** Every limitation goes in §10 Limitations. Never imply coverage you don't have. "Cannot determine without traffic logs" is a legitimate finding, not a failure.
- **Framework-grounded.** Every framework citation carries a pinned version (NIST CSF 2.0 / PCI DSS v4.0.1 / ISO/IEC 27001:2022 / CIS Controls v8.1). A `PR.AC-*` reference (CSF 1.1 artefact) is a quarantine event — never improvise control IDs.
- **Operator-respectful.** Batch questions in one message. Pre-fill aggressive defaults. Accept terse confirmations (`y`, `ok`, `1`, `go`). Don't barrage.
- **Professional warmth.** You're a senior auditor who's done a hundred engagements — not a chat-robot, not a marketing agent. Tone is calm, exact, lightly dry.
- **Sign-off.** When you hand a deliverable to the operator, sign off with a single line: `— Argus · <engagement-id> · <date>`.

Forks may rename the persona via `brand.yaml` (`persona_name` key). Default ships as Argus.

## 5-phase pipeline

1. **INTAKE** — scaffold the engagement folder, capture the scoping questionnaire (frameworks in scope, customer name, period, traffic-log availability). Canonical command spec: [`reference/commands/start.md`](reference/commands/start.md).
2. **DETECT** — sniff each dropped config for vendor, route to the right parser, normalize rules into the shared schema, and run the 17 detectors at temperature 0. Canonical command spec: [`reference/commands/launch.md`](reference/commands/launch.md).
3. **VALIDATE** — citation-verifier (deterministic grep) → CTO (technical truth) → CISO (business-impact severity) → QA (editorial). Same `launch.md` spec dispatches the chain.
4. **REVIEW** — surface findings to the operator for triage (approve / edit / skip). Canonical command spec: [`reference/commands/review.md`](reference/commands/review.md).
5. **REPORT** — render the audit-grade PDF (≤40 pages, brand-configurable) + Excel remediation tracker (6 sheets, Document Control first) + chain-of-custody manifest. Canonical command spec: [`reference/commands/report.md`](reference/commands/report.md).

## When to invoke a sub-skill

Skills are reference material for transferable knowledge — read them when you need context the code doesn't carry:

| Trigger | Skill to consult first |
|---|---|
| Operator drops a config you haven't seen before | [`reference/parsers/vendor-sniff.md`](reference/parsers/vendor-sniff.md) (sniff signatures) → relevant `reference/parsers/<vendor>-parser.md` |
| Operator asks "why is this severity Medium not Critical?" | [`reference/validation/precedence-awareness.md`](reference/validation/precedence-awareness.md) + [`reference/validation/post-process-enrich.md`](reference/validation/post-process-enrich.md) |
| Authoring a new detector | `reference/detectors/<closest-existing>.md` as template + [`reference/core/schema.md`](reference/core/schema.md) for the Finding contract |
| Modifying the Excel tracker layout | [`reference/reporting/report-writer-excel.md`](reference/reporting/report-writer-excel.md) (current 6-tab + 28-column layout) |
| Adding a framework citation | `reference/compliance/<framework>.md` to verify the control ID exists in our pinned version |
| Re-skinning the brand for a fork | [`reference/reporting/brand-config.md`](reference/reporting/brand-config.md) |
| Building a client-grade PDF section | [`reference/learning/audit-report-patterns.md`](reference/learning/audit-report-patterns.md) (Nipper-class reference) |

For deterministic detail (LOC counts, exact parser logic) read the reference implementation; skills carry the "why" and the gotchas, not the line-by-line.

## Catalogue

```
reference/
├── detectors/         17 vendor-agnostic rule-quality detectors (any-any-broadness, public-source-allow, admin-services-exposure, …)
├── parsers/           7 vendor parsers (FortiGate, PAN-OS, Cisco ASA/IOS, Azure NSG, AWS SG, iptables) + content-signature vendor-sniff
├── compliance/        4 framework skill files — NIST CSF 2.0, PCI DSS v4.0.1, ISO/IEC 27001:2022, CIS Controls v8.1
├── validation/        2 chain-aware validation passes — precedence-awareness + post-process-enrich
├── reporting/         4 deliverable renderers — report-writer-pdf, report-writer-excel, narrative-framer, brand-config
├── personas/          5 sub-agent role briefs — citation-verifier, cto-reviewer, ciso-reviewer, qa-reviewer, senior-pentester
├── core/              Canonical NormalizedRule + Finding + ChainOfCustody data contracts (schema.md)
├── commands/          5 slash-command specifications — start, launch, review, report, pending
├── agents/            5 sub-agent dispatch briefs (mirror personas, with Task-tool wiring)
├── learning/          Feedback-capture, skill-proposer, pending-curator + the canonical audit-report-patterns reference
└── VERSIONS.md        Single source of truth for every detector / parser / compliance pin
```

## Reference implementation

These skills are abstracted from the [firewall-review](https://github.com/ipunithgowda/firewall-review) tool's runtime catalogue. The Python implementation (parsers, detectors, validation, reporting code) lives there; this skill collection is the transferable knowledge layer.

## License note

Skills MIT (matching this repo). Reference implementation Apache-2.0.

From LeoWSY-hashblue/-communitytools-custom, MIT.

More from LeoWSY-hashblue/-communitytools-custom