Fix stale model picker shortcut labels by colonelpanic8 · Pull Request #4433 · pingdotgg/t3code

colonelpanic8 · 2026-07-24T07:08:18Z

What Changed

Include the positional shortcut-label map in the virtualized model list's invalidation state.
Include the other external row-render dependencies so cached model rows refresh when their presentation changes.
Add a focused regression test that verifies the list receives the current per-model shortcut mapping.

Why

LegendList caches rows by model key, but model-picker shortcuts are assigned by visible position. When models ahead of a cached row were inserted, removed, filtered, or reordered, the row could retain its previous label and display the same shortcut as another model.

Fixes#4432.

UI Changes

Behavior-only correction: after filtering the live model list to Opus models, the surviving rows refresh to unique shortcuts from Ctrl+1 through Ctrl+4. No layout or styling changes.

Checklist

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
vp fmt --check apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
vp lint apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
Verified the real picker in an isolated web environment by filtering/reordering the model list and confirming unique positional labels.
Focused web typecheck is currently blocked by unrelated existing dependency/type errors, including @effect/vitest exports and existing LegendList callback inference errors.

Note

Low Risk
Localized UI fix in the model picker virtualization layer with a regression test; no auth, data, or API changes.

Overview
Fixes stale positional shortcut labels (e.g. duplicate Ctrl+1) in the virtualized model picker when the visible list is filtered, reordered, or edited.

LegendList caches rows by model key; jump shortcuts are assigned by position, so cached rows kept old labels until something else invalidated them. ModelPickerContent now passes a memoized modelListExtraData object to extraData instead of only favoritesSet, including modelJumpLabelByKey plus active selection, favorites, lock state, disabled reasons, and favorite toggles so rows re-render when presentation changes.

Adds a focused Vitest test that asserts the list receives the current per-model shortcut map in extraData.

^{Reviewed by Cursor Bugbot for commit c7ac18d. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Fix stale shortcut labels in model picker rows by expanding `LegendList` cache key

ModelPickerContent.tsx passes only favoritesSet as extraData to LegendList, which caches rows by model key. When the list reorders (e.g. a model is added or removed), positional shortcut labels like Ctrl+1 become stale because rows are not invalidated.

Replaces the favoritesSetextraData prop with a useMemo-computed modelListExtraData object that includes activeInstanceId, activeModel, favoritesSet, getModelDisabledReason, isLocked, modelJumpLabelByKey, and toggleFavorite.
Any change to these values now triggers a full row re-render, keeping shortcut labels in sync with list position.
Adds a Vitest test suite in ModelPickerContent.test.tsx asserting the correct modelJumpLabelByKey map is passed through extraData.

^{Macroscope summarized c7ac18d.}

coderabbitai · 2026-07-24T07:08:27Z

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: da41e741-dc8b-441f-8430-06a31524d50e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

macroscopeapp · 2026-07-24T07:14:10Z

Approvability

Verdict: Approved

Straightforward bug fix that corrects stale shortcut label rendering in a virtualized list by including modelJumpLabelByKey in the cache invalidation dependencies. Small, self-contained change with clear intent and a test to verify the fix.

^{You can customize Macroscope's approvability policy. Learn more.}

…s when virtualized rows are reordered)

colonelpanic8 · 2026-07-27T02:00:12Z

Reproduced the stale shortcut-label behavior and verified the fix in #4433 at 2b107b9f8a50dd7999e94a55f61509bdc7a1ed57.

Using the same isolated web environment at a 390×844 viewport:

Before (unfixed checkout): filtering the live model list to opus leaves cached rows with their old full-list shortcuts (Ctrl+2 through Ctrl+6) instead of reassigning them from the filtered positions.
After (PR head): the same flow refreshes every row to unique positional shortcuts Ctrl+1 through Ctrl+5.

Before — stale cached labels

After — fixed positional labels

Focused regression test also passes:

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
Test Files 1 passed (1)
Tests 1 passed (1)

…s when virtualized rows are reordered)

…cuts when virtualized rows are reordered)

juliusmarminge · 2026-08-15T11:31:54Z

Closing as superseded by #5400, which has merged and covers this change on the current architecture. Thanks for the contribution.

colonelpanic8 · 2026-07-24T07:08:18Z

What Changed

Include the positional shortcut-label map in the virtualized model list's invalidation state.
Include the other external row-render dependencies so cached model rows refresh when their presentation changes.
Add a focused regression test that verifies the list receives the current per-model shortcut mapping.

