refactor(fetch-all): simplify pageId branch in fetchAllNotionData

- Skip applyFetchAllTransform in the single-page path: bulk-mode sorting,
  deduplication, and status-based child resolution are not needed for a
  single page retrieved directly by ID. Passing pageId+statusFilter no longer
  silently drops the page when statuses don't match.
- Preserve includeRemoved guard: pages with status "Remove" are still
  filtered out when includeRemoved=false.
- Remove dead candidateIds logic: only consumed by fetch-ready; return [] unconditionally.
- Remove redundant defensivelyFiltered step that re-applied the same Remove
  filter already handled above.

Author: luandro

scripts/notion-fetch-all/fetchAll.ts

@@ -76,24 +76,17 @@ export async function fetchAllNotionData(
     generateOptions = {},
   } = options;
 
-  // Single-page fetch uses a direct page retrieval, then generation-only path
-  // to avoid a full database query.
+  // Single-page fetch uses a direct page retrieval, skips bulk-mode transforms,
+  // then runs generation-only path to avoid a full database query.
   if (pageId) {
     const page = await notion.pages.retrieve({ page_id: pageId });
     const fetchedCount = 1;
-    let candidateIds: string[] = [];
     const basePage = page as Record<string, unknown>;
 
-    if (statusFilter && getStatusFromRawPage(basePage) === statusFilter) {
-      candidateIds = [(basePage as any).id];
-    }
-
-    const transformedPages = applyFetchAllTransform([basePage], {
-      statusFilter,
-      maxPages,
-      includeRemoved,
-    });
-    const rawData = Array.isArray(transformedPages) ? transformedPages : [];
+    const rawData =
+      !includeRemoved && getStatusFromRawPage(basePage) === "Remove"
+        ? []
+        : [basePage];
 
     let metrics: FetchAllResult["metrics"] | undefined;
     if (exportFiles) {
@@ -107,21 +100,15 @@ export async function fetchAllNotionData(
       metrics = generationResult.metrics;
     }
 
-    const defensivelyFiltered = rawData.filter((p) => {
-      const status = getStatusFromRawPage(p);
-      if (!includeRemoved && status === "Remove") return false;
-      return true;
-    });
-
-    const pages = defensivelyFiltered.map((rawPage) => transformPage(rawPage));
+    const pages = rawData.map((rawPage) => transformPage(rawPage));
     const sortedPages = sortPages(pages, sortBy, sortDirection);
 
     logStatusSummary(sortedPages);
 
     return {
       pages: sortedPages,
-      rawPages: defensivelyFiltered,
-      candidateIds,
+      rawPages: rawData,
+      candidateIds: [],
       metrics,
       fetchedCount,
       processedCount: sortedPages.length,