@pome-sh/checks 0.1.2 → 0.1.4

package/CHANGELOG.md

@@ -1,5 +1,49 @@
 # @pome-sh/checks
 
+## 0.1.4
+
+Carries twin-github's widened seed schema to the grader (F-1421). One thing
+moves in `dist/`: `@pome-sh/checks/github`'s re-exported `seedSchema` — and with
+it `parseSeed` — now accepts five entity types it used to strip silently.
+
+- `repositories[].milestones[]`, `repositories[].tags[]`,
+  `repositories[].releases[]`
+- `repositories[].issues[].comments[]` and
+  `repositories[].pull_requests[].comments[]`
+- `repositories[].pull_requests[].review_comments[]`
+
+Every one is optional with a `[]` default, so a seed that parsed before parses
+to the same value now. This is a widening, not a tightening: nothing a consumer
+already sends can start failing.
+
+Why it is owed a release rather than left to the twin: this package's job is to
+carry the twins' seed schemas to a consumer that has no twin, and a consumer
+validating a seed against 0.1.3 would strip exactly the keys twin-github 0.10.2
+now honors — reporting a seed as accepted and a world as seeded while five
+entities were dropped on the way. The two halves have to move together or the
+copy on the grader's side becomes the one that decides what a seed may say.
+
+No check declaration, template, polarity or vacuity sentinel changed, so no
+existing criterion's verdict moves.
+
+## 0.1.3
+
+No change to the published surface — `dist/` is byte-identical to 0.1.2, so no
+declaration moved and no consumer's verdict changes. The declaration layer has
+not been touched since 0.1.2 was cut (`38acaf9`), which is the version-bump
+gate doing its job rather than an absence of work.
+
+What this version is for: it is the first release cut by following
+[`RELEASING.md`](../../RELEASING.md) end to end since the `packages-v*` batch
+flow was replaced by version-diff-on-push, and it exists to prove that runbook
+is followable — bump, merge, `release.yml` publishes, pome-cloud pins. The
+consumer half is the part that was actually owed: pome-cloud has been pinned at
+0.1.1 since before 0.1.2 shipped, so F-1157's redaction-survival vocabulary has
+not reached the grader.
+
+Version-only bumps are accepted here rather than carrying an exception list,
+per `cli/CHANGELOG.md` 0.21.7.
+
 ## 0.1.2
 
 Carries F-1157 to the grader. Three things move in `dist/`:

package/dist/_types/twin-github/src/seed.d.ts

@@ -26,6 +26,29 @@ export declare const seedSchema: z.ZodObject<{
             content: z.ZodString;
             branch: z.ZodOptional<z.ZodString>;
         }, z.core.$strip>>>;
