Skip to content

Remove the review command - #155

Merged
hbrooks merged 1 commit into
mainfrom
claude/remove-code-reviews-rjgh9c
Sep 26, 2026
Merged

hbrooks merged 1 commit into
mainfrom
claude/remove-code-reviews-rjgh9c

Conversation

@hbrooks

@hbrooks hbrooks commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Code review has been removed from the platform (ellipsis-dev/ellipsis#6712 dropped the /v1/reviews routes), so the CLI drops everything that concerned it:

  • ellipsis review (start, get, list, init) and its hidden aliases reviews, code-review, cr, plus its test file.
  • The review types in src/lib/types.ts and the unused ListReviewsQuery.
  • The review entry in the Sessions help group, the README usage lines, and the code review sections of skills/ellipsis/SKILL.md.
  • The typo-suggestion tests now use variable as their sample command instead of review.

Kept on purpose: ellipsis analytics review / analytics reviewer, which report the organization's own GitHub pull request review activity (/v1/analytics/reviews, /v1/analytics/metrics), not Ellipsis reviews.

Rebased onto #154 (session turn model). That change added followConversation in src/commands/session.ts for the review command alone, so it goes too; the stream helper it wrapped still follows a whole conversation when called without a turn id.

The CLI pins @ellipsis-dev/sdk 0.31.0, which still carries the reviews client; this change only removes code, so it typechecks against that pin. The next paired SDK release drops the reviews surface and the version bump picks it up.

Checks run locally: bun install --frozen-lockfile, bun run check:versions, bun run typecheck, bun run test (217 passing), bun run build, and a --compile of the binary whose --help lists no review command.

🤖 Generated with Claude Code

https://claude.ai/code/session_016JiiZiz8r2z32pVp5at2qx


Generated by Claude Code

Code review is being removed from the platform, so the CLI loses the
`review` command (start, get, list, init) and its hidden aliases, the review
types, the README usage lines, and the code review sections of the ellipsis
skill. The GitHub review analytics under `analytics` are about the
organization's own pull request activity and stay.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016JiiZiz8r2z32pVp5at2qx
@hbrooks
hbrooks force-pushed the claude/remove-code-reviews-rjgh9c branch from 4d27d93 to af8b2d0 Compare September 26, 2026 17:56
@hbrooks
hbrooks merged commit 26f4388 into main Sep 26, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants