菜单
Review ScyllaDB/Cassandra C driver API usage in this extension — error handling, session/cluster lifecycle, type mapping, and async patterns. Use when touching any code that calls cass_* functions.
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
基于 SOC 职业分类
Comprehensive Zend Engine API reference for writing PHP 8.x extensions in C23. Covers module lifecycle, parameter parsing, zvals, zend_string, HashTable, object handlers, memory management, globals, INI settings, arginfo/stub workflow, and PHP 8 breakage traps derived from real codebase audit findings. Use whenever implementing or reviewing PHP extension C code.
Set up the ScyllaDB PHP driver for local development. Walks through installing all native dependencies, building PHP, compiling the extension, and verifying everything works. Use when onboarding or setting up a new machine.
Review CMakeLists.txt and cmake/ module files for modern CMake correctness, anti-patterns, portability bugs, and find-module issues. Use after modifying any CMake file.
Scaffold a new PHP extension module following the src/Cluster canonical pattern. Creates stub, handlers, implementation skeleton, CMakeLists.txt, and wires it into the build.
Review a PHP extension module (C23 style) for correctness, memory safety, and Zend API usage. Use after implementing or modifying any src/ module.
| name | scylladb-review |
| description | Review ScyllaDB/Cassandra C driver API usage in this extension — error handling, session/cluster lifecycle, type mapping, and async patterns. Use when touching any code that calls cass_* functions. |
| allowed-tools | Read Grep Glob |
Review the code at $ARGUMENTS (or recently changed files if no argument) for correct use of the ScyllaDB/Cassandra C driver API.
For each finding: file:line — issue — fix.
cass_* call that returns CassError must be checked — never silently droppedASSERT_SUCCESS(rc) for calls where failure is a programming error (wrong type, invalid arg)if (rc != CASS_OK) + php_error_docref(NULL, E_WARNING, "%s", cass_error_desc(rc)) for recoverable errors (network, timeout)CASS_ERROR_LIB_NOT_IMPLEMENTED must be handled specially — some features are optional in older driver versions (see Builder.cpp withHostnameResolution as reference)cass_error_desc() result after the CassFuture or object it came from is freedCassFuture * must be freed with cass_future_free() — check all exit paths including early returnscass_future_error_code() checked before accessing resultcass_future_get_result() only called when cass_future_error_code() == CASS_OKCassResult * from cass_future_get_result() freed with cass_result_free() — never leakedcass_cluster_new() always paired with cass_cluster_free() (or stored in persistent list)cass_session_new() always paired with cass_session_free()EG(persistent_list) with ZVAL_NEW_PERSISTENT_RES — not plain pointerscass_cluster_set_* call requires updating the hash key format in Builder::build()cass_session_connect) always freed even on errorcass_string_t / const char * returned by driver functions are not owned by caller — do not free themzend_string with zend_string_init(ptr, len, 0) if the value outlives the driver objectZSTR_VAL() / ZSTR_LEN() used when passing zend_string * to cass_* functions expecting const char *cass_value_get_* called only after cass_value_type() confirms the correct typeCASS_VALUE_TYPE_UNKNOWN and CASS_VALUE_TYPE_CUSTOM handled — don't assume a known typeCassCollection * freed with cass_collection_free() after cass_statement_bind_collection() (binding does not transfer ownership)CassTuple * / CassUserType * freed with their respective _free() functionscass_statement_bind_*_by_idx) preferred over name-based for performancecass_statement_free() called after cass_session_execute() — the future does not own the statementCassPrepared * freed with cass_prepared_free() when no longer neededCassIterator * always freed with cass_iterator_free() on all pathscass_iterator_next() checked before calling cass_iterator_get_*() — never read past endcass_result_first_row() for single-row results rather than creating an iteratorcass_* functions are thread-safe for distinct objects — document any shared state explicitlyCassCluster is NOT reused across sessions simultaneously — one cluster per session or use persistent list with separate session per requestPHP_DRIVER_G(persistent_clusters) counter incremented/decremented atomically with the persistent listcass_cluster_set_num_threads_io to a value lower than the ScyllaDB shard count on the target node — default of 1 is intentionally conservativecass_cluster_set_use_schema(false) disables schema metadata — appropriate for high-throughput write paths; flag it if enabled on a write-heavy builderReport all findings. If a section is clean, say "✓ clean". End with: PASS, NEEDS FIXES (n issues), or CRITICAL (leak/crash risk).