refactor: use destructured defaults over per-field ?? fallbacks

packages/annotation/src/useSubmitOnCmdEnter.test.ts

@@ -50,10 +50,11 @@ function buildKeyEvent(init: {
   ctrlKey?: boolean;
   preventDefault?: () => void;
 }): KeyboardEvent<HTMLTextAreaElement> {
+  const { key, metaKey = false, ctrlKey = false, preventDefault = () => {} } = init;
   return {
-    key: init.key,
-    metaKey: init.metaKey ?? false,
-    ctrlKey: init.ctrlKey ?? false,
-    preventDefault: init.preventDefault ?? (() => {}),
+    key,
+    metaKey,
+    ctrlKey,
+    preventDefault,
   } as unknown as KeyboardEvent<HTMLTextAreaElement>;
 }

packages/cli/src/testHelpers/annotationFakes.ts

@@ -33,8 +33,8 @@ export function createAnnotationDependencies(
     result?: Promise<AnnotationSubmission>;
   } = {},
 ): TrackedAnnotationDependencies {
+  const { submission = annotationSubmission.build(), result = Promise.resolve(submission) } = options;
   const payloads: AnnotationPayload[] = [];
-  const submission = options.submission ?? annotationSubmission.build();
   const sigintRegistration = createDeferred<void>();
   let closeCount = 0;
   let observedPort: number | undefined;
@@ -71,7 +71,7 @@ export function createAnnotationDependencies(
       return {
         port: 4312,
         url: 'http://localhost:4312',
-        result: options.result ?? Promise.resolve(submission),
+        result,
         close: () => {
           closeCount += 1;
           return Promise.resolve();

packages/cli/src/updater/UpdaterImpl.ts

@@ -58,14 +58,13 @@ export class UpdaterImpl {
   constructor(private readonly deps: UpdaterImplDeps) {}
 
   async checkForUpdate(options: CheckForUpdateOptions = {}): Promise<UpdateNotice | null> {
-    const { buildInfo, env, clock, logger } = this.deps;
+    const { buildInfo, env, clock, logger, ttl = Temporal.Duration.from({ hours: DEFAULT_TTL_HOURS }) } = this.deps;
     const { forceRefresh = false } = options;
 
     if (buildInfo.version === DEV_BUILD_VERSION) return null;
     if (env.CONTEXTBRIDGE_UPDATE_CHECK_DISABLED) return null;
 
     const now = clock();
-    const ttl = this.deps.ttl ?? Temporal.Duration.from({ hours: DEFAULT_TTL_HOURS });
 
     if (!forceRefresh) {
       const cached = this.readCache(buildInfo.channel, now, ttl);