archetech / archon
93%
main: 94%

Repo Added 12 Jan 2026 07:43PM UTC

Token VKRdMqMwqPTOZprp4p5Cb3v3uLt6LWUtk regen

Build 1946 Last

Committed 02 Aug 2026 08:21PM UTC coverage: 92.944% (+0.1%) from 92.801%

Committed by

Commit Message

test(mcp-server): cover the tool handlers' optional and failure branches

tools.ts sat at 78.2% branches against 99.2% lines — the suite drove every
tool with fully-populated arguments, leaving the "argument omitted" and
failure sides untaken. 8 tests added (42 -> 50), taking it to 96.2%
branches.

What the new tests pin down:

- `includeIds: false` must survive rather than collapsing to undefined.
  The handler writes `includeIds === undefined ? undefined : {...}`
  precisely to preserve an explicit false, and nothing tested it.
- get_property against non-object DID data (undefined, null, a string, an
  array, a number). The guard exists so indexing a string or array cannot
  return a nonsense "property" — which an agent would then act on.
- The legacy `tool()` registration path, for an MCP server without
  `registerTool`. Previously untested, so a regression there would have
  stayed invisible until an older client connected.
- view_credential with a missing credential returns proofValid false
  without calling verifyProof at all.
- Inline data decoded as utf8 as well as base64, and the default names
  ('item', 'attachment') when none is supplied.
- resolve_id falling back to the current ID, and erroring when there is
  none.

Three of my first attempts were wrong about the codebase rather than the
code: keymaster calls live on runtime.keymaster (not runtime.node), the
image tool is archon_get_asset_image, and signing is archon_sign_file
whose schema already rejects most non-objects — so signableJson's guard
is only reachable with an array, which JsonObjectSchema admits but
signing must refuse. That array case is the branch that needed covering.

Three branches remain, and they are defensive rather than reachable. One
is worth noting: the readOnly check inside `wrapped` (tools.ts:641) is
unreachable, because the identical condition above it returns before the
tool is ever registered — a mutating tool in read-only mode never gets a
`wrapped` at all. ... (continued)

Coverage Stats

3521 of 4060 branches covered (86.72%)

Branch coverage included in aggregate %.

8163 of 8511 relevant lines covered (95.91%)

673.79 hits per line

8511 RELEVANT LINES 8163 COVERED LINES

673.79 HITS PER LINE

Builds	Branch	Commit	Type	Ran	Committer	Via	Coverage
30765451617	test/mcp-tools-branches	test(mcp-server): cover the tool handlers' optional and failure branches tools.ts sat at 78.2% branches against 99.2% lines — the suite drove every tool with fully-populated arguments, leaving the "argument omitted" and failure sides untaken. 8 t...	Pull #827	02 Aug 2026 08:25PM UTC	macterra	github	92.94

archetech / archon
93%
main: 94%

archetech / archon 93% main: 94%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

CHANGE BRANCH x

Relevant lines Covered

Source Files on test/mcp-tools-branches

Recent builds

archetech / archon
93%
main: 94%

README BADGES
x

CHANGE BRANCH
x