codex-orchestrator 0.1.23 → 0.1.24
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 +8 -0
- package/README.md +14 -3
- package/dist/src/runner/review-gate-policy.d.ts.map +1 -1
- package/dist/src/runner/review-gate-policy.js +14 -2
- package/dist/src/runner/review-gate-policy.js.map +1 -1
- package/package.json +1 -1
- package/prompts/workflows/scoped-implementation.md +9 -3
package/CHANGELOG.md
CHANGED
|
@@ -6,6 +6,14 @@ The format is based on Keep a Changelog, and this project follows SemVer.
|
|
|
6
6
|
|
|
7
7
|
## [Unreleased]
|
|
8
8
|
|
|
9
|
+
## [0.1.24] - 2026-05-14
|
|
10
|
+
|
|
11
|
+
### Changed
|
|
12
|
+
- Mobile UI verification guidance now gives an explicit adb/emulator preflight
|
|
13
|
+
for Android mobile app work, prefers a connected device before emulator
|
|
14
|
+
fallback, tells agents to try loading Test Android Apps when unavailable, and
|
|
15
|
+
treats missing plugin/device targets as a warning rather than a blocker.
|
|
16
|
+
|
|
9
17
|
## [0.1.23] - 2026-05-14
|
|
10
18
|
|
|
11
19
|
### Added
|
package/README.md
CHANGED
|
@@ -303,9 +303,20 @@ Runtime and test paths are configurable through
|
|
|
303
303
|
`reviewGates.quality.testChangedPathGlobs`.
|
|
304
304
|
|
|
305
305
|
For UI or frontend issues, visual proof is runner-owned via a configurable
|
|
306
|
-
command (typically Playwright). Screenshot artifacts should be
|
|
307
|
-
`.codex-orchestrator/proofs/issue-<number>/`; the runner includes
|
|
308
|
-
and issue review report.
|
|
306
|
+
command (typically Playwright for browser/web UI). Screenshot artifacts should be
|
|
307
|
+
saved under `.codex-orchestrator/proofs/issue-<number>/`; the runner includes
|
|
308
|
+
them in the PR and issue review report.
|
|
309
|
+
|
|
310
|
+
For Android mobile app UI work, the implementation prompt directs Codex to use
|
|
311
|
+
device-backed proof instead of Playwright: run `adb devices -l`, prefer a
|
|
312
|
+
connected non-emulator device serial, and run `export ANDROID_SERIAL=<serial>`.
|
|
313
|
+
Otherwise run `emulator -list-avds`, start an AVD in a separate shell with
|
|
314
|
+
`emulator -avd <avd-name>`, and wait with `adb wait-for-device`. If Test Android
|
|
315
|
+
Apps skills are unavailable, the agent should try to enable or load that plugin
|
|
316
|
+
through the available Codex plugin/tool discovery mechanism. If the plugin cannot
|
|
317
|
+
be enabled, or no usable adb target is available, the agent should report the
|
|
318
|
+
mobile proof as a warning/skipped check with the concrete reason rather than
|
|
319
|
+
treating it as a blocker by itself.
|
|
309
320
|
|
|
310
321
|
Configure a runner-owned command:
|
|
311
322
|
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"review-gate-policy.d.ts","sourceRoot":"","sources":["../../../src/runner/review-gate-policy.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,qBAAqB,CAAC;AACnE,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,qBAAqB,CAAC;AAEvD,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,uBAAuB,CAAC;AAElE,wBAAgB,2BAA2B,CAAC,MAAM,EAAE,uBAAuB,EAAE,WAAW,EAAE,MAAM,GAAG,MAAM,EAAE,
|
|
1
|
+
{"version":3,"file":"review-gate-policy.d.ts","sourceRoot":"","sources":["../../../src/runner/review-gate-policy.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,qBAAqB,CAAC;AACnE,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,qBAAqB,CAAC;AAEvD,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,uBAAuB,CAAC;AAElE,wBAAgB,2BAA2B,CAAC,MAAM,EAAE,uBAAuB,EAAE,WAAW,EAAE,MAAM,GAAG,MAAM,EAAE,CA8B1G;AAaD,wBAAgB,2BAA2B,CAAC,MAAM,EAAE,uBAAuB,GAAG,MAAM,EAAE,CAerF;AAED,wBAAgB,0BAA0B,CAAC,KAAK,EAAE;IAChD,MAAM,EAAE,uBAAuB,CAAC;IAChC,KAAK,EAAE,WAAW,CAAC;IACnB,YAAY,EAAE,MAAM,EAAE,CAAC;CACxB,GAAG,OAAO,CAaV;AAED,wBAAgB,mBAAmB,CAAC,UAAU,EAAE,oBAAoB,EAAE,EAAE,QAAQ,EAAE,MAAM,EAAE,GAAG,OAAO,CAInG;AAED,wBAAgB,sBAAsB,CAAC,UAAU,EAAE,oBAAoB,EAAE,EAAE,QAAQ,EAAE,MAAM,EAAE,GAAG,OAAO,CAgBtG;AAED,wBAAgB,cAAc,CAAC,IAAI,EAAE,oBAAoB,GAAG,MAAM,CAEjE;AAED,wBAAgB,wBAAwB,CAAC,IAAI,EAAE,oBAAoB,GAAG,OAAO,CAE5E;AAED,wBAAgB,wBAAwB,CAAC,IAAI,EAAE,oBAAoB,GAAG,OAAO,CAE5E;AAED,wBAAgB,YAAY,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAEnE;AAED,wBAAgB,uBAAuB,CAAC,MAAM,EAAE,uBAAuB,GAAG;IACxE,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,WAAW,EAAE,MAAM,CAAC;IACpB,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,sBAAsB,EAAE,MAAM,CAAC;CAChC,CASA"}
|
|
@@ -4,7 +4,8 @@ export function buildVisualProofPromptLines(config, issueNumber) {
|
|
|
4
4
|
if (!command) {
|
|
5
5
|
return [
|
|
6
6
|
`For visual/UI work, if you can produce screenshot proof, save it under ${config.reviewGates.visualProof.artifactDir}/issue-${issueNumber}/ and include it as screenshot artifacts.`,
|
|
7
|
-
'
|
|
7
|
+
'For browser/web UI work, prefer a Playwright-based proof script when available; do not rely on in-session browser plugins for proof.',
|
|
8
|
+
...androidMobileProofPromptLines(),
|
|
8
9
|
'If screenshot proof is not possible in this environment, state that explicitly in skippedChecks along with the concrete reason (missing dependencies, missing credentials, dev server cannot start, etc.).',
|
|
9
10
|
];
|
|
10
11
|
}
|
|
@@ -16,7 +17,8 @@ export function buildVisualProofPromptLines(config, issueNumber) {
|
|
|
16
17
|
`For visual/UI work, prepare screenshot proof files under ${config.reviewGates.visualProof.artifactDir}/issue-${issueNumber}/ and include them as screenshot artifacts when you create them.`,
|
|
17
18
|
`After your run, the runner will execute this visual proof command outside the child Codex sandbox: ${command}.`,
|
|
18
19
|
'Prepare any project files this command needs, but do not execute this runner-owned command yourself or start long-lived browser/dev-server proof loops from child Codex.',
|
|
19
|
-
'
|
|
20
|
+
'For browser/web UI work, prefer Playwright-based screenshot proof via the runner-owned command; do not rely on in-session browser plugins for proof.',
|
|
21
|
+
...androidMobileProofPromptLines(),
|
|
20
22
|
'When this runner-owned proof command can validate the visual behavior, treat it as the primary visual proof path.',
|
|
21
23
|
'Do not report browser tool unavailability as a skipped check or residual risk when the runner-owned proof command is prepared.',
|
|
22
24
|
'For UI layout fixes, a focused visual proof script with concrete assertions can be the TDD evidence when regular unit tests cannot observe the layout.',
|
|
@@ -26,6 +28,16 @@ export function buildVisualProofPromptLines(config, issueNumber) {
|
|
|
26
28
|
'If required login environment variables are missing, the visual proof script must fail with a short clear error.',
|
|
27
29
|
];
|
|
28
30
|
}
|
|
31
|
+
function androidMobileProofPromptLines() {
|
|
32
|
+
return [
|
|
33
|
+
'For Android mobile app UI work, use device-backed proof instead of browser automation: run `adb devices -l`, prefer a connected non-emulator device serial, and run `export ANDROID_SERIAL=<serial>` for the proof session.',
|
|
34
|
+
'If no physical Android device is listed by adb, run `emulator -list-avds`, start an available AVD in a separate shell with `emulator -avd <avd-name>`, then wait for it with `adb wait-for-device` before testing.',
|
|
35
|
+
'If Test Android Apps skills are unavailable, try to enable or load that plugin through the available Codex plugin/tool discovery mechanism before falling back.',
|
|
36
|
+
'After selecting the adb target, use Test Android Apps skills for app launch, navigation, screenshots, logs, and performance evidence.',
|
|
37
|
+
'Do not use Playwright as the primary proof path for Android mobile app verification.',
|
|
38
|
+
'If Test Android Apps cannot be enabled, or no usable Android device or emulator is available, report the mobile proof as a warning/skipped check with the concrete plugin or adb/emulator reason; this is not a publication blocker by itself.',
|
|
39
|
+
];
|
|
40
|
+
}
|
|
29
41
|
export function buildQualityGatePromptLines(config) {
|
|
30
42
|
const quality = config.reviewGates.quality;
|
|
31
43
|
if (!quality.enabled) {
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"review-gate-policy.js","sourceRoot":"","sources":["../../../src/runner/review-gate-policy.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAGhD,MAAM,UAAU,2BAA2B,CAAC,MAA+B,EAAE,WAAmB;IAC9F,MAAM,OAAO,GAAG,MAAM,CAAC,WAAW,CAAC,WAAW,CAAC,uBAAuB,EAAE,IAAI,EAAE,CAAC;IAC/E,IAAI,CAAC,OAAO,EAAE,CAAC;QACb,OAAO;YACL,0EAA0E,MAAM,CAAC,WAAW,CAAC,WAAW,CAAC,WAAW,UAAU,WAAW,2CAA2C;YACpL,
|
|
1
|
+
{"version":3,"file":"review-gate-policy.js","sourceRoot":"","sources":["../../../src/runner/review-gate-policy.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAGhD,MAAM,UAAU,2BAA2B,CAAC,MAA+B,EAAE,WAAmB;IAC9F,MAAM,OAAO,GAAG,MAAM,CAAC,WAAW,CAAC,WAAW,CAAC,uBAAuB,EAAE,IAAI,EAAE,CAAC;IAC/E,IAAI,CAAC,OAAO,EAAE,CAAC;QACb,OAAO;YACL,0EAA0E,MAAM,CAAC,WAAW,CAAC,WAAW,CAAC,WAAW,UAAU,WAAW,2CAA2C;YACpL,sIAAsI;YACtI,GAAG,6BAA6B,EAAE;YAClC,4MAA4M;SAC7M,CAAC;IACJ,CAAC;IAED,MAAM,QAAQ,GAAG,MAAM,CAAC,WAAW,CAAC,WAAW,CAAC,cAAc,IAAI,EAAE,CAAC;IACrE,MAAM,YAAY,GAAG,QAAQ,CAAC,MAAM,GAAG,CAAC;QACtC,CAAC,CAAC,qGAAqG,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,iFAAiF;QAC3M,CAAC,CAAC,2LAA2L,CAAC;IAEhM,OAAO;QACL,4DAA4D,MAAM,CAAC,WAAW,CAAC,WAAW,CAAC,WAAW,UAAU,WAAW,kEAAkE;QAC7L,sGAAsG,OAAO,GAAG;QAChH,0KAA0K;QAC1K,sJAAsJ;QACtJ,GAAG,6BAA6B,EAAE;QAClC,mHAAmH;QACnH,gIAAgI;QAChI,wJAAwJ;QACxJ,oHAAoH;QACpH,uIAAuI;QACvI,YAAY;QACZ,kHAAkH;KACnH,CAAC;AACJ,CAAC;AAED,SAAS,6BAA6B;IACpC,OAAO;QACL,6NAA6N;QAC7N,oNAAoN;QACpN,iKAAiK;QACjK,uIAAuI;QACvI,sFAAsF;QACtF,gPAAgP;KACjP,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,2BAA2B,CAAC,MAA+B;IACzE,MAAM,OAAO,GAAG,MAAM,CAAC,WAAW,CAAC,OAAO,CAAC;IAC3C,IAAI,CAAC,OAAO,CAAC,OAAO,EAAE,CAAC;QACrB,OAAO,EAAE,CAAC;IACZ,CAAC;IAED,OAAO;QACL,0BAA0B;QAC1B,uNAAuN;QACvN,6IAA6I;QAC7I,gDAAgD,OAAO,CAAC,uBAAuB,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG;QAC7F,6CAA6C,OAAO,CAAC,oBAAoB,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG;QACvF,kGAAkG;QAClG,kFAAkF,OAAO,CAAC,aAAa,CAAC,oBAAoB,4DAA4D;KACzL,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,0BAA0B,CAAC,KAI1C;IACC,MAAM,WAAW,GAAG,KAAK,CAAC,MAAM,CAAC,WAAW,CAAC,WAAW,CAAC;IACzD,IAAI,CAAC,WAAW,CAAC,OAAO,EAAE,CAAC;QACzB,OAAO,KAAK,CAAC;IACf,CAAC;IAED,MAAM,SAAS,GAAG,GAAG,KAAK,CAAC,KAAK,CAAC,KAAK,KAAK,KAAK,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC;IAC9D,MAAM,gBAAgB,GAAG,WAAW,CAAC,iBAAiB,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC,CAAC;IAC3G,MAAM,cAAc,GAAG,KAAK,CAAC,YAAY,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CACxD,WAAW,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,WAAW,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC,CAC3E,CAAC;IAEF,OAAO,gBAAgB,IAAI,cAAc,CAAC,MAAM,GAAG,CAAC,CAAC;AACvD,CAAC;AAED,MAAM,UAAU,mBAAmB,CAAC,UAAkC,EAAE,QAAkB;IACxF,OAAO,UAAU,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAC9B,IAAI,CAAC,MAAM,KAAK,QAAQ,IAAI,QAAQ,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,CACpG,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,sBAAsB,CAAC,UAAkC,EAAE,QAAkB;IAC3F,IAAI,UAAU,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAC3B,IAAI,CAAC,MAAM,KAAK,QAAQ;WACnB,CAAC,sBAAsB,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC;WAC7C,QAAQ,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,CAC7E,EAAE,CAAC;QACF,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,WAAW,GAAG,UAAU;SAC3B,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,MAAM,KAAK,QAAQ,CAAC;SAC1C,GAAG,CAAC,cAAc,CAAC;SACnB,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,sBAAsB,CAAC,IAAI,CAAC,CAAC,CAAC;IACnD,MAAM,cAAc,GAAG,WAAW,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;IAC3D,MAAM,gBAAgB,GAAG,WAAW,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC;IAC/D,OAAO,cAAc,IAAI,gBAAgB,CAAC;AAC5C,CAAC;AAED,MAAM,UAAU,cAAc,CAAC,IAA0B;IACvD,OAAO,GAAG,IAAI,CAAC,OAAO,KAAK,IAAI,CAAC,OAAO,EAAE,CAAC;AAC5C,CAAC;AAED,MAAM,UAAU,wBAAwB,CAAC,IAA0B;IACjE,OAAO,oCAAoC,CAAC,IAAI,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC;AACzE,CAAC;AAED,MAAM,UAAU,wBAAwB,CAAC,IAA0B;IACjE,OAAO,uBAAuB,CAAC,IAAI,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC;AAC5D,CAAC;AAED,MAAM,UAAU,YAAY,CAAC,OAAe,EAAE,IAAY;IACxD,OAAO,IAAI,MAAM,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC9C,CAAC;AAED,MAAM,UAAU,uBAAuB,CAAC,MAA+B;IAOrE,MAAM,WAAW,GAAG,MAAM,CAAC,WAAW,CAAC,WAAW,CAAC;IACnD,OAAO;QACL,eAAe,EAAE,WAAW,CAAC,uBAAuB,EAAE,IAAI,EAAE,IAAI,SAAS;QACzE,WAAW,EAAE,WAAW,CAAC,WAAW;QACpC,cAAc,EAAE,WAAW,CAAC,cAAc,IAAI,EAAE;QAChD,SAAS,EAAE,WAAW,CAAC,eAAe;QACtC,sBAAsB,EAAE,WAAW,CAAC,sBAAsB;KAC3D,CAAC;AACJ,CAAC;AAED,SAAS,iBAAiB,CAAC,IAAY;IACrC,OAAO,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,2DAA2D,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC5G,CAAC;AAED,SAAS,mBAAmB,CAAC,IAAY;IACvC,OAAO,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC;WAC5B,mFAAmF,CAAC,IAAI,CAAC,IAAI,CAAC;WAC9F,wFAAwF,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC3G,CAAC;AAED,SAAS,sBAAsB,CAAC,IAAY;IAC1C,OAAO,2EAA2E,CAAC,IAAI,CAAC,IAAI,CAAC;WACxF,2EAA2E,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC9F,CAAC"}
|
package/package.json
CHANGED
|
@@ -11,6 +11,12 @@ Run code-review before completion for runtime changes and report the result.
|
|
|
11
11
|
Report validation, skipped checks, and risks.
|
|
12
12
|
For UI or visual changes, follow the orchestration prompt's visual proof
|
|
13
13
|
contract. If a runner-owned visual proof command is configured, prepare its
|
|
14
|
-
script/artifacts (prefer Playwright) but let the runner
|
|
15
|
-
|
|
16
|
-
|
|
14
|
+
script/artifacts (prefer Playwright for browser/web UI) but let the runner
|
|
15
|
+
execute it. For Android mobile app UI work, use device-backed proof instead of
|
|
16
|
+
Playwright: run `adb devices -l`, prefer a connected non-emulator device serial,
|
|
17
|
+
and run `export ANDROID_SERIAL=<serial>`. Otherwise run `emulator -list-avds`,
|
|
18
|
+
start an AVD in a separate shell with `emulator -avd <avd-name>`, and wait with
|
|
19
|
+
`adb wait-for-device`. If Test Android Apps skills are unavailable, try to enable
|
|
20
|
+
or load that plugin through the available Codex plugin/tool discovery mechanism.
|
|
21
|
+
If the plugin cannot be enabled, or no usable device or emulator is available,
|
|
22
|
+
report that as a warning/skipped check with the concrete reason and proceed.
|