chore: merge main into Next.js 16 upgrade branch

Resolves conflicts between this branch and main's parallel work:
- #67 (chore: remove lootbox feature)
- #69 (test: add Vitest + unit tests for proposal encoding, funding totals,
  middleware) plus the CI workflow that runs `pnpm lint` + `pnpm test`

Key merge decisions:
- Rename `src/middleware.test.ts` → `src/proxy.test.ts` and update its
  `import` / symbols / describe label to `proxy`, matching this branch's
  `middleware.ts` → `proxy.ts` rename for the Next.js 16 file convention.
- Regenerated `pnpm-lock.yaml` from the merged `package.json` so both
  Next.js 16 / React 19.2.5 and the new `vitest` + `@vitest/coverage-v8`
  devDeps resolve.
- Accept Next.js 16's tsconfig auto-update: add `.next/dev/types/**/*.ts`
  to include, switch `jsx` to `react-jsx`. `pnpm exec tsc --noEmit`
  passes; `tsc` doesn't emit so the jsx change is non-functional here.
- CLAUDE.md: refresh the `md/` comment to reference `proxy.ts` instead
  of middleware.

Validation:
- `pnpm lint` → 0 errors, 137 warnings (pre-existing react-compiler warnings)
- `pnpm exec tsc --noEmit` → clean
- `pnpm test` → 37 pass, 1 todo (includes the renamed proxy tests)
- `pnpm dev` → Ready in 903ms; `curl -H "Accept: text/markdown" /` → 200

https://claude.ai/code/session_01QVjceB1PFubwe6TCpRq3xx

Author: claude

.github/workflows/ci.yml

@@ -0,0 +1,32 @@
+name: CI
+
+on:
+  pull_request:
+  push:
+    branches:
+      - main
+
+jobs:
+  lint-format-test:
+    name: Lint, format, unit tests
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: pnpm/action-setup@v4
+        with:
+          version: 10
+
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 22
+          cache: pnpm
+
+      - name: Install dependencies
+        run: pnpm install --frozen-lockfile
+
+      - name: Lint
+        run: pnpm lint
+
+      - name: Unit tests
+        run: pnpm test

CLAUDE.md

@@ -34,16 +34,16 @@ No unit test runner; Playwright e2e only. `tests/e2e/propdates.spec.ts`, `tests/
 
 ```
 src/
-├── app/              # App Router — 25+ routes incl. /auctions /proposals /propose /tv /members /treasury /lootbox /feed /propdates /droposals /installations /blogs /coin-proposal /community/bounties /map /mural
+├── app/              # App Router — 25+ routes incl. /auctions /proposals /propose /tv /members /treasury /feed /propdates /droposals /installations /blogs /coin-proposal /community/bounties /map /mural
 │   ├── api/          # 18 route groups (alchemy, coins, ens, og, pinata, propdates, proposals, treasury, tv, …)
-│   └── md/           # markdown content-negotiation target (see middleware)
+│   └── md/           # markdown content-negotiation target (see proxy.ts)
 ├── components/       # 24 feature dirs + ui/ (shadcn)
 ├── hooks/            # 40 hooks — see naming note below
 ├── services/         # 16 data-layer modules (auctions, proposals, treasury, feed, members, farcaster, poidh, snapshot, …)
 ├── lib/              # config.ts, thirdweb.ts, wagmi.ts, subgraph.ts, ipfs.ts, zora-*, proposal-*, og-*, schemas/, types/
 ├── data/             # static JSON (installations.json)
 ├── types/            # shared TS interfaces
