fix: cap API per_page, guard null alert names, fix log hasMore logic

src/commands/alert/issues/list.ts

@@ -349,7 +349,9 @@ async function handleResolvedTargets(
   );
   const filteredRows = flags.query
     ? allRows.filter((row) =>
-        row.rule.name.toLowerCase().includes(flags.query?.toLowerCase() ?? "")
+        (row.rule.name ?? "")
+          .toLowerCase()
+          .includes(flags.query?.toLowerCase() ?? "")
       )
     : allRows;
 
@@ -471,7 +473,7 @@ function formatIssueAlertListHuman(result: IssueAlertListResult): string {
 
   const tableRows: Row[] = rows.map(({ rule: r, target }) => ({
     id: r.id,
-    name: escapeMarkdownCell(r.name),
+    name: escapeMarkdownCell(r.name ?? "(untitled)"),
     ...(isMultiProject && {
       project: `${target.org}/${target.project}`,
     }),

src/commands/alert/issues/rule-resolve.ts

@@ -133,10 +133,12 @@ export async function resolveIssueAlertRule(
   for (const target of targets) {
     const rules = await listAllIssueRulesForTarget(target);
     for (const r of rules) {
-      allRuleNames.push(r.name);
+      if (r.name) {
+        allRuleNames.push(r.name);
+      }
     }
     const exact = rules.find(
-      (rule) => rule.name.toLowerCase() === ref.toLowerCase()
+      (rule) => rule.name && rule.name.toLowerCase() === ref.toLowerCase()
     );
     if (exact) {
       hits.push({ target, rule: exact });

src/commands/alert/metrics/list.ts

@@ -347,7 +347,9 @@ async function handleResolvedOrgs(
   );
   const filteredRows = flags.query
     ? allRows.filter((row) =>
-        row.rule.name.toLowerCase().includes(flags.query?.toLowerCase() ?? "")
+        (row.rule.name ?? "")
+          .toLowerCase()
+          .includes(flags.query?.toLowerCase() ?? "")
       )
     : allRows;
 
@@ -470,7 +472,7 @@ function formatMetricAlertListHuman(result: MetricAlertListResult): string {
 
   const tableRows: Row[] = rows.map(({ rule: r, orgSlug }) => ({
     id: r.id,
-    name: escapeMarkdownCell(r.name),
+    name: escapeMarkdownCell(r.name ?? "(untitled)"),
     ...(isMultiOrg && { org: orgSlug }),
     aggregate: r.aggregate,
     dataset: r.dataset,

src/commands/alert/metrics/rule-resolve.ts

@@ -129,10 +129,12 @@ export async function resolveMetricAlertRule(
   for (const orgSlug of orgSlugs) {
     const rules = await listAllMetricRulesForOrg(orgSlug);
     for (const r of rules) {
-      allNames.push(r.name);
+      if (r.name) {
+        allNames.push(r.name);
+      }
     }
     const exact = rules.find(
-      (rule) => rule.name.toLowerCase() === ref.toLowerCase()
+      (rule) => rule.name && rule.name.toLowerCase() === ref.toLowerCase()
     );
     if (exact) {
       hits.push({ orgSlug, rule: exact });

src/commands/log/list.ts

@@ -10,6 +10,7 @@
 import * as Sentry from "@sentry/node-core/light";
 import type { SentryContext } from "../../context.js";
 import {
+  API_MAX_PER_PAGE,
   type LogSortDirection,
   listLogs,
   listTraceLogs,
@@ -196,7 +197,8 @@ async function executeSingleFetch(
     };
   }
 
-  const hasMore = logs.length >= flags.limit;
+  const effectivePerPage = Math.min(flags.limit, API_MAX_PER_PAGE);
+  const hasMore = logs.length >= effectivePerPage;
   const countText = `Showing ${logs.length} log${logs.length === 1 ? "" : "s"}.`;
   const tip = hasMore ? " Use --limit to show more, or -f to follow." : "";
 
@@ -501,7 +503,8 @@ async function executeTraceSingleFetch(
     };
   }
 
-  const hasMore = logs.length >= flags.limit;
+  const effectivePerPage = Math.min(flags.limit, API_MAX_PER_PAGE);
+  const hasMore = logs.length >= effectivePerPage;
   const countText = `Showing ${logs.length} log${logs.length === 1 ? "" : "s"} for trace ${traceId}.`;
   const tip = hasMore ? " Use --limit to show more." : "";
 

src/commands/project/list.ts

@@ -12,6 +12,7 @@
 
 import type { SentryContext } from "../../context.js";
 import {
+  API_MAX_PER_PAGE,
   findProjectsBySlug,
   getProject,
   listOrganizations,
@@ -451,7 +452,7 @@ export async function handleOrgAll(
     () =>
       listProjectsPaginated(org, {
         cursor,
-        perPage: flags.limit,
+        perPage: Math.min(flags.limit, API_MAX_PER_PAGE),
       })
   );
 

src/lib/api/logs.ts

@@ -161,7 +161,7 @@ export async function listLogs(
       field: fields,
       project: isNumericProject ? [Number(projectSlug)] : undefined,
       query: fullQuery || undefined,
-      per_page: options.limit || API_MAX_PER_PAGE,
+      per_page: Math.min(options.limit || API_MAX_PER_PAGE, API_MAX_PER_PAGE),
       statsPeriod:
         options.start || options.end
           ? undefined
@@ -343,7 +343,7 @@ export async function listTraceLogs(
             : (options.statsPeriod ?? "14d"),
         start: options.start,
         end: options.end,
-        per_page: options.limit ?? API_MAX_PER_PAGE,
+        per_page: Math.min(options.limit ?? API_MAX_PER_PAGE, API_MAX_PER_PAGE),
         query: options.query,
         sort: toApiSort(options.sort),
       },

src/lib/org-list.ts

@@ -31,6 +31,7 @@
  */
 
 import {
+  API_MAX_PER_PAGE,
   findProjectsBySlug,
   listOrganizations,
   type PaginatedResponse,
@@ -465,7 +466,7 @@ export async function handleOrgAll<TEntity, TWithOrg>(
     () =>
       config.listPaginated(org, {
         cursor,
-        perPage: flags.limit,
+        perPage: Math.min(flags.limit, API_MAX_PER_PAGE),
       })
   );