+        milestones: z.ZodDefault<z.ZodArray<z.ZodObject<{
+            number: z.ZodOptional<z.ZodNumber>;
+            title: z.ZodString;
+            description: z.ZodDefault<z.ZodString>;
+            state: z.ZodDefault<z.ZodEnum<{
+                open: "open";
+                closed: "closed";
+            }>>;
+            due_on: z.ZodOptional<z.ZodString>;
+        }, z.core.$strip>>>;
+        tags: z.ZodDefault<z.ZodArray<z.ZodObject<{
+            name: z.ZodString;
+            target: z.ZodOptional<z.ZodString>;
+        }, z.core.$strip>>>;
+        releases: z.ZodDefault<z.ZodArray<z.ZodObject<{
+            tag_name: z.ZodString;
+            name: z.ZodOptional<z.ZodString>;
+            body: z.ZodDefault<z.ZodString>;
+            target_commitish: z.ZodOptional<z.ZodString>;
+            draft: z.ZodDefault<z.ZodBoolean>;
+            prerelease: z.ZodDefault<z.ZodBoolean>;
+            author: z.ZodOptional<z.ZodString>;
+        }, z.core.$strip>>>;
         issues: z.ZodDefault<z.ZodArray<z.ZodObject<{
             number: z.ZodOptional<z.ZodNumber>;
             title: z.ZodString;
@@ -36,6 +59,10 @@ export declare const seedSchema: z.ZodObject<{
             }>>;
             labels: z.ZodDefault<z.ZodArray<z.ZodString>>;
             assignees: z.ZodDefault<z.ZodArray<z.ZodString>>;
+            comments: z.ZodDefault<z.ZodArray<z.ZodObject<{
+                body: z.ZodString;
+                author: z.ZodOptional<z.ZodString>;
+            }, z.core.$strip>>>;
         }, z.core.$strip>>>;
         pull_requests: z.ZodDefault<z.ZodArray<z.ZodObject<{
             number: z.ZodOptional<z.ZodNumber>;
@@ -61,12 +88,26 @@ export declare const seedSchema: z.ZodObject<{
                 context: z.ZodDefault<z.ZodString>;
                 state: z.ZodDefault<z.ZodEnum<{
                     error: "error";
+                    success: "success";
                     failure: "failure";
                     pending: "pending";
-                    success: "success";
                 }>>;
                 description: z.ZodDefault<z.ZodString>;
             }, z.core.$strip>>>;
+            comments: z.ZodDefault<z.ZodArray<z.ZodObject<{
+                body: z.ZodString;
+                author: z.ZodOptional<z.ZodString>;
+            }, z.core.$strip>>>;
+            review_comments: z.ZodDefault<z.ZodArray<z.ZodObject<{
+                body: z.ZodString;
+                path: z.ZodString;
+                line: z.ZodDefault<z.ZodNumber>;
+                side: z.ZodDefault<z.ZodEnum<{
+                    LEFT: "LEFT";
+                    RIGHT: "RIGHT";
+                }>>;
+                author: z.ZodOptional<z.ZodString>;
+            }, z.core.$strip>>>;
         }, z.core.$strip>>>;
     }, z.core.$strip>>;
 }, z.core.$strip>;

package/dist/_types/twin-github/src/types.d.ts

@@ -27,6 +27,26 @@ export type SeedRepository = {
         content: string;
         branch?: string;
     }>;
+    milestones?: Array<{
+        number?: number;
+        title: string;
+        description?: string;
+        state?: "open" | "closed";
+        due_on?: string;
+    }>;
+    tags?: Array<{
+        name: string;
+        target?: string;
+    }>;
+    releases?: Array<{
+        tag_name: string;
+        name?: string;
+        body?: string;
+        target_commitish?: string;
+        draft?: boolean;
+        prerelease?: boolean;
+        author?: string;
+    }>;
     issues?: Array<{
         number?: number;
         title: string;
@@ -34,6 +54,7 @@ export type SeedRepository = {
         state?: "open" | "closed";
         labels?: string[];
         assignees?: string[];
+        comments?: SeedComment[];
     }>;
     pull_requests?: Array<{
         number?: number;
@@ -53,8 +74,24 @@ export type SeedRepository = {
             state?: "error" | "failure" | "pending" | "success";
             description?: string;
         }>;
+        comments?: SeedComment[];
+        review_comments?: Array<{
+            body: string;
+            path: string;
+            line?: number;
+            side?: "LEFT" | "RIGHT";
+            author?: string;
+        }>;
     }>;
 };
