@weareikko/code-review 0.0.1 → 0.8.1

package/dist/platform.d.ts

@@ -0,0 +1,71 @@
+import type { Config } from './config.js';
+import type { Discussion } from './gitlab.js';
+import type { PostingMode, PostResult, SummaryResult, UpsertSummaryOptions } from './posting.js';
+import type { DiffRefs, GeneratedComment, ReviewComment } from './types.js';
+/**
+ * The change under review, normalized across source-control platforms. Mirrors
+ * the subset of a GitLab merge request (and a GitHub pull request) that the
+ * review engine consumes: the branch pair plus the author's declared intent.
+ */
+export interface MergeRequestMeta {
+    source_branch: string;
+    target_branch: string;
+    /** One-line declared intent of the change. May be empty. */
+    title?: string;
+    /** The author's full reasoning / decision log. May be empty or null. */
+    description?: string | null;
+}
+/**
+ * Minimal, telemetry-agnostic view of the most recent HTTP response a platform
+ * issued. The diagnostics/OTel layer maps these onto HTTP semantic-convention
+ * span attributes without coupling the seam to any observability SDK. Carries no
+ * secrets — the auth token travels in a request header, never the URL.
+ */
+export interface ScmResponseInfo {
+    method: string;
+    url: string;
+    status: number;
+    /** Parsed Content-Length header in bytes, when the response provided one. */
+    responseContentLength?: number;
+}
+/**
+ * The seam every source-control backend implements so `run()` can drive a review
+ * without knowing whether it is talking to GitLab or GitHub. The review core
+ * (local-git diff, agent, prompt, skills, parser, fingerprints, dedup, summary
+ * carryover) stays platform-agnostic; only target identification, reading
+ * existing comments, and posting differ per platform and live behind this
+ * interface.
+ */
+export interface ReviewPlatform {
+    /** Fetch the change's branch pair and declared intent. */
+    getMergeRequest(): Promise<MergeRequestMeta>;
+    /**
+     * Resolve the diff refs. On GitLab these are the MR version base/start/head
+     * SHAs; `head_sha` is the reviewed commit used for the reviewed-commit skip
+     * marker and the comment/summary footers.
+     */
+    getRefs(): Promise<DiffRefs>;
+    /** Existing comments, normalized so the shared dedup/summary helpers work unchanged. */
+    getDiscussions(): Promise<Discussion[]>;
+    /** Turn parsed reviewer findings into platform-specific posting payloads. */
+    buildComments(comments: ReviewComment[], diff: string, refs: DiffRefs, existingFingerprints: Set<string>): GeneratedComment[];
+    /** Post (or draft-then-publish) the non-duplicate generated comments. */
+    postComments(generated: GeneratedComment[], mode: PostingMode): Promise<PostResult>;
+    /** Create or update the single MR-level summary note. */
+    upsertSummary(summary: string, discussions: Discussion[], options: UpsertSummaryOptions): Promise<SummaryResult>;
+    /**
+     * The most recent HTTP response this platform issued, for telemetry stamping.
+     * Returns `undefined` until the first request completes. Each request reports
+     * a fresh object so a phase can compare identity and stamp only its own call.
+     */
+    lastResponse(): ScmResponseInfo | undefined;
+}
+/**
+ * Build the {@link ReviewPlatform} for this run from the resolved config. The
+ * platform was already selected (auto-detected from the environment or forced by
+ * `--platform`/`CODE_REVIEW_PLATFORM`) during config resolution; this only
+ * constructs the matching backend. GitHub target identifiers are parsed here so a
+ * malformed `owner/repo` or pull number fails fast with an actionable hint.
+ */
+export declare function createPlatform(config: Config): ReviewPlatform;
+//# sourceMappingURL=platform.d.ts.map

package/dist/platform.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"platform.d.ts","sourceRoot":"","sources":["../src/platform.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,aAAa,CAAC;AAC1C,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AAO9C,OAAO,KAAK,EAAE,WAAW,EAAE,UAAU,EAAE,aAAa,EAAE,oBAAoB,EAAE,MAAM,cAAc,CAAC;AACjG,OAAO,KAAK,EAAE,QAAQ,EAAE,gBAAgB,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAE5E;;;;GAIG;AACH,MAAM,WAAW,gBAAgB;IAC/B,aAAa,EAAE,MAAM,CAAC;IACtB,aAAa,EAAE,MAAM,CAAC;IACtB,4DAA4D;IAC5D,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,wEAAwE;IACxE,WAAW,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC7B;AAED;;;;;GAKG;AACH,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,MAAM,CAAC;IACf,GAAG,EAAE,MAAM,CAAC;IACZ,MAAM,EAAE,MAAM,CAAC;IACf,6EAA6E;IAC7E,qBAAqB,CAAC,EAAE,MAAM,CAAC;CAChC;AAED;;;;;;;GAOG;AACH,MAAM,WAAW,cAAc;IAC7B,0DAA0D;IAC1D,eAAe,IAAI,OAAO,CAAC,gBAAgB,CAAC,CAAC;IAC7C;;;;OAIG;IACH,OAAO,IAAI,OAAO,CAAC,QAAQ,CAAC,CAAC;IAC7B,wFAAwF;IACxF,cAAc,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC,CAAC;IACxC,6EAA6E;IAC7E,aAAa,CACX,QAAQ,EAAE,aAAa,EAAE,EACzB,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,QAAQ,EACd,oBAAoB,EAAE,GAAG,CAAC,MAAM,CAAC,GAChC,gBAAgB,EAAE,CAAC;IACtB,yEAAyE;IACzE,YAAY,CAAC,SAAS,EAAE,gBAAgB,EAAE,EAAE,IAAI,EAAE,WAAW,GAAG,OAAO,CAAC,UAAU,CAAC,CAAC;IACpF,yDAAyD;IACzD,aAAa,CACX,OAAO,EAAE,MAAM,EACf,WAAW,EAAE,UAAU,EAAE,EACzB,OAAO,EAAE,oBAAoB,GAC5B,OAAO,CAAC,aAAa,CAAC,CAAC;IAC1B;;;;OAIG;IACH,YAAY,IAAI,eAAe,GAAG,SAAS,CAAC;CAC7C;AAED;;;;;;GAMG;AACH,wBAAgB,cAAc,CAAC,MAAM,EAAE,MAAM,GAAG,cAAc,CAY7D"}

package/dist/platforms/github.d.ts

