jskelet 0.1.1 → 0.1.3

package/src/server/prewarm.js

@@ -11,6 +11,12 @@
  * middleware zinciri normal trafikle bire bir aynı olsun. Hangi yolların
  * ısıtılacağını uygulama `hooks.prewarmPaths()` ile bildirir; genelde
  * sitemap üreten fonksiyonun aynısıdır.
+ *
+ * On binlerce yolluk bir sitede tur bir "damla damla" tarayıcıya dönüşür:
+ * `priority` desenleri her turda başa alınır, geri kalan kuyruk turlar
+ * arasında kaldığı yerden devam eder (`rotate`) ve `rps` toplam hızı upstream
+ * kotasının altında tutar. Amaç, kimse gelmese bile hiçbir sayfanın soğuk
+ * kalmaması — ama bunu API'yi düşürmeden yapmak.
  */
 import process from "node:process";
 import { getConfig, hook } from "../config/index.js";
@@ -64,7 +70,7 @@ async function collectPaths() {
   const paths = await hook("prewarmPaths", []);
 
   if (!Array.isArray(paths)) {
-    console.warn("[prewarm] hooks.prewarmPaths() dizi döndürmeli, yok sayıldı");
+    console.warn("[prewarm] hooks.prewarmPaths() must return an array, ignoring it");
     return [];
   }
 
@@ -78,6 +84,100 @@ async function collectPaths() {
   );
 }
 
+/**
+ * `cache().prewarm.priority` desenlerine göre sıralar. Eşleşmeyen yollar
+ * listenin sonuna, kendi aralarındaki sırayı koruyarak gider — uygulamanın
+ * verdiği sıra hâlâ anlamlı olsun.
+ *
+ * @param {string[]} paths
+ * @returns {{ head: string[], tail: string[] }}
+ *   `head` öncelikli yollar (her turda ısıtılır), `tail` geri kalan kuyruk
+ *   (turlar arasında dolaşılır).
+ */
+function byPriority(paths) {
+  const rules = getConfig().prewarmPriority;
+  if (!rules.length) return { head: [], tail: paths };
+
+  /** @type {string[][]} */
+  const buckets = rules.map(() => []);
+  /** @type {string[]} */
+  const tail = [];
+
+  for (const candidate of paths) {
+    const rank = rules.findIndex((rule) => rule.test(candidate));
+    if (rank === -1) tail.push(candidate);
+    else buckets[rank].push(candidate);
+  }
+
+  return { head: buckets.flat(), tail };
+}
+
+/**
+ * Kuyruğun kaldığı yer. Periyodik turlar listeyi baştan ısıtıp aynı ilk
+ * `max` yolu tekrar tekrar tazelemesin: her tur bir sonraki dilimi alır ve
+ * yeterli tur sonunda liste baştan sona ısınır.
+ */
+let queueCursor = 0;
+
+/**
+ * Bir turda ısıtılacak dilimi seçer: önce `priority` eşleşenler, sonra
+ * kuyruğun sırası gelen parçası. Dışa açık olması bilinçli — sıralama ve
+ * rotasyon, tur çalışmadan doğrulanabilen tek davranış.
+ *
+ * @param {string[]} all
+ * @param {number} limit
+ * @param {boolean} rotate
+ * @returns {string[]}
+ */
+export function selectPrewarmPaths(all, limit, rotate = true) {
+  if (all.length <= limit) return all;
+
+  const { head, tail } = byPriority(all);
+  const selected = head.slice(0, limit);
+  const room = limit - selected.length;
+  if (room <= 0 || !tail.length) return selected;
+
+  if (!rotate) return [...selected, ...tail.slice(0, room)];
+
+  // Dilim listenin sonunu aşarsa başa sarar: kuyruk halkasal dolaşılır.
+  const start = queueCursor % tail.length;
+  const slice = tail.slice(start, start + room);
+  if (slice.length < room) slice.push(...tail.slice(0, room - slice.length));
+  queueCursor = (start + room) % tail.length;
+
+  return [...selected, ...slice];
+}
+
+/** @param {number} ms */
+function sleep(ms) {
+  return new Promise((resolve) => {
+    setTimeout(resolve, ms).unref?.();
+  });
+}
+
+/**
+ * Saniyedeki istek sayısını sınırlar. Fren `concurrency`'den bağımsız
+ * olmalı: paralellik gecikmeyi kapatmak için var, kotayı koruyan şey toplam
+ * hız. İşçiler aynı sayacı paylaştığı için sıra kimde olursa olsun tur
+ * verilen hızın üstüne çıkmaz.
+ *
+ * @param {number} rps 0 → sınırsız
+ * @returns {() => Promise<void>}
+ */
+function createPacer(rps) {
+  if (!rps) return async () => {};
+
+  const gap = 1000 / rps;
+  let nextSlot = 0;
+
+  return async () => {
+    const now = Date.now();
+    const slot = Math.max(now, nextSlot);
+    nextSlot = slot + gap;
+    if (slot > now) await sleep(slot - now);
+  };
+}
+
 /**
  * `DEV_TOKEN` ayarlıyken `devGate` token taşımayan her isteğe 404 döner.
  * Isıtma kendi sunucusuna istek attığı için token'ı çerez olarak taşımalı;
@@ -100,9 +200,10 @@ function devGateHeader() {
  * @param {(ok: number, failed: number) => void} [report]
  *   Tur ilerlemesini `prewarmProgress`'e yazar. Tekrar turunda sayaçların
  *   anlamı değiştiği için çağıran taraf kendi formülünü verir.
+ * @param {() => Promise<void>} [pace] İstek başına beklenen hız freni.
  * @returns {Promise<{ ok: number, failed: number, failedPaths: string[] }>}
  */