Why

LegendList caches rows by model key, but model-picker shortcuts are assigned by visible position. When models ahead of a cached row were inserted, removed, filtered, or reordered, the row could retain its previous label and display the same shortcut as another model.

Fixes#4432.

UI Changes

Behavior-only correction: after filtering the live model list to Opus models, the surviving rows refresh to unique shortcuts from Ctrl+1 through Ctrl+4. No layout or styling changes.

Checklist

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
vp fmt --check apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
vp lint apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
Verified the real picker in an isolated web environment by filtering/reordering the model list and confirming unique positional labels.
Focused web typecheck is currently blocked by unrelated existing dependency/type errors, including @effect/vitest exports and existing LegendList callback inference errors.

Note

Low Risk
Localized UI fix in the model picker virtualization layer with a regression test; no auth, data, or API changes.

Overview
Fixes stale positional shortcut labels (e.g. duplicate Ctrl+1) in the virtualized model picker when the visible list is filtered, reordered, or edited.

LegendList caches rows by model key; jump shortcuts are assigned by position, so cached rows kept old labels until something else invalidated them. ModelPickerContent now passes a memoized modelListExtraData object to extraData instead of only favoritesSet, including modelJumpLabelByKey plus active selection, favorites, lock state, disabled reasons, and favorite toggles so rows re-render when presentation changes.

Adds a focused Vitest test that asserts the list receives the current per-model shortcut map in extraData.

^{Reviewed by Cursor Bugbot for commit c7ac18d. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Fix stale shortcut labels in model picker rows by expanding `LegendList` cache key

ModelPickerContent.tsx passes only favoritesSet as extraData to LegendList, which caches rows by model key. When the list reorders (e.g. a model is added or removed), positional shortcut labels like Ctrl+1 become stale because rows are not invalidated.

Replaces the favoritesSetextraData prop with a useMemo-computed modelListExtraData object that includes activeInstanceId, activeModel, favoritesSet, getModelDisabledReason, isLocked, modelJumpLabelByKey, and toggleFavorite.
Any change to these values now triggers a full row re-render, keeping shortcut labels in sync with list position.
Adds a Vitest test suite in ModelPickerContent.test.tsx asserting the correct modelJumpLabelByKey map is passed through extraData.

^{Macroscope summarized c7ac18d.}

coderabbitai · 2026-07-24T07:08:27Z

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: da41e741-dc8b-441f-8430-06a31524d50e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

macroscopeapp · 2026-07-24T07:14:10Z

Approvability

Verdict: Approved

Straightforward bug fix that corrects stale shortcut label rendering in a virtualized list by including modelJumpLabelByKey in the cache invalidation dependencies. Small, self-contained change with clear intent and a test to verify the fix.

^{You can customize Macroscope's approvability policy. Learn more.}

…s when virtualized rows are reordered)

colonelpanic8 · 2026-07-27T02:00:12Z

Reproduced the stale shortcut-label behavior and verified the fix in #4433 at 2b107b9f8a50dd7999e94a55f61509bdc7a1ed57.

Using the same isolated web environment at a 390×844 viewport:

Before (unfixed checkout): filtering the live model list to opus leaves cached rows with their old full-list shortcuts (Ctrl+2 through Ctrl+6) instead of reassigning them from the filtered positions.
After (PR head): the same flow refreshes every row to unique positional shortcuts Ctrl+1 through Ctrl+5.

Before — stale cached labels

After — fixed positional labels

Focused regression test also passes:

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
Test Files 1 passed (1)
Tests 1 passed (1)

…s when virtualized rows are reordered)

…cuts when virtualized rows are reordered)

juliusmarminge · 2026-08-15T11:31:54Z

Closing as superseded by #5400, which has merged and covers this change on the current architecture. Thanks for the contribution.

colonelpanic8 · 2026-07-24T07:08:18Z

What Changed

Include the positional shortcut-label map in the virtualized model list's invalidation state.
Include the other external row-render dependencies so cached model rows refresh when their presentation changes.
Add a focused regression test that verifies the list receives the current per-model shortcut mapping.

Why

LegendList caches rows by model key, but model-picker shortcuts are assigned by visible position. When models ahead of a cached row were inserted, removed, filtered, or reordered, the row could retain its previous label and display the same shortcut as another model.

Fixes#4432.

UI Changes

Behavior-only correction: after filtering the live model list to Opus models, the surviving rows refresh to unique shortcuts from Ctrl+1 through Ctrl+4. No layout or styling changes.

