vouchington-tooling 0.13.2 → 0.13.4

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (36) hide show
  1. package/README.md +5 -0
  2. package/ast-grep/rules/no-three-sequential-awaits-tsx.yml +178 -0
  3. package/ast-grep/rules/no-three-sequential-awaits.yml +169 -0
  4. package/dist/ast-grep-examples/companion-parity.d.mts +19 -0
  5. package/dist/ast-grep-examples/companion-parity.mjs +156 -0
  6. package/dist/ast-grep-examples/index.d.mts +2 -1
  7. package/dist/ast-grep-examples/index.mjs +1 -0
  8. package/dist/coverage-transport/outcome.d.mts +6 -5
  9. package/dist/coverage-transport/outcome.mjs +6 -8
  10. package/dist/gha-check-run/cli.d.mts +2 -0
  11. package/dist/gha-check-run/cli.mjs +81 -0
  12. package/dist/gha-check-run/github.d.mts +30 -0
  13. package/dist/gha-check-run/github.mjs +63 -0
  14. package/dist/gha-check-run/index.d.mts +2 -0
  15. package/dist/gha-check-run/index.mjs +2 -0
  16. package/dist/gha-post-review/cli.mjs +1 -1
  17. package/dist/gha-post-review/github.d.mts +6 -22
  18. package/dist/gha-post-review/github.mjs +8 -92
  19. package/dist/gha-post-review/post.d.mts +7 -1
  20. package/dist/gha-post-review/post.mjs +11 -3
  21. package/dist/gha-post-review/pull-io.d.mts +13 -0
  22. package/dist/gha-post-review/pull-io.mjs +88 -0
  23. package/dist/index.d.mts +2 -1
  24. package/dist/index.mjs +1 -1
  25. package/dist/scc-complexity/baseline.mjs +5 -2
  26. package/dist/scc-complexity/constants.d.mts +1 -0
  27. package/dist/scc-complexity/constants.mjs +1 -0
  28. package/dist/scc-complexity/index.d.mts +1 -1
  29. package/dist/scc-complexity/index.mjs +24 -36
  30. package/dist/scc-complexity/paths.d.mts +2 -1
  31. package/dist/scc-complexity/paths.mjs +35 -2
  32. package/dist/scc-complexity/scope-config.d.mts +3 -0
  33. package/dist/scc-complexity/scope-config.mjs +35 -0
  34. package/dist/scc-complexity/workflow-command.d.mts +2 -0
  35. package/dist/scc-complexity/workflow-command.mjs +6 -0
  36. package/package.json +6 -1
