fix(wordpress): use ?? instead of || for zero-valued numeric fields in delete_category/tag

waleedlatif1 · waleedlatif1 · commit 8cb03d73e0c5 · 2026-07-01T23:57:56.000-07:00
count and parent can legitimately be 0 (empty term, top-level category); || was
dropping those values, same antipattern already fixed for `deleted` in this PR.
Flagged independently by Greptile and Cursor Bugbot.
diff --git a/apps/sim/tools/wordpress/delete_category.ts b/apps/sim/tools/wordpress/delete_category.ts
@@ -60,14 +60,14 @@ export const deleteCategoryTool: ToolConfig<
       output: {
         deleted: data.deleted ?? true,
         category: {
-          id: data.id || data.previous?.id,
-          count: data.count || data.previous?.count,
+          id: data.id ?? data.previous?.id,
+          count: data.count ?? data.previous?.count,
           description: data.description || data.previous?.description,
           link: data.link || data.previous?.link,
           name: data.name || data.previous?.name,
           slug: data.slug || data.previous?.slug,
           taxonomy: data.taxonomy || data.previous?.taxonomy,
-          parent: data.parent || data.previous?.parent,
+          parent: data.parent ?? data.previous?.parent,
         },
       },
     }
diff --git a/apps/sim/tools/wordpress/delete_tag.ts b/apps/sim/tools/wordpress/delete_tag.ts
@@ -57,8 +57,8 @@ export const deleteTagTool: ToolConfig<WordPressDeleteTagParams, WordPressDelete
       output: {
         deleted: data.deleted ?? true,
         tag: {
-          id: data.id || data.previous?.id,
-          count: data.count || data.previous?.count,
+          id: data.id ?? data.previous?.id,
+          count: data.count ?? data.previous?.count,
           description: data.description || data.previous?.description,
           link: data.link || data.previous?.link,
           name: data.name || data.previous?.name,
