$npx -y skills add warpdotdev/oz-for-oss --skill review-pr-localRepo-specific review guidance for oz-for-oss. Only the categories declared overridable by the core review-pr skill may be specialized here.
| 1 | # Repo-specific review guidance for `oz-for-oss` |
| 2 | |
| 3 | This file is a companion to the core `review-pr` skill. It does not |
| 4 | redefine the review output schema, severity labels, safety rules, or |
| 5 | evidence rules. It only specializes the override categories the core |
| 6 | skill marks as overridable. |
| 7 | |
| 8 | ## User-facing strings |
| 9 | |
| 10 | - Flag interpolated text that would read unnaturally at runtime (e.g. wrong casing after a sentence fragment like "The triage concluded that {summary}"). |
| 11 | - Link text should be descriptive (e.g. "triage session on Warp"), not bare URLs or generic "click here" labels. |
| 12 | - Verify that terminology is consistent across related messages in the same PR. |
| 13 | |
| 14 | ## Graceful degradation |
| 15 | |
| 16 | - When code renders optional dynamic data (URLs, session links, metadata), flag cases where a missing value would produce empty or broken output. The fix is usually to omit the element entirely and show a short fallback message. |
| 17 | - Prefer starting with generic, user-safe error messages over exposing internal details. |
| 18 | |
| 19 | ## Debugging and observability |
| 20 | |
| 21 | - Do not suggest removing session links, workflow URLs, or other debugging context from error paths. These are valuable for post-incident investigation even when the operation failed. |