@mpgd/cli 0.8.0 → 0.9.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mpgd/cli",
-  "version": "0.8.0",
+  "version": "0.9.0",
   "description": "Gunshi CLI for mpgd-kit starter generation and target workflow orchestration.",
   "license": "MIT",
   "repository": {
@@ -51,14 +51,14 @@
     "typescript": "7.0.2",
     "@mpgd/adapter-ait": "0.4.2",
     "@mpgd/adapter-browser": "0.4.2",
-    "@mpgd/adapter-capacitor": "0.4.2",
-    "@mpgd/adapter-devvit": "0.6.0",
     "@mpgd/analytics": "0.3.5",
+    "@mpgd/adapter-capacitor": "0.4.2",
+    "@mpgd/adapter-devvit": "0.7.0",
     "@mpgd/bridge": "0.6.0",
-    "@mpgd/game-services": "0.6.0",
     "@mpgd/catalog": "0.3.5",
-    "@mpgd/i18n": "0.5.1",
+    "@mpgd/game-services": "0.7.0",
     "@mpgd/phaser-assets": "0.4.1",
+    "@mpgd/i18n": "0.5.1",
     "@mpgd/platform": "0.5.1",
     "@mpgd/target-config": "0.6.1"
   },

package/templates/phaser-game/apps/target-devvit/README.md

@@ -38,9 +38,8 @@ not required. Devvit-owned menu, scheduler, trigger, and form callbacks remain
 thin `/internal/...` routes and delegate to shared service functions. oRPC
 Publisher helpers can broadcast results after a task completes, but they do not
 replace the `devvit.json` scheduler endpoint and should not use process-local
-memory when delivery must cross instances. A thin `/api/mpgd/bridge` JSON route
-remains available for callers that explicitly configure the adapter's legacy
-`endpoint` option; new traffic uses oRPC by default.
+memory when delivery must cross instances. The generated target exposes only
+the oRPC bridge route; the former JSON compatibility route is not generated.
 
 The default post entry uses `index.html` for a lightweight inline preview. Its
 Play button requests the `game` entrypoint, which loads the separate

package/templates/phaser-game/apps/target-devvit/package.json

@@ -19,15 +19,15 @@
     "fix": "ttsc fix"
   },
   "dependencies": {
-    "@devvit/web": "0.13.7",
+    "@devvit/web": "0.13.8",
     "@mpgd/adapter-devvit": "__MPGD_DEPENDENCY_VERSION_ADAPTER_DEVVIT__",
     "@mpgd/bridge": "__MPGD_DEPENDENCY_VERSION_BRIDGE__"
   },
   "devDependencies": {
-    "@devvit/start": "0.13.7",
+    "@devvit/start": "0.13.8",
     "@ttsc/unplugin": "0.18.4",
     "@types/node": "^24.0.0",
-    "devvit": "0.13.7",
+    "devvit": "0.13.8",
     "ttsc": "0.18.4",
     "typescript": "7.0.2",
     "vite": "^8.1.3"

package/templates/phaser-game/apps/target-devvit/src/server/index.ts

@@ -3,7 +3,6 @@ import type { IncomingMessage, ServerResponse } from 'node:http';
 import { createServer, context, getServerPort, reddit, redis } from '@devvit/web/server';
 import type { UiResponse } from '@devvit/web/shared';
 import {
-  assertBridgeRequest,
   createBridgeError,
   type BridgeRequest,
   type BridgeResponse,
@@ -17,8 +16,7 @@ import { createBridgeRpcNodeHandler } from '@mpgd/bridge/orpc/node';
 const redisKeyComponentPattern = /^[A-Za-z0-9:_-]{1,128}$/;
 const maxStorageKeyLength = 128;
 const maxEncodedStorageKeyLength = 384;
-const maxRequestBodySize = 1_000_000;
-const legacyBridgeEndpoint = '/api/mpgd/bridge';
+const maxRequestBodySize = 1_048_576;
 const leaderboardUpdateMaxAttempts = 3;
 const leaderboardBackoffBaseMs = 25;
 const leaderboardLockTtlMs = 2_000;
@@ -59,11 +57,6 @@ async function handleHttpRequest(
     return;
   }
 
-  if (request.method === 'POST' && requestPathname(request) === legacyBridgeEndpoint) {
-    await handleLegacyBridgeRequest(request, response);
-    return;
-  }
-
   if (await bridgeRpcHandler(request, response)) {
     return;
   }
@@ -73,73 +66,6 @@ async function handleHttpRequest(
   });
 }
 
