@danmoisan/drm-copilot-mcp 1.1.22 → 1.1.24

package/resources/powershell/PoshQC/PoshQC.Shard.psm1

@@ -235,6 +235,8 @@ Total number of shards.
 Optional path that receives the shard manifest JSON.
 .PARAMETER WriteShardFileList
 Seam that writes the manifest JSON; receives the path and the JSON text.
+.PARAMETER ReadWeightsText
+Optional seam passed to Get-PoshQCShardTestFileWeight that returns the weights-file text for RootPath, or $null (issue #1048).
 #>
 function Select-PoshQCShardTestFile {
     [CmdletBinding()]
@@ -245,11 +247,17 @@ function Select-PoshQCShardTestFile {
         [Parameter(Mandatory)][int] $ShardIndex,
         [Parameter(Mandatory)][int] $ShardCount,
         [string] $ShardFileListPath,
-        [Parameter(Mandatory)][scriptblock] $WriteShardFileList
+        [Parameter(Mandatory)][scriptblock] $WriteShardFileList,
+        [scriptblock] $ReadWeightsText
     )
 
     $paths = [string[]] @($TestFile | ForEach-Object { if ($_ -is [string]) { $_ } else { $_.FullName } })
-    $weights = [long[]] @(Get-PoshQCShardTestFileWeight -TestFile $TestFile)
+    # Issue #1048: the root lets the weight lookup read config/poshqc-shard-weights.json; a supplied reader replaces the default one.
+    $weightArguments = @{ TestFile = $TestFile; RootPath = $RootPath }
+    if ($PSBoundParameters.ContainsKey('ReadWeightsText')) {
+        $weightArguments['ReadWeightsText'] = $ReadWeightsText
+    }
+    $weights = [long[]] @(Get-PoshQCShardTestFileWeight @weightArguments)
     $manifest = New-PoshQCShardManifest -RootPath $RootPath -TestFile $paths -ShardIndex $ShardIndex -ShardCount $ShardCount -Weight $weights
     if (@($manifest.selected).Count -eq 0) {
         throw "Shard $ShardIndex of $ShardCount selects no test file; $($paths.Count) test files cannot fill $ShardCount shards. Lower the shard count."
@@ -262,25 +270,115 @@ function Select-PoshQCShardTestFile {
 
 <#
 .SYNOPSIS
-Returns the shard weight of each enumerated test file (issue #953, CR-19).
+Parses and validates the text of config/poshqc-shard-weights.json (issue #1048).
 .DESCRIPTION
-A FileInfo item weighs the UTF-8 byte count of its text after every CRLF pair is replaced by LF, so a checkout
-with CRLF line endings computes the same weights, and therefore the same shard assignment, as a checkout with
-LF line endings. Another object weighs its Length property when it has one; a plain path string, or an object
-without a Length property, weighs 1. Returns one weight per item, in input order.
+The document must be a JSON object with version 1, unit 'ms', and a non-empty files map whose keys are
+repository-relative forward-slash paths (no backslash, no leading slash, no colon) and whose values are
+non-negative integers. Throws a message that names poshqc-shard-weights.json otherwise. Returns a dictionary
+of milliseconds keyed by path, compared case-insensitively.
+.PARAMETER Json
+The text of the weights file.
+#>
+function ConvertFrom-PoshQCShardWeightJson {
+    [CmdletBinding()]
+    [OutputType([System.Collections.Generic.Dictionary[string, long]])]
+    param(
+        [Parameter(Mandatory)][AllowEmptyString()][string] $Json
+    )
+
+    $name = 'config/poshqc-shard-weights.json'
+    try {
+        $document = $Json | ConvertFrom-Json -ErrorAction Stop
+    } catch {
+        throw "$name is not valid JSON: $($_.Exception.Message)"
+    }
+    if ($null -eq $document -or $document -isnot [System.Management.Automation.PSCustomObject]) {
+        throw "$name must hold a JSON object."
+    }
+    $isInteger = { param($Value) $Value -is [int] -or $Value -is [long] }
+    $version = $document.PSObject.Properties['version']
+    if ($null -eq $version -or -not (& $isInteger $version.Value) -or $version.Value -ne 1) {
+        throw "$name must declare version 1."
+    }
+    $unit = $document.PSObject.Properties['unit']
+    if ($null -eq $unit -or [string] $unit.Value -cne 'ms') {
+        throw "$name must declare unit 'ms'."
+    }
+    $files = $document.PSObject.Properties['files']
+    if ($null -eq $files -or $files.Value -isnot [System.Management.Automation.PSCustomObject] -or @($files.Value.PSObject.Properties).Count -eq 0) {
+        throw "$name must list at least one file in a 'files' object."
+    }
+
+    $weights = New-Object 'System.Collections.Generic.Dictionary[string,long]' ([System.StringComparer]::OrdinalIgnoreCase)
+    # Validate each entry: a repository-relative forward-slash key and a non-negative integer of milliseconds.
+    foreach ($entry in $files.Value.PSObject.Properties) {
+        if ([string]::IsNullOrWhiteSpace($entry.Name) -or $entry.Name.Contains('\') -or $entry.Name.StartsWith('/') -or $entry.Name.Contains(':')) {
+            throw "$name lists '$($entry.Name)', which is not a repository-relative forward-slash path."
+        }
+        if (-not (& $isInteger $entry.Value) -or $entry.Value -lt 0) {
+            throw "$name gives '$($entry.Name)' the weight '$($entry.Value)'; weights must be non-negative integers."
+        }
+        $weights[$entry.Name] = [long] $entry.Value
+    }
+    # A dictionary is not enumerated by the pipeline, so it is returned as one object.
+    $weights
+}
+
+<#
+.SYNOPSIS
+Returns the shard weight of each enumerated test file (issue #953, CR-19; measured weights issue #1048).
+.DESCRIPTION
+When RootPath is supplied and ReadWeightsText returns text, the text is parsed with
+ConvertFrom-PoshQCShardWeightJson and each test file weighs the milliseconds listed for its repository-relative
+forward-slash path; a file without an entry weighs the median of all listed values, so bytes and milliseconds
+are never mixed. Otherwise the size path runs: a FileInfo item weighs the UTF-8 byte count of its text after
+every CRLF pair is replaced by LF, so a checkout with CRLF line endings computes the same weights, and therefore
+the same shard assignment, as a checkout with LF line endings. Another object weighs its Length property when it
+has one; a plain path string, or an object without a Length property, weighs 1. Returns one weight per item, in
+input order.
 .PARAMETER TestFile
 Enumerated test files: FileInfo items, objects with FullName (and optionally Length), or path strings.
 .PARAMETER ReadTestFileText
 Seam that returns the text of a test file from its full path. Defaults to [System.IO.File]::ReadAllText.
+.PARAMETER RootPath
+Optional workspace root. Without it the size path runs.
+.PARAMETER ReadWeightsText
+Seam that receives RootPath and returns the text of RootPath/config/poshqc-shard-weights.json, or $null when the
+file does not exist. Defaults to Test-Path and Get-Content -Raw.
 #>
 function Get-PoshQCShardTestFileWeight {
     [CmdletBinding()]
     [OutputType([long[]])]
     param(
         [Parameter(Mandatory)][AllowEmptyCollection()][object[]] $TestFile,
-        [scriptblock] $ReadTestFileText = { param([string] $Path) [System.IO.File]::ReadAllText($Path) }
+        [scriptblock] $ReadTestFileText = { param([string] $Path) [System.IO.File]::ReadAllText($Path) },
+        [string] $RootPath,
+        [scriptblock] $ReadWeightsText = {
+            param([string] $Root)
+            $weightsPath = Join-Path (Join-Path $Root 'config') 'poshqc-shard-weights.json'
+            if (Test-Path -LiteralPath $weightsPath) { Get-Content -Raw -LiteralPath $weightsPath } else { $null }
+        }
     )
 
+    $weightsText = if ($RootPath) { & $ReadWeightsText $RootPath } else { $null }
+    if ($null -ne $weightsText) {
+        $measured = ConvertFrom-PoshQCShardWeightJson -Json ([string] $weightsText)
+        [long[]] $listed = @($measured.Values)
+        [System.Array]::Sort($listed)
+        $middle = [int] [math]::Floor($listed.Count / 2)
+        $median = if ($listed.Count % 2 -eq 1) { $listed[$middle] } else { [long] [math]::Floor(($listed[$middle - 1] + $listed[$middle]) / 2) }
+        $prefix = ($RootPath -replace '\\', '/').TrimEnd('/') + '/'
+        # Look up each file by its root-relative forward-slash path; an unlisted file weighs the median.
+        $measuredWeights = New-Object 'System.Collections.Generic.List[long]'
+        foreach ($item in $TestFile) {
+            $path = $(if ($item -is [string]) { $item } else { [string] $item.FullName }) -replace '\\', '/'
+            $relative = if ($path.StartsWith($prefix, [System.StringComparison]::OrdinalIgnoreCase)) { $path.Substring($prefix.Length) } else { $path }
+            $weight = [long] 0
+            if ($measured.TryGetValue($relative, [ref] $weight)) { $measuredWeights.Add($weight) } else { $measuredWeights.Add($median) }
+        }
+        return [long[]] $measuredWeights.ToArray()
+    }
+
     $weights = New-Object 'System.Collections.Generic.List[long]'
     foreach ($item in $TestFile) {
         if ($item -is [System.IO.FileInfo]) {

package/resources/powershell/PoshQC/PoshQC.Testing.psm1

@@ -158,6 +158,12 @@ Number of shards. When greater than 0, Run.Path is set to this shard's size-bala
 Optional path that receives the shard manifest (shardIndex, shardCount, all, selected) as JSON.
 .PARAMETER WriteShardFileList
 Injectable seam that writes the shard manifest JSON. Defaults to Set-Content.
+.PARAMETER TestResultPath
+Optional JUnit output path that replaces the settings TestResult.OutputPath for this invocation, so concurrent
+shards write distinct files (issue #1048). A relative path is resolved against Root. Omitted or blank keeps the settings value.
+.PARAMETER CoverageOutputPath
+Optional coverage output path that replaces the settings CodeCoverage.OutputPath for this invocation (issue #1048).
+A relative path is resolved against Root. Omitted or blank keeps the settings value.
 #>
 function Invoke-PoshQCTest {
     [CmdletBinding()]
@@ -171,6 +177,8 @@ function Invoke-PoshQCTest {
         [int] $ShardIndex = 0,
         [int] $ShardCount = 0,
         [string] $ShardFileListPath,
+        [string] $TestResultPath,
+        [string] $CoverageOutputPath,
         [scriptblock] $WriteShardFileList = { param([string] $Path, [string] $Json) Set-Content -LiteralPath $Path -Value $Json -Encoding utf8 },
         [scriptblock] $EnsureModule = {
             param([string] $Name, [string] $ErrorMessage, [string] $MinimumVersion, [string] $MaximumVersion)
@@ -325,6 +333,10 @@ function Invoke-PoshQCTest {
 
     $settings = & $LoadSettings $SettingsPath
     $config = & $BuildConfiguration $settings
+    # Issue #1048: per-invocation output paths, applied before EnsureResultPath and ExpandCoveragePaths so the
+    # relative-to-root resolution and directory creation also apply to them.
+    if (-not [string]::IsNullOrWhiteSpace($TestResultPath)) { $config.TestResult.OutputPath = $TestResultPath }
+    if (-not [string]::IsNullOrWhiteSpace($CoverageOutputPath) -and $config.CodeCoverage) { $config.CodeCoverage.OutputPath = $CoverageOutputPath }
     $config = & $ExpandRunPaths $config $Root $ExcludeDirs
     $config = & $EnsureResultPath $config $Root
     $config = & $ExpandCoveragePaths $config $Root

package/resources/powershell/PoshQC/PoshQC.psd1

@@ -22,7 +22,12 @@
         'Get-PoshQCCoverageLineSummary',
         'New-PoshQCShardManifest',
         'Test-PoshQCShardCompleteness',
-        'Test-PoshQCShardTestCount'
+        'Test-PoshQCShardTestCount',
+        'Get-PoshQCParallelWorkerCount',
+        'Merge-PoshQCJunitReport',
+        'ConvertTo-PoshQCShardWeight',
+        'Export-PoshQCShardWeight',
+        'Invoke-PoshQCParallelTest'
     )
     CmdletsToExport      = @()
     VariablesToExport    = @()

package/resources/powershell/PoshQC/PoshQC.psm1

@@ -121,6 +121,7 @@ foreach ($subModuleName in @(
         'PoshQC.Analyzer.psm1',
         'PoshQC.Shard.psm1',
         'PoshQC.CoverageMerge.psm1',
+        'PoshQC.Parallel.psm1',
         'PoshQC.Testing.psm1'
     )) {
     $subModulePath = Join-Path $script:ModuleRoot $subModuleName
@@ -155,6 +156,11 @@ Export-ModuleMember -Function @(
     'Get-PoshQCCoverageLineSummary',
     'New-PoshQCShardManifest',
     'Test-PoshQCShardCompleteness',
-    'Test-PoshQCShardTestCount'
+    'Test-PoshQCShardTestCount',
+    'Get-PoshQCParallelWorkerCount',
+    'Merge-PoshQCJunitReport',
+    'ConvertTo-PoshQCShardWeight',
+    'Export-PoshQCShardWeight',
+    'Invoke-PoshQCParallelTest'
 ) -Alias @('Install-PoshQCTools')
 

package/resources/powershell/PoshQC/README.md

@@ -65,6 +65,10 @@ PoshQC is a lightweight PowerShell quality gate that wraps Invoke-Formatter, PSS
 - `Test-PoshQCShardTestCount`: throw unless the shard test counts sum to a discovery-pass test count.
 - `Merge-PoshQCCoverageReport`: merge the CoverageGutters XML texts of the shards at LINE granularity (no file I/O, no threshold).
 - `Get-PoshQCCoverageLineSummary`: return covered, missed, total, and percent from a report's LINE counter.
+- `Invoke-PoshQCParallelTest`: run the Pester suite as concurrent shard processes on one machine and merge the results (see Local concurrent runs below).
+- `Get-PoshQCParallelWorkerCount`: return the default worker count, half the logical processors, at least 1 and at most 8.
+- `Merge-PoshQCJunitReport`: merge the JUnit XML texts of the shards into one report (no file I/O).
+- `ConvertTo-PoshQCShardWeight` / `Export-PoshQCShardWeight`: build the measured shard-weights document from JUnit reports, and write it to a file.
 
 ## Configuration
 
@@ -81,10 +85,25 @@ PoshQC is a lightweight PowerShell quality gate that wraps Invoke-Formatter, PSS
   - An empty population disables coverage for that invocation, logs `Code coverage disabled for this invocation`, and still runs the tests.
   - Each run logs `Code coverage population: source=<source>; files=<count>` before Pester starts.
 - Sharded runs (issue #953): `Invoke-PoshQCTest -ShardIndex <n> -ShardCount <N>` runs only shard n of N.
-  - Test files are weighted by the UTF-8 size in bytes of their text with CRLF line endings counted as LF, and assigned heaviest first to the shard with the least assigned weight (ties: fewer files, then the lower shard index; equal sizes are taken in ordinal path order), so runners whose checkouts differ only in line-ending conversion compute the same assignment.
+  - Test files are weighted by measured duration first (issue #1048): when `config/poshqc-shard-weights.json` exists at the workspace root, a listed file weighs its milliseconds and a file without an entry weighs the median of all listed values, so bytes and milliseconds are never mixed. Without the weights file, test files are weighted by the UTF-8 size in bytes of their text with CRLF line endings counted as LF, so runners whose checkouts differ only in line-ending conversion compute the same assignment. Files are assigned heaviest first to the shard with the least assigned weight (ties: fewer files, then the lower shard index; equal weights are taken in ordinal path order).
+  - Weights file format: `{"version": 1, "unit": "ms", "files": {"<repository-relative forward-slash path>": <non-negative integer milliseconds>}}`. A document with another `version`, another `unit`, an empty `files` map, a non-relative key, or a negative or non-integer value is rejected with an error that names `poshqc-shard-weights.json`. Entries for files that no longer exist are ignored. The file is committed, so every CI shard computes the same assignment; stale or missing entries affect wall-clock time only, because the completeness and test-count checks still guard that every file runs once.
+  - Regenerate the weights from the shard JUnit files of a full CI run, not from a local run (local durations differ from the runner's). `Export-PoshQCShardWeight` maps each JUnit `testsuite` name, an absolute path on the runner, to the path from its first `tests` or `scripts` segment, sums seconds per file, and writes integer milliseconds with sorted keys:
+
+    ```powershell
+    gh run download <run id> --pattern 'poshqc-test-results-shard-*' --dir <download dir>
+    Import-Module ./scripts/powershell/PoshQC/PoshQC.psd1
+    Export-PoshQCShardWeight -JunitPath (Get-ChildItem -LiteralPath <download dir> -Recurse -Filter pester-junit.xml).FullName -OutputPath config/poshqc-shard-weights.json
+    ```
   - `-ShardFileListPath <path>` writes the shard manifest (`shardIndex`, `shardCount`, `all`, `selected`) as JSON through the `-WriteShardFileList` seam.
   - A shard that selects no test file (more shards than test files) stops with an error before Pester runs.
   - `Merge-PoshQCCoverageReport` merges the shard reports with `ci = max(ci)` per line and recomputes the LINE and INSTRUCTION counters, so the merged LINE figures are exact. It omits `class` and `method` elements and METHOD and CLASS counters, because those cannot be recomputed from line hits.
+- Output-path parameters (issue #1048): `Invoke-PoshQCTest -TestResultPath <path> -CoverageOutputPath <path>` replace the JUnit and coverage output paths of the settings file for one invocation; a relative path is resolved against `-Root`. When a parameter is omitted or blank, the settings value is used.
+- Local concurrent runs (issue #1048): `Invoke-PoshQCParallelTest -Root . [-WorkerCount <n>]` runs the suite as concurrent shard processes on one machine. Pester 5 has no in-process parallel mode, so each shard is a separate `pwsh` process; the VS Code task `PoshQC: 4 test (Pester, parallel)` runs it.
+  - The worker count defaults to half the logical processor count, at least 1 and at most 8 (`Get-PoshQCParallelWorkerCount`); `-WorkerCount` overrides it.
+  - Each shard runs `Invoke-PoshQCTest -ShardIndex <i> -ShardCount <N>` with its JUnit, coverage, and manifest files and its console log (`shard.log`, standard error in `shard.log.err`) under `artifacts/pester/parallel/shard-<i>/`.
+  - After every shard has exited, the runner checks the manifests with `Test-PoshQCShardCompleteness`, merges the JUnit reports with `Merge-PoshQCJunitReport` and the coverage reports with `Merge-PoshQCCoverageReport`, and writes `artifacts/pester/pester-junit.xml`, `artifacts/pester/powershell-coverage.xml`, and the `powershell-coverage.koverage.xml` copy.
+  - The merged JUnit root carries the summed `tests`, `errors`, `failures`, and `disabled` counts, every shard's `testsuite` elements with `id` renumbered from 0, and a root `time` equal to the longest shard's root `time`: the shards run at the same time, so the longest shard is the wall-clock time of the run, not the sum of test time.
+  - The runner returns an object with `ExitCode`, `WorkerCount`, `WallClockSeconds`, and per-shard exit codes and log paths. `ExitCode` is 0 only when every shard exited 0, wrote all three outputs, passed the completeness check, and merged; otherwise it is 1 and no merged artifact is written, so a partial result never replaces a full one. The runner never calls `exit`; the VS Code task exits with the returned `ExitCode`.
 
 ## Typical workflow
 

package/resources/templates/policy_audit/code-review.yyyy-MM-ddTHH-mm.md

@@ -9,7 +9,7 @@
 > 2. Replace all placeholders with concrete branch, file, test, and evidence details.
 > 3. Keep the sections that apply to the reviewed scope and delete inapplicable language subsections.
 > 4. Base all findings on verified evidence: diff inspection, toolchain output, coverage data, and feature-folder artifacts.
-> 5. Use severity labels consistently: `Blocker`, `Major`, `Minor`, `Nit`, `Info`.
+> 5. Use severity labels consistently: `Blocker`, `Major`, `Minor`, `Nit`, `Info`, `Warning`. `Warning` is the non-blocking file-size advisory for a covered file at or above 700 logical lines; it names the file and its line count and is never carried into remediation inputs.
 > 6. End with a clear readiness recommendation that matches the actual findings.
 >
 > **When to use:**
@@ -52,8 +52,8 @@
 
 | Severity | File | Location | Finding | Recommendation | Rationale | Evidence |
 |---|---|---|---|---|---|---|
-| [Blocker/Major/Minor/Nit/Info] | `[path]` | `[line range / section / n/a]` | [Concrete issue] | [Concrete corrective action or note] | [Why it matters] | [Command output, artifact path, or inspected file reference] |
-| [Blocker/Major/Minor/Nit/Info] | `[path]` | `[line range / section / n/a]` | [Concrete issue] | [Concrete corrective action or note] | [Why it matters] | [Command output, artifact path, or inspected file reference] |
+| [Blocker/Major/Minor/Nit/Info/Warning] | `[path]` | `[line range / section / n/a]` | [Concrete issue] | [Concrete corrective action or note] | [Why it matters] | [Command output, artifact path, or inspected file reference] |
+| [Blocker/Major/Minor/Nit/Info/Warning] | `[path]` | `[line range / section / n/a]` | [Concrete issue] | [Concrete corrective action or note] | [Why it matters] | [Command output, artifact path, or inspected file reference] |
 
 [If applicable, add a short line such as `No Blockers or Major findings.`]
 
@@ -162,4 +162,4 @@
 
 ## Verdict
 
-[Provide the final review conclusion in 1-2 paragraphs. State whether the change is ready for normal PR flow, ready after a specific follow-up, or blocked. Ensure this conclusion is consistent with the Findings Table and PR readiness recommendation above.]
+[Provide the final review conclusion in 1-2 paragraphs. State whether the change is ready for normal PR flow, ready after a specific follow-up, or blocked. Ensure this conclusion is consistent with the Findings Table and PR readiness recommendation above.]

package/resources/templates/policy_audit/policy-audit.yyyy-MM-ddTHH-mm.md

@@ -159,7 +159,7 @@ Repeat one bullet per in-scope language that has coverage requirements. Keep the
 | Requirement | Status | Evidence |
 |------------|--------|----------|
 | **Cohesive modules** | [✅/❌/N/A] [PASS/FAIL/N/A] | [Describe the purpose and cohesion of each module/file. Note organization strategy.] |
-| **Under 500 lines** | [✅/❌/N/A] [PASS/FAIL/N/A] | [Report line counts for all files. Note any exceptions and justifications.] |
+| **At or below the 800-line hard limit** | [✅/❌/N/A] [PASS/FAIL/N/A] | [Report logical line counts for all covered files. FAIL for a file above 800 lines without a `config/file-size-baseline.json` entry, a baselined file that grew, or an added or increased baseline entry. Note each file at or above 700 lines as a non-blocking Warning. Note any exceptions.] |
 | **Public vs internal** | [✅/❌/N/A] [PASS/FAIL/N/A] | [Describe the public API surface. Note how internals are hidden.] |
 | **No circular dependencies** | [✅/❌/N/A] [PASS/FAIL/N/A] | [Describe the dependency graph. Confirm no circular dependencies exist.] |
 
@@ -253,7 +253,7 @@ Repeat one bullet per in-scope language that has coverage requirements. Keep the
 
 | Requirement | Status | Evidence |
 |------------|--------|----------|
-| **Cohesive and under 500 lines** | [✅/❌/N/A] [PASS/FAIL/N/A] | [Report file line counts. Confirm all under limit.] |
+| **Cohesive and at or below the 800-line hard limit** | [✅/❌/N/A] [PASS/FAIL/N/A] | [Report logical line counts. FAIL for a file above 800 lines without a baseline entry or a baselined file that grew. Note each file at or above 700 lines as a non-blocking Warning.] |
 | **Approved verbs** | [✅/❌/N/A] [PASS/FAIL/N/A] | [List all function names and confirm verbs are approved.] |
 | **Comment why** | [✅/❌/N/A] [PASS/FAIL/N/A] | [Describe commenting approach. Confirm focus on rationale.] |
 
@@ -285,7 +285,7 @@ Repeat one bullet per in-scope language that has coverage requirements. Keep the
 |------------|--------|----------|
 | **Portable shebang** | [✅/❌/N/A] [PASS/FAIL/N/A] | [Confirm `#!/usr/bin/env bash` or `#!/bin/bash` used.] |
 | **Error handling** | [✅/❌/N/A] [PASS/FAIL/N/A] | [Describe use of `set -e`, `set -u`, `set -o pipefail`, error traps.] |
-| **Under 500 lines** | [✅/❌/N/A] [PASS/FAIL/N/A] | [Report file line counts.] |
+| **At or below the 800-line hard limit** | [✅/❌/N/A] [PASS/FAIL/N/A] | [Report logical line counts. FAIL for a file above 800 lines without a baseline entry or a baselined file that grew. Note each file at or above 700 lines as a non-blocking Warning.] |
 
 ---
 
@@ -424,7 +424,7 @@ Repeat one bullet per in-scope language that has coverage requirements. Keep the
 | Average Time per Test | [X]ms | [✅/❌] Fast/Slow |
 | Discovery Time | [X]ms | [✅/❌] |
 | Functions/Classes Tested | [N]/[M] ([X]%) | [✅/❌] |
-| Test File Size | [N] lines | [✅/❌] Maintainable |
+| Test File Size | [N] logical lines | [✅/❌] PASS at or below 800 lines, FAIL above 800 without a baseline entry; note a non-blocking Warning at or above 700 lines |
 | Code Coverage (if applicable) | [X]% lines, [Y]% branches | [✅/❌] |
 
 ---