fix: swap j/k keybindings to match Vim conventions

scripts/build-browser.ts

@@ -28,7 +28,10 @@ async function build() {
   const indexHtml = await Bun.file(indexPath).text();
   await Bun.write(indexPath, indexHtml.replaceAll("./", "/"));
 
-  await cp(resolve(process.cwd(), "src", "browser", "logo.svg"), resolve(process.cwd(), "dist", "browser", "logo.svg"));
+  await cp(
+    resolve(process.cwd(), "src", "browser", "logo.svg"),
+    resolve(process.cwd(), "dist", "browser", "logo.svg")
+  );
 
   // Build worker separately with document shim for Prism/refractor
   const workerResult = await Bun.build({

scripts/build-vercel.ts

@@ -17,7 +17,9 @@ await import("./build-browser");
 // Copy browser build contents to public (served by Vercel CDN)
 const browserFiles = await readdir(browserDistDir);
 for (const file of browserFiles) {
-  await cp(resolve(browserDistDir, file), resolve(publicDir, file), { recursive: true });
+  await cp(resolve(browserDistDir, file), resolve(publicDir, file), {
+    recursive: true,
+  });
 }
 console.log("Copied browser files to public/");
 

src/browser/components/pr-overview.tsx

@@ -711,7 +711,7 @@ export const PROverview = memo(function PROverview() {
                           key={`comment-${comment.id}`}
                           user={comment.user}
                           createdAt={comment.created_at}
-                          body={comment.body}
+                          body={comment.body ?? null}
                           reactions={reactions[`comment-${comment.id}`]}
                           onAddReaction={(content) =>
                             handleAddCommentReaction(comment.id, content)
@@ -747,7 +747,11 @@ export const PROverview = memo(function PROverview() {
                 {canWrite && pr.state === "open" && !pr.merged && (
                   <>
                     <MergeSection
-                      pr={pr}
+                      pr={{
+                        ...pr,
+                        requested_reviewers:
+                          pr.requested_reviewers ?? undefined,
+                      }}
                       checkStatus={checkStatus}
                       checks={checks}
                       canMerge={canMergePR}

src/browser/components/pr-review.tsx

@@ -2839,7 +2839,12 @@ const SubmitReviewDropdown = memo(function SubmitReviewDropdown() {
               e.stopPropagation();
               handleSubmit();
             }}
-            disabled={submitting || (reviewType === "COMMENT" && pendingCount === 0 && !reviewBody.trim())}
+            disabled={
+              submitting ||
+              (reviewType === "COMMENT" &&
+                pendingCount === 0 &&
+                !reviewBody.trim())
+            }
             className={cn(
               "flex items-center gap-1.5 px-2 py-1 text-xs font-medium rounded-md transition-colors disabled:opacity-50",
               reviewType === "APPROVE" &&

src/browser/contexts/github.tsx

@@ -1348,19 +1348,20 @@ function createGitHubStore() {
     if (pending) return pending;
 
     const promise = octokit
-      .paginate("GET /repos/{owner}/{repo}/labels", {
+      .request("GET /repos/{owner}/{repo}/labels", {
         owner,
         repo,
         per_page: 100,
       })
-      .then((labels) => {
-        const result = labels.map((l) => ({
-          name: l.name,
-          color: l.color,
-          description: l.description ?? null,
-        }));
+      .then(({ data: labels }) => {
+        const result = labels.map(
+          (l: { name: string; color: string; description: string | null }) => ({
+            name: l.name,
+            color: l.color,
+            description: l.description ?? null,
+          })
+        );
         cache.set(cacheKey, result);
-        cache.clearPending(cacheKey);
         return result;
       });
 

src/browser/contexts/pr-review.tsx

@@ -2047,14 +2047,14 @@ export function useKeyboardNavigation() {
 
       // Shortcuts
       switch (e.key.toLowerCase()) {
-        case "k":
+        case "j":
           e.preventDefault();
           // Use startTransition to allow React to interrupt rendering during rapid navigation
           startTransition(() => {
             store.navigateToNextUnviewedFile();
           });
           break;
-        case "j":
+        case "k":
           e.preventDefault();
           // Use startTransition to allow React to interrupt rendering during rapid navigation
           startTransition(() => {

src/browser/ui/user-hover-card.tsx

@@ -110,10 +110,12 @@ function UserHoverCardContent({
           </a>
           <div className="flex items-center gap-1.5 text-sm text-muted-foreground">
             <span>{profile.login}</span>
-            {profile.pronouns && (
+            {(profile as UserProfile & { pronouns?: string }).pronouns && (
               <>
                 <span>·</span>
-                <span>{profile.pronouns}</span>
+                <span>
+                  {(profile as UserProfile & { pronouns?: string }).pronouns}
+                </span>
               </>
             )}
           </div>

src/index.ts

@@ -6,7 +6,6 @@ import { serveStatic } from "@hono/node-server/serve-static";
 
 const app = new Hono();
 
-
 // Debug route to see filesystem structure on Vercel
 app.get("/_debug", (c) => {
   const listDir = (path: string, depth = 0): string[] => {
@@ -18,10 +17,12 @@ app.get("/_debug", (c) => {
         return results;
       }
       const entries = readdirSync(path, { withFileTypes: true });
-      for (const entry of entries.slice(0, 50)) { // Limit to 50 entries
+      for (const entry of entries.slice(0, 50)) {
+        // Limit to 50 entries
         if (entry.isDirectory()) {
           results.push(`${indent}${entry.name}/`);
-          if (depth < 2) { // Limit depth
+          if (depth < 2) {
+            // Limit depth
             results.push(...listDir(resolve(path, entry.name), depth + 1));
           }
         } else {
@@ -36,7 +37,7 @@ app.get("/_debug", (c) => {
 
   const cwd = process.cwd();
   const metaDirname = import.meta.dirname;
-  
+
   const info = {
     cwd,
     metaDirname,
@@ -54,7 +55,6 @@ app.route("/", api);
 
 app.use("/*", serveStatic({ root: resolve(process.cwd(), "public") }));
 
-
 // SPA fallback - serve index.html for client-side routing
 // Static files are served by Vercel CDN from public/
 app.get("*", (c) => {