smol-symphony 0.2.0 → 0.3.0

package/dist/tests/core/reconcile-issue.test.js

@@ -0,0 +1,116 @@
+// Pure unit tests for src/core/schedule/reconcile-issue.ts.
+//
+// decideReconcileForIssue is pure (data) -> data, so no mocks are needed:
+// the fresh tracker view and the states map are plain data constructed inline.
+import { test } from 'node:test';
+import assert from 'node:assert/strict';
+import { decideReconcileForIssue, decideReconcileForState } from '../../src/core/schedule/reconcile-issue.js';
+import { makeIssue } from '../helpers/fixtures.js';
+// --- fixtures -------------------------------------------------------------
+function st(role) {
+    return { role };
+}
+/** A workflow with an active, a holding, and a terminal state. */
+const STATES = {
+    Todo: st('active'),
+    Doing: st('active'),
+    Blocked: st('holding'),
+    Done: st('terminal'),
+};
+// --- missing row ----------------------------------------------------------
+test('undefined fresh row → terminate without cleanup (tracker_state_missing)', () => {
+    const action = decideReconcileForIssue(undefined, STATES);
+    assert.deepEqual(action, {
+        kind: 'terminate',
+        cleanup: false,
+        reason: 'tracker_state_missing',
+    });
+});
+// --- terminal-role state --------------------------------------------------
+test('terminal-role state → terminate with cleanup (tracker_state_terminal)', () => {
+    const action = decideReconcileForIssue(makeIssue({ state: 'Done' }), STATES);
+    assert.deepEqual(action, {
+        kind: 'terminate',
+        cleanup: true,
+        reason: 'tracker_state_terminal',
+    });
+});
+test('terminal-role match is case-insensitive (tracker stores lowercased dir)', () => {
+    const action = decideReconcileForIssue(makeIssue({ state: 'done' }), STATES);
+    assert.deepEqual(action, {
+        kind: 'terminate',
+        cleanup: true,
+        reason: 'tracker_state_terminal',
+    });
+});
+// --- active-role state ----------------------------------------------------
+test('active-role state → refresh', () => {
+    const action = decideReconcileForIssue(makeIssue({ state: 'Doing' }), STATES);
+    assert.deepEqual(action, { kind: 'refresh' });
+});
+test('active-role match is case-insensitive', () => {
+    const action = decideReconcileForIssue(makeIssue({ state: 'TODO' }), STATES);
+    assert.deepEqual(action, { kind: 'refresh' });
+});
+// --- holding / other non-active states ------------------------------------
+test('holding-role state → terminate without cleanup (tracker_state_non_active)', () => {
+    const action = decideReconcileForIssue(makeIssue({ state: 'Blocked' }), STATES);
+    assert.deepEqual(action, {
+        kind: 'terminate',
+        cleanup: false,
+        reason: 'tracker_state_non_active',
+    });
+});
+test('undeclared state → terminate without cleanup (tracker_state_non_active)', () => {
+    const action = decideReconcileForIssue(makeIssue({ state: 'Limbo' }), STATES);
+    assert.deepEqual(action, {
+        kind: 'terminate',
+        cleanup: false,
+        reason: 'tracker_state_non_active',
+    });
+});
+// --- branch precedence: terminal wins over active when both contain the name
+// (defensive — the reference checks terminal first). With a sane workflow a
+// name is only one role, but if a name were declared twice we keep ordering.
+test('terminal check precedes active check', () => {
+    // A workflow that (pathologically) lists the same lowercased name in both
+    // roles via distinct keys: terminal must win because it is consulted first.
+    const states = {
+        Review: st('terminal'),
+        review: st('active'),
+    };
+    const action = decideReconcileForIssue(makeIssue({ state: 'Review' }), states);
+    assert.equal(action.kind, 'terminate');
+    if (action.kind === 'terminate') {
+        assert.equal(action.reason, 'tracker_state_terminal');
+        // Canonical lookup scans declared keys in order; the FIRST whose lowercased
+        // form equals 'review' is "Review" (terminal), so cleanup is true.
+        assert.equal(action.cleanup, true);
+    }
+});
+// --- empty states map -----------------------------------------------------
+test('empty states map → any state is non-active → terminate without cleanup', () => {
+    const action = decideReconcileForIssue(makeIssue({ state: 'Whatever' }), {});
+    assert.deepEqual(action, {
+        kind: 'terminate',
+        cleanup: false,
+        reason: 'tracker_state_non_active',
+    });
+});
+// --- decideReconcileForState (the state-keyed core the operator-move reducer shares) --
+// decideReconcileForIssue is a thin wrapper over decideReconcileForState (reading only
+// `fresh.state`), so the two MUST agree for every state — the property that keeps the
+// dashboard move path and the poll-loop reconcile from drifting (issue 211).
+test('decideReconcileForState mirrors decideReconcileForIssue across every role', () => {
+    for (const state of ['Done', 'Todo', 'Doing', 'Blocked', 'Limbo']) {
+        assert.deepEqual(decideReconcileForState(state, STATES), decideReconcileForIssue(makeIssue({ state }), STATES), `state ${state} must decide identically via the state- and issue-keyed deciders`);
+    }
+});
+test('decideReconcileForState(undefined) → terminate without cleanup (missing row)', () => {
+    assert.deepEqual(decideReconcileForState(undefined, STATES), {
+        kind: 'terminate',
+        cleanup: false,
+        reason: 'tracker_state_missing',
+    });
+});
+//# sourceMappingURL=reconcile-issue.test.js.map