-async function crawl(origin, paths, concurrency, report = undefined) {
+async function crawl(origin, paths, concurrency, report = undefined, pace = undefined) {
   const { brand } = getConfig();
   const cacheHeader = brand.cacheHeader.toLowerCase();
 
@@ -117,6 +218,8 @@ async function crawl(origin, paths, concurrency, report = undefined) {
       const target = paths[index];
       index += 1;
 
+      if (pace) await pace();
+
       const startedAt = Date.now();
 
       try {
@@ -191,8 +294,15 @@ export async function prewarm({ origin, quiet = false, paths: only }) {
     process.env.NODE_ENV === "development" ? 2 : 4,
   );
 
+  const rps = num(process.env.PREWARM_RPS, num(getConfig().prewarm?.rps, 0));
+  const pace = createPacer(rps);
+
   const all = only?.length ? only : await collectPaths();
-  const paths = all.slice(0, limit);
+  // Elle verilen liste budanmaz ve sıralanmaz: çağıran tam olarak neyi
+  // istediğini biliyor (dev panelindeki "tekrar dene" bunu kullanır).
+  const paths = only?.length
+    ? all
+    : selectPrewarmPaths(all, limit, getConfig().prewarm?.rotate !== false);
 
   Object.assign(prewarmProgress, {
     active: true,
@@ -211,7 +321,13 @@ export async function prewarm({ origin, quiet = false, paths: only }) {
   try {
     /** @type {string[]} */
     let failedPaths;
-    ({ ok, failed, failedPaths } = await crawl(origin, paths, concurrency));
+    ({ ok, failed, failedPaths } = await crawl(
+      origin,
+      paths,
+      concurrency,
+      undefined,
+      pace,
+    ));
 
     // Hatalar çoğunlukla upstream rate limit'i (429): ilk tur yüzlerce sayfayı
     // aynı anda çekerken API'yi zorluyor. Tek seri tekrar turu bu sayfaların
@@ -219,11 +335,23 @@ export async function prewarm({ origin, quiet = false, paths: only }) {
     if (failedPaths.length) {
       const firstOk = ok;
       const firstFailed = failed;
-      const retry = await crawl(origin, failedPaths, 1, (retriedOk) => {
-        // Tekrar turunda her başarı bir hatayı başarıya çevirir.
-        prewarmProgress.ok = firstOk + retriedOk;
-        prewarmProgress.failed = firstFailed - retriedOk;
-      });
+
+      // Rate limit pencereleri saniye mertebesinde; hemen tekrar denemek aynı
+      // 429'u almak demek.
+      const retryDelay = setting("PREWARM_RETRY_DELAY_MS", "retryDelayMs", 2000);
+      await sleep(retryDelay);
+
+      const retry = await crawl(
+        origin,
+        failedPaths,
+        1,
+        (retriedOk) => {
+          // Tekrar turunda her başarı bir hatayı başarıya çevirir.
+          prewarmProgress.ok = firstOk + retriedOk;
+          prewarmProgress.failed = firstFailed - retriedOk;
+        },
+        pace,
+      );
       recovered = retry.ok;
       ok += retry.ok;
       failed -= retry.ok;
@@ -236,11 +364,15 @@ export async function prewarm({ origin, quiet = false, paths: only }) {
 
   if (!quiet && paths.length) {
     const skipped = all.length - paths.length;
+    // Rotasyon açıkken sınırın dışında kalan yollar kaybolmuyor, bir sonraki
+    // tura kalıyor; log bunu ayırt etmeli, yoksa "400 yol atlandı" satırı
+    // hatalı bir kurulum sanılıyor.
+    const rotate = !only?.length && getConfig().prewarm?.rotate !== false;
     console.log(
-      `[prewarm] ${ok}/${paths.length} sayfa ısıtıldı` +
-        `${failed ? `, ${failed} hata` : ""}` +
-        `${recovered ? `, ${recovered} sayfa tekrar turunda kurtarıldı` : ""}` +
-        `${skipped > 0 ? `, ${skipped} sayfa limit dışı` : ""}` +
+      `[prewarm] warmed ${ok}/${paths.length} pages` +
+        `${failed ? `, ${failed} failed` : ""}` +
+        `${recovered ? `, ${recovered} recovered on the retry pass` : ""}` +
+        `${skipped > 0 ? `, ${skipped} ${rotate ? "deferred to the next pass" : "over the limit"}` : ""}` +
         ` (${(elapsed / 1000).toFixed(1)}s)`,
     );
   }
@@ -264,20 +396,33 @@ export function startPrewarm({ port }) {
 
   const isDev = process.env.NODE_ENV === "development";
   const origin = `http://127.0.0.1:${port}`;
-  const run = () =>
-    prewarm({ origin }).catch((error) => {
-      console.error("[prewarm] başarısız", error);
-    });
+
+  // Hız frenli bir tur `intervalSeconds`'tan uzun sürebilir; üst üste binen
+  // turlar `prewarmProgress`'i bozar ve upstream'e iki kat yük bindirir.
+  let running = false;
+  const run = async () => {
+    if (running) return;
+    running = true;
+    try {
+      await prewarm({ origin });
+    } catch (error) {
+      console.error("[prewarm] failed", error);
+    } finally {
+      running = false;
+    }
+  };
 
   // Isıtma ilk isteklerle yarışmasın diye gecikmeyle başlar. Dev'de gecikme
   // daha uzun: dosya kaydı süreci yeniden başlattığı için zamanlayıcı da
   // ölür; yalnızca sunucu bir süre sakin kalınca ısınır.
   const delay = setting("PREWARM_DELAY_MS", "delayMs", isDev ? 3000 : 500);
-  setTimeout(run, delay).unref();
+  setTimeout(() => void run(), delay).unref();
 
   // Girdiler `revalidate` ile yaşlanır; stale-while-revalidate sayesinde
   // ziyaretçi beklemez. Periyodik tur, hiç ziyaret edilmeyen sayfaları da
   // sıcak tutmak isteyen kurulumlar için opsiyoneldir.
+  // `rotate` ile birlikte bu ayar "damla damla ısıtma"ya dönüşür: her tur
+  // kuyruğun bir dilimini alır, yeterli tur sonunda liste baştan sona ısınır.
   const interval = setting("PREWARM_INTERVAL_SECONDS", "intervalSeconds", 0);
-  if (interval > 0) setInterval(run, interval * 1000).unref();
+  if (interval > 0) setInterval(() => void run(), interval * 1000).unref();
 }

package/src/server/render.js

@@ -21,6 +21,12 @@ import { matchPattern } from "../config/pattern.js";
 import { encodeText, negotiateEncoding } from "./middleware/compression.js";
 import { navigationHints, preconnectHints } from "./head-hints.js";
 import { withRequestCache } from "../http/request-cache.js";
+import {
+  createRequestContext,
+  getRequestContext,
+  guardRequest,
+  withRequestContext,
+} from "../http/request-context.js";
 import { getUpstreamFailures, withUpstreamTracking } from "./upstream-tracking.js";
 import { isNotFoundError, isRedirectError } from "../http/control-flow.js";
 import { renderHeadMeta } from "./metadata.js";
@@ -153,50 +159,129 @@ export async function renderPage(page) {
   );
 }
 
+/**
+ * Kişiye özel yanıtın cache direktifi. Dinamik (cache'lenmeyen) her sayfa da
+ * bunu alır: bir yanıt hiçbir direktif taşımadığında HTTP onu "sezgisel olarak
+ * cache'lenebilir" sayar ve araya giren bir proxy ya da tarayıcının geri
+ * tuşu kullanıcıya özel HTML'i saklayabilir.
+ */
+const PRIVATE_CACHE = "private, no-store";
+
 /**
  * Controller'ı çalıştırıp yanıtı yazar; notFound/redirect kontrol akışını,
  * HTML cache'ini ve hata yönetimini üstlenir.
  *
+ * `private: true` kişiye özel sayfaları public cache yolundan tamamen ayırır:
+ * HTML cache devre dışı kalır, config'in `cache.html` deseni bu kararı
+ * ezemez, yanıt `no-store` ile ve ETag'siz gider. Dashboard tipi sayfalarda
+ * bu bayrak olmadan çalışmak, bir kullanıcının HTML'inin bir başkasına
+ * servis edilmesi anlamına gelir.
+ *
  * @param {(ctx: { params: object, query: object, pathname: string,
  *   req: import('express').Request }) => Promise<object>} controller
- * @param {{ revalidate?: number }} [options]
+ * @param {{ revalidate?: number, private?: boolean }} [options]
  * @returns {import('express').RequestHandler}
  */
 export function route(controller, options = {}) {
+  const isPrivate = options.private === true;
+
   return async (req, res, next) => {
+    const context = createRequestContext({ private: isPrivate, res });
     const ctx = {
       params: req.params ?? {},
       query: req.query ?? {},
       pathname: req.path,
-      req,
+      // Cookie/Authorization okunursa çıktı kullanıcıya bağlıdır; cache'e
+      // yazılmaması için işaretlenmesi gerekiyor.
+      req: guardRequest(req),
     };
 
-    const revalidate = resolveRevalidate(req.path, options.revalidate);
-    const cacheable = req.method === "GET" && Boolean(revalidate);
+    // Private route'ta desen taraması hiç yapılmaz: `cache.html` altındaki
+    // geniş bir kural (`/**` gibi) bu sayfayı cache'lenebilir hâle
+    // getirmesin. Kilit tek yönlü — route "özel" dediyse config açamaz.
+    const revalidate = isPrivate
+      ? undefined
+      : resolveRevalidate(req.path, options.revalidate);
+    const cacheable = !isPrivate && req.method === "GET" && Boolean(revalidate);
     const cacheKey = `${req.path}?${new URLSearchParams(
       Object.entries(ctx.query).map(([k, v]) => [k, String(v)]),
     ).toString()}`;
 
     try {
-      const result = await withHtmlCache(cacheKey, cacheable ? revalidate : 0, () =>
-        withUpstreamTracking(() => withRequestCache(() => produce(controller, ctx))),
+      const result = await withRequestContext(context, () =>
+        withHtmlCache(cacheKey, cacheable ? revalidate : 0, () =>
+          withUpstreamTracking(() => withRequestCache(() => produce(controller, ctx))),
+        ),
       );
 
+      // Cache'lenebilir bir route kimliğe dokunduysa yanıt yine gider ama
+      // saklanmaz (`produce` bunu `storable: false` ile bildirdi) ve public
+      // direktif yazılmaz.
+      const leaked = cacheable && context.tainted;
+      if (leaked && isDev) {
+        throw new Error(
+          `[render] ${req.path} is a cacheable route but read identity-bound data ` +
+            `(${context.taintReasons.join(", ")}). This page must be registered with ` +
+            `'route(fn, { private: true })'; otherwise one user's HTML is served to another.`,
+        );
+      }
+
+      // Eksik veriyle üretilen çıktı süreç içi önbelleğe yazılmıyor; aynı
+      // çıktıya CDN'de `s-maxage` vermek o kararı bir katman yukarıda geri
+      // almak olurdu. Geçici bir 429 yüzünden üretilen 503, ters proxy'de
+      // dakikalarca yaşamamalı.
+      const publicCache = cacheable && !leaked && !result.degraded;
+
       res.status(result.status);
       res.setHeader("Content-Type", "text/html; charset=utf-8");
-      if (cacheable) {
+
+      // Geçici upstream hatası: istemciye ve bota "bu kalıcı değil, sonra gel"
+      // demenin standart yolu.
+      if (result.retryAfter) {
+        res.setHeader("Retry-After", String(result.retryAfter));
+      }
+
+      // Teşhis başlığı `degraded` yanıtta da yazılır: "MISS" görmek, sayfanın
+      // önbellek yolundan geçtiğini ama saklanmadığını anlatan tek ipucu.
+      if (cacheable && !leaked) {
+        res.setHeader(
+          getConfig().brand.cacheHeader,
+          result.cached ? (result.stale ? "STALE" : "HIT") : "MISS",
+        );
+      }
+
+      if (publicCache) {
         res.setHeader(
           "Cache-Control",
           `public, max-age=0, s-maxage=${revalidate}, stale-while-revalidate=60`,
         );
+      } else {
+        res.setHeader("Cache-Control", PRIVATE_CACHE);
+        // Anahtarında cookie olmayan bir cache'in bu yanıtı paylaşmasını
+        // engeller; `no-store`'a uymayan bir katman için ikinci savunma.
+        res.setHeader("Vary", "Cookie");
+
+        if (leaked) {
+          console.warn(
+            `[render] ${req.path} read identity-bound data (${context.taintReasons.join(", ")}), ` +
+              `not cached. The route should be registered with 'private: true'.`,
+          );
+        }
       }
-      res.setHeader(
-        getConfig().brand.cacheHeader,
-        result.cached ? (result.stale ? "STALE" : "HIT") : "MISS",
-      );
-      await sendHtml(req, res, result.html, result.encoded);
+
+      // ETag kişiye özel HTML için kullanıcıya özgü bir doğrulayıcıdır ve
+      // `no-store` ile birlikte hiçbir işe yaramaz; üretilmesi engellenir.
+      await sendHtml(req, res, result.html, result.encoded, { etag: publicCache });
     } catch (error) {
       if (isRedirectError(error)) {
+        // Oturuma bağlı bir yönlendirme de kişiye özeldir: "giriş yapmalısın"
+        // kararının cache'lenmesi, oturum açmış kullanıcıyı da login sayfasına
+        // atan türde hatalara yol açıyor.
+        if (isPrivate || context.tainted) {
+          res.setHeader("Cache-Control", PRIVATE_CACHE);
+          res.setHeader("Vary", "Cookie");
+        }
+
         res.redirect(error.statusCode, error.location);
         return;
       }
@@ -205,6 +290,105 @@ export function route(controller, options = {}) {
   };
 }
 
+/**
+ * Layout'suz, asla cache'lenmeyen parça yanıtı.
+ *
+ * Fragment uçları (tablo sayfası, sekme paneli, canlı tazelenen kart) her
+ * projede elle yazılıyor ve `no-store` yazmayı unutmak sessiz bir sızıntıya
+ * dönüşüyor. Burada politika sabit: HTML cache'e hiç uğramaz, `no-store` ile
+ * ve ETag'siz gider.
+ *
+ * Hata durumunda tüm sayfa yerine küçük bir hata parçası döner: takas edilen
+ * bölge bir hata sayfasının tamamını içine almasın.
+ *
+ * @param {(ctx: { params: object, query: object, pathname: string,
+ *   req: import('express').Request }) => Promise<{ view: string, data?: object,
+ *   status?: number } | string>} controller
+ * @returns {import('express').RequestHandler}
+ */
+export function fragment(controller) {
+  return async (req, res, next) => {
+    const context = createRequestContext({ private: true, res });
+    const ctx = {
+      params: req.params ?? {},
+      query: req.query ?? {},
+      pathname: req.path,
+      req: guardRequest(req),
+    };
+
+    try {
+      const result = await withRequestContext(context, () =>
+        withRequestCache(async () => {
+          const value = await controller(ctx);
+          if (typeof value === "string") return { html: value, status: 200 };
+
+          return {
+            html: await renderView(value.view, value.data ?? {}),
+            status: value.status ?? 200,
+          };
+        }),
+      );
+
+      res.status(result.status);
+      res.setHeader("Content-Type", "text/html; charset=utf-8");
+      res.setHeader("Cache-Control", PRIVATE_CACHE);
+      res.setHeader("Vary", "Cookie");
+      await sendHtml(req, res, result.html, undefined, { etag: false });
+    } catch (error) {
+      if (isRedirectError(error)) {
+        res.setHeader("Cache-Control", PRIVATE_CACHE);
+        res.redirect(error.statusCode, error.location);
+        return;
+      }
+
+      if (isNotFoundError(error)) {
+        res.status(404).setHeader("Cache-Control", PRIVATE_CACHE);
+        res.type("html").send(fragmentError("notFound"));
+        return;
+      }
+
+      console.error(`[fragment] ${req.method} ${req.originalUrl}`, error);
+
+      if (res.headersSent) {
+        next(error);
+        return;
+      }
+
+      res.status(500).setHeader("Cache-Control", PRIVATE_CACHE);
+      res.type("html").send(fragmentError("failed"));
+    }
+  };
+}
+
+/**
+ * Ziyaretçiye görünen parça hatası metinleri. Framework'ün kendi logları
+ * İngilizce ama bu satırlar ekranda okunuyor, bu yüzden durum sayfalarıyla
+ * aynı kuralı izliyorlar: dil `brand.lang`.
+ */
+const FRAGMENT_MESSAGES = {
+  tr: { notFound: "Bu içerik bulunamadı.", failed: "Bu bölüm yüklenemedi." },
+  en: { notFound: "This content was not found.", failed: "This section could not be loaded." },
+};
+
+/**
+ * Fragment hatası için minimal işaretleme. Şablona bağlı olmaması bilinçli:
+ * hata yolu, hatanın kaynağı olabilecek render katmanına geri dönmemeli.
+ *
+ * @param {"notFound" | "failed"} kind
+ * @returns {string}
+ */
+function fragmentError(kind) {
+  let lang = "en";
+  try {
+    lang = getConfig().brand.lang ?? "en";
+  } catch {
+    /* config yüklenmemişse İngilizce kalır */
+  }
+
+  const table = FRAGMENT_MESSAGES[lang.slice(0, 2).toLowerCase()] ?? FRAGMENT_MESSAGES.en;
+  return `<div role="alert" data-fragment-error>${html.esc(table[kind])}</div>`;
+}
+
 /**
  * `jskelet.config.mjs` → `cache().html` route'un kendi `revalidate`'ini ezer.
  * Sonuç yol başına hatırlanır: her istekte desen taraması yapılmaz.
@@ -253,13 +437,23 @@ function resolveRevalidate(pathname, fallback) {
  * @param {import('express').Response} res
  * @param {string} body
  * @param {Map<string, Buffer>} [encoded]
+ * @param {{ etag?: boolean }} [options] `etag: false` → `res.send()` atlanır,
+ *   böylece Express kullanıcıya özel gövde için doğrulayıcı üretmez.
  * @returns {Promise<void>}
  */
-async function sendHtml(req, res, body, encoded) {
+async function sendHtml(req, res, body, encoded, options = {}) {
   const encoding =
     req.method === "HEAD" ? null : negotiateEncoding(req.headers["accept-encoding"]);
 
   if (!encoding || !encoded) {
+    if (options.etag === false) {
+      // Sıkıştırma middleware'i devreye girerse Content-Length'i kendisi
+      // kaldırır; girmezse doğru uzunlukla gider.
+      res.setHeader("Content-Length", String(Buffer.byteLength(body)));
+      res.end(req.method === "HEAD" ? undefined : body);
+      return;
+    }
+
     res.send(body);
     return;
   }
@@ -279,7 +473,8 @@ async function sendHtml(req, res, body, encoded) {
 /**
  * @param {Function} controller
  * @param {{ pathname: string }} ctx
- * @returns {Promise<{ html: string, status: number, degraded?: boolean }>}
+ * @returns {Promise<{ html: string, status: number, degraded?: boolean,
+ *   storable?: boolean, retryAfter?: number }>}
  */
 async function produce(controller, ctx) {
   try {
@@ -289,15 +484,45 @@ async function produce(controller, ctx) {
       html: rendered,
       status: page.status ?? 200,
       degraded: hasUpstreamFailures(ctx.pathname),
+      // Kimliğe bağlı çıktı önbelleğe yazılmaz. Karar burada verilmeli:
+      // `withHtmlCache` yazma anında controller'ın ne okuduğunu bilemez.
+      storable: getRequestContext()?.tainted !== true,
     };
   } catch (error) {
     if (isNotFoundError(error)) {
+      // Veri gelmediği için `notFound()` çağrılmış olabilir: geçici bir
+      // upstream hatası (429, 5xx, ağ) varken bunu 404 olarak servis etmek iki
+      // kere yanlış. Önbelleğe girip TTL boyunca "bu sayfa yok" cevabını
+      // sabitler ve arama motoru geçici bir rate limit'i kalıcı 404 sanar.
+      // Doğrusu 503: cache'lenmez, `Retry-After` ile gider, sonraki istek
+      // gerçek içeriği üretir.
+      const transient = transientUpstreamFailures();
+      if (transient.length) {
+        console.warn(
+          `[render] ${ctx.pathname} returned notFound() while upstream is failing ` +
+            `(${summarizeFailures(transient)}), serving an uncached 503 instead`,
+        );
+        return {
+          html: await renderStatusPage(503),
+          status: 503,
+          degraded: true,
+          retryAfter: RETRY_AFTER_SECONDS,
+        };
+      }
+
       return { html: await renderNotFound(), status: 404 };
     }
     throw error;
   }
 }
 
+/**
+ * Geçici bir upstream hatası yüzünden üretilemeyen sayfanın `Retry-After`
+ * değeri. Kısa tutuluyor: ziyaretçi de bot da birkaç saniye sonra gerçek
+ * içeriği bulabilsin.
+ */
+const RETRY_AFTER_SECONDS = 30;
+
 /** Ağ hatası (0) ve geçici olduğu varsayılan durumlar. */
 const TRANSIENT_STATUSES = new Set([0, 408, 425, 429]);
 
@@ -323,28 +548,39 @@ function hasUpstreamFailures(pathname) {
   const failures = getUpstreamFailures();
   if (!failures.length) return false;
 
-  /** @param {typeof failures} list */
-  const summarize = (list) =>
-    list.map((failure) => `${failure.status} ${failure.path}`).join(", ");
-
   const transient = failures.filter((failure) => isTransient(failure.status));
   const permanent = failures.filter((failure) => !isTransient(failure.status));
 
   if (permanent.length) {
     console.warn(
-      `[render] ${pathname} eksik veriyle üretildi, upstream kalıcı hata veriyor (${summarize(permanent)})`,
+      `[render] ${pathname} was produced with missing data, upstream is failing permanently (${summarizeFailures(permanent)})`,
     );
   }
 
   if (!transient.length) return false;
 
   console.warn(
-    `[render] ${pathname} eksik veriyle üretildi, önbelleğe alınmıyor (${summarize(transient)})`,
+    `[render] ${pathname} was produced with missing data, not caching it (${summarizeFailures(transient)})`,
   );
 
   return true;
 }
 
+/**
+ * @returns {import('./upstream-tracking.js').UpstreamFailure[]}
+ */
+function transientUpstreamFailures() {
+  return getUpstreamFailures().filter((failure) => isTransient(failure.status));
+}
+
+/**
+ * @param {import('./upstream-tracking.js').UpstreamFailure[]} failures
+ * @returns {string}
+ */
+function summarizeFailures(failures) {
+  return failures.map((failure) => `${failure.status} ${failure.path}`).join(", ");
+}
+
 /**
  * 404 sayfası. `renderStatusPage(404)` için kısayol; route dosyalarında en sık
  * ihtiyaç duyulan durum bu olduğu için ayrı bir ad taşımaya devam ediyor.