Checklist

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
vp fmt --check apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
vp lint apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
Verified the real picker in an isolated web environment by filtering/reordering the model list and confirming unique positional labels.
Focused web typecheck is currently blocked by unrelated existing dependency/type errors, including @effect/vitest exports and existing LegendList callback inference errors.

Note

Low Risk
Localized UI fix in the model picker virtualization layer with a regression test; no auth, data, or API changes.

Overview
Fixes stale positional shortcut labels (e.g. duplicate Ctrl+1) in the virtualized model picker when the visible list is filtered, reordered, or edited.

LegendList caches rows by model key; jump shortcuts are assigned by position, so cached rows kept old labels until something else invalidated them. ModelPickerContent now passes a memoized modelListExtraData object to extraData instead of only favoritesSet, including modelJumpLabelByKey plus active selection, favorites, lock state, disabled reasons, and favorite toggles so rows re-render when presentation changes.

Adds a focused Vitest test that asserts the list receives the current per-model shortcut map in extraData.

^{Reviewed by Cursor Bugbot for commit c7ac18d. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Fix stale shortcut labels in model picker rows by expanding `LegendList` cache key

ModelPickerContent.tsx passes only favoritesSet as extraData to LegendList, which caches rows by model key. When the list reorders (e.g. a model is added or removed), positional shortcut labels like Ctrl+1 become stale because rows are not invalidated.

Replaces the favoritesSetextraData prop with a useMemo-computed modelListExtraData object that includes activeInstanceId, activeModel, favoritesSet, getModelDisabledReason, isLocked, modelJumpLabelByKey, and toggleFavorite.
Any change to these values now triggers a full row re-render, keeping shortcut labels in sync with list position.
Adds a Vitest test suite in ModelPickerContent.test.tsx asserting the correct modelJumpLabelByKey map is passed through extraData.

^{Macroscope summarized c7ac18d.}

coderabbitai · 2026-07-24T07:08:27Z

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: da41e741-dc8b-441f-8430-06a31524d50e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

macroscopeapp · 2026-07-24T07:14:10Z

Approvability

Verdict: Approved

Straightforward bug fix that corrects stale shortcut label rendering in a virtualized list by including modelJumpLabelByKey in the cache invalidation dependencies. Small, self-contained change with clear intent and a test to verify the fix.

^{You can customize Macroscope's approvability policy. Learn more.}

…s when virtualized rows are reordered)

colonelpanic8 · 2026-07-27T02:00:12Z

Reproduced the stale shortcut-label behavior and verified the fix in #4433 at 2b107b9f8a50dd7999e94a55f61509bdc7a1ed57.

Using the same isolated web environment at a 390×844 viewport:

Before (unfixed checkout): filtering the live model list to opus leaves cached rows with their old full-list shortcuts (Ctrl+2 through Ctrl+6) instead of reassigning them from the filtered positions.
After (PR head): the same flow refreshes every row to unique positional shortcuts Ctrl+1 through Ctrl+5.

Before — stale cached labels

After — fixed positional labels

Focused regression test also passes:

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
Test Files 1 passed (1)
Tests 1 passed (1)

…s when virtualized rows are reordered)

…cuts when virtualized rows are reordered)

juliusmarminge · 2026-08-15T11:31:54Z

Closing as superseded by #5400, which has merged and covers this change on the current architecture. Thanks for the contribution.

colonelpanic8 · 2026-07-24T07:08:18Z

What Changed

Include the positional shortcut-label map in the virtualized model list's invalidation state.
Include the other external row-render dependencies so cached model rows refresh when their presentation changes.
Add a focused regression test that verifies the list receives the current per-model shortcut mapping.

Why

LegendList caches rows by model key, but model-picker shortcuts are assigned by visible position. When models ahead of a cached row were inserted, removed, filtered, or reordered, the row could retain its previous label and display the same shortcut as another model.

Fixes#4432.

UI Changes

Behavior-only correction: after filtering the live model list to Opus models, the surviving rows refresh to unique shortcuts from Ctrl+1 through Ctrl+4. No layout or styling changes.

Checklist

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
vp fmt --check apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
vp lint apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
Verified the real picker in an isolated web environment by filtering/reordering the model list and confirming unique positional labels.
Focused web typecheck is currently blocked by unrelated existing dependency/type errors, including @effect/vitest exports and existing LegendList callback inference errors.