@@ -0,0 +1,115 @@
+import { type IssueComment, type PullRequestReviewComment } from '../github.js';
+import type { Discussion } from '../gitlab.js';
+import type { MergeRequestMeta, ReviewPlatform, ScmResponseInfo } from '../platform.js';
+import { type PostingMode, type PostResult, type SummaryResult, type UpsertSummaryOptions } from '../posting.js';
+import type { DiffRefs, GeneratedComment, ReviewComment, Side } from '../types.js';
+/**
+ * A single inline comment in a batched GitHub review, positioned against the
+ * diff. GitHub takes one `line` plus a `side` (unlike GitLab's two-sided
+ * position). Built by {@link buildGitHubReviewPayload}; `null` when the finding
+ * cannot be anchored to the diff (see {@link buildGitHubComments}).
+ */
+export interface GitHubReviewCommentPayload {
+    path: string;
+    body: string;
+    line: number;
+    side: Side;
+}
+/** The `owner` and `repo` halves of a GitHub `owner/repo` slug. */
+export interface GitHubRepository {
+    owner: string;
+    repo: string;
+}
+/**
+ * Split a `GITHUB_REPOSITORY` slug into its `owner` and `repo` halves. GitHub
+ * repository slugs contain exactly one `/` (neither owner nor repo may be empty
+ * or contain a slash), so anything else is a configuration error surfaced with an
+ * actionable hint rather than a confusing 404 from the API.
+ */
+export declare function parseGitHubRepository(slug: string): GitHubRepository;
+/**
+ * Parse a pull-request number from its string form. Rejects non-integer or
+ * non-positive values so a malformed `--pr`/`GITHUB_REF` fails fast with a hint
+ * instead of hitting `/pulls/NaN`.
+ */
+export declare function parseGitHubPullNumber(value: string): number;
+/** What a {@link GitHubPlatform} needs to talk to one pull request. */
+export interface GitHubPlatformOptions {
+    /** REST API base (default `https://api.github.com`; GHE sets an `/api/v3` base). */
+    apiUrl?: string;
+    token: string;
+    owner: string;
+    repo: string;
+    /** Pull-request number. */
+    pull: number;
+    fetchImpl?: typeof fetch;
+    requestTimeout?: number;
+}
+/**
+ * Resolve one reviewer finding to a GitHub review-comment payload, or `null`
+ * when it cannot be anchored to GitHub's diff. GitHub uses a single `line` +
+ * `side`:
+ *   - a pure added line takes the new-file line with `side: 'RIGHT'`,
+ *   - a pure removed line takes the old-file line with `side: 'LEFT'`,
+ *   - an unchanged (context) line takes the new-file line with `side: 'RIGHT'`,
+ *     regardless of the finding's side — GitHub's API prescribes RIGHT for
+ *     "unchanged lines ... shown for context", so a LEFT context comment is
+ *     doc-noncompliant and risks a 422.
+ *
+ * Returning `null` for an off-diff line is deliberate: GitHub answers 422 when a
+ * review comment points at a line outside the diff, and one such comment fails
+ * the whole batched review, so the poster drops these rather than post them. A
+ * context line is only kept when it lies within GitHub's default 3-line hunk
+ * context of a change — our local diff carries far more context (`--unified=20`)
+ * than GitHub's PR diff, so a distant context line is inside our diff but off
+ * GitHub's.
+ */
+export declare function buildGitHubReviewPayload(comment: ReviewComment, body: string, diff: string): GitHubReviewCommentPayload | null;
+/**
+ * Turn parsed reviewer findings into GitHub review-comment payloads, mirroring
+ * the GitLab `buildGeneratedComments` contract: same hunk-context fingerprints,
+ * same dedup semantics, same comment body/footer. The payload is
+ * `GitHubReviewCommentPayload | null` — `null` for a finding that cannot be
+ * placed on the diff, which the poster skips.
+ */
+export declare function buildGitHubComments(comments: ReviewComment[], diff: string, refs: DiffRefs, existingFingerprints: Set<string>): GeneratedComment<GitHubReviewCommentPayload | null>[];
+/**
+ * Normalize GitHub's two comment streams into the {@link Discussion}[] shape the
+ * platform-agnostic helpers consume unchanged:
+ *   - Inline **review comments** are threaded by `in_reply_to_id` (a reply joins
+ *     its root comment's discussion, in arrival order) so `extractPriorThreads`
+ *     sees a bot note followed by human replies, exactly like GitLab.
+ *   - Non-positional **issue comments** each become a single-note discussion;
+ *     the MR-level summary note lives here and is found via its summary marker.
+ *
+ * The fingerprint markers, summary marker, and reviewed-commit footer are HTML
+ * comments that render identically on GitHub, so `extractExistingFingerprints`,
+ * `findExistingSummaryNote`, and the reviewed-commit scan all work as-is.
+ */
+export declare function normalizeGitHubDiscussions(reviewComments: PullRequestReviewComment[], issueComments: IssueComment[]): Discussion[];
+/**
+ * {@link ReviewPlatform} backed by the GitHub pull-request API. Positioned
+ * findings post as ONE batched review (`event: 'COMMENT'`) — atomic and free of
+ * per-comment secondary rate limits — and the MR-level summary is upserted as a
+ * single issue comment. The review core is unchanged; only target
+ * identification, reading existing comments, and posting differ from GitLab.
+ */
+export declare class GitHubPlatform implements ReviewPlatform {
+    private readonly client;
+    private readonly owner;
+    private readonly repo;
+    private readonly pull;
+    private last;
+    private pullRequestPromise;
+    private commitId;
+    constructor(options: GitHubPlatformOptions);
+    lastResponse(): ScmResponseInfo | undefined;
+    private pullRequest;
+    getMergeRequest(): Promise<MergeRequestMeta>;
+    getRefs(): Promise<DiffRefs>;
+    getDiscussions(): Promise<Discussion[]>;
+    buildComments(comments: ReviewComment[], diff: string, refs: DiffRefs, existingFingerprints: Set<string>): GeneratedComment[];
+    postComments(generated: GeneratedComment[], _mode: PostingMode): Promise<PostResult>;
+    upsertSummary(summary: string, discussions: Discussion[], options: UpsertSummaryOptions): Promise<SummaryResult>;
+}
+//# sourceMappingURL=github.d.ts.map

package/dist/platforms/github.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"github.d.ts","sourceRoot":"","sources":["../../src/platforms/github.ts"],"names":[],"mappings":"AAGA,OAAO,EAEL,KAAK,YAAY,EAEjB,KAAK,wBAAwB,EAC9B,MAAM,cAAc,CAAC;AACtB,OAAO,KAAK,EAAE,UAAU,EAA0C,MAAM,cAAc,CAAC;AAEvF,OAAO,KAAK,EAAE,gBAAgB,EAAE,cAAc,EAAE,eAAe,EAAE,MAAM,gBAAgB,CAAC;AACxF,OAAO,EAEL,KAAK,WAAW,EAChB,KAAK,UAAU,EACf,KAAK,aAAa,EAClB,KAAK,oBAAoB,EAC1B,MAAM,eAAe,CAAC;AACvB,OAAO,KAAK,EAAE,QAAQ,EAAE,gBAAgB,EAAE,aAAa,EAAE,IAAI,EAAE,MAAM,aAAa,CAAC;AAEnF;;;;;GAKG;AACH,MAAM,WAAW,0BAA0B;IACzC,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,IAAI,CAAC;CACZ;AAED,mEAAmE;AACnE,MAAM,WAAW,gBAAgB;IAC/B,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;CACd;AAED;;;;;GAKG;AACH,wBAAgB,qBAAqB,CAAC,IAAI,EAAE,MAAM,GAAG,gBAAgB,CASpE;AAED;;;;GAIG;AACH,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAQ3D;AAED,uEAAuE;AACvE,MAAM,WAAW,qBAAqB;IACpC,oFAAoF;IACpF,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;IACb,2BAA2B;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,CAAC,EAAE,OAAO,KAAK,CAAC;IACzB,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAgB,wBAAwB,CACtC,OAAO,EAAE,aAAa,EACtB,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,MAAM,GACX,0BAA0B,GAAG,IAAI,CAenC;AAED;;;;;;GAMG;AACH,wBAAgB,mBAAmB,CACjC,QAAQ,EAAE,aAAa,EAAE,EACzB,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,QAAQ,EACd,oBAAoB,EAAE,GAAG,CAAC,MAAM,CAAC,GAChC,gBAAgB,CAAC,0BAA0B,GAAG,IAAI,CAAC,EAAE,CAqBvD;AAoBD;;;;;;;;;;;;GAYG;AACH,wBAAgB,0BAA0B,CACxC,cAAc,EAAE,wBAAwB,EAAE,EAC1C,aAAa,EAAE,YAAY,EAAE,GAC5B,UAAU,EAAE,CA0Bd;AAED;;;;;;GAMG;AACH,qBAAa,cAAe,YAAW,cAAc;IACnD,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAe;IACtC,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAS;IAC/B,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAS;IAC9B,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAS;IAG9B,OAAO,CAAC,IAAI,CAA8B;IAI1C,OAAO,CAAC,kBAAkB,CAAmC;IAG7D,OAAO,CAAC,QAAQ,CAAqB;IAErC,YAAY,OAAO,EAAE,qBAAqB,EAazC;IAED,YAAY,IAAI,eAAe,GAAG,SAAS,CAE1C;IAED,OAAO,CAAC,WAAW;IAOb,eAAe,IAAI,OAAO,CAAC,gBAAgB,CAAC,CAQjD;IAEK,OAAO,IAAI,OAAO,CAAC,QAAQ,CAAC,CAYjC;IAEK,cAAc,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC,CAM5C;IAED,aAAa,CACX,QAAQ,EAAE,aAAa,EAAE,EACzB,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,QAAQ,EACd,oBAAoB,EAAE,GAAG,CAAC,MAAM,CAAC,GAChC,gBAAgB,EAAE,CAGpB;IAIK,YAAY,CAAC,SAAS,EAAE,gBAAgB,EAAE,EAAE,KAAK,EAAE,WAAW,GAAG,OAAO,CAAC,UAAU,CAAC,CA8CzF;IAEK,aAAa,CACjB,OAAO,EAAE,MAAM,EACf,WAAW,EAAE,UAAU,EAAE,EACzB,OAAO,EAAE,oBAAoB,GAC5B,OAAO,CAAC,aAAa,CAAC,CAQxB;CACF"}

