jackherd 1.9.0 → 1.11.0

package/src/watch/asking-lanes.ts

@@ -0,0 +1,40 @@
+import { questionSections } from './report-questions'
+
+/** The questions sections of a report as one string; empty without a report or without an open question. */
+export const questionsFingerprint = (text: string | null): string => {
+  if (text === null) return ''
+  const lines = text.split('\n')
+  return questionSections(lines)
+    .map(({ start, end }) => lines.slice(start, end).join('\n'))
+    .join('\n')
+}
+
+/**
+ * The working lanes whose report asks something other than what was last remembered for them (`current` and
+ * `remembered` are fingerprints per lane): a lane waiting on its background subagents ends no turn, so its question
+ * would wait for them. A lane without a remembered entry is unseen and asks nothing; its report is the baseline.
+ */
+export const askingLanes = (
+  working: readonly string[],
+  current: Readonly<Record<string, string>>,
+  remembered: Readonly<Record<string, string>>,
+): string[] =>
+  working.filter((lane) => lane in remembered && (current[lane] ?? '') !== '' && current[lane] !== remembered[lane])
+
+/**
+ * The fingerprints to remember for the watched lanes (the keys of `current`): a lane whose report was `shown`, that
+ * asks nothing now or is seen for the first time takes what it has now; any other keeps what it had, so a question
+ * not yet shown still fires. A lane no longer watched is forgotten.
+ */
+export const recordReportQuestions = (
+  remembered: Readonly<Record<string, string>>,
+  current: Readonly<Record<string, string>>,
+  shown: readonly string[],
+): Record<string, string> =>
+  Object.fromEntries(
+    Object.entries(current).map(([lane, questions]) => {
+      const kept = remembered[lane]
+      const takesCurrent = kept === undefined || questions === '' || shown.includes(lane)
+      return [lane, takesCurrent ? questions : kept]
+    }),
+  )

package/src/watch/base-tip.ts

@@ -1,3 +1,5 @@
+import { pullHeadRef } from '../plans/plan-heads'
+
 const COMMIT = /^[0-9a-f]{40}$/
 
 /** The commit of `refs/heads/<branch>` in a `git ls-remote` answer; null when it is not listed. */
@@ -16,3 +18,12 @@ export const remoteTipOf = (lsRemote: string, branch: string): string | null =>
  */
 export const baseFetchNeeded = (remoteTip: string | null, localTip: string | null): boolean =>
   remoteTip !== null && remoteTip !== localTip
+
+/**
+ * The pull requests whose head moved past the herd's ref of it (`pullHeadRef`), or that have none yet: the
+ * watcher's list carries each head commit, so an unchanged head costs no fetch.
+ */
+export const movedHeads = <Head extends { number: number; headCommit: string }>(
+  heads: readonly Head[],
+  localCommitOf: (ref: string) => string | null,
+): Head[] => heads.filter(({ number, headCommit }) => baseFetchNeeded(headCommit, localCommitOf(pullHeadRef(number))))