ci(release): publish latest release

Author: hello-happy-puppy

.agents/skills/argent-react-native-app-workflow/SKILL.md

@@ -151,6 +151,34 @@ Once you discover the correct build/run workflow for a project, **save it to pro
 
 For full simulator setup workflow, refer to the `argent-ios-simulator-setup` skill.
 
+### 3.6 Verifying Local Release-Variant Builds — Stale-Code Gotchas
+
+**Applies to Android local release-variant builds** (`bun mobile android:<flavor>:release:local`, i.e. `apps/mobile/scripts/runAndroidLocal.sh`). Release variants **embed the JS bundle in the APK**, so a verify-on-device loop can silently test **old code** through **two independent caching layers**. Each makes a change look like it had "no effect" when it was never in the installed APK. **Both must be defeated — disabling one alone is not enough.** `runAndroidLocal.sh` handles both by default; this section is for recognizing the symptoms if a build looks stale or you build with another command.
+
+#### Gotcha 1 — EAS build cache reinstalls a stale APK
+
+`apps/mobile/app.config.ts` sets `experiments.buildCacheProvider: 'eas'`. `expo run:android` computes a **native** fingerprint and, on a match, downloads + installs a cached APK instead of building. A **JS-only** change does not move the native fingerprint, so for a release variant it reinstalls an APK with old JS.
+
+- **Symptom (logs):** `Searching builds with matching fingerprint on EAS servers` → `Successfully downloaded cached build` → install path under `…/eas-build-run-cache/….apk`.
+- **Detection:** you changed only JS/TS, yet the install line points at an `eas-build-run-cache` path.
+- **Fix:** build via `runAndroidLocal.sh` — it exports `EXPO_LOCAL_NO_BUILD_CACHE=1`, which gates the provider off in `app.config.ts` so a real build runs. Confirm `› Building app...` in the logs (not "downloaded cached build"). If invoking `expo run:android` directly, export `EXPO_LOCAL_NO_BUILD_CACHE=1` yourself.
+
+#### Gotcha 2 — gradle `createBundle<Variant>JsAndAssets` stays UP-TO-DATE
+
+Even with the EAS cache off, gradle's incremental check misses changes to monorepo JS sources (e.g. `packages/uniswap/**`), so the bundle task stays `UP-TO-DATE` and the APK embeds the **previously-bundled** `index.android.bundle`.
+
+- **Symptom (logs):** `> Task :app:createBundleDevReleaseJsAndAssets UP-TO-DATE`.
+- **Detection:** compare the embedded bundle mtime to your edit time:
+  ```bash
+  ls -la apps/mobile/android/app/build/generated/assets/createBundle<Variant>JsAndAssets/index.android.bundle
+  ```
+- **Fix:** `runAndroidLocal.sh` deletes the bundle task outputs by default (pass `--no-clean-js` to skip for native-only rebuilds). To clear them manually, remove:
+  - `apps/mobile/android/app/build/generated/assets/createBundle<Variant>JsAndAssets`
+  - `apps/mobile/android/app/build/generated/res/createBundle<Variant>JsAndAssets`
+  - `apps/mobile/android/app/build/intermediates/assets/<variant>`
+
+  A successful regen logs `> Task :app:createBundle<Variant>JsAndAssets` (no `UP-TO-DATE`) and `Writing bundle output to …`. `<Variant>` is the capitalized gradle variant (`DevRelease`, `BetaRelease`, `ProdRelease`); `<variant>` is the camelCase form (`devRelease`, …).
+
 ---
 
 ## 4. Runtime Problems in the App

RELEASE

@@ -1,6 +1,6 @@
 IPFS hash of the deployment:
