Repository navigation
Say what the coverage matrix actually means, in ticks and dashes - #269
Merged
Merged
Conversation
Yes/No became ✓/– with a legend, and several cells that said No were wrong about why. The table presented four programs. Three of them are one binary: `netscli` with a command is the CLI, with no command it opens the terminal UI, and `netscli serve` starts the MCP server. Installing one installs all three. The page now says so before the table, because otherwise a dash in the MCP column reads as "you need another install for that", which is never true. "Setup and doctor" was one row for two different commands -- an interactive first-run wizard and a headless capability report that works on every build. Split. "ARP neighbor cache" was one row for reading the table and changing it. The MCP server can read it and deliberately cannot change it: that needs administrator rights and edits machine state. Split, with the reason. Two dashes that were really "not applicable" now say which: - Result bundles are the desktop app's save format. The CLI's `--json`, the TUI's `/export` and MCP's JSON-RPC results are the equivalent, not an absence -- so "Structured output" is a tick on all four and the row says what it means on each. - Reverse DNS over MCP works through `dns_lookup` with a PTR record; what is missing is the convenience of passing an address. Two are genuine gaps and are labelled as such: trace route has no MCP tool, and everything it needs is already in the core. The build-availability section said the desktop app hides tools whose feature is absent. It greys Packet Capture with an explanation instead, since #259 -- a tool that vanishes leaves nowhere to say why. Verified from the built page: 19 rows, ticks and dashes rendering, legend present, table still inside its horizontal scroll wrapper with no page overflow at 500px or 1440px. astro check 0 errors, a11y 0 violations.
Contributor
|
Site preview: https://pr-269.netscli-site-preview.pages.dev Built from 1af1566 with Production is unaffected: netscli.com is served from GitHub Pages via |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two of the open items: the matrix should use ticks and dashes, and several of its cells were wrong about why something was absent.
It presented four programs. Three are one binary.
netscliwith a command is the CLI, with no command it opens the terminal UI, andnetscli servestarts the MCP server. Installing one installs all three; the desktop app is the only separate download.That is now stated above the table, because without it a dash in the MCP column reads as "you need another install for that" — which is never true.
Rows that conflated two things
netscli setupis an interactive wizard,netscli doctoris a headless capability report that works on every build. Split.Dashes that meant "not applicable", not "missing"
--jsonon the CLI,/exportin the TUI and JSON-RPC over MCP are the equivalent, so Structured output is a tick on all four with a note on what it means on each.dns_lookuptakes aPTRrecord. What is missing is passing an address directly rather than anin-addr.arpaname.Dashes that are real gaps, now labelled as such
Also corrected
The build-availability section claimed the desktop app hides tools whose feature is absent from the build. Since #259 it greys Packet Capture with an explanation instead — a tool that vanishes leaves nowhere to say why.
The per-surface prose below the table was rewritten to name the actual commands (
netscli setup,netscli doctor,netscli serve,netscli mcp-service, the TUI's slash commands) rather than describe them abstractly, and the MCP section now mentions that results are bounded before a model sees them.Verified from the built page
19 rows · ticks and dashes rendering · legend present · table still inside its horizontal scroll wrapper with no page overflow at 500px or 1440px ·
astro check0 errors · a11y 0 violations in both themes.Every cell was checked against the real surfaces rather than the old table:
netscli --helpfor the CLI, the TUI's slash-command list, the GUI tool registry, and the MCPtools/listnames.