package/dist/tests/core/reconcile-issue.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"reconcile-issue.test.js","sourceRoot":"","sources":["../../../tests/core/reconcile-issue.test.ts"],"names":[],"mappings":"AAAA,4DAA4D;AAC5D,EAAE;AACF,0EAA0E;AAC1E,+EAA+E;AAE/E,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,MAAM,MAAM,oBAAoB,CAAC;AAExC,OAAO,EAAE,uBAAuB,EAAE,uBAAuB,EAAE,MAAM,4CAA4C,CAAC;AAE9G,OAAO,EAAE,SAAS,EAAE,MAAM,wBAAwB,CAAC;AAEnD,6EAA6E;AAE7E,SAAS,EAAE,CAAC,IAAyB;IACnC,OAAO,EAAE,IAAI,EAAE,CAAC;AAClB,CAAC;AAED,kEAAkE;AAClE,MAAM,MAAM,GAAgC;IAC1C,IAAI,EAAE,EAAE,CAAC,QAAQ,CAAC;IAClB,KAAK,EAAE,EAAE,CAAC,QAAQ,CAAC;IACnB,OAAO,EAAE,EAAE,CAAC,SAAS,CAAC;IACtB,IAAI,EAAE,EAAE,CAAC,UAAU,CAAC;CACrB,CAAC;AAEF,6EAA6E;AAE7E,IAAI,CAAC,yEAAyE,EAAE,GAAG,EAAE;IACnF,MAAM,MAAM,GAAG,uBAAuB,CAAC,SAAS,EAAE,MAAM,CAAC,CAAC;IAC1D,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE;QACvB,IAAI,EAAE,WAAW;QACjB,OAAO,EAAE,KAAK;QACd,MAAM,EAAE,uBAAuB;KAChC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,6EAA6E;AAE7E,IAAI,CAAC,uEAAuE,EAAE,GAAG,EAAE;IACjF,MAAM,MAAM,GAAG,uBAAuB,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,EAAE,MAAM,CAAC,CAAC;IAC7E,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE;QACvB,IAAI,EAAE,WAAW;QACjB,OAAO,EAAE,IAAI;QACb,MAAM,EAAE,wBAAwB;KACjC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,yEAAyE,EAAE,GAAG,EAAE;IACnF,MAAM,MAAM,GAAG,uBAAuB,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,EAAE,MAAM,CAAC,CAAC;IAC7E,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE;QACvB,IAAI,EAAE,WAAW;QACjB,OAAO,EAAE,IAAI;QACb,MAAM,EAAE,wBAAwB;KACjC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,6EAA6E;AAE7E,IAAI,CAAC,6BAA6B,EAAE,GAAG,EAAE;IACvC,MAAM,MAAM,GAAG,uBAAuB,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC,EAAE,MAAM,CAAC,CAAC;IAC9E,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;AAChD,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,uCAAuC,EAAE,GAAG,EAAE;IACjD,MAAM,MAAM,GAAG,uBAAuB,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,EAAE,MAAM,CAAC,CAAC;IAC7E,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;AAChD,CAAC,CAAC,CAAC;AAEH,6EAA6E;AAE7E,IAAI,CAAC,2EAA2E,EAAE,GAAG,EAAE;IACrF,MAAM,MAAM,GAAG,uBAAuB,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,SAAS,EAAE,CAAC,EAAE,MAAM,CAAC,CAAC;IAChF,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE;QACvB,IAAI,EAAE,WAAW;QACjB,OAAO,EAAE,KAAK;QACd,MAAM,EAAE,0BAA0B;KACnC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,yEAAyE,EAAE,GAAG,EAAE;IACnF,MAAM,MAAM,GAAG,uBAAuB,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC,EAAE,MAAM,CAAC,CAAC;IAC9E,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE;QACvB,IAAI,EAAE,WAAW;QACjB,OAAO,EAAE,KAAK;QACd,MAAM,EAAE,0BAA0B;KACnC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,8EAA8E;AAC9E,4EAA4E;AAC5E,6EAA6E;AAE7E,IAAI,CAAC,sCAAsC,EAAE,GAAG,EAAE;IAChD,0EAA0E;IAC1E,4EAA4E;IAC5E,MAAM,MAAM,GAAgC;QAC1C,MAAM,EAAE,EAAE,CAAC,UAAU,CAAC;QACtB,MAAM,EAAE,EAAE,CAAC,QAAQ,CAAC;KACrB,CAAC;IACF,MAAM,MAAM,GAAG,uBAAuB,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,QAAQ,EAAE,CAAC,EAAE,MAAM,CAAC,CAAC;IAC/E,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,IAAI,EAAE,WAAW,CAAC,CAAC;IACvC,IAAI,MAAM,CAAC,IAAI,KAAK,WAAW,EAAE,CAAC;QAChC,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,MAAM,EAAE,wBAAwB,CAAC,CAAC;QACtD,4EAA4E;QAC5E,mEAAmE;QACnE,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC;IACrC,CAAC;AACH,CAAC,CAAC,CAAC;AAEH,6EAA6E;AAE7E,IAAI,CAAC,wEAAwE,EAAE,GAAG,EAAE;IAClF,MAAM,MAAM,GAAG,uBAAuB,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,UAAU,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;IAC7E,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE;QACvB,IAAI,EAAE,WAAW;QACjB,OAAO,EAAE,KAAK;QACd,MAAM,EAAE,0BAA0B;KACnC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,yFAAyF;AACzF,uFAAuF;AACvF,sFAAsF;AACtF,6EAA6E;AAE7E,IAAI,CAAC,2EAA2E,EAAE,GAAG,EAAE;IACrF,KAAK,MAAM,KAAK,IAAI,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,EAAE,SAAS,EAAE,OAAO,CAAC,EAAE,CAAC;QAClE,MAAM,CAAC,SAAS,CACd,uBAAuB,CAAC,KAAK,EAAE,MAAM,CAAC,EACtC,uBAAuB,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,MAAM,CAAC,EACrD,SAAS,KAAK,kEAAkE,CACjF,CAAC;IACJ,CAAC;AACH,CAAC,CAAC,CAAC;AAEH,IAAI,CAAC,8EAA8E,EAAE,GAAG,EAAE;IACxF,MAAM,CAAC,SAAS,CAAC,uBAAuB,CAAC,SAAS,EAAE,MAAM,CAAC,EAAE;QAC3D,IAAI,EAAE,WAAW;QACjB,OAAO,EAAE,KAAK;QACd,MAAM,EAAE,uBAAuB;KAChC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}