claude-dev-env 1.94.0 → 2.0.0

package/skills/codex-review/scripts/codex_down_classifier.py

@@ -0,0 +1,98 @@
+"""Classify Codex run streams into down-detail and gate outcome classes.
+
+::
+
+    classification = classify_codex_run(exit_code=1, stream_text=stderr_text)
+    classification.detail_class   # usage_limit | auth_failure | ...
+    classification.outcome_class  # codex_down | completed
+"""
+
+from __future__ import annotations
+
+from dataclasses import dataclass
+
+from codex_review_scripts_constants.classifier_constants import (
+    ALL_AUTH_FAILURE_MARKERS,
+    ALL_CONFIG_ERROR_MARKERS,
+    ALL_MODEL_ERROR_MARKERS,
+    ALL_USAGE_LIMIT_MARKERS,
+    FAILURE_CLASS_AUTH_FAILURE,
+    FAILURE_CLASS_CONFIG_ERROR,
+    FAILURE_CLASS_MODEL_ERROR,
+    FAILURE_CLASS_UNKNOWN,
+    FAILURE_CLASS_USAGE_LIMIT,
+    SUCCESS_EXIT_CODE,
+)
+from codex_review_scripts_constants.run_constants import (
+    OUTCOME_CLASS_CODEX_DOWN,
+    OUTCOME_CLASS_COMPLETED,
+)
+
+
+@dataclass(frozen=True)
+class CodexRunClassification:
+    """Fine-grained failure class plus the gate-level outcome class.
+
+    ::
+
+        CodexRunClassification(
+            detail_class="config_error",
+            outcome_class="codex_down",
+        )
+
+    Attributes:
+        detail_class: ``completed`` or a failure class name.
+        outcome_class: ``completed`` or ``codex_down`` for the conditional gate.
+    """
+
+    detail_class: str
+    outcome_class: str
+
+
+def _stream_contains_marker(stream_text: str, all_markers: tuple[str, ...]) -> bool:
+    lowered_stream = stream_text.lower()
+    return any(each_marker in lowered_stream for each_marker in all_markers)
+
+
+def _classify_failure_detail(stream_text: str) -> str:
+    if _stream_contains_marker(stream_text, ALL_CONFIG_ERROR_MARKERS):
+        return FAILURE_CLASS_CONFIG_ERROR
+    if _stream_contains_marker(stream_text, ALL_MODEL_ERROR_MARKERS):
+        return FAILURE_CLASS_MODEL_ERROR
+    if _stream_contains_marker(stream_text, ALL_USAGE_LIMIT_MARKERS):
+        return FAILURE_CLASS_USAGE_LIMIT
+    if _stream_contains_marker(stream_text, ALL_AUTH_FAILURE_MARKERS):
+        return FAILURE_CLASS_AUTH_FAILURE
+    return FAILURE_CLASS_UNKNOWN
+
+
+def classify_codex_run(*, exit_code: int, stream_text: str) -> CodexRunClassification:
+    """Map exit code and stream text to detail and gate outcome classes.
+
+    ::
+
+        classify_codex_run(exit_code=0, stream_text=success_jsonl)
+        # ok: detail_class=completed, outcome_class=completed
+        classify_codex_run(exit_code=1, stream_text="rate limit")
+        # ok: detail_class=usage_limit, outcome_class=codex_down
+
+    Exit zero is a completed review. Every other class — including
+    ``unknown`` for nonzero exits and unparseable text — maps to
+    ``codex_down`` so the conditional gate fails closed.
+
+    Args:
+        exit_code: Process exit code from the Codex review invocation.
+        stream_text: Captured stdout JSONL and/or stderr text.
+
+    Returns:
+        Detail class and gate outcome class for the run.
+    """
+    if exit_code == SUCCESS_EXIT_CODE:
+        return CodexRunClassification(
+            detail_class=OUTCOME_CLASS_COMPLETED,
+            outcome_class=OUTCOME_CLASS_COMPLETED,
+        )
+    return CodexRunClassification(
+        detail_class=_classify_failure_detail(stream_text),
+        outcome_class=OUTCOME_CLASS_CODEX_DOWN,
+    )

