neon 6.2.4 → 6.4.0

package/README.md

@@ -665,7 +665,37 @@ export default async function hello(): Promise<Response> {
 }
 ```
 
-An existing `neon.ts` (or `hello.ts`) is never overwritten.
+An existing `neon.ts` (or `hello.ts`) is never overwritten. To change one, use `config add`.
+
+### Editing a `neon.ts` (`config add`)
+
+`neon config add` declares one service in the `neon.ts` you already have and leaves the rest of the file (comments, the `branch` closure, formatting) as it was. It is purely local — no auth, no API calls — and creates the starter `neon.ts` first when there is none.
+
+```bash
+neon config add auth                        # auth: true
+neon config add data-api                    # dataApi: true, and auth: true (the default provider needs it)
+neon config add ai-gateway                  # aiGateway: true
+neon config add function sendemail          # functions.sendemail + functions/sendemail.ts
+neon config add function sendemail --name "Send email" --source ./src/send.ts
+neon config add bucket uploads              # buckets.uploads = { access: 'private' }
+neon config add bucket assets --access public_read
+```
+
+Each command prints what it changed and the next step:
+
+```
+$ neon config add function sendemail
+INFO: Created functions/sendemail.ts.
+INFO: Updated neon.ts: added functions.sendemail.
+INFO: Next: `neon dev` to run it locally, `neon config apply` to deploy.
+```
+
+- **Functions** take a slug of 1-20 lowercase letters and digits, the same rule `plan` enforces. The handler file is written first and an existing file is never overwritten, so pointing `--source` at code you already have works. A `.js` `neon.ts` gets a `.js` handler.
+- Enabling a service that `neon.ts` already enables changes nothing and exits 0. Adding a function slug or bucket name that already exists is an error.
+- `--config <path>` edits a specific file; otherwise the closest `neon.ts` above the current directory is used, and the suggested `plan` / `apply` commands repeat the flag.
+- `--no-install` applies when the command had to create `neon.ts`; editing an existing file never installs anything.
+
+`config add` edits the file as text and checks its own output: it re-applies the edit to the result and fails if that is not a no-op. When the file is in a shape it cannot edit safely (a single-line `defineConfig({ auth: true })`, a service declared under `preview`, a value that is not a literal), it changes nothing and prints the exact lines to paste.
 
 Four sub-commands plus two top-level aliases drive it:
 
@@ -792,7 +822,7 @@ Recommended installs the Neon plugin for every detected agent (global config, pr
 
 Unauthenticated `-y` skips linking and prints the next step: sign up at https://neon.com/signup, then `neon auth`, `neon link`, or `neon claim create` (no account, expires in 72 hours unless claimed). `-y` never opens a browser. `-y` with `--project-id` or other account flags links when the CLI is already signed in; otherwise it errors and names `neon auth` or `--claimable`.
 
-Custom asks how to add Neon to coding agents (plugin, skills and MCP separately, or skip), then how to get a project (sign in and link, or a claimable project when you are not signed in), then which services `neon.ts` should declare. Postgres is listed first as always included. The package manager is inferred from the directory; Custom asks only when none is detected. `--skill` selects skills (not the plugin) and skips that picker. MCP flags (`--mcp-config-location`, `--mcp-auth`, `--mcp-project-scoped`) select skills and MCP. `--mcp-config-location` is where the config is written: `global` maps to `neon mcp`, and `project` maps to `neon mcp --project`. With `-y`, the location defaults to `global`; interactive Custom asks. If some selected agents cannot write that location, init warns and skips MCP for those agents while continuing with the supported agents. It fails when none of the selected agents support the location. `--mcp-project-scoped` limits MCP tools to the linked Neon project by passing its ID to `neon mcp --project-id`. Omitted is false, including in Custom; there is no prompt. `--no-agent-setup` skips agent setup.
+Custom asks how to add Neon to coding agents (plugin, skills and MCP separately, or skip), then how to get a project (sign in and link, or a claimable project when you are not signed in), then which services `neon.ts` should declare. Skills and MCP use separate agent pickers, and either can be skipped. Postgres is listed first as always included. The package manager is inferred from the directory; Custom asks only when none is detected. `--skill` selects skills (not the plugin) and skips that picker. MCP flags (`--mcp-config-location`, `--mcp-auth`, `--mcp-project-scoped`) select skills and MCP. `--mcp-config-location` is where the config is written: `global` maps to `neon mcp`, and `project` maps to `neon mcp --project`. With `-y`, the location defaults to `global`; interactive Custom asks. If some selected agents cannot write that location, init warns and skips MCP for those agents while continuing with the supported agents. It fails when none of the selected agents support the location. `--mcp-project-scoped` limits MCP tools to the linked Neon project by passing its ID to `neon mcp --project-id`. Omitted is false, including in Custom; there is no prompt. `--no-agent-setup` skips agent setup.
 
 Empty directories are set up in place. `neon init` does not scaffold starter files.
 

package/dist/commands/auth.js

@@ -5,7 +5,7 @@ import { log } from "../log.js";
 import { getApiClient, isNeonApiError } from "../api.js";
 import { setAuthContext } from "../auth_context.js";
 import { a as isOwnedCredentialPath } from "../_chunks/paths-DMq0Lt7a.js";
-import { currentContextFile, isAskCommand, isClaimCommand, isConfigInit, isCurrentBranchProbe, isGitLocalCommand, isInspectDbUrl, isMcpCommand, isMcpOauth, isPluginsCommand, isProfileCommand, isSkillsCommand, isUnfollowedGitHookSync, readContextFile } from "../context.js";
+import { currentContextFile, isAskCommand, isClaimCommand, isConfigAdd, isConfigInit, isCurrentBranchProbe, isGitLocalCommand, isInspectDbUrl, isMcpCommand, isMcpOauth, isPluginsCommand, isProfileCommand, isSkillsCommand, isUnfollowedGitHookSync, readContextFile } from "../context.js";
 import { _ as upsertProfile, d as newProfileLocation, g as selectProfileName, h as resolveProfile, i as assertValidProfileName, l as locationForName, m as readProfiles, n as KEYRING_CREDENTIALS, p as profilesUsingPath, r as assertProfilesUsable, s as isKeyringPointer, t as DEFAULT_PROFILE, u as locationOf } from "../_chunks/profiles-CvnFEQyd.js";
 import { extendTokenSet } from "../utils/auth.js";
 import { AuthRefreshError, auth, classifyRefreshFailure, refreshToken } from "../auth.js";
@@ -296,6 +296,7 @@ const ensureAuth = async (props) => {
 	if (props._.length === 0 || props.help) return;
 	if (isCurrentBranchProbe(props)) return;
 	if (isConfigInit(props)) return;
+	if (isConfigAdd(props)) return;
 	if (isProfileCommand(props)) return;
 	if (isGitLocalCommand(props)) return;
 	if (isUnfollowedGitHookSync(props)) return;

package/dist/commands/claim.js

@@ -6,8 +6,8 @@ import { applyContext, contextBranch, readContextFile } from "../context.js";
 import { t as credentialInputs } from "../_chunks/auth_selection-pitMFzDe.js";
 import { ClaimableClient, ClaimableServiceError } from "../claimable/api.js";
 import { assertionHasExpired, listClaimableCredentials, readClaimableCredentials, readLinkedClaimableCredentials, removeClaimableCredentials, writeClaimableCredentials } from "../claimable/state.js";
-import { claimableDataApiCreateBody, declaredNeonServices } from "../config_services.js";
 import { NEON_SERVICES, deprecatedServiceMessage, parseServices, servicesFlagValue, servicesOption } from "../neon_services.js";
+import { claimableDataApiCreateBody, declaredNeonServices } from "../config_services.js";
 import { resolveEnvFilePath } from "../env_file.js";
 import { writer } from "../writer.js";
 import { noPassthrough } from "../utils/flags.js";

package/dist/commands/config.js

@@ -4,10 +4,11 @@ import { log } from "../log.js";
 import { getApiClient } from "../api.js";
 import { contextBranch, ensureDirectoryGitignored, readContextFile } from "../context.js";
 import { getCliName } from "../utils/cli_name.js";
+import { deprecatedServiceMessage, parseServices, servicesFlagValue, servicesOption } from "../neon_services.js";
+import { CONFIG_INIT_NONE_MEANS, CONFIG_INIT_SERVICES, CONFIG_INIT_UNAVAILABLE, FUNCTION_FILENAME, FUNCTION_SLUG, FUNCTION_TEMPLATE, REQUIRED_PACKAGES, renderFunctionSource, renderNeonConfig, renderNeonConfigFromView } from "../config_template.js";
+import { editNeonConfig } from "../config_edit.js";
 import { toNeonConfigView } from "../config_format.js";
 import { declaredNeonServices } from "../config_services.js";
-import { deprecatedServiceMessage, parseServices, servicesFlagValue, servicesOption } from "../neon_services.js";
-import { CONFIG_INIT_NONE_MEANS, CONFIG_INIT_SERVICES, CONFIG_INIT_UNAVAILABLE, FUNCTION_FILENAME, FUNCTION_SLUG, FUNCTION_TEMPLATE, REQUIRED_PACKAGES, renderNeonConfig, renderNeonConfigFromView } from "../config_template.js";
 import { loadEnvFileIntoProcess } from "../env_file.js";
 import { writer } from "../writer.js";
 import { fillSingleProject, resolveBranchRef } from "../utils/enrichers.js";
@@ -21,14 +22,16 @@ import { resolveHookEnv, runDeployAfterHook, runDeployBeforeHook } from "../util
 import { pickServicesInteractively } from "../utils/service_picker.js";
 import { zipBundle } from "../utils/zip.js";
 import { autoPullEnvAfterPin } from "./env.js";
-import { existsSync, readFileSync, writeFileSync } from "node:fs";
-import { dirname, join } from "node:path";
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
+import { dirname, join, relative, resolve } from "node:path";
+import { homedir } from "node:os";
 import chalk from "chalk";
 import { packagesToStage, previewGaWarningForConfig, resolveConfig } from "@neon/config";
 import { PushConflictError, apply, assertZipWithinLimits, bundleAsIs, createBranch, describeNativeFinding, enforceLimits, findUndeclaredNativePackages, inspect, isPartialBranchCreateError, loadConfigFromFile as loadConfigFromFile$1, plan, resolveEsbuildEntry, traceNativePackages, zipFunctionBundle } from "@neon/config-runtime";
 //#region src/commands/config.ts
 var config_exports = /* @__PURE__ */ __exportAll({
 	ConfigInstallFailed: () => ConfigInstallFailed,
+	addCmd: () => addCmd,
 	applyCmd: () => applyCmd,
 	applyFlags: () => applyFlags,
 	applyPolicyOnCreate: () => applyPolicyOnCreate,
@@ -213,6 +216,28 @@ const seedFromBranch = async (props) => {
 		branchName: live.branch.name
 	};
 };
+/** Install the config packages the project is missing, or say how to. */
+const ensureConfigPackages = async (cwd, props) => {
+	const run = props.run ?? runCommand;
+	const missing = missingDependencies(cwd);
+	if (missing.length === 0) {
+		if (!props.silent) log.info("%s are already installed.", REQUIRED_PACKAGES.join(" and "));
+		return;
+	}
+	const pm = props.packageManager ?? resolvePackageManager(cwd);
+	const args = installArgs(pm, missing);
+	if (props.install === false) {
+		if (!props.silent) log.info("Install the Neon config packages to use neon.ts: %s", formatInstallCommand(pm, missing));
+		return;
+	}
+	ensureDirectoryGitignored(join(cwd, "node_modules"));
+	if (!props.silent) log.info("Installing %s with %s…", missing.join(", "), pm);
+	if (!await run(pm, args, cwd)) {
+		const command = formatInstallCommand(pm, missing);
+		if (props.requireInstall) throw new ConfigInstallFailed(command);
+		log.warning("Could not install the config packages automatically. Run by hand: %s", command);
+	}
+};
 /**
 * Scaffold a `neon.ts` policy and make sure the Neon config packages are
 * installed, so a project can go straight to `neon config plan` / `apply`.
@@ -220,10 +245,12 @@ const seedFromBranch = async (props) => {
 */
 const initCmd = async (props) => {
 	const cwd = props.cwd ?? process.cwd();
-	const run = props.run ?? runCommand;
 	const existing = neonConfigFilename(cwd);
 	if (existing) {
-		if (!props.silent) log.info("Found an existing %s — leaving it untouched.", existing);
+		if (!props.silent) {
+			log.info("Found an existing %s — leaving it untouched.", existing);
+			if (props.services !== void 0) log.warning("--services was ignored. To declare services in the existing %s, use `%s config add`.", existing, getCliName());
+		}
 	} else if (props.fromBranch) {
 		const { source, seeded, branchName } = await seedFromBranch(props);
 		writeFileSync(join(cwd, "neon.ts"), source);
@@ -240,26 +267,139 @@ const initCmd = async (props) => {
 		}
 		if (services.includes("functions")) scaffoldFunction(cwd);
 	}
-	const missing = missingDependencies(cwd);
-	if (missing.length === 0) {
-		if (!props.silent) log.info("%s are already installed.", REQUIRED_PACKAGES.join(" and "));
-	} else {
-		const pm = props.packageManager ?? resolvePackageManager(cwd);
-		const args = installArgs(pm, missing);
-		if (props.install === false) {
-			if (!props.silent) log.info("Install the Neon config packages to use neon.ts: %s", formatInstallCommand(pm, missing));
-		} else {
-			ensureDirectoryGitignored(join(cwd, "node_modules"));
-			if (!props.silent) log.info("Installing %s with %s…", missing.join(", "), pm);
-			if (!await run(pm, args, cwd)) {
-				const command = formatInstallCommand(pm, missing);
-				if (props.requireInstall) throw new ConfigInstallFailed(command);
-				log.warning("Could not install the config packages automatically. Run by hand: %s", command);
+	await ensureConfigPackages(cwd, props);
+	if (!props.silent) log.info("Next: edit neon.ts, then run `neon config plan` to preview and `neon config apply`.");
+};
+/** The rule the platform enforces on function slugs, so a bad one fails before any write. */
+const FUNCTION_SLUG_PATTERN = /^[a-z0-9]{1,20}$/;
+const SCRIPT_EXTENSION = /\.(?:ts|mts|cts|js|mjs|cjs)$/;
+const TYPESCRIPT_EXTENSION = /\.(?:ts|mts|cts)$/;
+/**
+* The nearest neon.ts, found the way the config loader finds it: walk up from `start`, stop
+* at the first directory holding `.git` (or at the home directory).
+*/
+const findNeonConfigUpward = (start) => {
+	let dir = resolve(start);
+	for (;;) {
+		const name = neonConfigFilename(dir);
+		if (name) return join(dir, name);
+		if (existsSync(join(dir, ".git")) || dir === homedir()) return void 0;
+		const parent = dirname(dir);
+		if (parent === dir) return void 0;
+		dir = parent;
+	}
+};
+const asRelativeModulePath = (path) => path.startsWith("./") || path.startsWith("../") || path.startsWith("/") ? path : `./${path}`;
+const shellQuote = (value) => /^[\w@%+=:,./-]+$/.test(value) ? value : `'${value.replace(/'/g, "'\\''")}'`;
+/**
+* `explicitConfig` is the `--config` the user passed. The commands suggested afterwards must
+* carry it, or they would act on whichever `neon.ts` a walk up from cwd finds instead.
+*/
+const planAdd = (target, configPath, explicitConfig) => {
+	const language = TYPESCRIPT_EXTENSION.test(configPath) ? "ts" : "js";
+	const cli = getCliName();
+	const flag = explicitConfig ? ` --config ${shellQuote(explicitConfig)}` : "";
+	const provision = `Next: \`${cli} config plan${flag}\` to preview, \`${cli} config apply${flag}\` to provision.`;
+	switch (target.kind) {
+		case "service": return {
+			edit: {
+				kind: "service",
+				service: target.service
+			},
+			next: provision
+		};
+		case "bucket":
+			if (target.name.length < 1 || target.name.length > 255) throw new Error("A bucket name must be 1-255 characters.");
+			if (target.name === "__proto__") throw new Error("\"__proto__\" cannot be used as a bucket name.");
+			return {
+				edit: {
+					kind: "bucket",
+					name: target.name,
+					access: target.access ?? "private"
+				},
+				next: provision
+			};
+		case "function": {
+			if (!FUNCTION_SLUG_PATTERN.test(target.slug)) {
+				const suggestion = target.slug.toLowerCase().replace(/[^a-z0-9]/g, "").slice(0, 20);
+				throw new Error(`"${target.slug}" is not a valid function slug: use 1-20 lowercase letters and digits, no hyphens.` + (suggestion ? ` Try "${suggestion}".` : ""));
 			}
+			if (target.source !== void 0 && !SCRIPT_EXTENSION.test(target.source)) throw new Error(`--source must be a .ts or .js file, got "${target.source}". Declare a directory source in neon.ts by hand.`);
+			const source = asRelativeModulePath(target.source ?? `functions/${target.slug}.${language}`);
+			return {
+				edit: {
+					kind: "function",
+					slug: target.slug,
+					name: target.name ?? target.slug,
+					source
+				},
+				sourceFile: resolve(dirname(configPath), source),
+				next: explicitConfig ? `Next: \`${cli} config apply${flag}\` to deploy.` : `Next: \`${cli} dev\` to run it locally, \`${cli} config apply\` to deploy.`
+			};
 		}
 	}
-	if (!props.silent) log.info("Next: edit neon.ts, then run `neon config plan` to preview and `neon config apply`.");
 };
