ts-reviewer 3.7.0 → 3.7.1

package/README.md

@@ -4,6 +4,8 @@ An AI skill for deep code review and auto-fix of pure TypeScript codebases. Find
 
 Built for one fixed stack — **TypeScript 5.9.x, ES2024, Node 24** — without any framework-specific checks (no React, Vue, Angular, etc.). Anything below the stack is a finding, anything above it is never recommended.
 
+Supported AI agents: **Claude Code** and **Codex** only. Other agents are not supported by the installer or tested with the skill.
+
 ## What It Does
 
 Four modes, one skill:
@@ -17,6 +19,8 @@ Four modes, one skill:
 
 ## What's New
 
+**3.7.1 — 1 message on Codex.** Codex has no question menu outside Plan mode, so the start questions come as 1 chat message with a 1-line answer instead of 4 round trips.
+
 **3.7.0 — 3 questions before a scan.** A scan you start asks, in this order: which domains (2 pages, nothing pre-ticked — page 1 the 4 cheap groups, page 2 Security and Architecture), whether to run the skill lint, and which model runs the passes (every run, your last answer first). A flag answers its own question, a resumed scan asks nothing, and `--defaults` asks nothing at all — for an agent that starts the scan for you. The default set drops Security: 8 domains. `--pick` is gone: the menu now shows without it.
 
 **3.6.1 — the agent menu waits.** On Windows, after you picked the install scope, the agent menu took no keys: it confirmed both agents unasked or quit without installing. It now waits for your choice.
@@ -50,6 +54,8 @@ The review has ten domains, each with its own detailed checklist. A scan asks wh
 
 ## Installation
 
+The installer supports 2 AI agents: Claude Code and Codex. Antigravity was dropped in 3.6.0.
+
 ### Install with npx
 
 ```bash
@@ -133,7 +139,7 @@ The passes run in groups: Type Safety with Boundary Validation, Async Patterns w
 
 A scan or auto run you start asks 3 questions before it reads the project, in this order:
 
-1. **Domains** — a multi-select in 2 pages, with nothing ticked in advance (a numbered list such as `1,3` on a host with no multi-select):
+1. **Domains** — a multi-select in 2 pages, with nothing ticked in advance:
    - page 1: Type Safety + Boundary Validation, Async Patterns + Error Handling, Config + Dependency Hygiene, Modernization + Code Quality — a usual scan ticks all 4;
    - page 2: Security, Architecture — the expensive ones.
 2. **Skill lint** — whether to download and run the pinned ESLint config. Asked only when a picked domain has lint-owned lines; Knip and dependency-cruiser are approved in the same question when Architecture is picked and missing locally.
@@ -141,6 +147,8 @@ A scan or auto run you start asks 3 questions before it reads the project, in th
 
 A resumed scan asks none of them: the queue holds the answers. Leaving a question unanswered takes the default, except the skill lint, which an unanswered approval declines.
 
+**On Codex** there is no question menu outside Plan mode, so the scan asks all of them in 1 chat message: the 6 groups as 1 numbered list, then the lint and the model. Answer in 1 line, e.g. `1-4,6; lint yes; sonnet high`. `codex exec` has nobody to answer: pass `--defaults`.
+
 #### Domain flags
 
 A flag answers its question, so the scan does not ask it:

package/assets/skills/ts-reviewer/SKILL.md

@@ -114,7 +114,23 @@ start_questions:
 - a question with no operator answer takes the `--defaults` answer, except the skill lint: an unanswered approval is a decline
 - the domain menu has 2 pages, each 1 multi-select question holding the `pass_groups` rows of its page in row order, each option naming its domains
 - tick no option in advance, and say in the page 1 question that a usual scan ticks every option of page 1
-- ask a page as a numbered list read from a reply such as `1,3` on a host with no multi-select
+- ask every pending start question in 1 message on a host with no question tool, such as Codex in default mode: 1 chat round trip per question ends the agent's turn 4 times
+- write that message in the shape below, with the `pass_groups` rows of page 1 and then page 2, and leave out a line whose question a flag answers
+- offer no ready-made answer such as "defaults" in that message: the operator ticks the groups, as on a menu
+```text
+Before the scan, answer in 1 line, e.g. `1-4; lint yes; sonnet high`.
+Domains (a usual scan picks 1-4; 5 and 6 are expensive):
+  1. Type Safety, Boundary Validation
+  2. Async Patterns, Error Handling
+  3. Config, Dependency Hygiene
+  4. Modernization, Code Quality
+  5. Security
+  6. Architecture
+Skill lint: download and run the pinned ESLint config? yes / no
+Pass model: <the scout file's model and effort> / the main agent's model / another model and effort
+```
+- read every answer from 1 reply such as `1-4,6; lint yes; sonnet high`, and ask again only the part the reply leaves unreadable
+- ask a page as a numbered list in the question text on a host whose question tool takes 1 choice per question
 - ask the domain menu again when the operator ticks no option on either page: an empty scan is never the intent
 - ask the skill lint only when an active domain owns a line carrying "lint-owned by", and approve Knip and dependency-cruiser in the same question when Architecture is active and missing locally
 - ask the pass model as `pass_agent` states

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ts-reviewer",
-  "version": "3.7.0",
+  "version": "3.7.1",
   "description": "Install the TypeScript Code Reviewer skill for Claude Code or Codex, globally or per project",
   "license": "MIT",
   "type": "module",