fix: address self-review — clone before mutating filterData, filter empty fields, consistent 429 exit code in health

Co-Authored-By: Chris K <ckorhonen@gmail.com>

Author: devin-ai-integration[bot]

src/cli.ts

@@ -87,7 +87,10 @@ function getFilters(): OutputFilterOptions {
     maxItems?: string
   }>()
   return {
-    fields: opts.fields?.split(",").map(f => f.trim()),
+    fields: opts.fields
+      ?.split(",")
+      .map(f => f.trim())
+      .filter(Boolean),
     maxItems: opts.maxItems
       ? parseIntOption(opts.maxItems, "--max-items")
       : undefined,

src/commands/health.ts

@@ -26,7 +26,7 @@ export function healthCommand(getClient: () => OpenSeaClient): Command {
               2,
             ),
           )
-          process.exit(1)
+          process.exit(error.statusCode === 429 ? 3 : 1)
         }
         console.error(
           JSON.stringify(

src/output.ts

@@ -20,7 +20,7 @@ export function filterData(
     result &&
     typeof result === "object"
   ) {
-    const obj = result as Record<string, unknown>
+    const obj = { ...(result as Record<string, unknown>) }
     for (const [key, value] of Object.entries(obj)) {
       if (Array.isArray(value)) {
         obj[key] = value.slice(0, options.maxItems)