@openhands/agent-canvas 1.15.0 → 1.16.0

package/dist/manifests/automation-interface.js

@@ -40,12 +40,15 @@ function f(e) {
 	return u(r.setup, e);
 }
 function p(e) {
+	return e === r.list || e.startsWith(`${r.list}/`);
+}
+function m(e) {
 	return l().endpoints[e] ?? "";
 }
-function m(e, t) {
-	return p(e).replace("{id}", encodeURIComponent(t));
+function h(e, t) {
+	return m(e).replace("{id}", encodeURIComponent(t));
 }
-function h() {
+function g() {
 	let e = l();
 	return {
 		sidebarLabel: e.navigation.sidebar.label,
@@ -58,19 +61,19 @@ function h() {
 		editTitle: e.pages.edit.title
 	};
 }
-function g() {
+function _() {
 	return l().importExport;
 }
-function _() {
+function v() {
 	return l().docsUrl;
 }
-function v() {
+function y() {
 	return l().featuredAutomationIds;
 }
-function y() {
+function b() {
 	return l().responderIntegrationIds;
 }
 //#endregion
-export { d as automationListPath, f as automationSetupPath, p as getAutomationEndpoint, m as getAutomationIdEndpoint, _ as getAutomationsDocsUrl, v as getFeaturedAutomationIds, g as getImportExportSpec, h as getInterfaceCopy, y as getResponderIntegrationIds, c as hasAutomationInterface };
+export { d as automationListPath, f as automationSetupPath, m as getAutomationEndpoint, h as getAutomationIdEndpoint, v as getAutomationsDocsUrl, y as getFeaturedAutomationIds, _ as getImportExportSpec, g as getInterfaceCopy, b as getResponderIntegrationIds, c as hasAutomationInterface, p as isAutomationsRoute };
 
 //# sourceMappingURL=automation-interface.js.map

package/dist/manifests/automation-interface.js.map

@@ -1 +1 @@
-{"version":3,"file":"automation-interface.js","names":[],"sources":["../../src/manifests/automation-interface.ts"],"sourcesContent":["/**\n * The automation interface seam.\n *\n * Every automation-specific datum the host's surfaces render — navigation and\n * page copy, the settable attributes, the import/export envelope, endpoint\n * paths, the featured and responder id lists, the sub-page surface — is served\n * from the interface manifest the pinned `@openhands/extensions` publishes.\n * The host holds none of it. When the package publishes no manifest, or the\n * published one fails admission, there is nothing to serve: the nav entries do\n * not render and the routes 404. `hasAutomationInterface()` is that gate, and\n * every accessor that needs the manifest is reachable only from behind it.\n *\n * Routes are the exception, and they are not a definition: `src/routes.ts`\n * mounts them, so the host states them here and admission checks a manifest's\n * routes against them. That check is what keeps a published deep link\n * resolving to the page it named.\n */\n\nimport { AUTOMATION_CATALOG } from \"@openhands/extensions/automations\";\nimport { validateInterfaceManifest } from \"./interface-validation\";\nimport { AUTOMATION_INTERFACE_CANDIDATE } from \"./manifest-sources\";\nimport type {\n  AutomationAttributeName,\n  InterfaceDashboardFilter,\n  InterfaceDashboardSort,\n  InterfaceEndpointName,\n  InterfaceIconSlug,\n  InterfaceImportExport,\n  InterfaceListInsights,\n  InterfaceManifest,\n  InterfaceOverview,\n  InterfaceRoutes,\n  InterfaceSubPageId,\n  InterfaceTemplatesPage,\n} from \"./types\";\n\n/** The routes this host has registrations for, in `src/routes.ts`. */\nconst MOUNTED_ROUTES = {\n  list: \"/automations\",\n  setup: \"/automations/new/:automationId\",\n  detail: \"/automations/:automationId\",\n  templates: \"/automations/templates\",\n} satisfies InterfaceRoutes;\n\n/**\n * File-format section for portable `.automation.json` export/import. A docs\n * pointer for the host's import picker, not interface data — the manifest's\n * `importExport` block has no docs field.\n */\nexport const AUTOMATION_FILE_FORMAT_DOCS_URL =\n  \"https://docs.openhands.dev/openhands/usage/agent-canvas/managing-automations#exported-file-format\";\n\nfunction isRecord(value: unknown): value is Record<string, unknown> {\n  return typeof value === \"object\" && value !== null && !Array.isArray(value);\n}\n\nfunction getCatalogIds(): ReadonlySet<string> {\n  return new Set(\n    (AUTOMATION_CATALOG as readonly unknown[]).flatMap((entry) =>\n      isRecord(entry) && typeof entry.id === \"string\" ? [entry.id] : [],\n    ),\n  );\n}\n\nfunction admitInterfaceManifest(candidate: unknown): InterfaceManifest | null {\n  if (candidate === undefined) return null;\n\n  const result = validateInterfaceManifest(candidate, {\n    catalogIds: getCatalogIds(),\n    mountedRoutes: MOUNTED_ROUTES,\n  });\n  if (!result.valid) {\n    // Mirrors the setup registry: a rejected manifest is skipped loudly, and\n    // the surfaces it would have described stay unrendered.\n    console.warn(\n      \"Rejected the automation interface manifest:\",\n      result.errors.join(\"; \"),\n    );\n    return null;\n  }\n  return candidate as InterfaceManifest;\n}\n\nconst ADMITTED = admitInterfaceManifest(AUTOMATION_INTERFACE_CANDIDATE);\n\n/**\n * Whether this deployment has an automation interface at all. The nav entries\n * and the route loaders ask before rendering; nothing else has to.\n */\nexport function hasAutomationInterface(): boolean {\n  return ADMITTED !== null;\n}\n\n/**\n * The admitted manifest, for the accessors that cannot answer without one.\n * Reaching one of those without a manifest means a surface rendered past its\n * gate — a wiring mistake, not a state to render.\n */\nfunction requireInterface(): InterfaceManifest {\n  if (!ADMITTED) {\n    throw new Error(\n      \"No automation interface manifest is admitted, so this surface should not have rendered.\",\n    );\n  }\n  return ADMITTED;\n}\n\nfunction substituteRouteParam(pattern: string, id: string): string {\n  return pattern.replace(\":automationId\", encodeURIComponent(id));\n}\n\nexport function automationListPath(): string {\n  return MOUNTED_ROUTES.list;\n}\n\nexport function automationSetupPath(id: string): string {\n  return substituteRouteParam(MOUNTED_ROUTES.setup, id);\n}\n\nexport function automationDetailPath(id: string): string {\n  return substituteRouteParam(MOUNTED_ROUTES.detail, id);\n}\n\nexport function automationTemplatesPath(): string {\n  return MOUNTED_ROUTES.templates;\n}\n\n/**\n * A declared endpoint path. Empty for one the manifest may omit - the two a\n * bundle needs were added after the block shipped - so a caller that needs one\n * says so rather than reading a host-held default that does not exist.\n */\nexport function getAutomationEndpoint(name: InterfaceEndpointName): string {\n  return requireInterface().endpoints[name] ?? \"\";\n}\n\n/** An id-parameterized endpoint with `{id}` substituted, encoded. */\nexport function getAutomationIdEndpoint(\n  name: \"detail\" | \"dispatch\" | \"runs\" | \"tarball\",\n  id: string,\n): string {\n  return getAutomationEndpoint(name).replace(\"{id}\", encodeURIComponent(id));\n}\n\nexport interface InterfaceCopy {\n  sidebarLabel: string;\n  commandMenuTitle: string;\n  commandMenuDescription: string;\n  commandMenuKeywords: string;\n  listTitle: string;\n  listSubtitle: string;\n  detailBackLabel: string;\n  editTitle: string;\n}\n\nexport function getInterfaceCopy(): InterfaceCopy {\n  const manifest = requireInterface();\n  return {\n    sidebarLabel: manifest.navigation.sidebar.label,\n    commandMenuTitle: manifest.navigation.commandMenu.title,\n    commandMenuDescription: manifest.navigation.commandMenu.description,\n    commandMenuKeywords: manifest.navigation.commandMenu.keywords,\n    listTitle: manifest.pages.list.title,\n    listSubtitle: manifest.pages.list.subtitle,\n    detailBackLabel: manifest.pages.detail.backLabel,\n    editTitle: manifest.pages.edit.title,\n  };\n}\n\nexport interface AttributeSpec {\n  present: boolean;\n  label: string;\n  /** Null when the manifest states no help text for the attribute. */\n  help: string | null;\n  required: boolean;\n  min: number | null;\n  max: number | null;\n}\n\n/** An attribute the manifest does not declare is not offered at all. */\nconst ABSENT_ATTRIBUTE: AttributeSpec = {\n  present: false,\n  label: \"\",\n  help: null,\n  required: false,\n  min: null,\n  max: null,\n};\n\nexport function getAttributeSpec(name: AutomationAttributeName): AttributeSpec {\n  const attribute = requireInterface().attributes[name];\n  if (!attribute) return ABSENT_ATTRIBUTE;\n  return {\n    present: true,\n    label: attribute.label,\n    help: attribute.help ?? null,\n    required: attribute.required,\n    min: attribute.constraints?.min ?? null,\n    max: attribute.constraints?.max ?? null,\n  };\n}\n\nexport function getImportExportSpec(): InterfaceImportExport {\n  return requireInterface().importExport;\n}\n\nexport function getAutomationsDocsUrl(): string {\n  return requireInterface().docsUrl;\n}\n\nexport function getFeaturedAutomationIds(): readonly string[] {\n  return requireInterface().featuredAutomationIds;\n}\n\nexport function getResponderIntegrationIds(): readonly string[] {\n  return requireInterface().responderIntegrationIds;\n}\n\nexport interface SubPageNavSpec {\n  page: InterfaceSubPageId;\n  /** The page's route, resolved through the host's route table. */\n  to: string;\n  label: string;\n  icon: InterfaceIconSlug;\n}\n\n/**\n * The sub-page navigation, or null when the manifest does not declare the\n * sub-page surface.\n */\nexport function getSubPagesSpec(): SubPageNavSpec[] | null {\n  const subPages = ADMITTED?.navigation.subPages;\n  if (!subPages) return null;\n  return subPages.map((item) => ({\n    page: item.page,\n    to:\n      item.page === \"templates\"\n        ? automationTemplatesPath()\n        : automationListPath(),\n    label: item.label,\n    icon: item.icon,\n  }));\n}\n\nexport interface DashboardSpec {\n  overview: InterfaceOverview;\n  filters: InterfaceDashboardFilter[];\n  sort: InterfaceDashboardSort;\n  insights: InterfaceListInsights;\n}\n\n/**\n * The list page's dashboard composition, or null when the manifest does not\n * declare it. Admission accepts the sub-page surface whole or not at all, so\n * these four are present together.\n */\nexport function getDashboardSpec(): DashboardSpec | null {\n  const list = ADMITTED?.pages.list;\n  if (!list?.overview || !list.filters || !list.sort || !list.insights) {\n    return null;\n  }\n  return {\n    overview: list.overview,\n    filters: list.filters,\n    sort: list.sort,\n    insights: list.insights,\n  };\n}\n\n/** The templates page identity, or null when the manifest does not declare it. */\nexport function getTemplatesPageSpec(): InterfaceTemplatesPage | null {\n  return ADMITTED?.pages.templates ?? null;\n}\n"],"mappings":";;;;AAqCA,IAAM,IAAiB;CACrB,MAAM;CACN,OAAO;CACP,QAAQ;CACR,WAAW;AACb;AAUA,SAAS,EAAS,GAAkD;CAClE,OAAO,OAAO,KAAU,cAAY,KAAkB,CAAC,MAAM,QAAQ,CAAK;AAC5E;AAEA,SAAS,IAAqC;CAC5C,OAAO,IAAI,IACR,EAA0C,SAAS,MAClD,EAAS,CAAK,KAAK,OAAO,EAAM,MAAO,WAAW,CAAC,EAAM,EAAE,IAAI,CAAC,CAClE,CACF;AACF;AAEA,SAAS,EAAuB,GAA8C;CAC5E,IAAI,MAAc,KAAA,GAAW,OAAO;CAEpC,IAAM,IAAS,EAA0B,GAAW;EAClD,YAAY,EAAc;EAC1B,eAAe;CACjB,CAAC;CAUD,OATK,EAAO,QASL,KANL,QAAQ,KACN,+CACA,EAAO,OAAO,KAAK,IAAI,CACzB,GACO;AAGX;AAEA,IAAM,IAAW,EAAuB,CAA8B;AAMtE,SAAgB,IAAkC;CAChD,OAAO,MAAa;AACtB;AAOA,SAAS,IAAsC;CAC7C,IAAI,CAAC,GACH,MAAU,MACR,yFACF;CAEF,OAAO;AACT;AAEA,SAAS,EAAqB,GAAiB,GAAoB;CACjE,OAAO,EAAQ,QAAQ,iBAAiB,mBAAmB,CAAE,CAAC;AAChE;AAEA,SAAgB,IAA6B;CAC3C,OAAO,EAAe;AACxB;AAEA,SAAgB,EAAoB,GAAoB;CACtD,OAAO,EAAqB,EAAe,OAAO,CAAE;AACtD;AAeA,SAAgB,EAAsB,GAAqC;CACzE,OAAO,EAAiB,EAAE,UAAU,MAAS;AAC/C;AAGA,SAAgB,EACd,GACA,GACQ;CACR,OAAO,EAAsB,CAAI,EAAE,QAAQ,QAAQ,mBAAmB,CAAE,CAAC;AAC3E;AAaA,SAAgB,IAAkC;CAChD,IAAM,IAAW,EAAiB;CAClC,OAAO;EACL,cAAc,EAAS,WAAW,QAAQ;EAC1C,kBAAkB,EAAS,WAAW,YAAY;EAClD,wBAAwB,EAAS,WAAW,YAAY;EACxD,qBAAqB,EAAS,WAAW,YAAY;EACrD,WAAW,EAAS,MAAM,KAAK;EAC/B,cAAc,EAAS,MAAM,KAAK;EAClC,iBAAiB,EAAS,MAAM,OAAO;EACvC,WAAW,EAAS,MAAM,KAAK;CACjC;AACF;AAmCA,SAAgB,IAA6C;CAC3D,OAAO,EAAiB,EAAE;AAC5B;AAEA,SAAgB,IAAgC;CAC9C,OAAO,EAAiB,EAAE;AAC5B;AAEA,SAAgB,IAA8C;CAC5D,OAAO,EAAiB,EAAE;AAC5B;AAEA,SAAgB,IAAgD;CAC9D,OAAO,EAAiB,EAAE;AAC5B"}
+{"version":3,"file":"automation-interface.js","names":[],"sources":["../../src/manifests/automation-interface.ts"],"sourcesContent":["/**\n * The automation interface seam.\n *\n * Every automation-specific datum the host's surfaces render — navigation and\n * page copy, the settable attributes, the import/export envelope, endpoint\n * paths, the featured and responder id lists, the sub-page surface — is served\n * from the interface manifest the pinned `@openhands/extensions` publishes.\n * The host holds none of it. When the package publishes no manifest, or the\n * published one fails admission, there is nothing to serve: the nav entries do\n * not render and the routes 404. `hasAutomationInterface()` is that gate, and\n * every accessor that needs the manifest is reachable only from behind it.\n *\n * Routes are the exception, and they are not a definition: `src/routes.ts`\n * mounts them, so the host states them here and admission checks a manifest's\n * routes against them. That check is what keeps a published deep link\n * resolving to the page it named.\n */\n\nimport { AUTOMATION_CATALOG } from \"@openhands/extensions/automations\";\nimport { validateInterfaceManifest } from \"./interface-validation\";\nimport { AUTOMATION_INTERFACE_CANDIDATE } from \"./manifest-sources\";\nimport type {\n  AutomationAttributeName,\n  InterfaceDashboardFilter,\n  InterfaceDashboardSort,\n  InterfaceEndpointName,\n  InterfaceIconSlug,\n  InterfaceImportExport,\n  InterfaceListInsights,\n  InterfaceManifest,\n  InterfaceOverview,\n  InterfaceRoutes,\n  InterfaceSubPageId,\n  InterfaceTemplatesPage,\n} from \"./types\";\n\n/** The routes this host has registrations for, in `src/routes.ts`. */\nconst MOUNTED_ROUTES = {\n  list: \"/automations\",\n  setup: \"/automations/new/:automationId\",\n  detail: \"/automations/:automationId\",\n  templates: \"/automations/templates\",\n} satisfies InterfaceRoutes;\n\n/**\n * File-format section for portable `.automation.json` export/import. A docs\n * pointer for the host's import picker, not interface data — the manifest's\n * `importExport` block has no docs field.\n */\nexport const AUTOMATION_FILE_FORMAT_DOCS_URL =\n  \"https://docs.openhands.dev/openhands/usage/agent-canvas/managing-automations#exported-file-format\";\n\nfunction isRecord(value: unknown): value is Record<string, unknown> {\n  return typeof value === \"object\" && value !== null && !Array.isArray(value);\n}\n\nfunction getCatalogIds(): ReadonlySet<string> {\n  return new Set(\n    (AUTOMATION_CATALOG as readonly unknown[]).flatMap((entry) =>\n      isRecord(entry) && typeof entry.id === \"string\" ? [entry.id] : [],\n    ),\n  );\n}\n\nfunction admitInterfaceManifest(candidate: unknown): InterfaceManifest | null {\n  if (candidate === undefined) return null;\n\n  const result = validateInterfaceManifest(candidate, {\n    catalogIds: getCatalogIds(),\n    mountedRoutes: MOUNTED_ROUTES,\n  });\n  if (!result.valid) {\n    // Mirrors the setup registry: a rejected manifest is skipped loudly, and\n    // the surfaces it would have described stay unrendered.\n    console.warn(\n      \"Rejected the automation interface manifest:\",\n      result.errors.join(\"; \"),\n    );\n    return null;\n  }\n  return candidate as InterfaceManifest;\n}\n\nconst ADMITTED = admitInterfaceManifest(AUTOMATION_INTERFACE_CANDIDATE);\n\n/**\n * Whether this deployment has an automation interface at all. The nav entries\n * and the route loaders ask before rendering; nothing else has to.\n */\nexport function hasAutomationInterface(): boolean {\n  return ADMITTED !== null;\n}\n\n/**\n * The admitted manifest, for the accessors that cannot answer without one.\n * Reaching one of those without a manifest means a surface rendered past its\n * gate — a wiring mistake, not a state to render.\n */\nfunction requireInterface(): InterfaceManifest {\n  if (!ADMITTED) {\n    throw new Error(\n      \"No automation interface manifest is admitted, so this surface should not have rendered.\",\n    );\n  }\n  return ADMITTED;\n}\n\nfunction substituteRouteParam(pattern: string, id: string): string {\n  return pattern.replace(\":automationId\", encodeURIComponent(id));\n}\n\nexport function automationListPath(): string {\n  return MOUNTED_ROUTES.list;\n}\n\nexport function automationSetupPath(id: string): string {\n  return substituteRouteParam(MOUNTED_ROUTES.setup, id);\n}\n\nexport function automationDetailPath(id: string): string {\n  return substituteRouteParam(MOUNTED_ROUTES.detail, id);\n}\n\nexport function automationTemplatesPath(): string {\n  return MOUNTED_ROUTES.templates;\n}\n\n/**\n * Whether `path` (a location pathname) is inside the automation surface: the\n * list route itself or anything nested under it (detail, setup, templates,\n * git-sync). Prefix-safe: \"/automations-foo\" does not match.\n */\nexport function isAutomationsRoute(path: string): boolean {\n  return (\n    path === MOUNTED_ROUTES.list || path.startsWith(`${MOUNTED_ROUTES.list}/`)\n  );\n}\n\n/**\n * A declared endpoint path. Empty for one the manifest may omit - the two a\n * bundle needs were added after the block shipped - so a caller that needs one\n * says so rather than reading a host-held default that does not exist.\n */\nexport function getAutomationEndpoint(name: InterfaceEndpointName): string {\n  return requireInterface().endpoints[name] ?? \"\";\n}\n\n/** An id-parameterized endpoint with `{id}` substituted, encoded. */\nexport function getAutomationIdEndpoint(\n  name: \"detail\" | \"dispatch\" | \"runs\" | \"tarball\",\n  id: string,\n): string {\n  return getAutomationEndpoint(name).replace(\"{id}\", encodeURIComponent(id));\n}\n\nexport interface InterfaceCopy {\n  sidebarLabel: string;\n  commandMenuTitle: string;\n  commandMenuDescription: string;\n  commandMenuKeywords: string;\n  listTitle: string;\n  listSubtitle: string;\n  detailBackLabel: string;\n  editTitle: string;\n}\n\nexport function getInterfaceCopy(): InterfaceCopy {\n  const manifest = requireInterface();\n  return {\n    sidebarLabel: manifest.navigation.sidebar.label,\n    commandMenuTitle: manifest.navigation.commandMenu.title,\n    commandMenuDescription: manifest.navigation.commandMenu.description,\n    commandMenuKeywords: manifest.navigation.commandMenu.keywords,\n    listTitle: manifest.pages.list.title,\n    listSubtitle: manifest.pages.list.subtitle,\n    detailBackLabel: manifest.pages.detail.backLabel,\n    editTitle: manifest.pages.edit.title,\n  };\n}\n\nexport interface AttributeSpec {\n  present: boolean;\n  label: string;\n  /** Null when the manifest states no help text for the attribute. */\n  help: string | null;\n  required: boolean;\n  min: number | null;\n  max: number | null;\n}\n\n/** An attribute the manifest does not declare is not offered at all. */\nconst ABSENT_ATTRIBUTE: AttributeSpec = {\n  present: false,\n  label: \"\",\n  help: null,\n  required: false,\n  min: null,\n  max: null,\n};\n\nexport function getAttributeSpec(name: AutomationAttributeName): AttributeSpec {\n  const attribute = requireInterface().attributes[name];\n  if (!attribute) return ABSENT_ATTRIBUTE;\n  return {\n    present: true,\n    label: attribute.label,\n    help: attribute.help ?? null,\n    required: attribute.required,\n    min: attribute.constraints?.min ?? null,\n    max: attribute.constraints?.max ?? null,\n  };\n}\n\nexport function getImportExportSpec(): InterfaceImportExport {\n  return requireInterface().importExport;\n}\n\nexport function getAutomationsDocsUrl(): string {\n  return requireInterface().docsUrl;\n}\n\nexport function getFeaturedAutomationIds(): readonly string[] {\n  return requireInterface().featuredAutomationIds;\n}\n\nexport function getResponderIntegrationIds(): readonly string[] {\n  return requireInterface().responderIntegrationIds;\n}\n\nexport interface SubPageNavSpec {\n  page: InterfaceSubPageId;\n  /** The page's route, resolved through the host's route table. */\n  to: string;\n  label: string;\n  icon: InterfaceIconSlug;\n}\n\n/**\n * The sub-page navigation, or null when the manifest does not declare the\n * sub-page surface.\n */\nexport function getSubPagesSpec(): SubPageNavSpec[] | null {\n  const subPages = ADMITTED?.navigation.subPages;\n  if (!subPages) return null;\n  return subPages.map((item) => ({\n    page: item.page,\n    to:\n      item.page === \"templates\"\n        ? automationTemplatesPath()\n        : automationListPath(),\n    label: item.label,\n    icon: item.icon,\n  }));\n}\n\nexport interface DashboardSpec {\n  overview: InterfaceOverview;\n  filters: InterfaceDashboardFilter[];\n  sort: InterfaceDashboardSort;\n  insights: InterfaceListInsights;\n}\n\n/**\n * The list page's dashboard composition, or null when the manifest does not\n * declare it. Admission accepts the sub-page surface whole or not at all, so\n * these four are present together.\n */\nexport function getDashboardSpec(): DashboardSpec | null {\n  const list = ADMITTED?.pages.list;\n  if (!list?.overview || !list.filters || !list.sort || !list.insights) {\n    return null;\n  }\n  return {\n    overview: list.overview,\n    filters: list.filters,\n    sort: list.sort,\n    insights: list.insights,\n  };\n}\n\n/** The templates page identity, or null when the manifest does not declare it. */\nexport function getTemplatesPageSpec(): InterfaceTemplatesPage | null {\n  return ADMITTED?.pages.templates ?? null;\n}\n"],"mappings":";;;;AAqCA,IAAM,IAAiB;CACrB,MAAM;CACN,OAAO;CACP,QAAQ;CACR,WAAW;AACb;AAUA,SAAS,EAAS,GAAkD;CAClE,OAAO,OAAO,KAAU,cAAY,KAAkB,CAAC,MAAM,QAAQ,CAAK;AAC5E;AAEA,SAAS,IAAqC;CAC5C,OAAO,IAAI,IACR,EAA0C,SAAS,MAClD,EAAS,CAAK,KAAK,OAAO,EAAM,MAAO,WAAW,CAAC,EAAM,EAAE,IAAI,CAAC,CAClE,CACF;AACF;AAEA,SAAS,EAAuB,GAA8C;CAC5E,IAAI,MAAc,KAAA,GAAW,OAAO;CAEpC,IAAM,IAAS,EAA0B,GAAW;EAClD,YAAY,EAAc;EAC1B,eAAe;CACjB,CAAC;CAUD,OATK,EAAO,QASL,KANL,QAAQ,KACN,+CACA,EAAO,OAAO,KAAK,IAAI,CACzB,GACO;AAGX;AAEA,IAAM,IAAW,EAAuB,CAA8B;AAMtE,SAAgB,IAAkC;CAChD,OAAO,MAAa;AACtB;AAOA,SAAS,IAAsC;CAC7C,IAAI,CAAC,GACH,MAAU,MACR,yFACF;CAEF,OAAO;AACT;AAEA,SAAS,EAAqB,GAAiB,GAAoB;CACjE,OAAO,EAAQ,QAAQ,iBAAiB,mBAAmB,CAAE,CAAC;AAChE;AAEA,SAAgB,IAA6B;CAC3C,OAAO,EAAe;AACxB;AAEA,SAAgB,EAAoB,GAAoB;CACtD,OAAO,EAAqB,EAAe,OAAO,CAAE;AACtD;AAeA,SAAgB,EAAmB,GAAuB;CACxD,OACE,MAAS,EAAe,QAAQ,EAAK,WAAW,GAAG,EAAe,KAAK,EAAE;AAE7E;AAOA,SAAgB,EAAsB,GAAqC;CACzE,OAAO,EAAiB,EAAE,UAAU,MAAS;AAC/C;AAGA,SAAgB,EACd,GACA,GACQ;CACR,OAAO,EAAsB,CAAI,EAAE,QAAQ,QAAQ,mBAAmB,CAAE,CAAC;AAC3E;AAaA,SAAgB,IAAkC;CAChD,IAAM,IAAW,EAAiB;CAClC,OAAO;EACL,cAAc,EAAS,WAAW,QAAQ;EAC1C,kBAAkB,EAAS,WAAW,YAAY;EAClD,wBAAwB,EAAS,WAAW,YAAY;EACxD,qBAAqB,EAAS,WAAW,YAAY;EACrD,WAAW,EAAS,MAAM,KAAK;EAC/B,cAAc,EAAS,MAAM,KAAK;EAClC,iBAAiB,EAAS,MAAM,OAAO;EACvC,WAAW,EAAS,MAAM,KAAK;CACjC;AACF;AAmCA,SAAgB,IAA6C;CAC3D,OAAO,EAAiB,EAAE;AAC5B;AAEA,SAAgB,IAAgC;CAC9C,OAAO,EAAiB,EAAE;AAC5B;AAEA,SAAgB,IAA8C;CAC5D,OAAO,EAAiB,EAAE;AAC5B;AAEA,SAAgB,IAAgD;CAC9D,OAAO,EAAiB,EAAE;AAC5B"}

package/dist/node_modules/@openhands/extensions/automations/bundle-index.cjs

@@ -42,6 +42,12 @@ REPOS = ["owner/repo"]
 TRIGGER_LABEL = "openhands-review"
 REVIEW_TONE = "thorough"
 REVIEW_STYLE_INSTRUCTIONS = ""
+# Path within the checked-out repository to a repo-specific review guide
+# (e.g. the repo's own code-review skill). When the file exists at this path
+# relative to the repo root, its contents are read and injected verbatim into
+# the review prompt so the guide is always applied deterministically, rather
+# than relying on the spawned agent's skill activation. Set to "" to disable.
+REPO_REVIEW_GUIDE_PATH = ".agents/skills/custom-codereview-guide.md"
 DEFAULT_OPENHANDS_URL = "http://localhost:8000"
 
 CONFIG_FILENAME = "config.json"
@@ -54,6 +60,7 @@ _CONFIG_TYPES: dict[str, type] = {
     "trigger_label": str,
     "review_tone": str,
     "review_style_instructions": str,
+    "repo_review_guide_path": str,
     "openhands_url": str,
 }
 
@@ -134,6 +141,7 @@ REPOS = _CONFIG.get("repos", REPOS)
 TRIGGER_LABEL = _CONFIG.get("trigger_label", TRIGGER_LABEL)
 REVIEW_TONE = _CONFIG.get("review_tone", REVIEW_TONE)
 REVIEW_STYLE_INSTRUCTIONS = _CONFIG.get("review_style_instructions", REVIEW_STYLE_INSTRUCTIONS)
+REPO_REVIEW_GUIDE_PATH = _CONFIG.get("repo_review_guide_path", REPO_REVIEW_GUIDE_PATH)
 DEFAULT_OPENHANDS_URL = _CONFIG.get("openhands_url", DEFAULT_OPENHANDS_URL)
 
 DONE_DEBOUNCE = 15
@@ -741,7 +749,28 @@ def _with_ai_disclosure(body: str) -> str:
     return f"{body}\\n\\n{disclosure}" if body else disclosure
 
 
-def _build_review_prompt(repo: str, pr: dict, head_sha: str, label_event: dict) -> str:
+def _load_repo_review_guide(workspace_dir: Path) -> str | None:
+    """Read the repo-specific review guide from the checked-out repository.
+
+    The path is taken from \`\`REPO_REVIEW_GUIDE_PATH\`\`. An empty path disables
+    the feature. Returns the file contents, or None if the file is absent or
+    unreadable — a missing guide is never fatal, the review simply proceeds
+    without it.
+    """
+    if not REPO_REVIEW_GUIDE_PATH:
+        return None
+    candidate = workspace_dir / REPO_REVIEW_GUIDE_PATH
+    try:
+        if candidate.is_file():
+            text = candidate.read_text(encoding="utf-8", errors="replace").strip()
+            if text:
+                return text
+    except Exception as exc:
+        print(f"  Warning: could not read repo review guide {candidate}: {exc}")
+    return None
+
+
+def _build_review_prompt(repo: str, pr: dict, head_sha: str, label_event: dict, repo_review_guide: str | None = None) -> str:
     number = pr.get("number", "?")
     title = pr.get("title", "(no title)")
     body = (pr.get("body") or "").strip() or "(no description)"
@@ -757,6 +786,10 @@ def _build_review_prompt(repo: str, pr: dict, head_sha: str, label_event: dict)
     deletions = pr.get("deletions", "?")
     tone = _TONE_INSTRUCTIONS.get(REVIEW_TONE, _TONE_INSTRUCTIONS["thorough"])
     extra = f"\\n\\nAdditional style instructions:\\n{REVIEW_STYLE_INSTRUCTIONS}" if REVIEW_STYLE_INSTRUCTIONS.strip() else ""
+    guide_section = (
+        f"\\n\\nRepo-specific review guide (from {REPO_REVIEW_GUIDE_PATH}):\\n---\\n{repo_review_guide}\\n---\\n"
+        if repo_review_guide else ""
+    )
 
     return (
         "You are an AI code reviewer. Review the GitHub pull request below and publish "
@@ -775,7 +808,12 @@ def _build_review_prompt(repo: str, pr: dict, head_sha: str, label_event: dict)
         "Required workflow:\\n"
         "1. The workspace is already the repository root at the exact Head SHA above. "
         "Do not clone, fetch, check out, or delete the repository.\\n"
-        "2. Inspect the PR discussion, existing review comments, changed files, and the diff, "
+        "2. Before reviewing, you MUST read the repository's own guidance to understand the repo first.\\n"
+        "   Read \`AGENTS.md\` at the repository root (and any nested \`AGENTS.md\` covering the "
+        "changed files), plus other relevant docs when present - e.g. \`CONTRIBUTING.md\`, "
+        "\`CLAUDE.md\`, \`.cursorrules\`, and any review or coding-guideline docs. Apply that "
+        "guidance to your review.\\n"
+        "   Then inspect the PR discussion, existing review comments, changed files, and the diff, "
         "together with the surrounding code in the workspace.\\n"
         "   Use \`gh\` or GitHub REST API calls with \`GITHUB_PERSONAL_ACCESS_TOKEN\`; never print secret values.\\n"
         "3. Ground every finding in the workspace code. Before using an inline location, verify that "
@@ -793,7 +831,7 @@ def _build_review_prompt(repo: str, pr: dict, head_sha: str, label_event: dict)
         "or \`🔄 CHANGES REQUESTED\`.\\n"
         "8. If there are no material issues, still publish a review saying so, with the "
         "disclosure and the verdict.\\n"
-        f"\\nReview instructions:\\n{tone}{extra}\\n\\n"
+        f"\\nReview instructions:\\n{tone}{extra}{guide_section}\\n\\n"
         "After GitHub accepts the review, output exactly \`GITHUB_REVIEW_POSTED\`. "
         "If publishing still fails after the fallback in step 5, output the complete review text "
         "so it can be posted as a comment instead."
@@ -841,7 +879,10 @@ def _process_review_request(
     workspace_dir = None
     try:
         workspace_dir = _prepare_repository(github_token, repo, number, head_sha)
-        prompt = _build_review_prompt(repo, pr, head_sha, label_event)
+        repo_review_guide = _load_repo_review_guide(workspace_dir)
+        if repo_review_guide:
+            print(f"  Injected repo review guide for PR #{number}")
+        prompt = _build_review_prompt(repo, pr, head_sha, label_event, repo_review_guide)
         conv_id = create_conversation(agent_url, api_key, prompt, workspace_dir)
     except Exception as exc:
         # The claim is dropped so the next poll retries this label event. The