@usesocial/cli 0.12.0 → 0.12.2

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # @usesocial/cli
 
+## 0.12.2
+
+### Patch Changes
+
+- [#73](https://github.com/usesocial/monorepo/pull/73) [`666b356`](https://github.com/usesocial/monorepo/commit/666b356706c40b7a3dc5e14ec53d76428b25f5c4) Thanks [@CyrusNuevoDia](https://github.com/CyrusNuevoDia)! - Stop double-wrapping X list mutation output under nested data objects.
+
+- [#73](https://github.com/usesocial/monorepo/pull/73) [`1cfedc9`](https://github.com/usesocial/monorepo/commit/1cfedc967b3fa82249bbca059856cc641c87868e) Thanks [@CyrusNuevoDia](https://github.com/CyrusNuevoDia)! - Detect Bun-installed social CLI when printing update commands.
+
+## 0.12.1
+
+### Patch Changes
+
+- [#72](https://github.com/usesocial/monorepo/pull/72) [`affa4d3`](https://github.com/usesocial/monorepo/commit/affa4d30c27cdad8512893065ebd2df080b79aca) Thanks [@CyrusNuevoDia](https://github.com/CyrusNuevoDia)! - Fix X list OAuth scopes and list read field presets.
+
 ## 0.12.0
 
 ### Minor Changes

package/dist/index.mjs

@@ -2,7 +2,7 @@
 import { createRequire } from "node:module";
 import * as fs from "node:fs";
 import { createReadStream, existsSync, mkdirSync, readFileSync, realpathSync, writeFileSync } from "node:fs";
-import { basename, dirname, extname, join, resolve } from "node:path";
+import { basename, delimiter, dirname, extname, join, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 import { homedir } from "node:os";
 import { parseArgs, styleText } from "node:util";
@@ -21098,9 +21098,9 @@ const buildDelete = (deps) => {
 	});
 };
 const listFields = "created_at,description,follower_count,id,member_count,name,owner_id,private";
-const listUserFields = "affiliation,confirmed_email,connection_status,created_at,description,entities,id,is_identity_verified,location,most_recent_tweet_id,name,parody,pinned_tweet_id,profile_banner_url,profile_image_url,protected,public_metrics,receives_your_dm,subscription,subscription_type,url,username,verified,verified_followers_count,verified_type,withheld";
+const listUserFields = "affiliation,created_at,description,entities,id,is_identity_verified,location,most_recent_tweet_id,name,parody,pinned_tweet_id,profile_banner_url,profile_image_url,protected,public_metrics,subscription,subscription_type,url,username,verified,verified_followers_count,verified_type,withheld";
 const listMemberExpansions = "affiliation.user_id,most_recent_tweet_id,pinned_tweet_id";
-const listMemberTweetFields = "article,attachments,author_id,card_uri,community_id,context_annotations,conversation_id,created_at,display_text_range,edit_controls,edit_history_tweet_ids,entities,geo,id,in_reply_to_user_id,lang,matched_media_notes,media_metadata,non_public_metrics,note_request_suggestions,note_tweet,organic_metrics,paid_partnership,possibly_sensitive,promoted_metrics,public_metrics,referenced_tweets,reply_settings,scopes,source,suggested_source_links,suggested_source_links_with_counts,text,withheld";
+const listMemberTweetFields = "article,attachments,author_id,card_uri,community_id,context_annotations,conversation_id,created_at,display_text_range,edit_controls,edit_history_tweet_ids,entities,geo,id,in_reply_to_user_id,lang,matched_media_notes,media_metadata,note_request_suggestions,note_tweet,paid_partnership,possibly_sensitive,public_metrics,referenced_tweets,reply_settings,scopes,source,suggested_source_links,suggested_source_links_with_counts,text,withheld";
 const listReadParams = () => {
 	const params = new URLSearchParams();
 	params.set("list.fields", listFields);
@@ -21189,14 +21189,23 @@ const listOutputSchema = object({
 	includes: record(string(), unknown()).optional(),
 	meta: record(string(), unknown()).optional()
 }).passthrough();
-const listWriteAggregateOutputSchema = object({ data: object({
+const ListWriteOutput = record(string(), unknown());
+const ListWriteResponse = object({
+	data: ListWriteOutput.optional(),
+	errors: array(record(string(), unknown())).optional()
+}).passthrough();
+const ListWriteAggregateOutput = object({
 	list_id: string(),
 	results: array(object({
 		input: string(),
 		user_id: string(),
 		response: unknown()
 	}))
-}) }).passthrough();
+}).passthrough();
+const listWriteDataFromResponse = async (response) => {
+	const data = ListWriteResponse.parse(await response.json());
+	return data.data ?? data;
+};
 const readCost = (category, opts) => {
 	const usage = priceUsage(costBPSForCategory(category));
 	return {
@@ -21270,7 +21279,7 @@ const listReadContract = (inputSchema, pagination) => manualSchemaContract({
 });
 const listWriteContract = (args) => manualSchemaContract({
 	inputSchema: args.inputSchema,
-	outputSchema: args.outputSchema ?? listOutputSchema,
+	outputSchema: args.outputSchema ?? ListWriteOutput,
 	auth: {
 		required: true,
 		scopes: ["read", "write"]
@@ -21376,13 +21385,13 @@ const listDeleteContract = listWriteContract({
 const listMemberWriteContract = listWriteContract({
 	inputSchema: ListMemberWriteInput,
 	method: "POST",
-	outputSchema: listWriteAggregateOutputSchema,
+	outputSchema: ListWriteAggregateOutput,
 	pricing: "list_manage"
 });
 const listMemberRemoveContract = listWriteContract({
 	inputSchema: ListMemberWriteInput,
 	method: "DELETE",
-	outputSchema: listWriteAggregateOutputSchema,
+	outputSchema: ListWriteAggregateOutput,
 	destructive: true,
 	pricing: "list_manage"
 });
@@ -21563,7 +21572,7 @@ const buildLists = (deps) => {
 			await deps.printResult({
 				account: own.account,
 				contract: listCreateContract,
-				data: await response.json(),
+				data: await listWriteDataFromResponse(response),
 				responseHeaders: response.headers
 			});
 		}
@@ -21624,7 +21633,7 @@ const buildLists = (deps) => {
 			await deps.printResult({
 				account: own.account,
 				contract: listUpdateContract,
-				data: await response.json(),
+				data: await listWriteDataFromResponse(response),
 				resolved: [target.record],
 				responseHeaders: response.headers
 			});
@@ -21660,7 +21669,7 @@ const buildLists = (deps) => {
 			await deps.printResult({
 				account: own.account,
 				contract: listDeleteContract,
-				data: await response.json(),
+				data: await listWriteDataFromResponse(response),
 				resolved: [target.record],
 				responseHeaders: response.headers
 			});
@@ -21792,16 +21801,16 @@ const buildList = (deps) => {
 				results.push({
 					input: user,
 					user_id: userResolved.id,
-					response: await response.json()
+					response: await listWriteDataFromResponse(response)
 				});
 			}
 			await deps.printResult({
 				account: own.account,
 				contract: kind === "add" ? listMemberWriteContract : listMemberRemoveContract,
-				data: { data: {
+				data: {
 					list_id: target.id,
 					results
-				} },
+				},
 				resolved
 			});
 		}
@@ -23913,7 +23922,7 @@ function createEnv(opts) {
 }
 //#endregion
 //#region package.json
-var version$1 = "0.12.0";
+var version$1 = "0.12.2";
 //#endregion
 //#region src/lib/env.ts
 const URLWithTrailingSlash = url().transform(ensureTrailingSlash);
@@ -25446,8 +25455,8 @@ const isArray = nativeIsArray || function(obj) {
 };
 const isObject = (x) => x === Object(x) && !isArray(x);
 const isUndefined = (x) => void 0 === x;
-const isString = (x) => "[object String]" == type_utils_toString.call(x);
-const isEmptyString = (x) => isString(x) && 0 === x.trim().length;
+const isString$1 = (x) => "[object String]" == type_utils_toString.call(x);
+const isEmptyString = (x) => isString$1(x) && 0 === x.trim().length;
 const isNumber = (x) => "[object Number]" == type_utils_toString.call(x) && x === x;
 const isPlainError = (x) => x instanceof Error;
 function isPrimitive(value) {
@@ -26272,7 +26281,7 @@ var ObjectCoercer = class {
 		return `${className && "Object" !== className ? `'${className}'` : "Object"} captured as exception with keys: ${keys}`;
 	}
 	isSeverityLevel(x) {
-		return isString(x) && !isEmptyString(x) && severityLevels.indexOf(x) >= 0;
+		return isString$1(x) && !isEmptyString(x) && severityLevels.indexOf(x) >= 0;
 	}
 	getErrorPropertyFromObject(obj) {
 		for (const prop in obj) if (Object.prototype.hasOwnProperty.call(obj, prop)) {
@@ -38025,12 +38034,47 @@ const compareSemver = (left, right) => {
 	}
 	return 0;
 };
-const detectPackageManager = (packageRoot) => {
-	const normalized = packageRoot.replaceAll("\\", "/");
+const normalizedPath = (path) => path.replaceAll("\\", "/");
+const managerFromPath = (path) => {
+	const normalized = normalizedPath(path);
 	if (normalized.includes("/Cellar/usesocial/") || normalized.includes("/Cellar/cli/") || normalized.includes("/Homebrew/")) return "brew";
-	if (normalized.includes("/.bun/install/global/")) return "bun";
+	if (normalized.includes("/.bun/bin/") || normalized.includes("/.bun/install/global/")) return "bun";
+};
+const defaultRealpath = (path) => {
+	try {
+		return realpathSync(path);
+	} catch {
+		return;
+	}
+};
+const isString = (value) => value !== void 0;
+const packageManagerPaths = (packageRoot, options) => {
+	const pathExists = options.pathExists ?? existsSync;
+	const realpath = options.realpath ?? defaultRealpath;
+	const paths = [options.executablePath];
+	if (options.pathEnv) for (const binDir of options.pathEnv.split(delimiter)) {
+		if (!binDir) continue;
+		for (const binary of ["social", "social.cmd"]) {
+			const candidate = join(binDir, binary);
+			if (pathExists(candidate)) paths.push(candidate);
+		}
+	}
+	paths.push(packageRoot);
+	return [...new Set(paths.flatMap((path) => path ? [path, realpath(path)] : []).filter(isString))];
+};
+const detectPackageManager = (packageRoot, options = {}) => {
+	for (const path of packageManagerPaths(packageRoot, options)) {
+		const manager = managerFromPath(path);
+		if (manager) return manager;
+	}
 	return "npm";
 };
+const detectPackageManagerFor = (deps) => detectPackageManager(deps.packageRoot ?? internalPackageRoot(), {
+	executablePath: deps.executablePath ?? process.argv[1],
+	pathEnv: deps.pathEnv ?? process.env.PATH,
+	pathExists: deps.pathExists,
+	realpath: deps.realpath
+});
 const cliUpdateCommand = (manager) => {
 	if (manager === "brew") return "brew upgrade usesocial/tap/cli";
 	if (manager === "bun") return "bun install -g @usesocial/cli@latest";
@@ -38074,12 +38118,12 @@ const unknownCLIStatus = () => ({
 	updateCommand: null
 });
 const checkCLIUpdate = async (deps = {}) => {
-	const packageRoot = deps.packageRoot ?? internalPackageRoot();
+	const manager = detectPackageManagerFor(deps);
 	const fetchImpl = deps.fetch ?? fetch;
 	try {
 		const response = await fetchImpl(NPM_REGISTRY_URL, { headers: { accept: "application/json" } });
 		if (!response.ok) return unknownCLIStatus();
-		return parseNPMUpdateResult(await response.json(), VERSION, detectPackageManager(packageRoot));
+		return parseNPMUpdateResult(await response.json(), VERSION, manager);
 	} catch {
 		return unknownCLIStatus();
 	}
@@ -38243,7 +38287,7 @@ const unknownSkillStatus = () => ({
 	updateCommand: null
 });
 const checkSkillUpdate = async (deps = {}) => {
-	const manager = detectPackageManager(deps.packageRoot ?? internalPackageRoot());
+	const manager = detectPackageManagerFor(deps);
 	const install = await installedSkillCandidate(deps);
 	if (!install) return {
 		scope: null,
@@ -38314,20 +38358,20 @@ const withNotice = (cache, key, now) => ({
 		[key]: now
 	}
 });
-const noticeLinesFor = (cache, now) => {
+const noticeLinesFor = (cache, now, manager) => {
 	const notices = [];
-	if (cache.cli.status === "update_available" && cache.cli.latestVersion && cache.cli.updateCommand) {
+	if (cache.cli.status === "update_available" && cache.cli.currentVersion === VERSION && cache.cli.latestVersion && compareSemver(cache.cli.latestVersion, VERSION) > 0) {
 		const key = `cli:${cache.cli.latestVersion}`;
 		if (noticeDue(cache, key, now)) notices.push({
 			key,
-			line: `Update available: social CLI ${cache.cli.latestVersion} (current ${cache.cli.currentVersion}). Run: ${cache.cli.updateCommand}`
+			line: `Update available: social CLI ${cache.cli.latestVersion} (current ${cache.cli.currentVersion}). Run: ${cliUpdateCommand(manager)}`
 		});
 	}
-	if (cache.skill.status === "update_available" && cache.skill.scope && cache.skill.updateCommand && cache.skill.remoteTreeSHA) {
+	if (cache.skill.status === "update_available" && cache.skill.scope && cache.skill.remoteTreeSHA) {
 		const key = `skill:${cache.skill.scope}:${cache.skill.remoteTreeSHA}`;
 		if (noticeDue(cache, key, now)) notices.push({
 			key,
-			line: `Update available: social skill (${cache.skill.scope}). Run: ${cache.skill.updateCommand}`
+			line: `Update available: social skill (${cache.skill.scope}). Run: ${skillUpdateCommand(cache.skill.scope, manager)}`
 		});
 	}
 	return notices;
@@ -38351,8 +38395,9 @@ const maybeRunUpdateAwareness = async (deps = {}) => {
 	const home = deps.home;
 	const cache = await readCachedUpdateStatus(home);
 	if (cache) {
+		const manager = detectPackageManagerFor(deps);
 		let next = cache;
-		for (const notice of noticeLinesFor(cache, now)) {
+		for (const notice of noticeLinesFor(cache, now, manager)) {
 			printStderrLine(notice.line);
 			next = withNotice(next, notice.key, now);
 		}