package/dist/platforms/gitlab.d.ts

@@ -0,0 +1,26 @@
+import type { Config } from '../config.js';
+import { type Discussion } from '../gitlab.js';
+import type { MergeRequestMeta, ReviewPlatform, ScmResponseInfo } from '../platform.js';
+import { type PostingMode, type PostResult, type SummaryResult, type UpsertSummaryOptions } from '../posting.js';
+import type { DiffRefs, GeneratedComment, ReviewComment } from '../types.js';
+/**
+ * {@link ReviewPlatform} backed by the GitLab merge-request API. A thin wrapper
+ * over the existing {@link GitLabClient}, `payloads.ts`, and `posting.ts`; it
+ * carries no review logic of its own so GitLab behavior is byte-for-byte
+ * identical to the pre-seam code path.
+ */
+export declare class GitLabPlatform implements ReviewPlatform {
+    private readonly client;
+    private readonly project;
+    private readonly mr;
+    private last;
+    constructor(config: Config);
+    lastResponse(): ScmResponseInfo | undefined;
+    getMergeRequest(): Promise<MergeRequestMeta>;
+    getRefs(): Promise<DiffRefs>;
+    getDiscussions(): Promise<Discussion[]>;
+    buildComments(comments: ReviewComment[], diff: string, refs: DiffRefs, existingFingerprints: Set<string>): GeneratedComment[];
+    postComments(generated: GeneratedComment[], mode: PostingMode): Promise<PostResult>;
+    upsertSummary(summary: string, discussions: Discussion[], options: UpsertSummaryOptions): Promise<SummaryResult>;
+}
+//# sourceMappingURL=gitlab.d.ts.map

package/dist/platforms/gitlab.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"gitlab.d.ts","sourceRoot":"","sources":["../../src/platforms/gitlab.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,cAAc,CAAC;AAC3C,OAAO,EAAE,KAAK,UAAU,EAAgB,MAAM,cAAc,CAAC;AAE7D,OAAO,KAAK,EAAE,gBAAgB,EAAE,cAAc,EAAE,eAAe,EAAE,MAAM,gBAAgB,CAAC;AACxF,OAAO,EAEL,KAAK,WAAW,EAChB,KAAK,UAAU,EACf,KAAK,aAAa,EAElB,KAAK,oBAAoB,EAC1B,MAAM,eAAe,CAAC;AACvB,OAAO,KAAK,EAAE,QAAQ,EAAE,gBAAgB,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAE7E;;;;;GAKG;AACH,qBAAa,cAAe,YAAW,cAAc;IACnD,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAe;IACtC,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAS;IACjC,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAS;IAG5B,OAAO,CAAC,IAAI,CAA8B;IAE1C,YAAY,MAAM,EAAE,MAAM,EAWzB;IAED,YAAY,IAAI,eAAe,GAAG,SAAS,CAE1C;IAED,eAAe,IAAI,OAAO,CAAC,gBAAgB,CAAC,CAE3C;IAEK,OAAO,IAAI,OAAO,CAAC,QAAQ,CAAC,CAOjC;IAED,cAAc,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC,CAEtC;IAED,aAAa,CACX,QAAQ,EAAE,aAAa,EAAE,EACzB,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,QAAQ,EACd,oBAAoB,EAAE,GAAG,CAAC,MAAM,CAAC,GAChC,gBAAgB,EAAE,CAEpB;IAED,YAAY,CAAC,SAAS,EAAE,gBAAgB,EAAE,EAAE,IAAI,EAAE,WAAW,GAAG,OAAO,CAAC,UAAU,CAAC,CAElF;IAED,aAAa,CACX,OAAO,EAAE,MAAM,EACf,WAAW,EAAE,UAAU,EAAE,EACzB,OAAO,EAAE,oBAAoB,GAC5B,OAAO,CAAC,aAAa,CAAC,CAExB;CACF"}

package/dist/posting.d.ts

