@wix/ditto-codegen-public 1.0.467 → 1.0.469

package/dist/opencode-tools/report-wix-feedback.ts

@@ -10,6 +10,36 @@ const execFileAsync = promisify(execFile);
 const FEEDBACK_API_URL = "https://www.wixapis.com/mcp-serverless/v1/feedback";
 const FEEDBACK_TIMEOUT_MS = 15_000;
 
+const WIX_MCP_TOOLS = [
+  "SearchWixSDKDocumentation",
+  "ReadFullDocsArticle",
+  "ReadFullDocsMethodSchema",
+  "SearchWixAPISpec",
+  "BrowseWixRESTDocsMenu",
+  "CallWixSiteAPI",
+  "ExecuteWixAPI",
+];
+
+// Lookbehind rather than \b so opencode's "wix-mcp_<tool>" form matches too.
+const WIX_DOCS_CITATION_RE = new RegExp(
+  `dev\\.wix\\.com|(?<![A-Za-z0-9])(?:${WIX_MCP_TOOLS.join("|")})\\b`,
+);
+
+const OUT_OF_SCOPE_RE = /\.md\b|\bskills?\b|@wix\/(?:patterns|design-system)\b/i;
+
+export const MISSING_CITATION_REPLY =
+  "Not sent: the report doesn't cite a dev.wix.com URL or the wix-mcp tool you called " +
+  "(e.g. SearchWixSDKDocumentation, ReadFullDocsMethodSchema). Only wix-mcp and " +
+  "dev.wix.com docs problems belong here — skill references, skill scripts, docs in " +
+  "node_modules, and your own code are out of scope. If it is a wix-mcp problem, resend " +
+  "with the citation; otherwise continue your task.";
+
+export const OUT_OF_SCOPE_REPLY =
+  "Not sent: the report mentions a skill, a .md reference, or @wix/patterns / " +
+  "@wix/design-system docs — none of those are wix-mcp or dev.wix.com docs problems. If " +
+  "part of it is a wix-mcp or dev.wix.com problem, resend only that part without " +
+  "mentioning them; otherwise continue your task.";
+
 async function getWixAccessToken(): Promise<string> {
   const { stdout } = await execFileAsync("wix", ["token", "--json"], {
     timeout: 30_000,
@@ -39,10 +69,11 @@ export default tool({
     "tools to the Wix MCP team: a search whose top results were off-target, a docs article " +
     "or example that is wrong or missing the fields you needed, a method schema that " +
     "doesn't match the installed SDK types, or a wix-mcp tool error. Include cases you " +
-    "recovered from. Do NOT report skill references (SKILL.md, references/*.md), package " +
-    "docs in node_modules, your own mistakes, or the user's request — those aren't wix-mcp " +
-    "issues. Every report must cite the wix-mcp tool or dev.wix.com URL involved. Call it " +
-    "at most ONCE per task, covering every friction point you've hit so far.",
+    "recovered from. Do NOT report skill references (SKILL.md, references/*.md), skill " +
+    "scripts, package docs in node_modules, your own mistakes, or the user's request — " +
+    "those aren't wix-mcp issues. Every report must cite the wix-mcp tool or dev.wix.com " +
+    "URL involved. Call it at most ONCE per task, covering every friction point you've " +
+    "hit so far.",
   args: {
     message: tool.schema
       .string()
@@ -63,6 +94,13 @@ export default tool({
       ),
   },
   async execute(args) {
+    if (!WIX_DOCS_CITATION_RE.test(args.message)) {
+      return MISSING_CITATION_REPLY;
+    }
+    if (OUT_OF_SCOPE_RE.test(args.message)) {
+      return OUT_OF_SCOPE_REPLY;
+    }
+
     const message = [
       buildProvenance(),
       args.message,

package/dist/out.js

@@ -14916,7 +14916,8 @@ ${environmentIdentity}`;
   - \`batch-write\` to create N new files in one call. NEVER call \`write\` more than once per turn \u2014 use batch-write instead. \`write\` is reserved for single-file creation.
   - \`batch-read\` to read N files in one call. NEVER call \`read\` more than once per turn \u2014 use batch-read instead.
   - \`multi-edit\` to apply N find-and-replace edits across one or more files in one call. NEVER call \`edit\` more than once per turn \u2014 use multi-edit instead. \`edit\` is reserved for a single edit to one file. When ${multiEditContextClause} across N files, use ONE \`multi-edit\` with all N edits.
-  - Tool arguments are strict JSON, and \`write\`/\`edit\`/\`batch-write\`/\`multi-edit\` write \`content\`/\`oldString\`/\`newString\` to disk EXACTLY as decoded from that JSON \u2014 there is no second unescaping pass, so encode for the ONE decode that happens. Use \`\\n\`/\`\\t\`/\`\\"\` in the JSON when you want the file to contain a REAL newline/tab/quote character (a raw unescaped newline inside a JSON string value breaks parsing). If the file itself must contain the two LITERAL characters backslash-then-n (e.g. inside a string literal like \`.join('\\n')\`, a regex, or an already-escaped JSON value), escape the backslash itself in your JSON so it survives the one decode intact \u2014 do NOT emit a bare \`\\n\` expecting a tool to turn it back into two literal characters for you; none of them do.
+  - \`apply_patch\` (when available): inside each \`*** Update File\` section, list hunks in FILE ORDER, top to bottom. The applier only searches forward from where the previous hunk matched, so a hunk whose lines sit ABOVE the previous one fails with "Failed to find expected lines" even though the lines exist \u2014 the whole patch is then rejected. Unsure of the order? Re-read the file first, or split into one patch per hunk.
+  - Tool arguments are strict JSON, and \`write\`/\`edit\`/\`batch-write\`/\`multi-edit\`/\`apply_patch\` write \`content\`/\`oldString\`/\`newString\`/\`patchText\` to disk EXACTLY as decoded from that JSON \u2014 there is no second unescaping pass, so encode for the ONE decode that happens. Use \`\\n\`/\`\\t\`/\`\\"\` in the JSON when you want the file to contain a REAL newline/tab/quote character (a raw unescaped newline inside a JSON string value breaks parsing). If the file itself must contain the two LITERAL characters backslash-then-n (e.g. inside a string literal like \`.join('\\n')\`, a regex, or an already-escaped JSON value), escape the backslash itself in your JSON so it survives the one decode intact \u2014 do NOT emit a bare \`\\n\` expecting a tool to turn it back into two literal characters for you; none of them do.
 - For tools where no batch variant exists (e.g. \`grep\`, \`glob\`), still emit MULTIPLE \`tool_use\` blocks in a SINGLE response when the calls are independent. Sequential turns waste a model round-trip per call.`;
     }
   }
@@ -14982,7 +14983,7 @@ FILE CREATION:
 
 GENERATED FILES:
 - \`src/extensions/site/components/**/*.generated.ts\` is an auto-generated editor-react-component manifest \u2014 NEVER hand-edit. Change a manifest VALUE in the component's \`*.extension.ts\`; change anything else in the \`*.tsx\` / \`*.module.css\` / \`*.props.ts\` source, then regenerate (workflow step 5). No other \`.generated.ts\` path is affected.
-- A write guard rejects \`write\`/\`edit\`/\`batch-write\`/\`multi-edit\` there, dropping the ENTIRE call \u2014 never batch that path with real edits, or those are lost too and must be resubmitted.
+- A write guard rejects \`write\`/\`edit\`/\`batch-write\`/\`multi-edit\`/\`apply_patch\` there, dropping the ENTIRE call \u2014 never batch that path with real edits (or put it in a patch alongside other hunks), or those are lost too and must be resubmitted.
 
 TYPESCRIPT:
 - The project enables \`verbatimModuleSyntax\`. Type-only imports MUST use \`import type\`.
@@ -15727,6 +15728,20 @@ export const SubagentToolLogger = async () => {
   }
 });
 
+// dist/opencode-integration/apply-patch.js
+var require_apply_patch = __commonJS({
+  "dist/opencode-integration/apply-patch.js"(exports2) {
+    "use strict";
+    Object.defineProperty(exports2, "__esModule", { value: true });
+    exports2.PATCH_FILE_HEADER_RE = void 0;
+    exports2.extractPatchPaths = extractPatchPaths;
+    exports2.PATCH_FILE_HEADER_RE = /^\*\*\* (?:Add File|Update File|Delete File|Move to):(.*)$/;
+    function extractPatchPaths(patchText) {
+      return patchText.split(/\r?\n/).map((line) => exports2.PATCH_FILE_HEADER_RE.exec(line)?.[1].trim() ?? "").filter((p) => p !== "");
+    }
+  }
+});
+
 // dist/opencode-integration/execution/parser.js
 var require_parser = __commonJS({
   "dist/opencode-integration/execution/parser.js"(exports2) {
@@ -15741,6 +15756,7 @@ var require_parser = __commonJS({
     exports2.summarizeOpenCodeLine = summarizeOpenCodeLine;
     var types_1 = require_types4();
     var subagent_tool_logger_1 = require_subagent_tool_logger();
+    var apply_patch_1 = require_apply_patch();
     function parseRequiredPermissions(output) {
       let permissions = [];
       const lines = output.split("\n");
@@ -15851,7 +15867,14 @@ var require_parser = __commonJS({
     function summarizeOutput(tool, output) {
       return FULL_OUTPUT_TOOLS.has(tool) ? truncate(output, FULL_OUTPUT_CAP) : truncate(output);
     }
+    var PATCH_TEXT_CAP = 4e3;
     function summarizeInput(input) {
+      if (typeof input.patchText === "string") {
+        return {
+          patchFiles: (0, apply_patch_1.extractPatchPaths)(input.patchText),
+          patchText: truncate(input.patchText, PATCH_TEXT_CAP)
+        };
+      }
       if (input.files) {
         return { files: input.files.map(({ path }) => ({ path })) };
       }
@@ -15866,6 +15889,18 @@ var require_parser = __commonJS({
         ...input.description && { description: truncate(input.description) }
       };
     }
+    function summarizeMetadata(metadata) {
+      const trimmed = { ...metadata };
+      delete trimmed.diff;
+      if (trimmed.files) {
+        trimmed.files = trimmed.files.map((file) => {
+          const copy = { ...file };
+          delete copy.patch;
+          return copy;
+        });
+      }
+      return trimmed;
+    }
     function summarizeOpenCodeLine(line) {
       if (line.type !== "tool_use" && line.type !== subagent_tool_logger_1.SUBAGENT_TOOL_EVENT_TYPE) {
         return line;
@@ -15881,6 +15916,9 @@ var require_parser = __commonJS({
             ...state?.input && { input: summarizeInput(state.input) },
             ...state?.output && {
               output: summarizeOutput(part.tool, state.output)
+            },
+            ...state?.metadata && part.tool === "apply_patch" && {
+              metadata: summarizeMetadata(state.metadata)
             }
           }
         }
@@ -23929,8 +23967,9 @@ var require_generated_files_guard = __commonJS({
     exports2.buildGeneratedFilesGuardSource = buildGeneratedFilesGuardSource;
     exports2.writeGeneratedFilesGuardPlugin = writeGeneratedFilesGuardPlugin;
     var write_plugin_1 = require_write_plugin();
+    var apply_patch_1 = require_apply_patch();
     var ERC_MANIFEST_RE = /(^|\/)src\/extensions\/site\/components\/.+\.generated\.tsx?$/;
-    exports2.GUARD_MESSAGE = "BLOCKED: this editor-react-component manifest is auto-generated. Do not create or edit it by hand \u2014 edit the *.tsx / *.module.css / *.props.ts source and regenerate with `npx wix build && npx wix generate manifest`, or override values in the component's *.extension.ts. Do not re-attempt the blocked path; the whole call was rejected, so resubmit any other paths it carried \u2014 they were not written. Only *.generated.ts under src/extensions/site/components/ is blocked.";
+    exports2.GUARD_MESSAGE = "BLOCKED: this editor-react-component manifest is auto-generated. Do not create or edit it by hand \u2014 edit the *.tsx / *.module.css / *.props.ts source and regenerate with `npx wix build && npx wix generate manifest`, or override values in the component's *.extension.ts. Do not re-attempt the blocked path; the whole call was rejected \u2014 including every other file or apply_patch hunk it carried \u2014 so resubmit those; they were not written. Only *.generated.ts under src/extensions/site/components/ is blocked.";
     function isEditorComponentManifest(path) {
       return ERC_MANIFEST_RE.test(path);
     }
@@ -23938,6 +23977,9 @@ var require_generated_files_guard = __commonJS({
       if (tool === "write" || tool === "edit") {
         return typeof args.filePath === "string" ? [args.filePath] : [];
       }
+      if (tool === "apply_patch") {
+        return typeof args.patchText === "string" ? (0, apply_patch_1.extractPatchPaths)(args.patchText) : [];
+      }
       const list = tool === "batch-write" ? args.files : tool === "multi-edit" ? args.edits : void 0;
       if (!list)
         return [];
@@ -23948,11 +23990,19 @@ var require_generated_files_guard = __commonJS({
       return `// Generated by @wix/ditto-codegen (generated-files-guard). Do not edit.
 const ERC_MANIFEST_RE = /${ERC_MANIFEST_RE.source}/;
 const GUARD_MESSAGE = ${JSON.stringify(exports2.GUARD_MESSAGE)};
+const PATCH_FILE_HEADER_RE = /${apply_patch_1.PATCH_FILE_HEADER_RE.source}/;
 
 function extractWritePaths(tool, args) {
   if (tool === "${"write"}" || tool === "${"edit"}") {
     return typeof args.filePath === "string" ? [args.filePath] : [];
   }
+  if (tool === "${"apply_patch"}") {
+    if (typeof args.patchText !== "string") return [];
+    return args.patchText
+      .split(/\\r?\\n/)
+      .map((line) => PATCH_FILE_HEADER_RE.exec(line)?.[1].trim() ?? "")
+      .filter((p) => p !== "");
+  }
   const list =
     tool === "${"batch-write"}" ? args.files : tool === "${"multi-edit"}" ? args.edits : undefined;
   if (!list) return [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wix/ditto-codegen-public",
-  "version": "1.0.467",
+  "version": "1.0.469",
   "description": "AI-powered Wix CLI app generator - standalone executable",
   "scripts": {
     "build": "node build.mjs",
@@ -29,5 +29,5 @@
     "esbuild": "^0.27.2",
     "vitest": "^4.0.16"
   },
-  "falconPackageHash": "6e0bea815a8cc927e371476a9322bb0e2056a44108fca77338b370ff"
+  "falconPackageHash": "e5fba022f3f6b012f98ca1d776f6a961488ea78cc5b9c837b512c528"
 }