Skip to content

Commit 7ce373e

Browse files
committed
fix(ci): unbreak main — drop the orphaned AUTOMATION_COUNTED_ACTIONS export
Closes #10109
1 parent b900ef0 commit 7ce373e

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

src/review/automation-rate.ts

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -47,13 +47,14 @@ export const AUTOMATION_RATE_PROVENANCE_HORIZON_ISO = "2026-07-29T00:00:00.000Z"
4747
const AUTOMATION_ENACTING_ACTIONS = ["merge", "close"] as const;
4848

4949
/** The action recording that the gate declined to decide and handed the PR to a person. Module-private for
50-
* the same reason as the set above. */
50+
* the same reason as the set above.
51+
*
52+
* Together with the enacting set, this is what DECIDES a PR -- the published set the fold keys `decided` on.
53+
* There is deliberately no exported union of the two: #10013 removed the `action IN (...)` SQL filter that
54+
* was its only consumer, and an exported constant naming a set nothing selects on invites a reader to
55+
* believe the read still restricts by it. The fold tests each half directly instead. */
5156
const AUTOMATION_HOLD_ACTION = "hold";
5257

53-
/** Every action that DECIDES a PR -- an enacted decision, or a hold. This is the published set the fold
54-
* keys `decided` on; the read no longer restricts by it (#10013), so it does not gate what rows are seen. */
55-
export const AUTOMATION_COUNTED_ACTIONS: readonly string[] = [...AUTOMATION_ENACTING_ACTIONS, AUTOMATION_HOLD_ACTION];
56-
5758
/** How completely a week could be measured. `full` weeks see every manual signal; `holds_only` weeks predate
5859
* the provenance fields and can only under-count manual work. */
5960
export type AutomationWeekBasis = "full" | "holds_only";

0 commit comments

Comments
 (0)