test(unit): address Copilot review — drop unused infoed, clarify coverage docs

- test/unit/config-parse.test.ts: the `infoed` capture buffer was never
  asserted. Dropped it and silenced `log.info` directly so
  resolveTargetList/parseTargets fallback paths don't print during tests.
- docs-site/development.md: `yarn coverage:e2e` uses `--clean=false` and
  therefore accumulates on top of whatever is in `coverage/tmp` — it is
  not "e2e-only" in general. Docs now state the actual behavior: fresh →
  e2e-only, after `yarn coverage:unit` → merged.

No script change — the `--clean=false` design is intentional per #267
(unit + e2e accumulate via a single temp dir).

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>

Author: robertsLando

docs-site/development.md

@@ -65,9 +65,17 @@ When running `node test/test.js all`, based on the options, each test will be ru
 `pkg` uses [c8](https://github.com/bcoe/c8) as a thin reporter over V8's built-in coverage. `NODE_V8_COVERAGE` propagates through child processes, so e2e coverage captures the `pkg` CLI executions spawned by the harness.
 
 ```bash
-yarn coverage:unit    # unit-only coverage → coverage/lcov.info
-yarn coverage:e2e     # e2e-only coverage (slow — full build matrix)
-yarn coverage         # unit + e2e merged into a single report
+# Run unit coverage alone — always deterministic; clears coverage/tmp first.
+yarn coverage:unit
+
+# Append e2e coverage on top of whatever is in coverage/tmp (uses `c8 --clean=false`).
+# After a fresh `yarn coverage:unit`, this produces a merged report. Without
+# a prior run it produces an e2e-only report. For a guaranteed e2e-only view,
+# delete `coverage/` first. (Slow — runs the full e2e matrix.)
+yarn coverage:e2e
+
+# Full merged report: cleans, runs unit, then appends e2e, then emits lcov+text.
+yarn coverage
 ```
 
 ### Example e2e test

test/unit/config-parse.test.ts

@@ -27,13 +27,13 @@ type LogFn = (..._a: unknown[]) => void;
 // validatePkgConfig calls log.warn; wasReported() (used elsewhere) calls
 // log.error before returning an Error. Capture the former for assertions,
 // silence the latter so assertion-triggered throws don't pollute output.
+// log.info is silenced too — some tests fire resolveTargetList/parseTargets
+// fallback paths that would otherwise print to stderr.
 let warned: string[];
-let infoed: string[];
 let originals: { warn: LogFn; info: LogFn; error: LogFn };
 
 beforeEach(() => {
   warned = [];
-  infoed = [];
   originals = {
     warn: log.warn as LogFn,
     info: log.info as LogFn,
@@ -42,9 +42,7 @@ beforeEach(() => {
   log.warn = ((...a: unknown[]) => {
     warned.push(a.join(' '));
   }) as typeof log.warn;
-  log.info = ((...a: unknown[]) => {
-    infoed.push(a.join(' '));
-  }) as typeof log.info;
+  log.info = (() => {}) as typeof log.info;
   log.error = (() => {}) as typeof log.error;
 });