Conversation
Mbeaulne
added this pull request to stack #2752
September 18, 2026 19:28
🎩 PreviewA preview build has been created at: |
This was referenced Sep 18, 2026
Mbeaulne
marked this pull request as ready for review
September 22, 2026 14:32
| const showingCached = !!query.error && !query.data; | ||
| const cached = useQuery({ | ||
| queryKey: [...queryKey, "cached"], | ||
| queryFn: () => storage.remote?.listCached() ?? Promise.resolve([]), |
Mbeaulne
force-pushed
the
mb/remote-pipelines-list
branch
from
September 22, 2026 19:05
b6d78da to
d7aa18e
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
List layer of the remote-pipeline stack. Base: #2749. Deployment activation remains hard-disabled until #2739.
Review focus: tab and account isolation, cursor/cache behavior, and preserving local workflows while avoiding per-row remote requests.
Related pull requests
#2746 → #2747 → #2748 → #2749 → #2750 → #2751 → #2739 → #2780 → #2783
How to test
Passed at this layer: 36 hook/filter/row tests, TypeScript, targeted source lint, and Prettier checks.
pnpm exec vitest run src/components/Home/PipelineSection/usePipelineList.test.ts src/components/Home/PipelineSection/useRemotePipelineList.test.ts src/components/Home/PipelineSection/usePipelineFilters.test.ts src/components/Home/PipelineSection/PipelineRow.test.tsx pnpm run typecheckCoverage includes cursor navigation and cached pages, global name/date/sort filters, account changes, refresh and mutation invalidation, pending drafts, and outage fallback. Full-width layout was also verified in the complete-stack browser at 1280px and 1920px; integrated browser-suite results are tracked in #2739.
For a manual local-mode check, verify search, filters, favorites, selection, canceling row deletion, bulk deletion, and folder navigation. With the rollout enabled, switch tabs, page through more than ten remote entries, search for a later-page name, and upload a Local entry; the selected tab should remain unchanged and the confirmed upload should appear in Remote.
Change Size
Relative to
mb/remote-pipelines-editorfor this PR only. Changed lines = additions + deletions.Application code excludes test files. Tests include unit, component, and browser test source; test-runner configuration, CI, package scripts, compiler configuration, and documentation are counted separately.