reposets 4.0.0 → 4.0.1

package/bin/reposets.js

@@ -31,7 +31,7 @@ import { Command } from "effect/cli";
 * `0.0.0 (dev build)` from every build until the version was threaded down
 * from here through {@link Invocation}.
 */
-const VERSION = "4.0.0";
+const VERSION = "4.0.1";
 /**
 * The application's directories: XDG resolution and the `reposets` namespace
 * under it.

package/cli/commands/credentials.js

@@ -96,24 +96,6 @@ const SECRET_IN_NAME = "That looks like a token, not a name. It was not stored."
 /** What `--env` names, and what the prompt accepts for it: a shell variable identifier. */
 const ENV_NAME = /^[A-Za-z_][A-Za-z0-9_]*$/;
 /**
-* Write credentials back to the file they were read from.
-*
-* @remarks
-* `ConfigFileShape.save` resolves its own default path — the XDG one — rather
-* than the path `discover` found. With a project-local
-* `reposets.credentials.toml` present, that splits reads from writes: `create`
-* lands in XDG while `list` keeps reading the local file, so a profile is
-* created that the user cannot see and that sync will not use. Writing back to
-* the discovered source keeps one file authoritative, and falls back to `save`
-* only when there is genuinely nothing to update.
-*/
-const saveWhereRead = (file, value) => Effect.gen(function* () {
-	const existing = (yield* file.discover.pipe(Effect.orElseSucceed(() => [])))[0];
-	if (existing === void 0) return yield* file.save(value);
-	yield* file.write(value, existing.path);
-	return existing.path;
-});
-/**
 * Refuse an invocation the user got wrong.
 *
 * @remarks
@@ -284,7 +266,7 @@ const createHandler = (input) => Effect.gen(function* () {
 		...actsAs,
 		github_token
 	};
-	const path = yield* saveWhereRead(credentialsFile, { profiles: {
+	const path = yield* credentialsFile.save({ profiles: {
 		...existing.profiles,
 		[profile]: record
 	} });
@@ -378,7 +360,7 @@ const deleteHandler = (profile) => Effect.gen(function* () {
 	}), "Provide --profile <name>: the profile to delete."));
 	if (credentials.profiles[name] === void 0) return yield* refuse(`Profile '${name}' not found.`);
 	const { [name]: _removed, ...remaining } = credentials.profiles;
-	const path = yield* saveWhereRead(credentialsFile, { profiles: remaining });
+	const path = yield* credentialsFile.save({ profiles: remaining });
 	yield* CliMessage.success(`Deleted profile '${name}' from ${path}.`);
 });
 const deleteCommand = Command.make("delete", { profile: profileFlag }, ({ profile }) => deleteHandler(Option.getOrUndefined(profile))).pipe(Command.withDescription("Remove a credential profile"));

package/cli/commands/doctor.js

@@ -157,31 +157,15 @@ const REQUIRED_PERMISSIONS = [
 		"resolve team slugs (org-owned repos only)"
 	]
 ];
-const levenshtein = (a, b) => {
-	const matrix = [];
-	for (let i = 0; i <= a.length; i++) matrix[i] = [i];
-	for (let j = 0; j <= b.length; j++) matrix[0][j] = j;
-	for (let i = 1; i <= a.length; i++) for (let j = 1; j <= b.length; j++) {
-		const cost = a[i - 1] === b[j - 1] ? 0 : 1;
-		const row = matrix[i];
-		const prev = matrix[i - 1];
-		row[j] = Math.min(prev[j] + 1, row[j - 1] + 1, prev[j - 1] + cost);
-	}
-	return matrix[a.length][b.length];
-};
-/** The nearest known key within edit distance 3, if any. */
-const findClosestMatch = (key, known) => {
-	let best;
-	let bestDist = Number.POSITIVE_INFINITY;
-	for (const candidate of known) {
-		const dist = levenshtein(key, candidate);
-		if (dist < bestDist && dist <= 3) {
-			bestDist = dist;
-			best = candidate;
-		}
-	}
-	return best;
-};
+/**
+* The nearest known key within edit distance 3, if any.
+*
+* @remarks
+* Three edits, the threshold this check had before it moved onto the kit;
+* `Fmt.suggest` defaults to two, `effect/cli`'s threshold. Ties keep the
+* first known key, as before.
+*/
+const findClosestMatch = (key, known) => Fmt.suggest(key, known, { maxDistance: 3 })[0];
 /**
 * The credential resolver `doctor`'s live token check runs through.
 *

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "reposets",
-	"version": "4.0.0",
+	"version": "4.0.1",
 	"private": false,
 	"description": "CLI tool to sync GitHub repo settings, secrets and rulesets across personal repositories",
 	"keywords": [
@@ -48,20 +48,22 @@
 	},
 	"dependencies": {
 		"@1password/sdk": "^0.5.0",
-		"@effect/platform-node": "^4.0.6",
-		"@effected/app": "^0.28.0",
-		"@effected/cli": "^0.23.0",
-		"@effected/config-file": "^0.21.0",
-		"@effected/env": "^0.6.0",
-		"@effected/github": "^0.25.1",
-		"@effected/glob": "^0.15.0",
-		"@effected/schemastore": "^0.26.3",
-		"@effected/store": "^0.20.0",
-		"@effected/toml": "^0.16.0",
-		"@effected/walker": "^0.21.0",
-		"@effected/xdg": "^0.16.0",
+		"@effect/platform-node": "^4.0.7",
+		"@effected/app": "^0.29.0",
+		"@effected/cli": "^0.24.0",
+		"@effected/config-file": "^0.22.0",
+		"@effected/env": "^0.7.0",
+		"@effected/github": "^0.26.0",
+		"@effected/glob": "^0.16.0",
+		"@effected/jsonc": "^0.21.0",
+		"@effected/schemastore": "^0.27.0",
+		"@effected/store": "^0.21.0",
+		"@effected/toml": "^0.17.0",
+		"@effected/walker": "^0.22.0",
+		"@effected/xdg": "^0.17.0",
+		"@effected/yaml": "^0.26.0",
 		"blakejs": "^1.2.1",
-		"effect": "^4.0.6",
+		"effect": "^4.0.7",
 		"ink": "^8.0.0",
 		"react": "^19.3.1"
 	},

package/services/ConfigFiles.js

@@ -78,6 +78,13 @@ const STRICT_KEYS = {
 * `upwardWalk` defaults to `process.cwd()` when given no `cwd`, which made
 * config discovery the one reader of `process` below the entrypoint.
 *
+* **Writes go back to the file that was found.** `AppConfig` pins
+* `saveTarget: "default"`, the XDG path, which would split reads from writes
+* when a project-local `reposets.credentials.toml` exists: `credentials create`
+* would land in XDG while `list` and `sync` keep reading the local file.
+* `"source"` writes to the highest-priority file the chain found, and to XDG
+* only when there is none.
+*
 * @public
 */
 const CredentialsFilesLive = Layer.unwrap(Effect.map(Invocation, ({ cwd }) => AppConfig.layer(ReposetsCredentialsFile, {
@@ -88,7 +95,8 @@ const CredentialsFilesLive = Layer.unwrap(Effect.map(Invocation, ({ cwd }) => Ap
 		filename: CREDENTIALS_FILENAME,
 		cwd
 	})],
-	parseOptions: STRICT_KEYS
+	parseOptions: STRICT_KEYS,
+	saveTarget: "source"
 })));
 /**
 * Raised when `--config` names a path that does not exist.

package/sync/phases/settings.js

@@ -237,7 +237,7 @@ const settingsPhase = Effect.gen(function* () {
 		};
 		let sent;
 		if (!ctx.dryRun) {
-			const outcome = yield* read(settingsClient.applySettings(desired));
+			const outcome = yield* read(settingsClient.applySettings(desired, { unknownKeys: "send" }));
 			if ("failed" in outcome) return {
 				changes: [],
 				errors: [...errors, {
@@ -245,6 +245,7 @@ const settingsPhase = Effect.gen(function* () {
 					message: outcome.failed
 				}]
 			};
+			for (const key of outcome.value.unverified) yield* logger.syncOperation("apply", "setting", key, "(unconfirmed: GitHub does not describe this field)");
 			sent = [...outcome.value.rest, ...outcome.value.graphql];
 			yield* applied.record({
 				repo: ctx.slug,

package/tsdoc-metadata.json

@@ -5,7 +5,7 @@
   "toolPackages": [
     {
       "packageName": "@microsoft/api-extractor",
-      "packageVersion": "7.59.4"
+      "packageVersion": "7.60.0"
     }
   ]
 }