Tighten landing page spacing and harden git status fallback (#81)

* Optimize landing page spacing and git status fallback

- Tighten marketing page layout spacing
- Return empty git status for missing workspace paths
- Clean up theme docs and branding tables

* Import Git status details type in GitCore

Author: BunsDev

apps/server/src/git/Layers/GitCore.test.ts

@@ -1,4 +1,4 @@
-import { existsSync } from "node:fs";
+import fs, { existsSync } from "node:fs";
 import path from "node:path";
 
 import * as NodeServices from "@effect/platform-node/NodeServices";
@@ -1277,6 +1277,35 @@ it.layer(TestLayer)("git integration", (it) => {
       }),
     );
 
+    it.effect("returns an empty status for a missing workspace path", () =>
+      Effect.gen(function* () {
+        const tmp = yield* makeTmpDir();
+        yield* initRepoWithCommit(tmp);
+        const core = yield* GitCore;
+
+        expect(existsSync(tmp)).toBe(true);
+        yield* Effect.sync(() => fs.rmSync(tmp, { recursive: true, force: true }));
+
+        const details = yield* core.statusDetails(tmp);
+        expect(details).toEqual(
+          expect.objectContaining({
+            branch: null,
+            hasWorkingTreeChanges: false,
+            hasConflicts: false,
+            conflictedFiles: [],
+            hasUpstream: false,
+            aheadCount: 0,
+            behindCount: 0,
+            upstreamRef: null,
+            pr: null,
+          }),
+        );
+        expect(details.workingTree.files).toEqual([]);
+        expect(details.workingTree.insertions).toBe(0);
+        expect(details.workingTree.deletions).toBe(0);
+      }),
+    );
+
     it.effect("computes ahead count against base branch when no upstream is configured", () =>
       Effect.gen(function* () {
         const tmp = yield* makeTmpDir();

apps/server/src/git/Layers/GitCore.ts

@@ -27,6 +27,7 @@ import {
   type GitCoreShape,
   type ExecuteGitInput,
   type ExecuteGitResult,
+  type GitStatusDetails,
 } from "../Services/GitCore.ts";
 import { ServerConfig } from "../../config.ts";
 import { decodeJsonResult } from "@okcode/shared/schemaJson";
@@ -121,6 +122,23 @@ function parsePorcelainConflictPath(line: string): string | null {
   return parsePorcelainPath(line);
 }
 
+const EMPTY_STATUS_DETAILS = {
+  branch: null,
+  hasWorkingTreeChanges: false,
+  hasConflicts: false,
+  conflictedFiles: [],
+  workingTree: {
+    files: [],
+    insertions: 0,
+    deletions: 0,
+  },
+  hasUpstream: false,
+  aheadCount: 0,
+  behindCount: 0,
+  upstreamRef: null,
+  pr: null,
+} satisfies GitStatusDetails;
+
 function parseBranchLine(line: string): { name: string; current: boolean } | null {
   const trimmed = line.trim();
   if (trimmed.length === 0) return null;
@@ -1042,15 +1060,34 @@ export const makeGitCore = (options?: { executeOverride?: GitCoreShape["execute"
 
     const statusDetails: GitCoreShape["statusDetails"] = (cwd) =>
       Effect.gen(function* () {
+        const cwdStat = yield* fileSystem.stat(cwd).pipe(Effect.catch(() => Effect.succeed(null)));
+        if (!cwdStat || cwdStat.type !== "Directory") {
+          return EMPTY_STATUS_DETAILS;
+        }
+
         yield* refreshStatusUpstreamIfStale(cwd).pipe(Effect.ignoreCause({ log: true }));
 
-        const [statusStdout, unstagedNumstatStdout, stagedNumstatStdout] = yield* Effect.all(
+        const status = yield* executeGit(
+          "GitCore.statusDetails.status",
+          cwd,
+          ["status", "--porcelain=2", "--branch"],
+          { allowNonZeroExit: true },
+        );
+        if (status.code !== 0) {
+          const stderr = status.stderr.trim();
+          if (stderr.toLowerCase().includes("not a git repository")) {
+            return EMPTY_STATUS_DETAILS;
+          }
+          return yield* createGitCommandError(
+            "GitCore.statusDetails.status",
+            cwd,
+            ["status", "--porcelain=2", "--branch"],
+            stderr.length > 0 ? stderr : `git status failed with code ${status.code}.`,
+          );
+        }
+
+        const [unstagedNumstatStdout, stagedNumstatStdout] = yield* Effect.all(
           [
-            runGitStdout("GitCore.statusDetails.status", cwd, [
-              "status",
-              "--porcelain=2",
-              "--branch",
-            ]),
             runGitStdout("GitCore.statusDetails.unstagedNumstat", cwd, ["diff", "--numstat"]),
             runGitStdout("GitCore.statusDetails.stagedNumstat", cwd, [
               "diff",
@@ -1061,6 +1098,8 @@ export const makeGitCore = (options?: { executeOverride?: GitCoreShape["execute"
           { concurrency: "unbounded" },
         );
 
+        const statusStdout = status.stdout;
+
         let branch: string | null = null;
         let upstreamRef: string | null = null;
         let aheadCount = 0;