package/skills/codex-review/scripts/codex_review_scripts_constants/CLAUDE.md

@@ -0,0 +1,18 @@
+# codex-review/scripts/codex_review_scripts_constants
+
+Python package of named constants imported by codex-review scripts. Importing from this package keeps magic values out of script bodies.
+
+## Modules
+
+| File | Constants for |
+|---|---|
+| `__init__.py` | Package marker and package docstring. |
+| `run_constants.py` | Binary name, flag strings, custom-instructions prompt, version-probe pattern, timeout default, decode/timeout exit sentinels, flag_token_pattern suffix (whole-token shape match), JSONL keys, and capture outcome class labels for `run_codex_review.py`. |
+| `findings_constants.py` | Finding field keys, fenced-JSON and freeform patterns for `parse_codex_findings.py`. |
+| `classifier_constants.py` | Failure class names and stream markers for `codex_down_classifier.py`. |
+| `run_constants.py` | Binary name, flag strings, custom-instructions prompt, version-probe pattern, timeout default, decode/timeout exit sentinels, flag_token_pattern suffix (whole-token shape match), JSONL keys, JSONL capture filename and newline mode, and capture outcome class labels for `run_codex_review.py`. |
+| `codex_usage_probe_constants.py` | The weekly-usage probe CLI: gate threshold, report JSON keys, app-server JSON-RPC surface, rate-limit field keys, weekly-window sizing, text-status parse patterns, source labels, and exit codes. |
+
+## Convention
+
+Scripts import from this package at module scope. No constant is defined inline in a script body — the hook enforces this at write time.

package/skills/codex-review/scripts/codex_review_scripts_constants/__init__.py

@@ -0,0 +1 @@
+"""Named constants for the codex-review skill scripts."""

package/skills/codex-review/scripts/codex_review_scripts_constants/classifier_constants.py

@@ -0,0 +1,35 @@
+"""Named constants for the Codex down-classifier."""
+
+from __future__ import annotations
+
+FAILURE_CLASS_USAGE_LIMIT = "usage_limit"
+FAILURE_CLASS_AUTH_FAILURE = "auth_failure"
+FAILURE_CLASS_CONFIG_ERROR = "config_error"
+FAILURE_CLASS_MODEL_ERROR = "model_error"
+FAILURE_CLASS_UNKNOWN = "unknown"
+ALL_CONFIG_ERROR_MARKERS = (
+    "error loading config.toml",
+    "unknown variant",
+)
+ALL_MODEL_ERROR_MARKERS = (
+    "invalid_request_error",
+    "model is not supported",
+    "requires a newer version of codex",
+)
+ALL_USAGE_LIMIT_MARKERS = (
+    "rate limit",
+    "too many requests",
+    "http 429",
+    " (429)",
+    "credits exhausted",
+    "out of credits",
+    "api quota",
+    "usage quota",
+)
+ALL_AUTH_FAILURE_MARKERS = (
+    "unauthorized",
+    "authentication failed",
+    "login required",
+    "not authenticated",
+)
+SUCCESS_EXIT_CODE = 0

package/skills/codex-review/scripts/codex_review_scripts_constants/codex_usage_probe_constants.py

