@rmdes/indiekit-endpoint-microsub 1.0.63 → 1.0.65

package/lib/cache/redis.js

@@ -119,24 +119,6 @@ export async function setCache(redis, key, value, ttl = 300) {
   }
 }
 
-/**
- * Delete value from cache
- * @param {object} redis - Redis client
- * @param {string} key - Cache key
- * @returns {Promise<void>}
- */
-export async function deleteCache(redis, key) {
-  if (!redis) {
-    return;
-  }
-
-  try {
-    await redis.del(nsKey(key));
-  } catch {
-    // Ignore cache errors
-  }
-}
-
 /**
  * Publish event to channel
  * @param {object} redis - Redis client
@@ -185,16 +167,3 @@ export async function subscribeToChannel(redis, channel, callback) {
   }
 }
 
-/**
- * Cleanup Redis connection on shutdown
- */
-export async function closeRedis() {
-  if (redisClient) {
-    try {
-      await redisClient.quit();
-      redisClient = undefined;
-    } catch {
-      // Ignore cleanup errors
-    }
-  }
-}

package/lib/controllers/block.js

@@ -82,5 +82,3 @@ export async function unblock(request, response) {
 
   response.json({ result: "ok" });
 }
-
-export const blockController = { list, block, unblock };

package/lib/controllers/channels.js

@@ -132,5 +132,3 @@ export async function get(request, response) {
     settings: channel.settings,
   });
 }
-
-export const channelsController = { list, action, get };

package/lib/controllers/events.js

@@ -53,5 +53,3 @@ export async function stream(request, response) {
     removeClient(response);
   });
 }
-
-export const eventsController = { stream };

package/lib/controllers/follow.js

@@ -163,5 +163,3 @@ export async function unfollow(request, response) {
 
   response.json({ result: "ok" });
 }
-
-export const followController = { list, follow, unfollow };

package/lib/controllers/mute.js

@@ -121,5 +121,3 @@ export async function unmute(request, response) {
 
   response.json({ result: "ok" });
 }
-
-export const muteController = { list, mute, unmute };

package/lib/controllers/preview.js

@@ -63,5 +63,3 @@ export async function preview(request, response) {
   const previewData = await fetchPreview(url);
   response.json(previewData);
 }
-
-export const previewController = { get, preview };

package/lib/controllers/reader/channel.js

@@ -10,7 +10,6 @@ import {
   updateChannelSettings,
   deleteChannel,
 } from "../../storage/channels.js";
-import { getFeedsForChannel } from "../../storage/feeds.js";
 import { getTimelineItems } from "../../storage/items.js";
 import { countReadItems } from "../../storage/items-read-state.js";
 import { getUserId } from "../../utils/auth.js";

package/lib/controllers/reader/index.js

@@ -1,45 +1,9 @@
 /**
- * Reader controller - barrel re-export
+ * Reader controller aggregator. Bundles handlers from sibling modules into the
+ * `readerController` object consumed by route wiring in the plugin root.
  * @module controllers/reader
  */
 
-export {
-  index,
-  channels,
-  newChannel,
-  createChannelAction,
-  channel,
-  channelHtml,
-  settings,
-  updateSettings,
-  deleteChannelAction,
-} from "./channel.js";
-
-export {
-  feeds,
-  addFeed,
-  removeFeed,
-  feedDetails,
-  editFeedForm,
-  updateFeedUrl,
-  rediscoverFeed,
-  refreshFeed,
-} from "./feed.js";
-
-export {
-  timeline,
-  timelineHtml,
-  markAllRead,
-  markViewRead,
-  item,
-} from "./timeline.js";
-
-export { compose, submitCompose } from "./compose.js";
-
-export { searchPage, searchFeeds, subscribe } from "./search.js";
-
-export { deck, deckSettings, saveDeckSettings } from "./deck.js";
-
 import {
   index,
   channels,

package/lib/controllers/search.js

@@ -145,5 +145,3 @@ export async function search(request, response) {
     return response.json({ results: [] });
   }
 }
