@osdlc/typed-gh 0.11.1 → 0.12.1

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @osdlc/typed-gh
 
+## 0.12.1
+
+### Patch Changes
+
+- Updated dependencies [82d9841]
+- Updated dependencies [ebe0e0d]
+  - @osdlc/typed-git@0.11.0
+
+## 0.12.0
+
+### Minor Changes
+
+- a4ca07d: New `@osdlc/typed-gh/testing` entry point with builders for faking `gh` in tests. `prRow`, `issueRow`, `repoRow`, `searchPrRow`, `labelRow`, `checkRunRow`, `statusContextRow`, `commentRow` and `reviewRow` return gh's raw `--json` rows, with defaults that fill every field the parsers require (a pull request's `number`, a repository's `nameWithOwner`, the `headRefName` that `prList({ head })` filters on), and take typed overrides merged shallowly. `ghResult(payload)` wraps rows in a successful `CommandResult` for a fake runner. The main entry point does not import it.
+  
+  Fixed: `searchPrs` reads gh's zero time (`0001-01-01T00:00:00Z`), which gh prints for a hit that isn't closed, as `closedAt: null`. Before, an open hit had a year-0001 close date.
+
 ## 0.11.1
 
 ### Patch Changes

package/README.md

@@ -488,6 +488,36 @@ Import the runners themselves, such as `spawnRunner` and `spawnRunnerAsync`, fro
 named `gh` in the working directory cannot run in its place. Pass `{ absolutePathOnly: true }` when
 you build a real runner yourself.
 
+## Testing
+
+`@osdlc/typed-gh/testing` builds the rows a fake runner answers with. Each builder returns gh's raw
+`--json` shape, so the real parsers read it. Its defaults fill every field a parser requires, such
+as a pull request's `number`, a repository's `nameWithOwner` and the `headRefName` that
+`prList({ head })` filters on. Without them a row is dropped with no error.
+
+- `prRow`, `issueRow`, `repoRow`, `searchPrRow` and `labelRow`: one row of `gh pr`, `gh issue`,
+  `gh repo`, `gh search prs` and `gh label` output.
+- `checkRunRow`, `statusContextRow`, `commentRow` and `reviewRow`: entries of a pull request's
+  `statusCheckRollup`, `comments` and `reviews`.
+- `ghResult(payload)`: a `CommandResult` with exit code 0 and the payload as JSON on stdout.
+- Row types: `GhPrJson`, `GhIssueJson`, `GhRepoJson`, `GhSearchPrJson`, `GhLabelJson`,
+  `GhCheckRunJson`, `GhStatusContextJson`, `GhCommentJson`, `GhReviewJson` and `GhActorJson`, and
+  `RowOverrides<T>`, what a builder takes.
+
+Each builder takes typed overrides, merged shallowly. Pass a field as `undefined` to leave it out.
+A pull request's `url` follows its `number`, and a repository's its `nameWithOwner`. Assert on what
+you set: a default value may change in a minor release.
+
+```ts
+import { ghResult, prRow } from '@osdlc/typed-gh/testing'
+
+const runner: CommandRunner = {
+  run: () => ghResult([prRow({ number: 7, headRefName: 'task/foo' }), prRow({ number: 8 })]),
+}
+
+new Gh('/repo', { runner }).openPrFor('task/foo')?.number // 7
+```
+
 ## API
 
 Everything below is exported from `@osdlc/typed-gh`. The Stability section says which items are
@@ -599,7 +629,8 @@ The experimental items:
 - `GraphqlOptions`
 - `GraphqlVariable`
 
-Every other export and method is stable, `prDiff` included.
+Every other export and method is stable, `prDiff` included. So is `@osdlc/typed-gh/testing`: its
+builders and their row types. Their default values may change in any minor release.
 
 ## Platforms
 

package/dist/parse.js

@@ -266,6 +266,14 @@ export function parsePrSearchHit(raw) {
         labels: parseLabels(raw['labels']),
         createdAt: strOrNull(raw['createdAt']),
         updatedAt: strOrNull(raw['updatedAt']),
-        closedAt: strOrNull(raw['closedAt']),
+        closedAt: searchTime(raw['closedAt']),
     };
 }
+/**
+ * gh search prints Go's zero time (`0001-01-01T00:00:00Z`) for a time that is
+ * not set, such as `closedAt` on an open pull request. That is no time.
+ */
+function searchTime(x) {
+    const s = strOrNull(x);
+    return s === null || s.startsWith('0001-01-01') ? null : s;
+}

package/dist/testing.d.ts

