shuke987 opened a new pull request, #8:
URL: https://github.com/apache/doris-skills/pull/8

   ## Summary
   
   - add `DORIS_REVIEW_EFFORT` as a tool-neutral effort source while preserving 
the existing `CLAUDE_EFFORT` fallback
   - require effort values to use a known runtime spelling or `unknown`
   - document safe effort reporting for Codex without guessing from global 
`config.toml`
   - shorten the skill description so it passes the skill validator
   
   ## Why
   
   The PASS comment is consumed as an auditable review receipt. Codex sessions 
do not expose
   `CLAUDE_EFFORT`, so the existing default records `unknown` even when a 
review ran at `xhigh`.
   Reading global Codex configuration is not reliable because a task-level 
override can differ.
   
   ## Validation
   
   - `quick_validate.py skills/doris-repo-review`
   - `bash -n skills/doris-repo-review/scripts/post-pass-comment.sh`
   - smoke-tested `DORIS_REVIEW_EFFORT` precedence, `CLAUDE_EFFORT` fallback, 
and invalid effort rejection
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to