Note

Low Risk
Localized UI fix in the model picker virtualization layer with a regression test; no auth, data, or API changes.

Overview
Fixes stale positional shortcut labels (e.g. duplicate Ctrl+1) in the virtualized model picker when the visible list is filtered, reordered, or edited.

LegendList caches rows by model key; jump shortcuts are assigned by position, so cached rows kept old labels until something else invalidated them. ModelPickerContent now passes a memoized modelListExtraData object to extraData instead of only favoritesSet, including modelJumpLabelByKey plus active selection, favorites, lock state, disabled reasons, and favorite toggles so rows re-render when presentation changes.

Adds a focused Vitest test that asserts the list receives the current per-model shortcut map in extraData.

^{Reviewed by Cursor Bugbot for commit c7ac18d. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Fix stale shortcut labels in model picker rows by expanding `LegendList` cache key

ModelPickerContent.tsx passes only favoritesSet as extraData to LegendList, which caches rows by model key. When the list reorders (e.g. a model is added or removed), positional shortcut labels like Ctrl+1 become stale because rows are not invalidated.

Replaces the favoritesSetextraData prop with a useMemo-computed modelListExtraData object that includes activeInstanceId, activeModel, favoritesSet, getModelDisabledReason, isLocked, modelJumpLabelByKey, and toggleFavorite.
Any change to these values now triggers a full row re-render, keeping shortcut labels in sync with list position.
Adds a Vitest test suite in ModelPickerContent.test.tsx asserting the correct modelJumpLabelByKey map is passed through extraData.

^{Macroscope summarized c7ac18d.}

coderabbitai · 2026-07-24T07:08:27Z

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: da41e741-dc8b-441f-8430-06a31524d50e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

macroscopeapp · 2026-07-24T07:14:10Z

Approvability

Verdict: Approved

Straightforward bug fix that corrects stale shortcut label rendering in a virtualized list by including modelJumpLabelByKey in the cache invalidation dependencies. Small, self-contained change with clear intent and a test to verify the fix.

^{You can customize Macroscope's approvability policy. Learn more.}

…s when virtualized rows are reordered)

colonelpanic8 · 2026-07-27T02:00:12Z

Reproduced the stale shortcut-label behavior and verified the fix in #4433 at 2b107b9f8a50dd7999e94a55f61509bdc7a1ed57.

Using the same isolated web environment at a 390×844 viewport:

Before (unfixed checkout): filtering the live model list to opus leaves cached rows with their old full-list shortcuts (Ctrl+2 through Ctrl+6) instead of reassigning them from the filtered positions.
After (PR head): the same flow refreshes every row to unique positional shortcuts Ctrl+1 through Ctrl+5.

Before — stale cached labels

After — fixed positional labels

Focused regression test also passes:

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
Test Files 1 passed (1)
Tests 1 passed (1)

…s when virtualized rows are reordered)

…cuts when virtualized rows are reordered)

juliusmarminge · 2026-08-15T11:31:54Z

Closing as superseded by #5400, which has merged and covers this change on the current architecture. Thanks for the contribution.

colonelpanic8 · 2026-07-24T07:08:18Z

What Changed

Include the positional shortcut-label map in the virtualized model list's invalidation state.
Include the other external row-render dependencies so cached model rows refresh when their presentation changes.
Add a focused regression test that verifies the list receives the current per-model shortcut mapping.

Why

LegendList caches rows by model key, but model-picker shortcuts are assigned by visible position. When models ahead of a cached row were inserted, removed, filtered, or reordered, the row could retain its previous label and display the same shortcut as another model.

Fixes#4432.

UI Changes

Behavior-only correction: after filtering the live model list to Opus models, the surviving rows refresh to unique shortcuts from Ctrl+1 through Ctrl+4. No layout or styling changes.

Checklist

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
vp fmt --check apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
vp lint apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
Verified the real picker in an isolated web environment by filtering/reordering the model list and confirming unique positional labels.
Focused web typecheck is currently blocked by unrelated existing dependency/type errors, including @effect/vitest exports and existing LegendList callback inference errors.

Note

Low Risk
Localized UI fix in the model picker virtualization layer with a regression test; no auth, data, or API changes.

Overview
Fixes stale positional shortcut labels (e.g. duplicate Ctrl+1) in the virtualized model picker when the visible list is filtered, reordered, or edited.

