@fprad0/skill-master-mcp 0.0.11 → 1.0.0

package/scripts/configure-private-registry.mjs

@@ -1,208 +1,208 @@
-import { spawnSync } from 'node:child_process';
-import { chmodSync, copyFileSync, existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
-import { dirname, resolve } from 'node:path';
-import {
-  DEFAULT_PACKAGE_NAME,
-  DEFAULT_REGISTRY_URL,
-  DEFAULT_SCOPE,
-  DEFAULT_TOKEN_ENV,
-  redactSensitiveNpmrc,
-  upsertNpmrc,
-} from './lib/private-registry.mjs';
-
-function printHelp() {
-  process.stdout.write(`Skill Master private registry configurator
-
-Usage:
-  skill-master-configure-private-registry [options]
-
-Options:
-  --npmrc <path>              Target .npmrc path. Defaults to the user .npmrc.
-  --scope <scope>             npm scope. Defaults to @fprad0.
-  --registry <url>            Registry URL. Defaults to https://npm.pkg.github.com.
-  --token <token>             Token for validation. Avoid in shared shell history.
-  --token-file <path>         Read token from a local file.
-  --token-stdin               Read token from stdin.
-  --token-env <name>          Environment variable name. Defaults to GITHUB_PACKAGES_TOKEN.
-  --store-token               Persist the token literal in .npmrc. Default stores \${TOKEN_ENV}.
-  --allow-insecure-registry   Allow http only for localhost/private test registries.
-  --no-backup                 Do not create a timestamped backup.
-  --dry-run                   Print a redacted preview without writing.
-  --validate                  Run npm view after writing.
-  --package <name>            Package used for validation.
-  --help                      Show this help.
-`);
-}
-
-function parseArgs(argv) {
-  const parsed = {
-    npmrc: resolve(process.env.USERPROFILE ?? process.env.HOME ?? '.', '.npmrc'),
-    scope: DEFAULT_SCOPE,
-    registry: DEFAULT_REGISTRY_URL,
-    tokenEnv: DEFAULT_TOKEN_ENV,
-    backup: true,
-    dryRun: false,
-    validate: false,
-    packageName: DEFAULT_PACKAGE_NAME,
-    storeToken: false,
-    allowInsecureRegistry: false,
-    help: false,
-  };
-
-  for (let i = 0; i < argv.length; i += 1) {
-    const arg = argv[i];
-    switch (arg) {
-      case '--npmrc':
-        parsed.npmrc = resolve(argv[++i]);
-        break;
-      case '--scope':
-        parsed.scope = argv[++i];
-        break;
-      case '--registry':
-        parsed.registry = argv[++i];
-        break;
-      case '--token':
-        parsed.token = argv[++i];
-        break;
-      case '--token-file':
-        parsed.tokenFile = resolve(argv[++i]);
-        break;
-      case '--token-stdin':
-        parsed.tokenStdin = true;
-        break;
-      case '--token-env':
-        parsed.tokenEnv = argv[++i];
-        break;
-      case '--store-token':
-        parsed.storeToken = true;
-        break;
-      case '--allow-insecure-registry':
-        parsed.allowInsecureRegistry = true;
-        break;
-      case '--no-backup':
-        parsed.backup = false;
-        break;
-      case '--dry-run':
-        parsed.dryRun = true;
-        break;
-      case '--validate':
-        parsed.validate = true;
-        break;
-      case '--package':
-        parsed.packageName = argv[++i];
-        break;
-      case '--help':
-      case '-h':
-        parsed.help = true;
-        break;
-      default:
-        throw new Error(`Unknown argument: ${arg}`);
-    }
-  }
-
-  return parsed;
-}
-
-function readToken(options) {
-  if (options.token) {
-    return options.token;
-  }
-
-  if (options.tokenFile) {
-    return readFileSync(options.tokenFile, 'utf8').trim();
-  }
-
-  if (options.tokenStdin) {
-    return readFileSync(0, 'utf8').trim();
-  }
-
-  return process.env[options.tokenEnv];
-}
-
-function buildBackupPath(npmrcPath) {
-  const timestamp = new Date().toISOString().replace(/[:.]/g, '-');
-  return `${npmrcPath}.bak.${timestamp}`;
-}
-
-function main() {
-  const options = parseArgs(process.argv.slice(2));
-
-  if (options.help) {
-    printHelp();
-    return;
-  }
-
-  const token = readToken(options);
-  const needsToken = options.storeToken || options.validate;
-  if (needsToken && !token) {
-    throw new Error(`Token not provided. Use --token-file, --token-stdin, --token, or define ${options.tokenEnv}.`);
-  }
-
-  const currentContent = existsSync(options.npmrc)
-    ? readFileSync(options.npmrc, 'utf8')
-    : '';
-
-  const nextContent = upsertNpmrc(currentContent, {
-    scope: options.scope,
-    registryUrl: options.registry,
-    tokenValue: token,
-    tokenEnv: options.tokenEnv,
-    storeToken: options.storeToken,
-    allowInsecureRegistry: options.allowInsecureRegistry,
-  });
-
-  const changed = nextContent !== currentContent.replace(/\r\n/g, '\n');
-
-  if (options.dryRun) {
-    process.stdout.write(redactSensitiveNpmrc(nextContent));
-  } else {
-    mkdirSync(dirname(options.npmrc), { recursive: true });
-    if (changed && options.backup && existsSync(options.npmrc)) {
-      copyFileSync(options.npmrc, buildBackupPath(options.npmrc));
-    }
-    writeFileSync(options.npmrc, nextContent, 'utf8');
-    if (process.platform !== 'win32') {
-      chmodSync(options.npmrc, 0o600);
-    }
-  }
-
-  if (options.validate) {
-    if (options.dryRun) {
-      throw new Error('Cannot use --validate together with --dry-run.');
-    }
-
-    const result = spawnSync(
-      'npm',
-      ['view', options.packageName, `--registry=${options.registry}`],
-      {
-        env: { ...process.env, [options.tokenEnv]: token },
-        stdio: 'inherit',
-        shell: process.platform === 'win32',
-      },
-    );
-
-    if (result.status !== 0) {
-      throw new Error(`Validation failed for ${options.packageName}.`);
-    }
-  }
-
-  process.stdout.write(
-    [
-      `[skill_master] ${options.dryRun ? 'Preview ready for' : 'Configured'} ${options.npmrc}`,
-      `[skill_master] Scope: ${options.scope}`,
-      `[skill_master] Registry: ${options.registry}`,
-      `[skill_master] Token storage: ${options.storeToken ? 'literal token in .npmrc' : `environment reference \${${options.tokenEnv}}`}`,
-      `[skill_master] Changed: ${changed ? 'yes' : 'no'}`,
-      `[skill_master] Validate with: npm view ${options.packageName} --registry=${options.registry}`,
-      `[skill_master] Run with: npx -y ${options.packageName}@latest`,
-    ].join('\n') + '\n',
-  );
-}
-
-try {
-  main();
-} catch (error) {
-  const message = error instanceof Error ? error.message : String(error);
-  process.stderr.write(`[skill_master] ${message}\n`);
-  process.exitCode = 1;
-}
+import { spawnSync } from 'node:child_process';
+import { chmodSync, copyFileSync, existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
+import { dirname, resolve } from 'node:path';
+import {
+  DEFAULT_PACKAGE_NAME,
+  DEFAULT_REGISTRY_URL,
+  DEFAULT_SCOPE,
+  DEFAULT_TOKEN_ENV,
+  redactSensitiveNpmrc,
+  upsertNpmrc,
+} from './lib/private-registry.mjs';
+
+function printHelp() {
+  process.stdout.write(`Skill Master private registry configurator
+
+Usage:
+  skill-master-configure-private-registry [options]
+
+Options:
+  --npmrc <path>              Target .npmrc path. Defaults to the user .npmrc.
+  --scope <scope>             npm scope. Defaults to @fprad0.
+  --registry <url>            Registry URL. Defaults to https://npm.pkg.github.com.
+  --token <token>             Token for validation. Avoid in shared shell history.
+  --token-file <path>         Read token from a local file.
+  --token-stdin               Read token from stdin.
+  --token-env <name>          Environment variable name. Defaults to GITHUB_PACKAGES_TOKEN.
+  --store-token               Persist the token literal in .npmrc. Default stores \${TOKEN_ENV}.
+  --allow-insecure-registry   Allow http only for localhost/private test registries.
+  --no-backup                 Do not create a timestamped backup.
+  --dry-run                   Print a redacted preview without writing.
+  --validate                  Run npm view after writing.
+  --package <name>            Package used for validation.
+  --help                      Show this help.
+`);
+}
+
+function parseArgs(argv) {
+  const parsed = {
+    npmrc: resolve(process.env.USERPROFILE ?? process.env.HOME ?? '.', '.npmrc'),
+    scope: DEFAULT_SCOPE,
+    registry: DEFAULT_REGISTRY_URL,
+    tokenEnv: DEFAULT_TOKEN_ENV,
+    backup: true,
+    dryRun: false,
+    validate: false,
+    packageName: DEFAULT_PACKAGE_NAME,
+    storeToken: false,
+    allowInsecureRegistry: false,
+    help: false,
+  };
+
+  for (let i = 0; i < argv.length; i += 1) {
+    const arg = argv[i];
+    switch (arg) {
+      case '--npmrc':
+        parsed.npmrc = resolve(argv[++i]);
+        break;
+      case '--scope':
+        parsed.scope = argv[++i];
+        break;
+      case '--registry':
+        parsed.registry = argv[++i];
+        break;
+      case '--token':
+        parsed.token = argv[++i];
+        break;
+      case '--token-file':
+        parsed.tokenFile = resolve(argv[++i]);
+        break;
+      case '--token-stdin':
+        parsed.tokenStdin = true;
+        break;
+      case '--token-env':
+        parsed.tokenEnv = argv[++i];
+        break;
+      case '--store-token':
+        parsed.storeToken = true;
+        break;
+      case '--allow-insecure-registry':
+        parsed.allowInsecureRegistry = true;
+        break;
+      case '--no-backup':
+        parsed.backup = false;
+        break;
+      case '--dry-run':
+        parsed.dryRun = true;
+        break;
+      case '--validate':
+        parsed.validate = true;
+        break;
+      case '--package':
+        parsed.packageName = argv[++i];
+        break;
+      case '--help':
+      case '-h':
+        parsed.help = true;
+        break;
+      default:
+        throw new Error(`Unknown argument: ${arg}`);
+    }
+  }
+
+  return parsed;
+}
+
+function readToken(options) {
+  if (options.token) {
+    return options.token;
+  }
+
+  if (options.tokenFile) {
+    return readFileSync(options.tokenFile, 'utf8').trim();
+  }
+
+  if (options.tokenStdin) {
+    return readFileSync(0, 'utf8').trim();
+  }
+
+  return process.env[options.tokenEnv];
+}
+
+function buildBackupPath(npmrcPath) {
+  const timestamp = new Date().toISOString().replace(/[:.]/g, '-');
+  return `${npmrcPath}.bak.${timestamp}`;
+}
+
+function main() {
+  const options = parseArgs(process.argv.slice(2));
+
+  if (options.help) {
+    printHelp();
+    return;
+  }
+
+  const token = readToken(options);
+  const needsToken = options.storeToken || options.validate;
+  if (needsToken && !token) {
+    throw new Error(`Token not provided. Use --token-file, --token-stdin, --token, or define ${options.tokenEnv}.`);
+  }
+
+  const currentContent = existsSync(options.npmrc)
+    ? readFileSync(options.npmrc, 'utf8')
+    : '';
+
+  const nextContent = upsertNpmrc(currentContent, {
+    scope: options.scope,
+    registryUrl: options.registry,
+    tokenValue: token,
+    tokenEnv: options.tokenEnv,
+    storeToken: options.storeToken,
+    allowInsecureRegistry: options.allowInsecureRegistry,
+  });
+
+  const changed = nextContent !== currentContent.replace(/\r\n/g, '\n');
+
+  if (options.dryRun) {
+    process.stdout.write(redactSensitiveNpmrc(nextContent));
+  } else {
+    mkdirSync(dirname(options.npmrc), { recursive: true });
+    if (changed && options.backup && existsSync(options.npmrc)) {
+      copyFileSync(options.npmrc, buildBackupPath(options.npmrc));
+    }
+    writeFileSync(options.npmrc, nextContent, 'utf8');
+    if (process.platform !== 'win32') {
+      chmodSync(options.npmrc, 0o600);
+    }
+  }
+
+  if (options.validate) {
+    if (options.dryRun) {
+      throw new Error('Cannot use --validate together with --dry-run.');
+    }
+
+    const result = spawnSync(
+      'npm',
+      ['view', options.packageName, `--registry=${options.registry}`],
+      {
+        env: { ...process.env, [options.tokenEnv]: token },
+        stdio: 'inherit',
+        shell: process.platform === 'win32',
+      },
+    );
+
+    if (result.status !== 0) {
+      throw new Error(`Validation failed for ${options.packageName}.`);
+    }
+  }
+
+  process.stdout.write(
+    [
+      `[skill_master] ${options.dryRun ? 'Preview ready for' : 'Configured'} ${options.npmrc}`,
+      `[skill_master] Scope: ${options.scope}`,
+      `[skill_master] Registry: ${options.registry}`,
+      `[skill_master] Token storage: ${options.storeToken ? 'literal token in .npmrc' : `environment reference \${${options.tokenEnv}}`}`,
+      `[skill_master] Changed: ${changed ? 'yes' : 'no'}`,
+      `[skill_master] Validate with: npm view ${options.packageName} --registry=${options.registry}`,
+      `[skill_master] Run with: npx -y ${options.packageName}@latest`,
+    ].join('\n') + '\n',
+  );
+}
+
+try {
+  main();
+} catch (error) {
+  const message = error instanceof Error ? error.message : String(error);
+  process.stderr.write(`[skill_master] ${message}\n`);
+  process.exitCode = 1;
+}

package/scripts/lib/private-registry.mjs

@@ -1,97 +1,97 @@
-export const DEFAULT_SCOPE = '@fprad0';
-export const DEFAULT_PACKAGE_NAME = '@fprad0/skill-master-mcp';
-export const DEFAULT_REGISTRY_URL = 'https://npm.pkg.github.com';
-export const DEFAULT_TOKEN_ENV = 'GITHUB_PACKAGES_TOKEN';
-
-export function normalizeRegistryUrl(registryUrl = DEFAULT_REGISTRY_URL, { allowInsecure = false } = {}) {
-  const parsed = new URL(registryUrl);
-  const isLocalhost = ['localhost', '127.0.0.1', '::1'].includes(parsed.hostname);
-
-  if (parsed.protocol !== 'https:' && !(allowInsecure && isLocalhost)) {
-    throw new Error(`Registry must use HTTPS: ${registryUrl}`);
-  }
-
-  parsed.hash = '';
-  parsed.search = '';
-  return parsed.toString().replace(/\/$/, '');
-}
-
-export function registryHostFromUrl(registryUrl = DEFAULT_REGISTRY_URL, { allowInsecure = false } = {}) {
-  return new URL(normalizeRegistryUrl(registryUrl, { allowInsecure })).host;
-}
-
-export function buildRegistryLine(scope = DEFAULT_SCOPE, registryUrl = DEFAULT_REGISTRY_URL) {
-  return `${scope}:registry=${normalizeRegistryUrl(registryUrl)}`;
-}
-
-export function buildTokenLine(registryUrl = DEFAULT_REGISTRY_URL, tokenValue) {
-  return `//${registryHostFromUrl(registryUrl)}/:_authToken=${tokenValue}`;
-}
-
-export function buildTokenEnvReference(tokenEnv = DEFAULT_TOKEN_ENV) {
-  if (!/^[A-Za-z_][A-Za-z0-9_]*$/.test(tokenEnv)) {
-    throw new Error(`Invalid token environment variable name: ${tokenEnv}`);
-  }
-
-  return '${' + tokenEnv + '}';
-}
-
-export function validateScope(scope = DEFAULT_SCOPE) {
-  if (!/^@[a-z0-9][a-z0-9._-]*$/i.test(scope)) {
-    throw new Error(`Invalid npm scope: ${scope}`);
-  }
-
-  return scope;
-}
-
-export function upsertNpmrc(content, {
-  scope = DEFAULT_SCOPE,
-  registryUrl = DEFAULT_REGISTRY_URL,
-  tokenValue,
-  tokenEnv = DEFAULT_TOKEN_ENV,
-  storeToken = false,
-  allowInsecureRegistry = false,
-} = {}) {
-  const normalizedScope = validateScope(scope);
-  const normalizedRegistryUrl = normalizeRegistryUrl(registryUrl, { allowInsecure: allowInsecureRegistry });
-  const tokenValueForFile = storeToken
-    ? tokenValue
-    : buildTokenEnvReference(tokenEnv);
-
-  if (!tokenValueForFile) {
-    throw new Error(storeToken ? 'tokenValue is required when storeToken is enabled' : 'tokenEnv is required');
-  }
-
-  const existing = (content ?? '').replace(/\r\n/g, '\n');
-  const registryLinePrefix = `${normalizedScope}:registry=`;
-  const tokenLinePrefix = `//${registryHostFromUrl(normalizedRegistryUrl, { allowInsecure: allowInsecureRegistry })}/:_authToken=`;
-
-  const filteredLines = existing
-    .split('\n')
-    .filter((line) => !line.startsWith(registryLinePrefix) && !line.startsWith(tokenLinePrefix));
-
-  while (filteredLines.length > 0 && filteredLines[filteredLines.length - 1] === '') {
-    filteredLines.pop();
-  }
-
-  if (filteredLines.length > 0) {
-    filteredLines.push('');
-  }
-
-  filteredLines.push(buildRegistryLine(normalizedScope, normalizedRegistryUrl));
-  filteredLines.push(buildTokenLine(normalizedRegistryUrl, tokenValueForFile));
-
-  return `${filteredLines.join('\n')}\n`;
-}
-
-export function hasRegistryConfig(content, {
-  scope = DEFAULT_SCOPE,
-  registryUrl = DEFAULT_REGISTRY_URL,
-} = {}) {
-  const normalized = (content ?? '').replace(/\r\n/g, '\n');
-  return normalized.includes(buildRegistryLine(scope, registryUrl));
-}
-
-export function redactSensitiveNpmrc(content) {
-  return (content ?? '').replace(/(:_authToken=)(.+)$/gm, '$1<redacted>');
-}
+export const DEFAULT_SCOPE = '@fprad0';
+export const DEFAULT_PACKAGE_NAME = '@fprad0/skill-master-mcp';
+export const DEFAULT_REGISTRY_URL = 'https://npm.pkg.github.com';
+export const DEFAULT_TOKEN_ENV = 'GITHUB_PACKAGES_TOKEN';
+
+export function normalizeRegistryUrl(registryUrl = DEFAULT_REGISTRY_URL, { allowInsecure = false } = {}) {
+  const parsed = new URL(registryUrl);
+  const isLocalhost = ['localhost', '127.0.0.1', '::1'].includes(parsed.hostname);
+
+  if (parsed.protocol !== 'https:' && !(allowInsecure && isLocalhost)) {
+    throw new Error(`Registry must use HTTPS: ${registryUrl}`);
+  }
+
+  parsed.hash = '';
+  parsed.search = '';
+  return parsed.toString().replace(/\/$/, '');
+}
+
+export function registryHostFromUrl(registryUrl = DEFAULT_REGISTRY_URL, { allowInsecure = false } = {}) {
+  return new URL(normalizeRegistryUrl(registryUrl, { allowInsecure })).host;
+}
+
+export function buildRegistryLine(scope = DEFAULT_SCOPE, registryUrl = DEFAULT_REGISTRY_URL) {
+  return `${scope}:registry=${normalizeRegistryUrl(registryUrl)}`;
+}
+
+export function buildTokenLine(registryUrl = DEFAULT_REGISTRY_URL, tokenValue) {
+  return `//${registryHostFromUrl(registryUrl)}/:_authToken=${tokenValue}`;
+}
+
+export function buildTokenEnvReference(tokenEnv = DEFAULT_TOKEN_ENV) {
+  if (!/^[A-Za-z_][A-Za-z0-9_]*$/.test(tokenEnv)) {
+    throw new Error(`Invalid token environment variable name: ${tokenEnv}`);
+  }
+
+  return '${' + tokenEnv + '}';
+}
+
+export function validateScope(scope = DEFAULT_SCOPE) {
+  if (!/^@[a-z0-9][a-z0-9._-]*$/i.test(scope)) {
+    throw new Error(`Invalid npm scope: ${scope}`);
+  }
+
+  return scope;
+}
+
+export function upsertNpmrc(content, {
+  scope = DEFAULT_SCOPE,
+  registryUrl = DEFAULT_REGISTRY_URL,
+  tokenValue,
+  tokenEnv = DEFAULT_TOKEN_ENV,
+  storeToken = false,
+  allowInsecureRegistry = false,
+} = {}) {
+  const normalizedScope = validateScope(scope);
+  const normalizedRegistryUrl = normalizeRegistryUrl(registryUrl, { allowInsecure: allowInsecureRegistry });
+  const tokenValueForFile = storeToken
+    ? tokenValue
+    : buildTokenEnvReference(tokenEnv);
+
+  if (!tokenValueForFile) {
+    throw new Error(storeToken ? 'tokenValue is required when storeToken is enabled' : 'tokenEnv is required');
+  }
+
+  const existing = (content ?? '').replace(/\r\n/g, '\n');
+  const registryLinePrefix = `${normalizedScope}:registry=`;
+  const tokenLinePrefix = `//${registryHostFromUrl(normalizedRegistryUrl, { allowInsecure: allowInsecureRegistry })}/:_authToken=`;
+
+  const filteredLines = existing
+    .split('\n')
+    .filter((line) => !line.startsWith(registryLinePrefix) && !line.startsWith(tokenLinePrefix));
+
+  while (filteredLines.length > 0 && filteredLines[filteredLines.length - 1] === '') {
+    filteredLines.pop();
+  }
+
+  if (filteredLines.length > 0) {
+    filteredLines.push('');
+  }
+
+  filteredLines.push(buildRegistryLine(normalizedScope, normalizedRegistryUrl));
+  filteredLines.push(buildTokenLine(normalizedRegistryUrl, tokenValueForFile));
+
+  return `${filteredLines.join('\n')}\n`;
+}
+
+export function hasRegistryConfig(content, {
+  scope = DEFAULT_SCOPE,
+  registryUrl = DEFAULT_REGISTRY_URL,
+} = {}) {
+  const normalized = (content ?? '').replace(/\r\n/g, '\n');
+  return normalized.includes(buildRegistryLine(scope, registryUrl));
+}
+
+export function redactSensitiveNpmrc(content) {
+  return (content ?? '').replace(/(:_authToken=)(.+)$/gm, '$1<redacted>');
+}