@narumitw/pi-sync 0.50.3 → 0.51.1

package/src/sync/startup-check.ts

@@ -7,122 +7,108 @@ import { withLock } from "../state/lock.js";
 import { withStateDirectoryAccess } from "../state/state-directory.js";
 import { readStateForConfig, syncStateFingerprint } from "../state/sync-state-store.js";
 import type { SyncAttentionController } from "../ui/sync-attention.js";
+import { setSyncStatus } from "../ui/sync-status.js";
 import { safeTerminalText } from "../ui/terminal-text.js";
 import { combineSignals, throwIfAborted } from "./signals.js";
 import { errorMessage } from "./sync-errors.js";
 import type { SyncLoaders } from "./sync-loaders.js";
 
-const STATUS_KEY = "sync";
-
 /** One owned task per extension session, never keyed by the shared headless UI. */
-export function createStartupCheck(
-	loaders: SyncLoaders,
-	attention: SyncAttentionController,
-	timeoutMs = 30_000,
-) {
-	let active:
-		| {
-				controller: AbortController;
-				promise: Promise<void>;
-				owner: ExtensionContext["sessionManager"];
-		  }
-		| undefined;
+export function createStartupCheck(loaders: SyncLoaders, attention: SyncAttentionController, timeoutMs = 30_000) {
+  let active:
+    | {
+        controller: AbortController;
+        promise: Promise<void>;
+        owner: ExtensionContext["sessionManager"];
+      }
+    | undefined;
 
-	return {
-		start(ctx: ExtensionContext, sessionSignal: AbortSignal, initialConfig?: AnySyncConfig) {
-			if (!ctx.hasUI || sessionSignal.aborted) return;
-			if (active) throw new Error("Previous startup check has not been drained.");
-			const controller = new AbortController();
-			const signal = combineSignals(sessionSignal, controller.signal);
-			const task = { controller, owner: ctx.sessionManager, promise: Promise.resolve() };
-			active = task;
-			const isCurrent = () =>
-				!sessionSignal.aborted && active === task && task.owner === ctx.sessionManager;
-			const timer = setTimeout(
-				() => controller.abort(new DOMException("Startup check timed out", "TimeoutError")),
-				timeoutMs,
-			);
-			let config: AnySyncConfig | undefined;
-			let checking = false;
-			// Handle every detached rejection here; Pi's awaited event runner cannot catch it.
-			task.promise = Promise.resolve()
-				.then(async () => {
-					try {
-						throwIfAborted(signal);
-						config = initialConfig ?? (await loadConfigForCheck());
-						throwIfAborted(signal);
-						if (!config.automatic) return;
-						const captured = config;
-						const identity = syncCheckConfigFingerprint(captured);
-						checking = true;
-						ctx.ui.setStatus(STATUS_KEY, "checking in background");
-						const { inspectSync } = await loaders.inspection();
-						throwIfAborted(signal);
-						const currentConfig = await loadConfigForCheck();
-						throwIfAborted(signal);
-						if (syncCheckConfigFingerprint(currentConfig) !== identity) return;
-						// Lock order: state migration guard → sync operation → backend cache.
-						// Read heads may write only private cache data. Never call doctor/syncBoth.
-						const inspection = await withStateDirectoryAccess(() => {
-							throwIfAborted(signal);
-							return withLock("startup-check", () => {
-								throwIfAborted(signal);
-								return inspectSync(captured, snapshotOptionsForContext(ctx, captured), signal);
-							});
-						});
-						throwIfAborted(signal);
-						const latest = await loadConfigForCheck();
-						throwIfAborted(signal);
-						if (syncCheckConfigFingerprint(latest) !== identity) return;
-						const state = await withStateDirectoryAccess(() => readStateForConfig(latest));
-						throwIfAborted(signal);
-						if (syncStateFingerprint(state) !== inspection.stateIdentity || !isCurrent()) return;
-						attention.observe({
-							setupName: captured.setupName,
-							configIdentity: identity,
-							checkedAt: new Date().toISOString(),
-							inspection,
-						});
-						if (ctx.mode === "rpc") attention.notifyObservation(ctx);
-					} catch (error) {
-						if (!isCurrent() || sessionSignal.aborted || isMissingConfigError(error)) return;
-						if (signal.aborted && controller.signal.reason?.name !== "TimeoutError") return;
-						// A settings edit invalidates failures as well as successful observations.
-						if (config) {
-							const latest = await loadConfigForCheck().catch(() => undefined);
-							if (
-								!isCurrent() ||
-								(signal.aborted && controller.signal.reason?.name !== "TimeoutError")
-							)
-								return;
-							if (
-								!latest ||
-								syncCheckConfigFingerprint(latest) !== syncCheckConfigFingerprint(config)
-							)
-								return;
-						}
-						ctx.ui.notify(
-							`pi-sync startup check skipped: ${safeTerminalText(errorMessage(error))}. Run /sync status to retry. No startup transfer was performed.`,
-							"warning",
-						);
-					} finally {
-						clearTimeout(timer);
-						if (isCurrent() && checking) await attention.publish(ctx, sessionSignal);
-						if (active === task) active = undefined;
-					}
-				})
-				.catch(() => {
-					// UI teardown can itself throw. No unhandled rejection or retained timer.
-					clearTimeout(timer);
-					if (active === task) active = undefined;
-				});
-		},
-		async stop() {
-			const task = active;
-			if (!task) return;
-			task.controller.abort(new DOMException("Startup check cancelled", "AbortError"));
-			// Drain the actual work, including child/ref cleanup, before foreground guards.
-			await task.promise;
-		},
-	};
+  return {
+    start(ctx: ExtensionContext, sessionSignal: AbortSignal, initialConfig?: AnySyncConfig) {
+      if (!ctx.hasUI || sessionSignal.aborted) return;
+      if (active) throw new Error("Previous startup check has not been drained.");
+      const controller = new AbortController();
+      const signal = combineSignals(sessionSignal, controller.signal);
+      const task = { controller, owner: ctx.sessionManager, promise: Promise.resolve() };
+      active = task;
+      const isCurrent = () => !sessionSignal.aborted && active === task && task.owner === ctx.sessionManager;
+      const timer = setTimeout(
+        () => controller.abort(new DOMException("Startup check timed out", "TimeoutError")),
+        timeoutMs,
+      );
+      let config: AnySyncConfig | undefined;
+      let checking = false;
+      // Handle every detached rejection here; Pi's awaited event runner cannot catch it.
+      task.promise = Promise.resolve()
+        .then(async () => {
+          try {
+            throwIfAborted(signal);
+            config = initialConfig ?? (await loadConfigForCheck());
+            throwIfAborted(signal);
+            if (!config.automatic) return;
+            const captured = config;
+            const identity = syncCheckConfigFingerprint(captured);
+            checking = true;
+            setSyncStatus(ctx, "sync ...");
+            const { inspectSync } = await loaders.inspection();
+            throwIfAborted(signal);
+            const currentConfig = await loadConfigForCheck();
+            throwIfAborted(signal);
+            if (syncCheckConfigFingerprint(currentConfig) !== identity) return;
+            // Lock order: state migration guard → sync operation → backend cache.
+            // Read heads may write only private cache data. Never call doctor/syncBoth.
+            const inspection = await withStateDirectoryAccess(() => {
+              throwIfAborted(signal);
+              return withLock("startup-check", () => {
+                throwIfAborted(signal);
+                return inspectSync(captured, snapshotOptionsForContext(ctx, captured), signal);
+              });
+            });
+            throwIfAborted(signal);
+            const latest = await loadConfigForCheck();
+            throwIfAborted(signal);
+            if (syncCheckConfigFingerprint(latest) !== identity) return;
+            const state = await withStateDirectoryAccess(() => readStateForConfig(latest));
+            throwIfAborted(signal);
+            if (syncStateFingerprint(state) !== inspection.stateIdentity || !isCurrent()) return;
+            attention.observe({
+              setupName: captured.setupName,
+              configIdentity: identity,
+              checkedAt: new Date().toISOString(),
+              inspection,
+            });
+            if (ctx.mode === "rpc") attention.notifyObservation(ctx);
+          } catch (error) {
+            if (!isCurrent() || sessionSignal.aborted || isMissingConfigError(error)) return;
+            if (signal.aborted && controller.signal.reason?.name !== "TimeoutError") return;
+            // A settings edit invalidates failures as well as successful observations.
+            if (config) {
+              const latest = await loadConfigForCheck().catch(() => undefined);
+              if (!isCurrent() || (signal.aborted && controller.signal.reason?.name !== "TimeoutError")) return;
+              if (!latest || syncCheckConfigFingerprint(latest) !== syncCheckConfigFingerprint(config)) return;
+            }
+            ctx.ui.notify(
+              `pi-sync startup check skipped: ${safeTerminalText(errorMessage(error))}. Run /sync status to retry. No startup transfer was performed.`,
+              "warning",
+            );
+          } finally {
+            clearTimeout(timer);
+            if (isCurrent() && checking) await attention.publish(ctx, sessionSignal);
+            if (active === task) active = undefined;
+          }
+        })
+        .catch(() => {
+          // UI teardown can itself throw. No unhandled rejection or retained timer.
+          clearTimeout(timer);
+          if (active === task) active = undefined;
+        });
+    },
+    async stop() {
+      const task = active;
+      if (!task) return;
+      task.controller.abort(new DOMException("Startup check cancelled", "AbortError"));
+      // Drain the actual work, including child/ref cleanup, before foreground guards.
+      await task.promise;
+    },
+  };
 }

package/src/sync/sync-decision.ts

@@ -9,100 +9,96 @@ import { includeFromSelectionConfig } from "./sync-policy.js";
 import { syncPolicyChanged } from "./sync-state.js";
 
 export type SyncDecisionKind =
-	| "remote-or-policy-changed"
-	| "both-changed"
-	| "first-sync-settings-diverged"
-	| "first-sync-sessions-diverged"
-	| "remote-empty";
+  | "remote-or-policy-changed"
+  | "both-changed"
+  | "first-sync-settings-diverged"
+  | "first-sync-sessions-diverged"
+  | "remote-empty";
 
 export type SyncResolutionDirection = "push" | "pull";
 
 export interface SyncDecision {
-	kind: SyncDecisionKind;
-	setupName: string;
-	configIdentity: string;
-	causes: {
-		localChanged: boolean;
-		remoteChanged: boolean;
-		policyChanged: boolean;
-	};
-	previousInclude?: readonly string[];
-	currentInclude: readonly string[];
-	review: string;
-	directions: readonly SyncResolutionDirection[];
-	directMessage: string;
+  kind: SyncDecisionKind;
+  setupName: string;
+  configIdentity: string;
+  causes: {
+    localChanged: boolean;
+    remoteChanged: boolean;
+    policyChanged: boolean;
+  };
+  previousInclude?: readonly string[];
+  currentInclude: readonly string[];
+  review: string;
+  directions: readonly SyncResolutionDirection[];
+  directMessage: string;
 }
 
 export {
-	isSyncDecisionRequiredError,
-	SyncDecisionRequiredError,
+  isSyncDecisionRequiredError,
+  SyncDecisionRequiredError,
 } from "./sync-errors.js";
 
 interface CreateSyncDecisionOptions {
-	kind: SyncDecisionKind;
-	config: AnySyncConfig;
-	state: SyncState;
-	local: Snapshot;
-	remote?: Snapshot;
-	localChanged: boolean;
-	remoteChanged: boolean;
-	directMessage: string;
+  kind: SyncDecisionKind;
+  config: AnySyncConfig;
+  state: SyncState;
+  local: Snapshot;
+  remote?: Snapshot;
+  localChanged: boolean;
+  remoteChanged: boolean;
+  directMessage: string;
 }
 
 export function createSyncDecision(options: CreateSyncDecisionOptions) {
-	const { config, state, local, remote, kind } = options;
-	const policyChanged = Boolean(state.lastAppliedSnapshot) && syncPolicyChanged(state, config);
-	const previousInclude = includeFromSelectionConfig(state);
-	const currentInclude = [...config.include];
-	const causes = {
-		localChanged: options.localChanged,
-		remoteChanged: options.remoteChanged,
-		policyChanged,
-	};
-	const causeLines =
-		kind === "first-sync-settings-diverged"
-			? ["This machine and the remote have different Pi settings on first sync."]
-			: kind === "first-sync-sessions-diverged"
-				? ["Pi settings match, but local and remote sessions differ on first sync."]
-				: kind === "remote-empty"
-					? ["The remote storage location is empty."]
-					: [
-							...(causes.localChanged ? ["Local content changed since the last sync."] : []),
-							...(causes.remoteChanged ? ["Remote content changed since the last sync."] : []),
-							...(causes.policyChanged ? ["Included content changed since the last sync."] : []),
-						];
-	const comparison = remote
-		? formatDiff(local, remote)
-		: formatSnapshotOnlyDiff("Remote is empty. Local push would upload", local);
-	const review = [
-		`Sync setup: ${safeTerminalText(config.setupName)}`,
-		"",
-		"Why a decision is required:",
-		...causeLines,
-		...(policyChanged
-			? [
-					"",
-					`Previously included: ${safeList(previousInclude)}`,
-					`Currently included: ${safeList(currentInclude)}`,
-				]
-			: []),
-		"",
-		"Observed differences:",
-		...comparison.split("\n").map(safeTerminalText),
-	].join("\n");
-	return new SyncDecisionRequiredError({
-		kind,
-		setupName: config.setupName,
-		configIdentity: syncConfigReviewIdentity(config),
-		causes,
-		...(policyChanged ? { previousInclude: [...previousInclude] } : {}),
-		currentInclude,
-		review,
-		directions: kind === "remote-empty" ? ["push"] : ["push", "pull"],
-		directMessage: options.directMessage,
-	});
+  const { config, state, local, remote, kind } = options;
+  const policyChanged = Boolean(state.lastAppliedSnapshot) && syncPolicyChanged(state, config);
+  const previousInclude = includeFromSelectionConfig(state);
+  const currentInclude = [...config.include];
+  const causes = {
+    localChanged: options.localChanged,
+    remoteChanged: options.remoteChanged,
+    policyChanged,
+  };
+  const causeLines =
+    kind === "first-sync-settings-diverged"
+      ? ["This machine and the remote have different Pi settings on first sync."]
+      : kind === "first-sync-sessions-diverged"
+        ? ["Pi settings match, but local and remote sessions differ on first sync."]
+        : kind === "remote-empty"
+          ? ["The remote storage location is empty."]
+          : [
+              ...(causes.localChanged ? ["Local content changed since the last sync."] : []),
+              ...(causes.remoteChanged ? ["Remote content changed since the last sync."] : []),
+              ...(causes.policyChanged ? ["Included content changed since the last sync."] : []),
+            ];
+  const comparison = remote
+    ? formatDiff(local, remote)
+    : formatSnapshotOnlyDiff("Remote is empty. Local push would upload", local);
+  const review = [
+    `Sync setup: ${safeTerminalText(config.setupName)}`,
+    "",
+    "Why a decision is required:",
+    ...causeLines,
+    ...(policyChanged
+      ? ["", `Previously included: ${safeList(previousInclude)}`, `Currently included: ${safeList(currentInclude)}`]
+      : []),
+    "",
+    "Observed differences:",
+    ...comparison.split("\n").map(safeTerminalText),
+  ].join("\n");
+  return new SyncDecisionRequiredError({
+    kind,
+    setupName: config.setupName,
+    configIdentity: syncConfigReviewIdentity(config),
+    causes,
+    ...(policyChanged ? { previousInclude: [...previousInclude] } : {}),
+    currentInclude,
+    review,
+    directions: kind === "remote-empty" ? ["push"] : ["push", "pull"],
+    directMessage: options.directMessage,
+  });
 }
 
 function safeList(values: readonly string[]) {
-	return values.length > 0 ? values.map(safeTerminalText).join(", ") : "none";
+  return values.length > 0 ? values.map(safeTerminalText).join(", ") : "none";
 }

package/src/sync/sync-error-guidance.ts

@@ -2,52 +2,45 @@ import { errorMessage } from "./sync-errors.js";
 
 /** Add recovery advice without changing backend outcome semantics or printing error causes. */
 export function syncErrorGuidance(error: unknown): string {
-	const detail = errorMessage(error);
-	const message = detail.length > 1200 ? `${detail.slice(0, 1200)}… (details truncated)` : detail;
-	const code = (error as NodeJS.ErrnoException | undefined)?.code;
-	const status = (error as { status?: number } | undefined)?.status;
-	let next: string;
-	if (code === "ENOSPC") {
-		next = "Free disk space for the private pi-sync settings/state directory, then retry.";
-	} else if (code && ["EACCES", "EPERM", "EROFS"].includes(code)) {
-		next = "Check write access to the private pi-sync settings/state directory, then retry.";
-	} else if (code && ["EBUSY", "EIO", "EMFILE", "ENFILE"].includes(code)) {
-		next =
-			"Check local disk availability and other processes using the private settings file, then retry.";
-	} else if (/changed while|type changed|reopen/iu.test(message)) {
-		next = "Reopen the item in /sync and review its current settings before saving again.";
-	} else if (
-		status === 401 ||
-		status === 403 ||
-		/Permission denied|Authentication failed|HTTP (401|403)|failed \((401|403)\)/iu.test(message)
-	) {
-		next =
-			"Check the storage connection's credentials and remote permissions in /sync → More → Storage connections.";
-	} else if (status === 404 || /HTTP 404|failed \(404\)/iu.test(message)) {
-		next =
-			"Verify the connection address and setup's bucket, branch, or path with your provider; this response alone does not prove which resource is missing.";
-	} else if (status !== undefined && status >= 500) {
-		next = "Check your storage provider's service status, then retry Check setup before syncing.";
-	} else if (
-		/timed out|timeout|request failed|resolve host|Could not resolve|ENOTFOUND|ECONNREFUSED/iu.test(
-			message,
-		)
-	) {
-		next =
-			"Check the server address and network connection, then run /sync doctor for the affected setup.";
-	} else if (/strong ETag|If-Match|If-None-Match|precondition|conditional header/iu.test(message)) {
-		next =
-			"Ask your provider to check strong ETags and conditional writes, including proxy behavior, or choose compatible storage. Do not bypass these safety checks.";
-	} else if (/spawn git ENOENT/iu.test(message)) {
-		next = "Install Git 2.30 or newer and make sure Pi can find git on PATH, then retry.";
-	} else if (/response exceeds.*byte limit/iu.test(message)) {
-		next =
-			"Verify the storage endpoint; the server returned an unexpectedly large response. Check your provider's logs before retrying.";
-	} else if (/settings|config|already exists|duplicates|same normalized/iu.test(message)) {
-		next =
-			"Correct the named field or item in /sync. If the settings file is invalid, repair it before retrying; do not replace it with defaults.";
-	} else {
-		next = "Open /sync → More → Check setup and review the reported checks before retrying.";
-	}
-	return `${message}\n${next}`;
+  const detail = errorMessage(error);
+  const message = detail.length > 1200 ? `${detail.slice(0, 1200)}… (details truncated)` : detail;
+  const code = (error as NodeJS.ErrnoException | undefined)?.code;
+  const status = (error as { status?: number } | undefined)?.status;
+  let next: string;
+  if (code === "ENOSPC") {
+    next = "Free disk space for the private pi-sync settings/state directory, then retry.";
+  } else if (code && ["EACCES", "EPERM", "EROFS"].includes(code)) {
+    next = "Check write access to the private pi-sync settings/state directory, then retry.";
+  } else if (code && ["EBUSY", "EIO", "EMFILE", "ENFILE"].includes(code)) {
+    next = "Check local disk availability and other processes using the private settings file, then retry.";
+  } else if (/changed while|type changed|reopen/iu.test(message)) {
+    next = "Reopen the item in /sync and review its current settings before saving again.";
+  } else if (
+    status === 401 ||
+    status === 403 ||
+    /Permission denied|Authentication failed|HTTP (401|403)|failed \((401|403)\)/iu.test(message)
+  ) {
+    next = "Check the storage connection's credentials and remote permissions in /sync → More → Storage connections.";
+  } else if (status === 404 || /HTTP 404|failed \(404\)/iu.test(message)) {
+    next =
+      "Verify the connection address and setup's bucket, branch, or path with your provider; this response alone does not prove which resource is missing.";
+  } else if (status !== undefined && status >= 500) {
+    next = "Check your storage provider's service status, then retry Check setup before syncing.";
+  } else if (/timed out|timeout|request failed|resolve host|Could not resolve|ENOTFOUND|ECONNREFUSED/iu.test(message)) {
+    next = "Check the server address and network connection, then run /sync doctor for the affected setup.";
+  } else if (/strong ETag|If-Match|If-None-Match|precondition|conditional header/iu.test(message)) {
+    next =
+      "Ask your provider to check strong ETags and conditional writes, including proxy behavior, or choose compatible storage. Do not bypass these safety checks.";
+  } else if (/spawn git ENOENT/iu.test(message)) {
+    next = "Install Git 2.30 or newer and make sure Pi can find git on PATH, then retry.";
+  } else if (/response exceeds.*byte limit/iu.test(message)) {
+    next =
+      "Verify the storage endpoint; the server returned an unexpectedly large response. Check your provider's logs before retrying.";
+  } else if (/settings|config|already exists|duplicates|same normalized/iu.test(message)) {
+    next =
+      "Correct the named field or item in /sync. If the settings file is invalid, repair it before retrying; do not replace it with defaults.";
+  } else {
+    next = "Open /sync → More → Check setup and review the reported checks before retrying.";
+  }
+  return `${message}\n${next}`;
 }

package/src/sync/sync-errors.ts

@@ -3,21 +3,21 @@ import type { SyncDecision } from "./sync-decision.js";
 export class SetupPullRequiresUiError extends Error {}
 
 export class SyncDecisionRequiredError extends Error {
-	readonly decision: SyncDecision;
+  readonly decision: SyncDecision;
 
-	constructor(decision: SyncDecision) {
-		super(decision.directMessage);
-		this.name = "SyncDecisionRequiredError";
-		this.decision = decision;
-	}
+  constructor(decision: SyncDecision) {
+    super(decision.directMessage);
+    this.name = "SyncDecisionRequiredError";
+    this.decision = decision;
+  }
 }
 
 export function isSyncDecisionRequiredError(error: unknown): error is SyncDecisionRequiredError {
-	return error instanceof SyncDecisionRequiredError;
+  return error instanceof SyncDecisionRequiredError;
 }
 
 export function errorMessage(error: unknown) {
-	const message = error instanceof Error ? error.message : String(error);
-	// biome-ignore lint/suspicious/noControlCharactersInRegex: Escape untrusted terminal controls.
-	return message.replace(/[\u0000-\u001f\u007f-\u009f]/gu, "?");
+  const message = error instanceof Error ? error.message : String(error);
+  // biome-ignore lint/suspicious/noControlCharactersInRegex: Escape untrusted terminal controls.
+  return message.replace(/[\u0000-\u001f\u007f-\u009f]/gu, "?");
 }

package/src/sync/sync-inspection.ts

@@ -9,43 +9,39 @@ import { hasLocalChanges, remoteChangedSinceState } from "./sync-state.js";
 
 /** Advisory baseline-relative observation, never an authorization to transfer files. */
 export async function inspectSync(
-	config: AnySyncConfig,
-	options: SnapshotOptions,
-	signal?: AbortSignal,
-	factory: SyncBackendFactory = createSyncBackend,
+  config: AnySyncConfig,
+  options: SnapshotOptions,
+  signal?: AbortSignal,
+  factory: SyncBackendFactory = createSyncBackend,
 ) {
-	throwIfAborted(signal);
-	const backend = await factory(config);
-	throwIfAborted(signal);
-	const local = await createSnapshot(config.snapshotIdentity, { ...options, signal });
-	throwIfAborted(signal);
-	const state = await readStateForConfig(config);
-	throwIfAborted(signal);
-	const head = await backend.readHead(signal);
-	throwIfAborted(signal);
-	return {
-		head,
-		selectionState: head
-			? inspectRemoteSelection(config.include, { selection: head.selection, files: [] })
-			: undefined,
-		localFiles: local.files.length,
-		localChanged: hasLocalChanges(local, state, config),
-		remoteChanged: remoteChangedSinceState(head, state, config, (left, right) =>
-			backend.sameRevision(left, right),
-		),
-		firstSync: !state.lastAppliedSnapshot,
-		emptyInclude: config.include.length === 0,
-		stateIdentity: syncStateFingerprint(state),
-		destination: backend.destination,
-		capability: backend.capability,
-	};
+  throwIfAborted(signal);
+  const backend = await factory(config);
+  throwIfAborted(signal);
+  const local = await createSnapshot(config.snapshotIdentity, { ...options, signal });
+  throwIfAborted(signal);
+  const state = await readStateForConfig(config);
+  throwIfAborted(signal);
+  const head = await backend.readHead(signal);
+  throwIfAborted(signal);
+  return {
+    head,
+    selectionState: head ? inspectRemoteSelection(config.include, { selection: head.selection, files: [] }) : undefined,
+    localFiles: local.files.length,
+    localChanged: hasLocalChanges(local, state, config),
+    remoteChanged: remoteChangedSinceState(head, state, config, (left, right) => backend.sameRevision(left, right)),
+    firstSync: !state.lastAppliedSnapshot,
+    emptyInclude: config.include.length === 0,
+    stateIdentity: syncStateFingerprint(state),
+    destination: backend.destination,
+    capability: backend.capability,
+  };
 }
 
 export type SyncInspection = Awaited<ReturnType<typeof inspectSync>>;
 
 export interface StartupObservation {
-	setupName: string;
-	configIdentity: string;
-	checkedAt: string;
-	inspection: SyncInspection;
+  setupName: string;
+  configIdentity: string;
+  checkedAt: string;
+  inspection: SyncInspection;
 }