@@ -0,0 +1,108 @@
+import type { Discussion, GitLabClient } from './gitlab.js';
+import type { GeneratedComment, SizeSkippedFile } from './types.js';
+export declare const SUMMARY_MARKER = "<!-- code-review:summary -->";
+export declare const SUMMARY_HISTORY_START = "<!-- code-review:summary-history:start -->";
+export declare const SUMMARY_HISTORY_END = "<!-- code-review:summary-history:end -->";
+export declare const SUMMARY_HISTORY_ENTRY_START = "<!-- code-review:summary-history-entry:start -->";
+export declare const SUMMARY_HISTORY_ENTRY_END = "<!-- code-review:summary-history-entry:end -->";
+export declare const SUMMARY_HISTORY_LIMIT = 10;
+/**
+ * Matches the reviewed-commit footer on read. Both the scope and the repo URL
+ * host path accept the current `weareikko` org and the legacy `ikko-dev` org,
+ * and both the name segment and the repo name accept the current `code-review`
+ * and the legacy `gitlab-review`. The GitHub repo was renamed
+ * `weareikko/gitlab-review` → `weareikko/code-review`, so writers now emit the
+ * `code-review` URL; matching both keeps the reviewed-commit skip working on
+ * MRs/PRs already reviewed under any prior identity.
+ */
+export declare const REVIEWED_COMMIT_FOOTER_PATTERN: RegExp;
+export type SummaryAction = 'created' | 'updated';
+export interface SummaryResult {
+    action: SummaryAction;
+    noteId?: number;
+}
+export interface SummaryNote {
+    id: number;
+    body: string;
+}
+/**
+ * MR-level "this change is too big" signal. Surfaced as a prominent callout at
+ * the top of the summary so a reviewer cannot miss that part of the change went
+ * unreviewed or that the MR is past a human's reviewability threshold.
+ */
+export interface SizeNotice {
+    /** Files dropped for exceeding the char budget. */
+    sizeSkippedFiles: SizeSkippedFile[];
+    /** Set when the reviewed diff's changed-line count crossed the configured threshold. */
+    decomposeHint?: {
+        lines: number;
+        threshold: number;
+    };
+    /** Diff coverage when files were dropped: reviewed vs total changed lines. */
+    coverage?: {
+        reviewedLines: number;
+        totalLines: number;
+    };
+}
+export interface SummaryBodyOptions {
+    historyEntries?: string[];
+    reviewedCommitSha?: string;
+    skillsFooter?: string;
+    runId?: string;
+    /** Prominent size/decompose callout rendered above the reviewer's summary. */
+    sizeNotice?: SizeNotice;
+}
+export interface UpsertSummaryOptions extends SummaryBodyOptions {
+    archivedAt?: Date;
+    costFooter?: string;
+}
+/**
+ * Render the prominent size/decompose callout that sits directly under the
+ * "### Code Review" heading. Returns an empty string when there is nothing to
+ * surface, so the summary body is byte-for-byte unchanged in the common case.
+ */
+export declare function buildSizeNoticeBlock(notice?: SizeNotice): string;
+export declare function buildSummaryBody(summary: string, costFooter?: string, options?: SummaryBodyOptions): string;
+export declare function buildReviewedCommitFooter(commitSha: string): string;
+export declare function extractReviewedCommitSha(body: string): string | null;
+export declare function findExistingReviewedCommitSha(discussions: Discussion[]): string | null;
+export declare function findExistingSummaryNote(discussions: Discussion[]): SummaryNote | null;
+export declare function findExistingSummaryNoteId(discussions: Discussion[]): number | null;
+export declare function buildArchivedSummaryEntry(body: string, archivedAt?: Date): string;
+export declare function extractSummaryHistoryEntries(body: string): string[];
+export declare function stripSummaryHistory(body: string): string;
+export declare function stripSummaryMarker(body: string): string;
+export declare function buildSummaryHistoryEntries(existingBody: string, archivedAt?: Date): string[];
+/**
+ * Resolve the summary upsert into the note body to write plus the existing note
+ * to update (or `null` to create). Platform-agnostic — it operates on the
+ * normalized {@link Discussion}[] and returns strings/ids — so both the GitLab
+ * and GitHub platforms share the exact same body-building and history-carryover
+ * logic and only differ in which API call posts the result.
+ */
+export declare function buildUpsertSummary(summary: string, discussions: Discussion[], options: UpsertSummaryOptions): {
+    body: string;
+    existing: SummaryNote | null;
+};
+export declare function upsertSummaryNote(gitlab: GitLabClient, project: string, mr: string, summary: string, discussions: Discussion[], costFooterOrOptions?: string | UpsertSummaryOptions): Promise<SummaryResult>;
+export type PostingMode = 'direct' | 'draft';
+export declare const POSTING_MODES: readonly PostingMode[];
+export declare const DRAFT_CONCURRENCY = 10;
+export interface PostResult {
+    posted: number;
+    drafts?: {
+        abandoned: number;
+        created: number;
+        deletedPrePublish: number;
+        published: number;
+        /**
+         * Drafts that survived the pre-publish race check but could not be
+         * published, counted only on the per-draft fallback path (a `bulk_publish`
+         * 500 forced individual publishes and some still failed). 0 on the normal
+         * path. A non-zero value means inline comments were silently dropped.
+         */
+        publishFailed: number;
+    };
+}
+export declare function postGeneratedComments(gitlab: GitLabClient, project: string, mr: string, generated: GeneratedComment[], mode?: PostingMode): Promise<PostResult>;
+//# sourceMappingURL=posting.d.ts.map

package/dist/posting.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"posting.d.ts","sourceRoot":"","sources":["../src/posting.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAE5D,OAAO,KAAK,EAAgB,gBAAgB,EAAE,eAAe,EAAE,MAAM,YAAY,CAAC;AAElF,eAAO,MAAM,cAAc,iCAAiC,CAAC;AAC7D,eAAO,MAAM,qBAAqB,+CAA+C,CAAC;AAClF,eAAO,MAAM,mBAAmB,6CAA6C,CAAC;AAC9E,eAAO,MAAM,2BAA2B,qDAAqD,CAAC;AAC9F,eAAO,MAAM,yBAAyB,mDAAmD,CAAC;AAC1F,eAAO,MAAM,qBAAqB,KAAK,CAAC;AAYxC;;;;;;;;GAQG;AACH,eAAO,MAAM,8BAA8B,QAC+I,CAAC;AAI3L,MAAM,MAAM,aAAa,GAAG,SAAS,GAAG,SAAS,CAAC;AAElD,MAAM,WAAW,aAAa;IAC5B,MAAM,EAAE,aAAa,CAAC;IACtB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,WAAW;IAC1B,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;CACd;AAED;;;;GAIG;AACH,MAAM,WAAW,UAAU;IACzB,mDAAmD;IACnD,gBAAgB,EAAE,eAAe,EAAE,CAAC;IACpC,wFAAwF;IACxF,aAAa,CAAC,EAAE;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,SAAS,EAAE,MAAM,CAAA;KAAE,CAAC;IACrD,8EAA8E;IAC9E,QAAQ,CAAC,EAAE;QAAE,aAAa,EAAE,MAAM,CAAC;QAAC,UAAU,EAAE,MAAM,CAAA;KAAE,CAAC;CAC1D;AAED,MAAM,WAAW,kBAAkB;IACjC,cAAc,CAAC,EAAE,MAAM,EAAE,CAAC;IAC1B,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,8EAA8E;IAC9E,UAAU,CAAC,EAAE,UAAU,CAAC;CACzB;AAED,MAAM,WAAW,oBAAqB,SAAQ,kBAAkB;IAC9D,UAAU,CAAC,EAAE,IAAI,CAAC;IAClB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAOD;;;;GAIG;AACH,wBAAgB,oBAAoB,CAAC,MAAM,CAAC,EAAE,UAAU,GAAG,MAAM,CAqChE;AAED,wBAAgB,gBAAgB,CAC9B,OAAO,EAAE,MAAM,EACf,UAAU,CAAC,EAAE,MAAM,EACnB,OAAO,GAAE,kBAAuB,GAC/B,MAAM,CAeR;AAED,wBAAgB,yBAAyB,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,CAEnE;AAED,wBAAgB,wBAAwB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAEpE;AAED,wBAAgB,6BAA6B,CAAC,WAAW,EAAE,UAAU,EAAE,GAAG,MAAM,GAAG,IAAI,CAGtF;AAED,wBAAgB,uBAAuB,CAAC,WAAW,EAAE,UAAU,EAAE,GAAG,WAAW,GAAG,IAAI,CAerF;AAED,wBAAgB,yBAAyB,CAAC,WAAW,EAAE,UAAU,EAAE,GAAG,MAAM,GAAG,IAAI,CAElF;AAED,wBAAgB,yBAAyB,CAAC,IAAI,EAAE,MAAM,EAAE,UAAU,OAAa,GAAG,MAAM,CASvF;AAED,wBAAgB,4BAA4B,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,EAAE,CAenE;AAED,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAqBxD;AAED,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAEvD;AAED,wBAAgB,0BAA0B,CACxC,YAAY,EAAE,MAAM,EACpB,UAAU,OAAa,GACtB,MAAM,EAAE,CAOV;AAED;;;;;;GAMG;AACH,wBAAgB,kBAAkB,CAChC,OAAO,EAAE,MAAM,EACf,WAAW,EAAE,UAAU,EAAE,EACzB,OAAO,EAAE,oBAAoB,GAC5B;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,QAAQ,EAAE,WAAW,GAAG,IAAI,CAAA;CAAE,CAYhD;AAED,wBAAsB,iBAAiB,CACrC,MAAM,EAAE,YAAY,EACpB,OAAO,EAAE,MAAM,EACf,EAAE,EAAE,MAAM,EACV,OAAO,EAAE,MAAM,EACf,WAAW,EAAE,UAAU,EAAE,EACzB,mBAAmB,CAAC,EAAE,MAAM,GAAG,oBAAoB,GAClD,OAAO,CAAC,aAAa,CAAC,CAYxB;AAkCD,MAAM,MAAM,WAAW,GAAG,QAAQ,GAAG,OAAO,CAAC;AAE7C,eAAO,MAAM,aAAa,EAAE,SAAS,WAAW,EAAwB,CAAC;AAEzE,eAAO,MAAM,iBAAiB,KAAK,CAAC;AAEpC,MAAM,WAAW,UAAU;IACzB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE;QACP,SAAS,EAAE,MAAM,CAAC;QAClB,OAAO,EAAE,MAAM,CAAC;QAChB,iBAAiB,EAAE,MAAM,CAAC;QAC1B,SAAS,EAAE,MAAM,CAAC;QAClB;;;;;WAKG;QACH,aAAa,EAAE,MAAM,CAAC;KACvB,CAAC;CACH;AAED,wBAAsB,qBAAqB,CACzC,MAAM,EAAE,YAAY,EACpB,OAAO,EAAE,MAAM,EACf,EAAE,EAAE,MAAM,EACV,SAAS,EAAE,gBAAgB,EAAE,EAC7B,IAAI,GAAE,WAAsB,GAC3B,OAAO,CAAC,UAAU,CAAC,CAKrB"}