@@ -0,0 +1,216 @@
+/**
+ * `@osdlc/typed-gh/testing`: rows of `gh --json` output, for faking `gh` in
+ * tests.
+ *
+ * A separate subpath, not part of the package entry, so production code does
+ * not carry fixture data. Nothing in `src/` but this file knows about it.
+ *
+ * Each builder returns gh's raw JSON shape, what `gh … --json` prints
+ * (`nameWithOwner`, `headRefName`, `defaultBranchRef: { name }`), not the
+ * parsed model. Feed the rows to a fake runner through {@link ghResult} and
+ * the real parsers read them, so a test exercises the same parsing a real
+ * `gh` call does.
+ *
+ * The defaults fill every field the parsers require, which is where hand-made
+ * rows go wrong: a row the parser rejects is dropped without an error.
+ *
+ *   - A pull request needs an integer `number` (`parsePullRequest`), and
+ *     `prList({ head })` also keeps only rows whose `headRefName` equals
+ *     `head`.
+ *   - An issue and a search hit need a numeric `number`.
+ *   - A repository needs `nameWithOwner` as `owner/name`, both parts
+ *     non-empty (`parseRepository`).
+ *   - A label needs a non-empty `name` (`parseLabels`).
+ *   - A check needs a `name`, or a `context` for a legacy status context
+ *     (`parseCheckRun`).
+ *
+ * Overrides merge shallowly: a field you pass replaces the default whole, and
+ * a field you pass as `undefined` is left out of the row, to test a payload
+ * that lacks it. A default that names another field (a pull request's `url`
+ * names its `number`) follows your override of that field. The default
+ * values may change in a minor release, as long as every parser still
+ * accepts them, so assert on what you set, not on a default.
+ */
+import type { CommandResult } from '@osdlc/command-seam';
+import type { CheckConclusion, MergeableState, MergeStateStatus, PullRequestState, RepositoryVisibility, ReviewState } from './types.ts';
+/**
+ * What a builder takes: any field of the row, or `undefined` to leave that
+ * field out.
+ */
+export type RowOverrides<T> = {
+    [K in keyof T]?: T[K] | undefined;
+};
+/**
+ * A successful `gh` call that printed `payload` as JSON: exit code 0, the
+ * payload on stdout with gh's trailing newline, and nothing on stderr. Pass an
+ * array of rows for a list verb and one row for a view verb.
+ */
+export declare function ghResult(payload: unknown): CommandResult;
+/**
+ * A pull request's or search hit's `author` as `gh --json` prints it. A bot
+ * is `{ is_bot: true, login: 'app/<name>' }`.
+ */
+export interface GhActorJson {
+    login: string;
+    is_bot?: boolean;
+    name?: string;
+    id?: string;
+}
+/**
+ * A comment's or review's `author` as `gh --json` prints it: only `login`.
+ * gh leaves out `id`, `name` and `is_bot` there, so a bot comment does not
+ * read as a bot unless its login ends in `[bot]`.
+ */
+export interface GhCommentAuthorJson {
+    login: string;
+}
+/** One entry of a `labels` field, or one row of `gh label list --json name,color,description`. */
+export interface GhLabelJson {
+    name: string;
+    /** Six hex digits, no `#`. */
+    color: string;
+    description: string;
+}
+/** A label row. The default is GitHub's own `bug` label. */
+export declare function labelRow(overrides?: RowOverrides<GhLabelJson>): GhLabelJson;
+/** A modern `CheckRun` node of `statusCheckRollup`. */
+export interface GhCheckRunJson {
+    __typename: 'CheckRun';
+    name: string;
+    workflowName: string;
+    /** `QUEUED`, `IN_PROGRESS` or `COMPLETED`. */
+    status: string;
+    /** gh prints `""` while the check runs. */
+    conclusion: Exclude<CheckConclusion, null> | '';
+    detailsUrl: string;
+    startedAt: string;
+    completedAt: string;
+}
+/** A check run. The default completed with `SUCCESS`, so it reads as a green check. */
+export declare function checkRunRow(overrides?: RowOverrides<GhCheckRunJson>): GhCheckRunJson;
+/** A legacy `StatusContext` node of `statusCheckRollup`. */
+export interface GhStatusContextJson {
+    __typename: 'StatusContext';
+    context: string;
+    /** `SUCCESS`, `PENDING`, `FAILURE` or `ERROR`. */
+    state: string;
+    targetUrl: string;
+    startedAt: string;
+}
+/** A legacy status context. The default is a `SUCCESS`. */
+export declare function statusContextRow(overrides?: RowOverrides<GhStatusContextJson>): GhStatusContextJson;
+/** One entry of a pull request's `comments`. */
+export interface GhCommentJson {
+    /** The GraphQL node id. */
+    id: string;
+    author: GhCommentAuthorJson | null;
+    authorAssociation: string;
+    body: string;
+    createdAt: string;
+    url: string;
+}
+/** A comment by `octocat`. */
+export declare function commentRow(overrides?: RowOverrides<GhCommentJson>): GhCommentJson;
+/** One entry of a pull request's `reviews`. */
+export interface GhReviewJson {
+    /** The GraphQL node id. */
+    id: string;
+    author: GhCommentAuthorJson | null;
+    authorAssociation: string;
+    body: string;
+    state: ReviewState;
+    submittedAt: string;
+    commit: {
+        oid: string;
+    };
+}
+/** An approving review by `octocat`. */
+export declare function reviewRow(overrides?: RowOverrides<GhReviewJson>): GhReviewJson;
+/** A pull request as `gh pr list`/`view --json` prints it, with every field the parser reads. */
+export interface GhPrJson {
+    /** Required: a row without an integer `number` is dropped. */
+    number: number;
+    state: PullRequestState;
+    title: string;
+    body: string;
+    url: string;
+    isDraft: boolean;
+    baseRefName: string;
+    /** Required by `prList({ head })`, which keeps only rows whose head is exactly `head`. */
+    headRefName: string;
+    headRefOid: string;
+    isCrossRepository: boolean;
+    mergeable: MergeableState;
+    mergeStateStatus: MergeStateStatus;
+    /** gh prints `""` when there is no decision. */
+    reviewDecision: 'APPROVED' | 'CHANGES_REQUESTED' | 'REVIEW_REQUIRED' | '';
+    author: GhActorJson | null;
+    labels: GhLabelJson[];
+    comments: GhCommentJson[];
+    reviews: GhReviewJson[];
+    statusCheckRollup: (GhCheckRunJson | GhStatusContextJson)[];
+    mergedAt: string | null;
+    createdAt: string;
+    updatedAt: string;
+}
+/**
+ * An open, mergeable pull request #1 from `feature/add-retries` into `main`
+ * of `octo-org/hello-world`, by `octocat`, with no labels, comments, reviews
+ * or checks. Its `url` follows `number`.
+ */
+export declare function prRow(overrides?: RowOverrides<GhPrJson>): GhPrJson;
+/** An issue as `gh issue list`/`view --json number,title,body,labels,state` prints it. */
+export interface GhIssueJson {
+    /** Required: a row without a numeric `number` is dropped. */
+    number: number;
+    title: string;
+    body: string;
+    labels: GhLabelJson[];
+    /** `OPEN` or `CLOSED`. */
+    state: string;
+}
+/** An open issue #1 with no labels. */
+export declare function issueRow(overrides?: RowOverrides<GhIssueJson>): GhIssueJson;
+/** A repository as `gh repo list`/`view --json` prints it, with every field the parser reads. */
+export interface GhRepoJson {
+    /** Required: `owner/name`, both parts non-empty, or the row is dropped. */
+    nameWithOwner: string;
+    description: string;
+    /** `{ name: '' }` for an empty repository, as gh prints it. `null` parses the same. */
+    defaultBranchRef: {
+        name: string;
+    } | null;
+    visibility: RepositoryVisibility;
+    isArchived: boolean;
+    url: string;
+}
+/** The public, unarchived `octo-org/hello-world` on `main`. Its `url` follows `nameWithOwner`. */
+export declare function repoRow(overrides?: RowOverrides<GhRepoJson>): GhRepoJson;
+/** A row of `gh search prs --json`, with every field the parser reads. */
+export interface GhSearchPrJson {
+    /** Required: a row without a numeric `number` is dropped. */
+    number: number;
+    title: string;
+    /** Search answers in lower case: `open`, `closed` or `merged`. */
+    state: string;
+    url: string;
+    isDraft: boolean;
+    repository: {
+        name: string;
+        nameWithOwner: string;
+    };
+    author: GhActorJson | null;
+    labels: GhLabelJson[];
+    createdAt: string;
+    updatedAt: string;
+    /**
+     * gh prints `0001-01-01T00:00:00Z`, Go's zero time, for a hit that is not
+     * closed. The parser reads that as `null`.
+     */
+    closedAt: string | null;
+}
+/**
+ * An open pull request #1 in `octo-org/hello-world`, as search returns it. Its
+ * `url` follows `number` and `repository`.
+ */
+export declare function searchPrRow(overrides?: RowOverrides<GhSearchPrJson>): GhSearchPrJson;