-async function handleLegacyBridgeRequest(
-  request: IncomingMessage,
-  response: ServerResponse,
-): Promise<void> {
-  let body: unknown;
-
-  try {
-    body = await readJsonRequestBody(request, maxRequestBodySize);
-  } catch (error) {
-    const responseError = describeLegacyBridgeBodyError(error);
-
-    if (error instanceof RequestBodyTooLargeError) {
-      discardOversizedRequestBody(request, response);
-    }
-
-    if (responseError.retryable) {
-      console.error(`devvit legacy bridge body read failed: ${errorMessage(error)}`, error);
-    }
-
-    sendJson(
-      response,
-      responseError.status,
-      createBridgeError(
-        'unknown',
-        responseError.code,
-        responseError.message,
-        responseError.retryable,
-      ),
-    );
-    return;
-  }
-
-  let bridgeRequest: BridgeRequest;
-
-  try {
-    bridgeRequest = assertBridgeRequest(body);
-  } catch (error) {
-    console.warn(`devvit legacy bridge validation failed: ${errorMessage(error)}`);
-    sendJson(
-      response,
-      400,
-      createBridgeError(
-        requestIdFromBody(body),
-        'INVALID_BRIDGE_REQUEST',
-        'Invalid bridge request.',
-      ),
-    );
-    return;
-  }
-
-  try {
-    sendJson(response, 200, await handleBridgeRequest(bridgeRequest));
-  } catch (error) {
-    console.error(`devvit legacy bridge request failed: ${errorMessage(error)}`, error);
-    sendJson(
-      response,
-      500,
-      createBridgeError(
-        bridgeRequest.id,
-        'DEVVIT_BRIDGE_INTERNAL_ERROR',
-        'Devvit bridge request failed.',
-        true,
-      ),
-    );
-  }
-}
-
 async function handleCreatePostMenu(response: ServerResponse): Promise<void> {
   const subredditName = currentSubredditName();
 
@@ -690,38 +616,6 @@ function requestPathname(request: IncomingMessage): string {
 
 class RequestBodyTooLargeError extends Error {}
 
-function describeLegacyBridgeBodyError(error: unknown): {
-  readonly status: number;
-  readonly code: string;
-  readonly message: string;
-  readonly retryable: boolean;
-} {
-  if (error instanceof RequestBodyTooLargeError) {
-    return {
-      status: 413,
-      code: 'BRIDGE_REQUEST_TOO_LARGE',
-      message: 'Bridge request body is too large.',
-      retryable: false,
-    };
-  }
-
-  if (error instanceof SyntaxError) {
-    return {
-      status: 400,
-      code: 'INVALID_BRIDGE_REQUEST',
-      message: 'Bridge request body is not valid JSON.',
-      retryable: false,
-    };
-  }
-
-  return {
-    status: 500,
-    code: 'DEVVIT_BRIDGE_INTERNAL_ERROR',
-    message: 'Bridge request body could not be read.',
-    retryable: true,
-  };
-}
-
 async function* readRequestBodyChunks(
   request: IncomingMessage,
   maxBodySize: number,
@@ -751,19 +645,6 @@ async function drainRequestBody(
   }
 }
 
-async function readJsonRequestBody(
-  request: IncomingMessage,
-  maxBodySize: number,
-): Promise<unknown> {
-  const chunks: Buffer[] = [];
-
-  for await (const chunk of readRequestBodyChunks(request, maxBodySize)) {
-    chunks.push(chunk);
-  }
-
-  return JSON.parse(Buffer.concat(chunks).toString('utf8'));
-}
-
 function assertContentLengthWithinLimit(
   request: IncomingMessage,
   maxBodySize: number,
@@ -787,15 +668,6 @@ function discardOversizedRequestBody(
   request.resume();
 }
 
-function requestIdFromBody(input: unknown): string {
-  if (typeof input !== 'object' || input === null || !('id' in input)) {
-    return 'unknown';
-  }
-
-  const id = (input as { readonly id?: unknown }).id;
-  return typeof id === 'string' && id.length > 0 ? id : 'unknown';
-}
-
 function setResponseSecurityHeaders(response: ServerResponse): void {
   response.setHeader('cache-control', 'no-store');
   response.setHeader('content-security-policy', "default-src 'none'; frame-ancestors 'none'");

package/templates/phaser-game/mpgd.targets.json

@@ -43,7 +43,6 @@
     },
     "reddit": {
       "kind": "devvit-web",
-      "buildStrategy": "devvit-vite",
       "gameApp": ".",
       "wrapperApp": "apps/target-devvit",
       "adapter": "devvit",

package/templates/phaser-game/vite.shared.ts

@@ -51,12 +51,8 @@ export function createGameViteSharedConfig(
       alias: {
         ...createCatalogAliases({
           gameRoot: input.gameRoot,
-          ...(input.productCatalogFile === undefined
-            ? {}
-            : { productCatalogFile: input.productCatalogFile }),
-          ...(input.adPlacementsFile === undefined
-            ? {}
-            : { adPlacementsFile: input.adPlacementsFile }),
+          productCatalogFile: input.productCatalogFile,
+          adPlacementsFile: input.adPlacementsFile,
         }),
         '#mpgd-platform-gateway': resolve(input.gameRoot, buildGatewayModule),
       },
@@ -99,8 +95,8 @@ export function resolveBuildGatewayModule(input: {
 
 function createCatalogAliases(input: {
   readonly gameRoot: string;
-  readonly productCatalogFile?: string;
-  readonly adPlacementsFile?: string;
+  readonly productCatalogFile: string | undefined;
+  readonly adPlacementsFile: string | undefined;
 }): Record<string, string> {
   const productCatalogFile = readConfiguredPath(
     input.productCatalogFile ?? process.env.MPGD_PRODUCT_CATALOG_FILE,
@@ -209,9 +205,8 @@ function resolveCatalogBaseDir(path: string, pairedPath: string, gameRoot: strin
     }
   }
 
-  console.warn(
+  throw new Error(
     `Could not locate catalog files (${path}, ${pairedPath}) in any expected directory; `
-    + `falling back to game root ${gameRoot}.`,
+      + `checked: ${candidates.filter((candidate) => candidate !== undefined).join(', ')}.`,
   );
-  return gameRoot;
 }