package/dist/prior-threads.d.ts

@@ -0,0 +1,40 @@
+import type { Discussion, DiscussionNote } from './gitlab.js';
+export interface PriorThread {
+    file: string;
+    line: number | null;
+    resolved: boolean;
+    /** Bot comment body with fingerprint markers and emoji prefixes stripped. */
+    botComment: string;
+    /** Human (non-system) reply bodies, in order. */
+    replies: string[];
+}
+/**
+ * Returns true when the note body contains a code-review fingerprint marker
+ * (current or legacy prefix).
+ * Used to identify notes posted by the bot without needing a getCurrentUser() call.
+ */
+export declare function isBotNote(note: DiscussionNote): boolean;
+/**
+ * Parses the `+++ b/<path>` lines from a unified diff and returns the set of
+ * new file paths. `/dev/null` (deleted files) is excluded.
+ */
+export declare function extractChangedFiles(diff: string): Set<string>;
+/**
+ * Extracts prior review threads from existing MR discussions that are relevant
+ * to the current diff.
+ *
+ * A thread is included when:
+ * - It contains at least one bot note (identified by fingerprint marker).
+ * - It contains at least one non-system human reply after the bot note.
+ * - The thread's file appears in `changedFiles`.
+ *
+ * Resolved threads are included but marked with `resolved: true` so the
+ * reviewer can reference them without re-raising the concern.
+ */
+export declare function extractPriorThreads(discussions: Discussion[], changedFiles: Set<string>): PriorThread[];
+/**
+ * Renders a `<prior_review_feedback>` XML block from a list of prior threads.
+ * Returns an empty string when `threads` is empty.
+ */
+export declare function renderPriorThreadsBlock(threads: PriorThread[]): string;
+//# sourceMappingURL=prior-threads.d.ts.map

package/dist/prior-threads.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"prior-threads.d.ts","sourceRoot":"","sources":["../src/prior-threads.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,UAAU,EAAE,cAAc,EAAE,MAAM,aAAa,CAAC;AAI9D,MAAM,WAAW,WAAW;IAC1B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,QAAQ,EAAE,OAAO,CAAC;IAClB,6EAA6E;IAC7E,UAAU,EAAE,MAAM,CAAC;IACnB,iDAAiD;IACjD,OAAO,EAAE,MAAM,EAAE,CAAC;CACnB;AAED;;;;GAIG;AACH,wBAAgB,SAAS,CAAC,IAAI,EAAE,cAAc,GAAG,OAAO,CAEvD;AAED;;;GAGG;AACH,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,GAAG,CAAC,MAAM,CAAC,CAS7D;AAkBD;;;;;;;;;;;GAWG;AACH,wBAAgB,mBAAmB,CACjC,WAAW,EAAE,UAAU,EAAE,EACzB,YAAY,EAAE,GAAG,CAAC,MAAM,CAAC,GACxB,WAAW,EAAE,CAsCf;AAED;;;GAGG;AACH,wBAAgB,uBAAuB,CAAC,OAAO,EAAE,WAAW,EAAE,GAAG,MAAM,CAmBtE"}

package/dist/product.d.ts

@@ -0,0 +1,17 @@
+/**
+ * The published package name, shown in review footers. Independent of the review
+ * platform: the same tool posts to GitLab and GitHub, so the footer identifies
+ * the tool, not the backend. Kept as a single source of truth so the inline and
+ * summary footers can never drift apart.
+ */
+export declare const PRODUCT_NAME = "@weareikko/code-review";
+/** Canonical project URL used in the footer's markdown link. */
+export declare const PRODUCT_URL = "https://github.com/weareikko/code-review";
+/**
+ * The `[name](url)` markdown link used verbatim in both the inline comment footer
+ * ({@link buildCommentBody}) and the reviewed-commit summary footer
+ * ({@link buildReviewedCommitFooter}). Changing this changes the reviewed-commit
+ * footer format, which is guarded by a migration test.
+ */
+export declare const PRODUCT_LINK = "[@weareikko/code-review](https://github.com/weareikko/code-review)";
+//# sourceMappingURL=product.d.ts.map

package/dist/product.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"product.d.ts","sourceRoot":"","sources":["../src/product.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AACH,eAAO,MAAM,YAAY,2BAA2B,CAAC;AAErD,gEAAgE;AAChE,eAAO,MAAM,WAAW,6CAA6C,CAAC;AAEtE;;;;;GAKG;AACH,eAAO,MAAM,YAAY,uEAAsC,CAAC"}

package/dist/review.d.ts

