fix: FileDownload cross-platform support and target-path handling (#98, #49)

- Expand PSDependMap.psd1 Supports to include core/macos/linux; no
  Windows-only code was blocking cross-platform use (#98)
- Root relative Target paths against $PWD before any path operations so
  callers are not burned by Split-Path against a relative string (#49)
- Replace parent-exists heuristic with file-extension check to distinguish
  file targets from container targets; directory targets are now created
  when they do not yet exist rather than erroring (#49)
- Fix hardcoded ";" path separator in Write-Verbose to use
  [IO.Path]::PathSeparator for correctness on non-Windows
- Add four tests: existing-dir target, new-dir creation, extension-based
  file-path target, and relative target rooted against $PWD

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: HeyItsGilbert

CHANGELOG.md

@@ -7,11 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Added
+
+- `FileDownload` is now supported on all platforms (`windows`, `core`, `macos`, `linux`); there was no Windows-only code blocking this (#98).
+- `FileDownload` relative `Target` paths are now rooted against `$PWD` before resolution, matching the intuitive expectation of callers (#49).
+
 ### Fixed
 
 - `Get-Dependency -InputObject` no longer mutates the caller's hashtable:
   `PSDependOptions` is now preserved after the call, so a second invocation
   with the same object still honors global options such as `Target` (#35).
+- `FileDownload` no longer misidentifies a directory-like `Target` (no file extension, or trailing slash) as a full file path when its parent happens to exist; the handler now uses a file-extension heuristic to distinguish file targets from container targets and creates the directory when it does not yet exist (#49).
 
 ## [0.4.1] - 2026-06-12
 

PSDepend/PSDependMap.psd1

@@ -27,7 +27,7 @@
     FileDownload    = @{
         Script      = 'FileDownload.ps1'
         Description = 'Download a file'
-        Supports    = 'windows'
+        Supports    = 'windows', 'core', 'macos', 'linux'
     }
 
     FileSystem      = @{

PSDepend/PSDependScripts/FileDownload.ps1

@@ -76,15 +76,16 @@ Write-Verbose "Using URL: $URL"
 
 # Act on target path....
 $ToInstall = $False # Anti pattern
+
+# Normalize relative paths against $PWD so callers don't get burned by cwd-dependent splits
+if (-not [IO.Path]::IsPathRooted($Target)) {
+    $Target = Join-Path $PWD $Target
+}
+
 $TargetParent = Split-Path $Target -Parent
 $PathToAdd = $Target
-if ( (Test-Path $TargetParent) -and -not (Test-Path $Target)) {
-    # They gave us a full path, don't parse the file name, use this!
-    $Path = $Target
-    $ToInstall = $True
-    Write-Verbose "Found parent [$TargetParent], not target [$Target], assuming this is target file path"
-}
-elseif (Test-Path $Target -PathType Leaf) {
+
+if (Test-Path $Target -PathType Leaf) {
     # File exists.  We should download to temp spot, compare hashes, take action as appropriate.
     # For now, skip the file.
     Write-Verbose "Skipping existing file [$Target]"
@@ -93,16 +94,26 @@ elseif (Test-Path $Target -PathType Leaf) {
     }
     $PathToAdd = Split-Path $Target -Parent
 }
-elseif (-not (Test-Path $Target)) {
-    # They gave us something that doesn't look like a new container for a new or existing file. Wat?
-    Write-Error "Could not find target path [$Target]"
-    if ($PSDependAction -contains 'Test') {
-        return $False
+elseif ([IO.Path]::GetExtension($Target) -and -not (Test-Path $Target -PathType Container)) {
+    # Target has a file extension — treat as a full destination file path
+    if (-not (Test-Path $TargetParent)) {
+        Write-Error "Could not find parent path [$TargetParent] for target [$Target]"
+        if ($PSDependAction -contains 'Test') {
+            return $False
+        }
+    }
+    else {
+        $Path = $Target
+        $ToInstall = $True
+        Write-Verbose "Target has extension, treating as file path [$Target]"
     }
 }
 else {
+    # No extension (or already a container) — treat target as a directory
     Write-Verbose "[$Target] is a container, creating path to file"
-    # We have a target container, now find the name
+    if (-not (Test-Path $Target)) {
+        New-Item -ItemType Directory -Path $Target -Force | Out-Null
+    }
     If ($Name) {
         # explicit name
         $FileName = $Name
@@ -138,6 +149,6 @@ if ($PSDependAction -contains 'Install' -and $ToInstall) {
 }
 
 if ($Dependency.AddToPath) {
-    Write-Verbose "Setting PATH to`n$($PathToAdd, $env:PATH -join ';' | Out-String)"
+    Write-Verbose "Setting PATH to`n$($PathToAdd, $env:PATH -join [IO.Path]::PathSeparator | Out-String)"
     Add-ToItemCollection -Reference Env:\Path -Item $PathToAdd
 }

Tests/FileDownload.Type.Tests.ps1

@@ -71,4 +71,45 @@ Describe 'FileDownload script' -Skip:$SkipUnsupported {
         }
         $result | Should -Be $true
     }
+
+    It 'Creates a new directory and downloads into it when Target has no extension and does not exist' {
+        $newDir = Join-Path (New-Item 'TestDrive:/dl5base' -ItemType Directory -Force).FullName 'newcontainer'
+        $dep = New-PSDependFixture -DependencyName 'https://example.com/sample.dll' -DependencyType 'FileDownload' -Target $newDir
+        InModuleScope PSDepend -Parameters @{ Dep = $dep; ScriptPath = $script:ScriptPath; T = $newDir } {
+            & $ScriptPath -Dependency $Dep
+        }
+        Should -Invoke -CommandName Get-WebFile -ModuleName PSDepend -Times 1 -Exactly -ParameterFilter {
+            $URL -eq 'https://example.com/sample.dll' -and ($Path -like "*newcontainer*sample.dll")
+        }
+        Test-Path $newDir -PathType Container | Should -Be $true
+    }
+
+    It 'Treats Target as a full file path when it has a file extension and parent exists' {
+        $targetDir = (New-Item 'TestDrive:/dl6' -ItemType Directory -Force).FullName
+        $targetFile = Join-Path $targetDir 'out.dll'
+        $dep = New-PSDependFixture -DependencyName 'https://example.com/other.dll' -DependencyType 'FileDownload' -Target $targetFile
+        InModuleScope PSDepend -Parameters @{ Dep = $dep; ScriptPath = $script:ScriptPath } {
+            & $ScriptPath -Dependency $Dep
+        }
+        Should -Invoke -CommandName Get-WebFile -ModuleName PSDepend -Times 1 -Exactly -ParameterFilter {
+            $URL -eq 'https://example.com/other.dll' -and $Path -eq $targetFile
+        }
+    }
+
+    It 'Roots a relative Target against $PWD and downloads to it' {
+        $baseDir = (New-Item 'TestDrive:/relbase' -ItemType Directory -Force).FullName
+        Push-Location $baseDir
+        try {
+            $dep = New-PSDependFixture -DependencyName 'https://example.com/sample.dll' -DependencyType 'FileDownload' -Target 'subdir'
+            InModuleScope PSDepend -Parameters @{ Dep = $dep; ScriptPath = $script:ScriptPath } {
+                & $ScriptPath -Dependency $Dep
+            }
+            Should -Invoke -CommandName Get-WebFile -ModuleName PSDepend -Times 1 -Exactly -ParameterFilter {
+                $Path -like "*subdir*sample.dll"
+            }
+        }
+        finally {
+            Pop-Location
+        }
+    }
 }