LegendList caches rows by model key; jump shortcuts are assigned by position, so cached rows kept old labels until something else invalidated them. ModelPickerContent now passes a memoized modelListExtraData object to extraData instead of only favoritesSet, including modelJumpLabelByKey plus active selection, favorites, lock state, disabled reasons, and favorite toggles so rows re-render when presentation changes.

Adds a focused Vitest test that asserts the list receives the current per-model shortcut map in extraData.

^{Reviewed by Cursor Bugbot for commit c7ac18d. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Fix stale shortcut labels in model picker rows by expanding `LegendList` cache key

ModelPickerContent.tsx passes only favoritesSet as extraData to LegendList, which caches rows by model key. When the list reorders (e.g. a model is added or removed), positional shortcut labels like Ctrl+1 become stale because rows are not invalidated.

Replaces the favoritesSetextraData prop with a useMemo-computed modelListExtraData object that includes activeInstanceId, activeModel, favoritesSet, getModelDisabledReason, isLocked, modelJumpLabelByKey, and toggleFavorite.
Any change to these values now triggers a full row re-render, keeping shortcut labels in sync with list position.
Adds a Vitest test suite in ModelPickerContent.test.tsx asserting the correct modelJumpLabelByKey map is passed through extraData.

^{Macroscope summarized c7ac18d.}

coderabbitai · 2026-07-24T07:08:27Z

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: da41e741-dc8b-441f-8430-06a31524d50e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

macroscopeapp · 2026-07-24T07:14:10Z

Approvability

Verdict: Approved

Straightforward bug fix that corrects stale shortcut label rendering in a virtualized list by including modelJumpLabelByKey in the cache invalidation dependencies. Small, self-contained change with clear intent and a test to verify the fix.

^{You can customize Macroscope's approvability policy. Learn more.}

…s when virtualized rows are reordered)

colonelpanic8 · 2026-07-27T02:00:12Z

Reproduced the stale shortcut-label behavior and verified the fix in #4433 at 2b107b9f8a50dd7999e94a55f61509bdc7a1ed57.

Using the same isolated web environment at a 390×844 viewport:

Before (unfixed checkout): filtering the live model list to opus leaves cached rows with their old full-list shortcuts (Ctrl+2 through Ctrl+6) instead of reassigning them from the filtered positions.
After (PR head): the same flow refreshes every row to unique positional shortcuts Ctrl+1 through Ctrl+5.

Before — stale cached labels

After — fixed positional labels

Focused regression test also passes:

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
Test Files 1 passed (1)
Tests 1 passed (1)

…s when virtualized rows are reordered)

…cuts when virtualized rows are reordered)

juliusmarminge · 2026-08-15T11:31:54Z

Closing as superseded by #5400, which has merged and covers this change on the current architecture. Thanks for the contribution.

colonelpanic8 · 2026-07-24T07:08:18Z

What Changed

Include the positional shortcut-label map in the virtualized model list's invalidation state.
Include the other external row-render dependencies so cached model rows refresh when their presentation changes.
Add a focused regression test that verifies the list receives the current per-model shortcut mapping.

Why

LegendList caches rows by model key, but model-picker shortcuts are assigned by visible position. When models ahead of a cached row were inserted, removed, filtered, or reordered, the row could retain its previous label and display the same shortcut as another model.

Fixes#4432.

UI Changes

Behavior-only correction: after filtering the live model list to Opus models, the surviving rows refresh to unique shortcuts from Ctrl+1 through Ctrl+4. No layout or styling changes.

Checklist

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
vp fmt --check apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
vp lint apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
Verified the real picker in an isolated web environment by filtering/reordering the model list and confirming unique positional labels.
Focused web typecheck is currently blocked by unrelated existing dependency/type errors, including @effect/vitest exports and existing LegendList callback inference errors.

Note

Low Risk
Localized UI fix in the model picker virtualization layer with a regression test; no auth, data, or API changes.

Overview
Fixes stale positional shortcut labels (e.g. duplicate Ctrl+1) in the virtualized model picker when the visible list is filtered, reordered, or edited.

LegendList caches rows by model key; jump shortcuts are assigned by position, so cached rows kept old labels until something else invalidated them. ModelPickerContent now passes a memoized modelListExtraData object to extraData instead of only favoritesSet, including modelJumpLabelByKey plus active selection, favorites, lock state, disabled reasons, and favorite toggles so rows re-render when presentation changes.

Adds a focused Vitest test that asserts the list receives the current per-model shortcut map in extraData.