@@ -0,0 +1,18 @@
+export type { DiagnosticContext, DiagnosticError, DiagnosticPhase, DiagnosticUsage, DiagnosticUsageBreakdown, } from './diagnostics.js';
+export { DIAGNOSTIC_CHANNEL_NAMES, DIAGNOSTIC_CHANNEL_PREFIX, createDiagnosticContext, createDiagnosticRunId, diagnosticChannels, traceDiagnostic, traceDiagnosticPhase, } from './diagnostics.js';
+export type { OtelBridge, OtelBridgeOptions, OtelRuntime } from './otel.js';
+export { isOtelEnabled, startOtelBridge } from './otel.js';
+export type { RunBridges, RunResult } from './cli.js';
+export { run } from './cli.js';
+export type { AgentLike, CreateAgent, CreateAgentParams, FilteredDiff, ModelUsage, ReviewIntent, ReviewSizeNotice, ReviewUsage, RunReviewOptions, UsageBreakdown, } from './gitlab-review.js';
+export { filterDiff } from './gitlab-review.js';
+export { runReview } from './gitlab-review.js';
+export type { DiffRefs, Fingerprints, GeneratedComment, GitLabDiscussionPayload, ReviewComment, Side, SizeSkippedFile, } from './types.js';
+export { normalizeSeverity, toGitLabReviewSeverity } from './types.js';
+export { parseReviewMarkdown, parseReviewMarkdownWithWarnings, type ParseFailure, type ParseFailureReason, type ParseResult, } from './parser.js';
+export { appendFingerprintMarkers, extractDiffHunkContext, extractExistingFingerprints, fingerprints, normalizeBody, sha256, } from './fingerprints.js';
+export { buildGeneratedComments, buildPayload } from './payloads.js';
+export type { LoadNamedSkillOptions, Skill, SkillSpec } from './skills.js';
+export { gitSkillCacheKey, loadNamedSkill, parseSkillSpec, resolveNpmSkillDir, resolveSkillCacheDir, } from './skills.js';
+export { SUMMARY_HISTORY_END, SUMMARY_HISTORY_ENTRY_END, SUMMARY_HISTORY_ENTRY_START, SUMMARY_HISTORY_LIMIT, SUMMARY_HISTORY_START, SUMMARY_MARKER, buildArchivedSummaryEntry, buildSizeNoticeBlock, buildSummaryBody, buildReviewedCommitFooter, buildSummaryHistoryEntries, extractReviewedCommitSha, extractSummaryHistoryEntries, findExistingReviewedCommitSha, findExistingSummaryNote, findExistingSummaryNoteId, stripSummaryHistory, stripSummaryMarker, upsertSummaryNote, type SizeNotice, type SummaryAction, type SummaryBodyOptions, type SummaryNote, type SummaryResult, type UpsertSummaryOptions, } from './posting.js';
+//# sourceMappingURL=review.d.ts.map

package/dist/review.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"review.d.ts","sourceRoot":"","sources":["../src/review.ts"],"names":[],"mappings":"AAAA,YAAY,EACV,iBAAiB,EACjB,eAAe,EACf,eAAe,EACf,eAAe,EACf,wBAAwB,GACzB,MAAM,kBAAkB,CAAC;AAC1B,OAAO,EACL,wBAAwB,EACxB,yBAAyB,EACzB,uBAAuB,EACvB,qBAAqB,EACrB,kBAAkB,EAClB,eAAe,EACf,oBAAoB,GACrB,MAAM,kBAAkB,CAAC;AAC1B,YAAY,EAAE,UAAU,EAAE,iBAAiB,EAAE,WAAW,EAAE,MAAM,WAAW,CAAC;AAC5E,OAAO,EAAE,aAAa,EAAE,eAAe,EAAE,MAAM,WAAW,CAAC;AAC3D,YAAY,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,UAAU,CAAC;AACtD,OAAO,EAAE,GAAG,EAAE,MAAM,UAAU,CAAC;AAC/B,YAAY,EACV,SAAS,EACT,WAAW,EACX,iBAAiB,EACjB,YAAY,EACZ,UAAU,EACV,YAAY,EACZ,gBAAgB,EAChB,WAAW,EACX,gBAAgB,EAChB,cAAc,GACf,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EAAE,UAAU,EAAE,MAAM,oBAAoB,CAAC;AAChD,OAAO,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAC;AAC/C,YAAY,EACV,QAAQ,EACR,YAAY,EACZ,gBAAgB,EAChB,uBAAuB,EACvB,aAAa,EACb,IAAI,EACJ,eAAe,GAChB,MAAM,YAAY,CAAC;AACpB,OAAO,EAAE,iBAAiB,EAAE,sBAAsB,EAAE,MAAM,YAAY,CAAC;AACvE,OAAO,EACL,mBAAmB,EACnB,+BAA+B,EAC/B,KAAK,YAAY,EACjB,KAAK,kBAAkB,EACvB,KAAK,WAAW,GACjB,MAAM,aAAa,CAAC;AACrB,OAAO,EACL,wBAAwB,EACxB,sBAAsB,EACtB,2BAA2B,EAC3B,YAAY,EACZ,aAAa,EACb,MAAM,GACP,MAAM,mBAAmB,CAAC;AAC3B,OAAO,EAAE,sBAAsB,EAAE,YAAY,EAAE,MAAM,eAAe,CAAC;AACrE,YAAY,EAAE,qBAAqB,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAC3E,OAAO,EACL,gBAAgB,EAChB,cAAc,EACd,cAAc,EACd,kBAAkB,EAClB,oBAAoB,GACrB,MAAM,aAAa,CAAC;AACrB,OAAO,EACL,mBAAmB,EACnB,yBAAyB,EACzB,2BAA2B,EAC3B,qBAAqB,EACrB,qBAAqB,EACrB,cAAc,EACd,yBAAyB,EACzB,oBAAoB,EACpB,gBAAgB,EAChB,yBAAyB,EACzB,0BAA0B,EAC1B,wBAAwB,EACxB,4BAA4B,EAC5B,6BAA6B,EAC7B,uBAAuB,EACvB,yBAAyB,EACzB,mBAAmB,EACnB,kBAAkB,EAClB,iBAAiB,EACjB,KAAK,UAAU,EACf,KAAK,aAAa,EAClB,KAAK,kBAAkB,EACvB,KAAK,WAAW,EAChB,KAAK,aAAa,EAClB,KAAK,oBAAoB,GAC1B,MAAM,cAAc,CAAC"}

package/dist/review.js

@@ -0,0 +1,2 @@
+import { $ as normalizeBody, A as SUMMARY_HISTORY_END, B as buildSummaryHistoryEntries, C as createDiagnosticContext, D as traceDiagnosticPhase, E as traceDiagnostic, F as SUMMARY_MARKER, G as findExistingSummaryNoteId, H as extractSummaryHistoryEntries, I as buildArchivedSummaryEntry, J as upsertSummaryNote, K as stripSummaryHistory, L as buildReviewedCommitFooter, M as SUMMARY_HISTORY_ENTRY_START, N as SUMMARY_HISTORY_LIMIT, O as normalizeSeverity, P as SUMMARY_HISTORY_START, Q as fingerprints, R as buildSizeNoticeBlock, S as DIAGNOSTIC_CHANNEL_PREFIX, T as diagnosticChannels, U as findExistingReviewedCommitSha, V as extractReviewedCommitSha, W as findExistingSummaryNote, X as extractDiffHunkContext, Y as appendFingerprintMarkers, Z as extractExistingFingerprints, _ as resolveNpmSkillDir, b as parseReviewMarkdownWithWarnings, c as buildGeneratedComments, d as startOtelBridge, et as sha256, f as filterDiff, g as parseSkillSpec, h as loadNamedSkill, j as SUMMARY_HISTORY_ENTRY_END, k as toGitLabReviewSeverity, l as buildPayload, m as gitSkillCacheKey, o as run, p as runReview, q as stripSummaryMarker, u as isOtelEnabled, v as resolveSkillCacheDir, w as createDiagnosticRunId, x as DIAGNOSTIC_CHANNEL_NAMES, y as parseReviewMarkdown, z as buildSummaryBody } from "./cli-7RGxyNy6.js";
+export { DIAGNOSTIC_CHANNEL_NAMES, DIAGNOSTIC_CHANNEL_PREFIX, SUMMARY_HISTORY_END, SUMMARY_HISTORY_ENTRY_END, SUMMARY_HISTORY_ENTRY_START, SUMMARY_HISTORY_LIMIT, SUMMARY_HISTORY_START, SUMMARY_MARKER, appendFingerprintMarkers, buildArchivedSummaryEntry, buildGeneratedComments, buildPayload, buildReviewedCommitFooter, buildSizeNoticeBlock, buildSummaryBody, buildSummaryHistoryEntries, createDiagnosticContext, createDiagnosticRunId, diagnosticChannels, extractDiffHunkContext, extractExistingFingerprints, extractReviewedCommitSha, extractSummaryHistoryEntries, filterDiff, findExistingReviewedCommitSha, findExistingSummaryNote, findExistingSummaryNoteId, fingerprints, gitSkillCacheKey, isOtelEnabled, loadNamedSkill, normalizeBody, normalizeSeverity, parseReviewMarkdown, parseReviewMarkdownWithWarnings, parseSkillSpec, resolveNpmSkillDir, resolveSkillCacheDir, run, runReview, sha256, startOtelBridge, stripSummaryHistory, stripSummaryMarker, toGitLabReviewSeverity, traceDiagnostic, traceDiagnosticPhase, upsertSummaryNote };