@@ -0,0 +1,86 @@
+"""Constants for the Codex weekly-usage probe CLI.
+
+Groups: gate threshold, probe report JSON keys, app-server JSON-RPC surface,
+rate-limit snapshot field keys, weekly-window sizing, parse patterns for
+text status output, source labels, and exit codes.
+"""
+
+from __future__ import annotations
+
+WEEKLY_USAGE_GATE_THRESHOLD_PERCENT = 10
+
+USAGE_REPORT_KEY_PERCENT_LEFT = "percent_left"
+USAGE_REPORT_KEY_WINDOW_RESET = "window_reset"
+USAGE_REPORT_KEY_SOURCE = "source"
+
+CODEX_BINARY_NAME = "codex"
+ALL_APP_SERVER_COMMAND_PARTS = ("app-server", "--listen", "stdio://")
+APP_SERVER_TIMEOUT_SECONDS = 30
+READER_THREAD_JOIN_TIMEOUT_SECONDS = 2
+PROCESS_TREE_WAIT_TIMEOUT_SECONDS = 5
+WINDOWS_OS_NAME = "nt"
+WINDOWS_COMMAND_SHELL = "cmd"
+WINDOWS_COMMAND_SHELL_RUN_FLAG = "/c"
+ALL_WINDOWS_SCRIPT_SUFFIXES = (".cmd", ".bat")
+WINDOWS_TASKKILL_COMMAND = "taskkill"
+WINDOWS_TASKKILL_FORCE_FLAG = "/F"
+WINDOWS_TASKKILL_TREE_FLAG = "/T"
+WINDOWS_TASKKILL_PID_FLAG = "/PID"
+
+JSONRPC_VERSION = "2.0"
+JSONRPC_KEY_ID = "id"
+JSONRPC_KEY_METHOD = "method"
+JSONRPC_KEY_PARAMS = "params"
+JSONRPC_KEY_REPLY_BODY = "result"
+JSONRPC_KEY_ERROR = "error"
+JSONRPC_KEY_VERSION = "jsonrpc"
+
+INITIALIZE_REQUEST_ID = 1
+RATE_LIMITS_REQUEST_ID = 2
+METHOD_INITIALIZE = "initialize"
+METHOD_INITIALIZED = "initialized"
+METHOD_RATE_LIMITS_READ = "account/rateLimits/read"
+
+CLIENT_INFO_KEY = "clientInfo"
+CLIENT_INFO_NAME_KEY = "name"
+CLIENT_INFO_VERSION_KEY = "version"
+CLIENT_INFO_NAME = "codex-usage-probe"
+CLIENT_INFO_VERSION = "1.0.0"
+CAPABILITIES_KEY = "capabilities"
+EXPERIMENTAL_API_KEY = "experimentalApi"
+
+RATE_LIMITS_KEY = "rateLimits"
+PRIMARY_WINDOW_KEY = "primary"
+SECONDARY_WINDOW_KEY = "secondary"
+USED_PERCENT_KEY = "usedPercent"
+RESETS_AT_KEY = "resetsAt"
+WINDOW_DURATION_MINS_KEY = "windowDurationMins"
+
+WEEKLY_WINDOW_DURATION_MINUTES = 10080
+PERCENT_FULL = 100
+PERCENT_EMPTY = 0
+
+SOURCE_APP_SERVER_RATE_LIMITS = "codex app-server account/rateLimits/read"
+SOURCE_TEXT_STATUS = "text-status"
+SOURCE_NO_USAGE_SURFACE = "no-usage-surface"
+
+TEXT_WEEKLY_PERCENT_LEFT_PATTERN = (
+    r"(?i)weekly(?:\s+(?:limit|usage|cap))?"
+    r"[^\n%]{0,40}?"
+    r"(?P<percent>\d{1,3}(?:\.\d+)?)\s*%\s*(?:left|remaining)"
+)
+TEXT_WEEKLY_USED_PERCENT_PATTERN = (
+    r"(?i)weekly(?:\s+(?:limit|usage|cap))?"
+    r"[^\n%]{0,40}?"
+    r"(?P<percent>\d{1,3}(?:\.\d+)?)\s*%\s*(?:used|consumed)"
+)
+TEXT_WINDOW_RESET_PATTERN = (
+    r"(?i)(?:weekly\s+)?(?:window\s+)?resets?(?:\s+at)?"
+    r"[:\s]+(?P<reset>[^\s)\].,;]+)"
+)
+
+EXIT_CODE_SUCCESS = 0
+EXIT_CODE_CRASH = 1
+
+UTF8_ENCODING = "utf-8"
+NEWLINE = "\n"

