@gajae-code/coding-agent 0.16.7 → 0.17.1

package/src/cli/mcp-cli.ts

@@ -19,6 +19,7 @@ import {
 	resolveScopeMCPConfigPath,
 	upsertScopeMCPServer,
 } from "../runtime-mcp/scope-config";
+import { type AutoloadStatus, computeAutoloadStatus } from "../runtime-mcp/startup-policy";
 import type { MCPServerConfig } from "../runtime-mcp/types";
 
 export type MCPAction = "add" | "list" | "remove";
@@ -51,9 +52,6 @@ interface ScopedPath {
 	path: string;
 }
 
-/** Runtime status of a stored registration for conventional autoload. */
-type AutoloadStatus = "autoload" | "autoload-off" | "disabled";
-
 interface RedactedServerEntry {
 	name: string;
 	config: MCPServerConfig;
@@ -72,29 +70,21 @@ const REDACTED = "<redacted>";
 const SENSITIVE_KEY_PATTERN =
 	/(?:token|secret|key|credential|password|passwd|pwd|authorization|auth|bearer|cookie|session)/i;
 
-/**
- * Compute the conventional-autoload status of a stored server.
- *
- * - `autoload-off`: the server opted out of startup connection via `autoload: false`.
- * - `disabled`: the server is disabled via `enabled: false` or the `disabledServers` list.
- * - `autoload`: connected by ordinary standalone sessions at startup.
- */
-export function computeAutoloadStatus(
-	name: string,
-	config: MCPServerConfig,
-	disabledServers: ReadonlySet<string>,
-): AutoloadStatus {
-	if (config.enabled === false || disabledServers.has(name)) return "disabled";
-	if (config.autoload === false) return "autoload-off";
-	return "autoload";
-}
-
 function autoloadStatusNote(status: AutoloadStatus): string {
 	switch (status) {
 		case "autoload":
 			return "Loaded by ordinary standalone gjc sessions at startup.";
 		case "autoload-off":
-			return "Configured but not auto-loaded at startup (autoload: false); connect on demand via /mcp.";
+			// Editing the stored flag is the startup path: ordinary startup
+			// (`discoverAndConnect` without `autoloadOnly`) skips opted-out
+			// servers, and exact-config loading (`--mcp-config`) sets
+			// `autoloadOnly`, so naming the file skips it too. Necessary, not
+			// sufficient — `enabled: false` and `disabledServers` block independently.
+			return (
+				"Configured but not auto-loaded at startup (autoload: false). To load it at startup, set autoload to true or " +
+				"remove the key in this config file, then start a new session; --mcp-config does not override the " +
+				"flag. It must also stay enabled and out of disabledServers."
+			);
 		case "disabled":
 			return "Disabled; not loaded by sessions. Re-enable to autoload.";
 	}

package/src/cli/nofile-limit.ts

@@ -38,7 +38,11 @@ export function warnIfMacOSNoFileLimitTooLow(deps: WarnIfMacOSNoFileLimitTooLowD
 	const run = deps.execFileSync ?? execFileSync;
 	let currentLimit: number | undefined;
 	try {
-		currentLimit = parseNoFileLimit(run("/bin/sh", ["-lc", "ulimit -n"], { encoding: "utf8" }));
+		// Plain `sh -c`, not `-lc`: the value that matters is the limit this process
+		// inherited (children report their inherited soft limit), and `-l` sources
+		// the user's shell profile on every launch purely to produce a number that
+		// may not even match what gjc is actually running under.
+		currentLimit = parseNoFileLimit(run("/bin/sh", ["-c", "ulimit -n"], { encoding: "utf8" }));
 	} catch {
 		return;
 	}

package/src/cli/plugin-cli.ts

@@ -1281,7 +1281,12 @@ async function handleSetEnabled(
 		}
 
 		try {
-			await manager.setEnabled(name, enabled);
+			const scope = flags.scope ?? "user";
+			const state = await manager.getEnablementState(name, scope);
+			if (state.status !== "ok") {
+				throw new Error(state.status === "malformed" ? "plugin config is malformed" : "not installed");
+			}
+			await manager.setEnabled(name, enabled, scope, state.baseline);
 			if (flags.json) {
 				console.log(JSON.stringify({ [jsonKey]: name }));
 			} else {