package/dist/skills.d.ts

@@ -0,0 +1,92 @@
+export interface Skill {
+    name: string;
+    description: string;
+    /** Absolute path to the SKILL.md file. Used to reference skill content in prompts. */
+    filePath: string;
+    rootDir: string;
+    resourceDirs: string[];
+    source: 'builtin' | 'project' | 'npm' | 'file' | 'git';
+}
+/**
+ * A parsed skill spec descriptor. Produced by `parseSkillSpec`.
+ *
+ * - `builtin`  — bare name resolved from the package's bundled `skills/` dir
+ * - `npm`      — package in `node_modules`, optionally with a sub-directory
+ * - `file`     — explicit filesystem path (relative or absolute)
+ * - `git`      — shallow git clone at a pinned ref, optionally with a sub-directory
+ */
+export type SkillSpec = {
+    protocol: 'builtin';
+    name: string;
+} | {
+    protocol: 'npm';
+    packageName: string;
+    subpath: string;
+} | {
+    protocol: 'file';
+    path: string;
+} | {
+    protocol: 'git';
+    url: string;
+    ref: string;
+    subpath: string;
+};
+export declare function loadSkillFromDir(dirPath: string, source: Skill['source']): Promise<Skill | null>;
+export declare function resolveBuiltinSkillsDir(): string;
+export declare function loadBuiltinSkill(name: string): Promise<Skill | null>;
+export declare function loadAutoDiscoveredSkills(cwd: string, gitRoot: string, warn?: (msg: string) => void): Promise<Skill[]>;
+/**
+ * Parse a skill spec string into a typed `SkillSpec` descriptor.
+ *
+ * Supported spec formats:
+ *
+ * | Input                              | Result                                            |
+ * |------------------------------------|---------------------------------------------------|
+ * | `code-review`                      | `{ protocol: 'builtin', name: 'code-review' }`    |
+ * | `npm:my-skill`                     | `{ protocol: 'npm', packageName: 'my-skill', ... }`|
+ * | `npm:@scope/pkg`                   | `{ protocol: 'npm', packageName: '@scope/pkg', ... }`|
+ * | `npm:@scope/bundle/security`       | `{ protocol: 'npm', packageName: '@scope/bundle', subpath: 'security' }`|
+ * | `npm:bundle/security`              | `{ protocol: 'npm', packageName: 'bundle', subpath: 'security' }`|
+ * | `file:./path/to/skill`             | `{ protocol: 'file', path: './path/to/skill' }`   |
+ * | `file:/absolute/path`              | `{ protocol: 'file', path: '/absolute/path' }`    |
+ * | `git:https://host/org/s.git`       | `{ protocol: 'git', url: 'https://host/org/s.git', ref: '', subpath: '' }` |
+ * | `git:https://host/org/b.git#v1/sec`| `{ protocol: 'git', url: 'https://host/org/b.git', ref: 'v1', subpath: 'sec' }` |
+ * | `git+ssh://git@host/org/s.git`     | `{ protocol: 'git', url: 'ssh://git@host/org/s.git', ref: '', subpath: '' }` |
+ */
+export declare function parseSkillSpec(spec: string): SkillSpec;
+/**
+ * Resolve the directory for an npm-installed skill by walking `node_modules`
+ * upward from `cwd` (supports monorepo hoisting). Returns the resolved path
+ * or `null` if the package / subpath cannot be found.
+ */
+export declare function resolveNpmSkillDir(packageName: string, subpath: string, cwd: string): Promise<string | null>;
+/** Base directory for cached git-skill clones (honours `XDG_CACHE_HOME`). */
+export declare function resolveSkillCacheDir(): string;
+/**
+ * Stable cache-directory name for a git skill. Keyed on the clone URL plus the
+ * pinned ref so that two refs of the same repo never share a cache entry.
+ */
+export declare function gitSkillCacheKey(url: string, ref: string): string;
+/** Options controlling how external skills are resolved. */
+export interface LoadNamedSkillOptions {
+    /** Override the git-skill clone cache directory (defaults to the XDG cache). */
+    cacheDir?: string;
+    /** Re-clone git skills even when a cached copy exists. */
+    refresh?: boolean;
+}
+/**
+ * Load a skill by its spec string (`code-review`, `npm:@scope/pkg`, `file:./path`,
+ * `git:https://…`, …).
+ *
+ * Resolution order:
+ * 1. `builtin` — package-bundled `skills/<name>/`
+ * 2. `npm:`    — `node_modules/<packageName>[/subpath]` walked up from `cwd`
+ * 3. `file:`   — direct filesystem path (relative paths resolved from `cwd`)
+ * 4. `git:` / `git+ssh:` — shallow clone at the pinned ref, cached on disk,
+ *    loading `SKILL.md` from the repo root or the `#<ref>/<subpath>` directory
+ *
+ * Throws a `ConfigError` if the spec cannot be resolved or the resolved
+ * directory does not contain a valid `SKILL.md`.
+ */
+export declare function loadNamedSkill(spec: string, cwd: string, options?: LoadNamedSkillOptions): Promise<Skill>;
+//# sourceMappingURL=skills.d.ts.map

package/dist/skills.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"skills.d.ts","sourceRoot":"","sources":["../src/skills.ts"],"names":[],"mappings":"AAUA,MAAM,WAAW,KAAK;IACpB,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,MAAM,CAAC;IACpB,sFAAsF;IACtF,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,MAAM,EAAE,SAAS,GAAG,SAAS,GAAG,KAAK,GAAG,MAAM,GAAG,KAAK,CAAC;CACxD;AAED;;;;;;;GAOG;AACH,MAAM,MAAM,SAAS,GACjB;IAAE,QAAQ,EAAE,SAAS,CAAC;IAAC,IAAI,EAAE,MAAM,CAAA;CAAE,GACrC;IAAE,QAAQ,EAAE,KAAK,CAAC;IAAC,WAAW,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,GACzD;IAAE,QAAQ,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,MAAM,CAAA;CAAE,GAClC;IAAE,QAAQ,EAAE,KAAK,CAAC;IAAC,GAAG,EAAE,MAAM,CAAC;IAAC,GAAG,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC;AA0BnE,wBAAsB,gBAAgB,CACpC,OAAO,EAAE,MAAM,EACf,MAAM,EAAE,KAAK,CAAC,QAAQ,CAAC,GACtB,OAAO,CAAC,KAAK,GAAG,IAAI,CAAC,CAmBvB;AAED,wBAAgB,uBAAuB,IAAI,MAAM,CAEhD;AAED,wBAAsB,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,KAAK,GAAG,IAAI,CAAC,CAE1E;AAED,wBAAsB,wBAAwB,CAC5C,GAAG,EAAE,MAAM,EACX,OAAO,EAAE,MAAM,EACf,IAAI,CAAC,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,IAAI,GAC3B,OAAO,CAAC,KAAK,EAAE,CAAC,CAqClB;AAED;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,cAAc,CAAC,IAAI,EAAE,MAAM,GAAG,SAAS,CAqCtD;AA4CD;;;;GAIG;AACH,wBAAsB,kBAAkB,CACtC,WAAW,EAAE,MAAM,EACnB,OAAO,EAAE,MAAM,EACf,GAAG,EAAE,MAAM,GACV,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAYxB;AAED,6EAA6E;AAC7E,wBAAgB,oBAAoB,IAAI,MAAM,CAG7C;AAED;;;GAGG;AACH,wBAAgB,gBAAgB,CAAC,GAAG,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,MAAM,CAEjE;AA2DD,4DAA4D;AAC5D,MAAM,WAAW,qBAAqB;IACpC,gFAAgF;IAChF,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,0DAA0D;IAC1D,OAAO,CAAC,EAAE,OAAO,CAAC;CACnB;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAsB,cAAc,CAClC,IAAI,EAAE,MAAM,EACZ,GAAG,EAAE,MAAM,EACX,OAAO,GAAE,qBAA0B,GAClC,OAAO,CAAC,KAAK,CAAC,CAoEhB"}

