@codedrifters/configulator 0.0.395 → 0.0.397
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/lib/index.d.mts +7 -7
- package/lib/index.d.ts +7 -7
- package/lib/index.js +120 -86
- package/lib/index.js.map +1 -1
- package/lib/index.mjs +120 -86
- package/lib/index.mjs.map +1 -1
- package/package.json +6 -6
package/lib/index.mjs
CHANGED
|
@@ -16492,7 +16492,7 @@ var orchestratorSubAgent = {
|
|
|
16492
16492
|
"### Step 3 \u2014 failure semantics",
|
|
16493
16493
|
"",
|
|
16494
16494
|
"A single PR's worker failure (missing or malformed `WORKER_*` final",
|
|
16495
|
-
"line, sub-agent error, unresolvable
|
|
16495
|
+
"line, sub-agent error, unresolvable merge conflict) **never stops",
|
|
16496
16496
|
"the drain**. Log the failure as",
|
|
16497
16497
|
"`WORKER_FAILED pr:#<n> reason:<observed>` and continue to the next",
|
|
16498
16498
|
"`PR_DRAIN` line. The next orchestrator invocation re-runs",
|
|
@@ -17109,43 +17109,55 @@ var issueWorkerSubAgent = {
|
|
|
17109
17109
|
" reviewer's signal that the PR's head branch is BEHIND the default",
|
|
17110
17110
|
" branch with merge conflicts that `gh pr update-branch` could not",
|
|
17111
17111
|
" resolve. For either item the work is **not** an editorial change",
|
|
17112
|
-
" \u2014 it is a
|
|
17113
|
-
" different recipes:",
|
|
17112
|
+
" \u2014 it is a merge of the default branch into the PR branch plus",
|
|
17113
|
+
" conflict resolution. The two ids select different recipes:",
|
|
17114
17114
|
"",
|
|
17115
17115
|
" - `synthetic:rebase-behind-main` \u2014 generic conflict. Resolve",
|
|
17116
17116
|
" each conflicting file by hand (read both sides, reconcile,",
|
|
17117
|
-
" stage), then `git
|
|
17118
|
-
"
|
|
17117
|
+
" stage), then `git commit --no-edit` to complete the merge",
|
|
17118
|
+
" commit.",
|
|
17119
17119
|
" - `synthetic:rebase-shared-index` \u2014 every conflicting file is a",
|
|
17120
17120
|
" row-insert race on a shared-index file (registry / index /",
|
|
17121
17121
|
" feature-matrix under a Starlight content root). Apply the",
|
|
17122
17122
|
" explicit re-insert recipe carried in the item's `instruction`",
|
|
17123
17123
|
" field \u2014 do **not** hand-merge.",
|
|
17124
17124
|
"",
|
|
17125
|
+
" **Marker name vs. operation.** The `comment_id` values keep their",
|
|
17126
|
+
" historical `rebase` spelling as a **stable identifier** the",
|
|
17127
|
+
" reviewer and worker agree on across sessions \u2014 the operation they",
|
|
17128
|
+
" select is a **merge** of the default branch into the PR branch,",
|
|
17129
|
+
" **not** a rebase. The PR branch is already pushed and diverged,",
|
|
17130
|
+
" so a rebase would rewrite its published history and require a",
|
|
17131
|
+
" `--force-with-lease` push that rule 8 forbids. A merge commit is",
|
|
17132
|
+
" fast-forward-pushable with a plain `git push`.",
|
|
17133
|
+
"",
|
|
17125
17134
|
" **For `synthetic:rebase-behind-main`.** Run the following",
|
|
17126
17135
|
" sequence:",
|
|
17127
17136
|
"",
|
|
17128
17137
|
" ```bash",
|
|
17129
17138
|
" git fetch origin",
|
|
17130
|
-
" git
|
|
17139
|
+
" git merge --no-edit origin/{{repository.defaultBranch}}",
|
|
17131
17140
|
" ```",
|
|
17132
17141
|
"",
|
|
17133
|
-
" When the
|
|
17134
|
-
" in turn (read both sides
|
|
17135
|
-
"
|
|
17136
|
-
"
|
|
17137
|
-
"
|
|
17138
|
-
"
|
|
17139
|
-
"
|
|
17140
|
-
"
|
|
17141
|
-
"
|
|
17142
|
-
"
|
|
17143
|
-
"",
|
|
17144
|
-
"
|
|
17145
|
-
"
|
|
17146
|
-
"
|
|
17147
|
-
"
|
|
17148
|
-
"
|
|
17142
|
+
" When the merge produces conflicts, resolve each conflicting file",
|
|
17143
|
+
" in turn (read both sides \u2014 `<<<<<<< HEAD` is this PR's change,",
|
|
17144
|
+
" `>>>>>>> origin/{{repository.defaultBranch}}` is the default",
|
|
17145
|
+
" branch's \u2014 reconcile, stage), then run `git commit --no-edit` to",
|
|
17146
|
+
" complete the merge commit. Do not run `git merge --abort` to give",
|
|
17147
|
+
" up casually \u2014 aborting drops the work the reviewer delegated.",
|
|
17148
|
+
" Abort **only** as the failure path: if you genuinely cannot",
|
|
17149
|
+
" resolve a conflict (the change is editorial enough that it",
|
|
17150
|
+
" requires human judgement, or the conflict touches a generated",
|
|
17151
|
+
" file you should not hand-edit), record the item as `failed` with",
|
|
17152
|
+
" a clear reason, run `git merge --abort`, and proceed to the",
|
|
17153
|
+
" report step. Do not force-push.",
|
|
17154
|
+
"",
|
|
17155
|
+
" When the merge completes cleanly, treat the merge commit itself",
|
|
17156
|
+
" as the deliverable. Skip the conventional commit step in Phase 6",
|
|
17157
|
+
" for this item \u2014 the merge already produced the commit. Push with",
|
|
17158
|
+
" a regular non-force `git push origin <branch>` (the merge commit",
|
|
17159
|
+
" is fast-forward-pushable, so no force-push is needed) and report",
|
|
17160
|
+
" the merged head SHA as the worker's commit.",
|
|
17149
17161
|
"",
|
|
17150
17162
|
" **For `synthetic:rebase-shared-index`.** Apply the typed",
|
|
17151
17163
|
" re-insert recipe step-by-step. Read the recipe verbatim from the",
|
|
@@ -17153,11 +17165,11 @@ var issueWorkerSubAgent = {
|
|
|
17153
17165
|
" contract the reviewer encodes and the precondition guards you",
|
|
17154
17166
|
" must enforce:",
|
|
17155
17167
|
"",
|
|
17156
|
-
" 1. **
|
|
17168
|
+
" 1. **Fetch and merge** the default branch into the PR branch:",
|
|
17157
17169
|
"",
|
|
17158
17170
|
" ```bash",
|
|
17159
17171
|
" git fetch origin",
|
|
17160
|
-
" git
|
|
17172
|
+
" git merge --no-edit origin/{{repository.defaultBranch}}",
|
|
17161
17173
|
" ```",
|
|
17162
17174
|
"",
|
|
17163
17175
|
" 2. **For each conflicting file**, inspect the conflict markers",
|
|
@@ -17174,18 +17186,20 @@ var issueWorkerSubAgent = {
|
|
|
17174
17186
|
" the frontmatter (lines between the opening / closing `---`",
|
|
17175
17187
|
" fences), the page H1, surrounding prose paragraphs, the",
|
|
17176
17188
|
" table header row, or the separator row, **stop**, run",
|
|
17177
|
-
" `git
|
|
17189
|
+
" `git merge --abort`, and record the item as `failed` with",
|
|
17178
17190
|
" the structured marker `BLOCKED <reason>` (e.g.",
|
|
17179
17191
|
" `BLOCKED conflict touches table header in <path>`). Do not",
|
|
17180
17192
|
" hand-merge \u2014 the typed recipe applies only to pure",
|
|
17181
17193
|
" row-insert races.",
|
|
17182
17194
|
"",
|
|
17183
|
-
" When the guards pass,
|
|
17184
|
-
"
|
|
17185
|
-
"
|
|
17186
|
-
"
|
|
17187
|
-
"
|
|
17188
|
-
"
|
|
17195
|
+
" When the guards pass, resolve the conflict in place: the",
|
|
17196
|
+
" `<<<<<<< HEAD` side carries this PR's row and the",
|
|
17197
|
+
" `>>>>>>> origin/{{repository.defaultBranch}}` side carries",
|
|
17198
|
+
" the other PR's row. Extract this PR's row and re-insert",
|
|
17199
|
+
" that row in declared sort order alongside the other PR's row",
|
|
17200
|
+
" using the file's documented sort key (alphabetical on the",
|
|
17201
|
+
" first column by default) so both rows survive, remove the",
|
|
17202
|
+
" conflict markers, and stage the file:",
|
|
17189
17203
|
"",
|
|
17190
17204
|
" ```bash",
|
|
17191
17205
|
" git add <path>",
|
|
@@ -17201,16 +17215,17 @@ var issueWorkerSubAgent = {
|
|
|
17201
17215
|
' [ "$count" = "1" ] || { echo "BLOCKED verification failed for <path>"; exit 1; }',
|
|
17202
17216
|
" ```",
|
|
17203
17217
|
"",
|
|
17204
|
-
" 4. **
|
|
17205
|
-
" every file is staged and verified
|
|
17218
|
+
" 4. **Complete the merge and push** with a plain non-force push",
|
|
17219
|
+
" once every file is staged and verified (the merge commit is",
|
|
17220
|
+
" fast-forward-pushable, so no force-push is needed):",
|
|
17206
17221
|
"",
|
|
17207
17222
|
" ```bash",
|
|
17208
|
-
" git
|
|
17223
|
+
" git commit --no-edit",
|
|
17209
17224
|
" git push origin <branch>",
|
|
17210
17225
|
" ```",
|
|
17211
17226
|
"",
|
|
17212
17227
|
" On any `BLOCKED` precondition failure, exit non-zero with the",
|
|
17213
|
-
" `BLOCKED <reason>` line, run `git
|
|
17228
|
+
" `BLOCKED <reason>` line, run `git merge --abort`, record the",
|
|
17214
17229
|
" item as `failed`, and proceed to the report step. The reviewer",
|
|
17215
17230
|
" will see the `failed` outcome on the next pass and fall through",
|
|
17216
17231
|
" to the human-required hand-off via `review:awaiting-human`. Do",
|
|
@@ -17277,20 +17292,21 @@ var issueWorkerSubAgent = {
|
|
|
17277
17292
|
"git push origin <branch-name>",
|
|
17278
17293
|
"```",
|
|
17279
17294
|
"",
|
|
17280
|
-
"**Synthetic
|
|
17295
|
+
"**Synthetic rebase items skip the `fix(review)` commit.** When the",
|
|
17281
17296
|
"fix-list contained a `comment_id` of `synthetic:rebase-behind-main`",
|
|
17282
|
-
"or `synthetic:rebase-shared-index` and you completed the
|
|
17283
|
-
"Phase 4, the
|
|
17284
|
-
"
|
|
17285
|
-
"commit
|
|
17286
|
-
|
|
17297
|
+
"or `synthetic:rebase-shared-index` and you completed the merge in",
|
|
17298
|
+
"Phase 4, the merge commit is itself the deliverable \u2014 there is no",
|
|
17299
|
+
"per-item editorial change to wrap in a `fix(review)` commit. The",
|
|
17300
|
+
"merge commit already exists from Phase 4, so skip the `git add` /",
|
|
17301
|
+
'`git commit -m "fix(review)..."` / `git pull --rebase` editorial',
|
|
17302
|
+
"steps for that item and push the merged branch directly:",
|
|
17287
17303
|
"",
|
|
17288
17304
|
"```bash",
|
|
17289
17305
|
"git push origin <branch-name>",
|
|
17290
17306
|
"```",
|
|
17291
17307
|
"",
|
|
17292
|
-
"If the fix-list mixed a synthetic
|
|
17293
|
-
"perform the editorial commit first (the
|
|
17308
|
+
"If the fix-list mixed a synthetic rebase item with editorial items,",
|
|
17309
|
+
"perform the editorial commit first (the merge already preceded it",
|
|
17294
17310
|
"in Phase 4), then push once at the end.",
|
|
17295
17311
|
"",
|
|
17296
17312
|
"**Normal mode:** Use conventional commit messages and push with `-u`",
|
|
@@ -17319,8 +17335,8 @@ var issueWorkerSubAgent = {
|
|
|
17319
17335
|
"",
|
|
17320
17336
|
"**This is the FIRST action after a successful push** \u2014 before any",
|
|
17321
17337
|
"summary, log line, or secondary step. Call the deterministic procedure",
|
|
17322
|
-
"with the PR number, the commit SHA you pushed (or the
|
|
17323
|
-
"for a synthetic
|
|
17338
|
+
"with the PR number, the commit SHA you pushed (or the merged head SHA",
|
|
17339
|
+
"for a synthetic rebase item), and the per-item outcomes JSON array:",
|
|
17324
17340
|
"",
|
|
17325
17341
|
"```bash",
|
|
17326
17342
|
".claude/procedures/complete-feedback-handoff.sh <pr-number> <commit-sha> '<items-json>'",
|
|
@@ -19071,22 +19087,22 @@ var prReviewerSubAgent = {
|
|
|
19071
19087
|
"Delegation** section. It evaluates the guards (auto-merge mode, the",
|
|
19072
19088
|
"delegation invocation guard, a free `review:fixing` lease, no",
|
|
19073
19089
|
"generated / projen / lockfile conflicts), classifies the conflicting",
|
|
19074
|
-
"files, and either delegates a
|
|
19075
|
-
"through to its own fallback. Invoke it with this entry
|
|
19076
|
-
"parameters:",
|
|
19090
|
+
"files, and either delegates a conflict resolution to `issue-worker`",
|
|
19091
|
+
"or falls through to its own fallback. Invoke it with this entry",
|
|
19092
|
+
"point's parameters:",
|
|
19077
19093
|
"",
|
|
19078
19094
|
"- **Entry point:** `already-conflicting`. There is no failed",
|
|
19079
19095
|
" `gh pr update-branch` to read conflicting paths from \u2014 identify",
|
|
19080
19096
|
" the conflicting files from the PR's merge state as documented in",
|
|
19081
|
-
" the shared procedure's entry-point step (a local
|
|
19097
|
+
" the shared procedure's entry-point step (a local merge attempt",
|
|
19082
19098
|
" whose `<<<<<<<` / `=======` / `>>>>>>>` markers name the",
|
|
19083
19099
|
" conflicting files). The guards and classification are identical to",
|
|
19084
19100
|
" the Phase 4 entry point.",
|
|
19085
|
-
"- **On the procedure returning `delegated`:** a
|
|
19086
|
-
" durably queued for `issue-worker` (the PR now carries",
|
|
19101
|
+
"- **On the procedure returning `delegated`:** a conflict resolution",
|
|
19102
|
+
" has been durably queued for `issue-worker` (the PR now carries",
|
|
19087
19103
|
" `review:needs-worker` and the `review:fixing` lease); stop this",
|
|
19088
19104
|
" pass. A top-level `issue-worker` session pulls the marker and",
|
|
19089
|
-
" pushes the
|
|
19105
|
+
" pushes the merged branch, then a follow-up reviewer pass confirms",
|
|
19090
19106
|
" the result and releases the lease \u2014 no human re-invocation is",
|
|
19091
19107
|
" required for the loop to complete.",
|
|
19092
19108
|
"- **On the procedure returning `skipped` (a guard denied delegation",
|
|
@@ -20045,13 +20061,13 @@ var prReviewerSubAgent = {
|
|
|
20045
20061
|
" with `gh pr view <pr-number> --json files`).",
|
|
20046
20062
|
"- **`already-conflicting`** \u2014 there is no failed `update-branch` to",
|
|
20047
20063
|
" read conflicting paths from. Identify the conflicting files from",
|
|
20048
|
-
" the PR's merge state instead: attempt a local
|
|
20049
|
-
" branch
|
|
20064
|
+
" the PR's merge state instead: attempt a local merge of the default",
|
|
20065
|
+
" branch into the head branch in a scratch checkout (or read the",
|
|
20050
20066
|
" conflicting paths git reports) and collect the files that carry",
|
|
20051
20067
|
" `<<<<<<<` / `=======` / `>>>>>>>` conflict markers. Use that set",
|
|
20052
20068
|
" as the conflicting-file list. The reviewer inspects these markers",
|
|
20053
20069
|
" read-only to classify; it does **not** stage, commit, or push any",
|
|
20054
|
-
" resolution \u2014 the actual
|
|
20070
|
+
" resolution \u2014 the actual merge-and-resolve is delegated to",
|
|
20055
20071
|
" `issue-worker` exactly as in the other entry point.",
|
|
20056
20072
|
"",
|
|
20057
20073
|
"### Guards",
|
|
@@ -20066,8 +20082,10 @@ var prReviewerSubAgent = {
|
|
|
20066
20082
|
" the human reviewer's consent. (Unlike the `update-branch` step,",
|
|
20067
20083
|
" which permits a size-only `human-required` carve-out, the",
|
|
20068
20084
|
" conflict-resolution delegation flow is auto-merge-only across",
|
|
20069
|
-
" the board: a worker
|
|
20070
|
-
"
|
|
20085
|
+
" the board: a worker's conflict-resolution merge reconciles both",
|
|
20086
|
+
" sides of each conflict into the branch, a stronger change to the",
|
|
20087
|
+
" reviewed diff than the clean merge-commit `gh pr update-branch`",
|
|
20088
|
+
" performs on a no-conflict branch.)",
|
|
20071
20089
|
"2. **Delegation invocation guard permits the hand-off** \u2014 the PR",
|
|
20072
20090
|
" carries the `origin:issue-worker` label, **or** the reviewer was",
|
|
20073
20091
|
" invoked with `--allow-human-author`. The same guard used for the",
|
|
@@ -20097,7 +20115,7 @@ var prReviewerSubAgent = {
|
|
|
20097
20115
|
"before composing the fix-list. The classification picks one of two",
|
|
20098
20116
|
"typed recipes \u2014 a precise `shared-index` resolver when every",
|
|
20099
20117
|
"conflict is a row-insert race on a registry / index / feature-matrix",
|
|
20100
|
-
"file, or the generic
|
|
20118
|
+
"file, or the generic conflict-resolution recipe in every other case.",
|
|
20101
20119
|
"",
|
|
20102
20120
|
"**Classification step.** For each conflicting path, decide whether",
|
|
20103
20121
|
"the file is `shared-index` or `generic` against these criteria (the",
|
|
@@ -20131,12 +20149,23 @@ var prReviewerSubAgent = {
|
|
|
20131
20149
|
' fix-list item with `comment_id: "synthetic:rebase-shared-index"`',
|
|
20132
20150
|
" carrying the explicit re-insert recipe (see step 3 below). The",
|
|
20133
20151
|
" worker mechanically re-inserts each row in declared sort order",
|
|
20134
|
-
" against the
|
|
20152
|
+
" against the merged file.",
|
|
20135
20153
|
"- **Any conflicting file is `generic`** \u2014 emit the existing generic",
|
|
20136
20154
|
' fix-list item with `comment_id: "synthetic:rebase-behind-main"`',
|
|
20137
20155
|
" (see step 3 below). The worker performs a hand-merge of the",
|
|
20138
20156
|
" conflict markers.",
|
|
20139
20157
|
"",
|
|
20158
|
+
"**Marker name vs. operation.** The two `comment_id` values retain",
|
|
20159
|
+
"their historical `rebase` spelling as a **stable cross-session",
|
|
20160
|
+
"identifier** \u2014 a fix-list posted by an older reviewer and consumed",
|
|
20161
|
+
"by a newer worker still matches. The operation they now select is a",
|
|
20162
|
+
"**merge** of the default branch into the PR branch, **not** a",
|
|
20163
|
+
"rebase: the PR branch is already pushed and diverged, so a rebase",
|
|
20164
|
+
"would rewrite its published history and force the worker into a",
|
|
20165
|
+
"`--force-with-lease` push that the never-force-push rule forbids. A",
|
|
20166
|
+
"merge commit is fast-forward-pushable with a plain `git push`, so",
|
|
20167
|
+
"the recipe stays publishable.",
|
|
20168
|
+
"",
|
|
20140
20169
|
"### Hand off to `issue-worker`",
|
|
20141
20170
|
"",
|
|
20142
20171
|
"Hand off to `issue-worker` with the chosen single synthetic",
|
|
@@ -20159,14 +20188,15 @@ var prReviewerSubAgent = {
|
|
|
20159
20188
|
" ```",
|
|
20160
20189
|
"",
|
|
20161
20190
|
"3. **Post a fix-list comment** containing exactly one synthetic item",
|
|
20162
|
-
" describing the
|
|
20163
|
-
" recipe
|
|
20191
|
+
" describing the conflict resolution. The `comment_id` field selects",
|
|
20192
|
+
" the typed recipe (both perform a **merge** of the default branch",
|
|
20193
|
+
" into the PR branch \u2014 see the marker-name note above):",
|
|
20164
20194
|
"",
|
|
20165
20195
|
" - `synthetic:rebase-behind-main` \u2014 generic conflict; the worker",
|
|
20166
20196
|
" resolves conflicting hunks by hand.",
|
|
20167
20197
|
" - `synthetic:rebase-shared-index` \u2014 every conflict is a pure",
|
|
20168
20198
|
" row-insert race on a shared-index file; the worker re-inserts",
|
|
20169
|
-
" each row in declared sort order against the
|
|
20199
|
+
" each row in declared sort order against the merged file.",
|
|
20170
20200
|
"",
|
|
20171
20201
|
" The next reviewer pass identifies the item by its `comment_id`.",
|
|
20172
20202
|
"",
|
|
@@ -20175,7 +20205,7 @@ var prReviewerSubAgent = {
|
|
|
20175
20205
|
" ```markdown",
|
|
20176
20206
|
" ## Reviewer: fix list for @issue-worker",
|
|
20177
20207
|
"",
|
|
20178
|
-
" - [ ] @reviewer \u2014
|
|
20208
|
+
" - [ ] @reviewer \u2014 merge origin/<default-branch> into the PR branch and resolve conflicts in: <space-separated list of conflicting files>",
|
|
20179
20209
|
"",
|
|
20180
20210
|
" ```json fix-list",
|
|
20181
20211
|
" {",
|
|
@@ -20183,7 +20213,7 @@ var prReviewerSubAgent = {
|
|
|
20183
20213
|
' "branch": "<head-ref-name>",',
|
|
20184
20214
|
' "generated_at": "<ISO-8601 timestamp>",',
|
|
20185
20215
|
' "items": [',
|
|
20186
|
-
' {"comment_id": "synthetic:rebase-behind-main", "author": "reviewer", "file": "<first-conflicting-file>", "instruction": "Branch is BEHIND default-branch with merge conflicts.
|
|
20216
|
+
' {"comment_id": "synthetic:rebase-behind-main", "author": "reviewer", "file": "<first-conflicting-file>", "instruction": "Branch is BEHIND default-branch with merge conflicts. Fetch and merge the default branch into the PR branch: `git fetch origin && git merge --no-edit origin/<default-branch>`. Resolve the conflicts in <conflicting-files> by reconciling both sides (`<<<<<<< HEAD` is this PR\'s change), `git add` each resolved file, then `git commit --no-edit` to complete the merge commit. Push the merged branch with a plain non-force `git push origin <branch>` \u2014 the merge commit is fast-forward-pushable, so no force-push is needed \u2014 and report success."}',
|
|
20187
20217
|
" ]",
|
|
20188
20218
|
" }",
|
|
20189
20219
|
" ```",
|
|
@@ -20196,7 +20226,7 @@ var prReviewerSubAgent = {
|
|
|
20196
20226
|
" ```markdown",
|
|
20197
20227
|
" ## Reviewer: fix list for @issue-worker",
|
|
20198
20228
|
"",
|
|
20199
|
-
" - [ ] @reviewer \u2014
|
|
20229
|
+
" - [ ] @reviewer \u2014 merge origin/<default-branch> into the PR branch and re-insert shared-index rows in: <space-separated list of conflicting files>",
|
|
20200
20230
|
"",
|
|
20201
20231
|
" ```json fix-list",
|
|
20202
20232
|
" {",
|
|
@@ -20204,7 +20234,7 @@ var prReviewerSubAgent = {
|
|
|
20204
20234
|
' "branch": "<head-ref-name>",',
|
|
20205
20235
|
' "generated_at": "<ISO-8601 timestamp>",',
|
|
20206
20236
|
' "items": [',
|
|
20207
|
-
' {"comment_id": "synthetic:rebase-shared-index", "author": "reviewer", "file": "<first-conflicting-file>", "instruction": "Branch is BEHIND default-branch with shared-index row-insert conflicts only. Apply this recipe: (1) Fetch and
|
|
20237
|
+
' {"comment_id": "synthetic:rebase-shared-index", "author": "reviewer", "file": "<first-conflicting-file>", "instruction": "Branch is BEHIND default-branch with shared-index row-insert conflicts only. Apply this recipe: (1) Fetch and merge the default branch into the PR branch: `git fetch origin && git merge --no-edit origin/<default-branch>`. (2) For each conflicting shared-index file: the conflict has this PR\'s row on the `<<<<<<< HEAD` side and the other PR\'s row on the `>>>>>>> origin/<default-branch>` side \u2014 extract this PR\'s row and re-insert that row in declared sort order alongside the other PR\'s row using the file\'s documented sort key (alphabetical on the first column by default) so both rows survive, remove the conflict markers, stage the file with `git add <path>`. (3) Run the commit-path verification for each row: `count=$(git show :<path> | grep -Fc <row-unique-marker>) && [ \\"$count\\" = \\"1\\" ] || exit 1`. (4) Complete the merge with `git commit --no-edit` and push with a plain non-force `git push origin <branch>` \u2014 the merge commit is fast-forward-pushable, so no force-push is needed. (5) If any conflict marker touches the frontmatter, table header row, `|---|---|` separator, or surrounding prose, abort with `git merge --abort` and emit `BLOCKED <reason>` \u2014 the precondition guards from the reviewer\'s classification step were violated."}',
|
|
20208
20238
|
" ]",
|
|
20209
20239
|
" }",
|
|
20210
20240
|
" ```",
|
|
@@ -20232,7 +20262,7 @@ var prReviewerSubAgent = {
|
|
|
20232
20262
|
" The worker runs in its own top-level session; a **subsequent**",
|
|
20233
20263
|
" reviewer pass detects the worker finished (PR carries",
|
|
20234
20264
|
" `review:fixing` but **not** `review:needs-worker`), reacts on the",
|
|
20235
|
-
" source comment, re-evaluates AC/CI against the
|
|
20265
|
+
" source comment, re-evaluates AC/CI against the merged branch,",
|
|
20236
20266
|
" re-enables auto-merge, and releases the `review:fixing` lease. See",
|
|
20237
20267
|
" the **Confirm pass** section below.",
|
|
20238
20268
|
"",
|
|
@@ -20251,7 +20281,7 @@ var prReviewerSubAgent = {
|
|
|
20251
20281
|
"and return `skipped` to the caller.",
|
|
20252
20282
|
"",
|
|
20253
20283
|
"```bash",
|
|
20254
|
-
"gh pr comment <pr-number> --body 'Branch is BEHIND <default-branch> with conflicts. <short reason delegation was skipped \u2014 e.g. human-required mode, generated-file conflicts, or in-flight review:fixing lease>. A human (or the next reviewer pass after
|
|
20284
|
+
"gh pr comment <pr-number> --body 'Branch is BEHIND <default-branch> with conflicts. <short reason delegation was skipped \u2014 e.g. human-required mode, generated-file conflicts, or in-flight review:fixing lease>. A human (or the next reviewer pass after the branch is updated) will need to resolve.'",
|
|
20255
20285
|
"```",
|
|
20256
20286
|
"",
|
|
20257
20287
|
"## Shared Procedure: Confirm a Completed Delegation",
|
|
@@ -20261,7 +20291,8 @@ var prReviewerSubAgent = {
|
|
|
20261
20291
|
"the fix-list, acquires the `review:fixing` lease, applies the durable",
|
|
20262
20292
|
"`review:needs-worker` marker, and exits returning `delegated`. A",
|
|
20263
20293
|
"**separate** top-level `issue-worker` session then discovers the",
|
|
20264
|
-
"marker, applies the fix-list (an editorial fix or a
|
|
20294
|
+
"marker, applies the fix-list (an editorial fix or a conflict-resolution",
|
|
20295
|
+
"merge), pushes,",
|
|
20265
20296
|
"writes a structured worker-report comment, and **clears**",
|
|
20266
20297
|
"`review:needs-worker` \u2014 leaving `review:fixing` in place for the",
|
|
20267
20298
|
"reviewer to release. The confirm pass below is how a later reviewer",
|
|
@@ -20300,12 +20331,12 @@ var prReviewerSubAgent = {
|
|
|
20300
20331
|
" used to perform.",
|
|
20301
20332
|
"2. **React on the source comments.** For each item:",
|
|
20302
20333
|
" - **`handled`** \u2014 react `rocket` on the original source comment",
|
|
20303
|
-
" (the fix-list comment for a synthetic
|
|
20304
|
-
" accepted change has landed.",
|
|
20334
|
+
" (the fix-list comment for a synthetic conflict-resolution item)",
|
|
20335
|
+
" to signal the accepted change has landed.",
|
|
20305
20336
|
" - **`failed`** \u2014 react `thinking_face` (`content=confused`) on the",
|
|
20306
20337
|
" source comment and post a reply citing the worker's failure",
|
|
20307
20338
|
" reason.",
|
|
20308
|
-
"3. **Re-evaluate AC/CI against the
|
|
20339
|
+
"3. **Re-evaluate AC/CI against the merged branch.** Re-run the Phase",
|
|
20309
20340
|
" 3 acceptance-criteria comparison and the **CI Verification** check",
|
|
20310
20341
|
" against the **updated** head SHA the worker pushed \u2014 do not reuse",
|
|
20311
20342
|
" any pre-delegation result.",
|
|
@@ -20432,8 +20463,9 @@ var prReviewerSubAgent = {
|
|
|
20432
20463
|
"been applied in this pass. `delegated` is never a terminal",
|
|
20433
20464
|
"completion: the loop closes only when a top-level worker consumes the",
|
|
20434
20465
|
"marker and a later reviewer confirm pass releases the lease. Likewise",
|
|
20435
|
-
"`Branch updated: delegated (PR #<n>)` records that a
|
|
20436
|
-
"queued for the worker, not that the branch is already
|
|
20466
|
+
"`Branch updated: delegated (PR #<n>)` records that a conflict",
|
|
20467
|
+
"resolution was queued for the worker, not that the branch is already",
|
|
20468
|
+
"current.",
|
|
20437
20469
|
"",
|
|
20438
20470
|
"---",
|
|
20439
20471
|
"",
|
|
@@ -20516,11 +20548,13 @@ var prReviewerSubAgent = {
|
|
|
20516
20548
|
" issue type) continue to block `update-branch`: pushing main into",
|
|
20517
20549
|
" those PRs expands the diff under review without the human",
|
|
20518
20550
|
" reviewer's consent. The same restriction applies to delegating",
|
|
20519
|
-
" conflict resolution to `issue-worker`: never delegate a
|
|
20520
|
-
" on a `human-required` PR regardless of
|
|
20521
|
-
" typed-recipe delegation flow stays
|
|
20522
|
-
"
|
|
20523
|
-
"
|
|
20551
|
+
" conflict resolution to `issue-worker`: never delegate a",
|
|
20552
|
+
" conflict resolution on a `human-required` PR regardless of",
|
|
20553
|
+
" `matched_rule` \u2014 the typed-recipe delegation flow stays",
|
|
20554
|
+
" auto-merge-only because a worker's merge injects reconciled",
|
|
20555
|
+
" content into a human-required branch, expanding the reviewed",
|
|
20556
|
+
" diff more than the clean merge-commit `gh pr update-branch`",
|
|
20557
|
+
" performs.",
|
|
20524
20558
|
"16. **Never delegate conflict resolution involving generated or",
|
|
20525
20559
|
" projen-managed files.** When `gh pr update-branch` fails on a",
|
|
20526
20560
|
" BEHIND PR with conflicts and any conflicting path is a lockfile,",
|
|
@@ -31523,23 +31557,23 @@ var VERSION = {
|
|
|
31523
31557
|
/**
|
|
31524
31558
|
* Version of Astro to pin for AstroProject scaffolding.
|
|
31525
31559
|
*/
|
|
31526
|
-
ASTRO_VERSION: "7.
|
|
31560
|
+
ASTRO_VERSION: "7.1.3",
|
|
31527
31561
|
/**
|
|
31528
31562
|
* CDK CLI for workflows and command line operations.
|
|
31529
31563
|
*
|
|
31530
31564
|
* CLI and lib are versioned separately, so this is the CLI version.
|
|
31531
31565
|
*/
|
|
31532
|
-
AWS_CDK_CLI_VERSION: "2.
|
|
31566
|
+
AWS_CDK_CLI_VERSION: "2.1132.1",
|
|
31533
31567
|
/**
|
|
31534
31568
|
* CDK Version to use for construct projects.
|
|
31535
31569
|
*
|
|
31536
31570
|
* CLI and lib are versioned separately, so this is the lib version.
|
|
31537
31571
|
*/
|
|
31538
|
-
AWS_CDK_LIB_VERSION: "2.
|
|
31572
|
+
AWS_CDK_LIB_VERSION: "2.262.0",
|
|
31539
31573
|
/**
|
|
31540
31574
|
* Version of the AWS Constructs library to use.
|
|
31541
31575
|
*/
|
|
31542
|
-
AWS_CONSTRUCTS_VERSION: "10.
|
|
31576
|
+
AWS_CONSTRUCTS_VERSION: "10.7.1",
|
|
31543
31577
|
/**
|
|
31544
31578
|
* Version of Node.js to use in CI workflows at github actions.
|
|
31545
31579
|
*/
|
|
@@ -31552,7 +31586,7 @@ var VERSION = {
|
|
|
31552
31586
|
/**
|
|
31553
31587
|
* Version of PNPM to use in workflows at github actions.
|
|
31554
31588
|
*/
|
|
31555
|
-
PNPM_VERSION: "11.
|
|
31589
|
+
PNPM_VERSION: "11.16.0",
|
|
31556
31590
|
/**
|
|
31557
31591
|
* Version of Projen to use.
|
|
31558
31592
|
*/
|
|
@@ -31570,11 +31604,11 @@ var VERSION = {
|
|
|
31570
31604
|
/**
|
|
31571
31605
|
* Version of `@astrojs/starlight` to pin for StarlightProject scaffolding.
|
|
31572
31606
|
*/
|
|
31573
|
-
STARLIGHT_VERSION: "0.41.
|
|
31607
|
+
STARLIGHT_VERSION: "0.41.4",
|
|
31574
31608
|
/**
|
|
31575
31609
|
* What version of the turborepo library should we use?
|
|
31576
31610
|
*/
|
|
31577
|
-
TURBO_VERSION: "2.10.
|
|
31611
|
+
TURBO_VERSION: "2.10.6",
|
|
31578
31612
|
/**
|
|
31579
31613
|
* Version of `@types/node` to use across all packages (pnpm catalog).
|
|
31580
31614
|
*/
|