Invoke when the user asks to review, check, audit, or look over Qt6 C++ code — or suggest before committing. Runs deterministic linting (60+ rules) then six parallel deep- analysis agents covering model contracts, ownership, threading, API correctness, error handling, and performance. Reports only h

Qt Code Review Read only review of Qt6 C++ code: a deterministic Python linter first, then agent driven deep analysis across six rule domains, then one consolidated report. When to use this skill Review related requests: "review", "check", "audit", "look over", "code review", "sanity check" Suggest it before committing code Arguments /qt cpp review — universal Qt6 C++ rules /qt cpp review framework — also apply Qt framework/module rules (binary compatibility, exports, d pointers, qdoc, QML versioning) Framework mode Enable when $ARGUMENTS contains "framework". Otherwise, if the lint run or a quick Grep shows two or more of these signals, suggest framework mode to the user but do not enable it yourself: QT BEGIN NAMESPACE , any Q EXPORT macro, <QtModule/private/ p.h includes, Q DECLARE PRIVATE / Q D() / Q Q() , qt internal add module in CMakeLists.txt, sync.profile or .qmake.conf at the repo root. When enabled: pass framework to the linter, and name references/qt framework checklist.md in the prompts of the agents covering A2 or A4 (they apply it as mapped at the end of agent rules.md ). In diff scope, also check the commit message against the FW CMT rules yourself; load only that section with sed n '/^ Commit Message/,$p' references/qt framework checklist.md . Scope detection Diff scope — "this commit", "these changes", "the diff", "what I changed", "staged changes", "before I commit". The range is HEAD (working tree plus index), cached for staged changes only, or HEAD~1..HEAD for "this commit". Do not load the diff itself: the linter's diff mode (Phase 1) lists the changed files with their changed line counts and reports only findings on changed lines. Codebase scope — "review the codebase", "audit src/", or an explicit path with no commit language. Glob .cpp , .h , .hpp under the path. Token budget rules These rules exist to keep the review cheap. Follow them. 1. Do not load the reference files into the main context. Agents read references/agent rules.md themselves; you pass its absolute path and the section IDs. Only an inline run (Phase 2) reads it in the main context. Lint messages are self describing; the checklists are for humans. 2. Pass lint results in summary form (grouped by rule, paths relative to a root printed once), never the raw per line listing. 3. Point agents at content, never paste it. Every token you write into a prompt is output on the orchestrating model, the most expensive tokens in the review. Prompts carry paths, commands and section IDs; the agents load the rules, the files and the diff. In diff scope an agent gets the git diff U50 <range <files command instead of the hunks, and reads other files only to trace a specific symbol. 4. Scale the agent count to the scope (see Phase 2). Launch no agents for a small scope and split a large one into shards of about equal line count. 5. Use the dedicated review agent where the harness has one. If the cpp review agent definition (listed as qt development skills:cpp review agent when installed with the plugin, plain cpp review agent when copied by hand) is available, launch the agents with it (read only, tools limited to Read, Grep, Glob and read only Bash); otherwise use the harness's general purpose agent. The definition lives in the plugin's agents/ directory, outside this skill, so it is missing when the skill was copied on its own or the harness does not install agents/ . Then add one line to each prompt: read only (Bash only for cat , grep , awk , git diff , git show ); take file:line from the printed numbers; in diff scope do not re read the files; Grep or Read other files only to trace a symbol; prefer few confirmed findings over many guesses. Execution order Never skip a phase. Phase 1: Deterministic linting Python 3.7+, no dependencies. If Python is unavailable, warn and continue (in diff scope, take the file list from git diff stat <range ). The linter is authoritative; do not second guess its output. Rule families: INC, DEP, PAT, MDL, ERR, LCY, API, ENM, TMO, CND, VAL, HDR, TRN, VAR (60+ checks). Exit code 1 simply means findings exist; 2 means a usage or git error. Phase 2: Deep analysis (inline, or one agent per shard) The six sections of references/agent rules.md are the analysis. Agents read it themselves; read it yourself only when running inline: Section Mission A1 Model Contracts A2 Ownership & Lifecycle A3 Thread Safety A4 API, Naming & C++ Correctness A5 Error Handling & Validation A6 Performance & Code Quality Choose the fan out from the scope size. Count the lines of the files in scope (the changed lines in diff scope), taken from the lint summary or wc l . A block is about 2,500 lines; the line count alone decides the split, never the number of files. Scope Agents Up to 2,500 lines 0 (inline) More than 2,500 lines one per shard, ceil(lines / 2500) shards Inline. A small scope does not repay the cost of agents, which each rebuild the context and roughly double the review's token cost for the same findings. Do the analysis yourself: read references/agent rules.md and the files (or the diff) with the one Bash call shown under the agent prompt below, and apply A1 A6 in a single pass under the same rules, confidence thresholds and output format. Do not re report lint rows, and do not run inline above the limit because the code looks simple. Report Agents: 0 (inline) . Shards. Cut the files, in directory order, into n = ceil(lines / 2500) shards of about lines / n lines each; how many files a shard holds does not matter. Keep a directory in one shard where it fits, so that a class's header and source, and the models and views that reference each other, land together; split a larger directory by file. A file larger than a shard gets a shard to itself. Each shard agent applies all six sections to its shard only and may Grep the whole scope to trace a symbol. Run up to six shard agents at a time and the rest in a second wave. Cross shard checks (ownership, model contracts, duplicated logic) are then best effort; say so in the report header. Launch the agents in parallel, named after their shard (e.g. "Review shard 2/6: src/models"). Keep each prompt short; it contains only: 1. The absolute path of references/agent rules.md and the section IDs to apply (A1 A6), plus the absolute path of references/qt framework checklist.md for A2 and A4 agents in framework mode 2. The root directory from the lint summary, the files in scope relative to it and, in diff scope, the git diff U50 <range <files command for them 3. The lint summary rows for that agent's own files only (a shard agent never needs the other shards' rows), with the instruction "do not re report these lines" 4. The instruction to load everything with one Bash call before analysing, and to return findings in the format at the top of the rules file. Each tool call re sends the agent's whole context, so one batched read instead of one Read per file is the single largest saving on large scopes. Both forms print per file line numbers for the file:line locations ( cat n on several files numbers them as one run, and git diff has no line numbers): Agents are read only and must never edit files. In tools without agents, run each mission as whatever prompt chain or subprocess the tool supports; the contract is the same. Phase 3: Consolidation Merge lint and agent findings. Deduplicate on file+line+issue. When duplicates disagree on confidence, keep the block with the highest score and its trace; never average scores. Write the report in the format below. Confidence scoring Confidence Meaning Action 80 100 Rule violation confirmed with a symbol trace Finding 60 79 Likely, not fully verifiable (e.g. noexcept needing whole program analysis, dead code with possible external callers) Investigation target (max 10) <60 Suspicion only Suppress Output format Use exactly this structure. Lint findings are grouped by rule with one mitigation each; only deep analysis findings get a full block. References references/agent rules.md — agent missions A1 A6 and the agent output format (read by the agents, not the orchestrator) references/qt review checklist.md — universal rule IDs and rationale (human reference, not loaded during a review) references/qt framework checklist.md — framework rules (read by A2 and A4 agents in framework mode) references/qt deprecated classes.md — background for the DEP lint rules (human reference) references/lint scripts/qt review lint.py — single pass linter, summary for grouped output, diff=<range for changed lines only, framework for module rules

qt-cpp-review