fix devvit logs (#15)

<!-- If this pull request closes an issue, please mention the issue
number below -->
Closes # <!-- Issue # here -->

## 💸 TL;DR
<!-- What's the three sentence summary of purpose of the PR -->

I gave llms too much power and it kept messing up stuff. Also, since is
required because we only stream this from the logs and I need to stop it
forcibly since we need a more rest styled request.

## 📜 Details
[Design Doc](<!-- insert Google Doc link here if applicable -->)

[Jira](<!-- insert Jira link if applicable -->)

<!-- Add additional details required for the PR: breaking changes,
screenshots, external dependency changes -->

## 🧪 Testing Steps / Validation
<!-- add details on how this PR has been tested, include reproductions
and screenshots where applicable -->

## ✅ Checks
<!-- Make sure your pr passes the CI checks and do check the following
fields as needed - -->
- [ ] CI tests (if present) are passing
- [ ] Adheres to code style for repo
- [ ] Contributor License Agreement (CLA) completed if not a Reddit
employee

Author: mwood23

package-lock.json

@@ -18,7 +18,7 @@
     "check": "npm run lint && npm run type-check && npm run prettier",
     "clean": "rm -rf ./dist ./fixtures ./db ./fixtures.tar.gz",
     "dev": "concurrently -p \"[{name}]\" -n \"ESBUILD,INSPECTOR\" -c \"blue,green\" \"npm run build:watch\" \"npm run dev:inspector\"",
-    "dev:inspector": "npx @modelcontextprotocol/inspector node ./dist/index.js",
+    "dev:inspector": "npx --yes @modelcontextprotocol/inspector@0.14.3 node ./dist/index.js",
     "lint": "eslint ./src",
     "lint:fix": "eslint --fix ./src",
     "prepare": "npm run check && npm run type-check && npm run build",
@@ -30,7 +30,7 @@
     "type-check": "tsc --noEmit"
   },
   "dependencies": {
-    "@modelcontextprotocol/sdk": "1.11.0",
+    "@modelcontextprotocol/sdk": "1.17.3",
     "@xenova/transformers": "2.17.2",
     "archiver": "^6.0.2",
     "better-sqlite3": "11.9.1",

package.json

@@ -12,29 +12,17 @@ This is a convenience wrapper around the \`devvit logs\` CLI command. Supply a s
 
 Examples:
   { subreddit: "mySubreddit" }
-  { subreddit: "r/myTestSubreddit", app: "my-app", json: true, since: "15m" }`,
+  { subreddit: "r/myTestSubreddit", app: "my-app", since: "15m" }`,
   inputSchema: z.object({
     subreddit: z.string().describe('Provide the subreddit name. The "r/" prefix is optional'),
+    since: z.string().describe('Start time for logs (e.g. "15s", "2w1d", "30m").'),
     app: z.string().optional().describe('Provide the app name'),
-    config: z.string().optional().describe('Path to devvit config file (default: devvit.yaml)'),
-    connect: z.boolean().optional().default(false).describe('Connect to local runtime'),
-    dateformat: z
-      .string()
-      .optional()
-      .describe(
-        'Format for rendering dates (default: "MMM d HH:mm:ss"). See https://date-fns.org/docs/format for formatting options.'
-      ),
-    json: z.boolean().optional().default(false).describe('Output JSON for each log line'),
-    since: z
-      .string()
-      .optional()
-      .describe('Start time for logs (e.g. "15s", "2w1d", "30m"). Defaults to 0m (now)'),
-    verbose: z.boolean().optional().default(true).describe('Enable verbose output'),
+    config: z.string().optional().describe('Path to devvit config file (default: devvit.json)'),
   }),
   handler: async ({ params }) => {
-    const { subreddit, app, config, connect, dateformat, json: jsonFlag, since, verbose } = params;
+    const { subreddit, app, config, since } = params;
 
-    const args: string[] = ['logs', subreddit];
+    const args: string[] = ['devvit', 'logs', subreddit];
     const repoRoot = resolveRepoRoot();
 
     if (!repoRoot && !params.app) {
@@ -53,16 +41,12 @@ Examples:
 
     // Flags ---------------------------------------------------------------
     if (config) args.push('--config', config);
-    if (connect) args.push('--connect');
-    if (dateformat) args.push('--dateformat', dateformat);
-    if (jsonFlag) args.push('--json');
-    if (since) args.push('--since', since);
-    if (verbose) args.push('--verbose');
+    args.push('--since', since);
 
     logger.info(`🚀 Executing: devvit ${args.join(' ')}`);
 
     return await new Promise((resolve) => {
-      const child = spawn('devvit', args, {
+      const child = spawn('npx', args, {
         stdio: ['ignore', 'pipe', 'pipe'],
         env: process.env,
         cwd: repoRoot,
@@ -96,10 +80,12 @@ Examples:
         });
       });
 
-      const KILL_TIMEOUT_MS = 2_000; // Prevent indefinite hanging by killing process after 10s
+      const KILL_TIMEOUT_MS = 4_000; // Prevent indefinite hanging by killing process after 4s
 
       const timeout = setTimeout(() => {
-        logger.info(`⌛ Reached timeout (${KILL_TIMEOUT_MS}ms) – terminating devvit logs process…`);
+        logger.error(
+          `⌛ Reached timeout (${KILL_TIMEOUT_MS}ms) – terminating devvit logs process…`
+        );
         child.kill('SIGINT');
       }, KILL_TIMEOUT_MS);