next-issue 2.7.0 → 2.9.0

package/README.md

@@ -379,14 +379,16 @@ silent skip.
 
 ## Retry an issue that waits for a person
 
-A hand-over leaves the state file with the used budgets in it, so the next run
-would hand the issue over again at once. To give the issue another try, take the
-`status:needs-human` label off and run with `--reset`. The flag puts the budgets
-and the findings back to zero and keeps the branch, the pull request and the
-step that the last run reached, so the work carries on where it stopped. A
-commit that you pushed to the pull request in the meantime is fetched first.
-The flag touches only an issue that the run handles, so the state of an issue
-that a label still holds back stays as it was.
+A hand-over marks the state file as handed over. To give the issue another try,
+take the `status:needs-human` label off. The next run that claims the issue puts
+the budgets and the findings back to zero, clears the mark and keeps the branch,
+the pull request and the step that the last run reached, so the work carries on
+where it stopped. A commit that you pushed to the pull request in the meantime
+is fetched first. An issue that still has the label is not claimed.
+
+The `--reset` flag does the same for any issue that the run handles, whether or
+not it was handed over. It touches only an issue that the run handles, so the
+state of an issue that a label still holds back stays as it was.
 
 ## Commands and options
 

package/dist/handover.mjs

@@ -55,7 +55,7 @@ export async function cleanUp(context, log, branch) {
 }
 export async function handOver(context, log, issue, state, reason) {
     log.event('hand-over', { reason }, 'quiet');
-    await writeState(context.repo, state);
+    await writeState(context.repo, { ...state, handedOver: true });
     await setStatus(context, issue, state.pr, context.config.labels.needsHuman);
     const body = `next-issue needs help: ${reason}`;
     if (state.pr === undefined) {

package/dist/pipeline.mjs

@@ -106,7 +106,11 @@ export async function processIssue(context, issue) {
         reviewRounds: 0,
         reviewLog: [],
     };
-    const state = saved === undefined ? fresh : context.reset ? resetState(saved) : saved;
+    const state = saved === undefined
+        ? fresh
+        : context.reset || saved.handedOver === true
+            ? resetState(saved)
+            : saved;
     try {
         const report = await work(context, issue, log, state, saved === undefined);
         log.event('issue.end', { outcome: report.outcome, reason: report.reason, ms: report.ms }, 'quiet');

package/dist/state.mjs

@@ -60,7 +60,8 @@ function isState(value) {
         (value.baseFix === undefined || isBaseFix(value.baseFix)) &&
         typeof value.reviewRounds === 'number' &&
         Array.isArray(value.reviewLog) &&
-        value.reviewLog.every(isReviewRound));
+        value.reviewLog.every(isReviewRound) &&
+        (value.handedOver === undefined || typeof value.handedOver === 'boolean'));
 }
 export async function savedIssues(repo) {
     let names;
@@ -79,7 +80,8 @@ export async function savedIssues(repo) {
     });
 }
 export function resetState(state) {
-    return { ...state, ciFixes: 0, commitFixes: 0, reviewRounds: 0, reviewLog: [] };
+    const { handedOver: _handedOver, ...rest } = state;
+    return { ...rest, ciFixes: 0, commitFixes: 0, reviewRounds: 0, reviewLog: [] };
 }
 export async function writeState(repo, state) {
     await mkdir(dir(repo), { recursive: true });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "next-issue",
-  "version": "2.7.0",
+  "version": "2.9.0",
   "description": "Autonomous issue harness driven by the pi SDK",
   "keywords": [
     "github",