+/**
+* Declare a service, function, or bucket in the project's `neon.ts` with a text edit that
+* leaves the rest of the file alone, creating the file (and installing the config packages)
+* when the project has none. Local only, like `config init`.
+*/
+const addCmd = async (props) => {
+	const cwd = props.cwd ?? process.cwd();
+	let existing;
+	if (props.config) {
+		existing = resolve(cwd, props.config);
+		if (!existsSync(existing)) throw new Error(`--config ${props.config} does not exist.`);
+	} else existing = findNeonConfigUpward(cwd);
+	const configPath = existing ?? join(cwd, "neon.ts");
+	const shown = (path) => relative(cwd, path) || path;
+	const plan = planAdd(props.target, configPath, props.config);
+	const { source, changes } = editNeonConfig(existing ? readFileSync(existing, "utf8") : renderNeonConfig([]), plan.edit);
+	if (changes.length === 0) {
+		if (props.target.kind === "service") {
+			log.info("%s already enables that; nothing to change.", shown(configPath));
+			return;
+		}
+		throw new Error(`${shown(configPath)} already declares ${props.target.kind} "${props.target.kind === "function" ? props.target.slug : props.target.name}". Edit it there, or pick another name.`);
+	}
+	if (plan.sourceFile) {
+		if (existsSync(plan.sourceFile)) log.info("Found an existing %s — leaving it untouched.", shown(plan.sourceFile));
+		else {
+			mkdirSync(dirname(plan.sourceFile), { recursive: true });
+			const slug = props.target.kind === "function" ? props.target.slug : "";
+			writeFileSync(plan.sourceFile, renderFunctionSource(slug, TYPESCRIPT_EXTENSION.test(plan.sourceFile) ? "ts" : "js"));
+			log.info("Created %s.", shown(plan.sourceFile));
+		}
+	}
+	writeFileSync(configPath, source);
+	log.info("%s %s: %s.", existing ? "Updated" : "Created", shown(configPath), changes.join(", "));
+	if (!existing) await ensureConfigPackages(dirname(configPath), props);
+	log.info("%s", plan.next);
+};
+/**
+* Options every `config add` sub-command takes. `--project-id` and `--branch` are inherited
+* from `config` but mean nothing to a command that never leaves the working tree, so they are
+* hidden from its help.
+*/
+const addSharedOptions = (yargs) => yargs.strict().options({
+	"project-id": { hidden: true },
+	branch: { hidden: true },
+	config: {
+		describe: "Path to the neon.ts to edit (defaults to walking up from cwd; created in cwd when there is none)",
+		type: "string"
+	},
+	install: {
+		describe: "Install @neon/config and @neon/env when creating neon.ts. On by default; use --no-install to just print the command.",
+		type: "boolean",
+		default: true
+	}
+});
+/** The `config add` props for a sub-command's parsed flags. */
+const addProps = (args, target) => ({
+	target,
+	install: args.install,
+	...args.config !== void 0 ? { config: args.config } : {}
+});
 const command = "config";
 const describe = "Manage a branch with a neon.ts policy";
 const builder = (argv) => argv.usage("$0 config <sub-command> [options]").options({
@@ -296,7 +436,47 @@ const builder = (argv) => argv.usage("$0 config <sub-command> [options]").option
 	...envFlag,
 	...applyFlags,
 	...envPullFlag
-}), (args) => applyCmd(args)).command("init", "Scaffold a neon.ts policy and install the Neon config packages", (yargs) => yargs.options({
+}), (args) => applyCmd(args)).command("add", "Add a service, function, or bucket to neon.ts (creates it if missing)", (yargs) => yargs.command("auth", "Enable Neon Auth (auth: true)", addSharedOptions, (args) => addCmd(addProps(args, {
+	kind: "service",
+	service: "auth"
+}))).command("data-api", "Enable the Data API (dataApi: true) and Neon Auth, which the default provider requires", addSharedOptions, (args) => addCmd(addProps(args, {
+	kind: "service",
+	service: "data-api"
+}))).command("ai-gateway", "Enable the AI Gateway (aiGateway: true)", addSharedOptions, (args) => addCmd(addProps(args, {
+	kind: "service",
+	service: "ai-gateway"
+}))).command("function <slug>", "Declare a Neon Function and create its handler file", (yargs) => addSharedOptions(yargs).example("$0 config add function sendemail", "Creates functions/sendemail.ts and declares it").example("$0 config add function sendemail --source src/send.ts", "Declares an existing handler").positional("slug", {
+	describe: "Function slug: 1-20 lowercase letters and digits",
+	type: "string",
+	demandOption: true
+}).options({
+	name: {
+		describe: "Display name. Defaults to the slug",
+		type: "string"
+	},
+	source: {
+		describe: "Handler file, relative to neon.ts. Defaults to functions/<slug>.ts. Created when missing, left alone when it exists",
+		type: "string"
+	}
+}), (args) => addCmd(addProps(args, {
+	kind: "function",
+	slug: args.slug,
+	...args.name !== void 0 ? { name: args.name } : {},
+	...args.source !== void 0 ? { source: args.source } : {}
+}))).command("bucket <name>", "Declare an Object Storage bucket", (yargs) => addSharedOptions(yargs).example("$0 config add bucket assets --access public_read", "Declares a bucket that allows anonymous reads").positional("name", {
+	describe: "Bucket name",
+	type: "string",
+	demandOption: true
+}).options({ access: {
+	describe: "Anonymous access: private requires credentials, public_read allows anonymous reads",
+	type: "string",
+	choices: ["private", "public_read"],
+	default: "private"
+} }), (args) => addCmd(addProps(args, {
+	kind: "bucket",
+	name: args.name,
+	access: args.access
+}))).demandCommand(1, "Specify what to add: auth, data-api, ai-gateway, function, or bucket.")).command("init", "Scaffold a neon.ts policy and install the Neon config packages", (yargs) => yargs.options({
 	install: {
 		describe: "Install @neon/config and @neon/env if they're missing. On by default; use --no-install to just print the command.",
 		type: "boolean",
@@ -803,4 +983,4 @@ const createBranchFromPolicyOnCheckout = async (props) => {
 	}
 };
 //#endregion
-export { ConfigInstallFailed, applyCmd, applyFlags, applyPolicyOnCreate, builder, command, createBranchFromPolicyOnCheckout, describe, envFlag, envPullFlag, handler, hasNeonConfigFile, initCmd, neonConfigFilename, planCmd, status, config_exports as t };
+export { ConfigInstallFailed, addCmd, applyCmd, applyFlags, applyPolicyOnCreate, builder, command, createBranchFromPolicyOnCheckout, describe, envFlag, envPullFlag, handler, hasNeonConfigFile, initCmd, neonConfigFilename, planCmd, status, config_exports as t };

package/dist/commands/mcp.js

@@ -6,10 +6,11 @@ import { getCliName } from "../utils/cli_name.js";
 import { helpCsv, helpEpilogue } from "../utils/help_text.js";
 import { writer } from "../writer.js";
 import { noPassthrough, single } from "../utils/flags.js";
+import { AGENT_MCP_SKIP_MESSAGE } from "../init/copy.js";
 import { NEON_MCP_CATEGORIES, NEON_MCP_URL, existingNeonApiKey, installNeonMcpServer, neonMcpUrl, parseMcpCategories, trackedProjectMcpConfig } from "../mcp/install.js";
 import { mcpInstallableAgents, resolveInstallTargets } from "../mcp/targets.js";
 import { mintMcpApiKey, mintedKeyRevokeCommand, withdrawMintedKey } from "../mcp/mint.js";
-import { canPickAgentsInteractively } from "../utils/agent_picker.js";
+import { canPickAgentsInteractively, skippableAgentPicker } from "../utils/agent_picker.js";
 import { resolveMcpPlan } from "../mcp/plan.js";
 //#region src/commands/mcp.ts
 var mcp_exports = /* @__PURE__ */ __exportAll({
@@ -113,7 +114,8 @@ const setupNeonMcp = async (options) => {
 		readOnly: options.readOnly === true,
 		projectId: options.projectId,
 		categories: options.category ?? [],
-		linkedProjectId
+		linkedProjectId,
+		pickAgents: skippableAgentPicker(options.allowAgentSkip, AGENT_MCP_SKIP_MESSAGE)
 	});
 	const { install, skipped } = resolveInstallTargets({
 		agents: plan.agents,

package/dist/commands/plugins.js

@@ -4,11 +4,12 @@ import { recordCommandSuccessExtras } from "../analytics.js";
 import { helpCsv, helpEpilogue } from "../utils/help_text.js";
 import { writer } from "../writer.js";
 import { noPassthrough } from "../utils/flags.js";
+import { AGENT_PLUGIN_SKIP_MESSAGE } from "../init/copy.js";
 import { getAgentDisplayName } from "../mcp/agents.js";
 import { NEON_MCP_URL } from "../mcp/install.js";
 import "../init/agents.js";
 import { pluginsInstallableAgents } from "../plugins/targets.js";
-import { canPickAgentsInteractively } from "../utils/agent_picker.js";
+import { canPickAgentsInteractively, skippableAgentPicker } from "../utils/agent_picker.js";
 import { resolvePluginsPlan } from "../plugins/plan.js";
 import { NEON_PLUGIN_NAME, PLUGIN_SKILLS, PLUGIN_SOURCE, neonPluginsRetryCommand, pluginsAddArgs, runPluginsCli } from "../plugins/run.js";
 //#region src/commands/plugins.ts
@@ -71,10 +72,12 @@ const installPlugins = async (options) => {
 		agents: options.agents ?? [],
 		yes,
 		cwd,
-		interactive
+		interactive,
+		pickAgents: skippableAgentPicker(options.allowAgentSkip, AGENT_PLUGIN_SKIP_MESSAGE)
 	});
 	for (const agent of plan.skipped) log.warning("Skipping %s: no plugins mapping.", getAgentDisplayName(agent));
 	for (const agent of plan.userScopeSkipped) log.warning("Skipping %s: plugins are user-level. Pass --global.", getAgentDisplayName(agent));
+	for (const { agent, command } of plan.missingCommands) log.warning("Skipping %s: the plugin installs through \"%s\", which was not found on PATH.", getAgentDisplayName(agent), command);
 	const rows = [];
 	const failed = [];
 	const scope = scopeLabel(plan.scope);
@@ -147,7 +150,8 @@ const reportPluginsInstall = (props, outcome) => {
 		title: "Plugins"
 	});
 	out.end();
-	if (outcome.failed.length === 0) log.info(outcome.scope === "project" ? "Installed the Neon plugin (project)." : "Installed the Neon plugin (user).");
+	if (outcome.rows.length === 0) log.info("No Neon plugins installed.");
+	else if (outcome.failed.length === 0) log.info(outcome.scope === "project" ? "Installed the Neon plugin (project)." : "Installed the Neon plugin (user).");
 };
 const handler = async (props) => {
 	const outcome = await installPlugins({

package/dist/commands/skills.js

@@ -4,10 +4,11 @@ import { recordCommandSuccessExtras } from "../analytics.js";
 import { helpCsv, helpEpilogue } from "../utils/help_text.js";
 import { writer } from "../writer.js";
 import { noPassthrough } from "../utils/flags.js";
+import { AGENT_SKILLS_SKIP_MESSAGE } from "../init/copy.js";
 import { getAgentDisplayName } from "../mcp/agents.js";
 import "../init/agents.js";
 import { mappedSkillsAgentNames, skillsInstallableAgents } from "../skills/targets.js";
-import { canPickAgentsInteractively } from "../utils/agent_picker.js";
+import { canPickAgentsInteractively, skippableAgentPicker } from "../utils/agent_picker.js";
 import { skillsHelpValues, skillsYesHelp } from "../skills/catalog.js";
 import { confirmSkillsUpdate } from "../skills/wizard.js";
 import { assertSkillsCanRun, resolveSkillsPlan } from "../skills/plan.js";
@@ -102,7 +103,8 @@ const installSkills = async (options) => {
 		skills: options.skills ?? [],
 		yes,
 		cwd,
-		interactive
+		interactive,
+		pickAgents: skippableAgentPicker(options.allowAgentSkip, AGENT_SKILLS_SKIP_MESSAGE)
 	});
 	const mapped = mappedSkillsAgentNames(plan.agents);
 	for (const agent of plan.skipped) log.warning("Skipping %s: no skills mapping.", getAgentDisplayName(agent));