Skip to content

Show successful TCP ports next to verify stage results#23

Merged
ehsanking merged 1 commit into
mainfrom
ehsan/indicate-successful-ports-in-cli
Apr 2, 2026
Merged

Show successful TCP ports next to verify stage results#23
ehsanking merged 1 commit into
mainfrom
ehsan/indicate-successful-ports-in-cli

Conversation

@ehsanking
Copy link
Copy Markdown
Owner

Motivation

  • The verify CLI output did not make it easy to see which tested TCP ports succeeded, forcing users to inspect the full report JSON to discover working ports.
  • Provide immediate, human-readable feedback in the CLI so operators can quickly identify reachable ports when running pipeline verification.

Description

  • Add port and protocol metadata to the L4 (tcp-443) and L5 (tcp-80) checks by updating src/checks/l4_tcp_443.js and src/checks/l5_tcp_80.js so pipeline levels include port information.
  • Render a new 🧪 STAGE RESULTS panel in the verify command inside cli.js to show each stage as PASS/FAIL.
  • Annotate successful TCP stages inline with (port <n> ✅) and include an aggregated Successful ports line in the verify summary panel.

Testing

  • Ran the test suite with npm test, all tests passed (6 tests, 0 failures).
  • Executed node cli.js verify 1.1.1.1 -t 1 to validate CLI output, which showed the new 🧪 STAGE RESULTS panel and the Successful ports summary as expected.

Codex Task

@ehsanking ehsanking merged commit 768028d into main Apr 2, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant