@compforge/harness-common 0.5.5 → 0.5.7

package/README.md

@@ -183,3 +183,10 @@ custom Entry now renders consistently when shown alone or through its Dictionary
 relationships without duplicating identity. `ResourceSnapshot` retains typed instances, unit-bearing
 measurements, independently known totals and complete/partial/unavailable coverage. Discovery and
 rendering remain consumer-owned. See [the shared contract](../../../docs/resource.md).
+
+## PullRequest
+
+`PullRequest` identifies a PR/MR by its target `Repository` and repository-local number.
+State, branches, source repository, revisions and provider timestamps are optional facts.
+See [the shared contract](../../../docs/pull-request.md) for identity and unknown-state semantics.
+Forge access and polling remain consumer/toolkit responsibilities.

package/dist/{context.d.ts → access/context.d.ts}

@@ -1,5 +1,5 @@
 import type { ClientManager } from "./client-manager.js";
-import type { Environment } from "./environment.js";
+import type { Environment } from "../system/environment.js";
 /**
  * Execution-scoped environment access, independent of fixture orchestration.
  * @rule Borrows clients; does not initialize or dispose resources.

package/dist/{datasource.d.ts → access/datasource.d.ts}

@@ -1,6 +1,6 @@
 import type { Client } from "./client.js";
 import type { ClientProvider } from "./client-provider.js";
-import type { Service } from "./service.js";
+import type { Service } from "../system/service.js";
 /** A source of data; accessible environments implement ClientProvider directly. */
 export interface DataSource<C extends Client> extends ClientProvider<C> {
     /** Safe descriptive metadata for discovery; never credentials or part of clientKey. */

package/dist/execution.d.ts

@@ -1,4 +1,4 @@
-import type { Service } from "./service.js";
+import type { Service } from "./system/service.js";
 /** Stable capability identity, independent of access configuration. */
 export interface Operation {
     readonly name: string;

package/dist/index.d.ts

@@ -1,18 +1,19 @@
-export * from "./client.js";
-export * from "./client-provider.js";
-export * from "./context.js";
-export { ClientManager } from "./client-manager.js";
-export * from "./datasource.js";
-export * from "./workload.js";
-export * from "./forge.js";
-export * from "./repository.js";
-export * from "./component.js";
-export * from "./host.js";
-export * from "./environment.js";
-export * from "./service.js";
+export * from "./access/client.js";
+export * from "./access/client-provider.js";
+export * from "./access/context.js";
+export { ClientManager } from "./access/client-manager.js";
+export * from "./access/datasource.js";
+export * from "./system/workload.js";
+export * from "./system/forge.js";
+export * from "./system/repository.js";
+export * from "./system/pull-request.js";
+export * from "./system/component.js";
+export * from "./system/host.js";
+export * from "./system/environment.js";
+export * from "./system/service.js";
 export * from "./execution.js";
-export type { Product } from "./product.js";
-export * from "./knowledge.js";
+export type { Product } from "./system/product.js";
+export * from "./system/knowledge.js";
 export type { Identity } from "./identity.js";
-export * from "./file.js";
-export * from "./resource.js";
+export * from "./access/file.js";
+export * from "./system/resource.js";

package/dist/index.js

@@ -1,16 +1,17 @@
-export * from "./client.js";
-export * from "./client-provider.js";
-export * from "./context.js";
-export { ClientManager } from "./client-manager.js";
-export * from "./datasource.js";
-export * from "./workload.js";
-export * from "./forge.js";
-export * from "./repository.js";
-export * from "./component.js";
-export * from "./host.js";
-export * from "./environment.js";
-export * from "./service.js";
+export * from "./access/client.js";
+export * from "./access/client-provider.js";
+export * from "./access/context.js";
+export { ClientManager } from "./access/client-manager.js";
+export * from "./access/datasource.js";
+export * from "./system/workload.js";
+export * from "./system/forge.js";
+export * from "./system/repository.js";
+export * from "./system/pull-request.js";
+export * from "./system/component.js";
+export * from "./system/host.js";
+export * from "./system/environment.js";
+export * from "./system/service.js";
 export * from "./execution.js";
-export * from "./knowledge.js";
-export * from "./file.js";
-export * from "./resource.js";
+export * from "./system/knowledge.js";
+export * from "./access/file.js";
+export * from "./system/resource.js";

package/dist/system/pull-request.d.ts

@@ -0,0 +1,27 @@
+import type { Identity } from "../identity.js";
+import { type Repository } from "./repository.js";
+export type PullRequestState = "open" | "merged" | "closed";
+/**
+ * A code-review proposal owned by its target Repository (GitHub PR or GitLab MR).
+ * @spec Identity is target Repository + repository-local number; source, revisions and state are metadata.
+ * @spec Omitted facts are unknown. A missing state or mergedAt never proves closure or merge.
+ * @see {@link component://docs/pull-request.md}
+ */
+export interface PullRequest extends Identity {
+    readonly repository: Repository;
+    /** Positive GitHub PR number or GitLab MR iid, not the provider's global database ID. */
+    readonly number: number;
+    readonly title?: string;
+    readonly url?: string;
+    readonly state?: PullRequestState;
+    readonly sourceRepository?: Repository;
+    readonly sourceBranch?: string;
+    readonly targetBranch?: string;
+    readonly headSha?: string;
+    readonly baseSha?: string;
+    /** Provider timestamps, represented as ISO 8601 strings. */
+    readonly updatedAt?: string;
+    readonly mergedAt?: string;
+}
+/** Derive identity from fields; never trust a consumer's display URL or repository id method. */
+export declare function pullRequestId(pr: Pick<PullRequest, "repository" | "number">): string;

package/dist/system/pull-request.js

@@ -0,0 +1,5 @@
+import { repositoryId } from "./repository.js";
+/** Derive identity from fields; never trust a consumer's display URL or repository id method. */
+export function pullRequestId(pr) {
+    return `${repositoryId(pr.repository)}#${pr.number}`;
+}

package/dist/{repository.d.ts → system/repository.d.ts}

@@ -1,5 +1,5 @@
 import type { Forge } from "./forge.js";
-import type { Identity } from "./identity.js";
+import type { Identity } from "../identity.js";
 /**
  * A repository identified by its Forge and path within that Forge.
  * @spec forkedFrom is the direct parent path within the same Forge instance, not repository identity.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@compforge/harness-common",
-  "version": "0.5.5",
+  "version": "0.5.7",
   "description": "Execution-neutral client lifecycle and datasource contracts for harnesses",
   "type": "module",
   "scripts": {