@jmoyers/harness 0.1.10 → 0.1.20

package/src/store/control-plane-store-normalize.ts

@@ -17,17 +17,12 @@ import type {
   ControlPlaneProjectTaskFocusMode,
   ControlPlaneProjectThreadSpawnMode,
   ControlPlaneRepositoryRecord,
-  ControlPlaneTaskLinearPriority,
-  ControlPlaneTaskLinearRecord,
   ControlPlaneTaskRecord,
   ControlPlaneTaskScopeKind,
   ControlPlaneTaskStatus,
   ControlPlaneTelemetryRecord,
-  TaskLinearInput,
 } from './control-plane-store-types.ts';
 
-const DATE_ONLY_PATTERN = /^\d{4}-\d{2}-\d{2}$/;
-
 export function asRecord(value: unknown): Record<string, unknown> {
   if (typeof value !== 'object' || value === null) {
     throw new Error('expected object row');
@@ -151,16 +146,8 @@ function normalizeRuntimeStatusModel(value: unknown): StreamSessionStatusModel |
   const attentionReason = asStringOrNull(model.attentionReason, 'attentionReason');
   const lastKnownWork = asStringOrNull(model.lastKnownWork, 'lastKnownWork');
   const lastKnownWorkAt = asStringOrNull(model.lastKnownWorkAt, 'lastKnownWorkAt');
-  const phaseHintRaw = asStringOrNull(model.phaseHint, 'phaseHint');
+  const activityHintRaw = normalizeLegacyActivityHint(model.activityHint);
   const observedAt = asString(model.observedAt, 'observedAt');
-  if (
-    phaseHintRaw !== null &&
-    phaseHintRaw !== 'needs-action' &&
-    phaseHintRaw !== 'working' &&
-    phaseHintRaw !== 'idle'
-  ) {
-    throw new Error('expected phaseHint enum value');
-  }
   return {
     runtimeStatus: runtimeStatusRaw,
     phase: phaseRaw,
@@ -170,11 +157,40 @@ function normalizeRuntimeStatusModel(value: unknown): StreamSessionStatusModel |
     attentionReason,
     lastKnownWork,
     lastKnownWorkAt,
-    phaseHint: phaseHintRaw,
+    activityHint: activityHintRaw,
     observedAt,
   };
 }
 
+function normalizeLegacyActivityHint(value: unknown): StreamSessionStatusModel['activityHint'] {
+  if (value === undefined || value === null) {
+    return null;
+  }
+  if (typeof value === 'string') {
+    if (value === 'needs-action' || value === 'working' || value === 'idle') {
+      return value;
+    }
+    throw new Error('expected activityHint enum value');
+  }
+  if (typeof value === 'object' && value !== null && !Array.isArray(value)) {
+    const legacy = value as Record<string, unknown>;
+    const legacyCandidate =
+      typeof legacy.kind === 'string'
+        ? legacy.kind
+        : typeof legacy.value === 'string'
+          ? legacy.value
+          : null;
+    if (
+      legacyCandidate === 'needs-action' ||
+      legacyCandidate === 'working' ||
+      legacyCandidate === 'idle'
+    ) {
+      return legacyCandidate;
+    }
+  }
+  return null;
+}
+
 export function normalizeStoredDirectoryRow(value: unknown): ControlPlaneDirectoryRecord {
   const row = asRecord(value);
   return {
@@ -287,231 +303,6 @@ function normalizeRepositoryMetadata(value: unknown): Record<string, unknown> {
   }
 }
 
-export function defaultTaskLinearRecord(): ControlPlaneTaskLinearRecord {
-  return {
-    issueId: null,
-    identifier: null,
-    url: null,
-    teamId: null,
-    projectId: null,
-    projectMilestoneId: null,
-    cycleId: null,
-    stateId: null,
-    assigneeId: null,
-    priority: null,
-    estimate: null,
-    dueDate: null,
-    labelIds: [],
-  };
-}
-
-function normalizeOptionalTaskLinearString(value: string | null, field: string): string | null {
-  if (value === null) {
-    return null;
-  }
-  return normalizeNonEmptyLabel(value, field);
-}
-
-function normalizeTaskLinearPriority(
-  value: number | null,
-  field: string,
-): ControlPlaneTaskLinearPriority | null {
-  if (value === null) {
-    return null;
-  }
-  if (!Number.isInteger(value) || value < 0 || value > 4) {
-    throw new Error(`expected integer [0..4] for ${field}`);
-  }
-  return value as ControlPlaneTaskLinearPriority;
-}
-
-function normalizeTaskLinearEstimate(value: number | null, field: string): number | null {
-  if (value === null) {
-    return null;
-  }
-  if (!Number.isInteger(value) || value < 0) {
-    throw new Error(`expected non-negative integer for ${field}`);
-  }
-  return value;
-}
-
-function normalizeTaskLinearDueDate(value: string | null, field: string): string | null {
-  if (value === null) {
-    return null;
-  }
-  const normalized = value.trim();
-  if (!DATE_ONLY_PATTERN.test(normalized)) {
-    throw new Error(`expected YYYY-MM-DD for ${field}`);
-  }
-  return normalized;
-}
-
-function normalizeTaskLinearLabelIds(
-  value: readonly string[] | null,
-  field: string,
-): readonly string[] {
-  if (value === null) {
-    return [];
-  }
-  return uniqueValues(
-    value.map((entry, idx) => normalizeNonEmptyLabel(entry, `${field}[${String(idx)}]`)),
-  );
-}
-
-function parseTaskLinearInputRecord(
-  record: Record<string, unknown>,
-  field: string,
-): TaskLinearInput {
-  const parsed: TaskLinearInput = {};
-  if ('issueId' in record) {
-    parsed.issueId = asStringOrNull(record.issueId, `${field}.issueId`);
-  }
-  if ('identifier' in record) {
-    parsed.identifier = asStringOrNull(record.identifier, `${field}.identifier`);
-  }
-  if ('url' in record) {
-    parsed.url = asStringOrNull(record.url, `${field}.url`);
-  }
-  if ('teamId' in record) {
-    parsed.teamId = asStringOrNull(record.teamId, `${field}.teamId`);
-  }
-  if ('projectId' in record) {
-    parsed.projectId = asStringOrNull(record.projectId, `${field}.projectId`);
-  }
-  if ('projectMilestoneId' in record) {
-    parsed.projectMilestoneId = asStringOrNull(
-      record.projectMilestoneId,
-      `${field}.projectMilestoneId`,
-    );
-  }
-  if ('cycleId' in record) {
-    parsed.cycleId = asStringOrNull(record.cycleId, `${field}.cycleId`);
-  }
-  if ('stateId' in record) {
-    parsed.stateId = asStringOrNull(record.stateId, `${field}.stateId`);
-  }
-  if ('assigneeId' in record) {
-    parsed.assigneeId = asStringOrNull(record.assigneeId, `${field}.assigneeId`);
-  }
-  if ('priority' in record) {
-    parsed.priority = asNumberOrNull(record.priority, `${field}.priority`);
-  }
-  if ('estimate' in record) {
-    parsed.estimate = asNumberOrNull(record.estimate, `${field}.estimate`);
-  }
-  if ('dueDate' in record) {
-    parsed.dueDate = asStringOrNull(record.dueDate, `${field}.dueDate`);
-  }
-  if ('labelIds' in record) {
-    const raw = record.labelIds;
-    if (raw === null) {
-      parsed.labelIds = null;
-    } else if (Array.isArray(raw) && raw.every((entry) => typeof entry === 'string')) {
-      parsed.labelIds = raw;
-    } else {
-      throw new Error(`expected string array or null for ${field}.labelIds`);
-    }
-  }
-  return parsed;
-}
-
-export function applyTaskLinearInput(
-  base: ControlPlaneTaskLinearRecord,
-  input: TaskLinearInput,
-): ControlPlaneTaskLinearRecord {
-  return {
-    issueId:
-      input.issueId === undefined
-        ? base.issueId
-        : normalizeOptionalTaskLinearString(input.issueId, 'linear.issueId'),
-    identifier:
-      input.identifier === undefined
-        ? base.identifier
-        : normalizeOptionalTaskLinearString(input.identifier, 'linear.identifier'),
-    url:
-      input.url === undefined
-        ? base.url
-        : normalizeOptionalTaskLinearString(input.url, 'linear.url'),
-    teamId:
-      input.teamId === undefined
-        ? base.teamId
-        : normalizeOptionalTaskLinearString(input.teamId, 'linear.teamId'),
-    projectId:
-      input.projectId === undefined
-        ? base.projectId
-        : normalizeOptionalTaskLinearString(input.projectId, 'linear.projectId'),
-    projectMilestoneId:
-      input.projectMilestoneId === undefined
-        ? base.projectMilestoneId
-        : normalizeOptionalTaskLinearString(input.projectMilestoneId, 'linear.projectMilestoneId'),
-    cycleId:
-      input.cycleId === undefined
-        ? base.cycleId
-        : normalizeOptionalTaskLinearString(input.cycleId, 'linear.cycleId'),
-    stateId:
-      input.stateId === undefined
-        ? base.stateId
-        : normalizeOptionalTaskLinearString(input.stateId, 'linear.stateId'),
-    assigneeId:
-      input.assigneeId === undefined
-        ? base.assigneeId
-        : normalizeOptionalTaskLinearString(input.assigneeId, 'linear.assigneeId'),
-    priority:
-      input.priority === undefined
-        ? base.priority
-        : normalizeTaskLinearPriority(input.priority, 'linear.priority'),
-    estimate:
-      input.estimate === undefined
-        ? base.estimate
-        : normalizeTaskLinearEstimate(input.estimate, 'linear.estimate'),
-    dueDate:
-      input.dueDate === undefined
-        ? base.dueDate
-        : normalizeTaskLinearDueDate(input.dueDate, 'linear.dueDate'),
-    labelIds:
-      input.labelIds === undefined
-        ? base.labelIds
-        : normalizeTaskLinearLabelIds(input.labelIds, 'linear.labelIds'),
-  };
-}
-
-export function serializeTaskLinear(linear: ControlPlaneTaskLinearRecord): string {
-  return JSON.stringify({
-    issueId: linear.issueId,
-    identifier: linear.identifier,
-    url: linear.url,
-    teamId: linear.teamId,
-    projectId: linear.projectId,
-    projectMilestoneId: linear.projectMilestoneId,
-    cycleId: linear.cycleId,
-    stateId: linear.stateId,
-    assigneeId: linear.assigneeId,
-    priority: linear.priority,
-    estimate: linear.estimate,
-    dueDate: linear.dueDate,
-    labelIds: [...linear.labelIds],
-  });
-}
-
-function normalizeTaskLinear(value: unknown): ControlPlaneTaskLinearRecord {
-  if (typeof value !== 'string') {
-    throw new Error('expected string for linear_json');
-  }
-  let parsed: unknown;
-  try {
-    parsed = JSON.parse(value) as unknown;
-  } catch {
-    return defaultTaskLinearRecord();
-  }
-  if (typeof parsed !== 'object' || parsed === null || Array.isArray(parsed)) {
-    return defaultTaskLinearRecord();
-  }
-  return applyTaskLinearInput(
-    defaultTaskLinearRecord(),
-    parseTaskLinearInputRecord(parsed as Record<string, unknown>, 'linear_json'),
-  );
-}
-
 function normalizeTaskStatus(value: unknown): ControlPlaneTaskStatus {
   const status = asString(value, 'status');
   if (
@@ -615,6 +406,7 @@ export function normalizeTaskRow(value: unknown): ControlPlaneTaskRecord {
   const row = asRecord(value);
   const repositoryId = asStringOrNull(row.repository_id, 'repository_id');
   const projectId = asStringOrNull(row.project_id, 'project_id');
+  const bodyValue = row.body ?? row.description;
   return {
     taskId: asString(row.task_id, 'task_id'),
     tenantId: asString(row.tenant_id, 'tenant_id'),
@@ -624,7 +416,7 @@ export function normalizeTaskRow(value: unknown): ControlPlaneTaskRecord {
     scopeKind: normalizeTaskScopeKind(row.scope_kind, repositoryId, projectId),
     projectId,
     title: asString(row.title, 'title'),
-    description: asString(row.description, 'description'),
+    body: asString(bodyValue, 'body'),
     status: normalizeTaskStatus(row.status),
     orderIndex: asNumberOrNull(row.order_index, 'order_index') as number,
     claimedByControllerId: asStringOrNull(row.claimed_by_controller_id, 'claimed_by_controller_id'),
@@ -633,7 +425,6 @@ export function normalizeTaskRow(value: unknown): ControlPlaneTaskRecord {
     baseBranch: asStringOrNull(row.base_branch, 'base_branch'),
     claimedAt: asStringOrNull(row.claimed_at, 'claimed_at'),
     completedAt: asStringOrNull(row.completed_at, 'completed_at'),
-    linear: normalizeTaskLinear(row.linear_json),
     createdAt: asString(row.created_at, 'created_at'),
     updatedAt: asString(row.updated_at, 'updated_at'),
   };

package/src/store/control-plane-store-types.ts

@@ -59,7 +59,6 @@ export interface ControlPlaneTelemetrySummary {
 
 export type ControlPlaneTaskStatus = 'draft' | 'ready' | 'in-progress' | 'completed';
 export type ControlPlaneTaskScopeKind = 'global' | 'repository' | 'project';
-export type ControlPlaneTaskLinearPriority = 0 | 1 | 2 | 3 | 4;
 export type ControlPlaneProjectTaskFocusMode = 'balanced' | 'own-only';
 export type ControlPlaneProjectThreadSpawnMode = 'new-thread' | 'reuse-thread';
 export type ControlPlaneAutomationPolicyScope = 'global' | 'repository' | 'project';
@@ -72,38 +71,6 @@ export type ControlPlaneGitHubCiRollup =
   | 'neutral'
   | 'none';
 
-export interface ControlPlaneTaskLinearRecord {
-  readonly issueId: string | null;
-  readonly identifier: string | null;
-  readonly url: string | null;
-  readonly teamId: string | null;
-  readonly projectId: string | null;
-  readonly projectMilestoneId: string | null;
-  readonly cycleId: string | null;
-  readonly stateId: string | null;
-  readonly assigneeId: string | null;
-  readonly priority: ControlPlaneTaskLinearPriority | null;
-  readonly estimate: number | null;
-  readonly dueDate: string | null;
-  readonly labelIds: readonly string[];
-}
-
-export interface TaskLinearInput {
-  issueId?: string | null;
-  identifier?: string | null;
-  url?: string | null;
-  teamId?: string | null;
-  projectId?: string | null;
-  projectMilestoneId?: string | null;
-  cycleId?: string | null;
-  stateId?: string | null;
-  assigneeId?: string | null;
-  priority?: number | null;
-  estimate?: number | null;
-  dueDate?: string | null;
-  labelIds?: readonly string[] | null;
-}
-
 export interface ControlPlaneRepositoryRecord {
   readonly repositoryId: string;
   readonly tenantId: string;
@@ -126,7 +93,7 @@ export interface ControlPlaneTaskRecord {
   readonly scopeKind: ControlPlaneTaskScopeKind;
   readonly projectId: string | null;
   readonly title: string;
-  readonly description: string;
+  readonly body: string;
   readonly status: ControlPlaneTaskStatus;
   readonly orderIndex: number;
   readonly claimedByControllerId: string | null;
@@ -135,7 +102,6 @@ export interface ControlPlaneTaskRecord {
   readonly baseBranch: string | null;
   readonly claimedAt: string | null;
   readonly completedAt: string | null;
-  readonly linear: ControlPlaneTaskLinearRecord;
   readonly createdAt: string;
   readonly updatedAt: string;
 }