+/**
+ * A conversation comment on an issue OR a pull request — one shape, because
+ * they are one table and one route on GitHub too (F-1151).
+ */
+export type SeedComment = {
+    body: string;
+    author?: string;
+};
 export type RepoRow = {
     id: number;
     owner: string;

package/dist/_types/twin-linear/src/types.d.ts

@@ -4,7 +4,44 @@ export type LinearWorkflowStateType = "backlog" | "unstarted" | "started" | "com
 export type LinearTokenType = "personal" | "oauth_access" | "oauth_refresh" | "client_credentials";
 export type LinearTokenActorType = "user" | "app";
 export type LinearIssuePriority = 0 | 1 | 2 | 3 | 4;
+/** Linear's `AgentActivityType` enum, member-for-member — the `content` discriminator. */
 export type LinearAgentActivityType = "thought" | "elicitation" | "action" | "response" | "error" | "prompt";
+/** Linear's `AgentActivitySignal` enum, member-for-member (F-1176). */
+export type LinearAgentActivitySignal = "stop" | "continue" | "auth" | "select";
+/**
+ * Linear's `AgentActivityContent` union, member-for-member (F-1176).
+ *
+ * Upstream this is a real GraphQL union discriminated on `type`, whose six
+ * members are `AgentActivity{Thought,Action,Response,Elicitation,Error,Prompt}
+ * Content`. Five of the six carry `body`; `action` carries `action` /
+ * `parameter` / `result` instead, which is why an activity's payload cannot be
+ * flattened back to the `{ type, body }` pair the twin used to accept.
+ *
+ * `bodyData` / `resultData` are upstream's structured companions to the text
+ * fields. The twin does not derive them, but it accepts and stores them rather
+ * than refusing an agent that was written against real Linear and sends them.
+ */
+export type LinearAgentActivityContent = {
+    type: "thought" | "response" | "elicitation";
+    body: string;
+    bodyData?: unknown;
+} | {
+    type: "prompt";
+    body: string;
+    title?: string | null;
+    bodyData?: unknown;
+} | {
+    type: "error";
+    body: string;
+    reasonCode?: string | null;
+    bodyData?: unknown;
+} | {
+    type: "action";
+    action: string;
+    parameter: string;
+    result?: string | null;
+    resultData?: unknown;
+};
 /** Linear's `AgentSessionStatus` enum, member-for-member (F-1172). */
 export type LinearAgentSessionStatus = "pending" | "active" | "awaitingInput" | "complete" | "error" | "stale";
 /** One entry of Linear's `AgentSession.externalUrls` JSON payload. */
@@ -208,10 +245,15 @@ export type LinearAgentSession = {
 };
 export type LinearAgentActivity = {
     id: string;
-    sessionId: string;
-    userId: string | null;
-    type: LinearAgentActivityType;
-    body: string;
+    agentSessionId: string;
+    /**
+     * Non-null, matching Linear's `AgentActivity.user: User!`. The column is
+     * still nullable — its FK is `ON DELETE SET NULL` — so this is enforced at
+     * the read boundary, next to the status check, rather than assumed.
+     */
+    userId: string;
+    content: LinearAgentActivityContent;
+    signal: LinearAgentActivitySignal | null;
     ephemeral: boolean;
     createdAt: string;
     updatedAt: string;

package/dist/{chunk-UCV6YLN2.js → chunk-JVCGWAZQ.js}

@@ -863,13 +863,61 @@ var seedSchema = z.object({
       description: z.string().default("")
     })).default([]),
     files: z.array(z.object({ path: z.string().min(1), content: z.string(), branch: z.string().optional() })).default([]),
+    // F-1421 — milestones, tags and releases are repository-level entities the
+    // twin already SERVES (`GET /milestones`, `/tags`, `/releases`,
+    // `/releases/latest`, `/releases/tags/:tag`) and the seed could not
+    // express. Zod strips unknown keys, so a seed naming one reached the
+    // domain as nothing at all and those routes could only ever answer `[]`
+    // — a shape of infidelity no shape-diff can see, because an empty array
+    // on both sides compares zero elements.
+    milestones: z.array(z.object({
+      // Assigned sequentially from 1 in seed order when omitted, the way
+      // `nextMilestoneNumber` hands them out. Honored when given, so a
+      // seed that pins `PATCH /milestones/2` addresses the milestone it
+      // named rather than one silently renumbered under it.
+      number: z.number().int().positive().optional(),
+      title: z.string().min(1),
+      description: z.string().default(""),
+      state: z.enum(["open", "closed"]).default("open"),
+      // GitHub's own spelling: an ISO 8601 timestamp. Absent means the
+      // milestone has no due date (`due_on: null` on the wire).
+      due_on: z.string().optional()
+    })).default([]),
+    // A tag names a commit. `target` is any ref the twin resolves — a branch
+    // name or a SHA — and defaults to the repository's default-branch head. A
+    // release whose `tag_name` matches an entry here reuses that tag rather
+    // than minting a second one: `createRelease` looks the tag up first.
+    tags: z.array(z.object({
+      name: z.string().min(1),
+      target: z.string().min(1).optional()
+    })).default([]),
+    releases: z.array(z.object({
+      tag_name: z.string().min(1),
+      // Nullable upstream, so an absent `name` means `null` — not `""`.
+      name: z.string().optional(),
+      body: z.string().default(""),
+      target_commitish: z.string().min(1).optional(),
+      draft: z.boolean().default(false),
+      prerelease: z.boolean().default(false),
+      author: z.string().min(1).optional()
+    })).default([]),
     issues: z.array(z.object({
       number: z.number().int().positive().optional(),
       title: z.string().min(1),
       body: z.string().default(""),
       state: z.enum(["open", "closed"]).default("open"),
       labels: z.array(z.string().min(1)).default([]),
-      assignees: z.array(z.string().min(1)).default([])
+      assignees: z.array(z.string().min(1)).default([]),
+      // The issue's conversation timeline, served at
+      // `GET /repos/:o/:r/issues/:n/comments` (F-1421). `author` is
+      // seeded honestly rather than taken from the write path, which
+      // stamps every comment `pome-agent`: a world in which the agent
+      // under test wrote every comment on the issue it is being asked to
+      // read is not one worth testing against.
+      comments: z.array(z.object({
+        body: z.string().min(1),
+        author: z.string().min(1).optional()
+      })).default([])
     })).default([]),
     pull_requests: z.array(z.object({
       number: z.number().int().positive().optional(),
@@ -893,6 +941,30 @@ var seedSchema = z.object({
         context: z.string().min(1).default("ci/build"),
         state: z.enum(["error", "failure", "pending", "success"]).default("success"),
         description: z.string().default("")
+      })).default([]),
+      // F-1421 — the PR's CONVERSATION timeline. Same table, same route
+      // and same number space as an issue's comments, because GitHub
+      // models a pull request as an issue (F-1151). This is the third
+      // thing a reader could call "a comment on the PR" and the seed
+      // keeps all three apart: `reviews[].body` is the prose on a review
+      // verdict, `review_comments[]` below is anchored to a file and
+      // line, and THIS one is the timeline.
+      comments: z.array(z.object({
+        body: z.string().min(1),
+        author: z.string().min(1).optional()
+      })).default([]),
+      // F-1421 — inline review comments, served at
+      // `GET /repos/:o/:r/pulls/:n/comments`. Seeded through the domain's
+      // own write path, so `path` must name a file the PR changes and
+      // `line` must exist in it: a seeded review comment is one
+      // `POST /pulls/:n/comments` could have produced, not a row only the
+      // seeder can make.
+      review_comments: z.array(z.object({
+        body: z.string().min(1),
+        path: z.string().min(1),
+        line: z.number().int().positive().default(1),
+        side: z.enum(["LEFT", "RIGHT"]).default("RIGHT"),
+        author: z.string().min(1).optional()
       })).default([])
     })).default([])
   }))