-├── utils/abis/       # contract ABIs (gnarsLootboxV4Abi, erc20, …)
+├── utils/abis/       # contract ABIs (erc20, …)
 ├── workers/          # client-side search workers (blog, proposal)
 └── proxy.ts          # Accept: text/markdown → rewrite to /md/* (Next.js 16 proxy convention, formerly middleware.ts)
 ```
@@ -70,13 +70,6 @@ src/
 - **Governance pre-checks** — write hooks that gate on voting power must pre-read `getPastVotes` / `getVotes` and bail with a toast before prompting signatures.
 - Full provider tree + decision matrix: `docs/architecture/thirdweb-wallet-layer.md`.
 
-### Lootbox (V4 only)
-
-- UI at `src/app/lootbox/page.tsx`, ABI `src/utils/abis/gnarsLootboxV4Abi.ts`, hooks `use-lootbox-contract.ts` + `use-lootbox-actions.ts`.
-- Admin controls for VRF config, allowlist, deposits, withdrawals, recovery.
-- Listens for `FlexOpened` event (in `use-lootbox-contract.ts`) to surface NFT win toasts.
-- Address from `DAO_ADDRESSES.lootbox` in `config.ts` — update after each deploy.
-
 ## Data Fetching
 
 - Server Components for initial load (SEO, perf).

package.json

@@ -9,6 +9,9 @@
     "lint": "eslint",
     "format": "prettier --write .",
     "format:check": "prettier --check .",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "test:coverage": "vitest run --coverage",
     "proposal:funding:poc": "tsx scripts/proposal-funding-poc.ts"
   },
   "dependencies": {
@@ -100,6 +103,7 @@
     "@types/react": "^19.2.14",
     "@types/react-dom": "^19.2.3",
     "@types/three": "^0.182.0",
+    "@vitest/coverage-v8": "^4.1.5",
     "eslint": "^9",
     "eslint-config-next": "16.2.4",
     "eslint-config-prettier": "^10.1.8",
@@ -108,6 +112,7 @@
     "tailwindcss": "^4",
     "tsx": "^4.21.0",
     "tw-animate-css": "^1.3.7",
-    "typescript": "^5"
+    "typescript": "^5",
+    "vitest": "^4.1.5"
   }
 }

pnpm-lock.yaml

@@ -235,12 +235,6 @@ export async function GET(): Promise<Response> {
       changeFrequency: "daily",
       priority: 0.7,
     },
-    {
-      url: toUrl("/lootbox"),
-      lastModified: now,
-      changeFrequency: "weekly",
-      priority: 0.6,
-    },
     {
       url: toUrl("/map"),
       lastModified: now,

public/gnars-lootbox.mp4

@@ -164,17 +164,6 @@ const navigationItems = [
         description: "Gnarly challenges from the action sports community",
         badge: "NEW!",
       },
-      {
-        title: "divider",
-        label: "Beta",
-      },
-      {
-        title: "Lootbox",
-        href: "/lootbox",
-        icon: Gift,
-        description: "Join Gnars DAO onboarding",
-        badge: "BETA",
-      },
     ],
   },
 ];
@@ -250,21 +239,6 @@ function DesktopNav() {
                 <NavigationMenuContent>
                   <ul className="grid w-[400px] gap-2 p-2">
                     {item.items?.map((subItem) => {
-                      // Divider item
-                      if (subItem.title === "divider") {
-                        return (
-                          <li key={`divider-${subItem.label}`} className="px-3 py-2">
-                            <div className="flex items-center gap-2">
-                              <div className="h-px flex-1 bg-border" />
-                              <span className="text-xs font-medium text-muted-foreground uppercase tracking-wider">
-                                {subItem.label}
-                              </span>
-                              <div className="h-px flex-1 bg-border" />
-                            </div>
-                          </li>
-                        );
-                      }
-
                       if (!subItem.icon) return null;
 
                       const SubIcon = subItem.icon;
@@ -419,21 +393,6 @@ function MobileNav() {
                     {item.title}
                   </div>
                   {item.items?.map((subItem) => {
-                    if (subItem.title === "divider") {
-                      return (
-                        <div key={subItem.title} className="relative my-2 mx-3">
-                          <div className="absolute inset-0 flex items-center">
-                            <div className="w-full border-t" />
-                          </div>
-                          <div className="relative flex justify-center text-xs uppercase">
-                            <span className="bg-background px-2 text-muted-foreground">
-                              {subItem.label}
-                            </span>
-                          </div>
-                        </div>
-                      );
-                    }
-
                     if (!subItem.icon) return null;
 
                     const SubIcon = subItem.icon;