feat: Feed (private alpha)

commit 296954896f722ba37b67ba4d77088a0b3336b9ef
Author: Tanner Linsley <[email protected]>
Date:   Sun Nov 30 21:46:31 2025 -0700

    checkpoint

commit 686548fc1fa64cfe4444504852f4bbbd260878f4
Author: Tanner Linsley <[email protected]>
Date:   Sun Nov 30 21:41:32 2025 -0700

    checkpoint

commit 6c17c1489158dae51a2cec39034ce5651535a743
Author: Tanner Linsley <[email protected]>
Date:   Thu Nov 27 10:57:49 2025 -0700

    checkpoint

commit b51b11fa3422a8c6b2c10785078598f1748192f6
Author: Tanner Linsley <[email protected]>
Date:   Mon Nov 24 22:11:17 2025 -0700

    checkpoint

Author: tannerlinsley

AGENTS.md

@@ -0,0 +1,69 @@
+# Agent Guidelines
+
+## Typesafety
+
+**Typesafety is of utmost importance.**
+
+### Avoid Type Casting
+
+We **never ever** cast types unless it's absolutely necessary. This includes:
+- Manual generic type parameters (e.g., `<Type>`)
+- Type assertions using `as`
+- Type assertions using `satisfies`
+- Any other form of type casting
+
+### Prefer Type Inference
+
+Always infer types and go up the logical chain as far as we can control to determine types. The preferred approach is:
+1. **Schema validation** - Use schema definitions (e.g., Convex schema, Zod, etc.) as the source of truth
+2. **Type inference from concrete sources** - Let TypeScript infer types from function return types, API responses, etc.
+3. **Go up the chain** - Trace types back to their source rather than casting at the point of use
+
+### Example
+
+❌ **Bad:**
+```typescript
+const result = api.getData() as MyType;
+const value = getValue<MyType>();
+```
+
+✅ **Good:**
+```typescript
+// Infer from schema or API definition
+const result = api.getData(); // Type inferred from api.getData return type
+const value = getValue(); // Type inferred from function implementation
+```
+
+If types need to be fixed, fix them at the source (schema, API definition, function signature) rather than casting at the point of use.
+
+## Route Loaders
+
+### loaderDeps Must Be Specific
+
+**loaderDeps must always be specific to what's actually used in the loader.**
+
+Only include the properties from `search` (or other sources) that are actually used in the loader function. This ensures proper cache invalidation and prevents unnecessary re-runs when unrelated search params change.
+
+❌ **Bad:**
+```typescript
+loaderDeps: ({ search }) => search, // Includes everything, even unused params
+loader: async ({ deps }) => {
+  // Only uses deps.page and deps.pageSize
+  await fetchData({ page: deps.page, pageSize: deps.pageSize })
+}
+```
+
+✅ **Good:**
+```typescript
+loaderDeps: ({ search }) => ({
+  page: search.page,
+  pageSize: search.pageSize,
+  // Only include what's actually used in the loader
+}),
+loader: async ({ deps }) => {
+  await fetchData({ page: deps.page, pageSize: deps.pageSize })
+}
+```
+
+This ensures the loader only re-runs when the specific dependencies change, not when unrelated search params (like `expanded`, `viewMode`, etc.) change.
+

agents/tasks/feed-admin-fixes.md

@@ -0,0 +1,72 @@
+# Feed Admin Feature - Fix List
+
+## Issues Found and Fixed
+
+### 1. Route Navigation - ✅ FIXED
+**File**: `src/routes/admin/feed.tsx`
+**Issue**: Using `/admin/feed/new` which doesn't exist as a route
+**Fix**: Changed to `/admin/feed/$id` with `params={{ id: 'new' }}`
+
+### 2. Navigation Search Params - ✅ FIXED
+**File**: `src/routes/admin/feed.tsx`
+**Issue**: Direct object assignment to search params
+**Fix**: Updated to use function form: `search: (s) => ({ ...s, ...updates })`
+
+### 3. Window Confirm - ✅ FIXED
+**File**: `src/routes/admin/feed.tsx`
+**Issue**: Using `confirm()` without `window.` prefix
+**Fix**: Changed to `window.confirm()`
+
+### 4. Type Error in FeedEntryEditor - ✅ FIXED
+**File**: `src/routes/admin/feed.$id.tsx`
+**Issue**: `entryQuery` could be `undefined` but type expects `FeedEntry | null`
+**Fix**: Added null coalescing: `entryQuery ?? null`
+
+### 5. Unused Imports - ✅ FIXED
+**Files**: Multiple admin files
+**Issue**: Unused imports causing warnings
+**Fix**: Removed unused imports (`useState`, `useEffect`, `validateManualEntry`)
+
+### 6. Library Filter Property - ✅ FIXED
+**File**: `src/components/admin/FeedEntryEditor.tsx`
+**Issue**: Accessing `lib.visible` property that doesn't exist
+**Fix**: Removed filter for non-existent property
+
+### 7. Actions Export - ✅ FIXED
+**File**: `src/components/admin/FeedSyncStatus.tsx`
+**Issue**: `api.feed.actions` doesn't exist in generated API
+**Fix**: Removed `syncAllSources` button and updated to use `useAction` hook. GitHub sync works via `api.feed.github.syncGitHubReleases`.
+**Note**: The `actions.ts` file exists but Convex may need to regenerate types. GitHub sync action works correctly.
+
+## Remaining Issues
+
+### 1. Actions Not Exported
+**File**: `convex/feed/actions.ts`
+**Issue**: Actions file exists but `api.feed.actions` is not available
+**Status**: Need to verify Convex file structure. Actions may need to be in root `convex/` directory or registered differently.
+
+### 2. Route Search Params Type Errors - ✅ FIXED
+**File**: `src/routes/admin/feed.tsx`
+**Issue**: TypeScript errors with search param updates
+**Fix**: Changed from `useNavigate()` to `Route.useNavigate()` and updated search param updates to use object spread instead of function form
+
+## Testing Status
+
+- ✅ Admin route requires authentication (working)
+- ✅ Create entry route exists (`/admin/feed/$id` with `id='new'`)
+- ✅ Edit entry route exists (`/admin/feed/$id`)
+- ⚠️ Cannot test CRUD operations without authentication
+- ⚠️ Sync status component has action import issues
+
+## Next Steps
+
+1. Fix Convex actions export structure
+2. Resolve route search param type errors
+3. Test with authenticated user:
+   - Create manual post
+   - View post in admin list
+   - Edit post
+   - Delete post
+   - Toggle visibility
+   - Toggle featured
+