filedepot 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3323f568d221fde7de4cd30f7450b46444f48bcfe71b2e0dd16acfe84d97fd14
-  data.tar.gz: 39318899de695422ef75292f4e5c5a8cf3271b91ef0d474fdc1305ebfd78dead
+  metadata.gz: 2e67db1d2be17a3af457047eb147926143e85ce7a41f398ff818fc83c404dcfa
+  data.tar.gz: 3a6616e9a4925732d04d78f55fb716127c115d63220b673165795beba8b9d9c7
 SHA512:
-  metadata.gz: 37f23733b8942db6547ab12960cef9308717f1836ff4c1ae6959c3a1196c00195508f4a5d996d8b9efd10a7ac93552799d633fa11e475f6575ddccee690985ee
-  data.tar.gz: 70f21d4037d84be1cc71c12032eb7e31253b304511af80e8d6fb7e397823e87a9c4718db35466bfe746ca3cfc249a089bc65e534d1e3330240196a5dc703c8a0
+  metadata.gz: 42946b05448b6bf6c3db84b7a8c372a8f5e61688ed63defa263401ac67273d74ef647b0642e52277515d1ae2910c17fcda810b5f7da965eb5b2855bbf2913409
+  data.tar.gz: d11483ddbf5f480956a7ae334e5bd238ff701e77e198d259b825db077ae1961c844cbd71e50a8cc04ac6cb39899884ad37688e598fcd91bc534d9bc1baec6872

data/README.md

