jonah-fleet 1.4.1 → 1.4.2

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/CHANGELOG.md CHANGED
@@ -5,16 +5,35 @@ All notable changes to `jonah-fleet` will be documented in this file.
5
5
  The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
6
6
  and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
7
7
 
8
+ ## [1.4.2] - 2026-09-03
9
+
10
+ ### Changed
11
+ - Strict Allowlist for Cloud Review Triggers:
12
+ - Updated `trigger-review-routine.yml` to strictly require `priority/P0` or `priority/P1` labels to fire immediate cloud reviews on `pull_request` events.
13
+ - Untagged PRs and lower-priority PRs (P2/P3) skip immediate cloud review, routing review execution exclusively to local peer-review daemons (with 48h scheduled cloud catchup fallback).
14
+
8
15
  ## [1.4.1] - 2026-09-03
9
16
 
10
17
  ### Added
11
18
  - Decoupled Multi-Cadence Daemon & Zero-Cost PR Preflight:
12
19
  - Added independent scheduling in `jonah-fleet daemon` via `--review-interval` (default: 3m) and `--autowork-interval` (default: 30m).
13
20
  - Added ultra-fast (~100ms) local PR preflight check (`countOpenReadyPRs`) in `src/lib/daemon.ts` that queries `gh pr list` and skips agent invocations with 0 token spend when 0 ready PRs are open.
21
+ - User-Friendly Terminal Cards & Daemon Ticker:
22
+ - Replaced noisy raw markdown streaming in `jonah-fleet daemon --foreground` and `jonah-fleet run` with a dynamic single-line status spinner showing target and active phase heuristics.
23
+ - Added real-time token stream redirection to `.jonah-fleet/daemon.log` with `-v, --verbose` escape hatch for debugging.
24
+ - Added formatted Unicode summary and error cards with ephemeral worktree path sanitization and disk log fallbacks (`src/lib/terminal-card.ts`).
25
+ - Added in-place countdown ticker (`\r`) during daemon watchdog idle states to avoid cluttering terminal scrollback.
14
26
 
15
27
  ## [1.4.0] - 2026-09-03
16
28
 
17
29
  ### Added