-
-export const searchController = { discover, search };

package/lib/controllers/timeline.js

@@ -150,5 +150,3 @@ export async function action(request, response) {
     }
   }
 }
-
-export const timelineController = { get, action };

package/lib/feeds/discovery.js

@@ -78,7 +78,7 @@ export async function discoverAndValidateFeeds(url) {
  * @param {Array} feeds - Array of feed objects
  * @returns {Array} Filtered array of main content feeds
  */
-export function filterMainFeeds(feeds) {
+function filterMainFeeds(feeds) {
   return feeds.filter((feed) => feed.valid && !feed.isCommentsFeed);
 }
 

package/lib/feeds/normalizer.js

@@ -30,7 +30,7 @@ export function generateItemUid(feedUrl, itemId) {
  * @param {string|Date} dateInput - Date string or Date object
  * @returns {Date|undefined} Parsed Date or undefined if invalid
  */
-export function parseDate(dateInput) {
+function parseDate(dateInput) {
   if (!dateInput) {
     return;
   }
@@ -108,7 +108,7 @@ export function extractPhotoUrl(photo) {
  * @param {object|string} value - URL string or object with url/value property
  * @returns {string|undefined} URL string
  */
-export function extractUrl(value) {
+function extractUrl(value) {
   if (!value) {
     return;
   }

package/lib/media/proxy.js

@@ -98,7 +98,7 @@ const ALLOWED_TYPES = new Set([
  * @param {string} url - Original image URL
  * @returns {string} URL-safe hash
  */
-export function hashUrl(url) {
+function hashUrl(url) {
   return crypto.createHash("sha256").update(url).digest("hex").slice(0, 16);
 }
 
@@ -107,7 +107,7 @@ export function hashUrl(url) {
  * @param {string} url - Original image URL
  * @returns {string} HMAC hex signature (16 chars)
  */
-export function signProxyUrl(url) {
+function signProxyUrl(url) {
   const secret = process.env.SECRET || "microsub-default-secret";
   return crypto
     .createHmac("sha256", secret)
@@ -122,7 +122,7 @@ export function signProxyUrl(url) {
  * @param {string} sig - Submitted signature
  * @returns {boolean} Whether signature is valid
  */
-export function verifyProxySignature(url, sig) {
+function verifyProxySignature(url, sig) {
   if (!sig) return false;
   const expected = signProxyUrl(url);
   // Constant-time comparison
@@ -136,7 +136,7 @@ export function verifyProxySignature(url, sig) {
  * @param {string} originalUrl - Original image URL
  * @returns {string} Proxied URL
  */
-export function getProxiedUrl(baseUrl, originalUrl) {
+function getProxiedUrl(baseUrl, originalUrl) {
   if (!originalUrl || !baseUrl) {
     return originalUrl;
   }
@@ -203,7 +203,7 @@ export function proxyItemImages(item, baseUrl) {
  * @param {string} url - Image URL to fetch
  * @returns {Promise<object|null>} Cached image data or null
  */
-export async function fetchImage(redis, url) {
+async function fetchImage(redis, url) {
   // Block private/internal URLs (defense-in-depth)
   if (await isPrivateUrl(url)) {
     console.error(`[Microsub] Media proxy blocked private URL: ${url}`);

package/lib/polling/scheduler.js

@@ -35,7 +35,7 @@ export function startScheduler(indiekit) {
   // Run immediately on start
   runSchedulerCycle();
 
-  console.log("[Microsub] Feed polling scheduler started");
+  console.info("[Microsub] Feed polling scheduler started");
 }
 
 /**
@@ -47,7 +47,7 @@ export function stopScheduler() {
     schedulerInterval = undefined;
   }
   indiekitInstance = undefined;
-  console.log("[Microsub] Feed polling scheduler stopped");
+  console.info("[Microsub] Feed polling scheduler stopped");
 }
 
 /**
@@ -74,13 +74,13 @@ async function runSchedulerCycle() {
       return;
     }
 
-    console.log(`[Microsub] Processing ${feeds.length} feeds due for refresh`);
+    console.info(`[Microsub] Processing ${feeds.length} feeds due for refresh`);
 
     const result = await processFeedBatch(application, feeds, {
       concurrency: BATCH_CONCURRENCY,
     });
 
-    console.log(
+    console.info(
       `[Microsub] Processed ${result.total} feeds: ${result.successful} successful, ` +
         `${result.failed} failed, ${result.itemsAdded} new items`,
     );
@@ -118,13 +118,3 @@ export async function refreshFeedNow(application, feedId) {
   return processFeed(application, feed);
 }
 
-/**
- * Get scheduler status
- * @returns {object} Scheduler status
- */
-export function getSchedulerStatus() {
-  return {
-    running: !!schedulerInterval,
-    processing: isRunning,
-  };
-}

package/lib/polling/tier.js

@@ -23,22 +23,24 @@ const MAX_TIER = 10;
 const DEFAULT_TIER = 1;
 
 /**
- * Get polling interval for a tier in milliseconds
+ * Get polling interval for a tier in milliseconds.
+ * Internal helper for getNextFetchTime.
  * @param {number} tier - Polling tier (0-10)
  * @returns {number} Interval in milliseconds
  */
-export function getIntervalForTier(tier) {
+function getIntervalForTier(tier) {
   const clampedTier = Math.max(MIN_TIER, Math.min(MAX_TIER, tier));
   const minutes = Math.pow(2, clampedTier);
   return minutes * 60 * 1000;
 }
 
 /**
- * Get next fetch time based on tier
+ * Get next fetch time based on tier.
+ * Internal helper for calculateNewTier.
  * @param {number} tier - Polling tier
  * @returns {Date} Next fetch time
  */
-export function getNextFetchTime(tier) {
+function getNextFetchTime(tier) {
   const interval = getIntervalForTier(tier);
   return new Date(Date.now() + interval);
 }
@@ -90,50 +92,3 @@ export function calculateNewTier(options) {
   };
 }
 
-/**
- * Get initial tier for a new feed subscription
- * @returns {object} Initial tier settings
- */
-export function getInitialTier() {
-  return {
-    tier: MIN_TIER, // Start at tier 0 for immediate first fetch
-    consecutiveUnchanged: 0,
-    nextFetchAt: new Date(), // Fetch immediately
-  };
-}
-
-/**
- * Determine if a feed is due for fetching
- * @param {object} feed - Feed document
- * @returns {boolean} Whether the feed should be fetched
- */
-export function isDueForFetch(feed) {
-  if (!feed.nextFetchAt) {
-    return true;
-  }
-
-  return new Date(feed.nextFetchAt) <= new Date();
-}
-
-/**
- * Get human-readable description of polling interval
- * @param {number} tier - Polling tier
- * @returns {string} Description
- */
-export function getTierDescription(tier) {
-  const minutes = Math.pow(2, tier);
-
-  if (minutes < 60) {
-    return `every ${minutes} minute${minutes === 1 ? "" : "s"}`;
-  }
-
-  const hours = minutes / 60;
-  if (hours < 24) {
-    return `every ${hours.toFixed(1)} hour${hours === 1 ? "" : "s"}`;
-  }
-
-  const days = hours / 24;
-  return `every ${days.toFixed(1)} day${days === 1 ? "" : "s"}`;
-}
-
-export { MIN_TIER, MAX_TIER, DEFAULT_TIER };

package/lib/realtime/broker.js

@@ -84,18 +84,6 @@ export function subscribeClient(response, channelId) {
   }
 }
 
-/**
- * Unsubscribe a client from a channel
- * @param {object} response - Express response object
- * @param {string} channelId - Channel ID
- */
-export function unsubscribeClient(response, channelId) {
-  const client = clients.get(response);
-  if (client) {
-    client.channels.delete(channelId);
-  }
-}
-
 /**
  * Send an event to a specific client
  * @param {object} response - Express response object
@@ -113,26 +101,13 @@ export function sendEvent(response, event, data) {
 }
 
 /**
- * Broadcast an event to all clients subscribed to a channel
- * @param {string} channelId - Channel ID
- * @param {string} event - Event name
- * @param {object} data - Event data
- */
-export function broadcastToChannel(channelId, event, data) {
-  for (const client of clients.values()) {
-    if (client.channels.has(channelId)) {
-      sendEvent(client.response, event, data);
-    }
-  }
-}
-
-/**
- * Broadcast an event to all clients for a user
+ * Broadcast an event to all clients for a user.
+ * Internal helper — only invoked by the Redis fan-out in handleRedisEvent.
  * @param {string} userId - User ID
  * @param {string} event - Event name
  * @param {object} data - Event data
  */
-export function broadcastToUser(userId, event, data) {
+function broadcastToUser(userId, event, data) {
   for (const client of clients.values()) {
     if (client.userId === userId) {
       sendEvent(client.response, event, data);
@@ -140,17 +115,6 @@ export function broadcastToUser(userId, event, data) {
   }
 }
 
-/**
- * Broadcast an event to all connected clients
- * @param {string} event - Event name
- * @param {object} data - Event data
- */
-export function broadcastToAll(event, data) {
-  for (const client of clients.values()) {
-    sendEvent(client.response, event, data);
-  }
-}
-
 /**
  * Set up Redis subscription for a user
  * @param {string} userId - User ID
@@ -208,39 +172,3 @@ function handleRedisEvent(userId, data) {
   }
 }
 
-/**
- * Get broker statistics
- * @returns {object} Statistics
- */
-export function getStats() {
-  const userCounts = new Map();
-  for (const client of clients.values()) {
-    const count = userCounts.get(client.userId) || 0;
-    userCounts.set(client.userId, count + 1);
-  }
-
-  return {
-    totalClients: clients.size,
-    uniqueUsers: userCounts.size,
-    userSubscribers: userSubscribers.size,
-  };
-}
-
-/**
- * Clean up all connections
- */
-export function cleanup() {
-  for (const client of clients.values()) {
-    clearInterval(client.pingInterval);
-  }
-  clients.clear();
-
-  for (const subscriber of userSubscribers.values()) {
-    try {
-      subscriber.quit();
-    } catch {
-      // Ignore cleanup errors
-    }
-  }
-  userSubscribers.clear();
-}

package/lib/storage/channels.js

@@ -29,7 +29,7 @@ const CHANNEL_COLORS = [
  * @param {number} order - Channel order index
  * @returns {string} Hex color
  */
-export function getChannelColor(order) {
+function getChannelColor(order) {
   return CHANNEL_COLORS[Math.abs(order) % CHANNEL_COLORS.length];
 }
 

package/lib/storage/feeds.js

@@ -45,7 +45,7 @@ export function normalizeUrl(url) {
  * @param {string} url - Feed URL to check
  * @returns {Promise<object|null>} Existing feed with channel info, or null
  */
-export async function findFeedAcrossChannels(application, url) {
+async function findFeedAcrossChannels(application, url) {
   const collection = getCollection(application);
   const normalized = normalizeUrl(url);
 
@@ -427,19 +427,3 @@ export async function updateFeedStatus(application, id, status) {
   });
 }
 
-/**
- * Get feeds with errors
- * @param {object} application - Indiekit application
- * @param {number} [minErrors=3] - Minimum consecutive errors
- * @returns {Promise<Array>} Array of feeds with errors
- */
-export async function getFeedsWithErrors(application, minErrors = 3) {
-  const collection = getCollection(application);
-
-  return collection
-    .find({
-      status: "error",
-      consecutiveErrors: { $gte: minErrors },
-    })
-    .toArray();
-}