open-code-review

mirror of https://github.com/alibaba/open-code-review.git synced 2026-08-20 06:04:11 +00:00

History

xujiejie f6e5e98564 refactor(skill): simplify review flow, use native severity (#1002 ) * refactor(skill): move prerequisites check to troubleshooting section - Remove upfront `which ocr` / `ocr llm test` from main workflow - Keep inline installation fallback hint in Step 2 - Add Troubleshooting section with install and LLM config guidance - Update Gotchas entry to reference Troubleshooting instead of requiring pre-run check * refactor(skill): use native OCR severity/category instead of manual classification - Step 3: remove manual High/Medium/Low classification; use OCR output severity/category directly - Step 4: reference severity field instead of custom priority levels - Output Format: add category/severity to field list; group template by severity - Remove duplicated Priority classification definition - Retain mispositioned comments handling and thinking field * refactor(skill): apply prerequisites/severity refactor to canonical skill Mirrors the plugin skill refactor: - Move prerequisites check to Troubleshooting section - Use native OCR severity/category instead of manual classification - Retain mispositioned comments handling and thinking field - Retain truncation guidance added in `4f7d78f` * docs(skill): sync truncation guidance from canonical to plugin skill The plugin skill copy missed the output-truncation guidance added to the canonical skill in `4f7d78f` (#809). Both files now diverge only by the plugin's self-descriptive mirror notice, as designed. * fix(skill): route command failures to troubleshooting section Step 2 had no failure directive, so the agent did not consult the Troubleshooting section when `ocr review` failed (e.g. LLM connection error). Add an explicit pointer so the agent looks up the matching fix before retrying. * docs(skill): address PR review feedback - Lead with the interactive `ocr config provider` wizard for LLM config (agent cannot obtain credentials); keep manual `ocr config set` as alternative; drop env-var option. - Fix the no-issues line to reflect that low-severity findings are discarded ("no critical, high, or medium severity issues remain after filtering"). - Apply to both canonical and plugin skill copies.	2026-08-19 20:19:02 +08:00
..
open-code-review	refactor(skill): simplify review flow, use native severity (#1002 )	2026-08-19 20:19:02 +08:00
open-code-review-delegate	feat: add QCA delegation integration (#762 )	2026-08-07 13:58:27 +08:00

refactor(skill): simplify review flow, use native severity (#1002 )

* refactor(skill): move prerequisites check to troubleshooting section

- Remove upfront `which ocr` / `ocr llm test` from main workflow
- Keep inline installation fallback hint in Step 2
- Add Troubleshooting section with install and LLM config guidance
- Update Gotchas entry to reference Troubleshooting instead of requiring pre-run check

* refactor(skill): use native OCR severity/category instead of manual classification

- Step 3: remove manual High/Medium/Low classification; use OCR output severity/category directly
- Step 4: reference severity field instead of custom priority levels
- Output Format: add category/severity to field list; group template by severity
- Remove duplicated Priority classification definition
- Retain mispositioned comments handling and thinking field

* refactor(skill): apply prerequisites/severity refactor to canonical skill

Mirrors the plugin skill refactor:
- Move prerequisites check to Troubleshooting section
- Use native OCR severity/category instead of manual classification
- Retain mispositioned comments handling and thinking field
- Retain truncation guidance added in 4f7d78f

* docs(skill): sync truncation guidance from canonical to plugin skill

The plugin skill copy missed the output-truncation guidance added to the
canonical skill in 4f7d78f (#809). Both files now diverge only by the
plugin's self-descriptive mirror notice, as designed.

* fix(skill): route command failures to troubleshooting section

Step 2 had no failure directive, so the agent did not consult the
Troubleshooting section when `ocr review` failed (e.g. LLM connection
error). Add an explicit pointer so the agent looks up the matching fix
before retrying.

* docs(skill): address PR review feedback

- Lead with the interactive `ocr config provider` wizard for LLM config
  (agent cannot obtain credentials); keep manual `ocr config set` as
  alternative; drop env-var option.
- Fix the no-issues line to reflect that low-severity findings are
  discarded ("no critical, high, or medium severity issues remain after
  filtering").
- Apply to both canonical and plugin skill copies.

2026-08-19 20:19:02 +08:00

open-code-review refactor(skill): simplify review flow, use native severity (#1002 ) 2026-08-19 20:19:02 +08:00

open-code-review-delegate feat: add QCA delegation integration (#762 ) 2026-08-07 13:58:27 +08:00