30
+ - Inquisitive Agent Fleet & Grilling Protocol:
31
+ - Added new `grill-me` engineering skill (`templates/skills/grill-me/SKILL.md` and `.agents/skills/grill-me/SKILL.md`) to stress-test proposals, requirements, and designs through a targeted, sequential interview to eliminate assumptions before implementation (#54, #55).
32
+ - Included `grill-me` in `standard` and `full` presets in `src/lib/presets.ts` and `agents-manifest.json` (#54, #55).
33
+ - Added Requirements Discovery & Inquisitive Stance rules to `templates/docs/AGENTS.template.md` and `AGENTS.md`, establishing default skepticism on necessity, zero-guesswork ambiguity gates, and high-leverage question structuring (#54, #55).
34
+ - Added Ambiguity & Missing Acceptance Criteria Gate to Step 12 in `autowork.md` (`templates/prompts/autowork.md` & `.github/prompts/autowork.md`) to post clarifying questions, label `needs-info`, and release the claim instead of making silent assumptions (#54, #55).
35
+ - Added Selective Server-Side Issue Querying in Step 8 of `autowork.md` using `gh --search` with exclusion filters (`no:assignee -label:measurement -label:needs-human -label:needs-design -label:wontfix -label:needs-info`) to avoid tool buffer truncation and token waste (#54, #55).
36
+ - Updated `triage.md` and `ORCHESTRATION.md` to integrate `/grill-me` into issue evaluation and planning (#54, #55).
18
37
  - Priority-Driven Dual Agent Execution (Local + GitHub Actions):
19
38
  - Added CLI `jonah-fleet run <routine>` command to execute prompt routines locally with automatic Git worktree isolation (`.jonah-fleet/worktrees/`), protecting active working copies and uncommitted editor files.
20
39
  - Added CLI `jonah-fleet daemon [start|stop|status]` background worker to continuously poll open issues and PRs for local execution.
package/dist/index.js CHANGED
@@ -49,7 +49,8 @@ var PRESET_CONFIGS = {
49
49
  "diagnosing-bugs",
50
50
  "resolving-merge-conflicts",
51
51
  "writing-for-agents",
52
- "triage"
52
+ "triage",
53
+ "grill-me"
53
54
  ]
54
55
  },
55
56
  full: {
@@ -71,6 +72,7 @@ var PRESET_CONFIGS = {
71
72
  "resolving-merge-conflicts",
72
73
  "writing-for-agents",
73
74
  "triage",
75
+ "grill-me",
74
76
  "to-spec",
75
77
  "to-tickets"
76
78
  ]
@@ -90,7 +92,7 @@ var ROUTINE_TO_WORKFLOW_MAP = {
90
92
  "product-planning": [],
91
93
  "analytics-review": []
92
94
  };
93
- var FLEET_VERSION = "1.4.1";
95
+ var FLEET_VERSION = "1.4.2";
94
96
  var SCHEMA_URL = "https://raw.githubusercontent.com/juliendurandeu/jonah-fleet/main/schema.json";
95
97
 
96
98
  // src/lib/manifest.ts
@@ -2161,7 +2163,7 @@ import pc9 from "picocolors";
2161
2163
  import fs11 from "fs";
2162
2164
  import path11 from "path";
2163
2165
  import os2 from "os";
2164
- import { spawn } from "child_process";
2166
+ import { spawn, execSync as execSync2 } from "child_process";
2165
2167
 
2166
2168
  // src/lib/worktree.ts
2167
2169
  import fs10 from "fs";
@@ -47,6 +47,6 @@ export declare const PRESET_CONFIGS: Record<Exclude<PresetName, 'custom'>, {
47
47
  }>;
48
48
  export declare const ROUTINE_TO_WORKFLOW_MAP: Record<keyof FleetManifest['routines'], string[]>;
49
49
  export declare const WORKFLOW_TO_ROUTINE_MAP: Record<string, keyof FleetManifest['routines'] | 'sync-fleet'>;
50
- export declare const FLEET_VERSION = "1.4.1";
50
+ export declare const FLEET_VERSION = "1.4.2";
51
51
  export declare const SCHEMA_URL = "https://raw.githubusercontent.com/juliendurandeu/jonah-fleet/main/schema.json";
52
52
  //# sourceMappingURL=presets.d.ts.map
@@ -1 +1 @@
1
- {"version":3,"file":"presets.d.ts","sourceRoot":"","sources":["../../src/lib/presets.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,UAAU,GAAG,SAAS,GAAG,UAAU,GAAG,MAAM,GAAG,QAAQ,CAAC;AAEpE,MAAM,WAAW,aAAa;IAC5B,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,UAAU,CAAC;IACnB,QAAQ,EAAE;QACR,QAAQ,EAAE,OAAO,CAAC;QAClB,aAAa,EAAE,OAAO,CAAC;QACvB,SAAS,EAAE,OAAO,CAAC;QACnB,qBAAqB,EAAE,OAAO,CAAC;QAC/B,kCAAkC,EAAE,OAAO,CAAC;QAC5C,kBAAkB,EAAE,OAAO,CAAC;QAC5B,kBAAkB,EAAE,OAAO,CAAC;KAC7B,CAAC;IACF,SAAS,CAAC,EAAE;QACV,QAAQ,CAAC,EAAE,MAAM,CAAC;QAClB,aAAa,CAAC,EAAE,MAAM,CAAC;QACvB,SAAS,CAAC,EAAE,MAAM,CAAC;QACnB,qBAAqB,CAAC,EAAE,MAAM,CAAC;QAC/B,kCAAkC,CAAC,EAAE,MAAM,CAAC;QAC5C,kBAAkB,CAAC,EAAE,MAAM,CAAC;QAC5B,YAAY,CAAC,EAAE,MAAM,CAAC;QACtB,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAAC;KACnC,CAAC;IACF,MAAM,EAAE,MAAM,EAAE,CAAC;IACjB,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;IACxB,UAAU,CAAC,EAAE;QACX,OAAO,EAAE,OAAO,CAAC;QACjB,OAAO,EAAE,QAAQ,GAAG,QAAQ,CAAC;KAC9B,CAAC;IACF,SAAS,CAAC,EAAE;QACV,OAAO,CAAC,EAAE,OAAO,CAAC;QAClB,QAAQ,CAAC,EAAE,MAAM,CAAC;QAClB,iBAAiB,CAAC,EAAE,MAAM,CAAC;KAC5B,CAAC;IACF,aAAa,CAAC,EAAE,mBAAmB,CAAC;CACrC;AAED,MAAM,WAAW,mBAAmB;IAClC,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,eAAe,CAAC,EAAE,MAAM,EAAE,CAAC;IAC3B,iBAAiB,CAAC,EAAE,MAAM,CAAC;CAC5B;AAED,eAAO,MAAM,6BAA6B,EAAE,mBAI3C,CAAC;AAEF,eAAO,MAAM,cAAc,EAAE,MAAM,CAAC,OAAO,CAAC,UAAU,EAAE,QAAQ,CAAC,EAAE;IAAE,QAAQ,EAAE,aAAa,CAAC,UAAU,CAAC,CAAC;IAAC,MAAM,EAAE,MAAM,EAAE,CAAA;CAAE,CA+D3H,CAAC;AAEF,eAAO,MAAM,uBAAuB,EAAE,MAAM,CAAC,MAAM,aAAa,CAAC,UAAU,CAAC,EAAE,MAAM,EAAE,CAarF,CAAC;AAEF,eAAO,MAAM,uBAAuB,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,aAAa,CAAC,UAAU,CAAC,GAAG,YAAY,CAOlG,CAAC;AAEF,eAAO,MAAM,aAAa,UAAU,CAAC;AACrC,eAAO,MAAM,UAAU,kFAAkF,CAAC"}
1
+ {"version":3,"file":"presets.d.ts","sourceRoot":"","sources":["../../src/lib/presets.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,UAAU,GAAG,SAAS,GAAG,UAAU,GAAG,MAAM,GAAG,QAAQ,CAAC;AAEpE,MAAM,WAAW,aAAa;IAC5B,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,UAAU,CAAC;IACnB,QAAQ,EAAE;QACR,QAAQ,EAAE,OAAO,CAAC;QAClB,aAAa,EAAE,OAAO,CAAC;QACvB,SAAS,EAAE,OAAO,CAAC;QACnB,qBAAqB,EAAE,OAAO,CAAC;QAC/B,kCAAkC,EAAE,OAAO,CAAC;QAC5C,kBAAkB,EAAE,OAAO,CAAC;QAC5B,kBAAkB,EAAE,OAAO,CAAC;KAC7B,CAAC;IACF,SAAS,CAAC,EAAE;QACV,QAAQ,CAAC,EAAE,MAAM,CAAC;QAClB,aAAa,CAAC,EAAE,MAAM,CAAC;QACvB,SAAS,CAAC,EAAE,MAAM,CAAC;QACnB,qBAAqB,CAAC,EAAE,MAAM,CAAC;QAC/B,kCAAkC,CAAC,EAAE,MAAM,CAAC;QAC5C,kBAAkB,CAAC,EAAE,MAAM,CAAC;QAC5B,YAAY,CAAC,EAAE,MAAM,CAAC;QACtB,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAAC;KACnC,CAAC;IACF,MAAM,EAAE,MAAM,EAAE,CAAC;IACjB,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;IACxB,UAAU,CAAC,EAAE;QACX,OAAO,EAAE,OAAO,CAAC;QACjB,OAAO,EAAE,QAAQ,GAAG,QAAQ,CAAC;KAC9B,CAAC;IACF,SAAS,CAAC,EAAE;QACV,OAAO,CAAC,EAAE,OAAO,CAAC;QAClB,QAAQ,CAAC,EAAE,MAAM,CAAC;QAClB,iBAAiB,CAAC,EAAE,MAAM,CAAC;KAC5B,CAAC;IACF,aAAa,CAAC,EAAE,mBAAmB,CAAC;CACrC;AAED,MAAM,WAAW,mBAAmB;IAClC,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,eAAe,CAAC,EAAE,MAAM,EAAE,CAAC;IAC3B,iBAAiB,CAAC,EAAE,MAAM,CAAC;CAC5B;AAED,eAAO,MAAM,6BAA6B,EAAE,mBAI3C,CAAC;AAEF,eAAO,MAAM,cAAc,EAAE,MAAM,CAAC,OAAO,CAAC,UAAU,EAAE,QAAQ,CAAC,EAAE;IAAE,QAAQ,EAAE,aAAa,CAAC,UAAU,CAAC,CAAC;IAAC,MAAM,EAAE,MAAM,EAAE,CAAA;CAAE,CAiE3H,CAAC;AAEF,eAAO,MAAM,uBAAuB,EAAE,MAAM,CAAC,MAAM,aAAa,CAAC,UAAU,CAAC,EAAE,MAAM,EAAE,CAarF,CAAC;AAEF,eAAO,MAAM,uBAAuB,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,aAAa,CAAC,UAAU,CAAC,GAAG,YAAY,CAOlG,CAAC;AAEF,eAAO,MAAM,aAAa,UAAU,CAAC;AACrC,eAAO,MAAM,UAAU,kFAAkF,CAAC"}
@@ -1 +1 @@
1
- {"version":3,"file":"runner.d.ts","sourceRoot":"","sources":["../../src/lib/runner.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,sBAAsB;IACrC,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;IACxB,EAAE,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;IACrB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,YAAY,CAAC,EAAE,OAAO,CAAC;IACvB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC7B,KAAK,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,IAAI,CAAC;CACjC;AAED,MAAM,WAAW,qBAAqB;IACpC,OAAO,EAAE,OAAO,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,CAoB9D;AAED;;GAEG;AACH,wBAAgB,kBAAkB,CAChC,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,MAAM,EACf,OAAO,GAAE;IAAE,KAAK,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;IAAC,EAAE,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;CAAO,GAC9D,MAAM,CAmBR;AAED;;GAEG;AACH,wBAAgB,cAAc,IAAI,OAAO,CAQxC;AAED;;GAEG;AACH,wBAAsB,eAAe,CAAC,OAAO,EAAE,sBAAsB,GAAG,OAAO,CAAC,qBAAqB,CAAC,CAwIrG"}
1
+ {"version":3,"file":"runner.d.ts","sourceRoot":"","sources":["../../src/lib/runner.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,sBAAsB;IACrC,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;IACxB,EAAE,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;IACrB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,YAAY,CAAC,EAAE,OAAO,CAAC;IACvB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC7B,KAAK,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,IAAI,CAAC;CACjC;AAED,MAAM,WAAW,qBAAqB;IACpC,OAAO,EAAE,OAAO,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,CAoB9D;AAED;;GAEG;AACH,wBAAgB,kBAAkB,CAChC,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,MAAM,EACf,OAAO,GAAE;IAAE,KAAK,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;IAAC,EAAE,CAAC,EAAE,MAAM,GAAG,MAAM,CAAA;CAAO,GAC9D,MAAM,CAmBR;AAED;;GAEG;AACH,wBAAgB,cAAc,IAAI,OAAO,CAOxC;AAED;;GAEG;AACH,wBAAsB,eAAe,CAAC,OAAO,EAAE,sBAAsB,GAAG,OAAO,CAAC,qBAAqB,CAAC,CAwIrG"}
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "jonah-fleet",
3
- "version": "1.4.1",
3
+ "version": "1.4.2",
4
4
  "description": "Standalone autonomous agent fleet with Symphony orchestration, claim protocols, and continuous improvement loops",
5
5
  "main": "dist/index.js",
6
6
  "types": "dist/index.d.ts",
@@ -41,6 +41,12 @@ tests/ # Test suites
41
41
  - **Error Handling**: Validate inputs at server and module boundaries. Return explicit structured errors.
42
42
  - **Testing Philosophy**: Write regression unit tests for every bug fix and integration tests for key workflows.
43
43
 
44
+ ## Requirements Discovery & Inquisitive Stance
45
+
46
+ - **Challenge Premise First**: Do not be a passive order-taker. Before writing code or drafting large plans, verify whether the requested change is necessary, whether it solves a root cause rather than a symptom, or if existing primitives in the codebase already handle it.
47
+ - **Zero-Guesswork Ambiguity Gate**: Never make silent assumptions about underspecified requirements, schemas, edge cases, or UX flows. Stop and ask clarifying questions before implementation.
48
+ - **Structured Clarifications & Grilling**: When asking questions, propose 1–2 high-leverage questions at a time with concrete options or trade-offs. For complex features or fuzzy requirements, invoke `/grill-me` to stress-test the design before writing code.
49
+
44
50
  ## Conventions
45
51
 
46
52
  - **Language**: All code, comments, commit messages, and PR titles must be written in **English**.
@@ -83,8 +83,11 @@ The routines invoke specialized engineering skills at key workflow checkpoints:
83
83
  - `/resolving-merge-conflicts`: Resolves merge conflicts mechanically before squash-merging.
84
84
  - **Issues Housekeeping (`issues-housekeeping.md`)**:
85
85
  - `/triage`: Evaluates incoming issues into canonical roles.
86
+ - **Triage & Interactive Planning**:
87
+ - `/grill-me`: Interrogates proposals and requirements to uncover edge cases and force explicit trade-offs before implementation.
86
88
  - **Product Planning (`product-planning.md`)**:
87
89
  - `/domain-modeling`: Pressure-tests proposals and records domain terms / ADRs.
90
+ - `/grill-me`: Stress-tests proposed roadmaps and user stories.
88
91
  - `/to-spec`: Authors formal PRDs for larger proposals.
89
92
  - `/to-tickets`: Decomposes approved epics/proposals into dependency-linked issues.
90
93
  - **Prompt Optimizer (`optimizer.md`)**:
@@ -43,6 +43,7 @@ If any criterion cannot be met, stop immediately and log FAILURE with the reason
43
43
  - Do not start implementing an issue before claiming it (both assignment AND claim comment).
44
44
  - Do not mark a PR ready while its `mergeable_state` is `dirty` — resolve merge conflicts first.
45
45
  - Do not fall into the **Telemetry Rabbit Hole**: do not spend cycles instrumenting elaborate fallback telemetry or defensive error handling for features that suffer from lack of user intent rather than software bugs.
46
+ - Do not guess or invent arbitrary specifications for ambiguous issues — post clarifying questions, label `needs-info`, and release the claim instead of blindly writing code.
46
47
 
47
48
  ## Instructions
48
49
 
@@ -85,10 +86,13 @@ b. **Eligible → claim, then implement.** Call `get_me` once to learn your own
85
86
  6. Count open PRs authored by automated sessions representing live reviewable work (excluding log-only PRs). If 3 or more non-log PRs are open, STOP — run is SUCCESS with "Too many open PRs, converging first".
86
87
  7. Call `get_me` once to learn your own GitHub account (`login`).
87
88
  8. List open issues sorted by priority labels (P1 > P2 > P3). Within the same priority tier, order by type (`bug`/`security` before others), then oldest first.
89
+ - **Selective Server-Side Querying**: To prevent tool buffer truncation and avoid ingesting un-actionable issues, query GitHub with server-side exclusion filters rather than dumping all issues. E.g.:
90
+ `gh issue list --state open --search "no:assignee -label:measurement -label:needs-human -label:needs-design -label:wontfix -label:needs-info" --limit 30`
91
+ (or filter by priority tier: `--search "label:priority/P0,priority/P1 no:assignee -label:measurement -label:needs-human -label:needs-design -label:wontfix -label:needs-info"`).
88
92
  - **Dual Execution Priority Routing**:
89
93
  - If running in **Cloud Actions** (`$GITHUB_ACTIONS` / `$CI`): Scan mode selects `priority/P0` or `priority/P1` issues first. Lower-priority `priority/P2` or `priority/P3` issues are eligible in cloud Scan mode ONLY if the issue has remained unclaimed for more than 48 hours (`created_at` older than 48h, acting as a cloud catchup sweep).
90
94
  - If running in **Local Agent** (`$LOCAL_AGENT`): Scan mode selects across all priorities (P0 → P1 → P2 → P3) without time gating, prioritizing active local backlog consumption.
91
- - Skip assigned issues (unless stale claim per `ORCHESTRATION.md`), issues with open PRs, issues with unclosed blocking dependencies (`Blocked by #N` / `Depends on #N`), issues labeled `needs-human` or `needs-design`, and issues under cross-run cooldown.
95
+ - Skip assigned issues (unless stale claim per `ORCHESTRATION.md`), issues with open PRs, issues with unclosed blocking dependencies (`Blocked by #N` / `Depends on #N`), issues labeled `needs-human`, `needs-design`, or `needs-info`, and issues under cross-run cooldown.
92
96
  9. If no eligible candidate exists, STOP — run is SUCCESS with "No unclaimed work available".
93
97
  10. If the candidate should be closed already (work done, PRs merged), close it and return to step 8.
94
98
  10a. **Stale-claim reclamation:** If candidate is a stale claim per `ORCHESTRATION.md`, re-read immediately before writing, unassign the dead owner, post reclamation comment, and proceed to claim.
@@ -103,6 +107,11 @@ b. **Eligible → claim, then implement.** Call `get_me` once to learn your own
103
107
  - Read the issue description, linked code, and comment thread.
104
108
  - If bug: use `/diagnosing-bugs` to establish reproduction test before fixing.
105
109
  - If large/complex: use `/domain-modeling` and `/codebase-design`.
110
+ - **Ambiguity & Missing Acceptance Criteria Gate**: Challenge underspecified or incomplete requests before writing any code. If the issue lacks observable acceptance criteria, relies on unverified assumptions, or leaves critical technical/UX decisions ambiguous:
111
+ - Do NOT guess or invent arbitrary requirements to force completion.
112
+ - Post a comment on the issue posing 1–3 focused clarifying questions that identify the exact decisions or trade-offs needed.
113
+ - Apply the `needs-info` label and release the claim (unassign).
114
+ - Select the next candidate (evaluating ambiguous issues counts toward step 12's infeasible-continuation cap).
106
115
  - **Intent vs. Defect Guardrail**: When investigating issues related to low conversion, zero-click events, or underperforming features: verify whether the issue is a software defect or a lack of user intent. If data indicates the root cause is **lack of user intent** (e.g. button is rendered above fold and functions correctly when clicked, but user interaction rate is <2%) rather than a software defect, do NOT fall into the **telemetry rabbit hole** (adding elaborate fallback telemetry, downstream error handling, or defensive rendering). Categorize the issue as a **product/UX question** (`needs-design` / `roadmap/*`), comment explaining the lack of user intent, release the claim (unassign), and select the next candidate.
107
116
  - If infeasible: comment explaining blocker, release claim (unassign), and select next candidate (up to 3 infeasible evaluations per run). If permanent blocker on 2nd strike, apply `needs-human` label and tag repo owner.
108
117
  12a. **Umbrella-issue handoff + batching:** If candidate is an umbrella epic:
@@ -0,0 +1,64 @@
1
+ ---
2
+ name: grill-me
3
+ description: Stress-test proposals, requirements, and designs through a targeted, sequential interview to eliminate assumptions before implementation.
4
+ ---
5
+
6
+ # Grill Me
7
+
8
+ Stress-test plans, requirements, and designs through an active, sequential interview. The agent acts as an inquisitor—a cross between a principal engineer and a rigorous technical product manager.
9
+
10
+ Your goal is **not** to validate or agree with the user. Your goal is to **uncover gaps, challenge necessity, expose hidden edge cases, and force clear decisions** before any code is written or formal spec drafted.
11
+
12
+ ## The Grilling Mindset
13
+
14
+ - **Default to skepticism**: Ask why this needs to be built at all. Can this be solved with existing codebase primitives, configuration, or by deleting code?
15
+ - **Zero-guesswork discipline**: Do not guess what the user meant when requirements are ambiguous. Force explicit choices.
16
+ - **One to two questions per turn**: Never dump a questionnaire or a wall of questions. Ask at most 1–2 high-leverage questions at a time, provide concrete trade-offs or recommended options, and wait for the response before proceeding.
17
+
18
+ ## Process
19
+
20
+ Execute these phases in sequence:
21
+
22
+ ### Phase 1: Challenge the Premise & Scope
23
+
24
+ 1. **Problem vs. Symptom**: Is the user solving the real root problem or patching a surface symptom?
25
+ 2. **Alternative & Simplicity**: Could this be handled without new abstractions? What is the simplest thing that could possibly work?
26
+ 3. **Necessity & ROI**: What happens if we do nothing? Who specifically benefits from this change?
27
+
28
+ Probe the premise:
29
+ - "What exact problem does this solve, and why can't we solve it with [existing module/primitive]?"
30
+ - "What is explicitly *out of scope* for this change?"
31
+
32
+ ### Phase 2: Stress-Test Technical Seams & Failure Modes
33
+
34
+ Once the premise is solid, probe failure modes and system boundaries:
35
+
36
+ 1. **Failure & Degraded States**: What happens if network calls fail, services time out, or dependencies crash?
37
+ 2. **Boundary & Empty States**: What happens with empty data, maximum limits, concurrent writes, or conflicting inputs?
38
+ 3. **Security & Auth**: Are there permission gates, auth boundaries, or data leakage risks?
39
+ 4. **Data Model & Schema**: Does this change existing contracts? Is it backward-compatible?
40
+
41
+ ### Phase 3: Establish Concrete Acceptance Criteria
42
+
43
+ Force observable, testable definitions of done:
44
+
45
+ 1. **Test Seams**: At which seam (unit, integration, end-to-end) will this be verified?
46
+ 2. **Observable Outcomes**: What is the exact input and expected output? How will a test or user prove this works?
47
+ 3. **Negative Paths**: What negative test cases must fail safely?
48
+
49
+ ### Phase 4: Hand-off & Synthesis
50
+
51
+ When all critical ambiguities are resolved:
52
+ 1. Summarize the agreed scope, explicit non-goals, architectural decisions, and acceptance criteria in a concise brief.
53
+ 2. Direct the next action:
54
+ - For formal requirements: run `/to-spec` or author an agent brief.
55
+ - For domain terminology: run `/domain-modeling`.
56
+ - For immediate implementation: proceed test-first via `/tdd`.
57
+
58
+ ## Completion Criteria
59
+
60
+ The grilling session is complete only when:
61
+ - [ ] Core motivation and explicit non-goals are established
62
+ - [ ] Edge cases and failure modes have explicit decisions recorded
63
+ - [ ] Acceptance criteria and verification seams are concrete and testable
64
+ - [ ] The user confirms alignment with the summarized decisions
@@ -0,0 +1,5 @@
1
+ interface:
2
+ display_name: "Grill Me"
3
+ short_description: "Stress-test proposals, requirements, and designs"
4
+ policy:
5
+ allow_implicit_invocation: false
@@ -73,7 +73,7 @@ Show counts and a one-line summary per item. Let the maintainer pick.
73
73
 
74
74
  3. **Verify the claim.** Before any grilling, check that the claim holds up. For a bug, reproduce it from the reporter's steps. For a PR, confirm the diff does what it claims: check it out, run the relevant tests or commands. Report what happened: confirmed (with code path), failed, or insufficient detail (a strong `needs-info` signal). A confirmed verification makes a much stronger agent brief.
75
75
 
76
- 4. **Grill (if needed).** If the request needs fleshing out, call the Skill tool twice, for "grilling" and "domain-modeling", and grill it into shape a round of questions at a time, sharpening domain terms and updating `CONTEXT.md`/ADRs inline as decisions land.
76
+ 4. **Grill (if needed).** If the request needs fleshing out, call the Skill tool twice, for "grill-me" and "domain-modeling", and grill it into shape a round of questions at a time, sharpening domain terms and updating `CONTEXT.md`/ADRs inline as decisions land.
77
77
 
78
78
  5. **Apply the outcome:**
79
79
  - `ready-for-agent`: post an agent brief comment ([AGENT-BRIEF.md](AGENT-BRIEF.md)).
@@ -107,12 +107,13 @@ jobs:
107
107
  fi
108
108
 
109
109
  # Priority-driven dual execution filter on direct pull_request triggers:
110
- # If PR is labeled priority/P2 or priority/P3, skip immediate cloud review to let local agent daemons review it
110
+ # Strict allowlist: only trigger immediate cloud review if explicitly labeled priority/P0 or priority/P1.
111
+ # Untagged PRs and lower-priority PRs (P2/P3) skip immediate cloud review and are handled by local peer-review daemons
111
112
  # (cloud scheduled watchdog sweep will catch up if unreviewed > 48h).
112
113
  if [ "${{ github.event_name }}" = "pull_request" ]; then
113
114
  labels=$(echo "$pr" | jq -r '.labels[].name' 2>/dev/null || echo "")
114
- if echo "$labels" | grep -qE '^priority/P[23]$'; then
115
- echo "PR is labeled as lower priority (P2/P3) — skipping immediate cloud review for local peer-review daemon."
115
+ if ! echo "$labels" | grep -qE '^priority/P[01]$'; then
116
+ echo "PR is not explicitly labeled priority/P0 or priority/P1 — skipping immediate cloud review for local peer-review daemon."
116
117
  echo "skip=true" >> "$GITHUB_OUTPUT"
117
118
  exit 0
118
119
  fi