hungry-ghost-hive 0.43.0 → 0.43.2

package/src/utils/story-status.test.ts

@@ -0,0 +1,74 @@
+// Licensed under the Hungry Ghost Hive License. See LICENSE.
+
+import { describe, expect, it } from 'vitest';
+import { STORY_STATUS_ORDER, isForwardTransition, isStatusRegression } from './story-status.js';
+
+describe('STORY_STATUS_ORDER', () => {
+  it('contains all expected statuses in order', () => {
+    expect(STORY_STATUS_ORDER).toEqual([
+      'draft',
+      'estimated',
+      'planned',
+      'in_progress',
+      'review',
+      'qa',
+      'qa_failed',
+      'pr_submitted',
+      'merged',
+    ]);
+  });
+});
+
+describe('isStatusRegression', () => {
+  it('detects regression to earlier status', () => {
+    expect(isStatusRegression('in_progress', 'draft')).toBe(true);
+    expect(isStatusRegression('merged', 'planned')).toBe(true);
+    expect(isStatusRegression('review', 'estimated')).toBe(true);
+  });
+
+  it('returns false for forward transitions', () => {
+    expect(isStatusRegression('draft', 'estimated')).toBe(false);
+    expect(isStatusRegression('planned', 'in_progress')).toBe(false);
+    expect(isStatusRegression('review', 'merged')).toBe(false);
+  });
+
+  it('returns false for same status', () => {
+    expect(isStatusRegression('planned', 'planned')).toBe(false);
+  });
+
+  it('returns false for unknown statuses', () => {
+    expect(isStatusRegression('unknown', 'planned')).toBe(false);
+    expect(isStatusRegression('planned', 'unknown')).toBe(false);
+  });
+});
+
+describe('isForwardTransition', () => {
+  it('allows forward transitions', () => {
+    expect(isForwardTransition('draft', 'estimated')).toBe(true);
+    expect(isForwardTransition('estimated', 'planned')).toBe(true);
+    expect(isForwardTransition('planned', 'in_progress')).toBe(true);
+    expect(isForwardTransition('in_progress', 'review')).toBe(true);
+    expect(isForwardTransition('review', 'pr_submitted')).toBe(true);
+    expect(isForwardTransition('pr_submitted', 'qa')).toBe(true);
+    expect(isForwardTransition('qa', 'merged')).toBe(true);
+  });
+
+  it('allows same-status (no-op)', () => {
+    expect(isForwardTransition('planned', 'planned')).toBe(true);
+    expect(isForwardTransition('in_progress', 'in_progress')).toBe(true);
+  });
+
+  it('prevents backward transitions', () => {
+    expect(isForwardTransition('in_progress', 'planned')).toBe(false);
+    expect(isForwardTransition('in_progress', 'draft')).toBe(false);
+    expect(isForwardTransition('review', 'planned')).toBe(false);
+    expect(isForwardTransition('merged', 'planned')).toBe(false);
+    expect(isForwardTransition('qa', 'planned')).toBe(false);
+  });
+
+  it('handles qa_failed ordering correctly', () => {
+    expect(isForwardTransition('review', 'qa_failed')).toBe(true);
+    expect(isForwardTransition('in_progress', 'qa_failed')).toBe(true);
+    expect(isForwardTransition('qa', 'qa_failed')).toBe(false);
+  });
+});

package/src/utils/story-status.ts

@@ -0,0 +1,62 @@
+// Licensed under the Hungry Ghost Hive License. See LICENSE.
+
+/**
+ * Ordered lifecycle stages for Hive stories.
+ * Used to prevent bidirectional sync from regressing story status
+ * and to determine canonical status during cluster story merges.
+ */
+export const STORY_STATUS_ORDER = [
+  'draft',
+  'estimated',
+  'planned',
+  'in_progress',
+  'review',
+  'qa',
+  'qa_failed',
+  'pr_submitted',
+  'merged',
+] as const;
+
+export type StoryStatusValue = (typeof STORY_STATUS_ORDER)[number];
+
+/**
+ * Status progression order for the Hive pipeline.
+ * Higher numbers represent further progress. Used to prevent
+ * sync from regressing stories backward.
+ *
+ * Note: qa_failed has the same order as review (4) because it
+ * represents a lateral step, not a regression.
+ */
+const STATUS_ORDER: Record<string, number> = {
+  draft: 0,
+  estimated: 1,
+  planned: 2,
+  in_progress: 3,
+  review: 4,
+  pr_submitted: 5,
+  qa: 6,
+  qa_failed: 4, // allowed as a backward step from qa
+  merged: 7,
+};
+
+/**
+ * Check if transitioning from currentStatus to newStatus would be a regression.
+ * Returns true if the new status is earlier in the lifecycle than the current one.
+ * Unknown statuses are allowed through (returns false).
+ */
+export function isStatusRegression(currentStatus: string, newStatus: string): boolean {
+  const currentIdx = STORY_STATUS_ORDER.indexOf(currentStatus as StoryStatusValue);
+  const newIdx = STORY_STATUS_ORDER.indexOf(newStatus as StoryStatusValue);
+  if (currentIdx === -1 || newIdx === -1) return false;
+  return newIdx < currentIdx;
+}
+
+/**
+ * Check whether transitioning from currentStatus to newStatus is a
+ * forward (or lateral) move in the pipeline.
+ */
+export function isForwardTransition(currentStatus: string, newStatus: string): boolean {
+  const currentOrder = STATUS_ORDER[currentStatus] ?? -1;
+  const newOrder = STATUS_ORDER[newStatus] ?? -1;
+  return newOrder >= currentOrder;
+}