@lpm-registry/cli 0.2.6 → 0.3.0

package/CHANGELOG.md

@@ -2,6 +2,22 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.3.0] - 2026-03-22
+
+### Added
+
+- **Upstream npm proxy mode (default)** — `lpm npmrc` and `lpm setup` now configure LPM as the default registry. All packages (LPM + npm) are fetched through LPM for faster, edge-cached installs via Cloudflare R2.
+- **`--scoped` flag** — Use `lpm npmrc --scoped` or `lpm setup --scoped` to only route `@lpm.dev` packages through LPM (old behavior).
+- **Custom registry detection** — If `.npmrc` already has a custom default registry (e.g., Artifactory), the CLI warns and automatically falls back to `--scoped` mode to avoid overriding it.
+- **Pre-publish skills staleness warning** — When Agent Skills haven't changed since the previous version, the warning now appears before the publish confirmation prompt (instead of after), so you can cancel and update skills before publishing.
+
+### Changed
+
+- `lpm npmrc` default output changed from `@lpm.dev:registry=...` (scoped) to `registry=...` (proxy mode)
+- `lpm setup` and `lpm setup --oidc` default output changed to proxy mode
+- Updated CLI output messaging to indicate proxy vs scoped mode
+- Updated README with new default behavior and `--scoped` flag
+
 ## [0.2.6] - 2026-03-21
 
 ### Added

package/README.md

@@ -69,10 +69,13 @@ lpm whoami
 
 Generate a 30-day read-only token for your project. Makes `npm install` work with LPM packages.
 
+By default, LPM is configured as your default registry — all packages (LPM + npm) are fetched through LPM for faster, edge-cached installs.
+
 ```bash
-lpm npmrc                  # 30-day token (default)
+lpm npmrc                  # 30-day token, proxy mode (default)
 lpm npmrc --days 7         # 7-day token
 lpm npmrc --days 90        # 90-day token
+lpm npmrc --scoped         # Only route @lpm.dev packages through LPM
 ```
 
 Automatically adds `.npmrc` to `.gitignore` to prevent token leaks.
@@ -82,8 +85,9 @@ Automatically adds `.npmrc` to `.gitignore` to prevent token leaks.
 Configure `.npmrc` for CI/CD environments.
 
 ```bash
-lpm setup                  # Writes ${LPM_TOKEN} placeholder
+lpm setup                  # Writes ${LPM_TOKEN} placeholder (proxy mode)
 lpm setup --oidc           # OIDC — no secrets needed (GitHub Actions, GitLab CI)
+lpm setup --scoped         # Only route @lpm.dev packages through LPM
 ```
 
 For deployment platforms (Vercel, Netlify), set the `LPM_TOKEN` environment variable.

package/bin/lpm.js

@@ -210,18 +210,26 @@ program
 
 program
 	.command("setup")
-	.description("Configure .npmrc for LPM packages (@lpm.dev scope)")
+	.description("Configure .npmrc for LPM registry")
 	.option("-r, --registry <url>", "Custom registry URL")
 	.option(
 		"--oidc",
 		"Exchange CI OIDC token for install access (no secrets needed)",
 	)
+	.option(
+		"--scoped",
+		"Only route @lpm.dev packages through LPM (don't proxy npm)",
+	)
 	.action(setup)
 
 program
 	.command("npmrc")
 	.description("Generate a read-only .npmrc token for local development")
 	.option("-d, --days <number>", "Token expiry in days (default: 30)")
+	.option(
+		"--scoped",
+		"Only route @lpm.dev packages through LPM (don't proxy npm)",
+	)
 	.action(npmrc)
 
 program

package/lib/commands/npmrc.js

