reddit-mcp-server 1.0.2 → 1.0.3

package/.claude/settings.local.json

@@ -13,7 +13,10 @@
       "Bash(ls:*)",
       "Bash(rm:*)",
       "Bash(pnpm build:*)",
-      "Bash(npx tsup:*)"
+      "Bash(npx tsup:*)",
+      "Bash(find:*)",
+      "Bash(rg:*)",
+      "Bash(grep:*)"
     ],
     "deny": []
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "reddit-mcp-server",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "A Model Context Protocol (MCP) that provides tools for fetching and creating Reddit content. Fork of the alexandros-lekkas/reddit-mcp-server.",
   "main": "dist/index.js",
   "bin": {
@@ -15,9 +15,9 @@
   "author": "Jordan Burke <jordan.burke@gmail.com>",
   "license": "ISC",
   "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.13.1",
+    "@modelcontextprotocol/sdk": "^1.13.2",
     "axios": "^1.10.0",
-    "dotenv": "^16.5.0"
+    "dotenv": "^16.6.1"
   },
   "devDependencies": {
     "@eslint/js": "^9.29.0",
@@ -27,7 +27,7 @@
     "eslint": "^9.29.0",
     "eslint-config-prettier": "^10.1.5",
     "eslint-plugin-prettier": "^5.5.1",
-    "prettier": "^3.6.1",
+    "prettier": "^3.6.2",
     "tsup": "^8.5.0",
     "typescript": "^5.8.3"
   },

package/src/client/reddit-client.ts

@@ -52,12 +52,12 @@ export class RedditClient {
       const authData = new URLSearchParams()
 
       if (this.username && this.password) {
-        console.error(`[Auth] Authenticating with user credentials for ${this.username}`)
+        // Authenticating with user credentials
         authData.append("grant_type", "password")
         authData.append("username", this.username)
         authData.append("password", this.password)
       } else {
-        console.error("[Auth] Authenticating with client credentials (read-only)")
+        // Authenticating with client credentials (read-only)
         authData.append("grant_type", "client_credentials")
       }
 
@@ -77,9 +77,9 @@ export class RedditClient {
       this.authenticated = true
       this.api.defaults.headers.common["Authorization"] = `Bearer ${this.accessToken}`
 
-      console.error("[Auth] Successfully authenticated with Reddit API")
+      // Successfully authenticated with Reddit API
     } catch (error) {
-      console.error("[Auth] Authentication error:", error)
+      // Authentication error occurred
       throw new Error("Failed to authenticate with Reddit API")
     }
   }
@@ -115,7 +115,7 @@ export class RedditClient {
         profileUrl: `https://reddit.com/user/${data.name}`,
       }
     } catch (error) {
-      console.error(`[Error] Failed to get user info for ${username}:`, error)
+      // Failed to get user info
       throw new Error(`Failed to get user info for ${username}`)
     }
   }
@@ -139,7 +139,7 @@ export class RedditClient {
         url: data.url,
       }
     } catch (error) {
-      console.error(`[Error] Failed to get subreddit info for ${subredditName}:`, error)
+      // Failed to get subreddit info
       throw new Error(`Failed to get subreddit info for ${subredditName}`)
     }
   }
@@ -177,7 +177,7 @@ export class RedditClient {
         }
       })
     } catch (error) {
-      console.error(`[Error] Failed to get top posts for ${subreddit || "home"}:`, error)
+      // Failed to get top posts
       throw new Error(`Failed to get top posts for ${subreddit || "home"}`)
     }
   }
@@ -220,7 +220,7 @@ export class RedditClient {
         permalink: post.permalink,
       }
     } catch (error) {
-      console.error(`[Error] Failed to get post with ID ${postId}:`, error)
+      // Failed to get post
       throw new Error(`Failed to get post with ID ${postId}`)
     }
   }
@@ -234,7 +234,7 @@ export class RedditClient {
 
       return response.data.data.children.map((child: any) => child.data.display_name)
     } catch (error) {
-      console.error("[Error] Failed to get trending subreddits:", error)
+      // Failed to get trending subreddits
       throw new Error("Failed to get trending subreddits")
     }
   }
@@ -268,7 +268,7 @@ export class RedditClient {
         throw new Error("Failed to create post")
       }
     } catch (error) {
-      console.error(`[Error] Failed to create post in ${subreddit}:`, error)
+      // Failed to create post
       throw new Error(`Failed to create post in ${subreddit}`)
     }
   }
