npm - @bobfrankston/gcards - Versions diffs - 0.1.21 → 0.1.22 - Mend

@bobfrankston/gcards 0.1.21 → 0.1.22

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (6) hide show

package/gcards.ts CHANGED Viewed

@@ -8,6 +8,7 @@ import fs from 'fs';
 import fp from 'fs/promises';
 import path from 'path';
 import crypto from 'crypto';
+import { parse as parseJsonc } from 'jsonc-parser';
 import { parseArgs, showUsage, showHelp } from './glib/parsecli.ts';
 import { authenticateOAuth } from '../../../projects/oauth/oauthsupport/index.ts';
 import type { GooglePerson, GoogleConnectionsResponse } from './glib/types.ts';
@@ -160,7 +161,11 @@ interface DeletedIndex {
 function loadDeleted(paths: UserPaths): DeletedIndex {
     if (fs.existsSync(paths.deletedFile)) {
-        return JSON.parse(fs.readFileSync(paths.deletedFile, 'utf-8'));
+        try {
+            return parseJsonc(fs.readFileSync(paths.deletedFile, 'utf-8'));
+        } catch (e: any) {
+            throw new Error(`Failed to parse ${paths.deletedFile}: ${e.message}`);
+        }
     }
     return { deleted: {} };
 }
@@ -194,7 +199,11 @@ interface PhotosIndex {
 function loadPhotos(paths: UserPaths): PhotosIndex {
     if (fs.existsSync(paths.photosFile)) {
-        return JSON.parse(fs.readFileSync(paths.photosFile, 'utf-8'));
+        try {
+            return parseJsonc(fs.readFileSync(paths.photosFile, 'utf-8'));
+        } catch (e: any) {
+            throw new Error(`Failed to parse ${paths.photosFile}: ${e.message}`);
+        }
     }
     return { photos: {} };
 }
@@ -208,12 +217,55 @@ function savePhotos(paths: UserPaths, photosIndex: PhotosIndex): void {
         sorted[key] = value;
     }
     photosIndex.photos = sorted;
+    // Save JSON data file
     fs.writeFileSync(paths.photosFile, JSON.stringify(photosIndex, null, 2));
+    // Generate HTML view
+    const html = `<!DOCTYPE html>
+<html>
+<head>
+    <meta charset="UTF-8">
+    <title>Saved Photos from Deleted Contacts</title>
+    <style>
+        body { font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, sans-serif; margin: 20px; }
+        h1 { color: #333; }
+        table { border-collapse: collapse; width: 100%; }
+        th, td { border: 1px solid #ddd; padding: 8px; text-align: left; vertical-align: middle; }
+        th { background: #f5f5f5; }
+        tr:hover { background: #f9f9f9; }
+        img { width: 100px; height: 100px; object-fit: cover; border-radius: 4px; }
+        a { color: #1a73e8; text-decoration: none; }
+        a:hover { text-decoration: underline; }
+        .deleted { font-size: 12px; color: #666; }
+    </style>
+</head>
+<body>
+    <h1>Saved Photos from Deleted Contacts (${entries.length})</h1>
+    <table>
+        <tr><th>Photo</th><th>Name</th></tr>
+${entries.map(([resourceName, entry]) => {
+    const id = resourceName.replace('people/', '');
+    const contactUrl = `https://contacts.google.com/person/${id}`;
+    const photoUrl = entry.photos[0]?.url || '';
+    return `        <tr>
+            <td>${photoUrl ? `<img src="${photoUrl}" alt="${entry.displayName}" loading="lazy">` : '(no photo)'}</td>
+            <td><a href="${contactUrl}" target="_blank">${entry.displayName}</a><br><span class="deleted">Deleted: ${entry.deletedAt?.split('T')[0] || 'unknown'}</span></td>
+        </tr>`;
+}).join('\n')}
+    </table>
+</body>
+</html>`;
+    fs.writeFileSync(paths.photosHtmlFile, html);
 }
 function loadDeleteQueue(paths: UserPaths): DeleteQueue {
     if (fs.existsSync(paths.deleteQueueFile)) {
-        return JSON.parse(fs.readFileSync(paths.deleteQueueFile, 'utf-8'));
+        try {
+            return parseJsonc(fs.readFileSync(paths.deleteQueueFile, 'utf-8'));
+        } catch (e: any) {
+            throw new Error(`Failed to parse ${paths.deleteQueueFile}: ${e.message}`);
+        }
     }
     return { updatedAt: '', entries: [] };
 }
@@ -249,8 +301,12 @@ function generateGuid(): string {
 function loadSyncToken(paths: UserPaths): string {
     if (fs.existsSync(paths.syncTokenFile)) {
-        const data = JSON.parse(fs.readFileSync(paths.syncTokenFile, 'utf-8'));
-        return data.syncToken || null;
+        try {
+            const data = parseJsonc(fs.readFileSync(paths.syncTokenFile, 'utf-8'));
+            return data.syncToken || null;
+        } catch (e: any) {
+            throw new Error(`Failed to parse ${paths.syncTokenFile}: ${e.message}`);
+        }
     }
     return null;
 }
@@ -439,7 +495,7 @@ async function syncContacts(user: string, options: { full: boolean; verbose: boo
                         // Save photos before deleting
                         if (fs.existsSync(contactFile)) {
                             try {
-                                const contact = JSON.parse(fs.readFileSync(contactFile, 'utf-8')) as GooglePerson;
+                                const contact = parseJsonc(fs.readFileSync(contactFile, 'utf-8')) as GooglePerson;
                                 const photos = extractNonDefaultPhotos(contact);
                                 if (photos.length > 0) {
                                     photosIndex.photos[person.resourceName] = {
@@ -461,12 +517,16 @@ async function syncContacts(user: string, options: { full: boolean; verbose: boo
                             if (!fs.existsSync(paths.deletedDir)) {
                                 fs.mkdirSync(paths.deletedDir, { recursive: true });
                             }
-                            const contactData = JSON.parse(fs.readFileSync(contactFile, 'utf-8'));
-                            contactData._deletedAt = new Date().toISOString();
-                            contactData._delete = 'server';
-                            const destPath = path.join(paths.deletedDir, `${id}.json`);
-                            fs.writeFileSync(destPath, JSON.stringify(contactData, null, 2));
-                            fs.unlinkSync(contactFile);
+                            try {
+                                const contactData = parseJsonc(fs.readFileSync(contactFile, 'utf-8'));
+                                contactData._deletedAt = new Date().toISOString();
+                                contactData._delete = 'server';
+                                const destPath = path.join(paths.deletedDir, `${id}.json`);
+                                fs.writeFileSync(destPath, JSON.stringify(contactData, null, 2));
+                                fs.unlinkSync(contactFile);
+                            } catch (e: any) {
+                                console.error(`\n[PARSE ERROR] ${id}.json: ${e.message}`);
+                            }
                         }
                         deleted++;
@@ -568,7 +628,11 @@ async function syncContacts(user: string, options: { full: boolean; verbose: boo
 function loadStatus(paths: UserPaths): PushStatus {
     if (fs.existsSync(paths.statusFile)) {
-        return JSON.parse(fs.readFileSync(paths.statusFile, 'utf-8'));
+        try {
+            return parseJsonc(fs.readFileSync(paths.statusFile, 'utf-8'));
+        } catch (e: any) {
+            throw new Error(`Failed to parse ${paths.statusFile}: ${e.message}`);
+        }
     }
     return { lastPush: '' };
 }
@@ -628,7 +692,7 @@ async function findPendingChanges(paths: UserPaths, logger: FileLogger, since?:
             checked++;
             let content: GooglePerson & { _delete?: boolean };
             try {
-                content = JSON.parse(await fp.readFile(filePath, 'utf-8'));
+                content = parseJsonc(await fp.readFile(filePath, 'utf-8'));
             } catch (e: any) {
                 const errMsg = `[PARSE ERROR] ${file}: ${e.message}`;
                 parseErrors.push(errMsg);
@@ -666,7 +730,7 @@ async function findPendingChanges(paths: UserPaths, logger: FileLogger, since?:
             const filePath = path.join(paths.toDeleteDir, file);
             let content: GooglePerson & { _delete?: string | boolean };
             try {
-                content = JSON.parse(await fp.readFile(filePath, 'utf-8'));
+                content = parseJsonc(await fp.readFile(filePath, 'utf-8'));
             } catch (e: any) {
                 const errMsg = `[PARSE ERROR] ${file}: ${e.message}`;
                 parseErrors.push(errMsg);
@@ -692,7 +756,7 @@ async function findPendingChanges(paths: UserPaths, logger: FileLogger, since?:
             const filePath = path.join(paths.toAddDir, file);
             let content: GooglePerson;
             try {
-                content = JSON.parse(await fp.readFile(filePath, 'utf-8'));
+                content = parseJsonc(await fp.readFile(filePath, 'utf-8'));
             } catch (e: any) {
                 const errMsg = `[PARSE ERROR] ${file}: ${e.message}`;
                 parseErrors.push(errMsg);
@@ -962,7 +1026,7 @@ async function pushContacts(user: string, options: { yes: boolean; verbose: bool
         processed++;
         try {
             if (change.type === 'add') {
-                const content = JSON.parse(fs.readFileSync(change.filePath, 'utf-8')) as GooglePerson;
+                const content = parseJsonc(fs.readFileSync(change.filePath, 'utf-8')) as GooglePerson;
                 process.stdout.write(`${ts()} [${String(processed).padStart(5)}/${total}] Creating ${change.displayName}...`);
                 const created = await createContactOnGoogle(content);
@@ -996,7 +1060,7 @@ async function pushContacts(user: string, options: { yes: boolean; verbose: bool
                 console.log(` done (${created.resourceName})`);
                 successCount++;
             } else if (change.type === 'update') {
-                const content = JSON.parse(fs.readFileSync(change.filePath, 'utf-8')) as GooglePerson;
+                const content = parseJsonc(fs.readFileSync(change.filePath, 'utf-8')) as GooglePerson;
                 process.stdout.write(`${ts()} [${String(processed).padStart(5)}/${total}] Updating ${change.displayName}...`);
                 await updateContactOnGoogle(content);
@@ -1017,7 +1081,7 @@ async function pushContacts(user: string, options: { yes: boolean; verbose: bool
                 let isForced = change._delete === 'force' || index.contacts[change.resourceName]?._delete === 'force';
                 if (fs.existsSync(fileToRead)) {
                     try {
-                        const contact = JSON.parse(fs.readFileSync(fileToRead, 'utf-8')) as GooglePerson;
+                        const contact = parseJsonc(fs.readFileSync(fileToRead, 'utf-8')) as GooglePerson;
                         contactHasPhoto = hasNonDefaultPhoto(contact);
                         contactIsStarred = isStarred(contact);
                         if ((contact as any)._delete === 'force') isForced = true;
@@ -1071,7 +1135,7 @@ async function pushContacts(user: string, options: { yes: boolean; verbose: bool
                     }
                     if (fs.existsSync(contactFile)) {
                         // Add deletion metadata to the JSON file
-                        const contactData = JSON.parse(fs.readFileSync(contactFile, 'utf-8'));
+                        const contactData = parseJsonc(fs.readFileSync(contactFile, 'utf-8'));
                         contactData._deletedAt = new Date().toISOString();
                         contactData._delete = change._delete || entry?._delete || 'unknown';
                         const destPath = path.join(paths.deletedDir, path.basename(contactFile));
@@ -1090,7 +1154,7 @@ async function pushContacts(user: string, options: { yes: boolean; verbose: bool
                 }
             }
-            await sleep(700);  // 90 writes/min limit = ~700ms between ops
+            await sleep(1500);  // Google tightened limits - need ~1.5s between ops
         } catch (error: any) {
             const fileName = path.basename(change.filePath);
             const errorMsg = `${fileName}: ${error.message}`;
@@ -1104,6 +1168,26 @@ async function pushContacts(user: string, options: { yes: boolean; verbose: bool
                 continue;
             }
+            // Check for 404 - contact deleted on server, remove locally
+            if (error.message?.includes('404') || error.message?.includes('NOT_FOUND')) {
+                console.log(`    → Contact deleted on Google. Removing local copy.`);
+                // Move to deleted folder
+                if (fs.existsSync(change.filePath)) {
+                    const id = path.basename(change.filePath, '.json');
+                    if (!fs.existsSync(paths.deletedDir)) {
+                        fs.mkdirSync(paths.deletedDir, { recursive: true });
+                    }
+                    const destPath = path.join(paths.deletedDir, `${id}.json`);
+                    fs.renameSync(change.filePath, destPath);
+                }
+                // Remove from index
+                if (index.contacts[change.resourceName]) {
+                    delete index.contacts[change.resourceName];
+                    saveIndex(paths, index);
+                }
+                continue;
+            }
             // Get AI explanation if available and error is from API
             if (error instanceof ApiError && error.payload) {
                 if (isAIAvailable()) {
@@ -1169,7 +1253,7 @@ async function pushContacts(user: string, options: { yes: boolean; verbose: bool
             let email = '';
             if (nd.filePath && fs.existsSync(nd.filePath)) {
                 try {
-                    const contact = JSON.parse(fs.readFileSync(nd.filePath, 'utf-8'));
+                    const contact = parseJsonc(fs.readFileSync(nd.filePath, 'utf-8'));
                     email = contact.emailAddresses?.[0]?.value || '';
                 } catch { /* ignore */ }
             }

package/gfix.ts CHANGED Viewed

@@ -19,7 +19,7 @@ import { mergeContacts as mergeContactData, collectSourcePhotos, type MergeEntry
 function loadDeleteQueue(paths: UserPaths): DeleteQueue {
     if (fs.existsSync(paths.deleteQueueFile)) {
-        return JSON.parse(fs.readFileSync(paths.deleteQueueFile, 'utf-8'));
+        return parseJsonc(fs.readFileSync(paths.deleteQueueFile, 'utf-8'));
     }
     return { updatedAt: '', entries: [] };
 }
@@ -323,7 +323,7 @@ async function runBirthdayExtract(user: string, mode: 'inspect' | 'apply'): Prom
     for (const file of files) {
         const filePath = path.join(paths.contactsDir, file);
-        const contact = JSON.parse(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
+        const contact = parseJsonc(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
         if (!contact.birthdays?.length) continue;
@@ -413,7 +413,7 @@ async function runFileAsFix(user: string, mode: 'inspect' | 'apply'): Promise<vo
     for (const file of files) {
         const filePath = path.join(paths.contactsDir, file);
-        const contact = JSON.parse(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
+        const contact = parseJsonc(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
         const name = contact.names?.[0];
         const givenName = name?.givenName || '';
@@ -1132,7 +1132,7 @@ async function runNamesFix(user: string, mode: 'inspect' | 'apply'): Promise<voi
                 const dst = path.join(paths.toDeleteDir, entry.file);
                 fs.renameSync(src, dst);
             }
-            const existing = fs.existsSync(emailOnlyFile) ? JSON.parse(fs.readFileSync(emailOnlyFile, 'utf-8')) : [];
+            const existing = fs.existsSync(emailOnlyFile) ? parseJsonc(fs.readFileSync(emailOnlyFile, 'utf-8')) : [];
             existing.push(...emailOnly.map(e => ({ file: e.file, resourceName: e.resourceName, email: e.email, movedAt: new Date().toISOString() })));
             fs.writeFileSync(emailOnlyFile, JSON.stringify(existing, null, 2));
         }
@@ -1143,7 +1143,7 @@ async function runNamesFix(user: string, mode: 'inspect' | 'apply'): Promise<voi
                 const dst = path.join(paths.toDeleteDir, entry.file);
                 fs.renameSync(src, dst);
             }
-            const existing = fs.existsSync(phoneOnlyFile) ? JSON.parse(fs.readFileSync(phoneOnlyFile, 'utf-8')) : [];
+            const existing = fs.existsSync(phoneOnlyFile) ? parseJsonc(fs.readFileSync(phoneOnlyFile, 'utf-8')) : [];
             existing.push(...phoneOnly.map(e => ({ file: e.file, resourceName: e.resourceName, phone: e.phone, movedAt: new Date().toISOString() })));
             fs.writeFileSync(phoneOnlyFile, JSON.stringify(existing, null, 2));
         }
@@ -1216,7 +1216,7 @@ async function runFix(user: string, mode: 'inspect' | 'apply'): Promise<void> {
     for (const file of files) {
         const filePath = path.join(paths.contactsDir, file);
-        const contact = JSON.parse(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
+        const contact = parseJsonc(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
         const changes = processContact(contact);
         if (changes.length > 0) {
@@ -1303,7 +1303,13 @@ async function runUndup(user: string): Promise<void> {
     for (const file of files) {
         const filePath = path.join(paths.contactsDir, file);
-        const contact = JSON.parse(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
+        let contact: GooglePerson;
+        try {
+            contact = parseJsonc(fs.readFileSync(filePath, 'utf-8'));
+        } catch (e: any) {
+            console.error(`[PARSE ERROR] ${file}: ${e.message}`);
+            continue;
+        }
         const name = contact.names?.[0];
         if (!name) continue;
@@ -1438,14 +1444,14 @@ async function runMerge(user: string, limit: number): Promise<void> {
     const paths = getUserPaths(user);
     const mergerPath = path.join(paths.userDir, 'merger.json');
     const mergedPath = path.join(paths.userDir, 'merged.json');
-    const photosPath = path.join(paths.userDir, 'photos.json');
+    const photosPath = paths.photosFile;  // _photos.json
     if (!fs.existsSync(mergerPath)) {
         console.error(`No merger.json found. Run 'gfix undup -u ${user}' first.`);
         process.exit(1);
     }
-    let entries: MergeEntry[] = JSON.parse(fs.readFileSync(mergerPath, 'utf-8'));
+    let entries: MergeEntry[] = parseJsonc(fs.readFileSync(mergerPath, 'utf-8'));
     if (entries.length === 0) {
         console.log('No entries in merger.json');
@@ -1478,7 +1484,8 @@ async function runMerge(user: string, limit: number): Promise<void> {
     // Load existing files
     let savedPhotos: PhotoEntry[] = [];
     if (fs.existsSync(photosPath)) {
-        savedPhotos = JSON.parse(fs.readFileSync(photosPath, 'utf-8'));
+        const parsed = parseJsonc(fs.readFileSync(photosPath, 'utf-8'));
+        savedPhotos = Array.isArray(parsed) ? parsed : [];
     }
     const deleteQueue = loadDeleteQueue(paths);
@@ -1489,7 +1496,8 @@ async function runMerge(user: string, limit: number): Promise<void> {
         const content = fs.readFileSync(mergedPath, 'utf-8').trim();
         if (content) {
             try {
-                mergedEntries = JSON.parse(content);
+                const parsed = parseJsonc(content);
+                mergedEntries = Array.isArray(parsed) ? parsed : [];
             } catch { /* ignore parse errors, start fresh */ }
         }
     }
@@ -1509,7 +1517,7 @@ async function runMerge(user: string, limit: number): Promise<void> {
                 const id = rn.replace('people/', '');
                 const filePath = path.join(paths.contactsDir, `${id}.json`);
                 if (fs.existsSync(filePath)) {
-                    contacts.push(JSON.parse(fs.readFileSync(filePath, 'utf-8')));
+                    contacts.push(parseJsonc(fs.readFileSync(filePath, 'utf-8')));
                 } else {
                     throw new Error(`Contact file not found: ${filePath}`);
                 }
@@ -1586,7 +1594,7 @@ async function runMerge(user: string, limit: number): Promise<void> {
             const id = rn.replace('people/', '');
             const filePath = path.join(paths.contactsDir, `${id}.json`);
             if (fs.existsSync(filePath)) {
-                const contact = JSON.parse(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
+                const contact = parseJsonc(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
                 const displayName = contact.names?.[0]?.displayName || entry.name;
                 deleteQueue.entries.push({
                     resourceName: rn,
@@ -1605,9 +1613,47 @@ async function runMerge(user: string, limit: number): Promise<void> {
         mergedEntries.push(entry);
     }
-    // Save photos.json
+    // Save photos (JSON data + HTML view)
     if (savedPhotos.length > 0) {
         fs.writeFileSync(photosPath, JSON.stringify(savedPhotos, null, 2));
+        // Generate HTML view
+        const htmlPath = paths.photosHtmlFile;
+        const html = `<!DOCTYPE html>
+<html>
+<head>
+    <meta charset="UTF-8">
+    <title>Saved Photos from Merged Contacts</title>
+    <style>
+        body { font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, sans-serif; margin: 20px; }
+        h1 { color: #333; }
+        table { border-collapse: collapse; width: 100%; }
+        th, td { border: 1px solid #ddd; padding: 8px; text-align: left; vertical-align: middle; }
+        th { background: #f5f5f5; }
+        tr:hover { background: #f9f9f9; }
+        img { width: 100px; height: 100px; object-fit: cover; border-radius: 4px; margin: 2px; }
+        a { color: #1a73e8; text-decoration: none; }
+        a:hover { text-decoration: underline; }
+    </style>
+</head>
+<body>
+    <h1>Saved Photos from Merged Contacts (${savedPhotos.length})</h1>
+    <table>
+        <tr><th>Photos</th><th>Name</th></tr>
+${savedPhotos.map(entry => {
+    const id = entry.contactId.replace('people/', '');
+    const contactUrl = `https://contacts.google.com/person/${id}`;
+    const photoImgs = entry.photos.map(url => `<img src="${url}" alt="${entry.name}" loading="lazy">`).join('');
+    return `        <tr>
+            <td>${photoImgs || '(no photo)'}</td>
+            <td><a href="${contactUrl}" target="_blank">${entry.name}</a></td>
+        </tr>`;
+}).join('\n')}
+    </table>
+</body>
+</html>`;
+        fs.writeFileSync(htmlPath, html);
+        console.log(`\nSaved ${savedPhotos.length} photo entries to ${htmlPath}`);
     }
     // Save merged.json (history of processed entries)
@@ -1620,7 +1666,7 @@ async function runMerge(user: string, limit: number): Promise<void> {
     fs.writeFileSync(paths.indexFile, JSON.stringify(index, null, 2));
     // Remove processed entries from merger.json
-    const remainingEntries = JSON.parse(fs.readFileSync(mergerPath, 'utf-8')) as MergeEntry[];
+    const remainingEntries = parseJsonc(fs.readFileSync(mergerPath, 'utf-8')) as MergeEntry[];
     const processedNames = new Set(entries.map(e => e.name));
     const newMerger = remainingEntries.filter(e => !processedNames.has(e.name));
     fs.writeFileSync(mergerPath, JSON.stringify(newMerger, null, 2));
@@ -1679,7 +1725,7 @@ async function runPhotos(user: string): Promise<void> {
     for (const file of files) {
         const filePath = path.join(paths.contactsDir, file);
-        const contact = JSON.parse(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
+        const contact = parseJsonc(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
         const photoUrl = getPhotoUrl(contact);
         if (!photoUrl) continue;
@@ -1766,7 +1812,7 @@ async function runPhotoCompare(user: string): Promise<void> {
     // First collect all contacts with photos
     for (const file of files) {
         const filePath = path.join(paths.contactsDir, file);
-        const contact = JSON.parse(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
+        const contact = parseJsonc(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
         const photoUrl = getPhotoUrl(contact);
         if (!photoUrl) continue;
@@ -1867,7 +1913,7 @@ function findContactById(paths: UserPaths, id: string): { contact: GooglePerson;
     const contactPath = path.join(paths.contactsDir, `${cleanId}.json`);
     if (fs.existsSync(contactPath)) {
         return {
-            contact: JSON.parse(fs.readFileSync(contactPath, 'utf-8')),
+            contact: parseJsonc(fs.readFileSync(contactPath, 'utf-8')),
             filePath: contactPath,
             source: 'contacts'
         };
@@ -1877,7 +1923,7 @@ function findContactById(paths: UserPaths, id: string): { contact: GooglePerson;
     const deletedPath = path.join(paths.deletedDir, `${cleanId}.json`);
     if (fs.existsSync(deletedPath)) {
         return {
-            contact: JSON.parse(fs.readFileSync(deletedPath, 'utf-8')),
+            contact: parseJsonc(fs.readFileSync(deletedPath, 'utf-8')),
             filePath: deletedPath,
             source: 'deleted'
         };
@@ -1906,7 +1952,7 @@ function searchContacts(paths: UserPaths, pattern: string): ContactMatch[] {
     if (fs.existsSync(paths.contactsDir)) {
         for (const file of fs.readdirSync(paths.contactsDir).filter(f => f.endsWith('.json'))) {
             const filePath = path.join(paths.contactsDir, file);
-            const contact = JSON.parse(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
+            const contact = parseJsonc(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
             if (matchesContact(contact, regex)) {
                 matches.push({
                     id: file.replace(/\.json$/, ''),
@@ -1923,7 +1969,7 @@ function searchContacts(paths: UserPaths, pattern: string): ContactMatch[] {
     if (fs.existsSync(paths.deletedDir)) {
         for (const file of fs.readdirSync(paths.deletedDir).filter(f => f.endsWith('.json'))) {
             const filePath = path.join(paths.deletedDir, file);
-            const contact = JSON.parse(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
+            const contact = parseJsonc(fs.readFileSync(filePath, 'utf-8')) as GooglePerson;
             if (matchesContact(contact, regex)) {
                 matches.push({
                     id: file.replace(/\.json$/, ''),
@@ -2167,7 +2213,7 @@ async function runReset(user: string): Promise<void> {
     // Clear from index.json
     if (fs.existsSync(paths.indexFile)) {
-        const index = JSON.parse(fs.readFileSync(paths.indexFile, 'utf-8'));
+        const index = parseJsonc(fs.readFileSync(paths.indexFile, 'utf-8'));
         for (const entry of Object.values(index.contacts) as any[]) {
             if (entry._delete) {
                 delete entry._delete;
@@ -2183,7 +2229,7 @@ async function runReset(user: string): Promise<void> {
         for (const file of files) {
             const filePath = path.join(paths.contactsDir, file);
             const stat = fs.statSync(filePath);
-            const content = JSON.parse(fs.readFileSync(filePath, 'utf-8'));
+            const content = parseJsonc(fs.readFileSync(filePath, 'utf-8'));
             if (content._delete || content._deleted) {
                 delete content._delete;
                 delete content._deleted;
@@ -2197,7 +2243,7 @@ async function runReset(user: string): Promise<void> {
     // Clear _delete.json queue
     if (fs.existsSync(paths.deleteQueueFile)) {
-        const queue = JSON.parse(fs.readFileSync(paths.deleteQueueFile, 'utf-8'));
+        const queue = parseJsonc(fs.readFileSync(paths.deleteQueueFile, 'utf-8'));
         queueCleared = queue.entries?.length || 0;
         fs.unlinkSync(paths.deleteQueueFile);
     }

package/glib/gctypes.ts CHANGED Viewed

@@ -99,7 +99,8 @@ export interface UserPaths {
     indexFile: string;
     deletedFile: string;       /** Deleted contacts index (deleted.json) */
     deleteQueueFile: string;   /** Pending deletions (_delete.json) */
-    photosFile: string;        /** Photos from deleted contacts (photos.json) */
+    photosFile: string;        /** Photos data store (_photos.json) */
+    photosHtmlFile: string;    /** Photos HTML view (photos.html) */
     statusFile: string;
     syncTokenFile: string;
     tokenFile: string;

package/glib/gmerge.ts CHANGED Viewed

@@ -261,8 +261,8 @@ export function collectSourcePhotos(sources: GooglePerson[]): string[] {
     const urls: string[] = [];
     for (const source of sources) {
         for (const photo of source.photos || []) {
-            // Only collect custom photos (default: false means custom)
-            if (photo.url && photo.default === false) {
+            // Only collect custom photos (!default means custom - undefined or false)
+            if (photo.url && !photo.default) {
                 urls.push(photo.url);
             }
         }

package/glib/gutils.ts CHANGED Viewed

@@ -64,7 +64,8 @@ export function getUserPaths(user: string): UserPaths {
         indexFile: path.join(userDir, 'index.json'),
         deletedFile: path.join(userDir, 'deleted.json'),
         deleteQueueFile: path.join(userDir, '_delete.json'),
-        photosFile: path.join(userDir, 'photos.json'),
+        photosFile: path.join(userDir, '_photos.json'),
+        photosHtmlFile: path.join(userDir, 'photos.html'),
         statusFile: path.join(userDir, 'status.json'),
         syncTokenFile: path.join(userDir, 'sync-token.json'),
         tokenFile: path.join(userDir, 'token.json'),

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/gcards",
-  "version": "0.1.21",
+  "version": "0.1.22",
   "description": "Google Contacts cleanup and management tool",
   "type": "module",
   "main": "gcards.ts",