fix(SigningRequirementValidator): remove signerHasSignRequest shortcut that bypassed visibleElements check

The shortcut added in e9ea794 caused needsCreateSignature to return true
whenever the signer had a signRequestId, regardless of whether any visual
element box was placed. This silently opened the draw modal for clickToSign
documents with no placed elements.

Remove the shortcut and keep only the visibleElements.some() check,
also normalising signRequestId comparison to String() to be type-safe.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>

Author: vitormattos

src/services/SigningRequirementValidator.ts

@@ -35,7 +35,6 @@ interface ValidatorConfig {
 	errors?: Array<{ code?: number; [key: string]: unknown }>
 	hasSignatures?: boolean
 	canCreateSignature?: boolean
-	signerHasSignRequest?: boolean
 	[key: string]: unknown
 }
 
@@ -122,11 +121,7 @@ export class SigningRequirementValidator {
 			return false
 		}
 
-		if (config.signerHasSignRequest) {
-			return true
-		}
-
 		const visibleElements = this.signStore.document?.visibleElements || []
-		return visibleElements.some(row => row.signRequestId === signRequestId)
+		return visibleElements.some(row => String(row.signRequestId) === String(signRequestId))
 	}
 }