@@ -82,15 +82,41 @@ export async function npmrc(options) {
 		: `${registryUrl}/api/registry`
 	const registryHost = fullRegistryUrl.replace(/^https?:/, "")
 
-	// Read existing .npmrc and remove old LPM config lines
+	// Determine proxy vs scoped mode
+	let useScoped = !!options?.scoped
+
+	// Read existing .npmrc and check for custom default registry
 	let npmrcContent = ""
 	if (fs.existsSync(npmrcPath)) {
-		npmrcContent = fs
-			.readFileSync(npmrcPath, "utf8")
+		const existingContent = fs.readFileSync(npmrcPath, "utf8")
+
+		// Detect custom default registry (not npmjs.org and not lpm.dev)
+		if (!useScoped) {
+			const registryLine = existingContent
+				.split("\n")
+				.find(
+					line =>
+						line.match(/^registry=/) &&
+						!line.includes("registry.npmjs.org") &&
+						!line.includes("lpm.dev"),
+				)
+			if (registryLine) {
+				log.warn(`Found existing default registry: ${registryLine.trim()}`)
+				log.warn("Using --scoped mode to avoid overriding it.")
+				log.warn(
+					"To use LPM as your default registry, remove the existing registry= line first.",
+				)
+				useScoped = true
+			}
+		}
+
+		// Remove old LPM config lines (both scoped and unscoped formats)
+		npmrcContent = existingContent
 			.split("\n")
 			.filter(line => {
 				return (
 					!line.includes("@lpm.dev:registry") &&
+					!line.match(/^registry=.*lpm\.dev/) &&
 					!line.includes("lpm.dev/api/registry/:_authToken") &&
 					!line.includes("_authToken=lpm_") &&
 					// biome-ignore lint/suspicious/noTemplateCurlyInString: matching literal npm token placeholder
@@ -102,8 +128,12 @@ export async function npmrc(options) {
 			.trim()
 	}
 
+	const registryLine = useScoped
+		? `@lpm.dev:registry=${fullRegistryUrl}`
+		: `registry=${fullRegistryUrl}`
+
 	const lpmConfig = `# LPM Registry (generated by lpm npmrc — do not commit)
-@lpm.dev:registry=${fullRegistryUrl}
+${registryLine}
 ${registryHost}/:_authToken=${readToken}`
 
 	npmrcContent = npmrcContent ? `${npmrcContent}\n\n${lpmConfig}` : lpmConfig
@@ -129,6 +159,13 @@ ${registryHost}/:_authToken=${readToken}`
 		log.info(".npmrc added to .gitignore to prevent token leaks.")
 	}
 
+	if (useScoped) {
+		log.info("Only @lpm.dev packages will route through lpm.dev.")
+	} else {
+		log.info("All packages (LPM + npm) will route through lpm.dev.")
+		log.info("Note: First install may update resolved URLs in your lockfile.")
+	}
+
 	const expiryDate = new Date(expiresAt).toLocaleDateString("en-US", {
 		year: "numeric",
 		month: "long",

package/lib/commands/publish.js

@@ -522,8 +522,63 @@ export async function publish(options = {}) {
 			swiftManifest,
 		})
 
+		// 7b. Check for stale skills (compare local with published)
+		spinner.text = "Checking skills..."
+		const prePublishWarnings = []
+		try {
+			const localSkillsDir = path.join(process.cwd(), ".lpm", "skills")
+			if (fs.existsSync(localSkillsDir)) {
+				const skillFiles = fs
+					.readdirSync(localSkillsDir)
+					.filter(f => f.endsWith(".md"))
+					.sort()
+
+				if (skillFiles.length > 0) {
+					// Read local skills content
+					const localSkills = skillFiles.map(f => ({
+						name: f.replace(/\.md$/, ""),
+						rawContent: fs.readFileSync(path.join(localSkillsDir, f), "utf8"),
+					}))
+
+					// Fetch published skills for comparison
+					const cleanedName = name.startsWith("@lpm.dev/")
+						? name.replace("@lpm.dev/", "")
+						: `${owner}.${packageName}`
+					const skillsResponse = await request(
+						`/skills?name=${encodeURIComponent(cleanedName)}`,
+						{ method: "GET" },
+					).catch(() => null)
+
+					if (skillsResponse?.ok) {
+						const published = await skillsResponse.json()
+						if (published.available && published.skills?.length > 0) {
+							const pubSorted = [...published.skills]
+								.map(s => ({ name: s.name, rawContent: s.rawContent }))
+								.sort((a, b) => a.name.localeCompare(b.name))
+
+							const isIdentical =
+								localSkills.length === pubSorted.length &&
+								localSkills.every(
+									(local, i) =>
+										local.name === pubSorted[i].name &&
+										local.rawContent === pubSorted[i].rawContent,
+								)
+
+							if (isIdentical) {
+								prePublishWarnings.push(
+									`Agent Skills haven't changed since v${published.version} - consider reviewing them for accuracy`,
+								)
+							}
+						}
+					}
+				}
+			}
+		} catch {
+			// Non-critical — skip skills staleness check
+		}
+
 		spinner.stop()
