Revert "docs: expand architecture and release documentation (#173)" (#174)

BunsDev · web-flow · commit a6021057ba7f · 2026-04-01T06:08:07.000-05:00
This reverts commit 075ba27.
diff --git a/README.md b/README.md
diff --git a/apps/marketing/app/globals.css b/apps/marketing/app/globals.css
@@ -78,10 +78,10 @@
 }
 
 @theme inline {
-  --font-sans:
-    var(--font-dm-sans), "DM Sans", -apple-system, BlinkMacSystemFont, "Segoe UI", system-ui,
-    sans-serif;
-  --font-mono: "SF Mono", "SFMono-Regular", Consolas, "Liberation Mono", Menlo, monospace;
+  --font-sans: var(--font-dm-sans), "DM Sans", -apple-system, BlinkMacSystemFont,
+    "Segoe UI", system-ui, sans-serif;
+  --font-mono: "SF Mono", "SFMono-Regular", Consolas, "Liberation Mono", Menlo,
+    monospace;
   --color-background: var(--background);
   --color-foreground: var(--foreground);
   --color-card: var(--card);
diff --git a/apps/marketing/components/ai-section.tsx b/apps/marketing/components/ai-section.tsx
@@ -310,7 +310,7 @@ export function AISection() {
 
                 {/* MCP Code Snippet - Use semantic tokens for code syntax */}
                 <div className="bg-card/30 border border-border/60 rounded-xl p-5 font-mono text-sm">
-                  <p className="text-muted-foreground/70 mb-3">{"//mcp.sprint.app/sse"}</p>
+                  <p className="text-muted-foreground/70 mb-3">//mcp.sprint.app/sse</p>
                   <div className="space-y-1 mb-6">
                     <p>
                       <span className="text-code-constant/70">"mcpServers"</span>
diff --git a/apps/marketing/components/dashboard/dashboard-inbox.tsx b/apps/marketing/components/dashboard/dashboard-inbox.tsx
@@ -184,7 +184,7 @@ function IssueRow({
   issue,
   selected,
   onClick,
-  isMobile: _isMobile,
+  isMobile,
 }: {
   issue: Issue;
   selected: boolean;
diff --git a/apps/marketing/components/dashboard/dashboard-issue-detail.tsx b/apps/marketing/components/dashboard/dashboard-issue-detail.tsx
@@ -484,9 +484,9 @@ export function DashboardIssueDetail({
             </div>
 
             <div className="space-y-4">
-              {issue.activity.map((activity) => (
+              {issue.activity.map((activity, index) => (
                 <ActivityItem
-                  key={`${activity.time}-${activity.user}-${activity.action}`}
+                  key={index}
                   user={activity.user}
                   action={activity.action}
                   time={activity.time}
diff --git a/apps/marketing/components/dashboard/dashboard-sidebar.tsx b/apps/marketing/components/dashboard/dashboard-sidebar.tsx
@@ -298,7 +298,7 @@ function NavItem({
   color,
   teamColor,
   onClick,
-  isMobile: _isMobile,
+  isMobile,
 }: {
   icon: React.ElementType;
   label: string;
diff --git a/apps/marketing/components/logo-cloud.tsx b/apps/marketing/components/logo-cloud.tsx
@@ -47,15 +47,15 @@ export function LogoCloud() {
           >
             {/* Logo grid with variety */}
             <div className="grid grid-cols-2 md:grid-cols-4 gap-x-16 gap-y-10 items-center justify-items-center transition-all duration-300 group-hover:blur-[2.5px] group-hover:opacity-50">
-              {companies.map((company, companyIndex) => {
+              {companies.map((company, i) => {
                 const Icon = company.icon;
                 return (
                   <motion.div
-                    key={company.name}
+                    key={i}
                     initial={{ opacity: 0, scale: 0.8 }}
                     whileInView={{ opacity: 1, scale: 1 }}
                     viewport={{ once: true }}
-                    transition={{ duration: 0.4, delay: companyIndex * 0.05 }}
+                    transition={{ duration: 0.4, delay: i * 0.05 }}
                     className="text-foreground font-semibold text-xl flex items-center gap-2"
                   >
                     <Icon className="w-5 h-5 fill-current" />
diff --git a/apps/marketing/components/product-direction-section.tsx b/apps/marketing/components/product-direction-section.tsx
@@ -77,13 +77,13 @@ export function ProductDirectionSection() {
               <div className="absolute top-0 left-0 right-0 flex items-end">
                 {/* Tick marks row */}
                 <div className="flex items-end gap-[3px] absolute bottom-0 left-[5%] right-0">
-                  {Array.from({ length: 60 }, (_, tickIndex) => tickIndex).map((tickIndex) => (
+                  {Array.from({ length: 60 }).map((_, i) => (
                     <div
-                      key={tickIndex}
+                      key={i}
                       className="bg-zinc-600/60"
                       style={{
                         width: "1px",
-                        height: tickIndex % 7 === 0 ? "16px" : "8px",
+                        height: i % 7 === 0 ? "16px" : "8px",
                       }}
                     />
                   ))}
diff --git a/apps/marketing/components/ui-panel-inbox.tsx b/apps/marketing/components/ui-panel-inbox.tsx
@@ -87,9 +87,9 @@ export function UIPanelInbox() {
 
       {/* List */}
       <div className="divide-y divide-zinc-800/30">
-        {items.map((item) => (
+        {items.map((item, index) => (
           <div
-            key={`${item.id}-${item.title}-${item.time || "0"}`}
+            key={index}
             className="px-4 py-2.5 hover:bg-zinc-800/30 transition-colors cursor-pointer"
           >
             <div className="flex items-start gap-3">
diff --git a/apps/marketing/components/ui/carousel.tsx b/apps/marketing/components/ui/carousel.tsx
@@ -102,22 +102,19 @@ function Carousel({
     };
   }, [api, onSelect]);
 
-  const contextValue = React.useMemo(
-    () => ({
-      carouselRef,
-      api: api,
-      opts,
-      orientation: orientation || (opts?.axis === "y" ? "vertical" : "horizontal"),
-      scrollPrev,
-      scrollNext,
-      canScrollPrev,
-      canScrollNext,
-    }),
-    [api, canScrollNext, canScrollPrev, carouselRef, orientation, opts, scrollNext, scrollPrev],
-  );
-
   return (
-    <CarouselContext.Provider value={contextValue}>
+    <CarouselContext.Provider
+      value={{
+        carouselRef,
+        api: api,
+        opts,
+        orientation: orientation || (opts?.axis === "y" ? "vertical" : "horizontal"),
+        scrollPrev,
+        scrollNext,
+        canScrollPrev,
+        canScrollNext,
+      }}
+    >
       <div
         onKeyDownCapture={handleKeyDown}
         className={cn("relative", className)}
diff --git a/apps/marketing/components/ui/chart.tsx b/apps/marketing/components/ui/chart.tsx
@@ -46,10 +46,9 @@ function ChartContainer({
 }) {
   const uniqueId = React.useId();
   const chartId = `chart-${id || uniqueId.replace(/:/g, "")}`;
-  const contextValue = React.useMemo(() => ({ config }), [config]);
 
   return (
-    <ChartContext.Provider value={contextValue}>
+    <ChartContext.Provider value={{ config }}>
       <div
         data-slot="chart"
         data-chart={chartId}
diff --git a/apps/marketing/components/ui/field.tsx b/apps/marketing/components/ui/field.tsx
@@ -197,11 +197,7 @@ function FieldError({
 
     return (
       <ul className="ml-4 flex list-disc flex-col gap-1">
-        {errors
-          .filter((error): error is { message: string } => typeof error?.message === "string")
-          .map((error) => (
-            <li key={error.message}>{error.message}</li>
-          ))}
+        {errors.map((error, index) => error?.message && <li key={index}>{error.message}</li>)}
       </ul>
     );
   }, [children, errors]);
diff --git a/apps/marketing/components/ui/form.tsx b/apps/marketing/components/ui/form.tsx
@@ -33,10 +33,8 @@ const FormField = <
 >({
   ...props
 }: ControllerProps<TFieldValues, TName>) => {
-  const fieldContextValue = React.useMemo(() => ({ name: props.name }), [props.name]);
-
   return (
-    <FormFieldContext.Provider value={fieldContextValue}>
+    <FormFieldContext.Provider value={{ name: props.name }}>
       <Controller {...props} />
     </FormFieldContext.Provider>
   );
@@ -73,10 +71,9 @@ const FormItemContext = React.createContext<FormItemContextValue>({} as FormItem
 
 function FormItem({ className, ...props }: React.ComponentProps<"div">) {
   const id = React.useId();
-  const itemContextValue = React.useMemo(() => ({ id }), [id]);
 
   return (
-    <FormItemContext.Provider value={itemContextValue}>
+    <FormItemContext.Provider value={{ id }}>
       <div data-slot="form-item" className={cn("grid gap-2", className)} {...props} />
     </FormItemContext.Provider>
   );
diff --git a/apps/marketing/components/ui/toggle-group.tsx b/apps/marketing/components/ui/toggle-group.tsx
@@ -19,7 +19,6 @@ function ToggleGroup({
   children,
   ...props
 }: React.ComponentProps<typeof ToggleGroupPrimitive.Root> & VariantProps<typeof toggleVariants>) {
-  const contextValue = React.useMemo(() => ({ variant, size }), [size, variant]);
   return (
     <ToggleGroupPrimitive.Root
       data-slot="toggle-group"
@@ -31,7 +30,9 @@ function ToggleGroup({
       )}
       {...props}
     >
-      <ToggleGroupContext.Provider value={contextValue}>{children}</ToggleGroupContext.Provider>
+      <ToggleGroupContext.Provider value={{ variant, size }}>
+        {children}
+      </ToggleGroupContext.Provider>
     </ToggleGroupPrimitive.Root>
   );
 }
diff --git a/apps/mobile/ios/App/CapApp-SPM/README.md b/apps/mobile/ios/App/CapApp-SPM/README.md
@@ -1,88 +1,5 @@
-# CapApp-SPM (Capacitor Swift Package)
+# CapApp-SPM
 
-This directory is a generated Swift Package integration point for the native mobile
-shell. It is managed by Capacitor tooling and should be treated as a reproducible
-native dependency boundary.
+This package is used to host SPM dependencies for your Capacitor project
 
-## 1) What this package is
-
-- Package name: `CapApp-SPM`
-- Product: `CapApp-SPM`
-- Target(s): `CapApp-SPM`
-- Platforms: iOS 15+
-
-Primary goals:
-
-- provide a stable dependency entrypoint into the native layer,
-- keep native package metadata separate from app JS source trees,
-- avoid hand-editing generated package internals.
-
-## 2) Files here
-
-- `Package.swift`
-- `Sources/` (package targets/source)
-
-The current `Package.swift` includes local package links into Bun-managed `node_modules`
-paths, which is why it is intentionally generated rather than hand-maintained.
-
-## 3) Install and wire into an Xcode project
-
-1. In Xcode, open your `.xcworkspace` / `.xcodeproj`.
-2. Go to **File → Add Packages…**.
-3. Add by local path:
-   - `apps/mobile/ios/App/CapApp-SPM`
-4. Add the `CapApp-SPM` product to the appropriate app target.
-
-### Target mapping quick reference
-
-- Package: `CapApp-SPM`
-- Product: `CapApp-SPM`
-- Usually added to the main app target and the extensions that depend on Capacitor modules.
-
-## 4) Regenerating after dependency changes
-
-When web/native dependency versions change in `apps/mobile`, regenerate this package
-using your normal Capacitor sync workflow (outside this folder) and commit generated
-updates if required.
-
-Recommended sequence:
-
-```bash
-cd apps/mobile
-bun run sync
-cd ios/App
-# regenerate native project references through your normal mobile tooling
-```
-
-Then reopen Xcode and run package resolve.
-
-## 5) Build and debug checks
-
-### Before a build
-
-- Open the workspace cleanly.
-- Remove stale derived data if dependency resolution changed.
-- Validate package resolve on local machine first.
-
-### Common failure states
-
-- **Package not found / cannot resolve product**
-  - verify path points to `CapApp-SPM/` and Xcode selected the correct package product.
-- **Build fails with module resolution issues**
-  - clean build folder and re-resolve package dependencies.
-- **Code signing breaks after package edits**
-  - re-check provisioning profile and signing identity for native targets.
-
-## 6) CI/release implications
-
-For release and package integrity:
-
-- keep native package updates deterministic,
-- avoid ad-hoc local path edits that are not reflected in source control,
-- verify any package-manager-driven changes are generated and committed with the
-  corresponding release or mobile update.
-
-## 7) Reference
-
-This package is separate from release docs in [`docs/releases/README.md`](/Users/buns/.okcode/worktrees/okcode/okcode-ddc899c0/docs/releases/README.md),
-which controls desktop artifact publication and GitHub release behavior.
+Do not modify the contents of it or there may be unintended consequences.
diff --git a/apps/web/src/components/DiffPanel.tsx b/apps/web/src/components/DiffPanel.tsx
@@ -418,11 +418,9 @@ export default function DiffPanel({ mode = "inline" }: DiffPanelProps) {
     () => renderableFiles.map((fileDiff) => resolveFileDiffPath(fileDiff)),
     [renderableFiles],
   );
-  const activeReviewState = useMemo(() => {
-    return patchReviewSelectionKey
-      ? (reviewStateBySelectionKey[patchReviewSelectionKey] ?? {})
-      : {};
-  }, [patchReviewSelectionKey, reviewStateBySelectionKey]);
+  const activeReviewState = patchReviewSelectionKey
+    ? (reviewStateBySelectionKey[patchReviewSelectionKey] ?? {})
+    : {};
   const acceptedFileCount = useMemo(
     () =>
       renderableFilePaths.reduce(
diff --git a/apps/web/src/components/pr-review/RawPatchViewer.tsx b/apps/web/src/components/pr-review/RawPatchViewer.tsx
@@ -446,27 +446,14 @@ export function RawPatchViewer({ text, reason }: { text: string; reason: string
         ) : (
           /* Fallback: no file structure detected, render all lines with highlighting */
           <div className="overflow-hidden rounded-xl border border-border/70">
-            {lines
-              .reduce<Array<{ line: string; key: string; lineNumber: number }>>(
-                (items, line, lineIndex) => {
-                  const lineNumber = lineIndex + 1;
-                  items.push({
-                    line,
-                    lineNumber,
-                    key: `${lineNumber}-${line}`,
-                  });
-                  return items;
-                },
-                [],
-              )
-              .map(({ key, line, lineNumber }) => (
-                <RawPatchLine
-                  key={key}
-                  line={line}
-                  lineNumber={lineNumber}
-                  kind={classifyLine(line)}
-                />
-              ))}
+            {lines.map((line, index) => (
+              <RawPatchLine
+                key={index}
+                line={line}
+                lineNumber={index + 1}
+                kind={classifyLine(line)}
+              />
+            ))}
           </div>
         )}
       </div>
diff --git a/apps/web/src/store.ts b/apps/web/src/store.ts
@@ -260,28 +260,26 @@ export function syncServerReadModel(state: AppState, readModel: OrchestrationRea
             }
           : null,
         messages: thread.messages.map((message) => {
-          const attachments = message.attachments?.map((attachment) => {
-            const baseAttachment = {
-              id: attachment.id,
-              name: attachment.name,
-              mimeType: attachment.mimeType,
-              sizeBytes: attachment.sizeBytes,
-              url: toAttachmentPreviewUrl(attachmentPreviewRoutePath(attachment.id)),
-            };
-
-            if (attachment.type === "image") {
-              return {
-                type: "image" as const,
-                previewUrl: toAttachmentPreviewUrl(attachmentPreviewRoutePath(attachment.id)),
-                ...baseAttachment,
-              };
-            }
-
-            return {
-              type: "file" as const,
-              ...baseAttachment,
-            };
-          });
+          const attachments = message.attachments?.map((attachment) => ({
+            ...(attachment.type === "image"
+              ? {
+                  type: "image" as const,
+                  id: attachment.id,
+                  name: attachment.name,
+                  mimeType: attachment.mimeType,
+                  sizeBytes: attachment.sizeBytes,
+                  url: toAttachmentPreviewUrl(attachmentPreviewRoutePath(attachment.id)),
+                  previewUrl: toAttachmentPreviewUrl(attachmentPreviewRoutePath(attachment.id)),
+                }
+              : {
+                  type: "file" as const,
+                  id: attachment.id,
+                  name: attachment.name,
+                  mimeType: attachment.mimeType,
+                  sizeBytes: attachment.sizeBytes,
+                  url: toAttachmentPreviewUrl(attachmentPreviewRoutePath(attachment.id)),
+                }),
+          }));
           const normalizedMessage: ChatMessage = {
             id: message.id,
             role: message.role,
diff --git a/docs/releases/README.md b/docs/releases/README.md
