🐛 Unblock CLI dependency update PR

Carry forward the Dependabot lockfile update and fix the Biome 2.4.13 lint findings so the dependency upgrade can merge cleanly.

Author: Robdel12

package-lock.json

@@ -173,7 +173,7 @@ export function buildTokenData(response, existingUser = null) {
  * @returns {{ valid: boolean, error: Error|null }}
  */
 export function validateTokens(auth, requiredField = 'accessToken') {
-  if (!auth || !auth[requiredField]) {
+  if (!auth?.[requiredField]) {
     let message =
       requiredField === 'refreshToken'
         ? 'No refresh token found. Please run "vizzly login" first.'

src/auth/core.js

@@ -28,7 +28,7 @@ export async function logoutCommand(options = {}, globalOptions = {}) {
     // Check if user is logged in
     let auth = await getAuthTokens();
 
-    if (!auth || !auth.accessToken) {
+    if (!auth?.accessToken) {
       if (globalOptions.json) {
         output.data({ loggedOut: false, reason: 'not_logged_in' });
       } else {

src/commands/logout.js

@@ -28,7 +28,7 @@ export async function whoamiCommand(options = {}, globalOptions = {}) {
     // Check if user is logged in
     let auth = await getAuthTokens();
 
-    if (!auth || !auth.accessToken) {
+    if (!auth?.accessToken) {
       if (globalOptions.json) {
         output.data({ authenticated: false });
       } else {

src/commands/whoami.js

@@ -408,7 +408,7 @@ export function serializeConfig(config, filepath) {
  * @returns {{ config: Object|null, filepath: string|null }}
  */
 export function extractCosmiconfigResult(result) {
-  if (!result || !result.config) {
+  if (!result?.config) {
     return { config: null, filepath: null };
   }
 

src/config/core.js

@@ -50,7 +50,7 @@ export function ScreenshotDisplay({
       const container = screenshotContainerRef.current;
       const image = container?.querySelector('img');
 
-      if (!container || !image || !image.naturalWidth) return;
+      if (!container || !image?.naturalWidth) return;
 
       // Always track natural size for overflow calculations
       setNaturalImageSize({

src/reporter/src/components/comparison/screenshot-display.jsx

@@ -523,7 +523,7 @@ function Header({ timestamp }) {
 }
 
 export default function StaticReportView({ reportData }) {
-  if (!reportData || !reportData.comparisons) {
+  if (!reportData?.comparisons) {
     return (
       <div className="min-h-screen bg-slate-900 text-white flex items-center justify-center">
         <div className="text-center">

src/reporter/src/components/static-report-view.jsx

@@ -231,7 +231,7 @@ export class VizzlySDK extends EventEmitter {
    * @throws {VizzlyError} When file cannot be read due to permissions or I/O errors
    */
   async screenshot(name, imageBuffer, options = {}) {
-    if (!this.server || !this.server.isRunning()) {
+    if (!this.server?.isRunning()) {
       throw new VizzlyError(
         'Server not running. Call start() first.',
         'SERVER_NOT_RUNNING'

src/sdk/index.js

@@ -20,11 +20,7 @@ export function calculateHotspotCoverage(diffClusters, hotspotAnalysis) {
     return { coverage: 0, linesInHotspots: 0, totalLines: 0 };
   }
 
-  if (
-    !hotspotAnalysis ||
-    !hotspotAnalysis.regions ||
-    hotspotAnalysis.regions.length === 0
-  ) {
+  if (!hotspotAnalysis?.regions || hotspotAnalysis.regions.length === 0) {
     return { coverage: 0, linesInHotspots: 0, totalLines: 0 };
   }
 

src/tdd/core/hotspot-coverage.js

@@ -26,7 +26,7 @@ export async function downloadHotspots(options) {
   try {
     let response = await api.getHotspots(screenshotNames);
 
-    if (!response || !response.hotspots) {
+    if (!response?.hotspots) {
       return { success: false, error: 'API returned no hotspot data' };
     }