pal-explorer-cli 0.4.11 → 0.4.13

package/lib/commands/cloud-backup.js

@@ -1,231 +1,231 @@
-import chalk from 'chalk';
-
-const VALID_PROVIDERS = ['s3', 'azure', 'gcs', 'b2'];
-
-export default function cloudBackupCommand(program) {
-  const cmd = program
-    .command('cloud-backup')
-    .description('encrypted cloud backup (Pro)')
-    .addHelpText('after', `
-Examples:
-  $ pe cloud-backup configure --provider s3 --bucket my-backups --region us-east-1
-  $ pe cloud-backup create                   Create a backup now
-  $ pe cloud-backup list                     List available backups
-  $ pe cloud-backup restore <id>             Restore from a backup
-  $ pe cloud-backup delete <id>              Delete a backup
-  $ pe cloud-backup status                   Show backup status
-`)
-    .action(() => { cmd.outputHelp(); });
-
-  cmd
-    .command('configure')
-    .description('set cloud backup provider')
-    .requiredOption('--provider <provider>', 'cloud provider (s3|azure|gcs|b2)')
-    .option('--bucket <name>', 'bucket name')
-    .option('--region <region>', 'AWS region (for S3)')
-    .option('--access-key <key>', 'access key')
-    .option('--secret-key <key>', 'secret key')
-    .option('--prefix <prefix>', 'key prefix')
-    .option('--encrypt', 'enable encryption (default)', true)
-    .action(async (opts) => {
-      try {
-        if (!VALID_PROVIDERS.includes(opts.provider)) {
-          console.log(chalk.red(`Invalid provider. Must be one of: ${VALID_PROVIDERS.join(', ')}`));
-          process.exitCode = 1;
-          return;
-        }
-
-        const extConfig = (await import('../utils/config.js')).default;
-        const existing = extConfig.get('ext.backup-cloud') || {};
-        const config = {
-          ...existing,
-          provider: opts.provider,
-          encryptBackups: opts.encrypt !== false,
-        };
-        if (opts.bucket) config.bucket = opts.bucket;
-        if (opts.region) config.region = opts.region;
-        if (opts.accessKey) config.accessKey = opts.accessKey;
-        if (opts.secretKey) {
-          try {
-            const keytar = (await import('keytar')).default;
-            await keytar.setPassword('palexplorer', 'cloudBackup.secretKey', opts.secretKey);
-          } catch {
-            config.secretKey = opts.secretKey;
-          }
-        }
-        if (opts.prefix) config.prefix = opts.prefix;
-
-        extConfig.set('ext.backup-cloud', config);
-        console.log(chalk.green(`✔ Cloud backup configured: ${opts.provider}`));
-        if (config.bucket) console.log(`  Bucket: ${chalk.white(config.bucket)}`);
-        if (config.region) console.log(`  Region: ${chalk.white(config.region)}`);
-        if (config.prefix) console.log(`  Prefix: ${chalk.white(config.prefix)}`);
-        console.log(`  Encryption: ${config.encryptBackups ? chalk.green('enabled') : chalk.yellow('disabled')}`);
-      } catch (err) {
-        console.log(chalk.red(`Configure failed: ${err.message}`));
-        process.exitCode = 1;
-      }
-    });
-
-  cmd
-    .command('create')
-    .description('create a backup now')
-    .action(async () => {
-      try {
-        const extConfig = (await import('../utils/config.js')).default;
-        const config = extConfig.get('ext.backup-cloud') || {};
-        if (!config.provider) {
-          console.log(chalk.red('No provider configured. Run: pe cloud-backup configure --provider <provider>'));
-          process.exitCode = 1;
-          return;
-        }
-
-        const backupId = Date.now().toString(36) + Math.random().toString(36).slice(2, 6);
-        const backup = {
-          id: backupId,
-          provider: config.provider,
-          bucket: config.bucket,
-          encrypted: config.encryptBackups !== false,
-          createdAt: new Date().toISOString(),
-          size: 0,
-          status: 'completed',
-        };
-
-        console.log(chalk.cyan(`Creating backup to ${config.provider}://${config.bucket || 'default'}...`));
-
-        const store = extConfig.get('ext_store.backup-cloud') || {};
-        const history = store.backupHistory || [];
-        history.push(backup);
-        store.backupHistory = history;
-        store.lastBackup = backup;
-        extConfig.set('ext_store.backup-cloud', store);
-
-        console.log(chalk.green(`✔ Backup created: ${backupId}`));
-        console.log(`  Provider: ${chalk.white(config.provider)}`);
-        console.log(`  Encrypted: ${backup.encrypted ? chalk.green('yes') : chalk.yellow('no')}`);
-        console.log(`  Time: ${chalk.dim(backup.createdAt)}`);
-      } catch (err) {
-        console.log(chalk.red(`Backup failed: ${err.message}`));
-        process.exitCode = 1;
-      }
-    });
-
-  cmd
-    .command('restore <id>')
-    .description('restore from a backup')
-    .action(async (id) => {
-      try {
-        const extConfig = (await import('../utils/config.js')).default;
-        const store = extConfig.get('ext_store.backup-cloud') || {};
-        const history = store.backupHistory || [];
-        const backup = history.find(b => b.id === id);
-        if (!backup) {
-          console.log(chalk.red(`Backup not found: ${id}`));
-          process.exitCode = 1;
-          return;
-        }
-
-        console.log(chalk.cyan(`Restoring from backup ${id}...`));
-        console.log(`  Provider: ${chalk.white(backup.provider)}`);
-        console.log(`  Created: ${chalk.dim(backup.createdAt)}`);
-        console.log(chalk.green('✔ Backup restored successfully.'));
-        console.log(chalk.dim('  Note: Restart palexplorer to apply restored settings.'));
-      } catch (err) {
-        console.log(chalk.red(`Restore failed: ${err.message}`));
-        process.exitCode = 1;
-      }
-    });
-
-  cmd
-    .command('list')
-    .description('list available backups')
-    .action(async () => {
-      try {
-        const extConfig = (await import('../utils/config.js')).default;
-        const store = extConfig.get('ext_store.backup-cloud') || {};
-        const history = store.backupHistory || [];
-        if (history.length === 0) {
-          console.log(chalk.dim('No backups found.'));
-          console.log(chalk.dim('  pe cloud-backup create'));
-          return;
-        }
-        console.log(chalk.bold(`Cloud Backups (${history.length})\n`));
-        for (const b of history) {
-          const statusColor = b.status === 'completed' ? chalk.green : b.status === 'failed' ? chalk.red : chalk.yellow;
-          const encrypted = b.encrypted ? chalk.green('🔒') : '';
-          console.log(`  ${chalk.white(b.id)} ${statusColor(b.status)} ${encrypted}`);
-          console.log(`    Provider: ${chalk.dim(b.provider)}  Created: ${chalk.dim(b.createdAt)}`);
-        }
-      } catch (err) {
-        console.log(chalk.red(`List failed: ${err.message}`));
-        process.exitCode = 1;
-      }
-    });
-
-  cmd
-    .command('delete <id>')
-    .description('delete a backup')
-    .action(async (id) => {
-      try {
-        const extConfig = (await import('../utils/config.js')).default;
-        const store = extConfig.get('ext_store.backup-cloud') || {};
-        const history = store.backupHistory || [];
-        const idx = history.findIndex(b => b.id === id);
-        if (idx === -1) {
-          console.log(chalk.red(`Backup not found: ${id}`));
-          process.exitCode = 1;
-          return;
-        }
-        const removed = history.splice(idx, 1)[0];
-        store.backupHistory = history;
-        if (store.lastBackup?.id === id) {
-          store.lastBackup = history.length > 0 ? history[history.length - 1] : null;
-        }
-        extConfig.set('ext_store.backup-cloud', store);
-        console.log(chalk.green(`✔ Backup deleted: ${id}`));
-        console.log(`  Provider: ${chalk.dim(removed.provider)}  Created: ${chalk.dim(removed.createdAt)}`);
-      } catch (err) {
-        console.log(chalk.red(`Delete failed: ${err.message}`));
-        process.exitCode = 1;
-      }
-    });
-
-  cmd
-    .command('status')
-    .description('show last backup info, provider, schedule')
-    .action(async () => {
-      try {
-        const extConfig = (await import('../utils/config.js')).default;
-        const config = extConfig.get('ext.backup-cloud') || {};
-        const store = extConfig.get('ext_store.backup-cloud') || {};
-
-        if (!config.provider) {
-          console.log(chalk.dim('Cloud backup not configured.'));
-          console.log(chalk.dim('  pe cloud-backup configure --provider <provider>'));
-          return;
-        }
-
-        console.log(chalk.bold('Cloud Backup Status\n'));
-        console.log(`  Provider:   ${chalk.cyan(config.provider)}`);
-        console.log(`  Bucket:     ${chalk.white(config.bucket || 'not set')}`);
-        if (config.region) console.log(`  Region:     ${chalk.white(config.region)}`);
-        if (config.prefix) console.log(`  Prefix:     ${chalk.white(config.prefix)}`);
-        console.log(`  Encryption: ${config.encryptBackups !== false ? chalk.green('enabled') : chalk.yellow('disabled')}`);
-        console.log(`  Schedule:   ${chalk.white(config.schedule || 'manual')}`);
-
-        const history = store.backupHistory || [];
-        console.log(`  Backups:    ${chalk.white(history.length)}`);
-
-        if (store.lastBackup) {
-          console.log('');
-          console.log(chalk.cyan('  Last Backup:'));
-          console.log(`    ID:      ${chalk.white(store.lastBackup.id)}`);
-          console.log(`    Status:  ${store.lastBackup.status === 'completed' ? chalk.green('completed') : chalk.red(store.lastBackup.status)}`);
-          console.log(`    Created: ${chalk.dim(store.lastBackup.createdAt)}`);
-        }
-      } catch (err) {
-        console.log(chalk.red(`Status failed: ${err.message}`));
-        process.exitCode = 1;
-      }
-    });
-}
+import chalk from 'chalk';
+
+const VALID_PROVIDERS = ['s3', 'azure', 'gcs', 'b2'];
+
+export default function cloudBackupCommand(program) {
+  const cmd = program
+    .command('cloud-backup')
+    .description('encrypted cloud backup (Pro)')
+    .addHelpText('after', `
+Examples:
+  $ pal cloud-backup configure --provider s3 --bucket my-backups --region us-east-1
+  $ pal cloud-backup create                   Create a backup now
+  $ pal cloud-backup list                     List available backups
+  $ pal cloud-backup restore <id>             Restore from a backup
+  $ pal cloud-backup delete <id>              Delete a backup
+  $ pal cloud-backup status                   Show backup status
+`)
+    .action(() => { cmd.outputHelp(); });
+
+  cmd
+    .command('configure')
+    .description('set cloud backup provider')
+    .requiredOption('--provider <provider>', 'cloud provider (s3|azure|gcs|b2)')
+    .option('--bucket <name>', 'bucket name')
+    .option('--region <region>', 'AWS region (for S3)')
+    .option('--access-key <key>', 'access key')
+    .option('--secret-key <key>', 'secret key')
+    .option('--prefix <prefix>', 'key prefix')
+    .option('--encrypt', 'enable encryption (default)', true)
+    .action(async (opts) => {
+      try {
+        if (!VALID_PROVIDERS.includes(opts.provider)) {
+          console.log(chalk.red(`Invalid provider. Must be one of: ${VALID_PROVIDERS.join(', ')}`));
+          process.exitCode = 1;
+          return;
+        }
+
+        const extConfig = (await import('../utils/config.js')).default;
+        const existing = extConfig.get('ext.backup-cloud') || {};
+        const config = {
+          ...existing,
+          provider: opts.provider,
+          encryptBackups: opts.encrypt !== false,
+        };
+        if (opts.bucket) config.bucket = opts.bucket;
+        if (opts.region) config.region = opts.region;
+        if (opts.accessKey) config.accessKey = opts.accessKey;
+        if (opts.secretKey) {
+          try {
+            const keytar = (await import('keytar')).default;
+            await keytar.setPassword('palexplorer', 'cloudBackup.secretKey', opts.secretKey);
+          } catch {
+            config.secretKey = opts.secretKey;
+          }
+        }
+        if (opts.prefix) config.prefix = opts.prefix;
+
+        extConfig.set('ext.backup-cloud', config);
+        console.log(chalk.green(`✔ Cloud backup configured: ${opts.provider}`));
+        if (config.bucket) console.log(`  Bucket: ${chalk.white(config.bucket)}`);
+        if (config.region) console.log(`  Region: ${chalk.white(config.region)}`);
+        if (config.prefix) console.log(`  Prefix: ${chalk.white(config.prefix)}`);
+        console.log(`  Encryption: ${config.encryptBackups ? chalk.green('enabled') : chalk.yellow('disabled')}`);
+      } catch (err) {
+        console.log(chalk.red(`Configure failed: ${err.message}`));
+        process.exitCode = 1;
+      }
+    });
+
+  cmd
+    .command('create')
+    .description('create a backup now')
+    .action(async () => {
+      try {
+        const extConfig = (await import('../utils/config.js')).default;
+        const config = extConfig.get('ext.backup-cloud') || {};
+        if (!config.provider) {
+          console.log(chalk.red('No provider configured. Run: pal cloud-backup configure --provider <provider>'));
+          process.exitCode = 1;
+          return;
+        }
+
+        const backupId = Date.now().toString(36) + Math.random().toString(36).slice(2, 6);
+        const backup = {
+          id: backupId,
+          provider: config.provider,
+          bucket: config.bucket,
+          encrypted: config.encryptBackups !== false,
+          createdAt: new Date().toISOString(),
+          size: 0,
+          status: 'completed',
+        };
+
+        console.log(chalk.cyan(`Creating backup to ${config.provider}://${config.bucket || 'default'}...`));
+
+        const store = extConfig.get('ext_store.backup-cloud') || {};
+        const history = store.backupHistory || [];
+        history.push(backup);
+        store.backupHistory = history;
+        store.lastBackup = backup;
+        extConfig.set('ext_store.backup-cloud', store);
+
+        console.log(chalk.green(`✔ Backup created: ${backupId}`));
+        console.log(`  Provider: ${chalk.white(config.provider)}`);
+        console.log(`  Encrypted: ${backup.encrypted ? chalk.green('yes') : chalk.yellow('no')}`);
+        console.log(`  Time: ${chalk.dim(backup.createdAt)}`);
+      } catch (err) {
+        console.log(chalk.red(`Backup failed: ${err.message}`));
+        process.exitCode = 1;
+      }
+    });
+
+  cmd
+    .command('restore <id>')
+    .description('restore from a backup')
+    .action(async (id) => {
+      try {
+        const extConfig = (await import('../utils/config.js')).default;
+        const store = extConfig.get('ext_store.backup-cloud') || {};
+        const history = store.backupHistory || [];
+        const backup = history.find(b => b.id === id);
+        if (!backup) {
+          console.log(chalk.red(`Backup not found: ${id}`));
+          process.exitCode = 1;
+          return;
+        }
+
+        console.log(chalk.cyan(`Restoring from backup ${id}...`));
+        console.log(`  Provider: ${chalk.white(backup.provider)}`);
+        console.log(`  Created: ${chalk.dim(backup.createdAt)}`);
+        console.log(chalk.green('✔ Backup restored successfully.'));
+        console.log(chalk.dim('  Note: Restart palexplorer to apply restored settings.'));
+      } catch (err) {
+        console.log(chalk.red(`Restore failed: ${err.message}`));
+        process.exitCode = 1;
+      }
+    });
+
+  cmd
+    .command('list')
+    .description('list available backups')
+    .action(async () => {
+      try {
+        const extConfig = (await import('../utils/config.js')).default;
+        const store = extConfig.get('ext_store.backup-cloud') || {};
+        const history = store.backupHistory || [];
+        if (history.length === 0) {
+          console.log(chalk.dim('No backups found.'));
+          console.log(chalk.dim('  pal cloud-backup create'));
+          return;
+        }
+        console.log(chalk.bold(`Cloud Backups (${history.length})\n`));
+        for (const b of history) {
+          const statusColor = b.status === 'completed' ? chalk.green : b.status === 'failed' ? chalk.red : chalk.yellow;
+          const encrypted = b.encrypted ? chalk.green('🔒') : '';
+          console.log(`  ${chalk.white(b.id)} ${statusColor(b.status)} ${encrypted}`);
+          console.log(`    Provider: ${chalk.dim(b.provider)}  Created: ${chalk.dim(b.createdAt)}`);
+        }
+      } catch (err) {
+        console.log(chalk.red(`List failed: ${err.message}`));
+        process.exitCode = 1;
+      }
+    });
+
+  cmd
+    .command('delete <id>')
+    .description('delete a backup')
+    .action(async (id) => {
+      try {
+        const extConfig = (await import('../utils/config.js')).default;
+        const store = extConfig.get('ext_store.backup-cloud') || {};
+        const history = store.backupHistory || [];
+        const idx = history.findIndex(b => b.id === id);
+        if (idx === -1) {
+          console.log(chalk.red(`Backup not found: ${id}`));
+          process.exitCode = 1;
+          return;
+        }
+        const removed = history.splice(idx, 1)[0];
+        store.backupHistory = history;
+        if (store.lastBackup?.id === id) {
+          store.lastBackup = history.length > 0 ? history[history.length - 1] : null;
+        }
+        extConfig.set('ext_store.backup-cloud', store);
+        console.log(chalk.green(`✔ Backup deleted: ${id}`));
+        console.log(`  Provider: ${chalk.dim(removed.provider)}  Created: ${chalk.dim(removed.createdAt)}`);
+      } catch (err) {
+        console.log(chalk.red(`Delete failed: ${err.message}`));
+        process.exitCode = 1;
+      }
+    });
+
+  cmd
+    .command('status')
+    .description('show last backup info, provider, schedule')
+    .action(async () => {
+      try {
+        const extConfig = (await import('../utils/config.js')).default;
+        const config = extConfig.get('ext.backup-cloud') || {};
+        const store = extConfig.get('ext_store.backup-cloud') || {};
+
+        if (!config.provider) {
+          console.log(chalk.dim('Cloud backup not configured.'));
+          console.log(chalk.dim('  pal cloud-backup configure --provider <provider>'));
+          return;
+        }
+
+        console.log(chalk.bold('Cloud Backup Status\n'));
+        console.log(`  Provider:   ${chalk.cyan(config.provider)}`);
+        console.log(`  Bucket:     ${chalk.white(config.bucket || 'not set')}`);
+        if (config.region) console.log(`  Region:     ${chalk.white(config.region)}`);
+        if (config.prefix) console.log(`  Prefix:     ${chalk.white(config.prefix)}`);
+        console.log(`  Encryption: ${config.encryptBackups !== false ? chalk.green('enabled') : chalk.yellow('disabled')}`);
+        console.log(`  Schedule:   ${chalk.white(config.schedule || 'manual')}`);
+
+        const history = store.backupHistory || [];
+        console.log(`  Backups:    ${chalk.white(history.length)}`);
+
+        if (store.lastBackup) {
+          console.log('');
+          console.log(chalk.cyan('  Last Backup:'));
+          console.log(`    ID:      ${chalk.white(store.lastBackup.id)}`);
+          console.log(`    Status:  ${store.lastBackup.status === 'completed' ? chalk.green('completed') : chalk.red(store.lastBackup.status)}`);
+          console.log(`    Created: ${chalk.dim(store.lastBackup.createdAt)}`);
+        }
+      } catch (err) {
+        console.log(chalk.red(`Status failed: ${err.message}`));
+        process.exitCode = 1;
+      }
+    });
+}

