@open-macaron/review 0.2.0 → 0.3.0

package/macaron-review.mjs

@@ -21994,7 +21994,7 @@ var Usd = external_exports.number().nonnegative().max(1e3);
 // ../contracts/src/mode.ts
 var Mode = external_exports.enum(["triage", "plan", "implement", "address_review", "reproduce", "verify", "retro", "review"]);
 var Harness = external_exports.enum(["claude", "codex", "opencode"]);
-var Capability = external_exports.enum(["gui", "video", "container"]);
+var Platform = external_exports.enum(["linux", "windows", "macos"]);
 var ResultAction = external_exports.enum(["reply", "plan", "ask", "commit", "escalate", "noop", "review"]);
 
 // ../contracts/src/config.ts
@@ -22039,6 +22039,8 @@ var Contribution = external_exports.strictObject({
   cla: external_exports.url().optional(),
   fixCi: external_exports.boolean()
 });
+var PlatformSettings = external_exports.strictObject({ runsOn: external_exports.string().min(1) });
+var OtherPlatform = Platform.exclude(["linux"]);
 var MAX_TIMEOUT_MINUTES = 360;
 var RepoSettings = external_exports.strictObject({
   harness: Harness,
@@ -22047,7 +22049,10 @@ var RepoSettings = external_exports.strictObject({
   models: external_exports.partialRecord(Mode, external_exports.string().min(1)),
   /** The harness's small background model (Claude Code's "haiku" slot). Names follow the LLM backend. */
   fastModel: external_exports.string().min(1),
+  /** The Linux runner label. */
   runsOn: external_exports.string().min(1),
+  /** The other platforms runs may ask for (`@macaron-bot reproduce on windows`). */
+  platforms: external_exports.partialRecord(OtherPlatform, PlatformSettings).default({}),
   draftPr: external_exports.boolean(),
   timeoutMinutes: external_exports.int().min(5).max(MAX_TIMEOUT_MINUTES),
   /** USD per run, enforced by the run's LLM key. `review` is the ceiling of `reviewCost`. */
@@ -22056,7 +22061,6 @@ var RepoSettings = external_exports.strictObject({
   caps: Caps,
   environment: Environment,
   commands: Commands,
-  capabilities: external_exports.array(Capability),
   contribution: Contribution
 });
 var RepoSettingsPatch = external_exports.strictObject({
@@ -22065,6 +22069,7 @@ var RepoSettingsPatch = external_exports.strictObject({
   models: external_exports.partialRecord(Mode, external_exports.string().min(1)).optional(),
   fastModel: external_exports.string().min(1).optional(),
   runsOn: external_exports.string().min(1).optional(),
+  platforms: external_exports.partialRecord(OtherPlatform, PlatformSettings.partial()).optional(),
   draftPr: external_exports.boolean().optional(),
   timeoutMinutes: external_exports.int().min(5).max(MAX_TIMEOUT_MINUTES).optional(),
   budgets: Budgets.partial().optional(),
@@ -22072,7 +22077,6 @@ var RepoSettingsPatch = external_exports.strictObject({
   caps: Caps.partial().optional(),
   environment: EnvironmentPatch.optional(),
   commands: Commands.optional(),
-  capabilities: external_exports.array(Capability).optional(),
   contribution: Contribution.partial().optional()
 });
 var GlobalFile = external_exports.strictObject({
@@ -22126,8 +22130,10 @@ var RuleFrontMatter = external_exports.strictObject({
   /** Where the rule comes from: review comments, documentation. */
   sources: external_exports.array(HttpsUrl).default([])
 });
+var LessonDescription = external_exports.string().min(1).max(1024);
 var LessonFrontMatter = external_exports.strictObject({
   topic: external_exports.string().min(1).max(120),
+  description: LessonDescription.optional(),
   applies_to: external_exports.array(Glob).min(1).optional(),
   /** Evidence: a lesson without a source can't be reviewed. */
   sources: external_exports.array(HttpsUrl).min(1),
@@ -22151,6 +22157,7 @@ var PublishedLesson = external_exports.strictObject({
   id: external_exports.string().min(1),
   scope: KnowledgeScope,
   topic: external_exports.string(),
+  description: LessonDescription.optional(),
   appliesTo: external_exports.array(Glob).nullable(),
   sources: external_exports.array(HttpsUrl),
   lastVerified: external_exports.iso.date(),
@@ -22176,7 +22183,7 @@ var KnowledgeIndex = external_exports.strictObject({
       /** Relative to the index, e.g. `projects/sandbox.json`. */
       path: external_exports.string(),
       rules: external_exports.array(PublishedRule.pick({ id: true, title: true, severity: true, appliesTo: true })),
-      lessons: external_exports.array(PublishedLesson.pick({ id: true, topic: true }))
+      lessons: external_exports.array(PublishedLesson.pick({ id: true, topic: true, description: true }))
     })
   )
 });
@@ -22224,6 +22231,10 @@ var ReviewContext = external_exports.strictObject({
   /** open-macaron/knowledge commit the rules come from; null when no slice exists for the repo. */
   knowledgeCommit: CommitSha.nullable()
 });
+var RunKnowledge = external_exports.strictObject({
+  commit: CommitSha.nullable(),
+  lessons: external_exports.array(PublishedLesson).max(200)
+});
 var RunContext = external_exports.strictObject({
   mode: Mode,
   repo: RepoSlug,
@@ -22244,7 +22255,13 @@ var RunContext = external_exports.strictObject({
   commands: Commands,
   attachments: AttachmentLimits,
   /** Set iff mode is `review`. */
-  review: ReviewContext.optional()
+  review: ReviewContext.optional(),
+  /** The machine's operating system. Defaults keep contexts from controllers that predate platforms valid. */
+  platform: Platform.default("linux"),
+  /** The platforms this repository's runs can use: where a triage run may escalate to. */
+  platforms: external_exports.array(Platform).min(1).default(["linux"]),
+  /** Empty for reviews (they get rules, in `review`) and in contexts from controllers that predate it. */
+  knowledge: RunKnowledge.default({ commit: null, lessons: [] })
 });
 var RuleVerdict = external_exports.strictObject({
   ruleId: external_exports.string().min(1).max(200),
@@ -22277,12 +22294,18 @@ var Reply = external_exports.strictObject({
 var Learning = external_exports.strictObject({
   scope: external_exports.enum(["global", "project", "repo"]),
   topic: external_exports.string().min(1).max(120),
+  /** The lesson's skill description; the topic stands in for it when absent. */
+  description: LessonDescription.optional(),
   text: external_exports.string().min(1).max(4096),
-  evidence: external_exports.url()
+  evidence: external_exports.url(),
+  /** The id of the lesson this revises (`<scope>/lessons/<slug>`), from `context.knowledge`. */
+  updates: external_exports.string().min(1).max(300).optional()
 });
 var RunResult = external_exports.strictObject({
   action: ResultAction,
   escalateTo: Mode.optional(),
+  /** The platform the escalated run should use; the escalating run's own when absent. */
+  escalateOn: Platform.optional(),
   commitMessage: external_exports.string().min(1).max(4096).optional(),
   pr: external_exports.strictObject({ title: external_exports.string().min(1).max(256), body: Body }).optional(),
   message: Body.optional(),
@@ -22298,6 +22321,9 @@ var RunResult = external_exports.strictObject({
   if (result.action === "escalate" && result.escalateTo === void 0) {
     ctx.addIssue({ code: "custom", path: ["escalateTo"], message: "required when action is escalate" });
   }
+  if (result.action !== "escalate" && result.escalateOn !== void 0) {
+    ctx.addIssue({ code: "custom", path: ["escalateOn"], message: "only when action is escalate" });
+  }
   if (result.action === "commit" && result.commitMessage === void 0) {
     ctx.addIssue({ code: "custom", path: ["commitMessage"], message: "required when action is commit" });
   }
@@ -22983,12 +23009,16 @@ var KnowledgeSource = class {
   baseUrl;
   cache = /* @__PURE__ */ new Map();
   async rules(options) {
+    const { commit, rules } = await this.knowledge(options);
+    return { commit, rules };
+  }
+  async knowledge(options) {
     const paths = ["global.json"];
     if (options.project !== void 0) paths.push(`projects/${options.project}.json`);
     if (options.repo !== void 0) paths.push(`repos/${options.repo}.json`);
     const slices = await Promise.all(paths.map((path) => this.slice(path)));
     const present = slices.filter((slice) => slice !== void 0);
-    return { commit: present.at(-1)?.commit ?? null, rules: present.flatMap((slice) => slice.rules) };
+    return { commit: present.at(-1)?.commit ?? null, rules: present.flatMap((slice) => slice.rules), lessons: present.flatMap((slice) => slice.lessons) };
   }
   /** A missing slice (404) is a level without knowledge yet, not an error. */
   async slice(path) {
@@ -23002,7 +23032,8 @@ var KnowledgeSource = class {
     }
     if (!response.ok) throw new KnowledgeUnavailable(`${path}: HTTP ${String(response.status)}`, response.status);
     const parsed = KnowledgeSlice.safeParse(await response.json());
-    if (!parsed.success) throw new KnowledgeUnavailable(`${path}: not a knowledge slice`, response.status);
+    if (!parsed.success) throw new KnowledgeUnavailable(`${path}: not a knowledge slice:
+${external_exports.prettifyError(parsed.error)}`, response.status);
     const etag = response.headers.get("etag");
     if (etag !== null) this.cache.set(url2, { etag, slice: parsed.data });
     return parsed.data;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-macaron/review",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "macaron-bot's review engine: open-macaron/knowledge rules selected by changed files, checked by an agent in a read-only session, evidence verified. Used by the bot and, as the macaron-review CLI, locally.",
   "license": "Apache-2.0",
   "type": "module",