From 73c6696873ce321d01e353072dfcd6e24a8fa82c Mon Sep 17 00:00:00 2001
From: Aiden McClelland <me@drbonez.dev>
Date: Mon, 9 Mar 2026 23:59:58 -0600
Subject: [PATCH] refactor: simplify AddPackageSignerParams merge field from
 Option<bool> to bool

---
 core/src/registry/package/signer.rs | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/core/src/registry/package/signer.rs b/core/src/registry/package/signer.rs
index 47ec7b13d..ee1cbc47a 100644
--- a/core/src/registry/package/signer.rs
+++ b/core/src/registry/package/signer.rs
@@ -59,8 +59,7 @@ pub struct AddPackageSignerParams {
     #[ts(type = "string | null")]
     pub versions: Option<VersionRange>,
     #[arg(long, help = "help.arg.merge")]
-    #[ts(optional)]
-    pub merge: Option<bool>,
+    pub merge: bool,
 }
 
 pub async fn add_package_signer(
@@ -89,7 +88,7 @@ pub async fn add_package_signer(
                 .as_authorized_mut()
                 .upsert(&signer, || Ok(VersionRange::None))?
                 .mutate(|existing| {
-                    *existing = if merge.unwrap_or(false) {
+                    *existing = if merge {
                         VersionRange::or(existing.clone(), versions)
                     } else {
                         versions