package/lib/commands/comment.js

@@ -1,99 +1,99 @@
-import chalk from 'chalk';
-import config from '../utils/config.js';
-import {
-  getShareComments,
-  addShareComment,
-  deleteShareComment
-} from '../core/groups.js';
-import { checkLimit } from '../core/pro.js';
-
-export default function commentCommand(program) {
-  const cmd = program
-    .command('comment')
-    .description('manage comments on shares')
-    .addHelpText('after', `
-Examples:
-  $ pe comment list <shareId>                  List comments on a share
-  $ pe comment add <shareId> "nice files!"     Add a comment
-  $ pe comment delete <shareId> <commentId>    Delete a comment
-`);
-
-  cmd
-    .command('list <shareId>')
-    .description('list comments on a share')
-    .action((shareId) => {
-      const comments = getShareComments(shareId);
-      if (comments.length === 0) {
-        console.log(chalk.gray('No comments on this share.'));
-        return;
-      }
-      console.log('');
-      console.log(chalk.cyan(`Comments on ${shareId}:`));
-      for (const c of comments) {
-        const time = new Date(c.createdAt).toLocaleString();
-        const author = c.authorHandle ? `@${c.authorHandle}` : c.authorName || 'unknown';
-        console.log(`  ${chalk.gray(c.id.slice(0, 8))}  ${chalk.white(author)}  ${chalk.gray(time)}`);
-        console.log(`    ${c.text}`);
-      }
-      console.log('');
-    });
-
-  cmd
-    .command('add <shareId> <text>')
-    .description('add a comment to a share')
-    .action((shareId, text) => {
-      if (!text || !text.trim()) {
-        console.log(chalk.red('Error: comment text cannot be empty.'));
-        process.exitCode = 1;
-        return;
-      }
-      try {
-        const comments = getShareComments(shareId);
-        checkLimit('maxCommentsPerShare', comments.length);
-
-        const identity = config.get('identity');
-        const comment = addShareComment(shareId, {
-          authorHandle: identity?.handle || null,
-          authorName: identity?.name || 'Anonymous',
-          text,
-        });
-        console.log(chalk.green(`Comment added.`));
-        console.log(chalk.gray(`ID: ${comment.id}`));
-      } catch (err) {
-        console.log(chalk.red(err.message));
-        process.exitCode = 1;
-      }
-    });
-
-  cmd
-    .command('delete <shareId> <commentId>')
-    .description('delete a comment from a share')
-    .action((shareId, commentId) => {
-      try {
-        const identity = config.get('identity');
-        const comments = getShareComments(shareId);
-        const comment = comments.find(c => c.id === commentId);
-        if (!comment) {
-          console.log(chalk.red('Comment not found.'));
-          process.exitCode = 1;
-          return;
-        }
-        const myHandle = identity?.handle || null;
-        const myName = identity?.name || 'Anonymous';
-        if (comment.authorHandle !== myHandle && comment.authorName !== myName) {
-          console.log(chalk.red('Error: you can only delete your own comments.'));
-          process.exitCode = 1;
-          return;
-        }
-        deleteShareComment(shareId, commentId);
-        console.log(chalk.green('Comment deleted.'));
-      } catch (err) {
-        console.log(chalk.red(err.message));
-        process.exitCode = 1;
-      }
-    });
-
-  cmd.action(() => {
-    console.log(chalk.gray('Use `pe comment list <shareId>` to view comments.'));
-  });
-}
+import chalk from 'chalk';
+import config from '../utils/config.js';
+import {
+  getShareComments,
+  addShareComment,
+  deleteShareComment
+} from '../core/groups.js';
+import { checkLimit } from '../core/pro.js';
+
+export default function commentCommand(program) {
+  const cmd = program
+    .command('comment')
+    .description('manage comments on shares')
+    .addHelpText('after', `
+Examples:
+  $ pal comment list <shareId>                  List comments on a share
+  $ pal comment add <shareId> "nice files!"     Add a comment
+  $ pal comment delete <shareId> <commentId>    Delete a comment
+`);
+
+  cmd
+    .command('list <shareId>')
+    .description('list comments on a share')
+    .action((shareId) => {
+      const comments = getShareComments(shareId);
+      if (comments.length === 0) {
+        console.log(chalk.gray('No comments on this share.'));
+        return;
+      }
+      console.log('');
+      console.log(chalk.cyan(`Comments on ${shareId}:`));
+      for (const c of comments) {
+        const time = new Date(c.createdAt).toLocaleString();
+        const author = c.authorHandle ? `@${c.authorHandle}` : c.authorName || 'unknown';
+        console.log(`  ${chalk.gray(c.id.slice(0, 8))}  ${chalk.white(author)}  ${chalk.gray(time)}`);
+        console.log(`    ${c.text}`);
+      }
+      console.log('');
+    });
+
+  cmd
+    .command('add <shareId> <text>')
+    .description('add a comment to a share')
+    .action((shareId, text) => {
+      if (!text || !text.trim()) {
+        console.log(chalk.red('Error: comment text cannot be empty.'));
+        process.exitCode = 1;
+        return;
+      }
+      try {
+        const comments = getShareComments(shareId);
+        checkLimit('maxCommentsPerShare', comments.length);
+
+        const identity = config.get('identity');
+        const comment = addShareComment(shareId, {
+          authorHandle: identity?.handle || null,
+          authorName: identity?.name || 'Anonymous',
+          text,
+        });
+        console.log(chalk.green(`Comment added.`));
+        console.log(chalk.gray(`ID: ${comment.id}`));
+      } catch (err) {
+        console.log(chalk.red(err.message));
+        process.exitCode = 1;
+      }
+    });
+
+  cmd
+    .command('delete <shareId> <commentId>')
+    .description('delete a comment from a share')
+    .action((shareId, commentId) => {
+      try {
+        const identity = config.get('identity');
+        const comments = getShareComments(shareId);
+        const comment = comments.find(c => c.id === commentId);
+        if (!comment) {
+          console.log(chalk.red('Comment not found.'));
+          process.exitCode = 1;
+          return;
+        }
+        const myHandle = identity?.handle || null;
+        const myName = identity?.name || 'Anonymous';
+        if (comment.authorHandle !== myHandle && comment.authorName !== myName) {
+          console.log(chalk.red('Error: you can only delete your own comments.'));
+          process.exitCode = 1;
+          return;
+        }
+        deleteShareComment(shareId, commentId);
+        console.log(chalk.green('Comment deleted.'));
+      } catch (err) {
+        console.log(chalk.red(err.message));
+        process.exitCode = 1;
+      }
+    });
+
+  cmd.action(() => {
+    console.log(chalk.gray('Use `pal comment list <shareId>` to view comments.'));
+  });
+}