@codedrifters/configulator 0.0.351 → 0.0.353

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.js CHANGED
@@ -12200,7 +12200,7 @@ function buildMeetingAnalystSubAgent(tier) {
12200
12200
  "| `planning` | Sprint/plan updates, task issues, requirement drafts | Extract every task assignment. Phase 4 updates the active sprint-plan doc directly and creates one issue per assigned task. |",
12201
12201
  "| `review` | Retro notes, follow-up issues, status updates, requirement revisions | Mark completed tasks in the active sprint-plan doc. Capture retrospective learnings inside the meeting notes \u2014 do not spawn a separate retro document. |",
12202
12202
  "| `brainstorm` | Future-feature candidates, research topics, very few Firm decisions | Lower the bar for Open Questions. Do **not** create requirement or ADR issues from brainstorm output unless the item is explicitly marked **Firm**. Prefer research/discovery issues for promising ideas. |",
12203
- "| `standup` | Action items, blockers, status updates | Do not produce requirement, ADR, or BDR drafts. Phase 3 (Draft) is almost always skipped. Focus on action-item issues and sprint-plan status updates. |",
12203
+ "| `standup` | Action items, blockers, status updates | Do not produce requirement, ADR, or BDR drafts. Phase 3 (Draft) is almost always skipped. Capture action items in the notes `## Action Items` table; file issues only for agent-workable doc deliverables (per the **Action-item filing policy**) and apply sprint-plan status updates. |",
12204
12204
  "| `external` | Customer/competitive intel, people/company profiles, lead capture | Default every attendee outside the team into **People of Interest**. Default every organization mentioned into **Companies of Interest** (signal threshold still applies). Capture stated customer pain points as candidate **BR** (business requirements), not FR. |",
12205
12205
  "| `other` | General meeting extraction | Apply the default workflow with no type-specific overrides. |",
12206
12206
  "",
@@ -12262,6 +12262,61 @@ function buildMeetingAnalystSubAgent(tier) {
12262
12262
  "",
12263
12263
  "---",
12264
12264
  "",
12265
+ "## Action-item filing policy",
12266
+ "",
12267
+ "Not every action item should become a GitHub issue. The pipeline",
12268
+ "can only ever close issues whose deliverable an automated agent",
12269
+ "(or this repo's existing channels) can produce. Filing an issue",
12270
+ "for a task only a human can do \u2014 in the real world or in a system",
12271
+ "this pipeline cannot touch \u2014 creates queue noise that no worker or",
12272
+ "orchestrator can ever resolve. Apply the **agent-workability",
12273
+ "test** below in phases 1, 3, and 4.",
12274
+ "",
12275
+ "### Agent-workability test",
12276
+ "",
12277
+ "An action item is **agent-workable** only when completing it",
12278
+ "produces a documentation deliverable an automated agent can author",
12279
+ "in this repo's docs tree \u2014 a requirement document, an ADR, a docs",
12280
+ "page, a BCM capability model, a research note, or a roadmap /",
12281
+ "product-doc update. Route agent-workable items through their",
12282
+ "**dedicated downstream channel** (`req:write`, `docs:write`,",
12283
+ "`bcm:*`, `research:scope`, roadmap/product-doc follow-up). Never",
12284
+ "file them as a generic `action-item` / `type:chore` issue.",
12285
+ "",
12286
+ "An action item is **human-owned** when it requires a person to act",
12287
+ "in the real world or in a system this pipeline cannot reach.",
12288
+ "Typical cues:",
12289
+ "",
12290
+ '- **send / email / schedule / invite / call** \u2014 e.g. "send out',
12291
+ ` company invoices", "invite X to next week's call".`,
12292
+ '- **install / configure / provision / set up** \u2014 e.g. "install and',
12293
+ ' configure Claude", "get codebase read access".',
12294
+ "- **decide / approve / choose / prioritise** (a human judgement",
12295
+ " call, not a documented decision record).",
12296
+ '- **communicate / notify / give a heads-up** \u2014 e.g. "give Dr. Y a',
12297
+ ' heads-up".',
12298
+ "- **get / request access or credentials**, **pay / invoice**.",
12299
+ "- **build or change code in another repo**, **design in an",
12300
+ " external tool** (Figma, etc.).",
12301
+ "",
12302
+ "Record human-owned items **only** in the notes `## Action Items`",
12303
+ "table (who / what / when) \u2014 that table is the **system of record**",
12304
+ "for human follow-ups. Do **not** file a GitHub issue for them.",
12305
+ "",
12306
+ "When an item is ambiguous, default to **human-owned**",
12307
+ "(notes-only): a missed issue is cheaper than queue noise the",
12308
+ "pipeline can never close.",
12309
+ "",
12310
+ "**Consumer tuning.** When the project supplies",
12311
+ "`AgentConfigOptions.meetings.actionItemFiling`, an",
12312
+ "**Action-item filing policy** subsection is rendered into the",
12313
+ "meetings taxonomy with any extra human-owned / agent-workable cues",
12314
+ "and a note when the split is disabled. Apply those overrides on",
12315
+ "top of the built-in cues above; if the split is disabled, fall",
12316
+ "back to filing a follow-up issue for every non-draft action item.",
12317
+ "",
12318
+ "---",
12319
+ "",
12265
12320
  ...PROJECT_CONTEXT_MAINTAINER_SECTION,
12266
12321
  "## Traceability",
12267
12322
  "",
@@ -12293,7 +12348,7 @@ function buildMeetingAnalystSubAgent(tier) {
12293
12348
  "",
12294
12349
  " | Artifact | Type | Issue/Path |",
12295
12350
  " |----------|------|------------|",
12296
- " | <title> | requirement / ADR / action-item / profile | #<N> or <path> |",
12351
+ " | <title> | requirement / ADR / docs / profile / research | #<N> or <path> |",
12297
12352
  " ```",
12298
12353
  "",
12299
12354
  "2. **Update the meeting notes** with a similar `## Related Issues` section",
@@ -12358,7 +12413,7 @@ function buildMeetingAnalystSubAgent(tier) {
12358
12413
  ` | **Decisions** | "We decided...", "Let's go with...", explicit choices |`,
12359
12414
  ' | **Requirements** | Feature descriptions, acceptance criteria, "it should..." |',
12360
12415
  " | **Technology discussions** | Platform comparisons, architecture options, tool evaluations |",
12361
- ' | **Action items** | "[Person] will...", "Next step is...", assigned tasks |',
12416
+ ' | **Action items** | "[Person] will...", "Next step is...", assigned tasks. Tag each with its **disposition** (`agent-workable` vs `human-owned`) per the **Action-item filing policy**. |',
12362
12417
  ' | **Open questions** | "We need to figure out...", unresolved debates |',
12363
12418
  " | **People of interest** | Industry contacts, domain experts mentioned |",
12364
12419
  " | **Companies of interest** | Competitors, vendors, partners discussed |",
@@ -12373,7 +12428,10 @@ function buildMeetingAnalystSubAgent(tier) {
12373
12428
  " - Decisions Made (with category and confidence: Firm / Tentative / Needs confirmation)",
12374
12429
  " - Requirements Identified (with category and priority estimate)",
12375
12430
  " - Technology Discussions (with status: Decided / Leaning toward / Open)",
12376
- " - Action Items (with assignee and due date if stated)",
12431
+ " - Action Items (with assignee and due date if stated, and a",
12432
+ " **disposition** tag \u2014 `agent-workable` or `human-owned` \u2014 per",
12433
+ " the **Action-item filing policy** so Phase 4 can route each",
12434
+ " item without re-classifying it)",
12377
12435
  " - Open Questions",
12378
12436
  " - People of Interest (with context and whether a profile exists)",
12379
12437
  " - Companies of Interest (with type and context)",
@@ -12503,7 +12561,11 @@ function buildMeetingAnalystSubAgent(tier) {
12503
12561
  " - Agenda / topics covered",
12504
12562
  " - Key Discussion Points (organized by topic)",
12505
12563
  " - Decisions (numbered, with rationale)",
12506
- " - Action Items (table: who, what, when)",
12564
+ " - Action Items (table: who, what, when). This table is the",
12565
+ " **system of record for human follow-ups** \u2014 human-owned",
12566
+ " action items live here and are never filed as GitHub issues",
12567
+ " (see the **Action-item filing policy**). Capture every",
12568
+ " action item here regardless of disposition.",
12507
12569
  " - Open Questions",
12508
12570
  " - Follow-up items",
12509
12571
  "4. **Plan the `notes/` tree index update \u2014 do not commit it",
@@ -12551,6 +12613,11 @@ function buildMeetingAnalystSubAgent(tier) {
12551
12613
  "This phase only exists if the extraction identified requirements, architectural",
12552
12614
  "decisions, or strategy changes. If this issue exists, execute it.",
12553
12615
  "",
12616
+ "Draft proposals only for **agent-workable** deliverables (per the",
12617
+ "**Action-item filing policy**). Action items tagged `human-owned`",
12618
+ "in the extraction are not drafted here \u2014 they stay in the notes",
12619
+ "`## Action Items` table and produce no follow-up issue.",
12620
+ "",
12554
12621
  "### Steps",
12555
12622
  "",
12556
12623
  "1. Read the extraction file from Phase 1 at",
@@ -12590,8 +12657,27 @@ function buildMeetingAnalystSubAgent(tier) {
12590
12657
  " gated by areas.",
12591
12658
  "3. Create ADR issues if technology decisions need formal records.",
12592
12659
  " Include a `## Traceability` section in each. Not gated by areas.",
12593
- "4. Create action item issues for tasks that are not document-related.",
12594
- " Include a `## Traceability` section in each. Not gated by areas.",
12660
+ "4. **Route action items per the Action-item filing policy \u2014 do",
12661
+ " not file a generic `action-item` issue per task.** Read each",
12662
+ " action item's `disposition` tag from the extraction (re-apply",
12663
+ " the **agent-workability test** if the tag is missing):",
12664
+ " - **Agent-workable** (yields a doc deliverable an agent can",
12665
+ " author): file it through its **dedicated downstream channel**",
12666
+ " \u2014 a `req:write` issue for a requirement, a `docs:write` issue",
12667
+ " for a docs page, `bcm:*` for a capability model, a",
12668
+ " `research:scope` issue for a research note, or a roadmap /",
12669
+ " product-doc follow-up. Use the matching template from the",
12670
+ " issue-templates page; include a `## Traceability` section.",
12671
+ " Issue creation is not gated by areas.",
12672
+ " - **Human-owned** (send/schedule/install/decide/communicate/",
12673
+ " get-access/build-elsewhere): record it **only** in the notes",
12674
+ " `## Action Items` table (step 8 already carries the table",
12675
+ " forward). **File no GitHub issue.** When ambiguous, treat the",
12676
+ " item as human-owned.",
12677
+ " If the project disabled the split via",
12678
+ " `meetings.actionItemFiling.enabled: false`, fall back to filing",
12679
+ " a follow-up issue for every action item not already covered by",
12680
+ " a Phase 3 draft.",
12595
12681
  "5. Cross-reference the meeting in any existing documents that were",
12596
12682
  " discussed **and that live under an in-scope `docRoot`**. For",
12597
12683
  " discussed documents under an out-of-scope `docRoot`, open a",
@@ -12628,7 +12714,10 @@ function buildMeetingAnalystSubAgent(tier) {
12628
12714
  "- Use `gh` CLI for all GitHub operations",
12629
12715
  "- Apply the appropriate `meeting:*` phase label to each phase issue",
12630
12716
  "- Use `type:docs` for documentation-producing issues, `type:chore` for",
12631
- " maintenance/organizational tasks",
12717
+ " maintenance/organizational tasks the pipeline can action",
12718
+ "- Do **not** file issues for human-owned action items \u2014 per the",
12719
+ " **Action-item filing policy** they live only in the notes",
12720
+ " `## Action Items` table",
12632
12721
  "- Use `status:` labels to track progress (`status:ready`, `status:in-progress`, `status:done`)",
12633
12722
  "- Reference the source meeting transcript in all created issues",
12634
12723
  "- Link phase issues with `Depends on: #N` to enforce ordering",
@@ -12699,7 +12788,10 @@ var processMeetingSkill = {
12699
12788
  "1. Resolve the basename, partition, and any sibling files using",
12700
12789
  " the **Basename-pairing algorithm** above",
12701
12790
  "2. Execute Phase 1 (Extract) \u2014 categorize the available content into",
12702
- " decisions, requirements, action items, open questions, and more",
12791
+ " decisions, requirements, action items, open questions, and more.",
12792
+ " Tag each action item's disposition (`agent-workable` vs",
12793
+ " `human-owned`) per the agent's **Action-item filing policy** so",
12794
+ " Phase 4 keeps human-owned tasks out of the issue queue",
12703
12795
  "3. Write the extraction to",
12704
12796
  " `<meetingsRoot>/insights/{type}/<basename>.md`",
12705
12797
  "4. Create downstream phase issues using `gh issue create`:",
@@ -12769,7 +12861,18 @@ function buildMeetingAnalysisBundle(tier = AGENT_MODEL.BALANCED) {
12769
12861
  "direct edits to the doc-root sub-folders declared in the project's",
12770
12862
  "**Area \u2192 doc-root mapping** table. When both fields are absent",
12771
12863
  "the pipeline falls back to the default workflow (kind `other`,",
12772
- "no area gating)."
12864
+ "no area gating).",
12865
+ "",
12866
+ "Phase 4 applies an **agent-workability split** to action items:",
12867
+ "items that yield a documentation deliverable an agent can author",
12868
+ "are filed through their dedicated downstream channel",
12869
+ "(`req:write`, `docs:write`, `bcm:*`, `research:scope`, roadmap",
12870
+ "follow-up); human-owned tasks (send / schedule / install /",
12871
+ "decide / communicate / get-access / build-elsewhere) are",
12872
+ "recorded **only** in the notes `## Action Items` table, which is",
12873
+ "the **system of record for human follow-ups** \u2014 no GitHub issue",
12874
+ "is filed for them. Consumers tune or disable the split via",
12875
+ "`AgentConfigOptions.meetings.actionItemFiling`."
12773
12876
  ].join("\n"),
12774
12877
  platforms: {
12775
12878
  cursor: { exclude: true }
@@ -13914,6 +14017,9 @@ var DEFAULT_BUNDLE_OVERRIDES = {
13914
14017
  "regulatory:research": {
13915
14018
  acceptanceCriteria: { smallMax: 3, mediumMax: 10 }
13916
14019
  },
14020
+ "standards:research": {
14021
+ acceptanceCriteria: { smallMax: 3, mediumMax: 10 }
14022
+ },
13917
14023
  "software:map": {
13918
14024
  acceptanceCriteria: { smallMax: 3, mediumMax: 12 },
13919
14025
  sources: { smallMax: 2, mediumMax: 15 }
@@ -14127,6 +14233,31 @@ function renderScopeGateSection(gate, excludeBundles = []) {
14127
14233
  "alphabetical order on the label name** \u2014 first-wins, deterministic."
14128
14234
  );
14129
14235
  }
14236
+ lines.push(
14237
+ "",
14238
+ "### Reconfiguring the thresholds",
14239
+ "",
14240
+ "Every threshold above is configurable per repo \u2014 the global",
14241
+ "`acceptance-criteria` / `sources` classification bands and each",
14242
+ "per-phase-label override. Retune them in the project's",
14243
+ "configulator config so the gate fits this repo's issue shapes:",
14244
+ "",
14245
+ "- `AgentConfigOptions.scopeGate.acceptanceCriteria` /",
14246
+ " `AgentConfigOptions.scopeGate.sources` \u2014 the global",
14247
+ " `smallMax` / `mediumMax` bands applied to every issue.",
14248
+ "- `AgentConfigOptions.scopeGate.bundleOverrides.<phase-label>` \u2014",
14249
+ " a per-phase-label override for **any** phase label, not just",
14250
+ " the ones shipped above. The map deep-merges with the shipped",
14251
+ " defaults (consumer-wins-per-key): add a key to raise a new",
14252
+ " label's AC / sources ceiling, set an existing key to replace a",
14253
+ " shipped default, or set a key to `undefined` to drop a shipped",
14254
+ " default and classify that label against the global bands.",
14255
+ "",
14256
+ "Each override entry's `acceptanceCriteria` and `sources` axes are",
14257
+ "independent \u2014 override one and the other falls through to the",
14258
+ "global band. Malformed thresholds (negative, non-integer, or",
14259
+ "`mediumMax <= smallMax`) fail the build at synth time."
14260
+ );
14130
14261
  lines.push(
14131
14262
  "",
14132
14263
  "### Decomposition-proposal template",
@@ -17993,7 +18124,11 @@ var prReviewerSubAgent = {
17993
18124
  "",
17994
18125
  "The PR is **eligible** only when **all** of the following hold:",
17995
18126
  "",
17996
- '1. `mergeable == "MERGEABLE"` (no merge conflicts).',
18127
+ '1. `mergeable == "MERGEABLE"` (no merge conflicts). A',
18128
+ ' `mergeable == "CONFLICTING"` PR is **not** an automatic',
18129
+ " disqualification \u2014 see the **already-`CONFLICTING` handling**",
18130
+ " below, which may auto-correct it via delegation before falling",
18131
+ " back to a hard reject.",
17997
18132
  "2. No **failing** required checks in `statusCheckRollup` \u2014 CI must be",
17998
18133
  " green or still pending. Any `FAILURE`, `TIMED_OUT`, `CANCELLED`, or",
17999
18134
  " `ERROR` conclusion on a required check disqualifies the PR. If the",
@@ -18005,8 +18140,9 @@ var prReviewerSubAgent = {
18005
18140
  "3. The PR body contains a linked issue via one of the closing keywords:",
18006
18141
  " `Closes #N`, `Fixes #N`, or `Resolves #N` (case-insensitive).",
18007
18142
  "",
18008
- "If **any** check fails, post a short comment explaining the reason and",
18009
- "stop. Do not proceed to full review.",
18143
+ "If the **CI** check (2) or the **linked-issue** check (3) fails, post",
18144
+ "a short comment explaining the reason and stop. Do not proceed to",
18145
+ "full review.",
18010
18146
  "",
18011
18147
  "```bash",
18012
18148
  "gh pr comment <pr-number> --body '<reason>'",
@@ -18014,10 +18150,50 @@ var prReviewerSubAgent = {
18014
18150
  "",
18015
18151
  "Typical reasons:",
18016
18152
  "",
18017
- "- `Not reviewable: merge conflicts \u2014 please rebase onto the default branch.`",
18018
18153
  "- `Not reviewable: required CI check <name> is failing.`",
18019
18154
  "- `Not reviewable: PR body is missing a linked issue (Closes #N / Fixes #N / Resolves #N).`",
18020
18155
  "",
18156
+ "### Already-`CONFLICTING` handling (auto-correct, then reject)",
18157
+ "",
18158
+ 'When the merge check (1) reports `mergeable == "CONFLICTING"`, do',
18159
+ "**not** unconditionally reject. A bot-authored PR that is already",
18160
+ "conflicting when the orchestrator finds it is exactly the case the",
18161
+ "conflict-resolution delegation flow exists to auto-correct \u2014 there",
18162
+ "is no reason to make a human rebase it by hand when `issue-worker`",
18163
+ "can. Before rejecting, evaluate checks (2) and (3) above first: a",
18164
+ "`CONFLICTING` PR that **also** has failing CI or no linked issue is",
18165
+ "rejected for that reason via the comment-and-stop path above, since",
18166
+ "the delegation cannot succeed without a linked issue and green-able",
18167
+ "CI anyway.",
18168
+ "",
18169
+ "Otherwise, hand off to the **Shared Procedure: Conflict-Resolution",
18170
+ "Delegation** section. It evaluates the guards (auto-merge mode, the",
18171
+ "delegation invocation guard, a free `review:fixing` lease, no",
18172
+ "generated / projen / lockfile conflicts), classifies the conflicting",
18173
+ "files, and either delegates a rebase to `issue-worker` or falls",
18174
+ "through to its own fallback. Invoke it with this entry point's",
18175
+ "parameters:",
18176
+ "",
18177
+ "- **Entry point:** `already-conflicting`. There is no failed",
18178
+ " `gh pr update-branch` to read conflicting paths from \u2014 identify",
18179
+ " the conflicting files from the PR's merge state as documented in",
18180
+ " the shared procedure's entry-point step (a local rebase attempt",
18181
+ " whose `<<<<<<<` / `=======` / `>>>>>>>` markers name the",
18182
+ " conflicting files). The guards and classification are identical to",
18183
+ " the Phase 4 entry point.",
18184
+ "- **On the procedure returning `delegated`:** a rebase has been",
18185
+ " delegated to `issue-worker`; stop this pass. A human re-invokes",
18186
+ " the reviewer once the worker pushes the rebased branch.",
18187
+ "- **On the procedure returning `skipped` (a guard denied delegation",
18188
+ " or the guards failed):** fall back to the original hard reject \u2014",
18189
+ " post the merge-conflict rejection comment and stop. Use this",
18190
+ " comment **only** in the skipped case, never before the shared",
18191
+ " procedure has had its chance to auto-correct:",
18192
+ "",
18193
+ "```bash",
18194
+ "gh pr comment <pr-number> --body 'Not reviewable: merge conflicts \u2014 please rebase onto the default branch.'",
18195
+ "```",
18196
+ "",
18021
18197
  "## Phase 2: Gather Context",
18022
18198
  "",
18023
18199
  "```bash",
@@ -18719,37 +18895,215 @@ var prReviewerSubAgent = {
18719
18895
  "##### Conflict-resolution delegation (BEHIND + conflicts)",
18720
18896
  "",
18721
18897
  "When `gh pr update-branch <pr-number>` fails because the merge would",
18722
- "produce conflicts, the reviewer **may** delegate conflict resolution to",
18723
- "`issue-worker` via the existing feedback-mode delegation contract (the",
18724
- "same path Phase 4's in-scope-fix delegation uses). The reviewer never",
18725
- "hand-resolves conflicts itself \u2014 branch mutations always belong to",
18726
- "`issue-worker`.",
18898
+ "produce conflicts, hand off to the **Shared Procedure:",
18899
+ "Conflict-Resolution Delegation** section below. That procedure owns",
18900
+ "the guards, the `shared-index` vs `generic` classification, the two",
18901
+ "typed recipes, and the fallback \u2014 defined once and shared with the",
18902
+ "Phase 1.5 already-`CONFLICTING` entry point. Invoke it with this",
18903
+ "entry point's parameters:",
18904
+ "",
18905
+ "- **Entry point:** `update-branch-conflict`. The set of conflicting",
18906
+ " files is the list reported by the failed `gh pr update-branch`",
18907
+ " (cross-referenced with `gh pr view <pr-number> --json files`).",
18908
+ "- **On the procedure returning `delegated`:** record",
18909
+ " `Branch updated: delegated (PR #<n>)` for the per-PR report.",
18910
+ "- **On the procedure returning `skipped` (a guard denied delegation",
18911
+ " or the guards failed):** the procedure has already posted the",
18912
+ " fallback comment and recorded",
18913
+ " `Branch updated: failed (conflicts; <short reason>)`. Stop.",
18914
+ "",
18915
+ "#### Mode `human-required`",
18916
+ "",
18917
+ "Do **not** run `gh pr merge --auto`. Instead, hand the PR off to a",
18918
+ "human reviewer:",
18919
+ "",
18920
+ "1. Apply the `review:awaiting-human` label so the PR is discoverable",
18921
+ " in the human-review queue:",
18922
+ "",
18923
+ " ```bash",
18924
+ " gh pr edit <pr-number> --add-label 'review:awaiting-human'",
18925
+ " ```",
18926
+ "",
18927
+ "2. **If `matched_rule == 6`** (size threshold was the sole trigger),",
18928
+ " run the `Update the branch when `mergeStateStatus` is `BEHIND``",
18929
+ " step from the `Mode auto-merge` branch above before exiting. The",
18930
+ " eligibility gate documented in that sub-section explicitly permits",
18931
+ " `gh pr update-branch` on size-only human-required PRs so the bot",
18932
+ " keeps the branch fresh against the default branch while the human",
18933
+ " reviews. Skip this sub-step for `matched_rule` in 2\u20135 \u2014 the gate",
18934
+ " denies `update-branch` there and the human owns branch-lifecycle.",
18935
+ "",
18936
+ "3. Exit cleanly after the acceptance-criteria check completes and any",
18937
+ " summary comment the reviewer posts. Proceed to Phase 5 only if a",
18938
+ " merge occurred \u2014 in `human-required` mode the reviewer stops at",
18939
+ " the hand-off and does not poll for merge.",
18940
+ "",
18941
+ "The reason captured in Phase 2.75 should be included in any summary",
18942
+ "comment so maintainers know why human review was required (path hit,",
18943
+ "issue type, size threshold, label, or default).",
18944
+ "",
18945
+ "### If any criterion is missing, partial, or CI is failing",
18946
+ "",
18947
+ "Post a plain comment (not a formal review block) with grouped findings:",
18948
+ "",
18949
+ "```bash",
18950
+ "gh pr comment <pr-number> --body '<grouped findings>'",
18951
+ "```",
18952
+ "",
18953
+ "Group findings by severity (**Blocking** / **Suggested** / **Nitpick**).",
18954
+ "For each blocking finding, cite the unmet acceptance criterion and the",
18955
+ "file or function the gap lives in. Do **not** merge, and do **not** push",
18956
+ "any commits to the PR's branch.",
18957
+ "",
18958
+ "Rationale for using a plain comment rather than `gh pr review",
18959
+ "--request-changes`: it is lighter-weight, doesn't require the author to",
18960
+ "dismiss a formal review, and composes cleanly with the multi-PR loop.",
18961
+ "",
18962
+ "## Phase 4.5: Update Sticky Summary",
18963
+ "",
18964
+ "On **every pass**, create or update a single **`## Reviewer notes`**",
18965
+ "comment on the PR. This sticky comment is the human-facing single",
18966
+ "source of truth for the PR's state \u2014 one comment per PR, edited in",
18967
+ "place across passes, never duplicated. Do not post a fresh",
18968
+ '"pass N summary" on each iteration.',
18969
+ "",
18970
+ "### Step 1: Find the existing sticky comment (if any)",
18971
+ "",
18972
+ "List PR-level comments and look for one authored by the reviewer",
18973
+ "whose body starts with `## Reviewer notes`:",
18974
+ "",
18975
+ "```bash",
18976
+ "gh api repos/{{repository.owner}}/{{repository.name}}/issues/<pr-number>/comments",
18977
+ "```",
18978
+ "",
18979
+ "If multiple candidates exist (an older run double-posted before this",
18980
+ "phase existed), keep the earliest and plan to delete the duplicates",
18981
+ "on a later pass. Never delete another author's comment.",
18982
+ "",
18983
+ "### Step 2: Compose the sticky body",
18984
+ "",
18985
+ "The body uses the following shape:",
18986
+ "",
18987
+ "```",
18988
+ "## Reviewer notes",
18989
+ "",
18990
+ "**Mode:** auto-merge | human-required \u2014 <reason from Phase 2.75>",
18991
+ "**AC status:** <met / partial / missing> (evidence links)",
18992
+ "**CI status:** green | pending | red",
18993
+ "**Outstanding:** <list of comments still open with their classification and author>",
18994
+ "**Pushbacks:** <list of disputes with reasons>",
18995
+ "**Last pass:** <ISO timestamp>",
18996
+ "```",
18997
+ "",
18998
+ "Populate each field from the phases above:",
18999
+ "",
19000
+ "- **Mode / reason** \u2014 the mode and reason recorded in Phase 2.75.",
19001
+ "- **AC status** \u2014 the checklist produced in Phase 3 (met, partial,",
19002
+ " or missing), with links to the files or tests that provide the",
19003
+ " evidence.",
19004
+ "- **CI status** \u2014 the verdict from the **CI Verification** section",
19005
+ " (primary check-runs read, or the Actions-runs fallback when",
19006
+ " check-runs returns a fine-grained-PAT 403).",
19007
+ "- **Outstanding** \u2014 the comments still carrying a non-terminal",
19008
+ " reviewer reaction from Phase 3.5 (typically `eyes` for queued",
19009
+ " in-scope items and `nit` / `question` items that remain open).",
19010
+ " List each as `<classification>: <author> \u2014 <short summary> (<url>)`.",
19011
+ "- **Pushbacks** \u2014 every comment the reviewer reacted",
19012
+ " `thinking_face` to on this or any prior pass, with the reason",
19013
+ " captured in the pushback reply. Empty list when there are none.",
19014
+ "- **Last pass** \u2014 the ISO 8601 timestamp of this run.",
19015
+ "",
19016
+ "### Step 3: Create or edit in place",
19017
+ "",
19018
+ "If no existing sticky comment was found in Step 1, create one:",
19019
+ "",
19020
+ "```bash",
19021
+ "gh pr comment <pr-number> --body '<sticky body>'",
19022
+ "```",
19023
+ "",
19024
+ "If an existing sticky comment was found, edit it in place using the",
19025
+ "comment id. Do not delete and re-create \u2014 editing preserves the",
19026
+ "comment's URL and any reactions humans have added to the summary:",
19027
+ "",
19028
+ "```bash",
19029
+ "gh api repos/{{repository.owner}}/{{repository.name}}/issues/comments/<sticky-comment-id> \\",
19030
+ " -X PATCH -f body='<sticky body>'",
19031
+ "```",
19032
+ "",
19033
+ "The sticky summary must be updated on every pass through Phase 4.5,",
19034
+ "including passes that ended in a pushback-gated skip, a",
19035
+ "`NEEDS_CHANGES` findings comment, or the `human-required` hand-off.",
19036
+ "Humans rely on the sticky comment to see the current state of the",
19037
+ "PR at a glance \u2014 it must never go stale while the reviewer is",
19038
+ "actively processing the PR.",
19039
+ "",
19040
+ "## Shared Procedure: Conflict-Resolution Delegation",
19041
+ "",
19042
+ "This procedure resolves a PR that cannot land because its head",
19043
+ "branch conflicts with the default branch. It is invoked from **two**",
19044
+ "entry points, both of which reuse the identical guards,",
19045
+ "classification, typed recipes, and fallback defined here \u2014 the flow",
19046
+ "is defined **once** and shared:",
19047
+ "",
19048
+ "- **Phase 4 `update-branch-conflict`** \u2014 a PR that was MERGEABLE at",
19049
+ " Phase 1.5 went BEHIND, and `gh pr update-branch` then failed",
19050
+ " because the merge would conflict.",
19051
+ "- **Phase 1.5 `already-conflicting`** \u2014 the orchestrator handed the",
19052
+ " reviewer a PR that is **already** `CONFLICTING` (its",
19053
+ ' `mergeable == "CONFLICTING"`) before any review work began.',
19054
+ "",
19055
+ "The reviewer **may** delegate conflict resolution to `issue-worker`",
19056
+ "via the feedback-mode delegation contract (the same path Phase 4's",
19057
+ "in-scope-fix delegation uses). The reviewer never hand-resolves",
19058
+ "conflicts itself \u2014 branch mutations always belong to `issue-worker`.",
19059
+ "",
19060
+ "### Entry-point parameters",
19061
+ "",
19062
+ "The caller supplies which entry point it is and how the conflicting",
19063
+ "files are identified. Everything after this step is identical across",
19064
+ "both entry points.",
19065
+ "",
19066
+ "- **`update-branch-conflict`** \u2014 the conflicting-file set is the",
19067
+ " list reported by the failed `gh pr update-branch` (cross-referenced",
19068
+ " with `gh pr view <pr-number> --json files`).",
19069
+ "- **`already-conflicting`** \u2014 there is no failed `update-branch` to",
19070
+ " read conflicting paths from. Identify the conflicting files from",
19071
+ " the PR's merge state instead: attempt a local rebase of the head",
19072
+ " branch onto the default branch in a scratch checkout (or read the",
19073
+ " conflicting paths git reports) and collect the files that carry",
19074
+ " `<<<<<<<` / `=======` / `>>>>>>>` conflict markers. Use that set",
19075
+ " as the conflicting-file list. The reviewer inspects these markers",
19076
+ " read-only to classify; it does **not** stage, commit, or push any",
19077
+ " resolution \u2014 the actual rebase-and-resolve is delegated to",
19078
+ " `issue-worker` exactly as in the other entry point.",
19079
+ "",
19080
+ "### Guards",
18727
19081
  "",
18728
19082
  "Delegate **only when all** of the following hold. If any guard fails,",
18729
- "fall through to the fallback at the end of this sub-section instead.",
19083
+ "fall through to the **Fallback** at the end of this procedure and",
19084
+ "return `skipped`.",
18730
19085
  "",
18731
19086
  "1. **Review mode is `auto-merge`.** Never delegate conflict",
18732
19087
  " resolution on `human-required` PRs \u2014 pushing worker-resolved",
18733
19088
  " merge content into them expands the diff under review without",
18734
- " the human reviewer's consent. (Unlike the `update-branch` step",
18735
- " above, which permits a size-only `human-required` carve-out, the",
19089
+ " the human reviewer's consent. (Unlike the `update-branch` step,",
19090
+ " which permits a size-only `human-required` carve-out, the",
18736
19091
  " conflict-resolution delegation flow is auto-merge-only across",
18737
19092
  " the board: a worker rebase push is a stronger branch mutation",
18738
19093
  " than the merge-commit `gh pr update-branch` performs.)",
18739
19094
  "2. **Delegation invocation guard permits the hand-off** \u2014 the PR",
18740
19095
  " carries the `origin:issue-worker` label, **or** the reviewer was",
18741
19096
  " invoked with `--allow-human-author`. The same guard used for the",
18742
- " in-scope-fix delegation flow above applies here unchanged.",
19097
+ " in-scope-fix delegation flow applies here unchanged.",
18743
19098
  "3. **The `review:fixing` lease is currently free.** If",
18744
19099
  " `review:fixing` is already on the PR, another delegation is",
18745
19100
  " in-flight; skip conflict-resolution delegation, log the contention",
18746
19101
  " to the sticky summary, and fall through to the fallback.",
18747
- "4. **No conflicting file is generated or projen-managed.** Run",
18748
- " `gh pr view <pr-number> --json files --jq '.files[].path'` and",
18749
- " inspect the conflicting paths reported by the failed",
18750
- " `update-branch`. If **any** conflicting path matches one of the",
18751
- " following globs, the conflict is unsafe to resolve mechanically",
18752
- " and the reviewer must skip delegation:",
19102
+ "4. **No conflicting file is generated or projen-managed.** Inspect",
19103
+ " the conflicting paths from the entry-point step above. If **any**",
19104
+ " conflicting path matches one of the following globs, the conflict",
19105
+ " is unsafe to resolve mechanically and the reviewer must skip",
19106
+ " delegation:",
18753
19107
  " - `**/*.lock` and `pnpm-lock.yaml` / `yarn.lock` / `package-lock.json`",
18754
19108
  " - `**/.projen/**`",
18755
19109
  " - any file whose first 5 lines contain the marker",
@@ -18760,17 +19114,18 @@ var prReviewerSubAgent = {
18760
19114
  " and conflicts there should be resolved by re-running synth, not by",
18761
19115
  " merging the conflict markers.",
18762
19116
  "",
19117
+ "### Classify each conflicting file",
19118
+ "",
18763
19119
  "When every guard above passes, **classify each conflicting file**",
18764
19120
  "before composing the fix-list. The classification picks one of two",
18765
19121
  "typed recipes \u2014 a precise `shared-index` resolver when every",
18766
19122
  "conflict is a row-insert race on a registry / index / feature-matrix",
18767
19123
  "file, or the generic rebase recipe in every other case.",
18768
19124
  "",
18769
- "**Classification step.** For each conflicting path reported by the",
18770
- "failed `update-branch`, decide whether the file is `shared-index` or",
18771
- "`generic` against these criteria (the shared-index glob set comes",
18772
- "from the `shared-editing-safety` rule \u2014 see the bundle's",
18773
- "`shared-editing.ts` for the canonical constant):",
19125
+ "**Classification step.** For each conflicting path, decide whether",
19126
+ "the file is `shared-index` or `generic` against these criteria (the",
19127
+ "shared-index glob set comes from the `shared-editing-safety` rule \u2014",
19128
+ "see the bundle's `shared-editing.ts` for the canonical constant):",
18774
19129
  "",
18775
19130
  "- `shared-index` \u2014 the path matches one of the shared-editing glob",
18776
19131
  " patterns:",
@@ -18805,6 +19160,8 @@ var prReviewerSubAgent = {
18805
19160
  " (see step 3 below). The worker performs a hand-merge of the",
18806
19161
  " conflict markers.",
18807
19162
  "",
19163
+ "### Hand off to `issue-worker`",
19164
+ "",
18808
19165
  "Hand off to `issue-worker` with the chosen single synthetic",
18809
19166
  "fix-list item:",
18810
19167
  "",
@@ -18892,146 +19249,24 @@ var prReviewerSubAgent = {
18892
19249
  "",
18893
19250
  "7. **Do not re-enable auto-merge in the same pass.** After delegation,",
18894
19251
  " exit and let a human re-invoke the reviewer. The next pass will",
18895
- " re-evaluate `mergeStateStatus` against the rebased branch and",
18896
- " re-enable auto-merge through the normal flow above.",
19252
+ " re-evaluate `mergeStateStatus` (or `mergeable`) against the",
19253
+ " rebased branch and re-enable auto-merge through the normal flow.",
19254
+ "",
19255
+ "Return `delegated` to the caller when delegation completes",
19256
+ "(regardless of the worker's outcome \u2014 the delegation itself is the",
19257
+ "action taken).",
18897
19258
  "",
18898
- "Record `Branch updated: delegated (PR #<n>)` for the per-PR report",
18899
- "when delegation completes (regardless of the worker's outcome \u2014 the",
18900
- "delegation itself is the action taken).",
19259
+ "### Fallback",
18901
19260
  "",
18902
19261
  "**Fallback when delegation is not permitted or guards fail.** Post a",
18903
19262
  "short comment explaining why the branch could not be updated",
18904
19263
  "automatically and stop. Do not push commits, do not force, do not",
18905
- "retry. Record `Branch updated: failed (conflicts; <short reason>)`.",
18906
- "",
18907
- "```bash",
18908
- "gh pr comment <pr-number> --body 'Auto-merge queued; 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 rebase) will need to resolve.'",
18909
- "```",
18910
- "",
18911
- "#### Mode `human-required`",
18912
- "",
18913
- "Do **not** run `gh pr merge --auto`. Instead, hand the PR off to a",
18914
- "human reviewer:",
18915
- "",
18916
- "1. Apply the `review:awaiting-human` label so the PR is discoverable",
18917
- " in the human-review queue:",
18918
- "",
18919
- " ```bash",
18920
- " gh pr edit <pr-number> --add-label 'review:awaiting-human'",
18921
- " ```",
18922
- "",
18923
- "2. **If `matched_rule == 6`** (size threshold was the sole trigger),",
18924
- " run the `Update the branch when `mergeStateStatus` is `BEHIND``",
18925
- " step from the `Mode auto-merge` branch above before exiting. The",
18926
- " eligibility gate documented in that sub-section explicitly permits",
18927
- " `gh pr update-branch` on size-only human-required PRs so the bot",
18928
- " keeps the branch fresh against the default branch while the human",
18929
- " reviews. Skip this sub-step for `matched_rule` in 2\u20135 \u2014 the gate",
18930
- " denies `update-branch` there and the human owns branch-lifecycle.",
18931
- "",
18932
- "3. Exit cleanly after the acceptance-criteria check completes and any",
18933
- " summary comment the reviewer posts. Proceed to Phase 5 only if a",
18934
- " merge occurred \u2014 in `human-required` mode the reviewer stops at",
18935
- " the hand-off and does not poll for merge.",
18936
- "",
18937
- "The reason captured in Phase 2.75 should be included in any summary",
18938
- "comment so maintainers know why human review was required (path hit,",
18939
- "issue type, size threshold, label, or default).",
18940
- "",
18941
- "### If any criterion is missing, partial, or CI is failing",
18942
- "",
18943
- "Post a plain comment (not a formal review block) with grouped findings:",
18944
- "",
18945
- "```bash",
18946
- "gh pr comment <pr-number> --body '<grouped findings>'",
18947
- "```",
18948
- "",
18949
- "Group findings by severity (**Blocking** / **Suggested** / **Nitpick**).",
18950
- "For each blocking finding, cite the unmet acceptance criterion and the",
18951
- "file or function the gap lives in. Do **not** merge, and do **not** push",
18952
- "any commits to the PR's branch.",
18953
- "",
18954
- "Rationale for using a plain comment rather than `gh pr review",
18955
- "--request-changes`: it is lighter-weight, doesn't require the author to",
18956
- "dismiss a formal review, and composes cleanly with the multi-PR loop.",
18957
- "",
18958
- "## Phase 4.5: Update Sticky Summary",
18959
- "",
18960
- "On **every pass**, create or update a single **`## Reviewer notes`**",
18961
- "comment on the PR. This sticky comment is the human-facing single",
18962
- "source of truth for the PR's state \u2014 one comment per PR, edited in",
18963
- "place across passes, never duplicated. Do not post a fresh",
18964
- '"pass N summary" on each iteration.',
18965
- "",
18966
- "### Step 1: Find the existing sticky comment (if any)",
18967
- "",
18968
- "List PR-level comments and look for one authored by the reviewer",
18969
- "whose body starts with `## Reviewer notes`:",
19264
+ "retry. Record `Branch updated: failed (conflicts; <short reason>)`",
19265
+ "and return `skipped` to the caller.",
18970
19266
  "",
18971
19267
  "```bash",
18972
- "gh api repos/{{repository.owner}}/{{repository.name}}/issues/<pr-number>/comments",
18973
- "```",
18974
- "",
18975
- "If multiple candidates exist (an older run double-posted before this",
18976
- "phase existed), keep the earliest and plan to delete the duplicates",
18977
- "on a later pass. Never delete another author's comment.",
18978
- "",
18979
- "### Step 2: Compose the sticky body",
18980
- "",
18981
- "The body uses the following shape:",
18982
- "",
19268
+ "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 rebase) will need to resolve.'",
18983
19269
  "```",
18984
- "## Reviewer notes",
18985
- "",
18986
- "**Mode:** auto-merge | human-required \u2014 <reason from Phase 2.75>",
18987
- "**AC status:** <met / partial / missing> (evidence links)",
18988
- "**CI status:** green | pending | red",
18989
- "**Outstanding:** <list of comments still open with their classification and author>",
18990
- "**Pushbacks:** <list of disputes with reasons>",
18991
- "**Last pass:** <ISO timestamp>",
18992
- "```",
18993
- "",
18994
- "Populate each field from the phases above:",
18995
- "",
18996
- "- **Mode / reason** \u2014 the mode and reason recorded in Phase 2.75.",
18997
- "- **AC status** \u2014 the checklist produced in Phase 3 (met, partial,",
18998
- " or missing), with links to the files or tests that provide the",
18999
- " evidence.",
19000
- "- **CI status** \u2014 the verdict from the **CI Verification** section",
19001
- " (primary check-runs read, or the Actions-runs fallback when",
19002
- " check-runs returns a fine-grained-PAT 403).",
19003
- "- **Outstanding** \u2014 the comments still carrying a non-terminal",
19004
- " reviewer reaction from Phase 3.5 (typically `eyes` for queued",
19005
- " in-scope items and `nit` / `question` items that remain open).",
19006
- " List each as `<classification>: <author> \u2014 <short summary> (<url>)`.",
19007
- "- **Pushbacks** \u2014 every comment the reviewer reacted",
19008
- " `thinking_face` to on this or any prior pass, with the reason",
19009
- " captured in the pushback reply. Empty list when there are none.",
19010
- "- **Last pass** \u2014 the ISO 8601 timestamp of this run.",
19011
- "",
19012
- "### Step 3: Create or edit in place",
19013
- "",
19014
- "If no existing sticky comment was found in Step 1, create one:",
19015
- "",
19016
- "```bash",
19017
- "gh pr comment <pr-number> --body '<sticky body>'",
19018
- "```",
19019
- "",
19020
- "If an existing sticky comment was found, edit it in place using the",
19021
- "comment id. Do not delete and re-create \u2014 editing preserves the",
19022
- "comment's URL and any reactions humans have added to the summary:",
19023
- "",
19024
- "```bash",
19025
- "gh api repos/{{repository.owner}}/{{repository.name}}/issues/comments/<sticky-comment-id> \\",
19026
- " -X PATCH -f body='<sticky body>'",
19027
- "```",
19028
- "",
19029
- "The sticky summary must be updated on every pass through Phase 4.5,",
19030
- "including passes that ended in a pushback-gated skip, a",
19031
- "`NEEDS_CHANGES` findings comment, or the `human-required` hand-off.",
19032
- "Humans rely on the sticky comment to see the current state of the",
19033
- "PR at a glance \u2014 it must never go stale while the reviewer is",
19034
- "actively processing the PR.",
19035
19270
  "",
19036
19271
  "## Phase 5: Branch Cleanup and Issue Closure Verification",
19037
19272
  "",
@@ -30312,13 +30547,20 @@ function renderFocusSection(focus) {
30312
30547
 
30313
30548
  // src/agent/bundles/meeting-types.ts
30314
30549
  var DEFAULT_AGENDA_TEMPLATE_ROOT = "<meetingsRoot>/_agenda-templates";
30550
+ function actionItemFilingHasOverride(filing) {
30551
+ if (!filing) {
30552
+ return false;
30553
+ }
30554
+ return filing.enabled === false || (filing.humanOwnedCues ?? []).length > 0 || (filing.agentWorkableCues ?? []).length > 0;
30555
+ }
30315
30556
  function renderMeetingTypesSection(meetings) {
30316
30557
  if (!meetings) {
30317
30558
  return "";
30318
30559
  }
30319
30560
  const types = meetings.meetingTypes ?? [];
30320
30561
  const areas = meetings.meetingAreas ?? [];
30321
- if (types.length === 0 && areas.length === 0) {
30562
+ const filingOverride = actionItemFilingHasOverride(meetings.actionItemFiling) ? meetings.actionItemFiling : void 0;
30563
+ if (types.length === 0 && areas.length === 0 && !filingOverride) {
30322
30564
  return "";
30323
30565
  }
30324
30566
  const sections = [];
@@ -30328,6 +30570,9 @@ function renderMeetingTypesSection(meetings) {
30328
30570
  if (areas.length > 0) {
30329
30571
  sections.push(renderMeetingAreas(areas));
30330
30572
  }
30573
+ if (filingOverride) {
30574
+ sections.push(renderActionItemFiling(filingOverride));
30575
+ }
30331
30576
  return sections.join("\n\n");
30332
30577
  }
30333
30578
  function renderMeetingTypes(types, agendaTemplateRoot) {
@@ -30381,6 +30626,54 @@ function renderMeetingAreas(areas) {
30381
30626
  }
30382
30627
  return lines.join("\n");
30383
30628
  }
30629
+ function renderActionItemFiling(filing) {
30630
+ const lines = [
30631
+ "## Action-item filing policy",
30632
+ "",
30633
+ "This project tunes the Phase 4 (Link) agent-workability split",
30634
+ "through `AgentConfigOptions.meetings.actionItemFiling`. See the",
30635
+ "**Action-item filing policy** section of the `meeting-analyst`",
30636
+ "agent for the full baked-in test.",
30637
+ ""
30638
+ ];
30639
+ if (filing.enabled === false) {
30640
+ lines.push(
30641
+ "- **Split disabled.** Phase 4 falls back to the legacy behaviour:",
30642
+ " file a follow-up issue for every action item that is not",
30643
+ " already covered by a requirement or ADR draft, including",
30644
+ " human-owned tasks. The notes `## Action Items` table is still",
30645
+ " written, but it is no longer the sole record for human",
30646
+ " follow-ups."
30647
+ );
30648
+ } else {
30649
+ lines.push(
30650
+ "- **Split enabled** (default). Agent-workable doc deliverables are",
30651
+ " filed through their dedicated downstream channel; human-owned",
30652
+ " tasks are recorded only in the notes `## Action Items` table."
30653
+ );
30654
+ }
30655
+ const humanCues = filing.humanOwnedCues ?? [];
30656
+ if (humanCues.length > 0) {
30657
+ lines.push(
30658
+ "",
30659
+ "**Additional human-owned cues** (recorded notes-only, never",
30660
+ "filed), in addition to the bundle's built-in list:",
30661
+ "",
30662
+ ...humanCues.map((cue) => `- ${cue}`)
30663
+ );
30664
+ }
30665
+ const agentCues = filing.agentWorkableCues ?? [];
30666
+ if (agentCues.length > 0) {
30667
+ lines.push(
30668
+ "",
30669
+ "**Additional agent-workable cues** (filed through the matching",
30670
+ "downstream channel), in addition to the bundle's built-in list:",
30671
+ "",
30672
+ ...agentCues.map((cue) => `- ${cue}`)
30673
+ );
30674
+ }
30675
+ return lines.join("\n");
30676
+ }
30384
30677
 
30385
30678
  // src/agent/bundles/priority-rules.ts
30386
30679
  function renderPriorityRulesSection(rules) {