@amerged/ohmyhost-mcp 0.1.15 → 0.1.18

package/src/apps/product-cli/runtime-contract.ts

@@ -1,3 +1,4 @@
+import { parseFeedbackSubmissionDetail } from "@ohmyhost/contracts/feedback";
 import { parseFrameworkConversionDetail } from "@ohmyhost/contracts/framework-admission";
 import {
   parseOperationDeploymentProgress,
@@ -21,12 +22,13 @@ import {
 } from "@ohmyhost/contracts/operation-failure";
 export type { PublicOperationFailure } from "@ohmyhost/contracts/operation-failure";
 
-const PROBLEM_POLICY = {
+/** Exported for the documentation build, which publishes one page per code. */
+export const PROBLEM_POLICY = {
   cloudflare_zone_not_bound: {
     status: 409,
     retryable: false,
     action:
-      "Configure the project domain first with 'ohmyhost domain paid plan' then 'ohmyhost domain paid apply', or configure its sender domain with 'ohmyhost mail domain set'. Then request Cloudflare authorization for that domain's zone.",
+      "Configure the project domain first with 'ohmyhost domain paid plan' then 'ohmyhost domain paid apply', or configure its sender domain with 'ohmyhost mail setup'. Then request Cloudflare authorization for that domain's zone.",
   },
   github_connection_required: {
     status: 409,
@@ -197,6 +199,12 @@ const PROBLEM_POLICY = {
     retryable: false,
     action: "Select the current succeeded dev deployment and target prod.",
   },
+  mail_capacity_unavailable: {
+    status: 503,
+    retryable: true,
+    action:
+      "Report this request ID through feedback. The stored mail setup continues automatically once ohmyho.st has capacity; follow mail status and keep the original idempotency key.",
+  },
   mail_domain_conflict: {
     status: 409,
     retryable: false,
@@ -205,7 +213,8 @@ const PROBLEM_POLICY = {
   mail_domain_required: {
     status: 409,
     retryable: false,
-    action: "Run 'ohmyhost mail domain set' for this project, then plan again.",
+    action:
+      "This app declares mail.enabled. Configure the project's own sender domain with 'ohmyhost mail setup' using its Prod environment ID, or set mail.enabled to false if the app sends no mail, then plan again.",
   },
   storage_jurisdiction_conflict: {
     status: 409,
@@ -213,6 +222,18 @@ const PROBLEM_POLICY = {
     action:
       "Set storage.jurisdiction in ohmyhost.yaml to the project's hosting region shown by project status, then plan again.",
   },
+  shared_data_requires_promotion: {
+    status: 409,
+    retryable: false,
+    action:
+      "Plan and deploy this commit to Dev, then run 'ohmyhost deployment promote plan' to reach Prod.",
+  },
+  production_deployment_required: {
+    status: 409,
+    retryable: false,
+    action:
+      "Deploy to Prod first: run 'ohmyhost plan --environment prod' and deploy that plan, or promote the current Dev deployment with 'ohmyhost deployment promote plan'. Then plan the domain again.",
+  },
   framework_conversion_required: {
     status: 409,
     retryable: false,
@@ -308,6 +329,8 @@ export class ResponseContractError extends Error {
 
 export interface SafeProblem {
   readonly requestId: string;
+  /** The error's documentation page, when the server names one on docs.ohmyho.st. */
+  readonly docsUrl?: string;
   readonly retryAfterSeconds?: number;
   readonly code: ProblemCode;
   readonly retryable: boolean;
@@ -690,6 +713,28 @@ export const parseCloudflareDnsAuthorizationStatus = (
   return input as unknown as PublicCloudflareDnsAuthorizationStatus;
 };
 
+export interface PublicDevAccessState {
+  readonly project_id: string;
+  readonly mode: "protected" | "public";
+  readonly share_url: string | null;
+}
+
+export const parseDevAccessState = (value: unknown): PublicDevAccessState => {
+  const input = exactRecord(value, ["project_id", "mode", "share_url"]);
+  if (
+    !isUlid(input["project_id"]) ||
+    (input["mode"] !== "protected" && input["mode"] !== "public") ||
+    (input["mode"] === "public" && input["share_url"] !== null) ||
+    (input["share_url"] !== null &&
+      (typeof input["share_url"] !== "string" ||
+        !/^https:\/\/dev-[a-z0-9]+(?:-[a-z0-9]+){0,4}\.(?:dev\.)?check\.omh\.st\/\.ohmyhost\/dev-access\/redeem\?stage=dev&ticket=[A-Za-z0-9_-]{43}$/u.test(
+          input["share_url"],
+        )))
+  )
+    throw new ResponseContractError();
+  return input as unknown as PublicDevAccessState;
+};
+
 export const parseDevAccessTicket = (value: unknown): PublicDevAccessTicket => {
   const input = exactRecord(value, [
     "ticket_id",
@@ -943,6 +988,7 @@ export const parseProjectStatus = (value: unknown): Readonly<Record<string, unkn
     "etag",
     "handle",
     "region",
+    "dev_access_mode",
     "lifecycle",
     "source",
     "default_environment",
@@ -961,6 +1007,7 @@ export const parseProjectStatus = (value: unknown): Readonly<Record<string, unkn
     !boundedString(handle, 3, 59) ||
     !/^[a-z0-9]+(?:-[a-z0-9]+){0,4}$/u.test(handle) ||
     (input["region"] !== "us" && input["region"] !== "eu") ||
+    (input["dev_access_mode"] !== "protected" && input["dev_access_mode"] !== "public") ||
     (input["lifecycle"] !== "active" &&
       input["lifecycle"] !== "deleting" &&
       input["lifecycle"] !== "deleted") ||
@@ -1308,6 +1355,7 @@ export const parseDeploymentPlan = (value: unknown): Readonly<Record<string, unk
   const input = exactRecord(value, [
     "id",
     "project_id",
+    "environment",
     "commit_sha",
     "source_digest",
     "build_plan_digest",
@@ -1329,6 +1377,7 @@ export const parseDeploymentPlan = (value: unknown): Readonly<Record<string, unk
     "currency",
     "rate_card_version",
     "credit_micros",
+    "credits",
     "provider_cost_micros",
     "scope",
     "reserved_seconds",
@@ -1336,6 +1385,7 @@ export const parseDeploymentPlan = (value: unknown): Readonly<Record<string, unk
   if (
     !isUlid(input["id"]) ||
     !isUlid(input["project_id"]) ||
+    (input["environment"] !== "dev" && input["environment"] !== "prod") ||
     !isCommitSha(input["commit_sha"]) ||
     !isDigest(input["source_digest"]) ||
     !isDigest(input["build_plan_digest"]) ||
@@ -1355,6 +1405,8 @@ export const parseDeploymentPlan = (value: unknown): Readonly<Record<string, unk
     !boundedString(cost["currency"], 1, 16) ||
     !boundedString(cost["rate_card_version"], 1, 128) ||
     !decimal(cost["credit_micros"]) ||
+    typeof cost["credits"] !== "string" ||
+    !/^[0-9]+(?:\.[0-9]{1,6})?$/u.test(cost["credits"]) ||
     !decimal(cost["provider_cost_micros"]) ||
     cost["scope"] !== "build_compute" ||
     !decimal(cost["reserved_seconds"]) ||
@@ -1497,14 +1549,18 @@ export const parseSafeProblem = (value: unknown): SafeProblem => {
       retryAfter > 86400)
   )
     throw new ResponseContractError();
+  const docsUrl = String(input["type"]);
   return {
     requestId: input["request_id"],
+    ...(docsUrl.startsWith("https://docs.ohmyho.st/errors/") ? { docsUrl } : {}),
     code: code as ProblemCode,
     retryable: policy.retryable,
     suggestedAction:
       (code === "framework_conversion_required"
         ? (parseFrameworkConversionDetail(input["detail"]) ?? policy.action)
-        : policy.action) +
+        : code === "invalid_request"
+          ? (parseFeedbackSubmissionDetail(input["detail"]) ?? policy.action)
+          : policy.action) +
       (retryAfter === undefined ? "" : ` Retry after ${String(retryAfter)} seconds.`),
     ...(retryAfter === undefined ? {} : { retryAfterSeconds: retryAfter as number }),
   };