Sync from main repo

Author: actions-user

src/cache.ts

@@ -1,5 +1,4 @@
 import "dotenv/config";
-import { FeedConfig } from "./types.js";
 
 // Environment variables validation
 const REQUIRED_ENV_VARS = ["API_SECRET"];
@@ -46,49 +45,3 @@ export const API_SECRET = process.env.API_SECRET!;
 export const ALLOWED_ORIGINS = process.env.ALLOWED_ORIGINS
   ? process.env.ALLOWED_ORIGINS.split(",")
   : ["*"];
-
-// Default feed ID - since we're focusing on a single feed
-export const DEFAULT_FEED_ID = "main";
-
-// Default configuration
-const DEFAULT_CONFIG: FeedConfig = {
-  id: DEFAULT_FEED_ID,
-  title: "Default RSS Feed",
-  description: "A feed of curated content",
-  siteUrl: "https://example.com",
-  copyright: "test",
-  language: "en",
-  maxItems: 100,
-  image: "https://example.com/logo.png",
-  author: { name: "Feed Author", email: "author@example.com" },
-};
-
-let currentConfig: FeedConfig | null = null;
-
-// Set feed configuration
-export function setFeedConfig(config: FeedConfig): void {
-  config.id = DEFAULT_FEED_ID;
-
-  // Set default values for optional fields if not provided
-  config.title = config.title || DEFAULT_CONFIG.title;
-  config.description = config.description || DEFAULT_CONFIG.description;
-  config.siteUrl = config.siteUrl || DEFAULT_CONFIG.siteUrl;
-
-  // Ensure maxItems is always a positive number
-  config.maxItems =
-    typeof config.maxItems === "number" && config.maxItems > 0
-      ? config.maxItems
-      : DEFAULT_CONFIG.maxItems;
-
-  config.language = config.language || DEFAULT_CONFIG.language;
-
-  // Update the in-memory configuration
-  currentConfig = config;
-  console.log("Updated feed configuration");
-}
-
-// Get the current feed configuration
-export const getFeedConfig = (): FeedConfig => {
-  const config = currentConfig || DEFAULT_CONFIG;
-  return JSON.parse(JSON.stringify(config)) as FeedConfig;
-};

src/config.ts

@@ -0,0 +1,39 @@
+export class RssServiceError extends Error {
+  constructor(
+    message: string,
+    public readonly statusCode: number,
+  ) {
+    super(message);
+    this.name = this.constructor.name;
+  }
+}
+
+export class FeedNotFoundError extends RssServiceError {
+  constructor(feedId: string) {
+    super(`Feed with ID '${feedId}' not found.`, 404);
+  }
+}
+
+export class DuplicateItemError extends RssServiceError {
+  constructor(guid: string) {
+    super(`Item with GUID '${guid}' already exists in this feed.`, 409);
+  }
+}
+
+export class InvalidPayloadError extends RssServiceError {
+  constructor(message: string) {
+    super(message, 400);
+  }
+}
+
+export class FeedConfigurationError extends RssServiceError {
+  constructor(message: string) {
+    super(`Feed configuration error: ${message}`, 400);
+  }
+}
+
+export class StorageError extends RssServiceError {
+  constructor(message: string) {
+    super(`Storage error: ${message}`, 500);
+  }
+}

src/errors.ts

@@ -1,6 +1,5 @@
 import { Feed, Item } from "feed";
-import { getFeedConfig } from "./config.js";
-import { ApiFormat, FeedFormat, RssItem } from "./types.js";
+import { ApiFormat, FeedFormat, RssItem, FeedConfig } from "./types.js";
 import { stripHtml } from "./utils.js";
 
 /**
@@ -71,9 +70,9 @@ export function formatItems(
  */
 export function generateFeed(
   items: string[],
+  feedConfig: FeedConfig,
   format: FeedFormat = "rss",
 ): { content: string; contentType: string } {
-  const feedConfig = getFeedConfig();
   // For raw format, return JSON with no HTML
   if (format === "raw") {
     const rawItems = formatItems(items, "raw");