@@ -0,0 +1,81 @@
1
+ import { appendFileSync } from 'node:fs';
2
+ import { fileURLToPath } from 'node:url';
3
+ import { createGhExec } from '../gha-post-review/github.mjs';
4
+ import { completeCheckRun, createCheckRun, CheckRunError, } from './github.mjs';
5
+ function requireEnv(name, env) {
6
+ const value = env[name];
7
+ if (!value)
8
+ throw new CheckRunError(`${name} is required.`);
9
+ return value;
10
+ }
11
+ function parseStatus(value) {
12
+ if (value !== 'in_progress' && value !== 'completed') {
13
+ throw new CheckRunError(`STATUS must be "in_progress" or "completed" (got "${value}").`);
14
+ }
15
+ return value;
16
+ }
17
+ function parseConclusion(value) {
18
+ if (value !== 'success' && value !== 'neutral') {
19
+ throw new CheckRunError(`CONCLUSION must be "success" or "neutral" (got "${value}").`);
20
+ }
21
+ return value;
22
+ }
23
+ function appendOutput(name, value, outputPath) {
24
+ if (!outputPath)
25
+ return;
26
+ appendFileSync(outputPath, `${name}=${value}\n`);
27
+ }
28
+ /** Builds a create-check-run input from env, requiring CONCLUSION only when STATUS is completed. */
29
+ function readCreateInput(env) {
30
+ const status = parseStatus(requireEnv('STATUS', env));
31
+ const shared = {
32
+ repository: requireEnv('GITHUB_REPOSITORY', env),
33
+ name: requireEnv('CHECK_NAME', env),
34
+ headSha: requireEnv('HEAD_SHA', env),
35
+ title: requireEnv('TITLE', env),
36
+ summary: requireEnv('SUMMARY', env),
37
+ };
38
+ if (status === 'completed') {
39
+ return { ...shared, status, conclusion: parseConclusion(requireEnv('CONCLUSION', env)) };
40
+ }
41
+ return { ...shared, status };
42
+ }
43
+ function runCreate(env, exec) {
44
+ const id = createCheckRun(readCreateInput(env), exec);
45
+ appendOutput('check_run_id', id, env.GITHUB_OUTPUT);
46
+ }
47
+ /** No-ops when CHECK_RUN_ID is unset: the create step may have already finalized the check. */
48
+ function runComplete(env, exec) {
49
+ const checkRunId = env.CHECK_RUN_ID ?? '';
50
+ if (checkRunId === '')
51
+ return;
52
+ completeCheckRun({
53
+ repository: requireEnv('GITHUB_REPOSITORY', env),
54
+ checkRunId,
55
+ conclusion: parseConclusion(requireEnv('CONCLUSION', env)),
56
+ title: requireEnv('TITLE', env),
57
+ summary: requireEnv('SUMMARY', env),
58
+ }, exec);
59
+ }
60
+ export function runCheckRunCli(argv = process.argv, env = process.env, exec = createGhExec()) {
61
+ const subcommand = argv[2];
62
+ try {
63
+ if (subcommand === 'create') {
64
+ runCreate(env, exec);
65
+ return 0;
66
+ }
67
+ if (subcommand === 'complete') {
68
+ runComplete(env, exec);
69
+ return 0;
70
+ }
71
+ throw new CheckRunError(`Unknown subcommand "${String(subcommand)}". Use create or complete.`);
72
+ }
73
+ catch (error) {
74
+ process.stderr.write(`Error: ${error instanceof Error ? error.message : String(error)}\n`);
75
+ return 1;
76
+ }
77
+ }
78
+ /* v8 ignore next 3 */
79
+ if (process.argv[1] === fileURLToPath(import.meta.url)) {
80
+ process.exitCode = runCheckRunCli();
81
+ }
@@ -0,0 +1,30 @@
1
+ import type { GhExec } from '../gha-post-review/github.mts';
2
+ export declare class CheckRunError extends Error {
3
+ constructor(message: string);
4
+ }
5
+ export type CheckRunStatus = 'in_progress' | 'completed';
6
+ export type CheckRunConclusion = 'success' | 'neutral';
7
+ export type CreateCheckRunInput = {
8
+ repository: string;
9
+ name: string;
10
+ headSha: string;
11
+ title: string;
12
+ summary: string;
13
+ } & ({
14
+ status: 'in_progress';
15
+ conclusion?: undefined;
16
+ } | {
17
+ status: 'completed';
18
+ conclusion: CheckRunConclusion;
19
+ });
20
+ export type CompleteCheckRunInput = {
21
+ repository: string;
22
+ checkRunId: string;
23
+ conclusion: CheckRunConclusion;
24
+ title: string;
25
+ summary: string;
26
+ };
27
+ /** Creates a GitHub check run, returning the API-assigned check run id. */
28
+ export declare function createCheckRun(input: CreateCheckRunInput, exec: GhExec): string;
29
+ /** Transitions an existing check run to `completed` with a final conclusion. */
30
+ export declare function completeCheckRun(input: CompleteCheckRunInput, exec: GhExec): void;
@@ -0,0 +1,63 @@
1
+ export class CheckRunError extends Error {
2
+ constructor(message) {
3
+ super(message);
4
+ this.name = 'CheckRunError';
5
+ }
6
+ }
7
+ function isFullCommitSha(value) {
8
+ return /^[0-9a-f]{40}$/u.test(value);
9
+ }
10
+ function isCheckRunId(value) {
11
+ return /^[0-9]+$/u.test(value);
12
+ }
13
+ function parseCheckRunId(raw) {
14
+ let parsed;
15
+ try {
16
+ parsed = JSON.parse(raw);
17
+ }
18
+ catch {
19
+ throw new CheckRunError('Check run creation response was not valid JSON.');
20
+ }
21
+ const id = parsed !== null && typeof parsed === 'object' ? parsed.id : undefined;
22
+ if (typeof id !== 'number' && typeof id !== 'string') {
23
+ throw new CheckRunError('Check run creation response did not include an id.');
24
+ }
25
+ return String(id);
26
+ }
27
+ /** Creates a GitHub check run, returning the API-assigned check run id. */
28
+ export function createCheckRun(input, exec) {
29
+ if (!isFullCommitSha(input.headSha)) {
30
+ throw new CheckRunError('headSha must be a full lowercase 40-character commit SHA.');
31
+ }
32
+ const body = {
33
+ name: input.name,
34
+ head_sha: input.headSha,
35
+ status: input.status,
36
+ output: { title: input.title, summary: input.summary },
37
+ };
38
+ if (input.status === 'completed')
39
+ body.conclusion = input.conclusion;
40
+ const raw = exec(['api', '--method', 'POST', `repos/${input.repository}/check-runs`, '--input', '-'], {
41
+ input: JSON.stringify(body),
42
+ });
43
+ return parseCheckRunId(raw);
44
+ }
45
+ /** Transitions an existing check run to `completed` with a final conclusion. */
46
+ export function completeCheckRun(input, exec) {
47
+ if (!isCheckRunId(input.checkRunId)) {
48
+ throw new CheckRunError('checkRunId must be a positive integer.');
49
+ }
50
+ const body = {
51
+ status: 'completed',
52
+ conclusion: input.conclusion,
53
+ output: { title: input.title, summary: input.summary },
54
+ };
55
+ exec([
56
+ 'api',
57
+ '--method',
58
+ 'PATCH',
59
+ `repos/${input.repository}/check-runs/${input.checkRunId}`,
60
+ '--input',
61
+ '-',
62
+ ], { input: JSON.stringify(body) });
63
+ }
@@ -0,0 +1,2 @@
1
+ export { CheckRunError, completeCheckRun, createCheckRun, type CheckRunConclusion, type CheckRunStatus, type CompleteCheckRunInput, type CreateCheckRunInput, } from './github.mts';
2
+ export { runCheckRunCli } from './cli.mts';
@@ -0,0 +1,2 @@
1
+ export { CheckRunError, completeCheckRun, createCheckRun, } from './github.mjs';
2
+ export { runCheckRunCli } from './cli.mjs';
@@ -3,7 +3,7 @@ import { writePostedOutput, postReviewFromEnv } from './github.mjs';
3
3
  export async function runPostReviewCli(env = process.env) {
4
4
  try {
5
5
  const result = await postReviewFromEnv(env);
6
- writePostedOutput(result.posted, env.GITHUB_OUTPUT);
6
+ writePostedOutput(result.posted, result.commentCount, env.GITHUB_OUTPUT);
7
7
  return 0;
8
8
  }
9
9
  catch (error) {
@@ -1,29 +1,13 @@
1
1
  import { execFileSync } from 'node:child_process';
2
- import { type PostResult, type PostReviewIo, type SanitizedReview } from './post.mts';
2
+ import { type PostReviewResult } from './post.mts';
3
+ export { createGhPostReviewIo, postWithGh } from './pull-io.mts';
3
4
  export type GhExec = (args: readonly string[], options?: {
4
5
  input?: string;
5
6
  env?: NodeJS.ProcessEnv;
6
7
  }) => string;
7
8
  export declare function createGhExec(exec?: typeof execFileSync): GhExec;
8
- export declare function postWithGh(repository: string, prNumber: string, payload: SanitizedReview, token: string, exec: GhExec): PostResult;
9
- export declare function writePostedOutput(posted: boolean, outputPath?: string | undefined): void;
10
- export declare function createGhPostReviewIo(options: {
11
- repository: string;
12
- prNumber: string;
13
- payloadPath: string;
14
- payloadBytes: Buffer;
15
- token: string;
16
- exec: GhExec;
17
- expectedHeadSha?: string;
18
- expectedBaseSha?: string;
19
- }): PostReviewIo;
9
+ export declare function writePostedOutput(posted: boolean, commentCount: number, outputPath?: string | undefined): void;
20
10
  /** @deprecated Use postReviewWithTokenFromEnv or postClaudeReviewFromEnv explicitly. */
21
- export declare function postReviewFromEnv(env?: NodeJS.ProcessEnv, exec?: GhExec, claudeIo?: import("./claude-token.mts").ClaudeTokenIo): Promise<{
22
- posted: boolean;
23
- }>;
24
- export declare function postReviewWithTokenFromEnv(env?: NodeJS.ProcessEnv, exec?: GhExec, token?: string | undefined): {
25
- posted: boolean;
26
- };
27
- export declare function postClaudeReviewFromEnv(env?: NodeJS.ProcessEnv, exec?: GhExec, claudeIo?: import("./claude-token.mts").ClaudeTokenIo): Promise<{
28
- posted: boolean;
29
- }>;
11
+ export declare function postReviewFromEnv(env?: NodeJS.ProcessEnv, exec?: GhExec, claudeIo?: import("./claude-token.mts").ClaudeTokenIo): Promise<PostReviewResult>;
12
+ export declare function postReviewWithTokenFromEnv(env?: NodeJS.ProcessEnv, exec?: GhExec, token?: string | undefined): PostReviewResult;
13
+ export declare function postClaudeReviewFromEnv(env?: NodeJS.ProcessEnv, exec?: GhExec, claudeIo?: import("./claude-token.mts").ClaudeTokenIo): Promise<PostReviewResult>;
@@ -1,11 +1,12 @@
1
1
  import { execFileSync } from 'node:child_process';
2
- import { appendFileSync, rmSync } from 'node:fs';
3
- import { parseReviewFilesJson } from '../gha-review-payload/index.mjs';
2
+ import { appendFileSync } from 'node:fs';
4
3
  import { ReviewPayloadError } from '../gha-review-payload/index.mjs';
4
+ import { readRegularReviewPayload } from '../gha-review-payload/index.mjs';
5
5
  import { createActionsClaudeTokenIo, withClaudeAppToken } from './claude-token.mjs';
6
6
  import { runPostReview } from './post.mjs';
7
+ import { createGhPostReviewIo } from './pull-io.mjs';
7
8
  import { requireEnv, resolveReviewPostToken } from './token.mjs';
8
- import { readRegularReviewPayload } from '../gha-review-payload/index.mjs';
9
+ export { createGhPostReviewIo, postWithGh } from './pull-io.mjs';
9
10
  export function createGhExec(exec = execFileSync) {
10
11
  return (args, options) => exec('gh', [...args], {
11
12
  encoding: 'utf8',
@@ -13,96 +14,10 @@ export function createGhExec(exec = execFileSync) {
13
14
  env: options?.env ?? process.env,
14
15
  }).trim();
15
16
  }
16
- export function postWithGh(repository, prNumber, payload, token, exec) {
17
- try {
18
- exec(['api', '--method', 'POST', `repos/${repository}/pulls/${prNumber}/reviews`, '--input', '-'], {
19
- input: JSON.stringify(payload),
20
- env: { ...process.env, GH_TOKEN: token, GITHUB_TOKEN: token },
21
- });
22
- return { ok: true, status: 201, body: '' };
23
- }
24
- catch (error) {
25
- // oxlint-disable-next-line no-mistakes/ts-no-const-aliases -- isolate the process-error shape assertion at the catch boundary
26
- const err = error;
27
- const text = `${err.stdout ?? ''}${err.stderr ?? ''}${err.message ?? ''}`;
28
- const status = Number(/HTTP\s+(\d{3})/u.exec(text)?.[1] ?? 0);
29
- return { ok: false, status, body: text };
30
- }
31
- }
32
- export function writePostedOutput(posted, outputPath = process.env.GITHUB_OUTPUT) {
17
+ export function writePostedOutput(posted, commentCount, outputPath = process.env.GITHUB_OUTPUT) {
33
18
  if (!outputPath)
34
19
  return;
35
- appendFileSync(outputPath, `posted=${posted ? 'true' : 'false'}\n`);
36
- }
37
- function readPullState(repository, prNumber, exec) {
38
- let lastError;
39
- for (let attempt = 1; attempt <= 3; attempt += 1) {
40
- try {
41
- return exec([
42
- 'api',
43
- `repos/${repository}/pulls/${prNumber}`,
44
- '--jq',
45
- '[.head.sha, .base.sha, .draft, .state] | @tsv',
46
- ]).split('\t');
47
- }
48
- catch (error) {
49
- lastError = error;
50
- }
51
- }
52
- throw lastError;
53
- }
54
- export function createGhPostReviewIo(options) {
55
- const { repository, prNumber, payloadBytes, token, exec, expectedHeadSha, expectedBaseSha } = options;
56
- return {
57
- readFile() {
58
- return payloadBytes;
59
- },
60
- removeFile(path) {
61
- rmSync(path, { force: true });
62
- },
63
- getHeadSha() {
64
- if (!/^[1-9][0-9]*$/u.test(prNumber)) {
65
- throw new ReviewPayloadError('PR_NUMBER must be a positive integer.');
66
- }
67
- if (Boolean(expectedHeadSha) !== Boolean(expectedBaseSha)) {
68
- throw new ReviewPayloadError('EXPECTED_HEAD_SHA and EXPECTED_BASE_SHA must be provided together.');
69
- }
70
- if (expectedHeadSha && !/^[0-9a-f]{40}$/u.test(expectedHeadSha)) {
71
- throw new ReviewPayloadError('EXPECTED_HEAD_SHA must be a full lowercase commit SHA.');
72
- }
73
- if (expectedBaseSha && !/^[0-9a-f]{40}$/u.test(expectedBaseSha)) {
74
- throw new ReviewPayloadError('EXPECTED_BASE_SHA must be a full lowercase commit SHA.');
75
- }
76
- const state = readPullState(repository, prNumber, exec);
77
- const [headSha = '', baseSha = '', draft = '', pullState = ''] = state;
78
- if (!/^[0-9a-f]{40}$/u.test(headSha)) {
79
- throw new ReviewPayloadError(`Could not resolve PR head SHA (got "${headSha}").`);
80
- }
81
- if (!/^[0-9a-f]{40}$/u.test(baseSha)) {
82
- throw new ReviewPayloadError(`Could not resolve PR base SHA (got "${baseSha}").`);
83
- }
84
- if (draft !== 'false') {
85
- throw new ReviewPayloadError('Pull request became a draft before posting the review.');
86
- }
87
- if (pullState !== 'open') {
88
- throw new ReviewPayloadError('Pull request closed before posting the review.');
89
- }
90
- if (expectedHeadSha && headSha !== expectedHeadSha) {
91
- throw new ReviewPayloadError('PR head changed before posting the selected review.');
92
- }
93
- if (expectedBaseSha && baseSha !== expectedBaseSha) {
94
- throw new ReviewPayloadError('PR base changed before posting the selected review.');
95
- }
96
- // Preserve the orchestrator-selected revision as the review commit_id after equality checks.
97
- return expectedHeadSha || headSha;
98
- },
99
- listPullFiles() {
100
- return parseReviewFilesJson(exec(['api', '--paginate', `repos/${repository}/pulls/${prNumber}/files?per_page=100`]));
101
- },
102
- postReview(payload) {
103
- return postWithGh(repository, prNumber, payload, token, exec);
104
- },
105
- };
20
+ appendFileSync(outputPath, `posted=${posted ? 'true' : 'false'}\ncomment_count=${commentCount}\n`);
106
21
  }
107
22
  /** @deprecated Use postReviewWithTokenFromEnv or postClaudeReviewFromEnv explicitly. */
108
23
  export async function postReviewFromEnv(env = process.env, exec = createGhExec(), claudeIo = createActionsClaudeTokenIo(env)) {
@@ -116,6 +31,7 @@ function createPostWithToken(env, exec) {
116
31
  const prNumber = requireEnv('PR_NUMBER', env);
117
32
  const payloadPath = requireEnv('CODE_REVIEW_PAYLOAD_PATH', env);
118
33
  const payloadBytes = readRegularReviewPayload(payloadPath, 'required');
34
+ const providerName = env.PROVIDER_NAME || undefined;
119
35
  return (token) => runPostReview(payloadPath, createGhPostReviewIo({
120
36
  repository,
121
37
  prNumber,
@@ -125,7 +41,7 @@ function createPostWithToken(env, exec) {
125
41
  exec,
126
42
  expectedHeadSha: env.EXPECTED_HEAD_SHA ?? '',
127
43
  expectedBaseSha: env.EXPECTED_BASE_SHA ?? '',
128
- }));
44
+ }), providerName);
129
45
  }
130
46
  function withTokenEnv(exec, env, token) {
131
47
  const tokenEnv = { ...env, GH_TOKEN: token, GITHUB_TOKEN: token };
@@ -13,6 +13,12 @@ export type PostReviewIo = {
13
13
  listPullFiles(): PullFile[];
14
14
  postReview(payload: SanitizedReview): PostResult;
15
15
  };
16
- export declare function runPostReview(payloadPath: string, io: PostReviewIo): {
16
+ export type PostReviewResult = {
17
17
  posted: boolean;
18
+ commentCount: number;
18
19
  };
20
+ /**
21
+ * Posts a validated review, optionally prefixing the body with a provider attribution line.
22
+ * The attribution is cosmetic (final posted text only); it never affects staged payload bytes.
23
+ */
24
+ export declare function runPostReview(payloadPath: string, io: PostReviewIo, providerName?: string): PostReviewResult;
@@ -6,7 +6,11 @@ MAX_REVIEW_COMMENTS as MAX_COMMENTS,
6
6
  MAX_REVIEW_PAYLOAD_BYTES as MAX_PAYLOAD_BYTES,
7
7
  // oxlint-disable-next-line no-mistakes/ts-no-export-renaming -- preserve the package's established public compatibility name
8
8
  ReviewPayloadError as PostReviewError, } from '../gha-review-payload/index.mjs';
9
- export function runPostReview(payloadPath, io) {
9
+ /**
10
+ * Posts a validated review, optionally prefixing the body with a provider attribution line.
11
+ * The attribution is cosmetic (final posted text only); it never affects staged payload bytes.
12
+ */
13
+ export function runPostReview(payloadPath, io, providerName) {
10
14
  try {
11
15
  const selectedHeadSha = io.getHeadSha();
12
16
  let review = parseReviewPayload(io.readFile(payloadPath), selectedHeadSha);
@@ -20,9 +24,13 @@ export function runPostReview(payloadPath, io) {
20
24
  if (io.getHeadSha() !== selectedHeadSha) {
21
25
  throw new ReviewPayloadError('PR head changed while preparing the selected review.');
22
26
  }
23
- const first = io.postReview(review);
27
+ const commentCount = review.comments.length;
28
+ const attributedReview = providerName
29
+ ? { ...review, body: `**${providerName} review**\n\n${review.body}` }
30
+ : review;
31
+ const first = io.postReview(attributedReview);
24
32
  if (first.ok)
25
- return { posted: true };
33
+ return { posted: true, commentCount };
26
34
  throw new ReviewPayloadError(`GitHub review POST failed (HTTP ${first.status}).`);
27
35
  }
28
36
  finally {
@@ -0,0 +1,13 @@
1
+ import type { GhExec } from './github.mts';
2
+ import type { PostResult, PostReviewIo, SanitizedReview } from './post.mts';
3
+ export declare function postWithGh(repository: string, prNumber: string, payload: SanitizedReview, token: string, exec: GhExec): PostResult;
4
+ export declare function createGhPostReviewIo(options: {
5
+ repository: string;
6
+ prNumber: string;
7
+ payloadPath: string;
8
+ payloadBytes: Buffer;
9
+ token: string;
10
+ exec: GhExec;
11
+ expectedHeadSha?: string;
12
+ expectedBaseSha?: string;
13
+ }): PostReviewIo;
@@ -0,0 +1,88 @@
1
+ import { rmSync } from 'node:fs';
2
+ import { parseReviewFilesJson, ReviewPayloadError } from '../gha-review-payload/index.mjs';
3
+ export function postWithGh(repository, prNumber, payload, token, exec) {
4
+ try {
5
+ exec(['api', '--method', 'POST', `repos/${repository}/pulls/${prNumber}/reviews`, '--input', '-'], {
6
+ input: JSON.stringify(payload),
7
+ env: { ...process.env, GH_TOKEN: token, GITHUB_TOKEN: token },
8
+ });
9
+ return { ok: true, status: 201, body: '' };
10
+ }
11
+ catch (error) {
12
+ // oxlint-disable-next-line no-mistakes/ts-no-const-aliases -- isolate the process-error shape assertion at the catch boundary
13
+ const err = error;
14
+ const text = `${err.stdout ?? ''}${err.stderr ?? ''}${err.message ?? ''}`;
15
+ const status = Number(/HTTP\s+(\d{3})/u.exec(text)?.[1] ?? 0);
16
+ return { ok: false, status, body: text };
17
+ }
18
+ }
19
+ function readPullState(repository, prNumber, exec) {
20
+ let lastError;
21
+ for (let attempt = 1; attempt <= 3; attempt += 1) {
22
+ try {
23
+ return exec([
24
+ 'api',
25
+ `repos/${repository}/pulls/${prNumber}`,
26
+ '--jq',
27
+ '[.head.sha, .base.sha, .draft, .state] | @tsv',
28
+ ]).split('\t');
29
+ }
30
+ catch (error) {
31
+ lastError = error;
32
+ }
33
+ }
34
+ throw lastError;
35
+ }
36
+ export function createGhPostReviewIo(options) {
37
+ const { repository, prNumber, payloadBytes, token, exec, expectedHeadSha, expectedBaseSha } = options;
38
+ return {
39
+ readFile() {
40
+ return payloadBytes;
41
+ },
42
+ removeFile(path) {
43
+ rmSync(path, { force: true });
44
+ },
45
+ getHeadSha() {
46
+ if (!/^[1-9][0-9]*$/u.test(prNumber)) {
47
+ throw new ReviewPayloadError('PR_NUMBER must be a positive integer.');
48
+ }
49
+ if (Boolean(expectedHeadSha) !== Boolean(expectedBaseSha)) {
50
+ throw new ReviewPayloadError('EXPECTED_HEAD_SHA and EXPECTED_BASE_SHA must be provided together.');
51
+ }
52
+ if (expectedHeadSha && !/^[0-9a-f]{40}$/u.test(expectedHeadSha)) {
53
+ throw new ReviewPayloadError('EXPECTED_HEAD_SHA must be a full lowercase commit SHA.');
54
+ }
55
+ if (expectedBaseSha && !/^[0-9a-f]{40}$/u.test(expectedBaseSha)) {
56
+ throw new ReviewPayloadError('EXPECTED_BASE_SHA must be a full lowercase commit SHA.');
57
+ }
58
+ const state = readPullState(repository, prNumber, exec);
59
+ const [headSha = '', baseSha = '', draft = '', pullState = ''] = state;
60
+ if (!/^[0-9a-f]{40}$/u.test(headSha)) {
61
+ throw new ReviewPayloadError(`Could not resolve PR head SHA (got "${headSha}").`);
62
+ }
63
+ if (!/^[0-9a-f]{40}$/u.test(baseSha)) {
64
+ throw new ReviewPayloadError(`Could not resolve PR base SHA (got "${baseSha}").`);
65
+ }
66
+ if (draft !== 'false') {
67
+ throw new ReviewPayloadError('Pull request became a draft before posting the review.');
68
+ }
69
+ if (pullState !== 'open') {
70
+ throw new ReviewPayloadError('Pull request closed before posting the review.');
71
+ }
72
+ if (expectedHeadSha && headSha !== expectedHeadSha) {
73
+ throw new ReviewPayloadError('PR head changed before posting the selected review.');
74
+ }
75
+ if (expectedBaseSha && baseSha !== expectedBaseSha) {
76
+ throw new ReviewPayloadError('PR base changed before posting the selected review.');
77
+ }
78
+ // Preserve the orchestrator-selected revision as the review commit_id after equality checks.
79
+ return expectedHeadSha || headSha;
80
+ },
81
+ listPullFiles() {
82
+ return parseReviewFilesJson(exec(['api', '--paginate', `repos/${repository}/pulls/${prNumber}/files?per_page=100`]));
83
+ },
84
+ postReview(payload) {
85
+ return postWithGh(repository, prNumber, payload, token, exec);
86
+ },
87
+ };
88
+ }
package/dist/index.d.mts CHANGED
@@ -25,7 +25,8 @@ export { requireUpToDate } from './require-up-to-date/index.mts';
25
25
  export type { GitExecutor, RequireUpToDateOptions } from './require-up-to-date/index.mts';
26
26
  export { gitleaksDirectoryScanArguments, runGitleaksDirectoryScan, } from './gitleaks-directory-scan/index.mts';
27
27
  export type { DirectoryScanExecutor, GitleaksDirectoryScanOptions, } from './gitleaks-directory-scan/index.mts';
28
- export { astGrepExamplesArguments, runAstGrepExamples } from './ast-grep-examples/index.mts';
28
+ export { astGrepExamplesArguments, compareAstGrepCompanions, runAstGrepExamples, } from './ast-grep-examples/index.mts';
29
+ export type { AstGrepCompanionContext, AstGrepCompanionDifference, AstGrepCompanionParityOptions, } from './ast-grep-examples/index.mts';
29
30
  export type { AstGrepExamplesExecutor, AstGrepExamplesOptions } from './ast-grep-examples/index.mts';
30
31
  export { astGrepPackPaths, astGrepPackPathsFrom } from './ast-grep-pack/index.mts';
31
32
  export type { AstGrepPackPaths } from './ast-grep-pack/index.mts';
package/dist/index.mjs CHANGED
@@ -13,7 +13,7 @@ export { auditCiJobRuntime, parseWorkflowNameMatch } from './gha-runtime-audit/i
13
13
  export { checkGhaWorkspacePolicy } from './gha-workspace-policy/index.mjs';
14
14
  export { requireUpToDate } from './require-up-to-date/index.mjs';
15
15
  export { gitleaksDirectoryScanArguments, runGitleaksDirectoryScan, } from './gitleaks-directory-scan/index.mjs';
16
- export { astGrepExamplesArguments, runAstGrepExamples } from './ast-grep-examples/index.mjs';
16
+ export { astGrepExamplesArguments, compareAstGrepCompanions, runAstGrepExamples, } from './ast-grep-examples/index.mjs';
17
17
  export { astGrepPackPaths, astGrepPackPathsFrom } from './ast-grep-pack/index.mjs';
18
18
  export { createVitestBlobManifest, createVitestReportAttempt, inspectVitestBlobBundle, parseVitestBlobManifest, parseVitestReportAttempt, readVitestReportAttempts, serializeVitestBlobManifest, serializeVitestReportAttempt, VITEST_BLOB_MANIFEST_FILENAME, VITEST_BLOB_MANIFEST_VERSION, VITEST_REPORT_ATTEMPT_PREFIX, VITEST_REPORT_ATTEMPT_VERSION, vitestBlobBundlePaths, writeVitestBlobManifest, writeVitestReportAttempt, } from './vitest-blob-manifest/index.mjs';
19
19
  export { prepareVitestReports } from './vitest-blob-manifest/reports.mjs';
@@ -1,4 +1,5 @@
1
- import { canonicalRepoPath, isInScope } from './paths.mjs';
1
+ import { assertWorkflowCommandData, canonicalRepoPath, isInScope } from './paths.mjs';
2
+ import { SCC_COMPLEXITY_LIMIT } from './constants.mjs';
2
3
  export const SCC_COMPLEXITY_BASELINE_VERSION = 1;
3
4
  export function parseSccComplexityBaseline(json) {
4
5
  const parsed = JSON.parse(json);
@@ -41,7 +42,7 @@ export function validateSccComplexityBaseline(baseline, scopes, trackedFileSet,
41
42
  throw new Error(`baseline entry ${key} is out of scope`);
42
43
  throw new Error(`baseline entry ${key} is stale`);
43
44
  }
44
- if (value.complexity <= (scope.limit ?? 50))
45
+ if (value.complexity <= (scope.limit ?? SCC_COMPLEXITY_LIMIT))
45
46
  throw new Error(`baseline entry ${key} is stale`);
46
47
  }
47
48
  }
@@ -76,6 +77,8 @@ function parseEntry(value, index) {
76
77
  throw new Error(`baseline entry ${index} has an invalid scope`);
77
78
  if (typeof value.file !== 'string' || value.file.length === 0)
78
79
  throw new Error(`baseline entry ${index} has an invalid file`);
80
+ assertWorkflowCommandData(value.scope, `baseline entry ${index}`);
81
+ assertWorkflowCommandData(value.file, `baseline entry ${index}`);
79
82
  if (typeof value.complexity !== 'number' ||
80
83
  !Number.isSafeInteger(value.complexity) ||
81
84
  value.complexity < 0)
@@ -0,0 +1 @@
1
+ export declare const SCC_COMPLEXITY_LIMIT = 50;
@@ -0,0 +1 @@
1
+ export const SCC_COMPLEXITY_LIMIT = 50;
@@ -1,8 +1,8 @@
1
1
  import type { SharedContext } from '../shared-context/index.mts';
2
2
  import type { SccComplexityOptions, SccComplexityScope, SccComplexityViolation } from './types.mts';
3
3
  export { parseSccComplexityBaseline, SCC_COMPLEXITY_BASELINE_VERSION } from './baseline.mts';
4
+ export { SCC_COMPLEXITY_LIMIT } from './constants.mts';
4
5
  export type { SccComplexityBaseline, SccComplexityBaselineEntry, SccComplexityOptions, SccComplexityScope, SccComplexityViolation, } from './types.mts';
5
- export declare const SCC_COMPLEXITY_LIMIT = 50;
6
6
  type RunScc = (outputPath: string, scope?: SccComplexityScope) => Promise<string>;
7
7
  export declare function buildSccArgs(options?: SccComplexityOptions): string[];
8
8
  export declare function checkSccComplexity(ctx: SharedContext, options?: SccComplexityOptions, runScc?: RunScc): Promise<{