@vypdev/copilot 3.3.0 → 3.5.0

package/build/api/src/api.d.ts

@@ -1,35 +1,81 @@
-import type { Execution } from './data/model/execution';
+import type { Result } from './data/model/result';
+import type { AgentConfiguration } from './domain/agent';
 import type { FindingsQueryPort } from './application/ports/agent_findings_ports';
-import type { BugbotContextPorts } from './application/ports/bugbot_context_ports';
-import type { BugbotFindingPublicationPorts } from './application/ports/bugbot_finding_publication_ports';
-import type { BugbotFindingResolutionPorts } from './application/ports/bugbot_finding_resolution_ports';
+import type { BugbotScmPorts } from './application/ports/bugbot_scm_ports';
 import type { BugbotTelemetryPort } from './application/ports/bugbot_telemetry_ports';
-import type { BugbotReviewCommandOverrides } from './domain/bugbot/review_command';
-export interface BugbotScmGateway {
-    readonly context: BugbotContextPorts;
-    readonly publication: BugbotFindingPublicationPorts;
-    readonly resolution: BugbotFindingResolutionPorts;
+import type { BugbotReviewConfiguration } from './domain/bugbot/review_configuration';
+import type { MessageCatalogResolutionPort } from './application/ports/message_catalog_ports';
+export type { MessageCatalogResolutionPort } from './application/ports/message_catalog_ports';
+/** Already-bound SCM authority for exactly one repository. */
+export interface BugbotScmGateway extends BugbotScmPorts {
+    readonly repository: {
+        readonly owner: string;
+        readonly name: string;
+    };
     readonly telemetry?: BugbotTelemetryPort;
 }
+export type BugbotMinimumSeverity = 'info' | 'low' | 'medium' | 'high';
+export type BugbotReviewTarget = {
+    readonly kind: 'pull-request';
+    readonly number: number;
+    readonly head: string;
+    readonly base?: string;
+    readonly linkedIssueNumber?: number;
+    readonly action?: 'opened' | 'reopened' | 'synchronize';
+    readonly expectedHeadSha?: string;
+    readonly before?: string;
+    readonly draft?: boolean;
+} | {
+    readonly kind: 'branch';
+    readonly branch: string;
+    readonly base?: string;
+    readonly issueNumber?: number;
+    readonly before?: string;
+    readonly after?: string;
+};
+/** Sole supported request for the programmatic Bugbot review entry point. */
+export interface BugbotReviewRequest {
+    readonly target: BugbotReviewTarget;
+    readonly agent: AgentConfiguration;
+    readonly configuration?: Partial<BugbotReviewConfiguration>;
+    readonly ignoreFiles?: readonly string[];
+    readonly minimumSeverity?: BugbotMinimumSeverity;
+    readonly commentLimit?: number;
+    readonly authenticatedUser?: string;
+    readonly locale?: {
+        readonly issue?: string;
+        readonly pullRequest?: string;
+    };
+}
 /** Provider-neutral programmatic entry point. Consumers supply agent and SCM adapters. */
 export declare class BugbotReviewService {
     private readonly useCase;
-    constructor(agent: FindingsQueryPort, scm: BugbotScmGateway);
-    review(execution: Execution, options?: BugbotReviewCommandOverrides): Promise<import("./data/model/result").Result[]>;
+    private readonly repository;
+    constructor(agent: FindingsQueryPort, scm: BugbotScmGateway, catalogResolver?: MessageCatalogResolutionPort);
+    review(request: BugbotReviewRequest): Promise<readonly Result[]>;
 }
 export { evaluateBugbotFindings, evaluateBugbotQualityGate } from './tooling/bugbot_quality_eval';
 export { evaluateBugbotBenchmark, loadBugbotBenchmark, loadBugbotPredictions } from './tooling/bugbot_benchmark';
 export { buildBugbotAnalytics, parseBugbotTelemetry } from './tooling/bugbot_analytics';
 export { buildSemanticFindingFingerprint, buildFindingFingerprint } from './domain/bugbot/finding_identity';
 export { normalizeBugbotReviewConfiguration, resolveBugbotReviewEffort } from './domain/bugbot/review_configuration';
-export { Execution } from './data/model/execution';
-export { Ai } from './data/model/ai';
+export { BUGBOT_FINDING_STATES, classifyBugbotFindingState, countActionableBugbotFindings, countBugbotFindingStates, isBugbotActionableState, isBugbotCleanState, } from './domain/bugbot/review_state';
+export { buildBugbotReviewProjection } from './domain/bugbot/review_projection';
+export { ApplicationError } from './application/errors/application_error';
+export type { ApplicationErrorCode, ApplicationErrorKind, ApplicationErrorPublicRecord, } from './application/errors/application_error';
+export type { AgentConfiguration } from './domain/agent';
 export type { FindingsQueryPort } from './application/ports/agent_findings_ports';
 export type { BugbotContextPorts } from './application/ports/bugbot_context_ports';
 export type { BugbotFindingPublicationPorts } from './application/ports/bugbot_finding_publication_ports';
 export type { BugbotFindingResolutionPorts } from './application/ports/bugbot_finding_resolution_ports';
+export type { BugbotScmPorts } from './application/ports/bugbot_scm_ports';
+export type { BugbotPresentationMutationPorts, BugbotReconciliationSnapshotPorts, } from './application/ports/bugbot_reconciliation_ports';
 export type { BugbotTelemetryPort } from './application/ports/bugbot_telemetry_ports';
+export type { BugbotReviewNavigation } from './application/ports/bugbot_review_navigation_ports';
 export type { Result } from './data/model/result';
-export type { BugbotFinding } from './application/usecases/steps/commit/bugbot/types';
+export type { BugbotFinding } from './domain/bugbot/finding';
 export type { BugbotReviewConfiguration } from './domain/bugbot/review_configuration';
 export type { BugbotReviewTelemetrySnapshot } from './application/ports/bugbot_telemetry_ports';
+export type { BugbotFindingState, BugbotFindingStateCounts, BugbotFindingEvidence, BugbotResolvedFindingState, } from './domain/bugbot/review_state';
+export type { BugbotProjectedFinding, BugbotProjectionOutcome, BugbotReviewProjection, } from './domain/bugbot/review_projection';
+export type { PullRequestReviewReference, PullRequestReviewSummary, } from './application/ports/pull_request_review_comment_ports';

package/build/api/src/application/errors/application_error.d.ts

@@ -0,0 +1,10 @@
+import { ApplicationError as SemanticApplicationError, type ApplicationErrorCode, type SemanticApplicationErrorOptions } from '../../data/model/application_error';
+export { APPLICATION_ERROR_METADATA, APPLICATION_ERROR_RECOVERY_IDS, type ApplicationErrorCode, type ApplicationErrorKind, type ApplicationErrorPublicRecord, type ApplicationErrorRecovery, type ApplicationErrorRecoveryId, } from '../../data/model/application_error';
+export interface ApplicationErrorOptions extends Omit<SemanticApplicationErrorOptions, 'correlationId'> {
+    readonly correlationId?: string;
+}
+/** Creates a semantic error and owns correlation identity outside the pure model. */
+export declare class ApplicationError extends SemanticApplicationError {
+    constructor(code: ApplicationErrorCode, message: string, options?: ApplicationErrorOptions);
+}
+export declare function toApplicationError(error: unknown, code: ApplicationErrorCode, message: string, options?: Omit<ApplicationErrorOptions, 'cause'>): ApplicationError;

package/build/api/src/application/ports/bugbot_context_ports.d.ts

@@ -1,9 +1,22 @@
-import type { BugbotIssueReadPort } from './bugbot_issue_read_ports';
-import type { BugbotPullRequestReadPort } from './bugbot_pull_request_read_ports';
 import type { BugbotRuleFileQueryPort } from './bugbot_rule_ports';
-export interface BugbotContextPorts {
-    issue: BugbotIssueReadPort;
-    pullRequest: BugbotPullRequestReadPort;
-    /** Optional for compatibility with embedders that do not expose a workspace. */
-    rules?: BugbotRuleFileQueryPort;
+import type { BugbotPullRequestIdentity, BugbotSourceCoverage } from '../../domain/bugbot/context';
+import type { PullRequestReviewDiffSnapshot } from './bugbot_pull_request_read_ports';
+import type { BugbotIssueComment } from './bugbot_issue_read_ports';
+import type { PullRequestReviewComment, PullRequestReviewThreadState } from './pull_request_review_comment_ports';
+export interface BoundedBugbotRead<T> {
+    readonly value: T;
+    readonly coverage: BugbotSourceCoverage;
 }
+export interface BoundBugbotContextReadPorts {
+    getPullRequest(pullRequestNumber: number): Promise<BugbotPullRequestIdentity>;
+    findOpenPullRequestsByExactHead(headOwner: string, headRef: string): Promise<readonly BugbotPullRequestIdentity[]>;
+    listIssueComments(issueNumber: number): Promise<BoundedBugbotRead<readonly BugbotIssueComment[]>>;
+    listPullRequestReviewComments(pullRequestNumber: number): Promise<BoundedBugbotRead<readonly PullRequestReviewComment[]>>;
+    listPullRequestReviewThreadStates(pullRequestNumber: number): Promise<BoundedBugbotRead<Readonly<Record<string, PullRequestReviewThreadState>>>>;
+    getReviewDiffSnapshot(pullRequestNumber: number): Promise<BoundedBugbotRead<PullRequestReviewDiffSnapshot>>;
+    getPullRequestHeadSha(pullRequestNumber: number): Promise<string | undefined>;
+    getPullRequestReviewCommentBody(pullRequestNumber: number, commentId: number): Promise<string | null>;
+    loadRules(paths: readonly string[], revision?: string): ReturnType<BugbotRuleFileQueryPort['loadRules']>;
+}
+/** Repository-bound, credential-free context reads exposed to Bugbot use cases. */
+export type BugbotContextPorts = BoundBugbotContextReadPorts;

package/build/api/src/application/ports/bugbot_finding_publication_ports.d.ts

@@ -1,7 +1,7 @@
-import type { BugbotIssueCommentWritePort } from "./bugbot_issue_write_ports";
-import type { BugbotPullRequestWritePort } from "./bugbot_pull_request_write_ports";
+import type { BoundBugbotIssueCommentWritePort } from "./bugbot_issue_write_ports";
+import type { BoundBugbotPullRequestWritePort } from "./bugbot_pull_request_write_ports";
 /** Minimum capabilities needed to publish or refresh findings. */
 export interface BugbotFindingPublicationPorts {
-    issueComments: BugbotIssueCommentWritePort;
-    pullRequestComments: BugbotPullRequestWritePort;
+    issueComments: BoundBugbotIssueCommentWritePort;
+    pullRequestComments: BoundBugbotPullRequestWritePort;
 }

package/build/api/src/application/ports/bugbot_finding_resolution_ports.d.ts

@@ -1,6 +1,6 @@
-import type { BugbotIssueCommentUpdatePort } from "./bugbot_issue_write_ports";
-import type { BugbotPullRequestResolutionPort } from "./bugbot_pull_request_resolution_ports";
+import type { BoundBugbotIssueCommentUpdatePort } from "./bugbot_issue_write_ports";
+import type { BoundBugbotPullRequestResolutionPort } from "./bugbot_pull_request_resolution_ports";
 export interface BugbotFindingResolutionPorts {
-    issueComments: BugbotIssueCommentUpdatePort;
-    pullRequestComments: BugbotPullRequestResolutionPort;
+    issueComments: BoundBugbotIssueCommentUpdatePort;
+    pullRequestComments: BoundBugbotPullRequestResolutionPort;
 }

package/build/api/src/application/ports/bugbot_issue_read_ports.d.ts

@@ -4,6 +4,9 @@ export interface BugbotIssueComment {
     user?: {
         login?: string;
     };
+    /** Provider-authenticated author classification; never inferred from the login. */
+    isAutomatedAuthor?: boolean;
+    createdAt?: string;
 }
 export interface BugbotIssueReadPort {
     listIssueComments(owner: string, repository: string, issueNumber: number, token: string): Promise<BugbotIssueComment[]>;

package/build/api/src/application/ports/bugbot_issue_write_ports.d.ts

@@ -10,3 +10,15 @@ export interface BugbotIssueCommentUpdatePort {
 }
 export interface BugbotIssueCommentWritePort extends BugbotIssueCommentCreatePort, BugbotIssueCommentUpdatePort {
 }
+export interface BoundBugbotIssueCommentCreatePort {
+    addComment(issueNumber: number, comment: string, options?: {
+        commitSha?: string;
+    }): Promise<void>;
+}
+export interface BoundBugbotIssueCommentUpdatePort {
+    updateComment(issueNumber: number, commentId: number, comment: string, options?: {
+        commitSha?: string;
+    }): Promise<void>;
+}
+export interface BoundBugbotIssueCommentWritePort extends BoundBugbotIssueCommentCreatePort, BoundBugbotIssueCommentUpdatePort {
+}

package/build/api/src/application/ports/bugbot_pull_request_read_ports.d.ts

@@ -1,4 +1,4 @@
-import type { PullRequestReviewComment } from "./pull_request_review_comment_ports";
+import type { PullRequestReviewComment, PullRequestReviewThreadState } from "./pull_request_review_comment_ports";
 export interface PullRequestDiffLocation {
     line: number;
     side: "LEFT" | "RIGHT";
@@ -22,26 +22,12 @@ export interface PullRequestReviewDiffSnapshot {
     }>;
 }
 export interface BugbotPullRequestQueryPort {
-    getHeadBranchForIssue(owner: string, repository: string, issueNumber: number, token: string): Promise<string | undefined>;
     getPullRequestReviewCommentBody(owner: string, repository: string, pullNumber: number, commentId: number, token: string): Promise<string | null>;
 }
 export interface BugbotPullRequestReadPort extends BugbotPullRequestQueryPort {
-    getOpenPullRequestNumbersByHeadBranch(owner: string, repository: string, branch: string, token: string): Promise<number[]>;
     listPullRequestReviewComments(owner: string, repository: string, pullNumber: number, token: string): Promise<PullRequestReviewComment[]>;
     getPullRequestHeadSha(owner: string, repository: string, pullNumber: number, token: string): Promise<string | undefined>;
-    getChangedFiles(owner: string, repository: string, pullNumber: number, token: string): Promise<Array<{
-        filename: string;
-        status: string;
-    }>>;
-    getFilesWithFirstDiffLine(owner: string, repository: string, pullNumber: number, token: string): Promise<Array<{
-        path: string;
-        firstLine: number;
-    }>>;
-    getFilesWithDiffLocations?(owner: string, repository: string, pullNumber: number, token: string): Promise<Array<{
-        path: string;
-        locations: PullRequestDiffLocation[];
-    }>>;
     /** Loads all diff projections from one paginated GitHub request. */
-    getReviewDiffSnapshot?(owner: string, repository: string, pullNumber: number, token: string): Promise<PullRequestReviewDiffSnapshot>;
-    listPullRequestReviewThreadStates?(owner: string, repository: string, pullNumber: number, token: string): Promise<Record<string, boolean>>;
+    getReviewDiffSnapshot(owner: string, repository: string, pullNumber: number, token: string): Promise<PullRequestReviewDiffSnapshot>;
+    listPullRequestReviewThreadStates(owner: string, repository: string, pullNumber: number, token: string): Promise<Record<string, PullRequestReviewThreadState>>;
 }

package/build/api/src/application/ports/bugbot_pull_request_resolution_ports.d.ts

@@ -1,3 +1,9 @@
-import type { PullRequestReviewCommentListQueryPort, PullRequestReviewCommentUpdatePort, PullRequestReviewThreadCommandPort } from "./pull_request_review_comment_ports";
+import type { PullRequestReviewComment, PullRequestReviewCommentListQueryPort, PullRequestReviewCommentUpdatePort, PullRequestReviewThreadCommandPort } from "./pull_request_review_comment_ports";
 export interface BugbotPullRequestResolutionPort extends PullRequestReviewCommentListQueryPort, PullRequestReviewCommentUpdatePort, PullRequestReviewThreadCommandPort {
 }
+export interface BoundBugbotPullRequestResolutionPort {
+    listPullRequestReviewComments(pullRequestNumber: number): Promise<PullRequestReviewComment[]>;
+    updatePullRequestReviewComment(commentIdentity: string, body: string): Promise<void>;
+    resolvePullRequestReviewThread(pullRequestNumber: number, commentIdentity: string): Promise<void>;
+    unresolvePullRequestReviewThread(pullRequestNumber: number, commentIdentity: string): Promise<void>;
+}

package/build/api/src/application/ports/bugbot_pull_request_write_ports.d.ts

@@ -1,4 +1,9 @@
-import type { PullRequestReviewCommentCommandPort, PullRequestReviewThreadCommandPort } from "./pull_request_review_comment_ports";
+import type { PullRequestReviewCommentDraft, PullRequestReviewReference, PullRequestReviewCommentCommandPort, PullRequestReviewThreadCommandPort } from "./pull_request_review_comment_ports";
 /** Publication may refresh a comment and reopen its thread when a finding returns. */
 export interface BugbotPullRequestWritePort extends PullRequestReviewCommentCommandPort, Pick<PullRequestReviewThreadCommandPort, "unresolvePullRequestReviewThread"> {
 }
+export interface BoundBugbotPullRequestWritePort {
+    createReviewWithComments(pullRequestNumber: number, commitId: string, body: string, comments: PullRequestReviewCommentDraft[]): Promise<PullRequestReviewReference | undefined>;
+    updatePullRequestReviewComment(commentIdentity: string, body: string): Promise<void>;
+    unresolvePullRequestReviewThread(pullRequestNumber: number, commentIdentity: string): Promise<void>;
+}

package/build/api/src/application/ports/bugbot_reconciliation_ports.d.ts

@@ -0,0 +1,18 @@
+import type { BugbotIssueComment } from './bugbot_issue_read_ports';
+import type { BoundBugbotIssueCommentWritePort } from './bugbot_issue_write_ports';
+import type { BugbotReviewNavigation } from './bugbot_review_navigation_ports';
+import type { PullRequestReviewComment, PullRequestReviewSummary, PullRequestReviewThreadState } from './pull_request_review_comment_ports';
+/** Repository-bound reads required to acquire one coherent final Bugbot snapshot. */
+export interface BugbotReconciliationSnapshotPorts {
+    listIssueComments(issueNumber: number): Promise<BugbotIssueComment[]>;
+    listPullRequestReviewComments(pullRequestNumber: number): Promise<PullRequestReviewComment[]>;
+    listPullRequestReviewThreadStates(pullRequestNumber: number): Promise<Record<string, PullRequestReviewThreadState>>;
+    listPullRequestReviews(pullRequestNumber: number): Promise<PullRequestReviewSummary[]>;
+    getPullRequestHeadSha(pullRequestNumber: number): Promise<string | undefined>;
+    navigationForPullRequest(pullRequestNumber: number, headSha: string): BugbotReviewNavigation;
+}
+/** Repository-bound presentation mutations; credentials never enter the use case. */
+export interface BugbotPresentationMutationPorts {
+    readonly comments: BoundBugbotIssueCommentWritePort;
+    updatePullRequestReview(pullRequestNumber: number, reviewIdentity: string, body: string): Promise<void>;
+}

package/build/api/src/application/ports/bugbot_review_navigation_ports.d.ts

@@ -0,0 +1,13 @@
+/** Provider-owned, user-facing navigation for one verified Bugbot projection. */
+export interface BugbotReviewNavigation {
+    readonly pullRequestUrl: string;
+    readonly commitUrl: string;
+    readonly runUrl?: string;
+}
+/**
+ * Keeps provider URL construction outside the use case and presentation policy.
+ * Implementations must return trusted, absolute HTTPS URLs only.
+ */
+export interface BugbotReviewNavigationPort {
+    forPullRequest(owner: string, repository: string, pullRequestNumber: number, headSha: string): BugbotReviewNavigation;
+}

package/build/api/src/application/ports/bugbot_rule_ports.d.ts

@@ -5,7 +5,7 @@ export interface BugbotReviewRule {
     readonly content: string;
 }
 export interface BugbotRuleFileQueryPort {
-    loadRules(changedFiles: readonly string[]): Promise<readonly BugbotReviewRule[]>;
+    loadRules(changedFiles: readonly string[], revision?: string): Promise<readonly BugbotReviewRule[]>;
 }
 export interface BugbotLearnedRuleCommandPort {
     rememberRule(rule: string): Promise<'created' | 'existing'>;

package/build/api/src/application/ports/bugbot_scm_ports.d.ts

@@ -0,0 +1,14 @@
+import type { BugbotContextPorts } from './bugbot_context_ports';
+import type { BugbotFindingPublicationPorts } from './bugbot_finding_publication_ports';
+import type { BugbotFindingResolutionPorts } from './bugbot_finding_resolution_ports';
+import type { BugbotPresentationMutationPorts, BugbotReconciliationSnapshotPorts } from './bugbot_reconciliation_ports';
+/** Complete repository-bound SCM surface consumed by one Bugbot review. */
+export interface BugbotScmPorts {
+    readonly context: BugbotContextPorts;
+    readonly publication: BugbotFindingPublicationPorts;
+    readonly resolution: BugbotFindingResolutionPorts;
+    readonly reconciliation: {
+        readonly snapshot: BugbotReconciliationSnapshotPorts;
+        readonly presentation: BugbotPresentationMutationPorts;
+    };
+}

package/build/api/src/application/ports/bugbot_telemetry_ports.d.ts

@@ -1,8 +1,10 @@
-export type BugbotReviewOutcome = 'completed' | 'no-findings' | 'dry-run' | 'superseded' | 'skipped' | 'failed';
+export type BugbotReviewOutcome = 'completed' | 'no-findings' | 'partial' | 'dry-run' | 'superseded' | 'skipped' | 'failed';
 export interface BugbotReviewTelemetrySnapshot {
     readonly schemaVersion: 1;
     readonly reviewId: string;
     readonly repository: string;
+    readonly repositoryId?: number;
+    readonly triggerKind: string;
     readonly pullRequestNumber?: number;
     readonly headSha?: string;
     readonly publicationMode: 'publish' | 'dry-run';
@@ -19,11 +21,34 @@ export interface BugbotReviewTelemetrySnapshot {
     readonly changedFiles: number;
     readonly changedLines: number;
     readonly rulesLoaded: number;
+    readonly contextSelectionReason?: "event" | "exact-head" | "none";
+    readonly contextCandidateBucket?: "0" | "1" | "2+";
+    readonly contextCoverageStatus?: "complete" | "partial";
+    readonly contextCoverage?: Readonly<Record<string, {
+        readonly status: "complete" | "partial";
+        readonly pagesFetched: number;
+        readonly itemsFetched: number;
+        readonly itemsRetained: number;
+        readonly omittedItems: number;
+        readonly truncatedItems: number;
+        readonly limitReached: boolean;
+        readonly providerLimitReached?: boolean;
+    }>>;
+    readonly contextLogicalProviderReads: number;
+    readonly contextRawProviderRequests: number;
+    readonly contextConcurrencyLimit: 2;
+    readonly analysisPartitions?: number;
+    readonly completedAnalysisPartitions?: number;
+    readonly analysisDiffFragments?: number;
+    readonly analysisAssignedFiles?: number;
+    readonly maximumAnalysisConcurrency?: number;
+    readonly failedAnalysisPartitionOrdinal?: number;
+    readonly failedAnalysisPartitionCategory?: string;
     readonly candidateFindings: number;
     readonly publishedFindings: number;
     readonly overflowFindings: number;
     readonly resolvedFindings: number;
-    readonly findingStates?: Readonly<Record<'open' | 'fixed' | 'obsolete' | 'dismissed' | 'reopened', number>>;
+    readonly findingStates?: Readonly<Partial<Record<import('../../domain/bugbot/review_state').BugbotFindingState, number>>>;
     readonly outcome: BugbotReviewOutcome;
     readonly errorCategory?: string;
 }

package/build/api/src/application/ports/message_catalog_ports.d.ts

@@ -0,0 +1,17 @@
+import type { AgentConfiguration } from '../../domain/agent';
+import type { CatalogMessage, MessageCatalogDefinition, ResolvedMessageCatalog, CatalogResolutionObservation } from '../../domain/message_catalog';
+export interface MessageCatalogResolutionRequest<Id extends string> {
+    readonly targetLocale: string;
+    readonly ids: readonly Id[];
+    readonly sourceCatalog: MessageCatalogDefinition<Id>;
+    readonly bundledCatalogs: readonly MessageCatalogDefinition<Id>[];
+    readonly configuration?: Readonly<AgentConfiguration>;
+}
+export interface MessageCatalogResolutionPort {
+    resolve<Id extends string>(request: MessageCatalogResolutionRequest<Id>): Promise<ResolvedMessageCatalog<Id>>;
+    observations?(): readonly CatalogResolutionObservation[];
+}
+export interface DynamicMessageCatalogResponse<Id extends string> {
+    readonly targetLocale: string;
+    readonly messages: Readonly<Record<Id, CatalogMessage>>;
+}

package/build/api/src/application/ports/pull_request_review_comment_ports.d.ts

@@ -7,6 +7,25 @@ export type PullRequestReviewComment = {
     path?: string;
     line?: number;
     authorLogin?: string;
+    /** Provider-authenticated author classification; never inferred from the login. */
+    isAutomatedAuthor?: boolean;
+    createdAt?: string;
+    /** Opaque identity of the submitted review that owns this comment. */
+    parentReviewIdentity?: string;
+    /** Safe provider URL for user-facing navigation. */
+    url?: string;
+};
+export type PullRequestReviewSummary = {
+    /** Lossless opaque provider identity used by mutations. */
+    identity: string;
+    body: string | null;
+    authorLogin?: string;
+    commitId?: string;
+    url?: string;
+};
+export type PullRequestReviewReference = {
+    identity: string;
+    url?: string;
 };
 export type PullRequestReviewCommentDraft = {
     path: string;
@@ -26,18 +45,28 @@ export interface PullRequestReviewCommentBodyQueryPort {
 export interface PullRequestReviewCommentQueryPort extends PullRequestReviewCommentListQueryPort, PullRequestReviewCommentBodyQueryPort {
 }
 export interface PullRequestReviewCommentCreatePort {
-    createReviewWithComments(owner: string, repository: string, pullRequestNumber: number, commitId: string, body: string, comments: PullRequestReviewCommentDraft[], token: string): Promise<void>;
+    createReviewWithComments(owner: string, repository: string, pullRequestNumber: number, commitId: string, body: string, comments: PullRequestReviewCommentDraft[], token: string): Promise<PullRequestReviewReference | undefined>;
 }
 export interface PullRequestReviewCommentUpdatePort {
     updatePullRequestReviewComment(owner: string, repository: string, commentIdentity: string, body: string, token: string): Promise<void>;
 }
 export interface PullRequestReviewCommentCommandPort extends PullRequestReviewCommentCreatePort, PullRequestReviewCommentUpdatePort {
 }
+export interface PullRequestReviewSummaryQueryPort {
+    listPullRequestReviews(owner: string, repository: string, pullRequestNumber: number, token: string): Promise<PullRequestReviewSummary[]>;
+}
+export interface PullRequestReviewSummaryUpdatePort {
+    updatePullRequestReview(owner: string, repository: string, pullRequestNumber: number, reviewIdentity: string, body: string, token: string): Promise<void>;
+}
 export interface PullRequestReviewThreadCommandPort {
     resolvePullRequestReviewThread(owner: string, repository: string, pullRequestNumber: number, commentIdentity: string, token: string): Promise<void>;
     unresolvePullRequestReviewThread(owner: string, repository: string, pullRequestNumber: number, commentIdentity: string, token: string): Promise<void>;
 }
 export interface PullRequestReviewThreadStateQueryPort {
     /** Maps review-comment node identities to their parent thread resolution state. */
-    listPullRequestReviewThreadStates(owner: string, repository: string, pullRequestNumber: number, token: string): Promise<Record<string, boolean>>;
+    listPullRequestReviewThreadStates(owner: string, repository: string, pullRequestNumber: number, token: string): Promise<Record<string, PullRequestReviewThreadState>>;
+}
+export interface PullRequestReviewThreadState {
+    resolved: boolean;
+    resolvedByLogin?: string;
 }

package/build/api/src/data/model/application_error.d.ts

@@ -0,0 +1,82 @@
+import { type BranchSyncValidationCode } from '../../domain/branch_sync_validation';
+export type ApplicationErrorKind = 'configuration' | 'authorization' | 'provider' | 'agent' | 'validation' | 'workflow' | 'unknown';
+export type ApplicationErrorCode = 'configuration.invalid' | 'configuration.unsupported' | 'authorization.denied' | 'authorization.credential-invalid' | 'provider.not-found' | 'provider.conflict' | 'provider.rate-limited' | 'provider.unavailable' | 'provider.contract-invalid' | 'agent.policy-rejected' | 'agent.failed' | 'locale.output-invalid' | 'locale.translation-failed' | 'validation.invalid-input' | 'workflow.invalid-event' | 'workflow.stale' | 'workflow.cancelled' | 'workflow.failed' | 'workflow.presentation-pending' | 'timeout' | 'unexpected';
+export declare const APPLICATION_ERROR_RECOVERY_IDS: readonly ["pull-request-link-restored", "pull-request-link-base-retained", "pull-request-link-reference-retained", "pull-request-link-base-and-reference-retained", "managed-branch-enrichment-failed", "inactivity-explanation-failed", "bugbot-review-blocks-pending", "branch-sync-agent-disabled", "branch-sync-fixer-unavailable", "branch-sync-resolution-invalid", "sdd-draft-size-exceeded", "sdd-draft-format-invalid", "workspace-changes-rejected"];
+export type ApplicationErrorRecoveryId = typeof APPLICATION_ERROR_RECOVERY_IDS[number];
+interface ApplicationErrorRecoveryVariables {
+    readonly 'pull-request-link-restored': Readonly<Record<string, never>>;
+    readonly 'pull-request-link-base-retained': Readonly<Record<string, never>>;
+    readonly 'pull-request-link-reference-retained': Readonly<Record<string, never>>;
+    readonly 'pull-request-link-base-and-reference-retained': Readonly<Record<string, never>>;
+    readonly 'managed-branch-enrichment-failed': Readonly<{
+        branchName: string;
+    }>;
+    readonly 'inactivity-explanation-failed': Readonly<{
+        issueNumber: number;
+    }>;
+    readonly 'bugbot-review-blocks-pending': Readonly<{
+        pendingCount: number;
+    }>;
+    readonly 'branch-sync-agent-disabled': Readonly<{
+        command: '/copilot sync-branch';
+    }>;
+    readonly 'branch-sync-fixer-unavailable': Readonly<{
+        command: '/copilot sync-branch';
+    }>;
+    readonly 'branch-sync-resolution-invalid': Readonly<{
+        validationCode: BranchSyncValidationCode;
+        command: '/copilot sync-branch';
+    }>;
+    readonly 'sdd-draft-size-exceeded': Readonly<Record<string, never>>;
+    readonly 'sdd-draft-format-invalid': Readonly<Record<string, never>>;
+    readonly 'workspace-changes-rejected': Readonly<Record<string, never>>;
+}
+export type ApplicationErrorRecovery = {
+    readonly [Id in ApplicationErrorRecoveryId]: Readonly<{
+        id: Id;
+        variables: ApplicationErrorRecoveryVariables[Id];
+    }>;
+}[ApplicationErrorRecoveryId];
+interface ApplicationErrorMetadata {
+    readonly kind: ApplicationErrorKind;
+    readonly retryable: boolean;
+    readonly impact: string;
+    readonly action: string;
+    readonly retainedState: string;
+}
+export declare const APPLICATION_ERROR_METADATA: Readonly<Record<ApplicationErrorCode, ApplicationErrorMetadata>>;
+export interface SemanticApplicationErrorOptions {
+    readonly retryable?: boolean;
+    readonly recovery?: ApplicationErrorRecovery;
+    readonly correlationId: string;
+    readonly cause?: unknown;
+}
+export interface ApplicationErrorPublicRecord {
+    readonly name: 'ApplicationError';
+    readonly message: string;
+    readonly code: ApplicationErrorCode;
+    readonly kind: ApplicationErrorKind;
+    readonly retryable: boolean;
+    readonly impact: string;
+    readonly action: string;
+    readonly retainedState: string;
+    readonly correlationId: string;
+    readonly recovery?: ApplicationErrorRecovery;
+}
+export declare function isApplicationErrorCorrelationId(value: string): boolean;
+/** Semantic error contract whose public fields are safe to serialize and present. */
+export declare class ApplicationError extends Error {
+    #private;
+    readonly name: 'ApplicationError';
+    readonly code: ApplicationErrorCode;
+    readonly kind: ApplicationErrorKind;
+    readonly retryable: boolean;
+    readonly impact: string;
+    readonly action: string;
+    readonly retainedState: string;
+    readonly correlationId: string;
+    readonly recovery?: ApplicationErrorRecovery;
+    constructor(code: ApplicationErrorCode, message: string, options: SemanticApplicationErrorOptions);
+    toJSON(): ApplicationErrorPublicRecord;
+}
+export {};

package/build/api/src/data/model/result.d.ts

@@ -1,3 +1,4 @@
+import type { ApplicationError } from './application_error';
 export type ResultStepFormat = 'plain' | 'markdown';
 export interface ResultInput {
     id?: string;
@@ -6,9 +7,7 @@ export interface ResultInput {
     steps?: string[];
     payload?: unknown;
     reminders?: string[];
-    errors?: unknown[];
-    /** Compatibility input while callers migrate to the plural property. */
-    error?: unknown;
+    errors?: readonly ApplicationError[];
     stepFormat?: ResultStepFormat;
 }
 export declare function getResultPayload(payload: unknown): Record<string, unknown> | undefined;
@@ -19,7 +18,7 @@ export declare class Result {
     steps: string[];
     payload: unknown;
     reminders: string[];
-    errors: Error[];
+    readonly errors: readonly ApplicationError[];
     stepFormat: ResultStepFormat;
     constructor(data: ResultInput);
 }

package/build/api/src/domain/agent.d.ts

@@ -3,18 +3,19 @@ export type AgentTask = 'findings' | 'fixer' | 'planner' | 'reviewer' | 'tester'
 export type AgentCapability = AgentTask | 'language';
 export declare const DEFAULT_AGENT_PROVIDER: AgentProvider;
 export declare const DEFAULT_MODEL_PROVIDER = "openai";
-export declare const DEFAULT_AGENT_MODEL = "gpt-5.6-luna";
+export declare const DEFAULT_AGENT_MODEL = "gpt-6-luna";
+export declare const AGENT_EXECUTABLE_BASENAMES: Readonly<Record<AgentProvider, string>>;
 /**
- * Agent configuration is the provider-neutral contract shared by the action,
- * CLI and infrastructure adapters. Provider-specific command flags are
- * derived at the boundary instead of being assembled by callers.
+ * Agent configuration is the provider-neutral contract shared by entrypoints
+ * and the execution planner. Provider argv and authority are never supplied by
+ * callers; each provider policy reconstructs them from these structured facts.
  */
 export interface AgentConfiguration {
     provider: AgentProvider;
     modelProvider?: string;
     model: string;
     effort?: string;
-    command?: string;
+    executable?: string;
 }
 export interface AgentTaskConfiguration {
     findings: AgentConfiguration;

package/build/api/src/domain/branch_sync_validation.d.ts

@@ -0,0 +1,4 @@
+/** Closed diagnostic vocabulary; workspace paths and provider output stay private. */
+export declare const BRANCH_SYNC_VALIDATION_CODES: readonly ["merge-state-mismatch", "head-changed", "merge-parent-changed", "unresolved-conflicts", "unexpected-paths", "unstaged-changes", "protected-index-changed", "sdd-catalog-invalid", "communication-budget-invalid", "staging-failed", "whitespace-or-conflict-markers", "parent-head-changed", "working-head-changed", "unclassified"];
+export type BranchSyncValidationCode = typeof BRANCH_SYNC_VALIDATION_CODES[number];
+export declare function admittedBranchSyncValidationCode(value: unknown): BranchSyncValidationCode;