[only21mil]

Problem

In a numbered list in the composer, Tab on item 2 nests it under item 1 but leaves the items after it numbered 3. and 4.. Enter and Shift+Tab already renumber in the same situation, so only Tab leaves the gap.

Shift+Tab has the reverse problem. Lifting the first nested item out of 1. b / 2. c leaves the remaining child numbered 2. c, so the sent Markdown starts that nested list at 2. Shift+Enter on an empty nested item in the plain-text composer lifts it the same way and has the same gap.

Change

renumberFollowingItems in composer-list-continuation.ts now renumbers a run of lists in one pass, each list from its own starting number, and folds the result into the edit. Every key goes through it:

• Tab renumbers the items after the nested one, counting on from the sibling it now follows. It does this only when both items are numbered with the same . or ).
• Shift+Tab, and Shift+Enter on an empty nested item, restart the siblings left behind at 1 when they become the lifted item's children, then count on in the parent list as before. Siblings that stay short of the lifted item's content column (for example under a bullet) keep their numbers.

Web and desktop share this composer. Mobile has no Tab list nesting.

Scope and approval

A small fix for an obvious bug. It extends the renumbering #14332 added for Enter to the nesting and lifting paths from #17138 and #27016, in the same file and test table. No new setting.

Verification

• vp test run src/composer-list-continuation.test.ts (from apps/web) on main (d1909d6) with the new rows: 3 failed, 32 passed (Tab, Shift+Tab, and Shift+Enter on an empty nested item). With the fix: 35 passed, including a row that keeps siblings under a bullet numbered (- a / 1. b / 2. c).
• vp run check:changed passes.
• Desktop app (vp run dev:desktop) on macOS, Apple silicon, default rich-text composer, no provider set up. Driven with key and mouse events sent through its remote debugging port; lists were typed with Shift+Enter for new items. main (d1909d6) first, then this branch. The web client matched.

Tab on "Add the new shortcuts". On main, the items after it stay 3. and 4.:

On this branch they read 2. and 3.:

Shift+Tab on "Add the new shortcuts", nested under item 1 with "Fix the dark theme" after it. On main, its new child still reads 2.:

On this branch the child restarts at 1. and the parent list counts on to 3.:

Not checked: Shift+Enter in the plain-text composer is covered by the test only. With rich text on, Enter in a list item goes through the editor's own list commands, which this PR does not touch.

Generated with Claude Opus 5.5 in T3 Code Nightly

[macroscopeapp]

Approvability

Verdict: Approved at 2632c69

Macroscope's review found this PR approvable — Nesting or lifting numbered composer items now updates surrounding list numbers consistently. The change is limited to draft editing, with no changes to submission, product defaults, or external systems.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

Ordered-list renumbering now applies to Enter continuation, Tab nesting, and Shift+Tab outdent edits. The shared logic handles multiple ordered runs and preserves delimiter matching, indentation boundaries, inline-token handling, and leading-zero formatting.

Changes: Ordered-list renumbering

Layer / File(s)	Summary
Ordered-run renumbering helpers apps/web/src/composer-list-continuation.ts	Adds shared helpers to recognize matching ordered markers and renumber numbered runs. Lift handling can renumber remaining ordered children and continue the parent list.
Renumbering after list edits apps/web/src/composer-list-continuation.ts, apps/web/src/composer-list-continuation.test.ts	Enter continuation, Tab nesting, and Shift+Tab outdent now use the shared renumbering logic. Tests check resulting numbering and cursor position.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Bug fix · Severity of issue fixed: Low

Possibly related PRs

• pingdotgg/t3code#17138: Adds numbered-list nesting and lifting in the same composer helper.
• pingdotgg/t3code#27016: Adds Shift+Tab outdent and renumbers matching ordered items.

Suggested labels: bugfix, size:M

Suggested reviewers: saphid, michal-billtech

Merge Risk: Minimal · up to 2632c

The change fixes numbered-list renumbering when nesting or lifting items in the composer. No actionable merge-blocking risk was identified.

Pre-merge checks: 3 · 1

Failed checks: 1 inconclusive

Check name	Status	Explanation	Resolution
Description check		The description includes the required Problem, Change, Scope and approval, and Verification sections, with specific test results and reported limitations. However, the supplied objectives conflict wit…	Reconcile the desktop verification details. Confirm whether the desktop app was launched and tested, then update the description or objectives so they report consistent results.

Passed checks: 3 passed

Check name	Status	Explanation
Title check		The title clearly summarizes the main change: renumbering numbered composer items when nesting or lifting them.
Linked Issues check		No active directly linked issue targets remain. Issue #14332 is closed and supplies historical context only, so no linked-issue coding requirements apply.
Out of Scope Changes check		The changes update numbered-list renumbering in composer-list-continuation.ts and add focused regression tests for Tab, Shift+Tab, and Shift+Enter. These changes support the PR’s stated list-nesting…

Full details: Out of Scope Changes check

Explanation

The changes update numbered-list renumbering in composer-list-continuation.ts and add focused regression tests for Tab, Shift+Tab, and Shift+Enter. These changes support the PR’s stated list-nesting and lifting fixes. The change summary shows no unrelated work.

Full details: Description check

Explanation

The description includes the required Problem, Change, Scope and approval, and Verification sections, with specific test results and reported limitations. However, the supplied objectives conflict with the description about whether the desktop app was launched and tested.

Resolution

Reconcile the desktop verification details. Confirm whether the desktop app was launched and tested, then update the description or objectives so they report consistent results.

Finishing Touches

Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Recent review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: main
• File: .coderabbit.config.ts
• Commit: e89abad
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: 3fe52c40-0f5e-4dae-904a-1163671df752

Commits

Reviewing files that changed from the base of the PR and between e89abad and 2632c69.

Files selected for processing: 2

• apps/web/src/composer-list-continuation.test.ts
• apps/web/src/composer-list-continuation.ts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 1 remain after this review.