^{Reviewed by Cursor Bugbot for commit c7ac18d. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Fix stale shortcut labels in model picker rows by expanding `LegendList` cache key

ModelPickerContent.tsx passes only favoritesSet as extraData to LegendList, which caches rows by model key. When the list reorders (e.g. a model is added or removed), positional shortcut labels like Ctrl+1 become stale because rows are not invalidated.

Replaces the favoritesSetextraData prop with a useMemo-computed modelListExtraData object that includes activeInstanceId, activeModel, favoritesSet, getModelDisabledReason, isLocked, modelJumpLabelByKey, and toggleFavorite.
Any change to these values now triggers a full row re-render, keeping shortcut labels in sync with list position.
Adds a Vitest test suite in ModelPickerContent.test.tsx asserting the correct modelJumpLabelByKey map is passed through extraData.

^{Macroscope summarized c7ac18d.}

coderabbitai · 2026-07-24T07:08:27Z

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: da41e741-dc8b-441f-8430-06a31524d50e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

macroscopeapp · 2026-07-24T07:14:10Z

Approvability

Verdict: Approved

Straightforward bug fix that corrects stale shortcut label rendering in a virtualized list by including modelJumpLabelByKey in the cache invalidation dependencies. Small, self-contained change with clear intent and a test to verify the fix.

^{You can customize Macroscope's approvability policy. Learn more.}

…s when virtualized rows are reordered)

colonelpanic8 · 2026-07-27T02:00:12Z

Reproduced the stale shortcut-label behavior and verified the fix in #4433 at 2b107b9f8a50dd7999e94a55f61509bdc7a1ed57.

Using the same isolated web environment at a 390×844 viewport:

Before (unfixed checkout): filtering the live model list to opus leaves cached rows with their old full-list shortcuts (Ctrl+2 through Ctrl+6) instead of reassigning them from the filtered positions.
After (PR head): the same flow refreshes every row to unique positional shortcuts Ctrl+1 through Ctrl+5.

Before — stale cached labels

After — fixed positional labels

Focused regression test also passes:

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
Test Files 1 passed (1)
Tests 1 passed (1)

…s when virtualized rows are reordered)

…cuts when virtualized rows are reordered)

juliusmarminge · 2026-08-15T11:31:54Z

Closing as superseded by #5400, which has merged and covers this change on the current architecture. Thanks for the contribution.

colonelpanic8 · 2026-07-24T07:08:18Z

What Changed

Include the positional shortcut-label map in the virtualized model list's invalidation state.
Include the other external row-render dependencies so cached model rows refresh when their presentation changes.
Add a focused regression test that verifies the list receives the current per-model shortcut mapping.

Why

LegendList caches rows by model key, but model-picker shortcuts are assigned by visible position. When models ahead of a cached row were inserted, removed, filtered, or reordered, the row could retain its previous label and display the same shortcut as another model.

Fixes#4432.

UI Changes

Behavior-only correction: after filtering the live model list to Opus models, the surviving rows refresh to unique shortcuts from Ctrl+1 through Ctrl+4. No layout or styling changes.

Checklist

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
vp fmt --check apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
vp lint apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
Verified the real picker in an isolated web environment by filtering/reordering the model list and confirming unique positional labels.
Focused web typecheck is currently blocked by unrelated existing dependency/type errors, including @effect/vitest exports and existing LegendList callback inference errors.

Note

Low Risk
Localized UI fix in the model picker virtualization layer with a regression test; no auth, data, or API changes.

Overview
Fixes stale positional shortcut labels (e.g. duplicate Ctrl+1) in the virtualized model picker when the visible list is filtered, reordered, or edited.

LegendList caches rows by model key; jump shortcuts are assigned by position, so cached rows kept old labels until something else invalidated them. ModelPickerContent now passes a memoized modelListExtraData object to extraData instead of only favoritesSet, including modelJumpLabelByKey plus active selection, favorites, lock state, disabled reasons, and favorite toggles so rows re-render when presentation changes.

Adds a focused Vitest test that asserts the list receives the current per-model shortcut map in extraData.

