You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Overall coverage is healthy across all metrics. Branch coverage (86.61%) is the lowest area and represents the primary improvement opportunity.
π‘οΈ Security-Critical Path Status
The security-critical firewall paths have excellent coverage:
File
Purpose
Statements
Branches
src/host-iptables.ts
Network isolation rules (L3/L4)
β High
β High
src/squid-config.ts
Domain ACL filtering (L7)
β High
β High
src/docker-manager.ts
Container lifecycle management
β High
β High
src/domain-patterns.ts
Domain pattern validation
β High
β High
src/cli.ts
CLI orchestration entry point
β οΈ 85.71%
β οΈ 50%
β οΈ Alert:src/cli.ts has 50% branch coverage β a split code path in the CLI wrapper is not exercised by the test suite. This should be addressed.
π Coverage Table β Critical Gaps
Severity
File
Statements
Branches
Issue
π΄ CRITICAL
src/bounded-execution/finite-disclosure.ts
38.12%
25.89%
Disclosure bounds logic untested; security boundary at risk
π HIGH
src/microvm/network-reservation.ts
50.22%
54.26%
microVM network isolation only half-tested
π‘ MEDIUM
src/enclave/runtime-preflight.ts
66.66%
41.02%
Enclave startup validation lacks branch coverage
π‘ MEDIUM
src/microvm/rootfs.ts
71.69%
54.54%
Filesystem setup logic branch coverage below 60%
π‘ MEDIUM
src/commands/validators/config-assembly.ts
73.68%
25%
Config validation branching severely under-tested
Recommendation: Focus first on finite-disclosure.ts (CRITICAL) as it governs output bounds. microVM network reservation is also high-priority for sandbox isolation.
Key observation: Recent changes are mostly refactoring and configuration. The --network-subnet change should be covered by existing network tests.
π Notable Findings
Security boundary exposure:finite-disclosure.ts (38.12%) is a CRITICAL gap β output bounds enforcement is the last line of defense against data exfiltration. This must be addressed.
microVM isolation risk:network-reservation.ts at 50.22% means half of the network isolation code is untested. Given microVM is a preview feature, this should be remediated before GA.
CLI edge cases: The 50% branch coverage in src/cli.ts suggests signal handling or error conditions are missing test cases β these are often security-relevant paths.
Overall health: Project-wide coverage is strong (92.76% statements, 94.09% lines), but branch coverage (86.61%) lags β this indicates missing conditional paths in happy-path tests.
π― Recommendations
Priority 1 (CRITICAL):
File:src/bounded-execution/finite-disclosure.ts
Action: Increase statement coverage from 38% to β₯80% by writing tests for all disclosure limit boundaries and truncation paths
Rationale: This controls output sanitization; security boundary violation if untested
Priority 2 (HIGH):
File:src/microvm/network-reservation.ts
Action: Test network pool exhaustion, IP reuse, and edge cases; aim for β₯75% statement coverage
Rationale: Network isolation is core to sandbox security
Priority 3 (MEDIUM):
File:src/cli.ts
Action: Identify the uncovered branch (likely signal handling) and add integration test for that code path
Rationale: Error handling paths in the CLI orchestrator are security-relevant
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
Overall Coverage
Current Project-Wide Coverage Metrics:
Overall coverage is healthy across all metrics. Branch coverage (86.61%) is the lowest area and represents the primary improvement opportunity.
π‘οΈ Security-Critical Path Status
The security-critical firewall paths have excellent coverage:
src/host-iptables.tssrc/squid-config.tssrc/docker-manager.tssrc/domain-patterns.tssrc/cli.tssrc/cli.tshas 50% branch coverage β a split code path in the CLI wrapper is not exercised by the test suite. This should be addressed.π Coverage Table β Critical Gaps
src/bounded-execution/finite-disclosure.tssrc/microvm/network-reservation.tssrc/enclave/runtime-preflight.tssrc/microvm/rootfs.tssrc/commands/validators/config-assembly.tsRecommendation: Focus first on
finite-disclosure.ts(CRITICAL) as it governs output bounds. microVM network reservation is also high-priority for sandbox isolation.π§ Function Audit
Key low-coverage functions require investigation:
src/bounded-execution/finite-disclosure.ts(38.12% statements)src/microvm/network-reservation.ts(50.22% statements)src/cli.ts(50% branch coverage)π Recent Source Changes (last 7 days)
Recent commits touching the firewall codebase:
407226fβ Add--network-subnetto relocate awf-net off colliding host/cluster subnets (Add --network-subnet to relocate awf-net off colliding host/cluster subnetsΒ #8398)a8ce656β Support Codex apply_patch through Copilot Responses API (Support Codex apply_patch through Copilot Responses APIΒ #8359)9e89ccaβ Refactor provider adapter registration (Refactor provider adapter registrationΒ #8335)f895fcaβ Refactor: centralize agent sandbox path policy (refactor: centralize agent sandbox path policyΒ #8334)e015efdβ Test: add end-to-end security coverage for dynamic repository enclaves (test: add end-to-end security coverage for dynamic repository enclavesΒ #8307)Key observation: Recent changes are mostly refactoring and configuration. The
--network-subnetchange should be covered by existing network tests.π Notable Findings
Security boundary exposure:
finite-disclosure.ts(38.12%) is a CRITICAL gap β output bounds enforcement is the last line of defense against data exfiltration. This must be addressed.microVM isolation risk:
network-reservation.tsat 50.22% means half of the network isolation code is untested. Given microVM is a preview feature, this should be remediated before GA.CLI edge cases: The 50% branch coverage in
src/cli.tssuggests signal handling or error conditions are missing test cases β these are often security-relevant paths.Overall health: Project-wide coverage is strong (92.76% statements, 94.09% lines), but branch coverage (86.61%) lags β this indicates missing conditional paths in happy-path tests.
π― Recommendations
Priority 1 (CRITICAL):
src/bounded-execution/finite-disclosure.tsPriority 2 (HIGH):
src/microvm/network-reservation.tsPriority 3 (MEDIUM):
src/cli.tsAll reactions