-- CIDv0: `QmcXGyQ2eF5eus7j3PKm4mRut7EV5X2qcB9ZMoufoZdCKG`
-- CIDv1: `bafybeigsxrw3dbnn44hhyi25tyc7dzcggbsbtkqhinpidom2hftu5jmmr4`
+- CIDv0: `QmeFNQ5YKGi3RfGVHatRvmBCpc95MLetgWruN5oDNCgGTX`
+- CIDv1: `bafybeihmmcsoeabmzjedko3molgj56em3smlek547ljl7fgj6wilejyzay`
 
 The latest release is always mirrored at [app.uniswap.org](https://app.uniswap.org).
 
@@ -10,5 +10,5 @@ You can also access the Uniswap Interface from an IPFS gateway.
 Your Uniswap settings are never remembered across different URLs.
 
 IPFS gateways:
-- https://bafybeigsxrw3dbnn44hhyi25tyc7dzcggbsbtkqhinpidom2hftu5jmmr4.ipfs.dweb.link/
-- [ipfs://QmcXGyQ2eF5eus7j3PKm4mRut7EV5X2qcB9ZMoufoZdCKG/](ipfs://QmcXGyQ2eF5eus7j3PKm4mRut7EV5X2qcB9ZMoufoZdCKG/)
+- https://bafybeihmmcsoeabmzjedko3molgj56em3smlek547ljl7fgj6wilejyzay.ipfs.dweb.link/
+- [ipfs://QmeFNQ5YKGi3RfGVHatRvmBCpc95MLetgWruN5oDNCgGTX/](ipfs://QmeFNQ5YKGi3RfGVHatRvmBCpc95MLetgWruN5oDNCgGTX/)

VERSION

@@ -1 +1 @@
-web/5.148.9
+web/5.149.0

apps/extension/package.json

@@ -117,8 +117,8 @@
     "dev:firefox": "nx dev:firefox extension",
     "env:local:download": "nx env:local:download extension",
     "env:local:upload": "nx env:local:upload extension",
-    "format": "nx format extension",
-    "format:check": "nx format:check extension",
+    "format": "nx run extension:format",
+    "format:check": "nx run extension:format:check",
     "lint": "nx lint extension",
     "lint:fix": "nx lint:fix extension",
     "postinstall": "nx postinstall extension",

apps/extension/project.json

@@ -94,7 +94,7 @@
       "outputs": ["{projectRoot}/.wxt/types", "{projectRoot}/.wxt/tsconfig.json", "{projectRoot}/.wxt/wxt.d.ts"]
     },
     "config:pull": {
-      "command": "if [ \"$USE_NEW_CONFIGS\" = \"true\" ]; then bun config:pull extension --env \"${CONFIG_ENV:-development}\"; fi",
+      "command": "if [ \"$USE_NEW_CONFIGS\" = \"true\" ] && [ \"$SKIP_CONFIG_PULL\" != \"true\" ]; then bun config:pull extension --env \"${CONFIG_ENV:-development}\"; fi",
       "options": { "cwd": "{workspaceRoot}" }
     },
     "snapshots": {
@@ -176,7 +176,7 @@
           "BUILD_NUM": "0"
         }
       },
-      "inputs": ["dependencies", "sourceFiles", "tsConfig"],
+      "inputs": ["sourceFiles"],
       "outputs": ["{projectRoot}/.output/chrome-mv3"],
       "cache": true,
       "dependsOn": ["^build", "prepare"]
@@ -186,7 +186,7 @@
       "options": {
         "cwd": "{projectRoot}"
       },
-      "inputs": ["dependencies", "sourceFiles", "tsConfig"],
+      "inputs": ["sourceFiles"],
       "outputs": ["{projectRoot}/build"],
       "cache": true,
       "dependsOn": ["^build", "prepare"]

apps/extension/src/app/components/tabs/PoolsTab.test.tsx

@@ -0,0 +1,166 @@
+import { PositionStatus } from '@uniswap/client-data-api/dist/data/v1/poolTypes_pb'
+import { PoolsTab } from 'src/app/components/tabs/PoolsTab'
+import { render, screen } from 'src/test/test-utils'
+import { useWalletPositions } from 'uniswap/src/features/positions/hooks/useWalletPositions'
+
+jest.mock('uniswap/src/features/positions/hooks/useWalletPositions', () => ({
+  useWalletPositions: jest.fn(),
+}))
+
+jest.mock('wallet/src/features/transactions/hooks/usePendingLiquidityTransactionsChangeListener', () => ({
+  usePendingLiquidityTransactionsChangeListener: jest.fn(),
+}))
+
+jest.mock('uniswap/src/features/chains/hooks/useEnabledChains', () => ({
+  useEnabledChains: () => ({ chains: [1] }),
+}))
+
+jest.mock('utilities/src/react/useInfiniteScroll', () => ({
+  useInfiniteScroll: () => ({ sentinelRef: { current: null } }),
+}))
+
+jest.mock('uniswap/src/components/portfolio/PositionItem/PositionItem', () => ({
+  PositionItem: ({ positionInfo }: { positionInfo: { poolId: string } }) => {
+    const { Text } = jest.requireActual('ui/src')
+    return <Text testID={`position-${positionInfo.poolId}`}>{positionInfo.poolId}</Text>
+  },
+}))
+
+const mockUseWalletPositions = useWalletPositions as jest.MockedFunction<typeof useWalletPositions>
+
+const baseResult = {
+  positions: [],
+  hiddenPositions: [],
+  allPositions: [],
+  isLoading: false,
+  isFetching: false,
+  isFetchingNextPage: false,
+  isPlaceholderData: false,
+  hasNextPage: false,
+  hasData: true,
+  error: null,
+  refetch: jest.fn(),
+  fetchNextPage: jest.fn(),
+} as unknown as ReturnType<typeof useWalletPositions>
+
+const position = (poolId: string, status: PositionStatus = PositionStatus.IN_RANGE): never =>
+  ({ poolId, tokenId: 't', chainId: 1, status }) as never
+
+describe('PoolsTab', () => {
+  beforeEach(() => {
+    jest.clearAllMocks()
+  })
+
+  it('renders the error state when the query errors before any data', () => {
+    mockUseWalletPositions.mockReturnValue({
+      ...baseResult,
+      hasData: false,
+      error: new Error('boom') as never,
+    })
+
+    render(<PoolsTab address="0xabc" />)
+
+    expect(screen.getByText('Pools balances currently unavailable')).toBeDefined()
+    expect(screen.getByText('Try again')).toBeDefined()
+  })
+
+  it('renders the loading skeleton while the first page loads (no data, no error)', () => {
+    mockUseWalletPositions.mockReturnValue({
+      ...baseResult,
+      isFetching: true,
+      hasData: false,
+    })
+
+    render(<PoolsTab address="0xabc" />)
+
+    expect(screen.getByText('Status')).toBeDefined()
+    expect(screen.getByTestId('pools-loading-skeleton')).toBeDefined()
+    expect(screen.queryByText('Pools balances currently unavailable')).toBeNull()
+  })
+
+  it('swaps the error view for the loading skeleton while a retry re-drives the list', () => {
+    mockUseWalletPositions.mockReturnValue({
+      ...baseResult,
+      hasData: false,
+      error: new Error('boom') as never,
+      isFetching: true,
+      isFetchingNextPage: false,
+    })
+
+    render(<PoolsTab address="0xabc" />)
+
+    expect(screen.getByTestId('pools-loading-skeleton')).toBeDefined()
+    expect(screen.queryByText('Pools balances currently unavailable')).toBeNull()
+  })
+
+  it('renders the status filter with the default "Open" selection', () => {
+    mockUseWalletPositions.mockReturnValue(baseResult)
+
+    render(<PoolsTab address="0xabc" />)
+
+    expect(screen.getByText('Status')).toBeDefined()
+    expect(screen.getByText('Open')).toBeDefined()
+  })
+
+  it('shows no no-results message for the empty open filter', () => {
+    mockUseWalletPositions.mockReturnValue({ ...baseResult, positions: [], hasData: true })
+
+    render(<PoolsTab address="0xabc" openPositionsCount={3} />)
+
+    expect(screen.queryByText('No closed positions')).toBeNull()
+    expect(screen.queryByText(/View .* open position/)).toBeNull()
+  })
+
+  it('renders visible position rows', () => {
+    mockUseWalletPositions.mockReturnValue({
+      ...baseResult,
+      positions: [position('pool-a'), position('pool-b')],
+    })
+
+    render(<PoolsTab address="0xabc" />)
+
+    expect(screen.getByTestId('position-pool-a')).toBeDefined()
+    expect(screen.getByTestId('position-pool-b')).toBeDefined()
+    expect(screen.queryByText('Pools balances currently unavailable')).toBeNull()
+  })
+
+  it('filters hidden positions by status, hiding the expando when none match the filter', () => {
+    mockUseWalletPositions.mockReturnValue({
+      ...baseResult,
+      positions: [],
+      hiddenPositions: [position('pool-hidden-closed', PositionStatus.CLOSED)],
+    })
+
+    // Default filter is Open; a closed hidden position must not surface the expando.
+    render(<PoolsTab address="0xabc" />)
+
+    expect(screen.queryByText(/Hidden/)).toBeNull()
+  })
+
+  it('shows the hidden expando with the hidden count and reveals rows when expanded', () => {
+    mockUseWalletPositions.mockReturnValue({
+      ...baseResult,
+      positions: [position('pool-a')],
+      hiddenPositions: [position('pool-hidden-1'), position('pool-hidden-2')],
+    })
+
+    render(<PoolsTab address="0xabc" />)
+
+    expect(screen.getByText(/Hidden .*\(2\)/)).toBeDefined()
+    expect(screen.queryByTestId('position-pool-hidden-1')).toBeNull()
+  })
+
+  it('filters out positions that do not match the active status filter (default open)', () => {
+    mockUseWalletPositions.mockReturnValue({
+      ...baseResult,
+      positions: [position('pool-open'), position('pool-closed', PositionStatus.CLOSED)],
+      hiddenPositions: [position('pool-hidden-closed', PositionStatus.CLOSED)],
+    })
+
+    render(<PoolsTab address="0xabc" />)
+
+    expect(screen.getByTestId('position-pool-open')).toBeDefined()
+    expect(screen.queryByTestId('position-pool-closed')).toBeNull()
+    expect(screen.queryByText(/Hidden/)).toBeNull()
+  })
+})