package/skills/codex-review/scripts/codex_review_scripts_constants/findings_constants.py

@@ -0,0 +1,18 @@
+"""Named constants for the Codex findings parser."""
+
+from __future__ import annotations
+
+FINDING_KEY_TITLE = "title"
+FINDING_KEY_PRIORITY = "priority"
+FINDING_KEY_FILE = "file"
+FINDING_KEY_LINE_RANGE = "line_range"
+FINDING_KEY_BODY = "body"
+FENCED_JSON_BLOCK_PATTERN = r"```(?:json)?\s*\n(.*?)\n```"
+FREEFORM_FINDING_LINE_PATTERN = (
+    r"^-\s*\[(?P<priority>P\d+)\]\s+"
+    r"(?P<title>.+?)\s+[—–-]\s+"
+    r"(?P<file_path>.+?):(?P<line_range>\d+-\d+)\s*$"
+)
+FREEFORM_BULLET_PREFIX = "- ["
+EMPTY_STRING = ""
+NEWLINE = "\n"

package/skills/codex-review/scripts/codex_review_scripts_constants/run_constants.py

@@ -0,0 +1,45 @@
+"""Named constants for the codex-review headless wrapper."""
+
+from __future__ import annotations
+
+CODEX_BINARY_NAME = "codex"
+EXEC_SUBCOMMAND = "exec"
+REVIEW_SUBCOMMAND = "review"
+HELP_FLAG = "--help"
+VERSION_FLAG = "--version"
+JSON_FLAG = "--json"
+BASE_TARGET_FLAG = "--base"
+UNCOMMITTED_TARGET_FLAG = "--uncommitted"
+COMMIT_TARGET_FLAG = "--commit"
+MODEL_FLAG = "-m"
+CODEX_MODEL_PIN = ""
+CODEX_HOME_ENV_VAR = "CODEX_HOME"
+DEFAULT_TIMEOUT_SECONDS = 600
+JSONL_CAPTURE_FILENAME = "codex-review.jsonl"
+JSONL_CAPTURE_NEWLINE = ""
+UTF8_ENCODING = "utf-8"
+VERSION_PROBE_PATTERN = r"codex(?:-cli)?\s+(\d+\.\d+\.\d+)"
+OUTCOME_CLASS_CODEX_DOWN = "codex_down"
+OUTCOME_CLASS_COMPLETED = "completed"
+MISSING_BINARY_EXIT_CODE = 127
+TIMEOUT_EXIT_CODE = 124
+SUBPROCESS_DECODE_EXIT_CODE = 70
+SHAPE_FLAG_TOKEN_TAIL_PATTERN = r"(?![\w-])"
+JSONL_EVENT_TYPE_KEY = "type"
+JSONL_ENTRY_KEY = "item"
+JSONL_ENTRY_COMPLETED_TYPE = "item.completed"
+JSONL_AGENT_MESSAGE_TYPE = "agent_message"
+JSONL_AGENT_MESSAGE_TEXT_KEY = "text"
+CUSTOM_INSTRUCTIONS_PROMPT = (
+    "Write a one-line review summary, then the heading Review comment:, then "
+    "zero or more finding bullets. Each bullet must use the form "
+    "- [P1] <title> — <path>:<start>-<end> "
+    "followed by an explanation paragraph. Use the same [P#] priority tags. "
+    "When there are no findings, omit finding bullets after the summary."
+)
+ALL_SHAPE_PROBE_REQUIRED_FLAGS = (
+    BASE_TARGET_FLAG,
+    UNCOMMITTED_TARGET_FLAG,
+    COMMIT_TARGET_FLAG,
+    JSON_FLAG,
+)