^{Reviewed by Cursor Bugbot for commit c7ac18d. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Fix stale shortcut labels in model picker rows by expanding `LegendList` cache key

ModelPickerContent.tsx passes only favoritesSet as extraData to LegendList, which caches rows by model key. When the list reorders (e.g. a model is added or removed), positional shortcut labels like Ctrl+1 become stale because rows are not invalidated.

Replaces the favoritesSetextraData prop with a useMemo-computed modelListExtraData object that includes activeInstanceId, activeModel, favoritesSet, getModelDisabledReason, isLocked, modelJumpLabelByKey, and toggleFavorite.
Any change to these values now triggers a full row re-render, keeping shortcut labels in sync with list position.
Adds a Vitest test suite in ModelPickerContent.test.tsx asserting the correct modelJumpLabelByKey map is passed through extraData.

^{Macroscope summarized c7ac18d.}

coderabbitai · 2026-07-24T07:08:27Z

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: da41e741-dc8b-441f-8430-06a31524d50e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

macroscopeapp · 2026-07-24T07:14:10Z

Approvability

Verdict: Approved

Straightforward bug fix that corrects stale shortcut label rendering in a virtualized list by including modelJumpLabelByKey in the cache invalidation dependencies. Small, self-contained change with clear intent and a test to verify the fix.

^{You can customize Macroscope's approvability policy. Learn more.}

…s when virtualized rows are reordered)

colonelpanic8 · 2026-07-27T02:00:12Z

Reproduced the stale shortcut-label behavior and verified the fix in #4433 at 2b107b9f8a50dd7999e94a55f61509bdc7a1ed57.

Using the same isolated web environment at a 390×844 viewport:

Before (unfixed checkout): filtering the live model list to opus leaves cached rows with their old full-list shortcuts (Ctrl+2 through Ctrl+6) instead of reassigning them from the filtered positions.
After (PR head): the same flow refreshes every row to unique positional shortcuts Ctrl+1 through Ctrl+5.

Before — stale cached labels

After — fixed positional labels

Focused regression test also passes:

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
Test Files 1 passed (1)
Tests 1 passed (1)

…s when virtualized rows are reordered)

…cuts when virtualized rows are reordered)

juliusmarminge · 2026-08-15T11:31:54Z

Closing as superseded by #5400, which has merged and covers this change on the current architecture. Thanks for the contribution.

colonelpanic8 · 2026-07-24T07:08:18Z

What Changed

Include the positional shortcut-label map in the virtualized model list's invalidation state.
Include the other external row-render dependencies so cached model rows refresh when their presentation changes.
Add a focused regression test that verifies the list receives the current per-model shortcut mapping.

Why

LegendList caches rows by model key, but model-picker shortcuts are assigned by visible position. When models ahead of a cached row were inserted, removed, filtered, or reordered, the row could retain its previous label and display the same shortcut as another model.

Fixes#4432.

UI Changes

Behavior-only correction: after filtering the live model list to Opus models, the surviving rows refresh to unique shortcuts from Ctrl+1 through Ctrl+4. No layout or styling changes.

Checklist

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
vp fmt --check apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
vp lint apps/web/src/components/chat/ModelPickerContent.tsx apps/web/src/components/chat/ModelPickerContent.test.tsx
Verified the real picker in an isolated web environment by filtering/reordering the model list and confirming unique positional labels.
Focused web typecheck is currently blocked by unrelated existing dependency/type errors, including @effect/vitest exports and existing LegendList callback inference errors.

Note

Low Risk
Localized UI fix in the model picker virtualization layer with a regression test; no auth, data, or API changes.

Overview
Fixes stale positional shortcut labels (e.g. duplicate Ctrl+1) in the virtualized model picker when the visible list is filtered, reordered, or edited.

LegendList caches rows by model key; jump shortcuts are assigned by position, so cached rows kept old labels until something else invalidated them. ModelPickerContent now passes a memoized modelListExtraData object to extraData instead of only favoritesSet, including modelJumpLabelByKey plus active selection, favorites, lock state, disabled reasons, and favorite toggles so rows re-render when presentation changes.

Adds a focused Vitest test that asserts the list receives the current per-model shortcut map in extraData.

