@cursor/july 0.1.109 → 0.1.111

package/src/types.ts

@@ -155,7 +155,7 @@ export function toPublicCloudInfo(
     ...rest,
     ...(repos !== undefined
       ? {
-          repos: repos.map((repo) => ({
+          repos: repos.map(repo => ({
             ...repo,
             url: redactHttpUrlCredentials(repo.url),
             ...(repo.prUrl !== undefined
@@ -283,6 +283,19 @@ export interface AgentConfig {
    * you to select a specific architecture.
    */
   architecture?: AgentArchitecture;
+  /**
+   * Persistent hosting for `architecture: "v2"`.
+   *
+   * Default (`false`): each session boots a pod from the release snapshot;
+   * framework and author state are mirrored to Cursor stores because pod
+   * disks are disposable.
+   *
+   * `true`: one long-lived pod per deployment runs each deployed version as
+   * a container over one state directory on the pod's disk. Nothing is
+   * mirrored: that directory is the durable copy. A redeploy starts the new
+   * version beside the old one on the same state, then drains the old one.
+   */
+  persistent?: boolean;
 }
 
 /** Hosted architecture on {@link AgentConfig.architecture}. */
@@ -1141,8 +1154,10 @@ export interface ExtensionOptions<TConfig, TInput = TConfig> {
  * `agent/extensions/<ns>.ts` to mount the extension with a configuration;
  * the extension's own modules import it to read that configuration.
  */
-export interface Extension<TConfig = undefined, TInput = TConfig>
-  extends ExtensionDeclaration {
+export interface Extension<
+  TConfig = undefined,
+  TInput = TConfig,
+> extends ExtensionDeclaration {
   (
     ...args: undefined extends TInput ? [config?: TInput] : [config: TInput]
   ): ExtensionMount;
@@ -1309,7 +1324,7 @@ export function normalizeCursorAccountServers(
   if (servers === undefined || servers === "*") {
     return undefined;
   }
-  const trimmed = servers.map((server) => server.trim());
+  const trimmed = servers.map(server => server.trim());
   if (trimmed.length === 1 && trimmed[0] === "*") {
     return undefined;
   }
@@ -1323,8 +1338,8 @@ export function cursorAccountServersHasMixedWildcard(
   if (servers === undefined || servers === "*") {
     return false;
   }
-  const trimmed = servers.map((server) => server.trim());
-  return trimmed.includes("*") && trimmed.some((server) => server !== "*");
+  const trimmed = servers.map(server => server.trim());
+  return trimmed.includes("*") && trimmed.some(server => server !== "*");
 }
 
 /**
@@ -2278,6 +2293,13 @@ export interface ChannelHandlerArgs {
    * pass `sessionId` in `tag` input to attribute one).
    */
   artifacts: ArtifactsApi;
+  /** Hosted Slack admission uses this to paint a warm follow-up. */
+  hasContinuationSession?(continuationToken: string): Promise<boolean>;
+  isContinuationBusy?(continuationToken: string): Promise<boolean>;
+  getContinuationLastBotMessageTs?(
+    continuationToken: string
+  ): Promise<string | undefined>;
+  interruptContinuation?(continuationToken: string): Promise<boolean>;
 }
 
 export type ChannelRouteHandler = (
@@ -2509,9 +2531,9 @@ export interface ChannelStartArgs {
 }
 
 /**
- * Descriptive metadata a channel pack attaches for tooling. Read by
- * discovery / `projectInfo` and CLI helpers (e.g. `agent-sdk github
- * forward`) — it never affects routing or dispatch.
+ * Metadata a channel pack attaches for discovery, `projectInfo`, and CLI
+ * helpers (e.g. `agent-sdk github forward`). Most fields are descriptive;
+ * explicitly documented protocol fields may also be consumed by the host.
  */
 export interface ChannelMeta {
   /** Well-known pack kind, e.g. `"github"` or `"slack"`. */
@@ -2556,6 +2578,16 @@ export interface ChannelMeta {
    * whatever the app manifest subscribed to).
    */
   slackEventTypes?: readonly string[];
+  /**
+   * Static agentic-reply protocol for hosted Slack ingress. Only literals
+   * belong here: the control plane persists this declaration at deploy time
+   * and may act before author code starts.
+   */
+  slackReply?: {
+    mode: "agentic";
+    ack: string;
+    placeholder?: string;
+  };
   /**
    * For Slack packs with the opt-in channel watch: the allowlisted channels
    * (ids or `#names`) this channel dispatches on without a mention. Surfaced
@@ -2622,10 +2654,33 @@ export interface HttpChannelConfig {
     message: string,
     info: { auth: AuthContext | null }
   ) => string | Promise<string>;
+  /**
+   * Bind host state after the control plane names this session's mailbox
+   * key and before `send()`.
+   */
+  onAdmit?: (info: HttpSessionAdmitInfo) => void | Promise<void>;
   /** Stream-event handlers for sessions created over HTTP. */
   events?: ChannelEventHandlers;
 }
 
+/** Subject of a hosted HTTP startSession, seen by {@link HttpChannelConfig.onAdmit}. */
+export type HttpSessionAdmitInfo = {
+  message: string;
+  sessionKey: string;
+  dryRun?: boolean;
+  asOf?: string;
+  host: HostContext;
+};
+
+/** Reject a hosted startSession subject from {@link HttpChannelConfig.onAdmit}. */
+export class HttpSessionAdmitError extends Error {
+  readonly status = 400 as const;
+  constructor(message: string) {
+    super(message);
+    this.name = "HttpSessionAdmitError";
+  }
+}
+
 export type HttpChannelDefinition = HttpChannelConfig &
   BrandedDefinition<"http-channel">;
 
@@ -2727,16 +2782,24 @@ export interface ResultContext {
   workspaceDir: string;
   /** Full assistant text for the turn. */
   text: string;
+  /**
+   * The commit did not apply; re-run this turn with `message` so the
+   * model can revise. After two repairs the turn fails like a throw.
+   */
+  reject(message: string): never;
 }
 
 /**
  * Input to `defineResult` (authored at `agent/result.ts`).
- * Parse `ctx.text` in `commit`. Throw to fail the turn.
+ * Parse `ctx.text` in `commit`. Throw to fail the turn. Call
+ * {@link ResultContext.reject} for a model-correctable miss so the
+ * same turn can revise before `turn.completed`.
  */
 export interface ResultConfig {
   /**
    * Host actuation after the model finishes, before `turn.completed`.
-   * Throw to fail the turn. A dry-run session skips this callback.
+   * Throw to fail the turn. `ctx.reject(message)` re-prompts the model
+   * in this turn (two repairs). A dry-run session skips this callback.
    */
   commit: (ctx: ResultContext) => void | Promise<void>;
 }
@@ -3256,6 +3319,7 @@ export interface ResolvedAgent {
    * continuation defaults onto channels that omitted a selector.
    */
   architecture?: AgentArchitecture;
+  persistent?: boolean;
   tools: DiscoveredTool[];
   skills: DiscoveredSkill[];
   /**
@@ -3281,7 +3345,7 @@ export interface AgentProject {
   agentDir: string;
   name: string;
   agent: ResolvedAgent;
-  /** Built-in HTTP channel config (authored override or default). */
+  /** Built-in HTTP session channel. */
   httpChannel: HttpChannelDefinition;
   channels: DiscoveredChannel[];
   schedules: DiscoveredSchedule[];
@@ -3461,11 +3525,12 @@ export interface RunnerTurnRequest {
   /** Cloud options when {@link runtime} is `"cloud"`. */
   cloud?: AgentCloudOptions;
   /**
-   * Composed instructions ({@link ResolvedAgent.instructions}) for runtimes
-   * whose harness cannot read the workspace `AGENTS.md` — grokbot turns
-   * carry them into the hosted session so they reach the Sand system
-   * prompt. Local turns read the materialized workspace instead and leave
-   * this unset.
+   * Composed instructions ({@link ResolvedAgent.instructions} plus identity
+   * and tool catalog) delivered as the harness system prompt. Local turns
+   * pass them to the Cursor SDK as `systemPrompt`, replacing Cursor's
+   * built-in prompt; grokbot turns carry them into the hosted session so
+   * they reach the Sand system prompt. Cloud turns leave this unset and
+   * inline the same text into the first prompt instead.
    */
   instructions?: string;
   /**

package/templates/agentic-owners/agent/channels/github.ts

@@ -1,7 +1,4 @@
-import {
-  defaultGitHubAuth,
-  githubChannel,
-} from "@cursor/july/channels/github";
+import { defaultGitHubAuth, githubChannel } from "@cursor/july/channels/github";
 import config from "../lib/config.js";
 import { githubOwnersForge } from "../lib/github.js";
 import { AgenticOwnersConfigValidationError } from "../lib/owners/parse.js";
@@ -27,8 +24,8 @@ export default githubChannel({
   cursorAccount: config.repos.length > 0 ? { repos: config.repos } : undefined,
   // The model never writes to GitHub; record_review actuates host-side.
   deliverReplies: false,
-  onStart: (args) => {
-    log = (line) => args.logger(`[agentic-owners] ${line}`);
+  onStart: args => {
+    log = line => args.logger(`[agentic-owners] ${line}`);
   },
   onPullRequest: async (ctx, pr) => {
     // A base retarget re-resolves policy against the new base. The fresh

package/templates/agentic-owners/agent/lib/github.ts

@@ -90,10 +90,10 @@ export function githubOwnersForge(
         pull_number: ref.number,
       });
       const requestedUsers = (data.requested_reviewers ?? []).flatMap(
-        (reviewer) => ("login" in reviewer ? [reviewer.login] : [])
+        reviewer => ("login" in reviewer ? [reviewer.login] : [])
       );
       const requestedTeams = (data.requested_teams ?? []).map(
-        (team) => `${ref.owner}/${team.slug}`
+        team => `${ref.owner}/${team.slug}`
       );
       return {
         number: data.number,
@@ -114,7 +114,7 @@ export function githubOwnersForge(
         pull_number: ref.number,
         per_page: 100,
       });
-      return files.map((file) => ({
+      return files.map(file => ({
         filename: file.filename,
         status: file.status,
         previousFilename: file.previous_filename,
@@ -201,7 +201,7 @@ export function githubOwnersForge(
         pull_number: ref.number,
         per_page: 100,
       });
-      return reviews.map((review) => ({
+      return reviews.map(review => ({
         id: review.id,
         state: review.state,
         body: review.body ?? "",
@@ -214,7 +214,7 @@ export function githubOwnersForge(
         octokit.rest.issues.listComments,
         { ...repo, issue_number: ref.number, per_page: 100 }
       );
-      return comments.map((comment) => ({
+      return comments.map(comment => ({
         id: comment.id,
         body: comment.body ?? "",
       }));

package/templates/agentic-owners/agent/lib/owners/decision.ts

@@ -144,7 +144,7 @@ export function authorOwnsRule(
     return false;
   }
   const handle = normalizeHandle(author.handle);
-  return rule.users.some((user) => normalizeHandle(user) === handle);
+  return rule.users.some(user => normalizeHandle(user) === handle);
 }
 
 /**
@@ -157,7 +157,7 @@ function selfReviewRuleForAuthor(
   author: ApprovalAuthor | undefined
 ): AgenticOwnersPathRule | undefined {
   return matchedRules.find(
-    (rule) => rule.selfReview && authorOwnsRule(rule, author)
+    rule => rule.selfReview && authorOwnsRule(rule, author)
   );
 }
 
@@ -185,12 +185,12 @@ export function authorSelfReviewCoversMatchedRules(
   author: ApprovalAuthor | undefined
 ): boolean {
   const reviewCapableRules = matchedRules.filter(
-    (rule) => !rule.alwaysApprove && !ruleIsNotifyOnly(rule)
+    rule => !rule.alwaysApprove && !ruleIsNotifyOnly(rule)
   );
   return (
     reviewCapableRules.length > 0 &&
     reviewCapableRules.every(
-      (rule) => rule.selfReview && authorOwnsRule(rule, author)
+      rule => rule.selfReview && authorOwnsRule(rule, author)
     )
   );
 }
@@ -248,11 +248,11 @@ function reviewersToRequest({
 function reasonForNoBlockingOwners(
   matchedRules: readonly AgenticOwnersPathRule[]
 ): string {
-  const anyNotifyOnly = matchedRules.some((rule) => ruleIsNotifyOnly(rule));
+  const anyNotifyOnly = matchedRules.some(rule => ruleIsNotifyOnly(rule));
   if (!anyNotifyOnly) {
     return "all matched policies set always_approve";
   }
-  const anyAlwaysApprove = matchedRules.some((rule) => rule.alwaysApprove);
+  const anyAlwaysApprove = matchedRules.some(rule => rule.alwaysApprove);
   return anyAlwaysApprove
     ? "matched policies are notify-only or always_approve (no blocking owners)"
     : "all matched policies are notify-only (no blocking owners)";
@@ -286,10 +286,10 @@ export function decideApproval(input: {
 }): ApprovalDecision {
   const { matchedRules, riskLevel, author } = input;
   const reviewRequiredRules = matchedRules.filter(
-    (rule) => !rule.alwaysApprove && !ruleIsNotifyOnly(rule)
+    rule => !rule.alwaysApprove && !ruleIsNotifyOnly(rule)
   );
   const riskBucket = approvalRiskBucket(riskLevel);
-  const humanReviewRequired = reviewRequiredRules.some((rule) =>
+  const humanReviewRequired = reviewRequiredRules.some(rule =>
     ruleIsHardGate(rule)
   );
   const requestReviewers = reviewersToRequest({
@@ -367,7 +367,10 @@ export function decideApproval(input: {
   const selfReviewClearsEvidence =
     authorSelfReviewCoversMatchedRules(matchedRules, author) &&
     !evidence.diffEmpty;
-  if ((evidence.diffEmpty || evidence.diffTruncated) && !selfReviewClearsEvidence) {
+  if (
+    (evidence.diffEmpty || evidence.diffTruncated) &&
+    !selfReviewClearsEvidence
+  ) {
     return hold(
       evidence.diffEmpty
         ? "no usable patch hunks — auto-approve disabled (fail closed)"
@@ -408,7 +411,7 @@ function baseApprovalVerdict(input: {
   // owner self_review on that same rule. Low-risk auto-approve never clears a
   // hard gate without self_review.
   const blockingHardGate = reviewRequiredRules.find(
-    (rule) =>
+    rule =>
       ruleIsHardGate(rule) && !(rule.selfReview && authorOwnsRule(rule, author))
   );
   if (blockingHardGate !== undefined) {

package/templates/agentic-owners/agent/lib/owners/handles.ts

@@ -28,8 +28,8 @@ export function sanitizeReviewerHandle(raw: string): string | undefined {
       .replace(/\.+$/, "")
       .toLowerCase()
       .split("/")
-      .map((segment) => segment.trim())
-      .filter((segment) => segment !== "");
+      .map(segment => segment.trim())
+      .filter(segment => segment !== "");
     // A GitHub team handle is always exactly `org/team`. Collapse a duplicated
     // leading org (the shape a double-prefix bug produces) before validating.
     while (segments.length > 2 && segments[0] === segments[1]) {

package/templates/agentic-owners/agent/lib/owners/match.ts

@@ -78,7 +78,7 @@ function ruleMatchesFile({
   rule: PathScopedRule;
   filepath: string;
 }): boolean {
-  return rule.paths.some((pattern) =>
+  return rule.paths.some(pattern =>
     patternMatchesFileFromPolicy({
       rule,
       pattern,
@@ -192,7 +192,7 @@ export function changedFilesWithoutWinningRule({
   rules: readonly AgenticOwnersPathRule[];
 }): string[] {
   return changedFiles.filter(
-    (filepath) => pickWinningRuleForFile({ filepath, rules }) === undefined
+    filepath => pickWinningRuleForFile({ filepath, rules }) === undefined
   );
 }
 
@@ -215,7 +215,7 @@ export function everyChangedFileGloballyExcluded({
     return false;
   }
   return changedFiles.every(
-    (filepath) =>
+    filepath =>
       pickWinningRuleForFile({ filepath, rules })?.globalExclusion === true
   );
 }

package/templates/agentic-owners/agent/lib/owners/matching.ts

@@ -71,7 +71,7 @@ export function patternSpecificity(pattern: string): number {
     globIndex === -1 ? normalized : normalized.slice(0, globIndex);
   const segments = literalPrefix
     .split("/")
-    .filter((segment) => segment !== "").length;
+    .filter(segment => segment !== "").length;
   return literalPrefix.length * 100 + segments;
 }
 

package/templates/agentic-owners/agent/lib/owners/parse.ts

@@ -106,16 +106,11 @@ function parsePaths(
     issues.push(`${label}: missing paths`);
     return undefined;
   }
-  if (
-    !Array.isArray(value) ||
-    value.some((entry) => typeof entry !== "string")
-  ) {
+  if (!Array.isArray(value) || value.some(entry => typeof entry !== "string")) {
     issues.push(`${label}: paths must be a list of strings`);
     return undefined;
   }
-  const paths = value
-    .map((entry) => entry.trim())
-    .filter((entry) => entry !== "");
+  const paths = value.map(entry => entry.trim()).filter(entry => entry !== "");
   if (paths.length === 0) {
     issues.push(`${label}: paths must not be empty`);
     return undefined;
@@ -286,7 +281,7 @@ function parseOwnerList({
   if (plural !== undefined) {
     if (
       !Array.isArray(plural) ||
-      plural.some((entry) => typeof entry !== "string")
+      plural.some(entry => typeof entry !== "string")
     ) {
       issues.push(`${label}.${field}: ${field} must be a list of strings`);
     } else {
@@ -352,7 +347,7 @@ function scopeDepthForConfigPath(configPath: string): number {
   if (prefix === "") {
     return 0;
   }
-  return prefix.split("/").filter((segment) => segment !== "").length;
+  return prefix.split("/").filter(segment => segment !== "").length;
 }
 
 function validatePathsStayWithinRepository({

package/templates/agentic-owners/agent/lib/owners/prompt.ts

@@ -36,7 +36,7 @@ export function composeReviewPrompt(args: {
   rulePrompts: readonly string[];
 }): string {
   const parts = [args.basePrompt, ...args.rulePrompts].filter(
-    (part) => part !== ""
+    part => part !== ""
   );
   const composed = parts.join("\n\n");
   if (composed.length > AGENTIC_OWNERS_MAX_COMPOSED_PROMPT_CHARS) {

package/templates/agentic-owners/agent/lib/review.ts

@@ -42,9 +42,7 @@ export interface ReviewVerdict {
   recommendedReviewers?: readonly string[];
 }
 
-export type ResolvedReview =
-  | { enrolled: false; reason: string }
-  | ReviewState;
+export type ResolvedReview = { enrolled: false; reason: string } | ReviewState;
 
 export interface ReviewState {
   enrolled: true;
@@ -137,8 +135,8 @@ function buildDiff(files: readonly ChangedFile[]): {
 } {
   const usablePatch = (file: ChangedFile): boolean =>
     file.patch !== undefined && file.patch.trim() !== "";
-  const noPatches = files.every((file) => !usablePatch(file));
-  const anyMissingPatch = files.some((file) => !usablePatch(file));
+  const noPatches = files.every(file => !usablePatch(file));
+  const anyMissingPatch = files.some(file => !usablePatch(file));
   const parts: string[] = [];
   let used = 0;
   for (const file of files) {
@@ -179,17 +177,17 @@ export async function loadReviewState(
   // Renames count both sides for matching; runtime-generated
   // `.agentic-owners/generated/` paths never participate.
   const reviewable = changed.filter(
-    (file) => !isGeneratedAgenticOwnersPath(file.filename)
+    file => !isGeneratedAgenticOwnersPath(file.filename)
   );
   const names = [
     ...new Set(
-      reviewable.flatMap((file) =>
+      reviewable.flatMap(file =>
         file.previousFilename === undefined
           ? [file.filename]
           : [file.filename, file.previousFilename]
       )
     ),
-  ].filter((path) => !isGeneratedAgenticOwnersPath(path));
+  ].filter(path => !isGeneratedAgenticOwnersPath(path));
   if (names.length === 0 && !filesTruncated) {
     return { enrolled: false, reason: "no reviewable files" };
   }
@@ -207,7 +205,10 @@ export async function loadReviewState(
     !filesTruncated &&
     everyChangedFileGloballyExcluded({ changedFiles: names, rules })
   ) {
-    return { enrolled: false, reason: "every changed file is globally excluded" };
+    return {
+      enrolled: false,
+      reason: "every changed file is globally excluded",
+    };
   }
 
   const matched = matchRepoConfigForChangedFiles({
@@ -254,9 +255,7 @@ export async function loadReviewState(
     return directory === "" ? rule.prompt : `${directory}/${rule.prompt}`;
   };
   const promptPaths = [
-    ...new Set(
-      committed.flatMap((rule) => rulePromptPath(rule) ?? [])
-    ),
+    ...new Set(committed.flatMap(rule => rulePromptPath(rule) ?? [])),
   ];
   const promptContents =
     promptPaths.length > 0
@@ -272,7 +271,9 @@ export async function loadReviewState(
     composedPaths.add(promptPath);
     const contents = promptContents.get(promptPath)?.trim();
     if (contents === undefined || contents === "") {
-      log(`prompt file missing at base ref: ${promptPath} (from ${rule.sourcePath})`);
+      log(
+        `prompt file missing at base ref: ${promptPath} (from ${rule.sourcePath})`
+      );
       continue;
     }
     rulePrompts.push(contents);
@@ -307,12 +308,12 @@ export function renderReviewWake(state: ReviewState): string {
   const rulesBlock =
     state.matchedRules.length > 0
       ? state.matchedRules
-          .map((rule) => `- ${describeRuleForContext(rule)}`)
+          .map(rule => `- ${describeRuleForContext(rule)}`)
           .join("\n")
       : "- (none — default risk review applies)";
   const candidatesBlock =
     state.candidates.length > 0
-      ? state.candidates.map((handle) => `- ${handle}`).join("\n")
+      ? state.candidates.map(handle => `- ${handle}`).join("\n")
       : "- (none — recommend no reviewers rather than guessing handles)";
   const existing =
     state.pr.requestedReviewers.length > 0
@@ -324,7 +325,7 @@ export function renderReviewWake(state: ReviewState): string {
   const fence = "`".repeat(
     Math.max(
       4,
-      ...[...state.diff.matchAll(/`+/g)].map((run) => run[0].length + 1)
+      ...[...state.diff.matchAll(/`+/g)].map(run => run[0].length + 1)
     )
   );
 
@@ -344,7 +345,7 @@ it.
 ${state.pr.baseRef}
 
 ## Changed files (${state.reviewableFiles.length})
-${state.reviewableFiles.map((file) => `- ${displayPath(file)}`).join("\n")}
+${state.reviewableFiles.map(file => `- ${displayPath(file)}`).join("\n")}
 
 ## Matched policy rules
 ${rulesBlock}
@@ -483,7 +484,9 @@ function commentBody(input: {
 }): string {
   const policies =
     input.matchedRules.length > 0
-      ? [...new Set(input.matchedRules.map((rule) => `\`${rule.sourcePath}\``))].join(", ")
+      ? [
+          ...new Set(input.matchedRules.map(rule => `\`${rule.sourcePath}\``)),
+        ].join(", ")
       : "none — default risk review";
   const reviewers =
     input.requested.length > 0
@@ -532,7 +535,7 @@ async function upsertMarkerComment(
   body: string,
   createIfMissing: boolean
 ): Promise<void> {
-  const existing = (await forge.listComments()).find((comment) =>
+  const existing = (await forge.listComments()).find(comment =>
     comment.body.startsWith(REVIEW_MARKER)
   );
   if (existing !== undefined) {
@@ -555,7 +558,7 @@ export async function dismissStaleApprovals(
   message: string
 ): Promise<void> {
   const stale = (await forge.listReviews()).filter(
-    (review) =>
+    review =>
       review.state === "APPROVED" &&
       review.body.includes(APPROVED_HEAD_MARKER_PREFIX)
   );
@@ -573,7 +576,7 @@ export async function reportPolicyError(
     REVIEW_MARKER,
     "**Agentic Owners** — policy validation failed; not reviewing until it parses.",
     "",
-    ...error.issues.map((issue) => `- ${issue}`),
+    ...error.issues.map(issue => `- ${issue}`),
   ].join("\n");
   await upsertMarkerComment(forge, body, true);
   await dismissStaleApprovals(
@@ -627,7 +630,8 @@ export async function actuateReview(input: {
   if (input.assessedHeadSha !== pr.headSha) {
     return {
       status: "skipped",
-      reason: "the head moved since the assessment; the new head gets its own review",
+      reason:
+        "the head moved since the assessment; the new head gets its own review",
     };
   }
 
@@ -637,7 +641,7 @@ export async function actuateReview(input: {
     recommended: verdict.recommendedReviewers ?? [],
     candidates: state.candidates,
   }).filter(
-    (handle) => normalizeHandle(handle) !== normalizeHandle(pr.authorLogin ?? "")
+    handle => normalizeHandle(handle) !== normalizeHandle(pr.authorLogin ?? "")
   );
   const decision = decideApproval({
     matchedRules: state.matchedRules,
@@ -645,7 +649,8 @@ export async function actuateReview(input: {
     author: { handle: pr.authorLogin },
     // Owners gate deterministically: when the model recommends nobody
     // usable, fall back to the policy-derived candidates.
-    recommendedReviewers: recommended.length > 0 ? recommended : state.candidates,
+    recommendedReviewers:
+      recommended.length > 0 ? recommended : state.candidates,
     evidence: {
       diffEmpty: state.diffEmpty,
       diffTruncated: state.diffTruncated,
@@ -656,7 +661,7 @@ export async function actuateReview(input: {
 
   if (decision.approve) {
     const alreadyApproved = (await forge.listReviews()).some(
-      (review) =>
+      review =>
         review.state === "APPROVED" &&
         review.body.includes(approvedHeadMarker(pr.headSha))
     );
@@ -665,7 +670,11 @@ export async function actuateReview(input: {
         // Pin the review to the assessed head: a push racing this write gets
         // a stale approval on the old commit, never a fresh one on unseen code.
         await forge.submitApproval({
-          body: reviewBody({ verdict, reason: decision.reason, headSha: pr.headSha }),
+          body: reviewBody({
+            verdict,
+            reason: decision.reason,
+            headSha: pr.headSha,
+          }),
           commitId: pr.headSha,
         });
       } catch (error) {
@@ -727,11 +736,9 @@ export async function actuateReview(input: {
     `Superseded by a fresh Agentic Owners decision: ${decision.reason}.`
   );
 
-  const alreadyRequested = new Set(
-    pr.requestedReviewers.map(normalizeHandle)
-  );
+  const alreadyRequested = new Set(pr.requestedReviewers.map(normalizeHandle));
   const newReviewers = decision.requestReviewers
-    .filter((handle) => !alreadyRequested.has(normalizeHandle(handle)))
+    .filter(handle => !alreadyRequested.has(normalizeHandle(handle)))
     .slice(
       0,
       Math.max(0, MAX_REQUESTED_REVIEWERS - pr.requestedReviewers.length)

package/templates/agentic-owners/evals/owners.eval.ts

@@ -31,7 +31,7 @@ function wake(input: {
       changedFilesCount: input.files.length,
       requestedReviewers: [],
     },
-    reviewableFiles: input.files.map((file) => file.filename),
+    reviewableFiles: input.files.map(file => file.filename),
     matchedRules: rules,
     composedPrompt: composeReviewPrompt({
       basePrompt: readBaseReviewPrompt(),
@@ -39,7 +39,7 @@ function wake(input: {
     }),
     candidates: input.candidates ?? [],
     diff: input.files
-      .map((file) => `--- ${file.filename} (modified)\n${file.patch}`)
+      .map(file => `--- ${file.filename} (modified)\n${file.patch}`)
       .join("\n\n"),
     diffTruncated: false,
     diffEmpty: false,

package/templates/agentic-owners/package.json

@@ -15,6 +15,6 @@
   },
   "devDependencies": {
     "@types/node": "^22.14.0",
-    "typescript": "^5.8.0"
+    "typescript": "6.0.3"
   }
 }