refactor(skill): streamline Xget skill documentation and remove deprecated test cases

Author: xixu-me

skills/xget/SKILL.md

@@ -6,8 +6,6 @@ description:
   self-hosted Xget instance. Use this skill when a task involves Xget URL
   rewriting, registry acceleration, proxy base URLs, self-hosting, or choosing
   the correct Xget prefix for Git, packages, OCI images, or inference APIs.
-  Prefer the user's own Xget domain; treat the public demo as a last-resort
-  fallback.
 license: GPL-3.0-or-later
 compatibility:
   Requires network access to refresh the live platform map. Optional Node.js 18+
@@ -17,10 +15,7 @@ allowed-tools: Bash(node:*) Bash(curl:*) Read
 
 # Xget
 
-Use this skill for Xget-specific tasks only. Default to the user's self-hosted
-Xget domain or an explicit internal instance. Do not default to
-`https://xget.xi-xu.me` unless the user explicitly wants the public demo or no
-self-hosted option exists.
+Use this skill for Xget-specific tasks only.
 
 ## Defaults
 
@@ -29,10 +24,8 @@ self-hosted option exists.
    - `XGET_BASE_URL` from the environment
    - if neither exists and the task is not just writing docs, ask the user for
      their self-hosted Xget domain and configure `XGET_BASE_URL`
-   - `https://xget.example.com` only for docs or templates when a real domain is
-     not available yet
-   - `https://xget.xi-xu.me` only as an explicitly labeled fallback after the
-     user declines or cannot provide a self-hosted domain
+   - `https://xget.example.com` only as a placeholder in docs or templates when
+     a real domain is not available yet
 2. Keep platform data fresh. Do not hardcode the full prefix list from memory.
    Run:
 
@@ -97,5 +90,6 @@ node scripts/xget.mjs platforms --format table
   Report that no current Xget mapping was found.
 - The default pip snippet should omit `trusted-host`; add it only when the
   deployment really needs it and keep it aligned with the actual host.
-- When generating docs or templates without a real domain, prefer
-  `https://xget.example.com` over the public demo.
+- When generating docs or templates without a real domain, use
+  `https://xget.example.com` as a clearly labeled placeholder rather than a
+  default instance.

skills/xget/references/REFERENCE.md

@@ -6,12 +6,10 @@ Use these defaults in order:
 
 1. User-provided Xget base URL
 2. `XGET_BASE_URL` from the environment
-3. `https://xget.example.com` only for templates and docs
-4. `https://xget.xi-xu.me` only as a clearly labeled public-demo fallback
+3. `https://xget.example.com` only as a placeholder in templates and docs
 
-The Xget README explicitly labels `xget.xi-xu.me` as a pre-deployed instance
-with no reliability guarantee, while the self-hosting docs and DigitalOcean
-guide show recommended self-hosted domains such as `xget.example.com`.
+This skill should not surface or assume a public demo domain. Prefer
+self-hosted guidance and custom-domain examples such as `xget.example.com`.
 
 ## Live platform source
 

skills/xget/scripts/xget.mjs

@@ -8,6 +8,7 @@ import vm from 'node:vm';
 const DEFAULT_SOURCE_URL = 'https://raw.gitcode.com/xixu-me/xget/raw/main/src/config/platforms.js';
 
 const DEFAULT_BASE_PLACEHOLDER = 'https://xget.example.com';
+const MISSING_BASE_URL_HINT = `Missing --base-url and XGET_BASE_URL. For docs, use ${DEFAULT_BASE_PLACEHOLDER}.`;
 
 const CRATES_API_PREFIX = '/api/v1/crates';
 
@@ -245,6 +246,16 @@ function normalizeBaseUrl(value) {
   }
 }
 
+/**
+ * Resolve an explicit or environment-provided base URL without inventing a fallback instance.
+ * @param {string | undefined} optionValue
+ * @param {string | undefined} envValue
+ * @returns {string | null}
+ */
+export function resolveBaseUrl(optionValue, envValue) {
+  return normalizeBaseUrl(optionValue ?? envValue);
+}
+
 /**
  * @param {string} value
  * @param {string} flagName
@@ -590,8 +601,8 @@ async function main() {
 
   if (command === 'convert') {
     const baseUrl =
-      normalizeBaseUrl(getStringOption(options, 'base-url') ?? process.env.XGET_BASE_URL) ??
-      fail(`Missing --base-url and XGET_BASE_URL. For docs, use ${DEFAULT_BASE_PLACEHOLDER}.`, 2);
+      resolveBaseUrl(getStringOption(options, 'base-url'), process.env.XGET_BASE_URL) ??
+      fail(MISSING_BASE_URL_HINT, 2);
 
     const rawUrl = getStringOption(options, 'url');
     if (!rawUrl) {
@@ -630,8 +641,8 @@ async function main() {
 
   if (command === 'snippet') {
     const baseUrl =
-      normalizeBaseUrl(getStringOption(options, 'base-url') ?? process.env.XGET_BASE_URL) ??
-      fail(`Missing --base-url and XGET_BASE_URL. For docs, use ${DEFAULT_BASE_PLACEHOLDER}.`, 2);
+      resolveBaseUrl(getStringOption(options, 'base-url'), process.env.XGET_BASE_URL) ??
+      fail(MISSING_BASE_URL_HINT, 2);
 
     const preset = getStringOption(options, 'preset');
     if (!preset) {