package/dist/testing.js

@@ -0,0 +1,186 @@
+/**
+ * `@osdlc/typed-gh/testing`: rows of `gh --json` output, for faking `gh` in
+ * tests.
+ *
+ * A separate subpath, not part of the package entry, so production code does
+ * not carry fixture data. Nothing in `src/` but this file knows about it.
+ *
+ * Each builder returns gh's raw JSON shape, what `gh … --json` prints
+ * (`nameWithOwner`, `headRefName`, `defaultBranchRef: { name }`), not the
+ * parsed model. Feed the rows to a fake runner through {@link ghResult} and
+ * the real parsers read them, so a test exercises the same parsing a real
+ * `gh` call does.
+ *
+ * The defaults fill every field the parsers require, which is where hand-made
+ * rows go wrong: a row the parser rejects is dropped without an error.
+ *
+ *   - A pull request needs an integer `number` (`parsePullRequest`), and
+ *     `prList({ head })` also keeps only rows whose `headRefName` equals
+ *     `head`.
+ *   - An issue and a search hit need a numeric `number`.
+ *   - A repository needs `nameWithOwner` as `owner/name`, both parts
+ *     non-empty (`parseRepository`).
+ *   - A label needs a non-empty `name` (`parseLabels`).
+ *   - A check needs a `name`, or a `context` for a legacy status context
+ *     (`parseCheckRun`).
+ *
+ * Overrides merge shallowly: a field you pass replaces the default whole, and
+ * a field you pass as `undefined` is left out of the row, to test a payload
+ * that lacks it. A default that names another field (a pull request's `url`
+ * names its `number`) follows your override of that field. The default
+ * values may change in a minor release, as long as every parser still
+ * accepts them, so assert on what you set, not on a default.
+ */
+/** `defaults` with `overrides` merged over them. An `undefined` override drops the field. */
+function build(defaults, overrides) {
+    const row = { ...defaults };
+    for (const [key, value] of Object.entries(overrides)) {
+        if (value === undefined)
+            delete row[key];
+        else
+            row[key] = value;
+    }
+    return row;
+}
+const HOST = 'https://github.com';
+const REPO = 'octo-org/hello-world';
+const CREATED_AT = '2024-01-15T09:30:00Z';
+const UPDATED_AT = '2024-01-16T14:05:00Z';
+const HEAD_OID = '6dcb09b5b57875f334f61aebed695e2e4193db5e';
+// ── the result ──────────────────────────────────────────────────────────────
+/**
+ * A successful `gh` call that printed `payload` as JSON: exit code 0, the
+ * payload on stdout with gh's trailing newline, and nothing on stderr. Pass an
+ * array of rows for a list verb and one row for a view verb.
+ */
+export function ghResult(payload) {
+    return { exitCode: 0, stdout: `${JSON.stringify(payload)}\n`, stderr: '' };
+}
+/** A new object each call, so a test that edits one row's author edits only that row. */
+function octocat() {
+    return { id: 'MDQ6VXNlcjU4MzIzMQ==', is_bot: false, login: 'octocat', name: 'The Octocat' };
+}
+/** A label row. The default is GitHub's own `bug` label. */
+export function labelRow(overrides = {}) {
+    return build({ name: 'bug', color: 'd73a4a', description: "Something isn't working" }, overrides);
+}
+/** A check run. The default completed with `SUCCESS`, so it reads as a green check. */
+export function checkRunRow(overrides = {}) {
+    return build({
+        __typename: 'CheckRun',
+        name: 'test',
+        workflowName: 'CI',
+        status: 'COMPLETED',
+        conclusion: 'SUCCESS',
+        detailsUrl: `${HOST}/${REPO}/actions/runs/1/job/1`,
+        startedAt: CREATED_AT,
+        completedAt: UPDATED_AT,
+    }, overrides);
+}
+/** A legacy status context. The default is a `SUCCESS`. */
+export function statusContextRow(overrides = {}) {
+    return build({
+        __typename: 'StatusContext',
+        context: 'ci/build',
+        state: 'SUCCESS',
+        targetUrl: 'https://ci.example.com/builds/1',
+        startedAt: CREATED_AT,
+    }, overrides);
+}
+/** A comment by `octocat`. */
+export function commentRow(overrides = {}) {
+    return build({
+        id: 'IC_kwDOAbCdEf5xYz12',
+        author: { login: 'octocat' },
+        authorAssociation: 'MEMBER',
+        body: 'Looks good to me.',
+        createdAt: UPDATED_AT,
+        url: `${HOST}/${REPO}/pull/1#issuecomment-1`,
+    }, overrides);
+}
+/** An approving review by `octocat`. */
+export function reviewRow(overrides = {}) {
+    return build({
+        id: 'PRR_kwDOAbCdEf5xYz34',
+        author: { login: 'octocat' },
+        authorAssociation: 'MEMBER',
+        body: '',
+        state: 'APPROVED',
+        submittedAt: UPDATED_AT,
+        commit: { oid: HEAD_OID },
+    }, overrides);
+}
+/**
+ * An open, mergeable pull request #1 from `feature/add-retries` into `main`
+ * of `octo-org/hello-world`, by `octocat`, with no labels, comments, reviews
+ * or checks. Its `url` follows `number`.
+ */
+export function prRow(overrides = {}) {
+    const number = overrides.number ?? 1;
+    return build({
+        number,
+        state: 'OPEN',
+        title: 'Add retries to the fetch loop',
+        body: 'Retries a failed fetch three times before giving up.',
+        url: `${HOST}/${REPO}/pull/${number}`,
+        isDraft: false,
+        baseRefName: 'main',
+        headRefName: 'feature/add-retries',
+        headRefOid: HEAD_OID,
+        isCrossRepository: false,
+        mergeable: 'MERGEABLE',
+        mergeStateStatus: 'CLEAN',
+        reviewDecision: '',
+        author: octocat(),
+        labels: [],
+        comments: [],
+        reviews: [],
+        statusCheckRollup: [],
+        mergedAt: null,
+        createdAt: CREATED_AT,
+        updatedAt: UPDATED_AT,
+    }, overrides);
+}
+/** An open issue #1 with no labels. */
+export function issueRow(overrides = {}) {
+    return build({
+        number: 1,
+        title: 'Fetch gives up after one failure',
+        body: 'A single network error ends the whole sync.',
+        labels: [],
+        state: 'OPEN',
+    }, overrides);
+}
+/** The public, unarchived `octo-org/hello-world` on `main`. Its `url` follows `nameWithOwner`. */
+export function repoRow(overrides = {}) {
+    const nameWithOwner = overrides.nameWithOwner ?? REPO;
+    return build({
+        nameWithOwner,
+        description: 'My first repository on GitHub.',
+        defaultBranchRef: { name: 'main' },
+        visibility: 'PUBLIC',
+        isArchived: false,
+        url: `${HOST}/${nameWithOwner}`,
+    }, overrides);
+}
+/**
+ * An open pull request #1 in `octo-org/hello-world`, as search returns it. Its
+ * `url` follows `number` and `repository`.
+ */
+export function searchPrRow(overrides = {}) {
+    const number = overrides.number ?? 1;
+    const repository = overrides.repository ?? { name: 'hello-world', nameWithOwner: REPO };
+    return build({
+        number,
+        title: 'Add retries to the fetch loop',
+        state: 'open',
+        url: `${HOST}/${repository.nameWithOwner}/pull/${number}`,
+        isDraft: false,
+        repository,
+        author: { id: 'MDQ6VXNlcjU4MzIzMQ==', is_bot: false, login: 'octocat' },
+        labels: [],
+        createdAt: CREATED_AT,
+        updatedAt: UPDATED_AT,
+        closedAt: '0001-01-01T00:00:00Z',
+    }, overrides);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@osdlc/typed-gh",
-  "version": "0.11.1",
+  "version": "0.12.1",
   "description": "A typed GitHub client for TypeScript over the gh CLI. A total PullRequest/Review/CheckRun model, pure argv builders and parsers, and a tri-modal client — queries return parsed values, mutations throw, raw() is the escape hatch. Uses gh's own credentials; reads no token.",
   "license": "MIT OR Apache-2.0",
   "author": "sksizer",
@@ -10,6 +10,10 @@
     ".": {
       "types": "./dist/index.d.ts",
       "default": "./dist/index.js"
+    },
+    "./testing": {
+      "types": "./dist/testing.d.ts",
+      "default": "./dist/testing.js"
     }
   },
   "files": [
@@ -23,7 +27,7 @@
   ],
   "dependencies": {
     "@osdlc/command-seam": "^0.14.0",
-    "@osdlc/typed-git": "^0.10.0",
+    "@osdlc/typed-git": "^0.11.1",
     "@osdlc/result": "^0.1.0"
   },
   "engines": {