-		displayQualityReport(qualityResult)
+		displayQualityReport({ ...qualityResult, warnings: prePublishWarnings })
 
 		// --dry-run: print summary and exit without uploading
 		if (dryRun) {
@@ -848,11 +903,16 @@ export async function publish(options = {}) {
 			)
 		}
 
-		// Show any server warnings (e.g., stale skills)
+		// Server may still return warnings for edge cases not caught client-side
 		if (publishResult.warnings?.length > 0) {
-			console.log("")
-			for (const warning of publishResult.warnings) {
-				log.warn(warning)
+			const newWarnings = publishResult.warnings.filter(
+				w => !prePublishWarnings.includes(w),
+			)
+			if (newWarnings.length > 0) {
+				console.log("")
+				for (const warning of newWarnings) {
+					log.warn(warning)
+				}
 			}
 		}
 	} catch (error) {

package/lib/commands/setup.js

@@ -8,6 +8,7 @@ import { log, printHeader } from "../ui.js"
 
 /**
  * Remove existing LPM config lines from .npmrc content.
+ * Handles both scoped (@lpm.dev:registry=) and unscoped (registry=...lpm.dev) formats.
  * @param {string} content
  * @returns {string}
  */
@@ -17,6 +18,7 @@ function removeLpmLines(content) {
 		.filter(line => {
 			return (
 				!line.includes("@lpm.dev:registry") &&
+				!line.match(/^registry=.*lpm\.dev/) &&
 				!line.includes("lpm.dev/api/registry/:_authToken") &&
 				!line.includes("_authToken=lpm_") &&
 				// biome-ignore lint/suspicious/noTemplateCurlyInString: matching literal npm token placeholder
@@ -28,6 +30,24 @@ function removeLpmLines(content) {
 		.trim()
 }
 
+/**
+ * Check if .npmrc has a custom default registry that is NOT npmjs.org or lpm.dev.
+ * @param {string} content - Existing .npmrc content
+ * @returns {string|null} The custom registry line, or null if none found
+ */
+function detectCustomRegistry(content) {
+	return (
+		content
+			.split("\n")
+			.find(
+				line =>
+					line.match(/^registry=/) &&
+					!line.includes("registry.npmjs.org") &&
+					!line.includes("lpm.dev"),
+			) || null
+	)
+}
+
 /**
  * Configure .npmrc for LPM packages.
  *
@@ -55,6 +75,9 @@ export async function setup(options) {
 		: `${registryUrl}/api/registry`
 	const registryHost = fullRegistryUrl.replace(/^https?:/, "")
 
+	// Determine proxy vs scoped mode
+	let useScoped = !!options?.scoped
+
 	// ── OIDC mode: exchange CI token for a real short-lived install token ──
 	if (options?.oidc) {
 		let oidcToken
@@ -70,11 +93,29 @@ export async function setup(options) {
 		if (oidcToken) {
 			let npmrcContent = ""
 			if (fs.existsSync(npmrcPath)) {
-				npmrcContent = removeLpmLines(fs.readFileSync(npmrcPath, "utf8"))
+				const existingContent = fs.readFileSync(npmrcPath, "utf8")
+
+				// Detect custom default registry
+				if (!useScoped) {
+					const customRegistry = detectCustomRegistry(existingContent)
+					if (customRegistry) {
+						log.warn(
+							`Found existing default registry: ${customRegistry.trim()}`,
+						)
+						log.warn("Using --scoped mode to avoid overriding it.")
+						useScoped = true
+					}
+				}
+
+				npmrcContent = removeLpmLines(existingContent)
 			}
 
+			const registryLine = useScoped
+				? `@lpm.dev:registry=${fullRegistryUrl}`
+				: `registry=${fullRegistryUrl}`
+
 			const lpmConfig = `# LPM Registry (OIDC — expires in 30 minutes)
-@lpm.dev:registry=${fullRegistryUrl}
+${registryLine}
 ${registryHost}/:_authToken=${oidcToken}`
 
 			npmrcContent = npmrcContent
@@ -83,13 +124,16 @@ ${registryHost}/:_authToken=${oidcToken}`
 			fs.writeFileSync(npmrcPath, `${npmrcContent}\n`)
 
 			p.note(
-				`@lpm.dev:registry=${fullRegistryUrl}\nToken: OIDC read-only (30 min)`,
+				`${registryLine}\nToken: OIDC read-only (30 min)`,
 				".npmrc configuration",
 			)
 
 			console.log("")
 			log.success(".npmrc configured with OIDC read-only token.")
 			log.info("Token expires in 30 minutes. No LPM_TOKEN secret needed.")
+			if (!useScoped) {
+				log.info("All packages (LPM + npm) will route through lpm.dev.")
+			}
 			console.log("")
 
 			p.outro("Setup complete!")
@@ -104,8 +148,24 @@ ${registryHost}/:_authToken=${oidcToken}`
 	if (fs.existsSync(npmrcPath)) {
 		npmrcContent = fs.readFileSync(npmrcPath, "utf8")
 
-		// Check if already configured
-		if (npmrcContent.includes("@lpm.dev:registry")) {
+		// Detect custom default registry
+		if (!useScoped) {
+			const customRegistry = detectCustomRegistry(npmrcContent)
+			if (customRegistry) {
+				log.warn(`Found existing default registry: ${customRegistry.trim()}`)
+				log.warn("Using --scoped mode to avoid overriding it.")
+				log.warn(
+					"To use LPM as your default registry, remove the existing registry= line first.",
+				)
+				useScoped = true
+			}
+		}
+
+		// Check if already configured (both scoped and unscoped formats)
+		if (
+			npmrcContent.includes("@lpm.dev:registry") ||
+			npmrcContent.match(/^registry=.*lpm\.dev/m)
+		) {
 			const overwrite = await p.confirm({
 				message: ".npmrc already has LPM configuration. Overwrite?",
 				initialValue: false,
@@ -120,10 +180,14 @@ ${registryHost}/:_authToken=${oidcToken}`
 		}
 	}
 
+	const registryLine = useScoped
+		? `@lpm.dev:registry=${fullRegistryUrl}`
+		: `registry=${fullRegistryUrl}`
+
 	// Add LPM registry config with placeholder
 	const lpmConfig = `
 # LPM Registry
-@lpm.dev:registry=${fullRegistryUrl}
+${registryLine}
 ${registryHost}/:_authToken=\${LPM_TOKEN}
 `.trim()
 
@@ -131,12 +195,18 @@ ${registryHost}/:_authToken=\${LPM_TOKEN}
 	fs.writeFileSync(npmrcPath, `${npmrcContent}\n`)
 
 	p.note(
-		`@lpm.dev:registry=${fullRegistryUrl}\n${registryHost}/:_authToken=\${LPM_TOKEN}`,
+		`${registryLine}\n${registryHost}/:_authToken=\${LPM_TOKEN}`,
 		".npmrc configuration",
 	)
 
 	console.log("")
-	log.success(".npmrc configured for LPM packages.")
+	log.success(".npmrc configured for LPM registry.")
+	if (useScoped) {
+		log.info("Only @lpm.dev packages will route through lpm.dev.")
+	} else {
+		log.info("All packages (LPM + npm) will route through lpm.dev.")
+		log.info("Note: First install may update resolved URLs in your lockfile.")
+	}
 	log.info("For local development: Run `lpm npmrc` to generate a token.")
 	log.info("For CI/CD: Set the LPM_TOKEN environment variable.")
 	log.info("For CI/CD with OIDC: Run `lpm setup --oidc` (no secrets needed).")

package/lib/quality/display.js

@@ -38,9 +38,9 @@ function progressBar(value, max, width = 18) {
 /**
  * Display quality report in the terminal.
  *
- * @param {{ score: number, checks: Array, meta: object }} result
+ * @param {{ score: number, checks: Array, meta: object, warnings?: string[] }} result
  */
-export function displayQualityReport({ score, checks, meta }) {
+export function displayQualityReport({ score, checks, meta, warnings }) {
 	const tierColor = TIER_COLORS[meta.tier] || chalk.white
 	const tierLabel = TIER_LABELS[meta.tier] || meta.tier
 
@@ -76,6 +76,14 @@ export function displayQualityReport({ score, checks, meta }) {
 		}
 	}
 
+	// Show warnings (e.g., stale skills)
+	if (warnings?.length > 0) {
+		if (failed.length === 0) console.log("")
+		for (const warning of warnings) {
+			console.log(`  ${chalk.yellow("⚠")} ${chalk.yellow(warning)}`)
+		}
+	}
+
 	if (failed.length > 0 && score < 90) {
 		console.log(
 			chalk.dim(

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@lpm-registry/cli",
-	"version": "0.2.6",
+	"version": "0.3.0",
 	"description": "CLI for Licensed Package Manager",
 	"author": "LPM <hello@lpm.dev> (https://lpm.dev)",
 	"repository": {