@wix/ditto-codegen-public 1.0.481 → 1.0.483

package/dist/opencode-tools/batch-read.ts

@@ -2,27 +2,75 @@ import { tool } from "@opencode-ai/plugin";
 import { readFile } from "fs/promises";
 import { join, isAbsolute } from "path";
 
+// opencode cuts any tool output over 2,000 lines or 50 KB and saves the rest to
+// a file the agent rarely reads, so every file after the cut is silently lost.
+// Stay well under both limits and name the files that didn't fit instead.
+export const OUTPUT_LIMIT = { bytes: 40 * 1024, lines: 1500 };
+
+const SECTION_SEPARATOR = "\n\n";
+
+async function readSection(
+  filePath: string,
+  directory: string,
+): Promise<string> {
+  const fullPath = isAbsolute(filePath) ? filePath : join(directory, filePath);
+  try {
+    const content = await readFile(fullPath, "utf-8");
+    return `--- ${filePath} ---\n${content}`;
+  } catch (e) {
+    const msg = e instanceof Error ? e.message : String(e);
+    return `--- ${filePath} ---\nERROR: ${msg}`;
+  }
+}
+
+function notReadSection(paths: string[]): string {
+  return (
+    `--- NOT READ (output limit) ---\n` +
+    `These files were not read, to keep this output under the tool output limit. ` +
+    `Read them with another batch-read call:\n` +
+    paths.map((path) => `- ${path}`).join("\n")
+  );
+}
+
 export default tool({
   description:
     "Read multiple files at once and return their contents. " +
-    "Use this instead of calling read multiple times sequentially.",
+    "Use this instead of calling read multiple times sequentially. " +
+    `Returns whole files, in order, up to about ${OUTPUT_LIMIT.bytes / 1024} KB and ${OUTPUT_LIMIT.lines} lines per call; ` +
+    "any files past that are listed as not read, so read them in another call.",
   args: {
     paths: tool.schema
       .array(tool.schema.string())
       .describe("Array of file paths (relative to project root) to read"),
   },
   async execute(args, context) {
-    const results: string[] = [];
+    const sections: string[] = [];
+    const notRead: string[] = [];
+    let bytes = 0;
+    let lines = 0;
     for (const filePath of args.paths) {
-      const fullPath = isAbsolute(filePath) ? filePath : join(context.directory, filePath);
-      try {
-        const content = await readFile(fullPath, "utf-8");
-        results.push(`--- ${filePath} ---\n${content}`);
-      } catch (e) {
-        const msg = e instanceof Error ? e.message : String(e);
-        results.push(`--- ${filePath} ---\nERROR: ${msg}`);
+      if (notRead.length > 0) {
+        notRead.push(filePath);
+        continue;
+      }
+      const section = await readSection(filePath, context.directory);
+      const sectionBytes =
+        Buffer.byteLength(section) + SECTION_SEPARATOR.length;
+      const sectionLines = section.split("\n").length + 1;
+      // The first file always goes in: alone, it is no worse than a plain read.
+      const fits =
+        sections.length === 0 ||
+        (bytes + sectionBytes <= OUTPUT_LIMIT.bytes &&
+          lines + sectionLines <= OUTPUT_LIMIT.lines);
+      if (!fits) {
+        notRead.push(filePath);
+        continue;
       }
+      sections.push(section);
+      bytes += sectionBytes;
+      lines += sectionLines;
     }
-    return results.join("\n\n");
+    if (notRead.length > 0) sections.push(notReadSection(notRead));
+    return sections.join(SECTION_SEPARATOR);
   },
 });

package/dist/out.js

@@ -14957,7 +14957,7 @@ var require_shared_rules = __commonJS({
   "dist/opencode-integration/rules/shared-rules.js"(exports2) {
     "use strict";
     Object.defineProperty(exports2, "__esModule", { value: true });
-    exports2.OUTPUT_MESSAGE_RULES = exports2.BOUNDARY_RULES = void 0;
+    exports2.OUTCOME_MESSAGE_RULES = exports2.OUTPUT_MESSAGE_RULES = exports2.BOUNDARY_RULES = void 0;
     exports2.buildEnvironmentRules = buildEnvironmentRules;
     exports2.buildFileOperationsRules = buildFileOperationsRules;
     function buildEnvironmentRules(environmentIdentity) {
@@ -14972,6 +14972,7 @@ ${environmentIdentity}`;
 - Match their technical level: include code/technical detail only if their request was technical; otherwise describe the outcome in plain language (no code, file/type names, or internals).
 - List any manual setup the user must still do for the feature to work, as short "next steps" (e.g. installing a dependent app or enabling a dashboard setting).
 - Be honest: if you didn't fully build what was asked (e.g. stored data but didn't send it), say so. Keep it short.`;
+    exports2.OUTCOME_MESSAGE_RULES = `- Lead with whether the request succeeded or failed and what changed (e.g. "Added\u2026", "Removed\u2026"). Never report validation, build, or type-check status as the outcome, and re-state the outcome in your last message even if you already said it earlier.`;
     function buildFileOperationsRules(multiEditContextClause) {
       return `- Package dependencies \u2014 \`npm install <pkg>\` any package not already in \`package.json\` BEFORE the first \`validate\` (do NOT wait for \`TS2307\`); install all needed packages in ONE command, except heavy ones: the sandbox has a memory limit and an install that exceeds it kills the machine and loses the whole run, so install those in several smaller commands. Do NOT hand-edit \`package.json\` for deps unless npm cannot be used. Never pass \`--package-lock=false\`.
 - File operations \u2014 pick the BATCH tool whenever you have 2+ files; the single-file variants are 5\u201310\xD7 slower:
@@ -15068,7 +15069,8 @@ The \`required-permissions\` tool records and surfaces permissions to the user a
         (0, shared_rules_1.buildEnvironmentRules)(environmentIdentity),
         shared_rules_1.BOUNDARY_RULES,
         buildCoreRules(validation),
-        shared_rules_1.OUTPUT_MESSAGE_RULES
+        `${shared_rules_1.OUTPUT_MESSAGE_RULES}
+${shared_rules_1.OUTCOME_MESSAGE_RULES}`
       ].join("\n\n");
     }
   }
@@ -15993,7 +15995,7 @@ var require_validation_state = __commonJS({
     exports2.VALIDATION_STATE_EVENT_TYPE = "validation_state";
     exports2.VALIDATION_LOADED_EVENT_TYPE = "validation_loaded";
     exports2.VALIDATION_STATE_PLUGIN_FILENAME = "validation-state.ts";
-    exports2.VALIDATION_NUDGE = "Project files need validation. Follow your validation workflow (delegate to codegen-validator if your instructions require it), run validate({ runBuild: true }), and fix any errors before finishing.";
+    exports2.VALIDATION_NUDGE = "Project files need validation. Follow your validation workflow (delegate to codegen-validator if your instructions require it), run validate({ runBuild: true }), and fix any errors before finishing. Your last message is shown to the user: once validation passes, end with a short plain-language summary of what changed for them, not the validation status.";
     function buildValidationStateSource() {
       return String.raw`// Generated by @wix/ditto-codegen (validation-state). Do not edit.
 const EVENT_TYPE = ${JSON.stringify(exports2.VALIDATION_STATE_EVENT_TYPE)};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wix/ditto-codegen-public",
-  "version": "1.0.481",
+  "version": "1.0.483",
   "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": "fb8725cd483f4c797567c369d6b0481e296080219712238314bf34dd"
+  "falconPackageHash": "08e5c0c0beef6a0acb833061c90a5e77d57ad5d9f552a1b4ee922c0b"
 }