pi-fold 3.0.0 → 3.1.0

package/extensions/settings.ts

@@ -50,17 +50,6 @@ const THRESHOLD_FIELDS = [
 
 export type FoldSettingId = typeof THRESHOLD_FIELDS[number];
 
-function readProportion(raw: string, field: string): number {
-	const value = Number(raw);
-	if (!Number.isFinite(value) || value <= 0 || value >= 1) {
-		throw new ThresholdPolicyError(
-			field,
-			`thresholds.${field} must be a proportion above 0 and below 1`,
-		);
-	}
-	return value;
-}
-
 // One edit, applied against the WHOLE draft: the merged thresholds object is
 // re-validated through resolveThresholds so cross-field invariants
 // (minTarget < maxTarget) hold at every saved state, never
@@ -71,18 +60,10 @@ export function applyFoldSettingsEdit(
 	rawValue: string,
 ): { ok: true; draft: FoldSettingsFile } | { ok: false; error: string } {
 	try {
-		if (id === "consolidateAfter" || id === "minFoldChars") {
-			const value = Number(rawValue.trim());
-			if (!Number.isSafeInteger(value)) {
-				return { ok: false, error: `thresholds.${id} must be a whole number` };
-			}
-			const thresholds = resolveThresholds({
-				...(draft.thresholds ?? DEFAULT_THRESHOLDS),
-				[id]: value,
-			});
-			return { ok: true, draft: { ...draft, thresholds } };
+		const value = Number(rawValue.trim());
+		if ((id === "consolidateAfter" || id === "minFoldChars") && !Number.isSafeInteger(value)) {
+			return { ok: false, error: `thresholds.${id} must be a whole number` };
 		}
-		const value = readProportion(rawValue.trim(), id);
 		const thresholds = resolveThresholds({
 			...(draft.thresholds ?? DEFAULT_THRESHOLDS),
 			[id]: value,
@@ -245,16 +226,15 @@ function shareCandidates(id: "maxTarget" | "minTarget"): number[] {
 	return values;
 }
 
-function allowedValues(id: FoldSettingId, thresholds: ActiveContextThresholds, budgetTokens: number): string[] {
-	if (id === "consolidateAfter") return CONSOLIDATE_CHOICES.map(String);
-	if (id === "minFoldChars") return MIN_FOLD_CHOICES.map((value) => `${value.toLocaleString("en-US")} chars`);
+// The lattice a row steps along, as numbers. Display belongs to rowDisplayValue and
+// only to it: this list is stepped, never shown.
+function allowedValues(id: FoldSettingId, thresholds: ActiveContextThresholds): number[] {
+	if (id === "consolidateAfter") return CONSOLIDATE_CHOICES;
+	if (id === "minFoldChars") return MIN_FOLD_CHOICES;
 	const { minTarget, maxTarget } = thresholds;
-	const candidates = id === "maxTarget"
+	return id === "maxTarget"
 		? shareCandidates("maxTarget").filter((v) => v > minTarget)
 		: shareCandidates("minTarget").filter((v) => v < maxTarget);
-	// Entries carry the SAME display format as currentValue: SettingsList cycles by
-	// indexOf(currentValue), so a format mismatch wraps every first press to the head.
-	return candidates.map((v) => `${v.toFixed(2)} · ${Math.round(v * budgetTokens).toLocaleString("en-US")} tok`);
 }
 
 function rowRawValue(settings: FoldSettingsFile, id: FoldSettingId): string {
@@ -385,7 +365,7 @@ export class FoldSettingsEditor extends Container {
 						row.label,
 						row.description,
 						rowRawValue(draft, row.id),
-						(raw) => this.applyEdit(row.id, raw),
+						(raw) => this.applyAndSave(row.id, raw),
 						submenuDone,
 					),
 			})),
@@ -405,8 +385,7 @@ export class FoldSettingsEditor extends Container {
 	// surface; an exact off-lattice value arrives only through Enter's editor.
 	private step(id: FoldSettingId, direction: number): void {
 		const thresholds = this.draft.thresholds ?? DEFAULT_THRESHOLDS;
-		const candidates = allowedValues(id, thresholds).map((entry) =>
-			Number.parseFloat(entry.replace(/,/g, "")));
+		const candidates = allowedValues(id, thresholds);
 		const current = Number(rowRawValue(this.draft, id));
 		const target = direction > 0
 			? candidates.find((candidate) => candidate > current + 1e-9)
@@ -427,10 +406,6 @@ export class FoldSettingsEditor extends Container {
 		return { ok: true, display: rowDisplayValue(this.draft, id, this.budgetTokens) };
 	}
 
-	private applyEdit(id: FoldSettingId, raw: string): { ok: true; display: string } | { ok: false; error: string } {
-		return this.applyAndSave(id, raw);
-	}
-
 	handleInput(data: string): void {
 		// An open submenu owns everything until it closes.
 		if (this.settingsList.submenuComponent) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-fold",
-  "version": "3.0.0",
+  "version": "3.1.0",
   "description": "Lossless, agent-governed context folding for the Pi coding agent: only the oldest context changes; the fresh window stays untouched.",
   "keywords": [
     "pi-package",