@@ -321,7 +321,7 @@ export class RedditClient {
         permalink: commentData.permalink,
       }
     } catch (error) {
-      console.error(`[Error] Failed to reply to post ${postId}:`, error)
+      // Failed to reply to post
       throw new Error(`Failed to reply to post ${postId}`)
     }
   }

package/src/index.ts

@@ -12,11 +12,6 @@ class RedditServer {
   private server: Server
 
   constructor() {
-    console.error("[Setup] Initializing Reddit Server...")
-
-    // Initialize the Reddit client
-    this.initializeRedditClient()
-
     this.server = new Server(
       {
         name: "reddit-mcp-server",
@@ -29,9 +24,19 @@ class RedditServer {
       },
     )
 
+    // Initialize the Reddit client
+    this.initializeRedditClient()
+
     this.setupToolHandlers()
 
-    this.server.onerror = (error) => console.error("[Error] Server error:", error)
+    this.server.onerror = async (error) => {
+      await this.server.sendLoggingMessage({
+        level: "error",
+        logger: "reddit-server",
+        data: `Server error: ${error}`,
+      })
+    }
+    
     process.on("SIGINT", async () => {
       await this.server.close()
       process.exit(0)
@@ -46,6 +51,8 @@ class RedditServer {
     const password = process.env.REDDIT_PASSWORD
 
     if (!clientId || !clientSecret) {
+      // Can't use server logging here as server isn't initialized yet
+      // Using console.error is OK for startup failures
       console.error(
         "[Error] Missing required Reddit API credentials. Please set REDDIT_CLIENT_ID and REDDIT_CLIENT_SECRET environment variables.",
       )
@@ -61,13 +68,9 @@ class RedditServer {
         password,
       })
 
-      console.error("[Setup] Reddit client initialized")
-      if (username && password) {
-        console.error(`[Setup] Authenticated as user: ${username}`)
-      } else {
-        console.error("[Setup] Running in read-only mode (no user authentication)")
-      }
+      // Client initialized successfully
     } catch (error) {
+      // Can't use server logging here as server isn't connected yet
       console.error("[Error] Failed to initialize Reddit client:", error)
       process.exit(1)
     }
@@ -222,7 +225,12 @@ class RedditServer {
         const toolName = request.params.name
         const toolParams = request.params.arguments || {}
 
-        console.error(`[Request] Tool call: ${toolName}`, toolParams)
+        // Log tool call
+        await this.server.sendLoggingMessage({
+          level: "debug",
+          logger: "reddit-server",
+          data: `Tool call: ${toolName}`,
+        })
 
         switch (toolName) {
           case "test_reddit_mcp_server":
@@ -280,7 +288,11 @@ class RedditServer {
         }
       } catch (error: unknown) {
         if (error instanceof Error) {
-          console.error("[Error] Error calling tool:", error.message)
+          await this.server.sendLoggingMessage({
+            level: "error",
+            logger: "reddit-server",
+            data: `Error calling tool: ${error.message}`,
+          })
 
           throw new McpError(ErrorCode.InternalError, `Failed to fetch data: ${error.message}`)
         }
@@ -293,7 +305,24 @@ class RedditServer {
   async run() {
     const transport = new StdioServerTransport()
     await this.server.connect(transport)
-    console.error("[Server] Server is running")
+    
+    // Log server startup
+    await this.server.sendLoggingMessage({
+      level: "info",
+      logger: "reddit-server",
+      data: "Reddit MCP Server is running",
+    })
+    
+    // Log authentication status
+    const username = process.env.REDDIT_USERNAME
+    const password = process.env.REDDIT_PASSWORD
+    await this.server.sendLoggingMessage({
+      level: "info",
+      logger: "reddit-server",
+      data: username && password 
+        ? `Authenticated as user: ${username}`
+        : "Running in read-only mode (no user authentication)",
+    })
   }
 }
 

package/src/tools/post-tools.ts

@@ -11,7 +11,7 @@ export async function getRedditPost(params: { subreddit: string; post_id: string
   }
 
   try {
-    console.error(`[Tool] Getting post ${post_id} from r/${subreddit}`)
+    // Getting post
     const post = await client.getPost(post_id, subreddit)
     const formattedPost = formatPostInfo(post)
 
@@ -54,7 +54,7 @@ ${formattedPost.bestTimeToEngage}
       ],
     }
   } catch (error) {
-    console.error(`[Error] Error getting post: ${error}`)
+    // Error will be logged by the server
     throw new McpError(ErrorCode.InternalError, `Failed to fetch post data: ${error}`)
   }
 }
@@ -68,7 +68,7 @@ export async function getTopPosts(params: { subreddit: string; time_filter?: str
   }
 
   try {
-    console.error(`[Tool] Getting top posts from r/${subreddit}`)
+    // Getting top posts
     const posts = await client.getTopPosts(subreddit, time_filter, limit)
     const formattedPosts = posts.map(formatPostInfo)
 
@@ -98,7 +98,7 @@ ${postSummaries}
       ],
     }
   } catch (error) {
-    console.error(`[Error] Error getting top posts: ${error}`)
+    // Error will be logged by the server
     throw new McpError(ErrorCode.InternalError, `Failed to fetch top posts: ${error}`)
   }
 }
@@ -112,7 +112,7 @@ export async function createPost(params: { subreddit: string; title: string; con
   }
 
   try {
-    console.error(`[Tool] Creating ${is_self ? "text" : "link"} post in r/${subreddit}`)
+    // Creating post
     const post = await client.createPost(subreddit, title, content, is_self)
     const formattedPost = formatPostInfo(post)
 
@@ -135,7 +135,7 @@ Your post has been successfully submitted to r/${formattedPost.subreddit}.
       ],
     }
   } catch (error) {
-    console.error(`[Error] Error creating post: ${error}`)
+    // Error will be logged by the server
     throw new McpError(ErrorCode.InternalError, `Failed to create post: ${error}`)
   }
 }
@@ -149,7 +149,7 @@ export async function replyToPost(params: { post_id: string; content: string; su
   }
 
   try {
-    console.error(`[Tool] Replying to post ${post_id}`)
+    // Replying to post
     const comment = await client.replyToPost(post_id, content)
     const formattedComment = formatCommentInfo(comment)
 
@@ -172,7 +172,7 @@ Your reply has been successfully posted.
       ],
     }
   } catch (error) {
-    console.error(`[Error] Error replying to post: ${error}`)
+    // Error will be logged by the server
     throw new McpError(ErrorCode.InternalError, `Failed to reply to post: ${error}`)
   }
 }

