From 73081a2b82e0335c4189120825758648c0fcfe8c Mon Sep 17 00:00:00 2001
From: Marius Storhaug <marstor@hotmail.com>
Date: Sat, 18 Jul 2026 20:10:27 +0200
Subject: [PATCH 01/13] Make class and enum loader dependency aware

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 .../src/helpers/Build/Add-ContentFromItem.ps1 | 139 ++++++++++++++++--
 .../Build/Build-PSModuleRootModule.ps1        |  37 +++--
 .../src/classes/public/A-BookList.ps1         |  86 +++++++++++
 tests/srcTestRepo/src/classes/public/Book.ps1 |  87 -----------
 .../src/classes/public/A-BookList.ps1         |  86 +++++++++++
 .../src/classes/public/Book.ps1               |  87 -----------
 6 files changed, 312 insertions(+), 210 deletions(-)
 create mode 100644 tests/srcTestRepo/src/classes/public/A-BookList.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/A-BookList.ps1

diff --git a/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1 b/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1
index 9046b204..1b4e0c8d 100644
--- a/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1
+++ b/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1
@@ -1,4 +1,103 @@
-function Add-ContentFromItem {
+function Get-DependencyOrderedScriptFiles {
+    [OutputType([System.IO.FileInfo[]])]
+    [CmdletBinding()]
+    param(
+        [Parameter(Mandatory)]
+        [System.IO.FileInfo[]] $Files
+    )
+
+    $sortedFiles = $Files | Sort-Object -Property FullName
+    if ($sortedFiles.Count -le 1) {
+        return $sortedFiles
+    }
+
+    $metadataByPath = @{}
+    $typeToPath = @{}
+    $duplicateTypes = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
+
+    foreach ($file in $sortedFiles) {
+        $content = Get-Content -Path $file.FullName -Raw
+        $declaredTypes = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
+        [Regex]::Matches($content, '(?im)^\s*(class|enum)\s+([^\s:{]+)') | ForEach-Object {
+            [void] $declaredTypes.Add($_.Groups[2].Value)
+        }
+
+        $metadataByPath[$file.FullName] = [pscustomobject]@{
+            File          = $file
+            Content       = $content
+            DeclaredTypes = $declaredTypes
+        }
+
+        foreach ($typeName in $declaredTypes) {
+            if ($typeToPath.ContainsKey($typeName)) {
+                [void]$duplicateTypes.Add($typeName)
+                continue
+            }
+
+            $typeToPath[$typeName] = $file.FullName
+        }
+    }
+
+    $dependenciesByPath = @{}
+    $dependentsByPath = @{}
+    foreach ($file in $sortedFiles) {
+        $dependenciesByPath[$file.FullName] = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
+        $dependentsByPath[$file.FullName] = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
+    }
+
+    foreach ($metadata in $metadataByPath.Values) {
+        foreach ($typeName in $typeToPath.Keys) {
+            if ($duplicateTypes.Contains($typeName)) {
+                continue
+            }
+
+            if ($metadata.DeclaredTypes.Contains($typeName)) {
+                continue
+            }
+
+            $typePattern = "(?<![\w\.])\[$([Regex]::Escape($typeName))\](?![\w\.])"
+            if ([Regex]::IsMatch($metadata.Content, $typePattern, [System.Text.RegularExpressions.RegexOptions]::IgnoreCase)) {
+                $dependencyPath = $typeToPath[$typeName]
+                [void]$dependenciesByPath[$metadata.File.FullName].Add($dependencyPath)
+                [void]$dependentsByPath[$dependencyPath].Add($metadata.File.FullName)
+            }
+        }
+    }
+
+    $remainingPaths = @($sortedFiles.FullName)
+    $ready = @(
+        $remainingPaths |
+        Where-Object { $dependenciesByPath[$_].Count -eq 0 } |
+        Sort-Object
+    )
+    $orderedPaths = [System.Collections.Generic.List[string]]::new()
+
+    while ($ready.Count -gt 0) {
+        $currentPath = $ready[0]
+        $ready = @($ready | Select-Object -Skip 1)
+        $orderedPaths.Add($currentPath)
+        $remainingPaths = @($remainingPaths | Where-Object { $_ -ne $currentPath })
+
+        $dependents = @($dependentsByPath[$currentPath] | Sort-Object)
+        foreach ($dependentPath in $dependents) {
+            $null = $dependenciesByPath[$dependentPath].Remove($currentPath)
+            if ($dependenciesByPath[$dependentPath].Count -eq 0) {
+                $ready = @($ready + $dependentPath | Sort-Object -Unique)
+            }
+        }
+    }
+
+    if ($orderedPaths.Count -lt $sortedFiles.Count) {
+        Write-Warning "Detected cyclical or unresolved class/enum dependencies. Falling back to lexical order for remaining files in [$($sortedFiles[0].DirectoryName)]."
+        $remainingPaths | Sort-Object | ForEach-Object {
+            $orderedPaths.Add($_)
+        }
+    }
+
+    return @($orderedPaths | ForEach-Object { $metadataByPath[$_].File })
+}
+
+function Add-ContentFromItem {
     <#
         .SYNOPSIS
         Add the content of a folder or file to the root module file.
@@ -20,15 +119,16 @@
 
         # The root path of the module.
         [Parameter(Mandatory)]
-        [string] $RootPath
+        [string] $RootPath,
+
+        # Whether the folder should be loaded using dependency ordering (class/enum aware).
+        [Parameter()]
+        [switch] $DependencyAware
     )
-    # Get the path separator for the current OS
-    $pathSeparator = [System.IO.Path]::DirectorySeparatorChar
+    $separators = [char[]]@([System.IO.Path]::DirectorySeparatorChar, [System.IO.Path]::AltDirectorySeparatorChar)
 
-    $relativeFolderPath = $Path -replace $RootPath, ''
-    $relativeFolderPath = $relativeFolderPath -replace $file.Extension, ''
-    $relativeFolderPath = $relativeFolderPath.TrimStart($pathSeparator)
-    $relativeFolderPath = $relativeFolderPath -split $pathSeparator | ForEach-Object { "[$_]" }
+    $relativeFolderPath = [System.IO.Path]::GetRelativePath($RootPath, $Path)
+    $relativeFolderPath = $relativeFolderPath.Split($separators, [System.StringSplitOptions]::RemoveEmptyEntries) | ForEach-Object { "[$_]" }
     $relativeFolderPath = $relativeFolderPath -join ' - '
 
     Add-Content -Path $RootModuleFilePath -Force -Value @"
@@ -36,12 +136,17 @@
 Write-Debug "[`$scriptName] - $relativeFolderPath - Processing folder"
 "@
 
-    $files = $Path | Get-ChildItem -File -Force -Filter '*.ps1' | Sort-Object -Property FullName
+    if ($DependencyAware) {
+        $files = $Path | Get-ChildItem -Recurse -File -Force -Filter '*.ps1' | Sort-Object -Property FullName
+        $files = Get-DependencyOrderedScriptFiles -Files $files
+    } else {
+        $files = $Path | Get-ChildItem -File -Force -Filter '*.ps1' | Sort-Object -Property FullName
+    }
+
     foreach ($file in $files) {
-        $relativeFilePath = $file.FullName -replace $RootPath, ''
-        $relativeFilePath = $relativeFilePath -replace $file.Extension, ''
-        $relativeFilePath = $relativeFilePath.TrimStart($pathSeparator)
-        $relativeFilePath = $relativeFilePath -split $pathSeparator | ForEach-Object { "[$_]" }
+        $relativeFilePath = [System.IO.Path]::GetRelativePath($RootPath, $file.FullName)
+        $relativeFilePath = [System.IO.Path]::ChangeExtension($relativeFilePath, $null).TrimEnd('.')
+        $relativeFilePath = $relativeFilePath.Split($separators, [System.StringSplitOptions]::RemoveEmptyEntries) | ForEach-Object { "[$_]" }
         $relativeFilePath = $relativeFilePath -join ' - '
 
         Add-Content -Path $RootModuleFilePath -Force -Value @"
@@ -55,9 +160,11 @@ Write-Debug "[`$scriptName] - $relativeFilePath - Done"
 "@
     }
 
-    $subFolders = $Path | Get-ChildItem -Directory -Force | Sort-Object -Property Name
-    foreach ($subFolder in $subFolders) {
-        Add-ContentFromItem -Path $subFolder.FullName -RootModuleFilePath $RootModuleFilePath -RootPath $RootPath
+    if (-not $DependencyAware) {
+        $subFolders = $Path | Get-ChildItem -Directory -Force | Sort-Object -Property Name
+        foreach ($subFolder in $subFolders) {
+            Add-ContentFromItem -Path $subFolder.FullName -RootModuleFilePath $RootModuleFilePath -RootPath $RootPath
+        }
     }
     Add-Content -Path $RootModuleFilePath -Force -Value @"
 Write-Debug "[`$scriptName] - $relativeFolderPath - Done"
diff --git a/.github/actions/Build-PSModule/src/helpers/Build/Build-PSModuleRootModule.ps1 b/.github/actions/Build-PSModule/src/helpers/Build/Build-PSModuleRootModule.ps1
index 02aaa984..4fa36797 100644
--- a/.github/actions/Build-PSModule/src/helpers/Build/Build-PSModuleRootModule.ps1
+++ b/.github/actions/Build-PSModule/src/helpers/Build/Build-PSModuleRootModule.ps1
@@ -10,15 +10,14 @@
 
         1. Module header from header.ps1 file. Usually to suppress code analysis warnings/errors and to add [CmdletBinding()] to the module.
         2. Data loader is added if data files are available.
-        3. Combines *.ps1 files from the following folders in alphabetical order from each folder:
+        3. Combines *.ps1 files from the following folders:
             1. init
-            2. classes/private
-            3. classes/public
-            4. functions/private
-            5. functions/public
-            6. variables/private
-            7. variables/public
-            8. Any remaining *.ps1 on module root.
+            2. classes (dependency-aware ordering for class/enum references)
+            3. functions/private
+            4. functions/public
+            5. variables/private
+            6. variables/public
+            7. Any remaining *.ps1 on module root.
         4. Adds a class loader for classes found in the classes/public folder.
         5. Export-ModuleMember by using the functions, cmdlets, variables and aliases found in the source files.
             - `Functions` will only contain functions that are from the `functions/public` folder.
@@ -48,8 +47,7 @@
         [System.IO.DirectoryInfo] $ModuleOutputFolder
     )
 
-    # Get the path separator for the current OS
-    $pathSeparator = [System.IO.Path]::DirectorySeparatorChar
+    $separators = [char[]]@([System.IO.Path]::DirectorySeparatorChar, [System.IO.Path]::AltDirectorySeparatorChar)
 
     Set-GitHubLogGroup 'Build root module' {
         $rootModuleFile = New-Item -Path $ModuleOutputFolder -Name "$ModuleName.psm1" -Force
@@ -176,8 +174,7 @@ Write-Debug "[$scriptName] - [data] - Done"
         #region - Add content from subfolders
         $scriptFoldersToProcess = @(
             'init',
-            'classes/private',
-            'classes/public',
+            'classes',
             'functions/private',
             'functions/public',
             'variables/private',
@@ -185,22 +182,22 @@ Write-Debug "[$scriptName] - [data] - Done"
         )
 
         foreach ($scriptFolder in $scriptFoldersToProcess) {
-            $scriptFolder = Join-Path -Path $ModuleOutputFolder -ChildPath $scriptFolder
-            if (-not (Test-Path -Path $scriptFolder)) {
+            $dependencyAware = $scriptFolder -eq 'classes'
+            $scriptFolderPath = Join-Path -Path $ModuleOutputFolder -ChildPath $scriptFolder
+            if (-not (Test-Path -Path $scriptFolderPath)) {
                 continue
             }
-            Add-ContentFromItem -Path $scriptFolder -RootModuleFilePath $rootModuleFile -RootPath $ModuleOutputFolder
-            Remove-Item -Path $scriptFolder -Force -Recurse
+            Add-ContentFromItem -Path $scriptFolderPath -RootModuleFilePath $rootModuleFile -RootPath $ModuleOutputFolder -DependencyAware:$dependencyAware
+            Remove-Item -Path $scriptFolderPath -Force -Recurse
         }
         #endregion - Add content from subfolders
 
         #region - Add content from *.ps1 files on module root
         $files = $ModuleOutputFolder | Get-ChildItem -File -Force -Filter '*.ps1' | Sort-Object -Property FullName
         foreach ($file in $files) {
-            $relativePath = $file.FullName -replace $ModuleOutputFolder, ''
-            $relativePath = $relativePath -replace $file.Extension, ''
-            $relativePath = $relativePath.TrimStart($pathSeparator)
-            $relativePath = $relativePath -split $pathSeparator | ForEach-Object { "[$_]" }
+            $relativePath = [System.IO.Path]::GetRelativePath($ModuleOutputFolder, $file.FullName)
+            $relativePath = [System.IO.Path]::ChangeExtension($relativePath, $null).TrimEnd('.')
+            $relativePath = $relativePath.Split($separators, [System.StringSplitOptions]::RemoveEmptyEntries) | ForEach-Object { "[$_]" }
             $relativePath = $relativePath -join ' - '
 
             Add-Content -Path $rootModuleFile -Force -Value @"
diff --git a/tests/srcTestRepo/src/classes/public/A-BookList.ps1 b/tests/srcTestRepo/src/classes/public/A-BookList.ps1
new file mode 100644
index 00000000..68a96673
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/A-BookList.ps1
@@ -0,0 +1,86 @@
+class BookList {
+    # Static property to hold the list of books
+    static [System.Collections.Generic.List[Book]] $Books
+    # Static method to initialize the list of books. Called in the other
+    # static methods to avoid needing to explicit initialize the value.
+    static [void] Initialize() { [BookList]::Initialize($false) }
+    static [bool] Initialize([bool]$force) {
+        if ([BookList]::Books.Count -gt 0 -and -not $force) {
+            return $false
+        }
+
+        [BookList]::Books = [System.Collections.Generic.List[Book]]::new()
+
+        return $true
+    }
+    # Ensure a book is valid for the list.
+    static [void] Validate([book]$Book) {
+        $Prefix = @(
+            'Book validation failed: Book must be defined with the Title,'
+            'Author, and PublishDate properties, but'
+        ) -join ' '
+        if ($null -eq $Book) { throw "$Prefix was null" }
+        if ([string]::IsNullOrEmpty($Book.Title)) {
+            throw "$Prefix Title wasn't defined"
+        }
+        if ([string]::IsNullOrEmpty($Book.Author)) {
+            throw "$Prefix Author wasn't defined"
+        }
+        if ([datetime]::MinValue -eq $Book.PublishDate) {
+            throw "$Prefix PublishDate wasn't defined"
+        }
+    }
+    # Static methods to manage the list of books.
+    # Add a book if it's not already in the list.
+    static [void] Add([Book]$Book) {
+        [BookList]::Initialize()
+        [BookList]::Validate($Book)
+        if ([BookList]::Books.Contains($Book)) {
+            throw "Book '$Book' already in list"
+        }
+
+        $FindPredicate = {
+            param([Book]$b)
+
+            $b.Title -eq $Book.Title -and
+            $b.Author -eq $Book.Author -and
+            $b.PublishDate -eq $Book.PublishDate
+        }.GetNewClosure()
+        if ([BookList]::Books.Find($FindPredicate)) {
+            throw "Book '$Book' already in list"
+        }
+
+        [BookList]::Books.Add($Book)
+    }
+    # Clear the list of books.
+    static [void] Clear() {
+        [BookList]::Initialize()
+        [BookList]::Books.Clear()
+    }
+    # Find a specific book using a filtering scriptblock.
+    static [Book] Find([scriptblock]$Predicate) {
+        [BookList]::Initialize()
+        return [BookList]::Books.Find($Predicate)
+    }
+    # Find every book matching the filtering scriptblock.
+    static [Book[]] FindAll([scriptblock]$Predicate) {
+        [BookList]::Initialize()
+        return [BookList]::Books.FindAll($Predicate)
+    }
+    # Remove a specific book.
+    static [void] Remove([Book]$Book) {
+        [BookList]::Initialize()
+        [BookList]::Books.Remove($Book)
+    }
+    # Remove a book by property value.
+    static [void] RemoveBy([string]$Property, [string]$Value) {
+        [BookList]::Initialize()
+        $Index = [BookList]::Books.FindIndex({
+                param($b)
+                $b.$Property -eq $Value
+            }.GetNewClosure())
+        if ($Index -ge 0) {
+            [BookList]::Books.RemoveAt($Index)
+        }
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/Book.ps1 b/tests/srcTestRepo/src/classes/public/Book.ps1
index 8917d9ab..a3c7a7ca 100644
--- a/tests/srcTestRepo/src/classes/public/Book.ps1
+++ b/tests/srcTestRepo/src/classes/public/Book.ps1
@@ -44,93 +44,6 @@
     }
 }
 
-class BookList {
-    # Static property to hold the list of books
-    static [System.Collections.Generic.List[Book]] $Books
-    # Static method to initialize the list of books. Called in the other
-    # static methods to avoid needing to explicit initialize the value.
-    static [void] Initialize() { [BookList]::Initialize($false) }
-    static [bool] Initialize([bool]$force) {
-        if ([BookList]::Books.Count -gt 0 -and -not $force) {
-            return $false
-        }
-
-        [BookList]::Books = [System.Collections.Generic.List[Book]]::new()
-
-        return $true
-    }
-    # Ensure a book is valid for the list.
-    static [void] Validate([book]$Book) {
-        $Prefix = @(
-            'Book validation failed: Book must be defined with the Title,'
-            'Author, and PublishDate properties, but'
-        ) -join ' '
-        if ($null -eq $Book) { throw "$Prefix was null" }
-        if ([string]::IsNullOrEmpty($Book.Title)) {
-            throw "$Prefix Title wasn't defined"
-        }
-        if ([string]::IsNullOrEmpty($Book.Author)) {
-            throw "$Prefix Author wasn't defined"
-        }
-        if ([datetime]::MinValue -eq $Book.PublishDate) {
-            throw "$Prefix PublishDate wasn't defined"
-        }
-    }
-    # Static methods to manage the list of books.
-    # Add a book if it's not already in the list.
-    static [void] Add([Book]$Book) {
-        [BookList]::Initialize()
-        [BookList]::Validate($Book)
-        if ([BookList]::Books.Contains($Book)) {
-            throw "Book '$Book' already in list"
-        }
-
-        $FindPredicate = {
-            param([Book]$b)
-
-            $b.Title -eq $Book.Title -and
-            $b.Author -eq $Book.Author -and
-            $b.PublishDate -eq $Book.PublishDate
-        }.GetNewClosure()
-        if ([BookList]::Books.Find($FindPredicate)) {
-            throw "Book '$Book' already in list"
-        }
-
-        [BookList]::Books.Add($Book)
-    }
-    # Clear the list of books.
-    static [void] Clear() {
-        [BookList]::Initialize()
-        [BookList]::Books.Clear()
-    }
-    # Find a specific book using a filtering scriptblock.
-    static [Book] Find([scriptblock]$Predicate) {
-        [BookList]::Initialize()
-        return [BookList]::Books.Find($Predicate)
-    }
-    # Find every book matching the filtering scriptblock.
-    static [Book[]] FindAll([scriptblock]$Predicate) {
-        [BookList]::Initialize()
-        return [BookList]::Books.FindAll($Predicate)
-    }
-    # Remove a specific book.
-    static [void] Remove([Book]$Book) {
-        [BookList]::Initialize()
-        [BookList]::Books.Remove($Book)
-    }
-    # Remove a book by property value.
-    static [void] RemoveBy([string]$Property, [string]$Value) {
-        [BookList]::Initialize()
-        $Index = [BookList]::Books.FindIndex({
-                param($b)
-                $b.$Property -eq $Value
-            }.GetNewClosure())
-        if ($Index -ge 0) {
-            [BookList]::Books.RemoveAt($Index)
-        }
-    }
-}
-
 enum Binding {
     Hardcover
     Paperback
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/A-BookList.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/A-BookList.ps1
new file mode 100644
index 00000000..68a96673
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/A-BookList.ps1
@@ -0,0 +1,86 @@
+class BookList {
+    # Static property to hold the list of books
+    static [System.Collections.Generic.List[Book]] $Books
+    # Static method to initialize the list of books. Called in the other
+    # static methods to avoid needing to explicit initialize the value.
+    static [void] Initialize() { [BookList]::Initialize($false) }
+    static [bool] Initialize([bool]$force) {
+        if ([BookList]::Books.Count -gt 0 -and -not $force) {
+            return $false
+        }
+
+        [BookList]::Books = [System.Collections.Generic.List[Book]]::new()
+
+        return $true
+    }
+    # Ensure a book is valid for the list.
+    static [void] Validate([book]$Book) {
+        $Prefix = @(
+            'Book validation failed: Book must be defined with the Title,'
+            'Author, and PublishDate properties, but'
+        ) -join ' '
+        if ($null -eq $Book) { throw "$Prefix was null" }
+        if ([string]::IsNullOrEmpty($Book.Title)) {
+            throw "$Prefix Title wasn't defined"
+        }
+        if ([string]::IsNullOrEmpty($Book.Author)) {
+            throw "$Prefix Author wasn't defined"
+        }
+        if ([datetime]::MinValue -eq $Book.PublishDate) {
+            throw "$Prefix PublishDate wasn't defined"
+        }
+    }
+    # Static methods to manage the list of books.
+    # Add a book if it's not already in the list.
+    static [void] Add([Book]$Book) {
+        [BookList]::Initialize()
+        [BookList]::Validate($Book)
+        if ([BookList]::Books.Contains($Book)) {
+            throw "Book '$Book' already in list"
+        }
+
+        $FindPredicate = {
+            param([Book]$b)
+
+            $b.Title -eq $Book.Title -and
+            $b.Author -eq $Book.Author -and
+            $b.PublishDate -eq $Book.PublishDate
+        }.GetNewClosure()
+        if ([BookList]::Books.Find($FindPredicate)) {
+            throw "Book '$Book' already in list"
+        }
+
+        [BookList]::Books.Add($Book)
+    }
+    # Clear the list of books.
+    static [void] Clear() {
+        [BookList]::Initialize()
+        [BookList]::Books.Clear()
+    }
+    # Find a specific book using a filtering scriptblock.
+    static [Book] Find([scriptblock]$Predicate) {
+        [BookList]::Initialize()
+        return [BookList]::Books.Find($Predicate)
+    }
+    # Find every book matching the filtering scriptblock.
+    static [Book[]] FindAll([scriptblock]$Predicate) {
+        [BookList]::Initialize()
+        return [BookList]::Books.FindAll($Predicate)
+    }
+    # Remove a specific book.
+    static [void] Remove([Book]$Book) {
+        [BookList]::Initialize()
+        [BookList]::Books.Remove($Book)
+    }
+    # Remove a book by property value.
+    static [void] RemoveBy([string]$Property, [string]$Value) {
+        [BookList]::Initialize()
+        $Index = [BookList]::Books.FindIndex({
+                param($b)
+                $b.$Property -eq $Value
+            }.GetNewClosure())
+        if ($Index -ge 0) {
+            [BookList]::Books.RemoveAt($Index)
+        }
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/Book.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/Book.ps1
index 8917d9ab..a3c7a7ca 100644
--- a/tests/srcWithManifestTestRepo/src/classes/public/Book.ps1
+++ b/tests/srcWithManifestTestRepo/src/classes/public/Book.ps1
@@ -44,93 +44,6 @@
     }
 }
 
-class BookList {
-    # Static property to hold the list of books
-    static [System.Collections.Generic.List[Book]] $Books
-    # Static method to initialize the list of books. Called in the other
-    # static methods to avoid needing to explicit initialize the value.
-    static [void] Initialize() { [BookList]::Initialize($false) }
-    static [bool] Initialize([bool]$force) {
-        if ([BookList]::Books.Count -gt 0 -and -not $force) {
-            return $false
-        }
-
-        [BookList]::Books = [System.Collections.Generic.List[Book]]::new()
-
-        return $true
-    }
-    # Ensure a book is valid for the list.
-    static [void] Validate([book]$Book) {
-        $Prefix = @(
-            'Book validation failed: Book must be defined with the Title,'
-            'Author, and PublishDate properties, but'
-        ) -join ' '
-        if ($null -eq $Book) { throw "$Prefix was null" }
-        if ([string]::IsNullOrEmpty($Book.Title)) {
-            throw "$Prefix Title wasn't defined"
-        }
-        if ([string]::IsNullOrEmpty($Book.Author)) {
-            throw "$Prefix Author wasn't defined"
-        }
-        if ([datetime]::MinValue -eq $Book.PublishDate) {
-            throw "$Prefix PublishDate wasn't defined"
-        }
-    }
-    # Static methods to manage the list of books.
-    # Add a book if it's not already in the list.
-    static [void] Add([Book]$Book) {
-        [BookList]::Initialize()
-        [BookList]::Validate($Book)
-        if ([BookList]::Books.Contains($Book)) {
-            throw "Book '$Book' already in list"
-        }
-
-        $FindPredicate = {
-            param([Book]$b)
-
-            $b.Title -eq $Book.Title -and
-            $b.Author -eq $Book.Author -and
-            $b.PublishDate -eq $Book.PublishDate
-        }.GetNewClosure()
-        if ([BookList]::Books.Find($FindPredicate)) {
-            throw "Book '$Book' already in list"
-        }
-
-        [BookList]::Books.Add($Book)
-    }
-    # Clear the list of books.
-    static [void] Clear() {
-        [BookList]::Initialize()
-        [BookList]::Books.Clear()
-    }
-    # Find a specific book using a filtering scriptblock.
-    static [Book] Find([scriptblock]$Predicate) {
-        [BookList]::Initialize()
-        return [BookList]::Books.Find($Predicate)
-    }
-    # Find every book matching the filtering scriptblock.
-    static [Book[]] FindAll([scriptblock]$Predicate) {
-        [BookList]::Initialize()
-        return [BookList]::Books.FindAll($Predicate)
-    }
-    # Remove a specific book.
-    static [void] Remove([Book]$Book) {
-        [BookList]::Initialize()
-        [BookList]::Books.Remove($Book)
-    }
-    # Remove a book by property value.
-    static [void] RemoveBy([string]$Property, [string]$Value) {
-        [BookList]::Initialize()
-        $Index = [BookList]::Books.FindIndex({
-                param($b)
-                $b.$Property -eq $Value
-            }.GetNewClosure())
-        if ($Index -ge 0) {
-            [BookList]::Books.RemoveAt($Index)
-        }
-    }
-}
-
 enum Binding {
     Hardcover
     Paperback

From 1ff1a90d31ae9d9c616deebb813e6052449da7d8 Mon Sep 17 00:00:00 2001
From: Marius Storhaug <marstor@hotmail.com>
Date: Sat, 18 Jul 2026 20:45:07 +0200
Subject: [PATCH 02/13] Extract dependency ordering helper script

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 .../src/helpers/Build/Add-ContentFromItem.ps1 | 99 -------------------
 .../Get-DependencyOrderedScriptFiles.ps1      | 98 ++++++++++++++++++
 2 files changed, 98 insertions(+), 99 deletions(-)
 create mode 100644 .github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1

diff --git a/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1 b/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1
index 1b4e0c8d..c3aa61bb 100644
--- a/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1
+++ b/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1
@@ -1,102 +1,3 @@
-function Get-DependencyOrderedScriptFiles {
-    [OutputType([System.IO.FileInfo[]])]
-    [CmdletBinding()]
-    param(
-        [Parameter(Mandatory)]
-        [System.IO.FileInfo[]] $Files
-    )
-
-    $sortedFiles = $Files | Sort-Object -Property FullName
-    if ($sortedFiles.Count -le 1) {
-        return $sortedFiles
-    }
-
-    $metadataByPath = @{}
-    $typeToPath = @{}
-    $duplicateTypes = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
-
-    foreach ($file in $sortedFiles) {
-        $content = Get-Content -Path $file.FullName -Raw
-        $declaredTypes = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
-        [Regex]::Matches($content, '(?im)^\s*(class|enum)\s+([^\s:{]+)') | ForEach-Object {
-            [void] $declaredTypes.Add($_.Groups[2].Value)
-        }
-
-        $metadataByPath[$file.FullName] = [pscustomobject]@{
-            File          = $file
-            Content       = $content
-            DeclaredTypes = $declaredTypes
-        }
-
-        foreach ($typeName in $declaredTypes) {
-            if ($typeToPath.ContainsKey($typeName)) {
-                [void]$duplicateTypes.Add($typeName)
-                continue
-            }
-
-            $typeToPath[$typeName] = $file.FullName
-        }
-    }
-
-    $dependenciesByPath = @{}
-    $dependentsByPath = @{}
-    foreach ($file in $sortedFiles) {
-        $dependenciesByPath[$file.FullName] = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
-        $dependentsByPath[$file.FullName] = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
-    }
-
-    foreach ($metadata in $metadataByPath.Values) {
-        foreach ($typeName in $typeToPath.Keys) {
-            if ($duplicateTypes.Contains($typeName)) {
-                continue
-            }
-
-            if ($metadata.DeclaredTypes.Contains($typeName)) {
-                continue
-            }
-
-            $typePattern = "(?<![\w\.])\[$([Regex]::Escape($typeName))\](?![\w\.])"
-            if ([Regex]::IsMatch($metadata.Content, $typePattern, [System.Text.RegularExpressions.RegexOptions]::IgnoreCase)) {
-                $dependencyPath = $typeToPath[$typeName]
-                [void]$dependenciesByPath[$metadata.File.FullName].Add($dependencyPath)
-                [void]$dependentsByPath[$dependencyPath].Add($metadata.File.FullName)
-            }
-        }
-    }
-
-    $remainingPaths = @($sortedFiles.FullName)
-    $ready = @(
-        $remainingPaths |
-        Where-Object { $dependenciesByPath[$_].Count -eq 0 } |
-        Sort-Object
-    )
-    $orderedPaths = [System.Collections.Generic.List[string]]::new()
-
-    while ($ready.Count -gt 0) {
-        $currentPath = $ready[0]
-        $ready = @($ready | Select-Object -Skip 1)
-        $orderedPaths.Add($currentPath)
-        $remainingPaths = @($remainingPaths | Where-Object { $_ -ne $currentPath })
-
-        $dependents = @($dependentsByPath[$currentPath] | Sort-Object)
-        foreach ($dependentPath in $dependents) {
-            $null = $dependenciesByPath[$dependentPath].Remove($currentPath)
-            if ($dependenciesByPath[$dependentPath].Count -eq 0) {
-                $ready = @($ready + $dependentPath | Sort-Object -Unique)
-            }
-        }
-    }
-
-    if ($orderedPaths.Count -lt $sortedFiles.Count) {
-        Write-Warning "Detected cyclical or unresolved class/enum dependencies. Falling back to lexical order for remaining files in [$($sortedFiles[0].DirectoryName)]."
-        $remainingPaths | Sort-Object | ForEach-Object {
-            $orderedPaths.Add($_)
-        }
-    }
-
-    return @($orderedPaths | ForEach-Object { $metadataByPath[$_].File })
-}
-
 function Add-ContentFromItem {
     <#
         .SYNOPSIS
diff --git a/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1 b/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1
new file mode 100644
index 00000000..45bc1f6e
--- /dev/null
+++ b/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1
@@ -0,0 +1,98 @@
+function Get-DependencyOrderedScriptFiles {
+    [OutputType([System.IO.FileInfo[]])]
+    [CmdletBinding()]
+    param(
+        [Parameter(Mandatory)]
+        [System.IO.FileInfo[]] $Files
+    )
+
+    $sortedFiles = $Files | Sort-Object -Property FullName
+    if ($sortedFiles.Count -le 1) {
+        return $sortedFiles
+    }
+
+    $metadataByPath = @{}
+    $typeToPath = @{}
+    $duplicateTypes = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
+
+    foreach ($file in $sortedFiles) {
+        $content = Get-Content -Path $file.FullName -Raw
+        $declaredTypes = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
+        [Regex]::Matches($content, '(?im)^\s*(class|enum)\s+([^\s:{]+)') | ForEach-Object {
+            [void] $declaredTypes.Add($_.Groups[2].Value)
+        }
+
+        $metadataByPath[$file.FullName] = [pscustomobject]@{
+            File          = $file
+            Content       = $content
+            DeclaredTypes = $declaredTypes
+        }
+
+        foreach ($typeName in $declaredTypes) {
+            if ($typeToPath.ContainsKey($typeName)) {
+                [void]$duplicateTypes.Add($typeName)
+                continue
+            }
+
+            $typeToPath[$typeName] = $file.FullName
+        }
+    }
+
+    $dependenciesByPath = @{}
+    $dependentsByPath = @{}
+    foreach ($file in $sortedFiles) {
+        $dependenciesByPath[$file.FullName] = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
+        $dependentsByPath[$file.FullName] = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
+    }
+
+    foreach ($metadata in $metadataByPath.Values) {
+        foreach ($typeName in $typeToPath.Keys) {
+            if ($duplicateTypes.Contains($typeName)) {
+                continue
+            }
+
+            if ($metadata.DeclaredTypes.Contains($typeName)) {
+                continue
+            }
+
+            $typePattern = "(?<![\w\.])\[$([Regex]::Escape($typeName))\](?![\w\.])"
+            if ([Regex]::IsMatch($metadata.Content, $typePattern, [System.Text.RegularExpressions.RegexOptions]::IgnoreCase)) {
+                $dependencyPath = $typeToPath[$typeName]
+                [void]$dependenciesByPath[$metadata.File.FullName].Add($dependencyPath)
+                [void]$dependentsByPath[$dependencyPath].Add($metadata.File.FullName)
+            }
+        }
+    }
+
+    $remainingPaths = @($sortedFiles.FullName)
+    $ready = @(
+        $remainingPaths |
+        Where-Object { $dependenciesByPath[$_].Count -eq 0 } |
+        Sort-Object
+    )
+    $orderedPaths = [System.Collections.Generic.List[string]]::new()
+
+    while ($ready.Count -gt 0) {
+        $currentPath = $ready[0]
+        $ready = @($ready | Select-Object -Skip 1)
+        $orderedPaths.Add($currentPath)
+        $remainingPaths = @($remainingPaths | Where-Object { $_ -ne $currentPath })
+
+        $dependents = @($dependentsByPath[$currentPath] | Sort-Object)
+        foreach ($dependentPath in $dependents) {
+            $null = $dependenciesByPath[$dependentPath].Remove($currentPath)
+            if ($dependenciesByPath[$dependentPath].Count -eq 0) {
+                $ready = @($ready + $dependentPath | Sort-Object -Unique)
+            }
+        }
+    }
+
+    if ($orderedPaths.Count -lt $sortedFiles.Count) {
+        Write-Warning "Detected cyclical or unresolved class/enum dependencies. Falling back to lexical order for remaining files in [$($sortedFiles[0].DirectoryName)]."
+        $remainingPaths | Sort-Object | ForEach-Object {
+            $orderedPaths.Add($_)
+        }
+    }
+
+    return @($orderedPaths | ForEach-Object { $metadataByPath[$_].File })
+}

From 3484520beb3aafcd66f4881f5c18b8050a7e412e Mon Sep 17 00:00:00 2001
From: Marius Storhaug <marstor@hotmail.com>
Date: Sat, 18 Jul 2026 20:49:30 +0200
Subject: [PATCH 03/13] Trigger workflow tests for action changes

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 .github/workflows/Workflow-Test-Default.yml      | 2 ++
 .github/workflows/Workflow-Test-WithManifest.yml | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/.github/workflows/Workflow-Test-Default.yml b/.github/workflows/Workflow-Test-Default.yml
index e5b012e5..511b4b1c 100644
--- a/.github/workflows/Workflow-Test-Default.yml
+++ b/.github/workflows/Workflow-Test-Default.yml
@@ -6,6 +6,7 @@ on:
   workflow_dispatch:
   pull_request:
     paths:
+      - '.github/actions/**'
       - '.github/workflows/**'
       - '!.github/workflows/Release.yml'
       - '!.github/workflows/Linter.yml'
@@ -43,4 +44,5 @@ jobs:
       ImportantFilePatterns: |
         ^src/
         ^README\.md$
+        ^\.github/actions/
         ^\.github/workflows/(?!Release\.yml$|Linter\.yml$)
diff --git a/.github/workflows/Workflow-Test-WithManifest.yml b/.github/workflows/Workflow-Test-WithManifest.yml
index 4b179d9a..82f80f55 100644
--- a/.github/workflows/Workflow-Test-WithManifest.yml
+++ b/.github/workflows/Workflow-Test-WithManifest.yml
@@ -6,6 +6,7 @@ on:
   workflow_dispatch:
   pull_request:
     paths:
+      - '.github/actions/**'
       - '.github/workflows/**'
       - '!.github/workflows/Release.yml'
       - '!.github/workflows/Linter.yml'
@@ -43,4 +44,5 @@ jobs:
       ImportantFilePatterns: |
         ^src/
         ^README\.md$
+        ^\.github/actions/
         ^\.github/workflows/(?!Release\.yml$|Linter\.yml$)

From 94c68b98c2e2a5a0ef4e28e598857e9566977cc6 Mon Sep 17 00:00:00 2001
From: Marius Storhaug <marstor@hotmail.com>
Date: Sat, 18 Jul 2026 20:50:19 +0200
Subject: [PATCH 04/13] Run release workflow for action changes

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 .github/workflows/Release.yml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.github/workflows/Release.yml b/.github/workflows/Release.yml
index a845c2a4..1e346454 100644
--- a/.github/workflows/Release.yml
+++ b/.github/workflows/Release.yml
@@ -13,6 +13,7 @@ on:
       - synchronize
       - labeled
     paths:
+      - '.github/actions/**'
       - '.github/workflows/**'
       - '!.github/workflows/Release.yml'
       - '!.github/workflows/Linter.yml'

From 676c55c539fc477c251eb3ae5446a12925521cf6 Mon Sep 17 00:00:00 2001
From: Marius Storhaug <marstor@hotmail.com>
Date: Sat, 18 Jul 2026 20:50:59 +0200
Subject: [PATCH 05/13] Treat action files as important by default

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 .github/workflows/Plan.yml     | 3 ++-
 .github/workflows/workflow.yml | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/Plan.yml b/.github/workflows/Plan.yml
index 442e14fe..d349b1c1 100644
--- a/.github/workflows/Plan.yml
+++ b/.github/workflows/Plan.yml
@@ -44,11 +44,12 @@ on:
         description: |
           Newline-separated list of regex patterns that identify important files.
           Changes matching these patterns trigger build, test, and publish stages.
-          When set, fully replaces the defaults (^src/ and ^README\.md$).
+          When set, fully replaces the defaults (^src/, ^README\.md$, and ^\.github/actions/).
         required: false
         default: |
           ^src/
           ^README\.md$
+          ^\.github/actions/
 
     outputs:
       Settings:
diff --git a/.github/workflows/workflow.yml b/.github/workflows/workflow.yml
index 1b9839a2..b0a6ed46 100644
--- a/.github/workflows/workflow.yml
+++ b/.github/workflows/workflow.yml
@@ -56,11 +56,12 @@ on:
         description: |
           Newline-separated list of regex patterns that identify important files.
           Changes matching these patterns trigger build, test, and publish stages.
-          When set, fully replaces the defaults (^src/ and ^README\.md$).
+          When set, fully replaces the defaults (^src/, ^README\.md$, and ^\.github/actions/).
         required: false
         default: |
           ^src/
           ^README\.md$
+          ^\.github/actions/
 
 permissions:
   contents: write # to checkout the repo and create releases on the repo

From eb844428da7a472456f12fea2c47a234e1673227 Mon Sep 17 00:00:00 2001
From: Marius Storhaug <marstor@hotmail.com>
Date: Sat, 18 Jul 2026 20:52:01 +0200
Subject: [PATCH 06/13] Keep default important patterns consumer-focused

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 .github/workflows/Plan.yml     | 3 +--
 .github/workflows/workflow.yml | 3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/Plan.yml b/.github/workflows/Plan.yml
index d349b1c1..442e14fe 100644
--- a/.github/workflows/Plan.yml
+++ b/.github/workflows/Plan.yml
@@ -44,12 +44,11 @@ on:
         description: |
           Newline-separated list of regex patterns that identify important files.
           Changes matching these patterns trigger build, test, and publish stages.
-          When set, fully replaces the defaults (^src/, ^README\.md$, and ^\.github/actions/).
+          When set, fully replaces the defaults (^src/ and ^README\.md$).
         required: false
         default: |
           ^src/
           ^README\.md$
-          ^\.github/actions/
 
     outputs:
       Settings:
diff --git a/.github/workflows/workflow.yml b/.github/workflows/workflow.yml
index b0a6ed46..1b9839a2 100644
--- a/.github/workflows/workflow.yml
+++ b/.github/workflows/workflow.yml
@@ -56,12 +56,11 @@ on:
         description: |
           Newline-separated list of regex patterns that identify important files.
           Changes matching these patterns trigger build, test, and publish stages.
-          When set, fully replaces the defaults (^src/, ^README\.md$, and ^\.github/actions/).
+          When set, fully replaces the defaults (^src/ and ^README\.md$).
         required: false
         default: |
           ^src/
           ^README\.md$
-          ^\.github/actions/
 
 permissions:
   contents: write # to checkout the repo and create releases on the repo

From 617de7406816c0446f6c3bfc8baf3ee210e7a6cd Mon Sep 17 00:00:00 2001
From: Marius Storhaug <marstor@hotmail.com>
Date: Sat, 18 Jul 2026 20:57:45 +0200
Subject: [PATCH 07/13] Expand class dependency fixtures for loader stress

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 .../Get-DependencyOrderedScriptFiles.ps1      |  2 +-
 .../src/classes/public/A-BookList.ps1         | 86 -------------------
 .../src/classes/public/ActiveLoanRegister.ps1 | 11 +++
 .../src/classes/public/AuthorIndex.ps1        | 11 +++
 .../src/classes/public/AuthorProfile.ps1      |  9 ++
 .../src/classes/public/BookCopy.ps1           | 11 +++
 .../src/classes/public/BookInventory.ps1      | 11 +++
 .../src/classes/public/BookList.ps1           | 57 ++++++++++++
 .../src/classes/public/BookLoan.ps1           | 13 +++
 .../src/classes/public/BranchLibrary.ps1      | 11 +++
 .../src/classes/public/CatalogSection.ps1     |  9 ++
 .../src/classes/public/DueDatePolicy.ps1      |  9 ++
 .../src/classes/public/LibraryReport.ps1      | 18 ++++
 .../src/classes/public/MemberAccount.ps1      | 11 +++
 .../src/classes/public/OpeningHours.ps1       |  9 ++
 .../src/classes/public/ReadingList.ps1        |  7 ++
 .../src/classes/public/ShelfLocation.ps1      | 11 +++
 .../src/classes/public/ZCatalog.ps1           | 11 +++
 .../src/classes/public/A-BookList.ps1         | 86 -------------------
 .../src/classes/public/ActiveLoanRegister.ps1 | 11 +++
 .../src/classes/public/AuthorIndex.ps1        | 11 +++
 .../src/classes/public/AuthorProfile.ps1      |  9 ++
 .../src/classes/public/BookCopy.ps1           | 11 +++
 .../src/classes/public/BookInventory.ps1      | 11 +++
 .../src/classes/public/BookList.ps1           | 57 ++++++++++++
 .../src/classes/public/BookLoan.ps1           | 13 +++
 .../src/classes/public/BranchLibrary.ps1      | 11 +++
 .../src/classes/public/CatalogSection.ps1     |  9 ++
 .../src/classes/public/DueDatePolicy.ps1      |  9 ++
 .../src/classes/public/LibraryReport.ps1      | 18 ++++
 .../src/classes/public/MemberAccount.ps1      | 11 +++
 .../src/classes/public/OpeningHours.ps1       |  9 ++
 .../src/classes/public/ReadingList.ps1        |  7 ++
 .../src/classes/public/ShelfLocation.ps1      | 11 +++
 .../src/classes/public/ZCatalog.ps1           | 11 +++
 35 files changed, 439 insertions(+), 173 deletions(-)
 delete mode 100644 tests/srcTestRepo/src/classes/public/A-BookList.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/ActiveLoanRegister.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/AuthorIndex.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/AuthorProfile.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/BookCopy.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/BookInventory.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/BookList.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/BookLoan.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/BranchLibrary.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/CatalogSection.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/DueDatePolicy.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/LibraryReport.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/MemberAccount.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/OpeningHours.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/ReadingList.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/ShelfLocation.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/ZCatalog.ps1
 delete mode 100644 tests/srcWithManifestTestRepo/src/classes/public/A-BookList.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/ActiveLoanRegister.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/AuthorIndex.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/AuthorProfile.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/BookCopy.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/BookInventory.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/BookList.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/BookLoan.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/BranchLibrary.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/CatalogSection.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/DueDatePolicy.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/LibraryReport.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/MemberAccount.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/OpeningHours.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/ReadingList.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/ShelfLocation.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/ZCatalog.ps1

diff --git a/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1 b/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1
index 45bc1f6e..6f2e7ac4 100644
--- a/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1
+++ b/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1
@@ -55,7 +55,7 @@ function Get-DependencyOrderedScriptFiles {
                 continue
             }
 
-            $typePattern = "(?<![\w\.])\[$([Regex]::Escape($typeName))\](?![\w\.])"
+            $typePattern = "(?<![\w\.])\[$([Regex]::Escape($typeName))(?:\[\])?\](?![\w\.])"
             if ([Regex]::IsMatch($metadata.Content, $typePattern, [System.Text.RegularExpressions.RegexOptions]::IgnoreCase)) {
                 $dependencyPath = $typeToPath[$typeName]
                 [void]$dependenciesByPath[$metadata.File.FullName].Add($dependencyPath)
diff --git a/tests/srcTestRepo/src/classes/public/A-BookList.ps1 b/tests/srcTestRepo/src/classes/public/A-BookList.ps1
deleted file mode 100644
index 68a96673..00000000
--- a/tests/srcTestRepo/src/classes/public/A-BookList.ps1
+++ /dev/null
@@ -1,86 +0,0 @@
-class BookList {
-    # Static property to hold the list of books
-    static [System.Collections.Generic.List[Book]] $Books
-    # Static method to initialize the list of books. Called in the other
-    # static methods to avoid needing to explicit initialize the value.
-    static [void] Initialize() { [BookList]::Initialize($false) }
-    static [bool] Initialize([bool]$force) {
-        if ([BookList]::Books.Count -gt 0 -and -not $force) {
-            return $false
-        }
-
-        [BookList]::Books = [System.Collections.Generic.List[Book]]::new()
-
-        return $true
-    }
-    # Ensure a book is valid for the list.
-    static [void] Validate([book]$Book) {
-        $Prefix = @(
-            'Book validation failed: Book must be defined with the Title,'
-            'Author, and PublishDate properties, but'
-        ) -join ' '
-        if ($null -eq $Book) { throw "$Prefix was null" }
-        if ([string]::IsNullOrEmpty($Book.Title)) {
-            throw "$Prefix Title wasn't defined"
-        }
-        if ([string]::IsNullOrEmpty($Book.Author)) {
-            throw "$Prefix Author wasn't defined"
-        }
-        if ([datetime]::MinValue -eq $Book.PublishDate) {
-            throw "$Prefix PublishDate wasn't defined"
-        }
-    }
-    # Static methods to manage the list of books.
-    # Add a book if it's not already in the list.
-    static [void] Add([Book]$Book) {
-        [BookList]::Initialize()
-        [BookList]::Validate($Book)
-        if ([BookList]::Books.Contains($Book)) {
-            throw "Book '$Book' already in list"
-        }
-
-        $FindPredicate = {
-            param([Book]$b)
-
-            $b.Title -eq $Book.Title -and
-            $b.Author -eq $Book.Author -and
-            $b.PublishDate -eq $Book.PublishDate
-        }.GetNewClosure()
-        if ([BookList]::Books.Find($FindPredicate)) {
-            throw "Book '$Book' already in list"
-        }
-
-        [BookList]::Books.Add($Book)
-    }
-    # Clear the list of books.
-    static [void] Clear() {
-        [BookList]::Initialize()
-        [BookList]::Books.Clear()
-    }
-    # Find a specific book using a filtering scriptblock.
-    static [Book] Find([scriptblock]$Predicate) {
-        [BookList]::Initialize()
-        return [BookList]::Books.Find($Predicate)
-    }
-    # Find every book matching the filtering scriptblock.
-    static [Book[]] FindAll([scriptblock]$Predicate) {
-        [BookList]::Initialize()
-        return [BookList]::Books.FindAll($Predicate)
-    }
-    # Remove a specific book.
-    static [void] Remove([Book]$Book) {
-        [BookList]::Initialize()
-        [BookList]::Books.Remove($Book)
-    }
-    # Remove a book by property value.
-    static [void] RemoveBy([string]$Property, [string]$Value) {
-        [BookList]::Initialize()
-        $Index = [BookList]::Books.FindIndex({
-                param($b)
-                $b.$Property -eq $Value
-            }.GetNewClosure())
-        if ($Index -ge 0) {
-            [BookList]::Books.RemoveAt($Index)
-        }
-    }
-}
diff --git a/tests/srcTestRepo/src/classes/public/ActiveLoanRegister.ps1 b/tests/srcTestRepo/src/classes/public/ActiveLoanRegister.ps1
new file mode 100644
index 00000000..fb4e98cd
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/ActiveLoanRegister.ps1
@@ -0,0 +1,11 @@
+class ActiveLoanRegister {
+    [System.Collections.Generic.List[BookLoan]] $Loans
+
+    ActiveLoanRegister() {
+        $this.Loans = [System.Collections.Generic.List[BookLoan]]::new()
+    }
+
+    [void] AddLoan([BookLoan]$Loan) {
+        $this.Loans.Add($Loan)
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/AuthorIndex.ps1 b/tests/srcTestRepo/src/classes/public/AuthorIndex.ps1
new file mode 100644
index 00000000..c379af92
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/AuthorIndex.ps1
@@ -0,0 +1,11 @@
+class AuthorIndex {
+    [string] $Name
+    [AuthorProfile[]] $Authors
+    [ZCatalog] $Catalog
+
+    AuthorIndex([string]$Name, [ZCatalog]$Catalog) {
+        $this.Name = $Name
+        $this.Catalog = $Catalog
+        $this.Authors = @()
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/AuthorProfile.ps1 b/tests/srcTestRepo/src/classes/public/AuthorProfile.ps1
new file mode 100644
index 00000000..6a86fa35
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/AuthorProfile.ps1
@@ -0,0 +1,9 @@
+class AuthorProfile {
+    [string] $Name
+    [Book[]] $PublishedBooks
+
+    AuthorProfile([string]$Name, [Book[]]$PublishedBooks) {
+        $this.Name = $Name
+        $this.PublishedBooks = $PublishedBooks
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/BookCopy.ps1 b/tests/srcTestRepo/src/classes/public/BookCopy.ps1
new file mode 100644
index 00000000..bee184b1
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/BookCopy.ps1
@@ -0,0 +1,11 @@
+class BookCopy {
+    [string] $InventoryId
+    [Book] $Book
+    [ShelfLocation] $Location
+
+    BookCopy([string]$InventoryId, [Book]$Book, [ShelfLocation]$Location) {
+        $this.InventoryId = $InventoryId
+        $this.Book = $Book
+        $this.Location = $Location
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/BookInventory.ps1 b/tests/srcTestRepo/src/classes/public/BookInventory.ps1
new file mode 100644
index 00000000..cf7948e3
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/BookInventory.ps1
@@ -0,0 +1,11 @@
+class BookInventory {
+    [string] $Name
+    [BookCopy[]] $Copies
+    [ZCatalog] $Catalog
+
+    BookInventory([string]$Name, [ZCatalog]$Catalog) {
+        $this.Name = $Name
+        $this.Catalog = $Catalog
+        $this.Copies = @()
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/BookList.ps1 b/tests/srcTestRepo/src/classes/public/BookList.ps1
new file mode 100644
index 00000000..bf421618
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/BookList.ps1
@@ -0,0 +1,57 @@
+class BookList {
+    static [System.Collections.Generic.List[Book]] $Books
+
+    static [void] Initialize() { [BookList]::Initialize($false) }
+    static [bool] Initialize([bool]$force) {
+        if ([BookList]::Books.Count -gt 0 -and -not $force) {
+            return $false
+        }
+
+        [BookList]::Books = [System.Collections.Generic.List[Book]]::new()
+
+        return $true
+    }
+
+    static [void] Validate([Book]$Book) {
+        $Prefix = @(
+            'Book validation failed: Book must be defined with the Title,'
+            'Author, and PublishDate properties, but'
+        ) -join ' '
+        if ($null -eq $Book) { throw "$Prefix was null" }
+        if ([string]::IsNullOrEmpty($Book.Title)) { throw "$Prefix Title wasn't defined" }
+        if ([string]::IsNullOrEmpty($Book.Author)) { throw "$Prefix Author wasn't defined" }
+        if ([datetime]::MinValue -eq $Book.PublishDate) { throw "$Prefix PublishDate wasn't defined" }
+    }
+
+    static [void] Add([Book]$Book) {
+        [BookList]::Initialize()
+        [BookList]::Validate($Book)
+        if ([BookList]::Books.Contains($Book)) { throw "Book '$Book' already in list" }
+
+        $findPredicate = {
+            param([Book]$b)
+
+            $b.Title -eq $Book.Title -and
+            $b.Author -eq $Book.Author -and
+            $b.PublishDate -eq $Book.PublishDate
+        }.GetNewClosure()
+        if ([BookList]::Books.Find($findPredicate)) { throw "Book '$Book' already in list" }
+
+        [BookList]::Books.Add($Book)
+    }
+
+    static [void] Clear() {
+        [BookList]::Initialize()
+        [BookList]::Books.Clear()
+    }
+
+    static [Book] Find([scriptblock]$Predicate) {
+        [BookList]::Initialize()
+        return [BookList]::Books.Find($Predicate)
+    }
+
+    static [Book[]] FindAll([scriptblock]$Predicate) {
+        [BookList]::Initialize()
+        return [BookList]::Books.FindAll($Predicate)
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/BookLoan.ps1 b/tests/srcTestRepo/src/classes/public/BookLoan.ps1
new file mode 100644
index 00000000..fc0f6163
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/BookLoan.ps1
@@ -0,0 +1,13 @@
+class BookLoan {
+    [BookCopy] $Copy
+    [MemberAccount] $Member
+    [datetime] $LoanDate
+    [datetime] $DueDate
+
+    BookLoan([BookCopy]$Copy, [MemberAccount]$Member, [DueDatePolicy]$Policy) {
+        $this.Copy = $Copy
+        $this.Member = $Member
+        $this.LoanDate = Get-Date
+        $this.DueDate = $this.LoanDate.AddDays($Policy.StandardLoanDays)
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/BranchLibrary.ps1 b/tests/srcTestRepo/src/classes/public/BranchLibrary.ps1
new file mode 100644
index 00000000..3ee2fd20
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/BranchLibrary.ps1
@@ -0,0 +1,11 @@
+class BranchLibrary {
+    [string] $Code
+    [string] $Name
+    [OpeningHours] $Hours
+
+    BranchLibrary([string]$Code, [string]$Name, [OpeningHours]$Hours) {
+        $this.Code = $Code
+        $this.Name = $Name
+        $this.Hours = $Hours
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/CatalogSection.ps1 b/tests/srcTestRepo/src/classes/public/CatalogSection.ps1
new file mode 100644
index 00000000..10352122
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/CatalogSection.ps1
@@ -0,0 +1,9 @@
+class CatalogSection {
+    [string] $Name
+    [Book[]] $Books
+
+    CatalogSection([string]$Name, [Book[]]$Books) {
+        $this.Name = $Name
+        $this.Books = $Books
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/DueDatePolicy.ps1 b/tests/srcTestRepo/src/classes/public/DueDatePolicy.ps1
new file mode 100644
index 00000000..2803f489
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/DueDatePolicy.ps1
@@ -0,0 +1,9 @@
+class DueDatePolicy {
+    [int] $StandardLoanDays
+    [int] $GraceDays
+
+    DueDatePolicy([int]$StandardLoanDays, [int]$GraceDays) {
+        $this.StandardLoanDays = $StandardLoanDays
+        $this.GraceDays = $GraceDays
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/LibraryReport.ps1 b/tests/srcTestRepo/src/classes/public/LibraryReport.ps1
new file mode 100644
index 00000000..dfa84fe7
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/LibraryReport.ps1
@@ -0,0 +1,18 @@
+class LibraryReport {
+    [AuthorIndex] $Authors
+    [ActiveLoanRegister] $ActiveLoans
+    [BookInventory] $Inventory
+    [ZCatalog] $Catalog
+
+    LibraryReport(
+        [AuthorIndex]$Authors,
+        [ActiveLoanRegister]$ActiveLoans,
+        [BookInventory]$Inventory,
+        [ZCatalog]$Catalog
+    ) {
+        $this.Authors = $Authors
+        $this.ActiveLoans = $ActiveLoans
+        $this.Inventory = $Inventory
+        $this.Catalog = $Catalog
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/MemberAccount.ps1 b/tests/srcTestRepo/src/classes/public/MemberAccount.ps1
new file mode 100644
index 00000000..c292f231
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/MemberAccount.ps1
@@ -0,0 +1,11 @@
+class MemberAccount {
+    [string] $MemberId
+    [string] $DisplayName
+    [ReadingList] $ReadingList
+
+    MemberAccount([string]$MemberId, [string]$DisplayName, [ReadingList]$ReadingList) {
+        $this.MemberId = $MemberId
+        $this.DisplayName = $DisplayName
+        $this.ReadingList = $ReadingList
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/OpeningHours.ps1 b/tests/srcTestRepo/src/classes/public/OpeningHours.ps1
new file mode 100644
index 00000000..4be09375
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/OpeningHours.ps1
@@ -0,0 +1,9 @@
+class OpeningHours {
+    [timespan] $OpenAt
+    [timespan] $CloseAt
+
+    OpeningHours([timespan]$OpenAt, [timespan]$CloseAt) {
+        $this.OpenAt = $OpenAt
+        $this.CloseAt = $CloseAt
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/ReadingList.ps1 b/tests/srcTestRepo/src/classes/public/ReadingList.ps1
new file mode 100644
index 00000000..9acadeff
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/ReadingList.ps1
@@ -0,0 +1,7 @@
+class ReadingList {
+    [Book[]] $Books
+
+    ReadingList([Book[]]$Books) {
+        $this.Books = $Books
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/ShelfLocation.ps1 b/tests/srcTestRepo/src/classes/public/ShelfLocation.ps1
new file mode 100644
index 00000000..e492d713
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/ShelfLocation.ps1
@@ -0,0 +1,11 @@
+class ShelfLocation {
+    [string] $BranchCode
+    [string] $Section
+    [string] $Shelf
+
+    ShelfLocation([string]$BranchCode, [string]$Section, [string]$Shelf) {
+        $this.BranchCode = $BranchCode
+        $this.Section = $Section
+        $this.Shelf = $Shelf
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/ZCatalog.ps1 b/tests/srcTestRepo/src/classes/public/ZCatalog.ps1
new file mode 100644
index 00000000..1458c4d7
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/ZCatalog.ps1
@@ -0,0 +1,11 @@
+class ZCatalog {
+    [string] $CatalogName
+    [CatalogSection[]] $Sections
+    [BranchLibrary[]] $Branches
+
+    ZCatalog([string]$CatalogName, [CatalogSection[]]$Sections, [BranchLibrary[]]$Branches) {
+        $this.CatalogName = $CatalogName
+        $this.Sections = $Sections
+        $this.Branches = $Branches
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/A-BookList.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/A-BookList.ps1
deleted file mode 100644
index 68a96673..00000000
--- a/tests/srcWithManifestTestRepo/src/classes/public/A-BookList.ps1
+++ /dev/null
@@ -1,86 +0,0 @@
-class BookList {
-    # Static property to hold the list of books
-    static [System.Collections.Generic.List[Book]] $Books
-    # Static method to initialize the list of books. Called in the other
-    # static methods to avoid needing to explicit initialize the value.
-    static [void] Initialize() { [BookList]::Initialize($false) }
-    static [bool] Initialize([bool]$force) {
-        if ([BookList]::Books.Count -gt 0 -and -not $force) {
-            return $false
-        }
-
-        [BookList]::Books = [System.Collections.Generic.List[Book]]::new()
-
-        return $true
-    }
-    # Ensure a book is valid for the list.
-    static [void] Validate([book]$Book) {
-        $Prefix = @(
-            'Book validation failed: Book must be defined with the Title,'
-            'Author, and PublishDate properties, but'
-        ) -join ' '
-        if ($null -eq $Book) { throw "$Prefix was null" }
-        if ([string]::IsNullOrEmpty($Book.Title)) {
-            throw "$Prefix Title wasn't defined"
-        }
-        if ([string]::IsNullOrEmpty($Book.Author)) {
-            throw "$Prefix Author wasn't defined"
-        }
-        if ([datetime]::MinValue -eq $Book.PublishDate) {
-            throw "$Prefix PublishDate wasn't defined"
-        }
-    }
-    # Static methods to manage the list of books.
-    # Add a book if it's not already in the list.
-    static [void] Add([Book]$Book) {
-        [BookList]::Initialize()
-        [BookList]::Validate($Book)
-        if ([BookList]::Books.Contains($Book)) {
-            throw "Book '$Book' already in list"
-        }
-
-        $FindPredicate = {
-            param([Book]$b)
-
-            $b.Title -eq $Book.Title -and
-            $b.Author -eq $Book.Author -and
-            $b.PublishDate -eq $Book.PublishDate
-        }.GetNewClosure()
-        if ([BookList]::Books.Find($FindPredicate)) {
-            throw "Book '$Book' already in list"
-        }
-
-        [BookList]::Books.Add($Book)
-    }
-    # Clear the list of books.
-    static [void] Clear() {
-        [BookList]::Initialize()
-        [BookList]::Books.Clear()
-    }
-    # Find a specific book using a filtering scriptblock.
-    static [Book] Find([scriptblock]$Predicate) {
-        [BookList]::Initialize()
-        return [BookList]::Books.Find($Predicate)
-    }
-    # Find every book matching the filtering scriptblock.
-    static [Book[]] FindAll([scriptblock]$Predicate) {
-        [BookList]::Initialize()
-        return [BookList]::Books.FindAll($Predicate)
-    }
-    # Remove a specific book.
-    static [void] Remove([Book]$Book) {
-        [BookList]::Initialize()
-        [BookList]::Books.Remove($Book)
-    }
-    # Remove a book by property value.
-    static [void] RemoveBy([string]$Property, [string]$Value) {
-        [BookList]::Initialize()
-        $Index = [BookList]::Books.FindIndex({
-                param($b)
-                $b.$Property -eq $Value
-            }.GetNewClosure())
-        if ($Index -ge 0) {
-            [BookList]::Books.RemoveAt($Index)
-        }
-    }
-}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/ActiveLoanRegister.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/ActiveLoanRegister.ps1
new file mode 100644
index 00000000..fb4e98cd
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/ActiveLoanRegister.ps1
@@ -0,0 +1,11 @@
+class ActiveLoanRegister {
+    [System.Collections.Generic.List[BookLoan]] $Loans
+
+    ActiveLoanRegister() {
+        $this.Loans = [System.Collections.Generic.List[BookLoan]]::new()
+    }
+
+    [void] AddLoan([BookLoan]$Loan) {
+        $this.Loans.Add($Loan)
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/AuthorIndex.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/AuthorIndex.ps1
new file mode 100644
index 00000000..c379af92
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/AuthorIndex.ps1
@@ -0,0 +1,11 @@
+class AuthorIndex {
+    [string] $Name
+    [AuthorProfile[]] $Authors
+    [ZCatalog] $Catalog
+
+    AuthorIndex([string]$Name, [ZCatalog]$Catalog) {
+        $this.Name = $Name
+        $this.Catalog = $Catalog
+        $this.Authors = @()
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/AuthorProfile.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/AuthorProfile.ps1
new file mode 100644
index 00000000..6a86fa35
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/AuthorProfile.ps1
@@ -0,0 +1,9 @@
+class AuthorProfile {
+    [string] $Name
+    [Book[]] $PublishedBooks
+
+    AuthorProfile([string]$Name, [Book[]]$PublishedBooks) {
+        $this.Name = $Name
+        $this.PublishedBooks = $PublishedBooks
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/BookCopy.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/BookCopy.ps1
new file mode 100644
index 00000000..bee184b1
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/BookCopy.ps1
@@ -0,0 +1,11 @@
+class BookCopy {
+    [string] $InventoryId
+    [Book] $Book
+    [ShelfLocation] $Location
+
+    BookCopy([string]$InventoryId, [Book]$Book, [ShelfLocation]$Location) {
+        $this.InventoryId = $InventoryId
+        $this.Book = $Book
+        $this.Location = $Location
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/BookInventory.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/BookInventory.ps1
new file mode 100644
index 00000000..cf7948e3
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/BookInventory.ps1
@@ -0,0 +1,11 @@
+class BookInventory {
+    [string] $Name
+    [BookCopy[]] $Copies
+    [ZCatalog] $Catalog
+
+    BookInventory([string]$Name, [ZCatalog]$Catalog) {
+        $this.Name = $Name
+        $this.Catalog = $Catalog
+        $this.Copies = @()
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/BookList.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/BookList.ps1
new file mode 100644
index 00000000..bf421618
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/BookList.ps1
@@ -0,0 +1,57 @@
+class BookList {
+    static [System.Collections.Generic.List[Book]] $Books
+
+    static [void] Initialize() { [BookList]::Initialize($false) }
+    static [bool] Initialize([bool]$force) {
+        if ([BookList]::Books.Count -gt 0 -and -not $force) {
+            return $false
+        }
+
+        [BookList]::Books = [System.Collections.Generic.List[Book]]::new()
+
+        return $true
+    }
+
+    static [void] Validate([Book]$Book) {
+        $Prefix = @(
+            'Book validation failed: Book must be defined with the Title,'
+            'Author, and PublishDate properties, but'
+        ) -join ' '
+        if ($null -eq $Book) { throw "$Prefix was null" }
+        if ([string]::IsNullOrEmpty($Book.Title)) { throw "$Prefix Title wasn't defined" }
+        if ([string]::IsNullOrEmpty($Book.Author)) { throw "$Prefix Author wasn't defined" }
+        if ([datetime]::MinValue -eq $Book.PublishDate) { throw "$Prefix PublishDate wasn't defined" }
+    }
+
+    static [void] Add([Book]$Book) {
+        [BookList]::Initialize()
+        [BookList]::Validate($Book)
+        if ([BookList]::Books.Contains($Book)) { throw "Book '$Book' already in list" }
+
+        $findPredicate = {
+            param([Book]$b)
+
+            $b.Title -eq $Book.Title -and
+            $b.Author -eq $Book.Author -and
+            $b.PublishDate -eq $Book.PublishDate
+        }.GetNewClosure()
+        if ([BookList]::Books.Find($findPredicate)) { throw "Book '$Book' already in list" }
+
+        [BookList]::Books.Add($Book)
+    }
+
+    static [void] Clear() {
+        [BookList]::Initialize()
+        [BookList]::Books.Clear()
+    }
+
+    static [Book] Find([scriptblock]$Predicate) {
+        [BookList]::Initialize()
+        return [BookList]::Books.Find($Predicate)
+    }
+
+    static [Book[]] FindAll([scriptblock]$Predicate) {
+        [BookList]::Initialize()
+        return [BookList]::Books.FindAll($Predicate)
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/BookLoan.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/BookLoan.ps1
new file mode 100644
index 00000000..fc0f6163
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/BookLoan.ps1
@@ -0,0 +1,13 @@
+class BookLoan {
+    [BookCopy] $Copy
+    [MemberAccount] $Member
+    [datetime] $LoanDate
+    [datetime] $DueDate
+
+    BookLoan([BookCopy]$Copy, [MemberAccount]$Member, [DueDatePolicy]$Policy) {
+        $this.Copy = $Copy
+        $this.Member = $Member
+        $this.LoanDate = Get-Date
+        $this.DueDate = $this.LoanDate.AddDays($Policy.StandardLoanDays)
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/BranchLibrary.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/BranchLibrary.ps1
new file mode 100644
index 00000000..3ee2fd20
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/BranchLibrary.ps1
@@ -0,0 +1,11 @@
+class BranchLibrary {
+    [string] $Code
+    [string] $Name
+    [OpeningHours] $Hours
+
+    BranchLibrary([string]$Code, [string]$Name, [OpeningHours]$Hours) {
+        $this.Code = $Code
+        $this.Name = $Name
+        $this.Hours = $Hours
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/CatalogSection.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/CatalogSection.ps1
new file mode 100644
index 00000000..10352122
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/CatalogSection.ps1
@@ -0,0 +1,9 @@
+class CatalogSection {
+    [string] $Name
+    [Book[]] $Books
+
+    CatalogSection([string]$Name, [Book[]]$Books) {
+        $this.Name = $Name
+        $this.Books = $Books
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/DueDatePolicy.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/DueDatePolicy.ps1
new file mode 100644
index 00000000..2803f489
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/DueDatePolicy.ps1
@@ -0,0 +1,9 @@
+class DueDatePolicy {
+    [int] $StandardLoanDays
+    [int] $GraceDays
+
+    DueDatePolicy([int]$StandardLoanDays, [int]$GraceDays) {
+        $this.StandardLoanDays = $StandardLoanDays
+        $this.GraceDays = $GraceDays
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/LibraryReport.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/LibraryReport.ps1
new file mode 100644
index 00000000..dfa84fe7
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/LibraryReport.ps1
@@ -0,0 +1,18 @@
+class LibraryReport {
+    [AuthorIndex] $Authors
+    [ActiveLoanRegister] $ActiveLoans
+    [BookInventory] $Inventory
+    [ZCatalog] $Catalog
+
+    LibraryReport(
+        [AuthorIndex]$Authors,
+        [ActiveLoanRegister]$ActiveLoans,
+        [BookInventory]$Inventory,
+        [ZCatalog]$Catalog
+    ) {
+        $this.Authors = $Authors
+        $this.ActiveLoans = $ActiveLoans
+        $this.Inventory = $Inventory
+        $this.Catalog = $Catalog
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/MemberAccount.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/MemberAccount.ps1
new file mode 100644
index 00000000..c292f231
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/MemberAccount.ps1
@@ -0,0 +1,11 @@
+class MemberAccount {
+    [string] $MemberId
+    [string] $DisplayName
+    [ReadingList] $ReadingList
+
+    MemberAccount([string]$MemberId, [string]$DisplayName, [ReadingList]$ReadingList) {
+        $this.MemberId = $MemberId
+        $this.DisplayName = $DisplayName
+        $this.ReadingList = $ReadingList
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/OpeningHours.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/OpeningHours.ps1
new file mode 100644
index 00000000..4be09375
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/OpeningHours.ps1
@@ -0,0 +1,9 @@
+class OpeningHours {
+    [timespan] $OpenAt
+    [timespan] $CloseAt
+
+    OpeningHours([timespan]$OpenAt, [timespan]$CloseAt) {
+        $this.OpenAt = $OpenAt
+        $this.CloseAt = $CloseAt
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/ReadingList.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/ReadingList.ps1
new file mode 100644
index 00000000..9acadeff
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/ReadingList.ps1
@@ -0,0 +1,7 @@
+class ReadingList {
+    [Book[]] $Books
+
+    ReadingList([Book[]]$Books) {
+        $this.Books = $Books
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/ShelfLocation.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/ShelfLocation.ps1
new file mode 100644
index 00000000..e492d713
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/ShelfLocation.ps1
@@ -0,0 +1,11 @@
+class ShelfLocation {
+    [string] $BranchCode
+    [string] $Section
+    [string] $Shelf
+
+    ShelfLocation([string]$BranchCode, [string]$Section, [string]$Shelf) {
+        $this.BranchCode = $BranchCode
+        $this.Section = $Section
+        $this.Shelf = $Shelf
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/ZCatalog.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/ZCatalog.ps1
new file mode 100644
index 00000000..1458c4d7
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/ZCatalog.ps1
@@ -0,0 +1,11 @@
+class ZCatalog {
+    [string] $CatalogName
+    [CatalogSection[]] $Sections
+    [BranchLibrary[]] $Branches
+
+    ZCatalog([string]$CatalogName, [CatalogSection[]]$Sections, [BranchLibrary[]]$Branches) {
+        $this.CatalogName = $CatalogName
+        $this.Sections = $Sections
+        $this.Branches = $Branches
+    }
+}

From 18d9d8bf1815cca463964ffa1c56ecba4c1fb06a Mon Sep 17 00:00:00 2001
From: Marius Storhaug <marstor@hotmail.com>
Date: Sat, 18 Jul 2026 21:14:49 +0200
Subject: [PATCH 08/13] Warn explicitly on cyclical class dependencies

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 .../helpers/Build/Get-DependencyOrderedScriptFiles.ps1   | 3 ++-
 .../src/classes/public/AlphaCirculationNode.ps1          | 9 +++++++++
 .../src/classes/public/ZetaCirculationNode.ps1           | 9 +++++++++
 .../src/classes/public/AlphaCirculationNode.ps1          | 9 +++++++++
 .../src/classes/public/ZetaCirculationNode.ps1           | 9 +++++++++
 5 files changed, 38 insertions(+), 1 deletion(-)
 create mode 100644 tests/srcTestRepo/src/classes/public/AlphaCirculationNode.ps1
 create mode 100644 tests/srcTestRepo/src/classes/public/ZetaCirculationNode.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/AlphaCirculationNode.ps1
 create mode 100644 tests/srcWithManifestTestRepo/src/classes/public/ZetaCirculationNode.ps1

diff --git a/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1 b/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1
index 6f2e7ac4..8fcddb3c 100644
--- a/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1
+++ b/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1
@@ -88,7 +88,8 @@ function Get-DependencyOrderedScriptFiles {
     }
 
     if ($orderedPaths.Count -lt $sortedFiles.Count) {
-        Write-Warning "Detected cyclical or unresolved class/enum dependencies. Falling back to lexical order for remaining files in [$($sortedFiles[0].DirectoryName)]."
+        $remainingNames = ($remainingPaths | ForEach-Object { [System.IO.Path]::GetFileName($_) }) -join ', '
+        Write-Warning "Cyclical class/enum dependencies detected in [$($sortedFiles[0].DirectoryName)]. Falling back to lexical order for unresolved files: $remainingNames"
         $remainingPaths | Sort-Object | ForEach-Object {
             $orderedPaths.Add($_)
         }
diff --git a/tests/srcTestRepo/src/classes/public/AlphaCirculationNode.ps1 b/tests/srcTestRepo/src/classes/public/AlphaCirculationNode.ps1
new file mode 100644
index 00000000..d7e10b5e
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/AlphaCirculationNode.ps1
@@ -0,0 +1,9 @@
+class AlphaCirculationNode {
+    [string] $Id
+    [ZetaCirculationNode] $Next
+
+    AlphaCirculationNode([string]$Id, [ZetaCirculationNode]$Next) {
+        $this.Id = $Id
+        $this.Next = $Next
+    }
+}
diff --git a/tests/srcTestRepo/src/classes/public/ZetaCirculationNode.ps1 b/tests/srcTestRepo/src/classes/public/ZetaCirculationNode.ps1
new file mode 100644
index 00000000..5ecd524e
--- /dev/null
+++ b/tests/srcTestRepo/src/classes/public/ZetaCirculationNode.ps1
@@ -0,0 +1,9 @@
+class ZetaCirculationNode {
+    [string] $Id
+    [AlphaCirculationNode] $Previous
+
+    ZetaCirculationNode([string]$Id, [AlphaCirculationNode]$Previous) {
+        $this.Id = $Id
+        $this.Previous = $Previous
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/AlphaCirculationNode.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/AlphaCirculationNode.ps1
new file mode 100644
index 00000000..d7e10b5e
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/AlphaCirculationNode.ps1
@@ -0,0 +1,9 @@
+class AlphaCirculationNode {
+    [string] $Id
+    [ZetaCirculationNode] $Next
+
+    AlphaCirculationNode([string]$Id, [ZetaCirculationNode]$Next) {
+        $this.Id = $Id
+        $this.Next = $Next
+    }
+}
diff --git a/tests/srcWithManifestTestRepo/src/classes/public/ZetaCirculationNode.ps1 b/tests/srcWithManifestTestRepo/src/classes/public/ZetaCirculationNode.ps1
new file mode 100644
index 00000000..5ecd524e
--- /dev/null
+++ b/tests/srcWithManifestTestRepo/src/classes/public/ZetaCirculationNode.ps1
@@ -0,0 +1,9 @@
+class ZetaCirculationNode {
+    [string] $Id
+    [AlphaCirculationNode] $Previous
+
+    ZetaCirculationNode([string]$Id, [AlphaCirculationNode]$Previous) {
+        $this.Id = $Id
+        $this.Previous = $Previous
+    }
+}

From 8d85266df491157a4591fb25363e23de93263885 Mon Sep 17 00:00:00 2001
From: Marius Storhaug <marstor@hotmail.com>
Date: Sat, 18 Jul 2026 21:26:09 +0200
Subject: [PATCH 09/13] Fail build on cyclical class dependencies

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 .../src/helpers/Build/Add-ContentFromItem.ps1 |  2 +-
 .../Build/Build-PSModuleRootModule.ps1        |  8 ++++-
 ...s1 => Get-DependencyOrderedScriptFile.ps1} | 32 +++++++++++++------
 3 files changed, 31 insertions(+), 11 deletions(-)
 rename .github/actions/Build-PSModule/src/helpers/Build/{Get-DependencyOrderedScriptFiles.ps1 => Get-DependencyOrderedScriptFile.ps1} (75%)

diff --git a/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1 b/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1
index c3aa61bb..2e1c0b54 100644
--- a/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1
+++ b/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1
@@ -39,7 +39,7 @@ Write-Debug "[`$scriptName] - $relativeFolderPath - Processing folder"
 
     if ($DependencyAware) {
         $files = $Path | Get-ChildItem -Recurse -File -Force -Filter '*.ps1' | Sort-Object -Property FullName
-        $files = Get-DependencyOrderedScriptFiles -Files $files
+        $files = Get-DependencyOrderedScriptFile -Files $files
     } else {
         $files = $Path | Get-ChildItem -File -Force -Filter '*.ps1' | Sort-Object -Property FullName
     }
diff --git a/.github/actions/Build-PSModule/src/helpers/Build/Build-PSModuleRootModule.ps1 b/.github/actions/Build-PSModule/src/helpers/Build/Build-PSModuleRootModule.ps1
index 4fa36797..f31667e3 100644
--- a/.github/actions/Build-PSModule/src/helpers/Build/Build-PSModuleRootModule.ps1
+++ b/.github/actions/Build-PSModule/src/helpers/Build/Build-PSModuleRootModule.ps1
@@ -187,7 +187,13 @@ Write-Debug "[$scriptName] - [data] - Done"
             if (-not (Test-Path -Path $scriptFolderPath)) {
                 continue
             }
-            Add-ContentFromItem -Path $scriptFolderPath -RootModuleFilePath $rootModuleFile -RootPath $ModuleOutputFolder -DependencyAware:$dependencyAware
+            $addContentParams = @{
+                Path               = $scriptFolderPath
+                RootModuleFilePath = $rootModuleFile
+                RootPath           = $ModuleOutputFolder
+                DependencyAware    = $dependencyAware
+            }
+            Add-ContentFromItem @addContentParams
             Remove-Item -Path $scriptFolderPath -Force -Recurse
         }
         #endregion - Add content from subfolders
diff --git a/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1 b/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFile.ps1
similarity index 75%
rename from .github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1
rename to .github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFile.ps1
index 8fcddb3c..c3774e24 100644
--- a/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFiles.ps1
+++ b/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFile.ps1
@@ -1,4 +1,16 @@
-function Get-DependencyOrderedScriptFiles {
+function Get-DependencyOrderedScriptFile {
+    <#
+        .SYNOPSIS
+        Sorts script files so class/enum dependencies are loaded before dependents.
+
+        .DESCRIPTION
+        Reads declared class/enum types and type references from each script and returns
+        files in dependency order. When a cyclical dependency is detected, the function
+        warns and throws to fail the build early.
+
+        .EXAMPLE
+        Get-DependencyOrderedScriptFile -Files (Get-ChildItem -Path '.\classes' -Filter '*.ps1')
+    #>
     [OutputType([System.IO.FileInfo[]])]
     [CmdletBinding()]
     param(
@@ -6,7 +18,7 @@ function Get-DependencyOrderedScriptFiles {
         [System.IO.FileInfo[]] $Files
     )
 
-    $sortedFiles = $Files | Sort-Object -Property FullName
+    [System.IO.FileInfo[]]$sortedFiles = $Files | Sort-Object -Property FullName
     if ($sortedFiles.Count -le 1) {
         return $sortedFiles
     }
@@ -67,8 +79,8 @@ function Get-DependencyOrderedScriptFiles {
     $remainingPaths = @($sortedFiles.FullName)
     $ready = @(
         $remainingPaths |
-        Where-Object { $dependenciesByPath[$_].Count -eq 0 } |
-        Sort-Object
+            Where-Object { $dependenciesByPath[$_].Count -eq 0 } |
+            Sort-Object
     )
     $orderedPaths = [System.Collections.Generic.List[string]]::new()
 
@@ -89,11 +101,13 @@ function Get-DependencyOrderedScriptFiles {
 
     if ($orderedPaths.Count -lt $sortedFiles.Count) {
         $remainingNames = ($remainingPaths | ForEach-Object { [System.IO.Path]::GetFileName($_) }) -join ', '
-        Write-Warning "Cyclical class/enum dependencies detected in [$($sortedFiles[0].DirectoryName)]. Falling back to lexical order for unresolved files: $remainingNames"
-        $remainingPaths | Sort-Object | ForEach-Object {
-            $orderedPaths.Add($_)
-        }
+        $message = @(
+            "Cyclical class/enum dependencies detected in [$($sortedFiles[0].DirectoryName)]."
+            "Build cannot continue with unresolved files: $remainingNames"
+        ) -join ' '
+        Write-Warning $message
+        throw $message
     }
 
-    return @($orderedPaths | ForEach-Object { $metadataByPath[$_].File })
+    return [System.IO.FileInfo[]]@($orderedPaths | ForEach-Object { $metadataByPath[$_].File })
 }

From 53fc898197031b3ee2534702e7f70d9a7f655ede Mon Sep 17 00:00:00 2001
From: Marius Storhaug <marstor@hotmail.com>
Date: Sat, 18 Jul 2026 21:42:40 +0200
Subject: [PATCH 10/13] Keep cycle detection as warning fallback

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 .../src/helpers/Build/Get-DependencyOrderedScriptFile.ps1   | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFile.ps1 b/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFile.ps1
index c3774e24..13adb811 100644
--- a/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFile.ps1
+++ b/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFile.ps1
@@ -103,10 +103,12 @@ function Get-DependencyOrderedScriptFile {
         $remainingNames = ($remainingPaths | ForEach-Object { [System.IO.Path]::GetFileName($_) }) -join ', '
         $message = @(
             "Cyclical class/enum dependencies detected in [$($sortedFiles[0].DirectoryName)]."
-            "Build cannot continue with unresolved files: $remainingNames"
+            "Falling back to lexical order for unresolved files: $remainingNames"
         ) -join ' '
         Write-Warning $message
-        throw $message
+        $remainingPaths | Sort-Object | ForEach-Object {
+            $orderedPaths.Add($_)
+        }
     }
 
     return [System.IO.FileInfo[]]@($orderedPaths | ForEach-Object { $metadataByPath[$_].File })

From 8c099789938f60d4cce1aacf8b0377791b7b0751 Mon Sep 17 00:00:00 2001
From: Marius Storhaug <marstor@hotmail.com>
Date: Sat, 18 Jul 2026 21:43:48 +0200
Subject: [PATCH 11/13] Exclude class fixture files from repo linter

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 .github/workflows/Linter.yml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.github/workflows/Linter.yml b/.github/workflows/Linter.yml
index 2c5664a2..e57c709c 100644
--- a/.github/workflows/Linter.yml
+++ b/.github/workflows/Linter.yml
@@ -28,6 +28,7 @@ jobs:
         uses: super-linter/super-linter@4ce20838b8ab83717e78138c5b3a1407148e0918 # v8.7.0
         env:
           GITHUB_TOKEN: ${{ github.token }}
+          FILTER_REGEX_EXCLUDE: 'tests/src(TestRepo|WithManifestTestRepo)/src/classes/public/.*\.ps1$'
           VALIDATE_BIOME_FORMAT: false
           VALIDATE_GITHUB_ACTIONS: false
           VALIDATE_JSCPD: false

From edc5b8c31afc199235c8f7447ef216b454ce1703 Mon Sep 17 00:00:00 2001
From: Marius Storhaug <marstor@hotmail.com>
Date: Sat, 18 Jul 2026 21:46:25 +0200
Subject: [PATCH 12/13] Scope repo-linter excludes to working directory

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 .github/workflows/Lint-Repository.yml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.github/workflows/Lint-Repository.yml b/.github/workflows/Lint-Repository.yml
index 02c77590..52822090 100644
--- a/.github/workflows/Lint-Repository.yml
+++ b/.github/workflows/Lint-Repository.yml
@@ -54,6 +54,7 @@ jobs:
           GITHUB_TOKEN: ${{ github.token }}
           DEFAULT_WORKSPACE: ${{ fromJson(env.Settings).WorkingDirectory }}
           FILTER_REGEX_INCLUDE: ${{ fromJson(env.Settings).WorkingDirectory }}
+          FILTER_REGEX_EXCLUDE: ${{ fromJson(env.Settings).WorkingDirectory }}/src/classes/public/.*\.ps1$
           ENABLE_GITHUB_ACTIONS_STEP_SUMMARY: false
           SAVE_SUPER_LINTER_SUMMARY: true
 

From 16be1e4ebd76683bb8405fc8093d96e2e2b66454 Mon Sep 17 00:00:00 2001
From: Marius Storhaug <marstor@hotmail.com>
Date: Sat, 18 Jul 2026 21:48:25 +0200
Subject: [PATCH 13/13] Rollback class sorting in build pipeline

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 .../src/helpers/Build/Add-ContentFromItem.ps1 |  21 +---
 .../Build/Build-PSModuleRootModule.ps1        |  27 ++--
 .../Build/Get-DependencyOrderedScriptFile.ps1 | 115 ------------------
 3 files changed, 16 insertions(+), 147 deletions(-)
 delete mode 100644 .github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFile.ps1

diff --git a/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1 b/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1
index 2e1c0b54..095b7d98 100644
--- a/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1
+++ b/.github/actions/Build-PSModule/src/helpers/Build/Add-ContentFromItem.ps1
@@ -20,11 +20,7 @@ function Add-ContentFromItem {
 
         # The root path of the module.
         [Parameter(Mandatory)]
-        [string] $RootPath,
-
-        # Whether the folder should be loaded using dependency ordering (class/enum aware).
-        [Parameter()]
-        [switch] $DependencyAware
+        [string] $RootPath
     )
     $separators = [char[]]@([System.IO.Path]::DirectorySeparatorChar, [System.IO.Path]::AltDirectorySeparatorChar)
 
@@ -37,12 +33,7 @@ function Add-ContentFromItem {
 Write-Debug "[`$scriptName] - $relativeFolderPath - Processing folder"
 "@
 
-    if ($DependencyAware) {
-        $files = $Path | Get-ChildItem -Recurse -File -Force -Filter '*.ps1' | Sort-Object -Property FullName
-        $files = Get-DependencyOrderedScriptFile -Files $files
-    } else {
-        $files = $Path | Get-ChildItem -File -Force -Filter '*.ps1' | Sort-Object -Property FullName
-    }
+    $files = $Path | Get-ChildItem -File -Force -Filter '*.ps1' | Sort-Object -Property FullName
 
     foreach ($file in $files) {
         $relativeFilePath = [System.IO.Path]::GetRelativePath($RootPath, $file.FullName)
@@ -61,11 +52,9 @@ Write-Debug "[`$scriptName] - $relativeFilePath - Done"
 "@
     }
 
-    if (-not $DependencyAware) {
-        $subFolders = $Path | Get-ChildItem -Directory -Force | Sort-Object -Property Name
-        foreach ($subFolder in $subFolders) {
-            Add-ContentFromItem -Path $subFolder.FullName -RootModuleFilePath $RootModuleFilePath -RootPath $RootPath
-        }
+    $subFolders = $Path | Get-ChildItem -Directory -Force | Sort-Object -Property Name
+    foreach ($subFolder in $subFolders) {
+        Add-ContentFromItem -Path $subFolder.FullName -RootModuleFilePath $RootModuleFilePath -RootPath $RootPath
     }
     Add-Content -Path $RootModuleFilePath -Force -Value @"
 Write-Debug "[`$scriptName] - $relativeFolderPath - Done"
diff --git a/.github/actions/Build-PSModule/src/helpers/Build/Build-PSModuleRootModule.ps1 b/.github/actions/Build-PSModule/src/helpers/Build/Build-PSModuleRootModule.ps1
index f31667e3..619a5d95 100644
--- a/.github/actions/Build-PSModule/src/helpers/Build/Build-PSModuleRootModule.ps1
+++ b/.github/actions/Build-PSModule/src/helpers/Build/Build-PSModuleRootModule.ps1
@@ -10,14 +10,15 @@
 
         1. Module header from header.ps1 file. Usually to suppress code analysis warnings/errors and to add [CmdletBinding()] to the module.
         2. Data loader is added if data files are available.
-        3. Combines *.ps1 files from the following folders:
+        3. Combines *.ps1 files from the following folders in alphabetical order from each folder:
             1. init
-            2. classes (dependency-aware ordering for class/enum references)
-            3. functions/private
-            4. functions/public
-            5. variables/private
-            6. variables/public
-            7. Any remaining *.ps1 on module root.
+            2. classes/private
+            3. classes/public
+            4. functions/private
+            5. functions/public
+            6. variables/private
+            7. variables/public
+            8. Any remaining *.ps1 on module root.
         4. Adds a class loader for classes found in the classes/public folder.
         5. Export-ModuleMember by using the functions, cmdlets, variables and aliases found in the source files.
             - `Functions` will only contain functions that are from the `functions/public` folder.
@@ -174,7 +175,8 @@ Write-Debug "[$scriptName] - [data] - Done"
         #region - Add content from subfolders
         $scriptFoldersToProcess = @(
             'init',
-            'classes',
+            'classes/private',
+            'classes/public',
             'functions/private',
             'functions/public',
             'variables/private',
@@ -182,18 +184,11 @@ Write-Debug "[$scriptName] - [data] - Done"
         )
 
         foreach ($scriptFolder in $scriptFoldersToProcess) {
-            $dependencyAware = $scriptFolder -eq 'classes'
             $scriptFolderPath = Join-Path -Path $ModuleOutputFolder -ChildPath $scriptFolder
             if (-not (Test-Path -Path $scriptFolderPath)) {
                 continue
             }
-            $addContentParams = @{
-                Path               = $scriptFolderPath
-                RootModuleFilePath = $rootModuleFile
-                RootPath           = $ModuleOutputFolder
-                DependencyAware    = $dependencyAware
-            }
-            Add-ContentFromItem @addContentParams
+            Add-ContentFromItem -Path $scriptFolderPath -RootModuleFilePath $rootModuleFile -RootPath $ModuleOutputFolder
             Remove-Item -Path $scriptFolderPath -Force -Recurse
         }
         #endregion - Add content from subfolders
diff --git a/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFile.ps1 b/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFile.ps1
deleted file mode 100644
index 13adb811..00000000
--- a/.github/actions/Build-PSModule/src/helpers/Build/Get-DependencyOrderedScriptFile.ps1
+++ /dev/null
@@ -1,115 +0,0 @@
-function Get-DependencyOrderedScriptFile {
-    <#
-        .SYNOPSIS
-        Sorts script files so class/enum dependencies are loaded before dependents.
-
-        .DESCRIPTION
-        Reads declared class/enum types and type references from each script and returns
-        files in dependency order. When a cyclical dependency is detected, the function
-        warns and throws to fail the build early.
-
-        .EXAMPLE
-        Get-DependencyOrderedScriptFile -Files (Get-ChildItem -Path '.\classes' -Filter '*.ps1')
-    #>
-    [OutputType([System.IO.FileInfo[]])]
-    [CmdletBinding()]
-    param(
-        [Parameter(Mandatory)]
-        [System.IO.FileInfo[]] $Files
-    )
-
-    [System.IO.FileInfo[]]$sortedFiles = $Files | Sort-Object -Property FullName
-    if ($sortedFiles.Count -le 1) {
-        return $sortedFiles
-    }
-
-    $metadataByPath = @{}
-    $typeToPath = @{}
-    $duplicateTypes = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
-
-    foreach ($file in $sortedFiles) {
-        $content = Get-Content -Path $file.FullName -Raw
-        $declaredTypes = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
-        [Regex]::Matches($content, '(?im)^\s*(class|enum)\s+([^\s:{]+)') | ForEach-Object {
-            [void] $declaredTypes.Add($_.Groups[2].Value)
-        }
-
-        $metadataByPath[$file.FullName] = [pscustomobject]@{
-            File          = $file
-            Content       = $content
-            DeclaredTypes = $declaredTypes
-        }
-
-        foreach ($typeName in $declaredTypes) {
-            if ($typeToPath.ContainsKey($typeName)) {
-                [void]$duplicateTypes.Add($typeName)
-                continue
-            }
-
-            $typeToPath[$typeName] = $file.FullName
-        }
-    }
-
-    $dependenciesByPath = @{}
-    $dependentsByPath = @{}
-    foreach ($file in $sortedFiles) {
-        $dependenciesByPath[$file.FullName] = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
-        $dependentsByPath[$file.FullName] = [System.Collections.Generic.HashSet[string]]::new([System.StringComparer]::OrdinalIgnoreCase)
-    }
-
-    foreach ($metadata in $metadataByPath.Values) {
-        foreach ($typeName in $typeToPath.Keys) {
-            if ($duplicateTypes.Contains($typeName)) {
-                continue
-            }
-
-            if ($metadata.DeclaredTypes.Contains($typeName)) {
-                continue
-            }
-
-            $typePattern = "(?<![\w\.])\[$([Regex]::Escape($typeName))(?:\[\])?\](?![\w\.])"
-            if ([Regex]::IsMatch($metadata.Content, $typePattern, [System.Text.RegularExpressions.RegexOptions]::IgnoreCase)) {
-                $dependencyPath = $typeToPath[$typeName]
-                [void]$dependenciesByPath[$metadata.File.FullName].Add($dependencyPath)
-                [void]$dependentsByPath[$dependencyPath].Add($metadata.File.FullName)
-            }
-        }
-    }
-
-    $remainingPaths = @($sortedFiles.FullName)
-    $ready = @(
-        $remainingPaths |
-            Where-Object { $dependenciesByPath[$_].Count -eq 0 } |
-            Sort-Object
-    )
-    $orderedPaths = [System.Collections.Generic.List[string]]::new()
-
-    while ($ready.Count -gt 0) {
-        $currentPath = $ready[0]
-        $ready = @($ready | Select-Object -Skip 1)
-        $orderedPaths.Add($currentPath)
-        $remainingPaths = @($remainingPaths | Where-Object { $_ -ne $currentPath })
-
-        $dependents = @($dependentsByPath[$currentPath] | Sort-Object)
-        foreach ($dependentPath in $dependents) {
-            $null = $dependenciesByPath[$dependentPath].Remove($currentPath)
-            if ($dependenciesByPath[$dependentPath].Count -eq 0) {
-                $ready = @($ready + $dependentPath | Sort-Object -Unique)
-            }
-        }
-    }
-
-    if ($orderedPaths.Count -lt $sortedFiles.Count) {
-        $remainingNames = ($remainingPaths | ForEach-Object { [System.IO.Path]::GetFileName($_) }) -join ', '
-        $message = @(
-            "Cyclical class/enum dependencies detected in [$($sortedFiles[0].DirectoryName)]."
-            "Falling back to lexical order for unresolved files: $remainingNames"
-        ) -join ' '
-        Write-Warning $message
-        $remainingPaths | Sort-Object | ForEach-Object {
-            $orderedPaths.Add($_)
-        }
-    }
-
-    return [System.IO.FileInfo[]]@($orderedPaths | ForEach-Object { $metadataByPath[$_].File })
-}