fix(e2e): broaden xiaoyuzhou skip logic for overseas CI runners (#316)

* fix: remove duplicate getErrorMessage import in discovery.ts

Squash merge left a duplicate import line causing TS2300 and oxc parse
errors in CI. Also clean up stale blank lines in discovery.ts and
execution.ts.

* fix(e2e): broaden xiaoyuzhou skip logic for overseas CI runners

The isExpectedChineseSiteRestriction function only matched FETCH_ERROR
with specific HTTP status codes. On overseas CI runners, xiaoyuzhou may
also return PARSE_ERROR (mangled HTML) or NOT_FOUND (geo-redirected
pages), causing false test failures. Now matches all CliError codes
from the adapter.

Author: jackwener

tests/e2e/public-commands.test.ts

@@ -8,7 +8,10 @@ import { parseJsonOutput, runCli } from './helpers.js';
 
 function isExpectedChineseSiteRestriction(code: number, stderr: string): boolean {
   if (code === 0) return false;
-  return /Error \[FETCH_ERROR\]: HTTP (403|429|451|503)\b/.test(stderr);
+  // Overseas CI runners may get HTTP errors, geo-blocks, DNS failures,
+  // or receive mangled HTML that fails parsing.
+  return /Error \[(FETCH_ERROR|PARSE_ERROR|NOT_FOUND)\]/.test(stderr)
+    || /fetch failed/.test(stderr);
 }
 
 function isExpectedApplePodcastsRestriction(code: number, stderr: string): boolean {