feedkeeper 0.28.2 → 0.28.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "feedkeeper",
-  "version": "0.28.2",
+  "version": "0.28.4",
   "description": "A lightweight, self-hosted RSS reader and remote MCP server for your news, blogs, and AI workflows.",
   "keywords": [
     "rss",

package/server/dist/api/v1/resources.js

@@ -644,6 +644,7 @@ resourcesRouter.get("/items/:id/image", imageLimiter, async (req, res) => {
             res.status(503).json({ error: "image_busy" });
             return;
         }
+        console.error(`[image_proxy] Failed to fetch image for item ${req.params.id} (${src}):`, error instanceof Error ? error.message : error);
         res.status(502).json({ error: "image_unavailable" });
     }
 });

package/server/dist/feeds/discovery.js

@@ -11,7 +11,7 @@ const FEED_MIME_TYPES = [
     "application/xml",
 ];
 const MAX_PROBES = 15;
-const COMMON_PATHS = ["/feed", "/rss.xml", "/atom.xml", "/feed.xml", "/index.xml", "/rss", "/rss.php?feed=RSS2.0", "/rss.php?feed=ATOM1.0"];
+const COMMON_PATHS = ["/feed/", "/feed", "/rss.xml", "/atom.xml", "/feed.xml", "/index.xml", "/rss/", "/rss", "/rss.php?feed=RSS2.0", "/rss.php?feed=ATOM1.0"];
 // Feed-looking URLs anywhere in a page, including JSON embedded in attributes.
 const FEED_URL_IN_TEXT = /https?:\/\/[^\s"'<>\\]+?(?:\.rss|\.atom|\/feed\/?|\/(?:rss|atom|feed)\.xml)(?=["'\s<>\\]|$)/gi;
 function sameUrl(a, b) {
@@ -28,18 +28,23 @@ function sameUrl(a, b) {
     return normalize(a) === normalize(b);
 }
 async function parseFeedAt(url) {
-    const fetched = await fetchFeed(url);
-    const finalUrl = fetched.finalUrl || url;
     try {
-        const parsed = await rssParser.parseString(fetched.body);
-        if (parsed.items && (parsed.title || parsed.description || parsed.items.length > 0)) {
-            return { feed: { url: finalUrl, title: plainTitle(parsed.title) ?? null, type: "application/rss+xml" }, html: null, finalUrl };
+        const fetched = await fetchFeed(url);
+        const finalUrl = fetched.finalUrl || url;
+        try {
+            const parsed = await rssParser.parseString(fetched.body);
+            if (parsed.items && (parsed.title || parsed.description || parsed.items.length > 0)) {
+                return { feed: { url: finalUrl, title: plainTitle(parsed.title) ?? null, type: "application/rss+xml" }, html: null, finalUrl };
+            }
+        }
+        catch {
+            // Not a feed; the caller may look for feed links in the HTML instead.
         }
+        return { feed: null, html: fetched.body, finalUrl };
     }
     catch {
-        // Not a feed; the caller may look for feed links in the HTML instead.
+        return { feed: null, html: null, finalUrl: url };
     }
-    return { feed: null, html: fetched.body, finalUrl };
 }
 /**
  * Keep only candidates that really are feeds. Sites sometimes announce their own
@@ -75,22 +80,29 @@ export async function discoverFeeds(inputUrl) {
     const page = await parseFeedAt(targetUrl);
     if (page.feed)
         return [page.feed];
-    if (!page.html)
-        return [];
+    // 1b. If the URL did not end in a slash, try with a trailing slash (e.g. /feed -> /feed/)
+    if (!targetUrl.endsWith("/")) {
+        const slashCandidate = await parseFeedAt(targetUrl + "/");
+        if (slashCandidate.feed)
+            return [slashCandidate.feed];
+    }
     // 2. Feeds announced or linked on the page.
-    const fromPage = await keepRealFeeds(extractFeedsFromHtml(page.html, page.finalUrl), page.finalUrl);
-    if (fromPage.length > 0)
-        return fromPage;
+    if (page.html) {
+        const fromPage = await keepRealFeeds(extractFeedsFromHtml(page.html, page.finalUrl), page.finalUrl);
+        if (fromPage.length > 0)
+            return fromPage;
+    }
     // 3. Common feed paths; some lead to an overview page that lists the site's feeds.
+    const probeBase = page.finalUrl || targetUrl;
     for (const path of COMMON_PATHS) {
         try {
-            const candidateUrl = new URL(path, page.finalUrl).toString();
+            const candidateUrl = new URL(path, probeBase).toString();
             if (sameUrl(candidateUrl, targetUrl))
                 continue;
             const candidate = await parseFeedAt(candidateUrl);
             if (candidate.feed)
                 return [candidate.feed];
-            if (candidate.html && !sameUrl(candidate.finalUrl, page.finalUrl)) {
+            if (candidate.html && !sameUrl(candidate.finalUrl, probeBase)) {
                 const listed = await keepRealFeeds(extractFeedsFromHtml(candidate.html, candidate.finalUrl), candidate.finalUrl);
                 if (listed.length > 0)
                     return listed;

package/server/dist/feeds/extractor.js

@@ -34,6 +34,31 @@ function cleanConsentDom(doc) {
         }
     }
 }
+function resolveLazyImages(doc) {
+    const images = doc.querySelectorAll("img");
+    for (const img of images) {
+        const lazySrc = img.getAttribute("data-lazy-src") ||
+            img.getAttribute("data-src") ||
+            img.getAttribute("data-original") ||
+            img.getAttribute("data-actualsrc");
+        const src = img.getAttribute("src");
+        if (lazySrc && (!src || src.startsWith("data:") || src.length < 10)) {
+            img.setAttribute("src", lazySrc);
+        }
+        const lazySrcset = img.getAttribute("data-lazy-srcset") || img.getAttribute("data-srcset");
+        const srcset = img.getAttribute("srcset");
+        if (lazySrcset && (!srcset || srcset.startsWith("data:"))) {
+            img.setAttribute("srcset", lazySrcset);
+        }
+    }
+    const sources = doc.querySelectorAll("source");
+    for (const source of sources) {
+        const lazySrcset = source.getAttribute("data-lazy-srcset") || source.getAttribute("data-srcset");
+        if (lazySrcset) {
+            source.setAttribute("srcset", lazySrcset);
+        }
+    }
+}
 /**
  * Checks if a given URL or extracted HTML/text is a cookie / consent wall rather than article content.
  */
@@ -183,6 +208,7 @@ async function extractArticleContent(rawUrl, fetchImpl, signal) {
         try {
             const paywall = /"isAccessibleForFree"\s*:\s*(?:false|"false")/i.test(fetched.html);
             cleanConsentDom(dom.window.document);
+            resolveLazyImages(dom.window.document);
             parsed = new Readability(dom.window.document, { charThreshold: 60 }).parse();
             if (isConsentContent(fetched.url, parsed?.content ?? fetched.html, parsed?.title, parsed?.textContent)) {
                 return { status: "consent_wall" };

package/server/dist/feeds/service.js

@@ -64,7 +64,10 @@ export async function subscribeToFeed(userId, inputUrl, label = null, folderId =
     // (e.g. a homepage that announces itself as RSS); don't keep the subscription.
     if (pollResult.error && !pollResult.deferred && !findFeedById(feed.id)?.last_success_at) {
         unsubscribeRepo(userId, feed.id);
-        throw new NoFeedsFoundError();
+        if (pollResult.error === "feed_invalid_or_unavailable") {
+            throw new NoFeedsFoundError();
+        }
+        throw new FeedError(pollResult.error);
     }
     const subscription = findSubscriptionForUser(userId, feed.id);
     if (!subscription)

package/server/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@feedkeeper/server",
-  "version": "0.28.2",
+  "version": "0.28.4",
   "private": true,
   "type": "module"
 }