appydave-tools 0.20.1 → 0.21.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ad2e56b8d8b3c9a1cb3e97bf6a020dff3c912ae6d22be2188104276cf952a0cf
-  data.tar.gz: ba06bb06e615d282df69ccb5b52a1f5b5264f02dfd7af574e86a71721cec1633
+  metadata.gz: c9c1294eb12c004bfbf7d5c3af2ad72e70202729867d5b2e1978d6710f573eca
+  data.tar.gz: 4c00d396ba791995706c8caea17eb36b65c0c6142533281ceb83d75e74a90d00
 SHA512:
-  metadata.gz: 53ba4ece9f3f7d7c8fe29107b9059018285a382acb02a1f26275d0b41a95745402dbe74d9609a6a089c73c0ec92ce320df58fa50b8bbd4b17578f0d84234bc1a
-  data.tar.gz: 927899c39261f330ceb73387a8598023a5c642cec24e5753ddde063af545a7aa373ac07969c6f0ee5c69ed2ee76b4f75d871c96cce2d015ef81d7fae6b532fab
+  metadata.gz: 9c9e76453f9594c21c6e5a3571369f715eda1c5262fc5e80429776ed1a4e6afc29a6f258c238df3ef68ed3fc6a769dc5186168674aa90c9d0ee8705e2d056da9
+  data.tar.gz: 1dac5dcaa14e3731fff08d6bfb0f3b6128549920503dd6e5869caf89696d40b6e4f745afeade7261d7ccc0a51baa93722b4204b8808b140537c041f393c77703

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [0.20.1](https://github.com/appydave/appydave-tools/compare/v0.20.0...v0.20.1) (2025-11-10)
+
+
+### Bug Fixes
+
+* remove noisy backup log message from self-healing config saves ([f9dacfd](https://github.com/appydave/appydave-tools/commit/f9dacfd74cce5f991c7f4bafb12d1494aa6ee4ab))
+
 # [0.20.0](https://github.com/appydave/appydave-tools/compare/v0.19.0...v0.20.0) (2025-11-10)
 
 

data/bin/bank_reconciliation.rb

@@ -3,7 +3,6 @@
 
 $LOAD_PATH.unshift(File.expand_path('../lib', __dir__))
 
-require 'pry'
 require 'appydave/tools'
 
 # !/usr/bin/env ruby

data/bin/configuration.rb

@@ -10,7 +10,6 @@
 
 $LOAD_PATH.unshift(File.expand_path('../lib', __dir__))
 
-require 'pry'
 require 'appydave/tools'
 
 options = { keys: [] }

data/bin/dam

@@ -25,6 +25,8 @@ class VatCLI
       'repo-status' => method(:repo_status_command),
       'repo-sync' => method(:repo_sync_command),
       'repo-push' => method(:repo_push_command),
+      's3-share' => method(:s3_share_command),
+      's3-discover' => method(:s3_discover_command),
       # Deprecated aliases (for backward compatibility)
       's3-cleanup' => method(:s3_cleanup_remote_command),
       'cleanup-local' => method(:s3_cleanup_local_command)
@@ -66,6 +68,10 @@ class VatCLI
       show_config_help
     when 'list'
       show_list_help
+    when 's3-share'
+      show_s3_share_help
+    when 's3-discover'
+      show_s3_discover_help
     when 's3-up', 's3-down', 's3-status', 's3-cleanup-remote', 's3-cleanup-local', 'archive', 'manifest', 'sync-ssd'
       show_s3_help(topic)
     when 's3-cleanup', 'cleanup-local'
@@ -151,6 +157,30 @@ class VatCLI
     exit 1
   end
 
+  # Share file via pre-signed URL
+  def s3_share_command(args)
+    options = parse_share_args(args)
+
+    share_ops = Appydave::Tools::Dam::ShareOperations.new(options[:brand], options[:project])
+    share_ops.generate_links(files: options[:file], expires: options[:expires], download: options[:download])
+  rescue StandardError => e
+    puts "❌ Error: #{e.message}"
+    exit 1
+  end
+
+  # Discover files in S3 for a project
+  def s3_discover_command(args)
+    options = parse_discover_args(args)
+    files = fetch_s3_files(options[:brand_key], options[:project_id])
+
+    return if handle_empty_files?(files, options[:brand_key], options[:project_id])
+
+    display_s3_files(files, options[:brand_key], options[:project_id], options[:shareable])
+  rescue StandardError => e
+    puts "❌ Error: #{e.message}"
+    exit 1
+  end
+
   # Archive project to SSD
   def archive_command(args)
     options = parse_s3_args(args, 'archive')
@@ -273,19 +303,14 @@ class VatCLI
     project_arg = args[1]
 
     if brand_arg.nil?
-      # Auto-detect from PWD
-      brand, project_id = Appydave::Tools::Dam::ProjectResolver.detect_from_pwd
-      if brand.nil?
-        puts '❌ Could not auto-detect brand from current directory'
-        puts 'Usage: dam status <brand> [project]'
-        exit 1
-      end
-      brand_key = brand
-    else
-      brand_key = brand_arg
-      project_id = project_arg ? Appydave::Tools::Dam::ProjectResolver.resolve(brand_arg, project_arg) : nil
+      puts '❌ Brand argument required'
+      puts 'Usage: dam status <brand> [project]'
+      exit 1
     end
 
+    brand_key = brand_arg
+    project_id = project_arg ? Appydave::Tools::Dam::ProjectResolver.resolve(brand_arg, project_arg) : nil
+
     status = Appydave::Tools::Dam::Status.new(brand_key, project_id)
     status.show
   rescue StandardError => e
@@ -402,6 +427,113 @@ class VatCLI
     { brand: brand_key, project: project_id, dry_run: dry_run, force: force }
   end
 
+  def parse_share_args(args)
+    # Extract --expires flag
+    expires = '7d' # default
+    if (expires_index = args.index('--expires'))
+      expires = args[expires_index + 1]
+      args.delete_at(expires_index + 1)
+      args.delete_at(expires_index)
+    end
+
+    # Extract --download flag
+    download = args.include?('--download')
+
+    # Remove other flags
+    args = args.reject { |arg| arg.start_with?('--') }
+
+    brand_arg = args[0]
+    project_arg = args[1]
+    file_arg = args[2]
+
+    show_share_usage_and_exit if brand_arg.nil? || project_arg.nil? || file_arg.nil?
+
+    brand_key = brand_arg
+    brand = Appydave::Tools::Dam::Config.expand_brand(brand_arg)
+    project_id = Appydave::Tools::Dam::ProjectResolver.resolve(brand_arg, project_arg)
+
+    # Set ENV for compatibility with ConfigLoader
+    ENV['BRAND_PATH'] = Appydave::Tools::Dam::Config.brand_path(brand)
+
+    { brand: brand_key, project: project_id, file: file_arg, expires: expires, download: download }
+  end
+
+  def show_share_usage_and_exit
+    puts 'Usage: dam s3-share <brand> <project> <file> [--expires 7d] [--download]'
+    puts ''
+    puts 'Options:'
+    puts '  --expires TIME    Expiry time (default: 7d)'
+    puts '  --download        Force download instead of viewing in browser'
+    puts ''
+    puts 'Examples:'
+    puts '  dam s3-share appydave b70 video.mp4'
+    puts '  dam s3-share appydave b70 video.mp4 --expires 24h'
+    puts '  dam s3-share appydave b70 video.mp4 --download'
+    puts '  dam s3-share voz boy-baker final-edit.mov --expires 3d --download'
+    exit 1
+  end
+
+  def parse_discover_args(args)
+    shareable = args.include?('--shareable')
+    args = args.reject { |arg| arg.start_with?('--') }
+
+    brand_arg = args[0]
+    project_arg = args[1]
+
+    if brand_arg.nil? || project_arg.nil?
+      puts 'Usage: dam s3-discover <brand> <project> [--shareable]'
+      puts ''
+      puts 'Examples:'
+      puts '  dam s3-discover appydave b70              # List files'
+      puts '  dam s3-discover appydave b70 --shareable  # Generate share commands'
+      exit 1
+    end
+
+    brand_key = brand_arg
+    brand = Appydave::Tools::Dam::Config.expand_brand(brand_arg)
+    project_id = Appydave::Tools::Dam::ProjectResolver.resolve(brand_arg, project_arg)
+
+    # Set ENV for compatibility with ConfigLoader
+    ENV['BRAND_PATH'] = Appydave::Tools::Dam::Config.brand_path(brand)
+
+    { brand_key: brand_key, project_id: project_id, shareable: shareable }
+  end
+
+  def fetch_s3_files(brand_key, project_id)
+    s3_ops = Appydave::Tools::Dam::S3Operations.new(brand_key, project_id)
+    s3_ops.list_s3_files
+  end
+
+  def handle_empty_files?(files, brand_key, project_id)
+    return false unless files.empty?
+
+    puts "📭 No files found in S3 for #{brand_key}/#{project_id}"
+    puts ''
+    puts 'Upload files first:'
+    puts "  dam s3-up #{brand_key} #{project_id}"
+    true
+  end
+
+  def display_s3_files(files, brand_key, project_id, shareable)
+    puts ''
+    puts "📁 Files in S3: #{brand_key}/#{project_id}"
+    puts ''
+
+    files.each do |file_info|
+      # list_s3_files returns array of hashes with 'Key', 'Size', 'ETag'
+      s3_key = file_info['Key']
+      filename = File.basename(s3_key)
+
+      if shareable
+        puts "dam s3-share #{brand_key} #{project_id} #{filename}"
+      else
+        puts filename
+      end
+    end
+
+    puts ''
+  end
+
   # Help text methods
   # rubocop:disable Metrics/MethodLength
   def show_main_help
@@ -415,7 +547,7 @@ class VatCLI
         list [brand] [pattern]      List brands/projects
 
         Status & Monitoring:
-        status [brand] [project]         Show unified status (local/S3/SSD/git)
+        status <brand> [project]         Show unified status (local/S3/SSD/git)
 
         Git Repository Commands:
         repo-status <brand> [--all]      Check git status for brand repos
@@ -429,6 +561,10 @@ class VatCLI
         s3-cleanup-remote <brand> <project>  Delete S3 files
         s3-cleanup-local <brand> <project>   Delete local s3-staging files
 
+        S3 Sharing Commands:
+        s3-discover <brand> <project>    List files available in S3
+        s3-share <brand> <project> <file> Generate shareable pre-signed URL
+
         Archive Commands:
         archive <brand> <project>        Copy project to SSD backup
         manifest <brand> [--all]         Generate project manifest
@@ -453,7 +589,11 @@ class VatCLI
         dam s3-up appydave b65
         dam s3-down voz boy-baker --dry-run
         dam s3-status appydave b65
-        dam s3-cleanup appydave b65 --force
+        dam s3-discover appydave b70
+        dam s3-discover appydave b70 --shareable
+        dam s3-share appydave b70 video.mp4
+        dam s3-share appydave b70 video.mp4 --expires 24h
+        dam s3-cleanup-remote appydave b65 --force
 
       For more information: https://github.com/appydave/appydave-tools
     HELP
@@ -474,9 +614,9 @@ class VatCLI
         ss        → v-supportsignal    (SupportSignal client)
 
       Usage:
-        dam list                       # Show all brands
-        dam list --summary             # Show brands with project counts
+        dam list                       # Show all brands with counts
         dam list appydave              # List all AppyDave projects
+        dam list appydave 'b6*'        # List AppyDave projects matching pattern
         dam s3-up voz boy-baker        # Upload VOZ project
     HELP
   end
@@ -585,6 +725,108 @@ class VatCLI
     HELP
   end
 
+  def show_s3_share_help
+    puts <<~HELP
+      S3 Share Command - Generate Shareable Pre-Signed URLs
+
+      Generate time-limited secure URLs for sharing video files with clients.
+      Recipients don't need AWS access - just click the link in their browser!
+
+      Usage: dam s3-share <brand> <project> <file> [OPTIONS]
+
+      Options:
+        --expires TIME    Expiry time (default: 7d)
+                         Examples: 24h, 48h, 3d, 7d
+        --download        Force download instead of viewing in browser (default: inline viewing)
+
+      How It Works:
+        1. Generates pre-signed S3 URL (time-limited, secure)
+        2. Copies URL to clipboard automatically
+        3. Shows expiry date/time
+        4. By default: Opens in browser (inline) - perfect for video playback
+        5. With --download: Forces file download
+
+      Examples:
+        dam s3-share appydave b70 video.mp4
+        # → View in browser (default), 7-day expiry
+
+        dam s3-share appydave b70 video.mp4 --download
+        # → Force download, 7-day expiry
+
+        dam s3-share appydave b70 final-edit.mp4 --expires 24h
+        # → View in browser, 24-hour expiry for quick review
+
+        dam s3-share voz boy-baker scene-01.mov --expires 3d --download
+        # → Force download, 3-day expiry for client approval
+
+      Viewing Modes:
+        🎬 Default (inline):  Opens in browser - great for video playback
+        📥 --download flag:   Forces download - better for archiving
+
+      Security:
+        ✅ Time-limited (expires automatically)
+        ✅ Cryptographically signed (can't be tampered with)
+        ✅ No AWS state stored (signature encodes everything)
+        ✅ Revocable (delete S3 file if needed)
+        ⚠️  Anyone with URL can access (don't share publicly)
+
+      Prerequisites:
+        - File must be uploaded to S3 first
+        - Run: dam s3-up <brand> <project>
+
+      Use Cases:
+        - Share videos with clients for review (inline)
+        - Send final files for download (--download)
+        - Provide temporary access to collaborators
+        - Quick video sharing without email attachments
+
+      For more information: https://github.com/appydave/appydave-tools
+    HELP
+  end
+
+  def show_s3_discover_help
+    puts <<~HELP
+      S3 Discover Command - List Files in S3
+
+      Discover what video files exist in S3 for a project.
+      Useful before sharing to see available files.
+
+      Usage: dam s3-discover <brand> <project> [--shareable]
+
+      Modes:
+        Default         List filenames only
+        --shareable     Generate ready-to-run share commands
+
+      Examples:
+        dam s3-discover appydave b70
+        # Output:
+        # xmen.mp4
+        # behind-the-scenes.mov
+
+        dam s3-discover appydave b70 --shareable
+        # Output:
+        # dam s3-share appydave b70 xmen.mp4
+        # dam s3-share appydave b70 behind-the-scenes.mov
+
+      Workflow:
+        1. Upload files: dam s3-up appydave b70
+        2. Discover files: dam s3-discover appydave b70
+        3. Share specific file: dam s3-share appydave b70 xmen.mp4
+
+      Use Cases:
+        - Find files available for sharing
+        - Generate multiple share commands at once
+        - Quick project file listing without downloading
+        - Copy-paste shareable commands for clients
+
+      Prerequisites:
+        - Files must be uploaded to S3 first
+        - Run: dam s3-up <brand> <project>
+
+      For more information: https://github.com/appydave/appydave-tools
+    HELP
+  end
+
   # rubocop:disable Metrics/CyclomaticComplexity
   def show_s3_help(command)
     case command

data/docs/dam/dam-testing-plan.md

@@ -7,7 +7,7 @@
 
 **Reference**: Original UAT plan at `/Users/davidcruwys/dev/video-projects/video-asset-tools/docs/testing-plan.md`
 
-**Windows Testing**: See [windows-testing-guide.md](./windows-testing-guide.md) for Windows-specific testing scenarios
+**Windows Testing**: See [dam-testing-plan-windows-powershell.md](./dam-testing-plan-windows-powershell.md) for Windows-specific testing scenarios
 
 ---
 
@@ -245,14 +245,14 @@ bundle exec rspec spec/appydave/tools/dam/config_loader_spec.rb
 #### Test 2.1: Help System
 ```bash
 # Test: Main help
-bin/dam help
+dam help
 
 # Test: Command-specific help
-bin/dam help list
-bin/dam help s3-up
-bin/dam help s3-down
-bin/dam help brands
-bin/dam help workflows
+dam help list
+dam help s3-up
+dam help s3-down
+dam help brands
+dam help workflows
 ```
 **Expected**:
 - Full help overview
@@ -265,40 +265,22 @@ bin/dam help workflows
 
 ---
 
-#### Test 2.3: List Brands (Mode 1)
-```bash
-# Test: List brands only
-bin/dam list
-```
-**Expected**: `Brands: aitldr, appydave, joy, kiros, ss, voz` (shortcuts)
-
-**Status**: [ ] Pass [ ] Fail
-**Notes**: ___________________________________________
-
----
-
-#### Test 2.4: List Brands with Summary (Mode 2)
+#### Test 2.3: List Brands
 ```bash
-# Test: Brands with project counts
-bin/dam list --summary
-```
-**Expected**:
-```
-aitldr: X projects
-appydave: Y projects
-joy: Z projects
-...
+# Test: List all brands with project counts
+dam list
 ```
+**Expected**: Tabular output showing all brands with project counts, sizes, and paths
 
 **Status**: [ ] Pass [ ] Fail
 **Notes**: ___________________________________________
 
 ---
 
-#### Test 2.5: List Brand Projects (Mode 3)
+#### Test 2.4: List Brand Projects
 ```bash
 # Test: All projects for brand
-bin/dam list appydave
+dam list appydave
 ```
 **Expected**: List of all appydave projects (excludes `archived/`, `.git`, etc.)
 
@@ -307,10 +289,10 @@ bin/dam list appydave
 
 ---
 
-#### Test 2.6: Pattern Matching (Mode 3b)
+#### Test 2.5: Pattern Matching
 ```bash
 # Test: Pattern matching
-bin/dam list appydave 'b6*'
+dam list appydave 'b6*'
 ```
 **Expected**: Only projects starting with `b6` (b60-b69)
 
@@ -319,7 +301,7 @@ bin/dam list appydave 'b6*'
 
 ---
 
-#### Test 2.7: S3 Upload with CLI Args
+#### Test 2.6: S3 Upload with CLI Args
 ```bash
 # Test: Upload with explicit args (dry-run)
 bin/s3_sync_up.rb appydave b65 --dry-run
@@ -331,7 +313,7 @@ bin/s3_sync_up.rb appydave b65 --dry-run
 
 ---
 
-#### Test 2.8: S3 Upload Auto-Detect
+#### Test 2.7: S3 Upload Auto-Detect
 ```bash
 # Test: Upload from project directory
 cd ~/dev/video-projects/v-appydave/b65-*
@@ -344,7 +326,7 @@ bin/s3_sync_up.rb --dry-run
 
 ---
 
-#### Test 2.9: S3 Download with CLI Args ⭐ NEW
+#### Test 2.8: S3 Download with CLI Args ⭐ NEW
 ```bash
 # Test: Download with explicit args (dry-run)
 bin/s3_sync_down.rb appydave b65 --dry-run
@@ -356,7 +338,7 @@ bin/s3_sync_down.rb appydave b65 --dry-run
 
 ---
 
-#### Test 2.10: S3 Download Auto-Detect ⭐ NEW
+#### Test 2.9: S3 Download Auto-Detect ⭐ NEW
 ```bash
 # Test: Download from project directory
 cd ~/dev/video-projects/v-appydave/b65-*
@@ -369,7 +351,7 @@ bin/s3_sync_down.rb --dry-run
 
 ---
 
-#### Test 2.11: S3 Status with CLI Args ⭐ NEW
+#### Test 2.10: S3 Status with CLI Args ⭐ NEW
 ```bash
 # Test: Check status with explicit args
 bin/s3_sync_status.rb appydave b65
@@ -381,7 +363,7 @@ bin/s3_sync_status.rb appydave b65
 
 ---
 
-#### Test 2.12: S3 Status Auto-Detect ⭐ NEW
+#### Test 2.11: S3 Status Auto-Detect ⭐ NEW
 ```bash
 # Test: Check status from project directory
 cd ~/dev/video-projects/v-appydave/b65-*
@@ -394,7 +376,7 @@ bin/s3_sync_status.rb
 
 ---
 
-#### Test 2.13: S3 Cleanup with CLI Args ⭐ NEW
+#### Test 2.12: S3 Cleanup with CLI Args ⭐ NEW
 ```bash
 # Test: Cleanup with explicit args (dry-run)
 bin/s3_sync_cleanup.rb appydave b65 --dry-run
@@ -406,7 +388,7 @@ bin/s3_sync_cleanup.rb appydave b65 --dry-run
 
 ---
 
-#### Test 2.14: S3 Cleanup Auto-Detect ⭐ NEW
+#### Test 2.13: S3 Cleanup Auto-Detect ⭐ NEW
 ```bash
 # Test: Cleanup from project directory
 cd ~/dev/video-projects/v-appydave/b65-*
@@ -419,10 +401,10 @@ bin/s3_sync_cleanup.rb --dry-run
 
 ---
 
-#### Test 2.15: Status - Brand Overview ⭐ PHASE 4
+#### Test 2.14: Status - Brand Overview ⭐ PHASE 4
 ```bash
 # Test: Show unified status for entire brand
-bin/dam status appydave
+dam status appydave
 ```
 **Expected**: Shows brand git status, manifest summary (total projects, local, S3, SSD counts)
 
@@ -431,10 +413,10 @@ bin/dam status appydave
 
 ---
 
-#### Test 2.16: Status - Specific Project ⭐ PHASE 4
+#### Test 2.15: Status - Specific Project ⭐ PHASE 4
 ```bash
 # Test: Show detailed status for specific project
-bin/dam status appydave b65
+dam status appydave b65
 ```
 **Expected**: Shows storage locations (local, S3 staging, SSD backup) and git status for project
 
@@ -443,23 +425,10 @@ bin/dam status appydave b65
 
 ---
 
-#### Test 2.17: Status - Auto-Detect ⭐ PHASE 4
-```bash
-# Test: Auto-detect brand and project from PWD
-cd ~/dev/video-projects/v-appydave/b65-*
-bin/dam status
-```
-**Expected**: Auto-detects and shows project status
-
-**Status**: [ ] Pass [ ] Fail
-**Notes**: ___________________________________________
-
----
-
-#### Test 2.18: Repo Status - Single Brand ⭐ PHASE 4
+#### Test 2.16: Repo Status - Single Brand ⭐ PHASE 4
 ```bash
 # Test: Show git status for single brand
-bin/dam repo-status appydave
+dam repo-status appydave
 ```
 **Expected**: Shows branch, remote, working directory status, sync status
 
@@ -468,10 +437,10 @@ bin/dam repo-status appydave
 
 ---
 
-#### Test 2.19: Repo Status - All Brands ⭐ PHASE 4
+#### Test 2.17: Repo Status - All Brands ⭐ PHASE 4
 ```bash
 # Test: Show git status for all brands
-bin/dam repo-status --all
+dam repo-status --all
 ```
 **Expected**: Shows git status for all configured brands (appydave, voz, aitldr, etc.)
 
@@ -480,10 +449,10 @@ bin/dam repo-status --all
 
 ---
 
-#### Test 2.20: Repo Sync - Single Brand ⭐ PHASE 4
+#### Test 2.18: Repo Sync - Single Brand ⭐ PHASE 4
 ```bash
 # Test: Pull updates for single brand
-bin/dam repo-sync appydave
+dam repo-sync appydave
 ```
 **Expected**: Pulls updates if available, skips if uncommitted changes, shows status
 
@@ -492,10 +461,10 @@ bin/dam repo-sync appydave
 
 ---
 
-#### Test 2.21: Repo Sync - All Brands ⭐ PHASE 4
+#### Test 2.19: Repo Sync - All Brands ⭐ PHASE 4
 ```bash
 # Test: Pull updates for all brands
-bin/dam repo-sync --all
+dam repo-sync --all
 ```
 **Expected**: Syncs all brands, shows summary, skips brands with uncommitted changes
 
@@ -504,10 +473,10 @@ bin/dam repo-sync --all
 
 ---
 
-#### Test 2.22: Repo Push - Basic ⭐ PHASE 4
+#### Test 2.20: Repo Push - Basic ⭐ PHASE 4
 ```bash
 # Test: Push changes for brand (no project validation)
-bin/dam repo-push appydave
+dam repo-push appydave
 ```
 **Expected**: Pushes commits, shows push summary
 
@@ -516,10 +485,10 @@ bin/dam repo-push appydave
 
 ---
 
-#### Test 2.23: Repo Push - With Project Validation ⭐ PHASE 4
+#### Test 2.21: Repo Push - With Project Validation ⭐ PHASE 4
 ```bash
 # Test: Push with project validation
-bin/dam repo-push appydave b65
+dam repo-push appydave b65
 ```
 **Expected**: Validates project exists in manifest before pushing, shows validation message