open-code-review/scripts
thxCode 0a74720560
feat(action): first-class review controls (effort, max_tokens_budget, llm_reasoning_effort) and live progress (#1154)
* feat(action): expose effort and max_tokens_budget as inputs

- Add validated, case-insensitive effort and base-10 max_tokens_budget
  workflow inputs, forwarded to ocr review as --effort and
  --max-tokens-budget; empty values omit the flags so CLI defaults apply
- Reject an explicit effort input on ocr older than v1.10.0, where the
  flag first shipped, instead of dying on an unknown flag
- Join both axes to the checkpoint config fingerprint

Closes #1147

Signed-off-by: thxCode <thxcode0824@gmail.com>

* feat(action): opt-in live review progress via stream_progress

- add stream_progress input (true|false, case-insensitive, default
  'false'), validated and normalized in the Validate inputs step
- the default keeps the original behavior: --audience agent in the
  review args, stderr captured to /tmp/ocr-stderr.log with no live tee
- stream_progress=true drops --audience agent so human-audience routing
  sends [ocr] progress lines to stderr while the result JSON stays on
  stdout; a FIFO feeds a background tee so progress streams live into
  the workflow log and is still captured to /tmp/ocr-stderr.log for
  artifacts and the posting step
- the tee runs as a real background job and is awaited before the log
  file is read, so the capture is fully flushed; OCR_EXIT_CODE is
  unaffected in either mode

Task 2 of action-review-controls.

Signed-off-by: thxCode <thxcode0824@gmail.com>

* feat(action): first-class llm_reasoning_effort input

- Merge llm_reasoning_effort into the effective llm.extra_body via node
  in the Configure step, riding the existing extra_body merge so any
  published CLI supports it; an explicit reasoning_effort key in
  llm_extra_body wins
- Reject llm_reasoning_effort on the anthropic protocol, and reject a
  malformed or non-object llm_extra_body with an actionable ::error::
- Fingerprint the normalized effort, max_tokens_budget, and
  llm_reasoning_effort values instead of the raw inputs, so equivalent
  spellings keep the checkpoint
- Keep the last 20000 characters of stderr in the unparseable-result
  summary so streamed progress cannot push the error report past
  GitHub's comment limit
- Document the new action inputs on the CI/CD docs page (en/zh/ja/ko/ru)

Closes #1149

Signed-off-by: thxCode <thxcode0824@gmail.com>

---------

Signed-off-by: thxCode <thxcode0824@gmail.com>
2026-09-07 14:10:56 +08:00
..
github-actions feat(action): first-class review controls (effort, max_tokens_budget, llm_reasoning_effort) and live progress (#1154) 2026-09-07 14:10:56 +08:00
publish chore: add SPDX license headers and automated verification (#740) 2026-08-05 21:26:27 +08:00
add-license.sh fix(scripts): stop license checks failing on valid headers (#971) 2026-08-17 17:25:35 +08:00
install.js chore: add SPDX license headers and automated verification (#740) 2026-08-05 21:26:27 +08:00
platform.js chore: add SPDX license headers and automated verification (#740) 2026-08-05 21:26:27 +08:00
update.js fix(updater): discard stale version hints (#720) 2026-08-17 22:35:46 +08:00
verify-action-pins.sh fix(action): pin nested action references to full commit SHAs (#836) 2026-08-11 15:36:13 +08:00
verify-english-only.go chore(ci): fail CI when unapproved non-English text appears in source files (#876) 2026-08-13 14:43:55 +08:00
verify-license.sh fix(scripts): stop license checks failing on valid headers (#971) 2026-08-17 17:25:35 +08:00
version.js fix(updater): discard stale version hints (#720) 2026-08-17 22:35:46 +08:00
version.test.js fix(updater): discard stale version hints (#720) 2026-08-17 22:35:46 +08:00