package/dist/github.js

@@ -1,2 +1,2 @@
-export { GITHUB_CHECKS, defaultSeedState, parseSeed, seedSchema } from './chunk-UCV6YLN2.js';
+export { GITHUB_CHECKS, defaultSeedState, parseSeed, seedSchema } from './chunk-JVCGWAZQ.js';
 import './chunk-W2JNYULF.js';

package/dist/index.js

@@ -1,6 +1,6 @@
 import './chunk-ZXE6LAM3.js';
-import { GITHUB_CHECKS } from './chunk-UCV6YLN2.js';
-export { GITHUB_CHECKS, defaultSeedState as defaultGitHubSeed, seedSchema as githubSeedSchema, parseSeed as parseGitHubSeed } from './chunk-UCV6YLN2.js';
+import { GITHUB_CHECKS } from './chunk-JVCGWAZQ.js';
+export { GITHUB_CHECKS, defaultSeedState as defaultGitHubSeed, seedSchema as githubSeedSchema, parseSeed as parseGitHubSeed } from './chunk-JVCGWAZQ.js';
 import { GMAIL_CHECKS } from './chunk-SA23X6PB.js';
 export { GMAIL_CHECKS, defaultSeedState as defaultGmailSeed, gmailSeedSchema, parseSeed as parseGmailSeed } from './chunk-SA23X6PB.js';
 import { LINEAR_CHECKS } from './chunk-THOSO63W.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pome-sh/checks",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "Pome's grading vocabulary — the check declarations, seed schemas and default seeds of all five digital twins, plus the check DSL they are written in. Declarations only: no twin server, no database, no routes, no tools.",
   "private": false,
   "type": "module",