@amerged/ohmyhost-mcp 0.1.31 → 0.1.33

package/src/packages/contracts/src/operation-failure.ts

@@ -1,3 +1,5 @@
+import { frameworkConversionDetail } from "./framework-admission.js";
+
 const failures = {
   authorization_changed: [
     "Authorization changed before this operation could finish.",
@@ -114,7 +116,7 @@ const failures = {
   ],
   auto_deploy_plan_rejected: [
     "The GitHub push could not be planned for deployment.",
-    "Read project status and the operation's error or deployment diagnostics. Correct the source or configuration named there, push the new commit and plan it before deploying. Replaying this failed operation cannot start a build.",
+    "Read error.detail: it names the file, the rule and the required change. Make that change, commit and push it; the next push is planned again. Without error.detail, plan the pushed commit explicitly with 'ohmyhost plan --project ULID --commit SHA --json' or MCP deployment_plan to read the reason. Replaying this failed operation cannot start a build.",
   ],
   promotion_source_stale: [
     "The Dev deployment changed before promotion could start.",
@@ -131,6 +133,8 @@ export interface PublicOperationFailure {
   readonly message: string;
   readonly retryable: false;
   readonly suggested_action: string;
+  /** The catalog sentence naming the refused file and its fix, only for an auto-deploy rejection. */
+  readonly detail?: string;
 }
 
 export interface OperationReconciliationObservation {
@@ -167,12 +171,26 @@ export function publicOperationReconciliation(
   });
 }
 
-/** Render only reviewed text; never return the internal/provider error message. */
-export function publicOperationFailure(internalCode: string): PublicOperationFailure {
+/**
+ * Render only reviewed text; never return the internal/provider error message. A stored framework
+ * conversion diagnostic of an auto-deploy rejection is re-rendered from the catalog as `detail`.
+ */
+export function publicOperationFailure(
+  internalCode: string,
+  diagnostic?: unknown,
+): PublicOperationFailure {
   const normalized = internalCode.toLowerCase();
   const code = Object.hasOwn(failures, normalized)
     ? (normalized as keyof typeof failures)
     : "operation_failed";
   const [message, suggested_action] = failures[code];
-  return Object.freeze({ code, message, retryable: false, suggested_action });
+  const detail =
+    code === "auto_deploy_plan_rejected" ? frameworkConversionDetail(diagnostic) : null;
+  return Object.freeze({
+    code,
+    message,
+    retryable: false,
+    suggested_action,
+    ...(detail === null ? {} : { detail }),
+  });
 }

package/src/packages/contracts/src/problem-policy.ts

@@ -350,10 +350,10 @@ export const PROBLEM_POLICY = {
     status: 409,
     retryable: false,
     action:
-      "Read detail for the file, reason and required change. Run ohmyhost init --dry-run --json at that commit, apply its blockers and the matching ohmyho.st framework Skill, then save or push the corrected version. Plan its commit; repeating the unchanged source cannot fix an admission error.",
+      "Read detail: a catalog sentence names the file, the rule and the required change. Apply it, save or push the corrected version and plan its commit. If detail names no file, run 'ohmyhost init --dry-run --json' at that commit and apply its blockers and the matching ohmyho.st framework Skill; init lists the install, build and server-only checks it cannot run locally under pending. Repeating the unchanged source cannot fix an admission error.",
     title: "Framework conversion required",
     restAction:
-      "Read detail for the file, reason and required change. Run ohmyhost init --dry-run --json at that commit, apply its blockers and the matching ohmyho.st framework Skill, then save or push the corrected version. Plan its commit; repeating the unchanged source cannot fix an admission error.",
+      "Read detail: a catalog sentence names the file, the rule and the required change. Apply it, save or push the corrected version and plan its commit. If detail names no file, run 'ohmyhost init --dry-run --json' at that commit and apply its blockers and the matching ohmyho.st framework Skill; init lists the install, build and server-only checks it cannot run locally under pending. Repeating the unchanged source cannot fix an admission error.",
   },
   repository_configuration_missing: {
     status: 409,

package/src/packages/sdk-ts/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ohmyhost/sdk-ts",
-  "version": "0.1.31",
+  "version": "0.1.33",
   "private": true,
   "type": "module",
   "exports": {

package/src/packages/sdk-ts/src/generated/sdk.gen.ts

@@ -3290,7 +3290,7 @@ export const applyPaidProjectDomain = <ThrowOnError extends boolean = true>(
 /**
  * Plan a deployment
  *
- * Resolves immutable inputs and estimates effects without provider mutation or billable work. The plan targets Dev unless `environment` is `prod`; a Prod plan builds straight into Prod, except that a project with shared data and a database only reaches Prod through promotion. Managed mail requires a confirmed Paid service period; it is checked again before build reservation. Selecting the managed Better Auth bridge does not require managed mail or Paid access. An old sender configuration is not Paid authority.
+ * Resolves immutable inputs and estimates effects without provider mutation or billable work. The plan targets Dev unless `environment` is `prod`; a Prod plan builds straight into Prod, except that a project with shared data and a database only reaches Prod through promotion. Managed mail requires a confirmed Paid service period; it is checked again before build reservation. Selecting the managed Better Auth bridge does not require managed mail or Paid access. An old sender configuration is not Paid authority. A commit the platform does not admit answers 409 with a nonretryable problem code; for framework_conversion_required, detail is a fixed catalog sentence that names the file, the rule and the required change whenever the refusal is typed.
  */
 export const planDeployment = <ThrowOnError extends boolean = true>(
   parameters: {

package/src/packages/sdk-ts/src/generated/types.gen.ts

@@ -1485,6 +1485,10 @@ export type OperationFailure = {
    */
   retryable: false;
   suggested_action: string;
+  /**
+   * Present only on auto_deploy_plan_rejected when planning the pushed commit returned a typed refusal. It is the same fixed catalog sentence that the planning 409 problem returns as detail, naming the file, the rule and the required change.
+   */
+  detail?: string;
 };
 
 export type AuditEvent = {
@@ -5839,6 +5843,10 @@ export type PlanDeploymentErrors = {
    * The resource does not exist or is not visible to the authenticated principal.
    */
   404: ProblemDetails;
+  /**
+   * The request failed.
+   */
+  409: ProblemDetails;
   /**
    * The request failed.
    */