Remove publisherName from azureSignOptions (TrustedSigning v0.4.1 compat)

[igorgudelj]

Summary

• Removes publisherName from azureSignOptions in electron-builder.mjs to fix Windows code signing failures with TrustedSigning PowerShell module v0.4.1+.
• Removes the now-unused azurePublisherName variable and simplifies the guard condition accordingly.

Problem

electron-builder delegates Windows Azure Trusted Signing to the Invoke-TrustedSigning PowerShell cmdlet (from the TrustedSigning module). In version 0.4.1 of that module, the -publisherName parameter was removed. Because NativePHP's config still passes publisherName in azureSignOptions, electron-builder forwards it as -publisherName to the cmdlet, which then fails with:

A parameter cannot be found that matches parameter name 'publisherName'.

You can verify the parameter was removed by comparing the TrustedSigning module v0.4.0 and v0.4.1 releases, or by inspecting electron-builder's windowsSignAzureManager.js which pins the install to TrustedSigning@0.4.1.

Fix

Remove the publisherName property from the azureSignOptions object, the azurePublisherName variable that fed it, and the azurePublisherName && check from the guard condition (since we no longer need that env var to be set).

🤖 Generated with Claude Code

[gwleuverink]

Good catch! Thanks 🙏

[simonhamp]

Seems reasonable