^{Reviewed by Cursor Bugbot for commit c7ac18d. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Fix stale shortcut labels in model picker rows by expanding `LegendList` cache key

ModelPickerContent.tsx passes only favoritesSet as extraData to LegendList, which caches rows by model key. When the list reorders (e.g. a model is added or removed), positional shortcut labels like Ctrl+1 become stale because rows are not invalidated.

Replaces the favoritesSetextraData prop with a useMemo-computed modelListExtraData object that includes activeInstanceId, activeModel, favoritesSet, getModelDisabledReason, isLocked, modelJumpLabelByKey, and toggleFavorite.
Any change to these values now triggers a full row re-render, keeping shortcut labels in sync with list position.
Adds a Vitest test suite in ModelPickerContent.test.tsx asserting the correct modelJumpLabelByKey map is passed through extraData.

^{Macroscope summarized c7ac18d.}

coderabbitai · 2026-07-24T07:08:27Z

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: da41e741-dc8b-441f-8430-06a31524d50e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

macroscopeapp · 2026-07-24T07:14:10Z

Approvability

Verdict: Approved

Straightforward bug fix that corrects stale shortcut label rendering in a virtualized list by including modelJumpLabelByKey in the cache invalidation dependencies. Small, self-contained change with clear intent and a test to verify the fix.

^{You can customize Macroscope's approvability policy. Learn more.}

…s when virtualized rows are reordered)

colonelpanic8 · 2026-07-27T02:00:12Z

Reproduced the stale shortcut-label behavior and verified the fix in #4433 at 2b107b9f8a50dd7999e94a55f61509bdc7a1ed57.

Using the same isolated web environment at a 390×844 viewport:

Before (unfixed checkout): filtering the live model list to opus leaves cached rows with their old full-list shortcuts (Ctrl+2 through Ctrl+6) instead of reassigning them from the filtered positions.
After (PR head): the same flow refreshes every row to unique positional shortcuts Ctrl+1 through Ctrl+5.

Before — stale cached labels

After — fixed positional labels

Focused regression test also passes:

vp test run apps/web/src/components/chat/ModelPickerContent.test.tsx
Test Files 1 passed (1)
Tests 1 passed (1)

…s when virtualized rows are reordered)

…cuts when virtualized rows are reordered)

juliusmarminge · 2026-08-15T11:31:54Z

Closing as superseded by #5400, which has merged and covers this change on the current architecture. Thanks for the contribution.

github-actionsBot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Jul 24, 2026

macroscopeappBot approved these changes Jul 24, 2026
View reviewed changes

colonelpanic8 force-pushed the fix/model-picker-shortcuts branch 3 times, most recently from fabe41a to 2b107b9Compare July 25, 2026 03:23

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 25, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
360f1b3
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 25, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
eba5ff4
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 25, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
0f37309
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
279347a
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
39c01c7
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
7d2921b
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
48067fc
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
af63287
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
9620fc0
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
2c7ee99
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
e3a72ab
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
698a1d7
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
20db57d
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
0604ab2
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
8e9eaba
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
79e4849
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 26, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
8b34780
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
5fbfc0e
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
415662f
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
205a594
…s when virtualized rows are reordered)

colonelpanic8 mentioned this pull request Jul 27, 2026
[Bug]: Model picker assigns duplicate keyboard shortcuts #4432
Closed

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
334664e
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
964f45e
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
b5d1f77
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
c00d52b
…s when virtualized rows are reordered)

colonelpanic8 force-pushed the fix/model-picker-shortcuts branch from 2b107b9 to e28ca5eCompare July 27, 2026 11:14

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
1d65b47
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
ebe4733
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
0dfcb22
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
bc05505
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
488bd51
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
6b7e54d
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
70f37d9
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 27, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
593edfd
…s when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 28, 2026
stack: mergepingdotgg#4433(Refresh positional model-picker shortcut…
b483b91
…s when virtualized rows are reordered)

colonelpanic8 force-pushed the fix/model-picker-shortcuts branch from e28ca5e to 722921aCompare July 28, 2026 02:07

Fix stale model picker shortcut labels
c7ac18d

colonelpanic8 force-pushed the fix/model-picker-shortcuts branch from 722921a to c7ac18dCompare July 28, 2026 02:51

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 28, 2026
assembly: mergepingdotgg#4433(Refresh positional model-picker short…
b8c2e92
…cuts when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Jul 28, 2026
assembly: mergepingdotgg#4433(Refresh positional model-picker short…
998c0c2
…cuts when virtualized rows are reordered)

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Aug 10, 2026
Rebase PRpingdotgg#4433onto orchestration v2 + latest main
d1ab8a4

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Aug 10, 2026
Rebase PRpingdotgg#4433onto orchestration v2 + latest main
5396906

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Aug 11, 2026
Rebase PRpingdotgg#4433onto orchestration v2 + latest main
bdf4e98

juliusmarminge closed this Aug 15, 2026

colonelpanic8 added a commit to colonelpanic8/t3code that referenced this pull request Aug 18, 2026
Rebase PRpingdotgg#4433onto orchestration v2 + latest main
a5fd0c1