[HalRiveria-Bot]

Follow-ups to #5, from a review after it merged.

What was wrong, and what changed:

• Shared collections could crash a run. With -NextJobProvider, the scriptblock scan runs while earlier workers are going. A results sink shared by every job (a synchronized ArrayList or hashtable) threw "Collection was modified" and stopped the run. Such a value is now skipped instead.
• Big arrays were walked per job. A byte[] of file contents or a large string[] cost a PowerShell loop over every element before each worker started. Collections whose element type can't hold a scriptblock aren't walked now.
• Nested scriptblocks got through. The scan looked one level deep, so @(@{ Run = { ... } }) passed. It now looks three levels deep and into [pscustomobject] properties.
• Get-Command cost on every call. The check ran Get-Command for each command name, and an unknown name searched every installed module. That happened on every Invoke-PkRunspaceJobs call, now that the check is on by default. It now reads the Alias: and Function: drives.
• Module functions were always trusted. Worker rules: allow cmdlets, check by default, refuse scriptblock job values #5 exempted any function from a module, so one imported by path, which a worker can't load by name, failed at runtime instead of being refused. It's flagged again, with a message saying to put the module on $env:PSModulePath or share the function.
• Storing a scriptblock got the run refused. $list.Add($sb), $map.Add($k, $sb) and $queue.Enqueue($sb) were flagged as delegates. Collection-storage methods are now exempt. The README notes the one case this misses (a collection typed to hold delegates).
• Alias coverage. Worker rules: allow cmdlets, check by default, refuse scriptblock job values #5 dropped the harness's alias-resolution assertions; a MissingFunction-through-alias case is back.

Testing. 188 assertions pass on Windows PowerShell 5.1, and on PowerShell 7.6 when merged with #6. Six of the new assertions fail against main's kit:

• storage methods;
• a module imported by path;
• a scriptblock two levels down;
• a [pscustomobject] property;
• a 20 MB byte[] taking 1.5s against under 1s;
• a concurrently modified sink throwing "Collection was modified".

The Get-Command change has no timing assertion, since a threshold would be flaky on CI.

🤖 Generated with Claude Code