@@ -30,17 +30,65 @@ sources:
     base_path: /Users/user/filedepot
 ```
 
+Optional `public_base_path` for public URLs (shown in info and after push):
+
+```yaml
+sources:
+  - name: test
+    ssh: ssh
+    host: 127.0.0.1
+    base_path: /data/filedepot
+    public_base_path: https://example.com/files
+```
+
+When `default_source` does not match any source name, the first source is used.
+
 ## Commands
 
 | Command | Description |
 |---------|-------------|
 | `filedepot` | Show current source and available commands |
 | `filedepot config` | Open config file with $EDITOR |
-| `filedepot push HANDLE` | Send file to current storage |
-| `filedepot pull HANDLE [VERSION]` | Get file from storage |
+| `filedepot push HANDLE FILE` | Send file to current storage |
+| `filedepot pull HANDLE [--path PATH] [--version N]` | Get file from storage |
 | `filedepot versions HANDLE` | List all versions of a handle |
+| `filedepot info HANDLE` | Show info for a handle |
 | `filedepot delete HANDLE [VERSION]` | Delete file(s) after confirmation |
 
+### Push
+
+```bash
+filedepot push test test.txt
+```
+
+Sends `test.txt` to storage with handle `test`. Each push creates a new version. When `public_base_path` is configured, the URL is shown after upload.
+
+### Pull
+
+```bash
+filedepot pull test
+filedepot pull test --path ./output/file.txt
+filedepot pull test --version 2
+filedepot pull test --version 2 --path ./output/file.txt
+```
+
+Gets the latest version by default, or a specific version with `--version`. Use `--path` to specify the local destination. Prompts before creating directories or overwriting files.
+
+### Versions
+
+Lists versions in descending order with creation datetime. Shows at most 10, with a summary if more exist.
+
+### Info
+
+Shows handle, remote base path, current version, updated-at datetime, and latest version URL (when `public_base_path` is set).
+
+## Testing
+
+```bash
+bundle install
+bundle exec rake test
+```
+
 ## License
 
 MIT

data/lib/filedepot/cli.rb

@@ -50,6 +50,20 @@ module Filedepot
 
     desc "push HANDLE FILE", "Send a file to the current storage with a specific handle"
     def push(handle, file_path)
+      if handle.to_s.strip.empty?
+        puts "Error: Handle is required."
+        return
+      end
+      if file_path.to_s.strip.empty?
+        puts "Error: File path is required."
+        return
+      end
+      path = File.expand_path(file_path)
+      unless File.file?(path)
+        puts "Error: File not found: #{path}"
+        return
+      end
+
       source = Config.current_source
       if source.nil?
         puts "Error: No storage source configured. Run 'filedepot config' to set up."
@@ -57,11 +71,13 @@ module Filedepot
       end
 
       storage = Storage::Base.for(source)
-      storage.push(handle, file_path)
+      storage.push(handle, path)
       version = storage.current_version(handle)
       puts "Pushed #{file_path} as #{handle} (version #{version})"
-      uploaded_url = storage.url(handle, version, File.basename(file_path))
+      uploaded_url = storage.url(handle, version, File.basename(path))
       puts uploaded_url if uploaded_url
+    rescue RuntimeError => e
+      puts "Error: #{e.message}"
     end
 
     desc "pull HANDLE", "Get file from storage"
@@ -100,6 +116,8 @@ module Filedepot
 
       storage.pull(handle, version, target_path)
       puts "pulled to #{target_path}"
+    rescue RuntimeError => e
+      puts "Error: #{e.message}"
     end
 
     desc "versions HANDLE", "List all versions of a handle (each version has an integer from 1 to n)"
@@ -118,7 +136,7 @@ module Filedepot
       storage = Storage::Base.for(source)
       versions_list = storage.versions(handle)
       if versions_list.empty?
-        puts "No versions found for handle: #{handle}"
+        puts "Error: Handle '#{handle}' not found."
       else
         max_display = 10
         to_show = versions_list.first(max_display)
@@ -144,6 +162,12 @@ module Filedepot
       end
 
       storage = Storage::Base.for(source)
+      versions_list = storage.versions(handle)
+      if versions_list.empty?
+        puts "Error: Handle '#{handle}' not found."
+        return
+      end
+
       data = storage.info(handle)
 
       puts "handle: #{data[:handle]}"
@@ -159,8 +183,46 @@ module Filedepot
         puts COMMAND_HELP[:delete]
         return
       end
+
+      source = Config.current_source
+      if source.nil?
+        puts "Error: No storage source configured. Run 'filedepot config' to set up."
+        return
+      end
+
+      storage = Storage::Base.for(source)
+      versions_list = storage.versions(handle)
+      if versions_list.empty?
+        puts "Error: Handle '#{handle}' not found."
+        return
+      end
+      if version
+        version_nums = versions_list.map(&:first)
+        unless version_nums.include?(version.to_i)
+          puts "Error: Version #{version} not found for handle '#{handle}'."
+          return
+        end
+      end
+
       version_str = version ? " version #{version}" : " all versions"
-      puts "delete: would delete file '#{handle}'#{version_str} after confirmation (not implemented)"
+      puts "This will delete '#{handle}'#{version_str}."
+      begin
+        print "Type the handle name to confirm: "
+        input = $stdin.gets&.strip
+      rescue Interrupt
+        puts
+        return
+      end
+
+      unless input == handle
+        puts "Aborted (handle name did not match)."
+        return
+      end
+
+      storage.delete(handle, version)
+      puts "Deleted #{handle}#{version ? " version #{version}" : ""}."
+    rescue RuntimeError => e
+      puts "Error: #{e.message}"
     end
 
     default_task :default

data/lib/filedepot/storage/base.rb

@@ -74,10 +74,11 @@ module Filedepot
       end
 
       def url(handle, version, filename)
-        base = @source["public_base_path"].to_s.sub(%r{/$}, "")
+        base = @source["public_base_path"].to_s.sub(%r{/+$}, "")
         return nil if base.empty? || filename.nil? || filename.empty?
 
-        [base, handle, version, filename].join("/").gsub(%r{/+}, "/")
+        path = [handle, version, filename].join("/")
+        "#{base}/#{path}"
       end
 
       protected

data/lib/filedepot/storage_ssh.rb

@@ -27,7 +27,7 @@ module Filedepot
 
       def push(handle, local_path)
         path = File.expand_path(local_path)
-        raise "File not found: #{local_path}" unless File.file?(path)
+        raise "File not found: #{path}" unless File.file?(path)
 
         push_path = next_version_path(handle)
 
@@ -57,14 +57,14 @@ module Filedepot
       def pull(handle, version = nil, local_path = nil)
         ssh_session do |ssh|
           versions_list = versions_for(ssh, handle)
-          raise "No versions found for handle: #{handle}" if versions_list.empty?
+          raise "Handle '#{handle}' not found" if versions_list.empty?
 
           version_num = version ? version.to_i : versions_list.max
-          raise "Version #{version} not found" unless versions_list.include?(version_num)
+          raise "Version #{version} not found for handle '#{handle}'" unless versions_list.include?(version_num)
 
           version_dir = File.join(remote_handle_path(handle), version_num.to_s)
           remote_file = first_file_in_dir(ssh, version_dir)
-          raise "No file found in version #{version_num}" if remote_file.nil?
+          raise "No file found in version #{version_num} for handle '#{handle}'" if remote_file.nil?
 
           remote_filename = File.basename(remote_file)
           target_path = resolve_local_path(local_path, remote_filename)
@@ -77,14 +77,14 @@ module Filedepot
       def pull_info(handle, version = nil, local_path = nil)
         ssh_session do |ssh|
           versions_list = versions_for(ssh, handle)
-          raise "No versions found for handle: #{handle}" if versions_list.empty?
+          raise "Handle '#{handle}' not found" if versions_list.empty?
 
           version_num = version ? version.to_i : versions_list.max
-          raise "Version #{version} not found" unless versions_list.include?(version_num)
+          raise "Version #{version} not found for handle '#{handle}'" unless versions_list.include?(version_num)
 
           version_dir = File.join(remote_handle_path(handle), version_num.to_s)
           remote_file = first_file_in_dir(ssh, version_dir)
-          raise "No file found in version #{version_num}" if remote_file.nil?
+          raise "No file found in version #{version_num} for handle '#{handle}'" if remote_file.nil?
 
           remote_filename = File.basename(remote_file)
           target_path = resolve_local_path(local_path, remote_filename)

data/lib/filedepot/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Filedepot
-  VERSION = "0.2.1"
+  VERSION = "0.2.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: filedepot
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Filedepot