package/src/tools/subreddit-tools.ts

@@ -11,7 +11,7 @@ export async function getSubredditInfo(params: { subreddit_name: string }) {
   }
 
   try {
-    console.error(`[Tool] Getting info for r/${subreddit_name}`)
+    // Getting subreddit info
     const subreddit = await client.getSubredditInfo(subreddit_name)
     const formattedSubreddit = formatSubredditInfo(subreddit)
 
@@ -56,7 +56,7 @@ ${formattedSubreddit.description.full}
       ],
     }
   } catch (error) {
-    console.error(`[Error] Error getting subreddit info: ${error}`)
+    // Error will be logged by the server
     throw new McpError(ErrorCode.InternalError, `Failed to fetch subreddit data: ${error}`)
   }
 }
@@ -69,7 +69,7 @@ export async function getTrendingSubreddits() {
   }
 
   try {
-    console.error("[Tool] Getting trending subreddits")
+    // Getting trending subreddits
     const trendingSubreddits = await client.getTrendingSubreddits()
 
     return {
@@ -85,7 +85,7 @@ ${trendingSubreddits.map((subreddit, index) => `${index + 1}. r/${subreddit}`).j
       ],
     }
   } catch (error) {
-    console.error(`[Error] Error getting trending subreddits: ${error}`)
+    // Error will be logged by the server
     throw new McpError(ErrorCode.InternalError, `Failed to fetch trending subreddits: ${error}`)
   }
 }

package/src/tools/user-tools.ts

@@ -11,7 +11,7 @@ export async function getUserInfo(params: { username: string }) {
   }
 
   try {
-    console.error(`[Tool] Getting info for u/${username}`)
+    // Getting user info
     const user = await client.getUser(username)
     const formattedUser = formatUserInfo(user)
 
@@ -42,7 +42,7 @@ export async function getUserInfo(params: { username: string }) {
       ],
     }
   } catch (error) {
-    console.error(`[Error] Error getting user info: ${error}`)
+    // Error will be logged by the server
     throw new McpError(ErrorCode.InternalError, `Failed to fetch user data: ${error}`)
   }
 }

package/dist/bin.d.ts

@@ -1 +0,0 @@
-#!/usr/bin/env node