package/dist/skipped-retrieval.d.ts

@@ -0,0 +1,26 @@
+/** Directory (relative to cwd) where dropped-file diffs are staged for retrieval. */
+export declare const SKIPPED_DIFF_DIR = ".code-review-skipped";
+export interface SkippedDiffFile {
+    /** Original repository path of the dropped file. */
+    path: string;
+    /** Path to the staged diff, relative to cwd — what the agent passes to its read tool. */
+    diskPath: string;
+}
+/**
+ * Write each size-dropped file's diff to `<cwd>/.code-review-skipped/` so an
+ * agentic reviewer can read the diffs it deems highest-risk on demand, instead
+ * of the diffs being lost to the char budget. Returns the on-disk manifest.
+ * Paths are staged under cwd because the reviewer's read tool is cwd-scoped.
+ */
+export declare function writeSkippedDiffs(cwd: string, sections: Array<{
+    path: string;
+    section: string;
+}>): Promise<SkippedDiffFile[]>;
+/** Remove the staged-diff directory. Safe to call when it was never created. */
+export declare function cleanupSkippedDiffs(cwd: string): Promise<void>;
+/**
+ * Render the `<skipped_files>` block for the retrieval mode: each dropped file
+ * with its on-disk diff path and an instruction to read the highest-risk ones.
+ */
+export declare function renderRetrievableSkippedBlock(files: SkippedDiffFile[]): string;
+//# sourceMappingURL=skipped-retrieval.d.ts.map

package/dist/skipped-retrieval.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"skipped-retrieval.d.ts","sourceRoot":"","sources":["../src/skipped-retrieval.ts"],"names":[],"mappings":"AAGA,qFAAqF;AACrF,eAAO,MAAM,gBAAgB,yBAAyB,CAAC;AAEvD,MAAM,WAAW,eAAe;IAC9B,oDAAoD;IACpD,IAAI,EAAE,MAAM,CAAC;IACb,yFAAyF;IACzF,QAAQ,EAAE,MAAM,CAAC;CAClB;AAOD;;;;;GAKG;AACH,wBAAsB,iBAAiB,CACrC,GAAG,EAAE,MAAM,EACX,QAAQ,EAAE,KAAK,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC,GACjD,OAAO,CAAC,eAAe,EAAE,CAAC,CAW5B;AAED,gFAAgF;AAChF,wBAAsB,mBAAmB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAEpE;AAED;;;GAGG;AACH,wBAAgB,6BAA6B,CAAC,KAAK,EAAE,eAAe,EAAE,GAAG,MAAM,CAG9E"}

package/dist/summary-carryover.d.ts

@@ -0,0 +1,48 @@
+import type { Discussion } from './gitlab.js';
+import type { Severity } from './types.js';
+/**
+ * A bot-posted inline finding from a previous review run whose thread is still
+ * open (unresolved). Carried into the current summary so an unresolved thread
+ * never silently disappears from the issue list when a later run doesn't
+ * re-emit it (#92).
+ */
+export interface CarryOverFinding {
+    file: string;
+    line: number | null;
+    severity: Severity;
+    /** Conventional Comment label incl. decoration, e.g. `issue (blocking)`. */
+    header: string;
+    subject: string;
+    /** All fingerprint hashes found in the bot note (primary + secondary). */
+    hashes: string[];
+}
+/**
+ * Extract still-open (unresolved) bot-posted inline findings from prior MR
+ * discussions. A discussion qualifies when its first bot note carries fingerprint
+ * markers, sits on a file position, and no note in it is resolved.
+ */
+export declare function extractOpenBotFindings(discussions: Discussion[]): CarryOverFinding[];
+/**
+ * From the still-open prior findings, keep only those the current run did NOT
+ * re-emit — i.e. none of their fingerprints appear in `currentFingerprints`.
+ * A re-emitted finding is already represented (posted or deduplicated), so it
+ * must not be double-listed.
+ */
+export declare function selectCarryOver(openFindings: CarryOverFinding[], currentFingerprints: Set<string>): CarryOverFinding[];
+/**
+ * Fold carried-over findings into a summary so unresolved prior threads stay
+ * visible and the risk line never drops below a still-open finding's level:
+ *
+ * 1. Bump the `**Risk: …**` line up (never down) if a carry-over outranks it.
+ * 2. Append a `**Still open from earlier reviews (N):**` block listing them.
+ *
+ * Returns the summary unchanged when there is nothing to carry over, so
+ * single-run / first-run output is byte-identical.
+ */
+export declare function applyCarryOverToSummary(summary: string, carryOvers: CarryOverFinding[]): string;
+/**
+ * Convenience wrapper: extract still-open prior findings, drop the ones the
+ * current run re-emitted, and fold the rest into the summary.
+ */
+export declare function withCarriedOverFindings(summary: string, discussions: Discussion[], currentFingerprints: Set<string>): string;
+//# sourceMappingURL=summary-carryover.d.ts.map

package/dist/summary-carryover.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"summary-carryover.d.ts","sourceRoot":"","sources":["../src/summary-carryover.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,UAAU,EAAkB,MAAM,aAAa,CAAC;AAE9D,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,YAAY,CAAC;AAK3C;;;;;GAKG;AACH,MAAM,WAAW,gBAAgB;IAC/B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,QAAQ,EAAE,QAAQ,CAAC;IACnB,4EAA4E;IAC5E,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,MAAM,CAAC;IAChB,0EAA0E;IAC1E,MAAM,EAAE,MAAM,EAAE,CAAC;CAClB;AA2CD;;;;GAIG;AACH,wBAAgB,sBAAsB,CAAC,WAAW,EAAE,UAAU,EAAE,GAAG,gBAAgB,EAAE,CAwBpF;AAED;;;;;GAKG;AACH,wBAAgB,eAAe,CAC7B,YAAY,EAAE,gBAAgB,EAAE,EAChC,mBAAmB,EAAE,GAAG,CAAC,MAAM,CAAC,GAC/B,gBAAgB,EAAE,CAEpB;AASD;;;;;;;;;GASG;AACH,wBAAgB,uBAAuB,CAAC,OAAO,EAAE,MAAM,EAAE,UAAU,EAAE,gBAAgB,EAAE,GAAG,MAAM,CAsB/F;AAED;;;GAGG;AACH,wBAAgB,uBAAuB,CACrC,OAAO,EAAE,MAAM,EACf,WAAW,EAAE,UAAU,EAAE,EACzB,mBAAmB,EAAE,GAAG,CAAC,MAAM,CAAC,GAC/B,MAAM,CAGR"}