Skip to content

Conversation

@KyleAMathews
Copy link
Contributor

No description provided.

KyleAMathews and others added 6 commits September 3, 2025 07:41
- Add src/lib/electric-proxy.ts with prepareElectricUrl() and proxyElectricRequest()
- Update API routes (projects, users, todos) to use new abstraction
- Reduces ~50 lines of duplicated code across 3 routes
- Centralizes Electric Cloud auth and proxy configuration
- Includes proper Vary header handling for cache invalidation

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Add authStateCollection for caching auth state in memory
- Implement smart baseURL detection (browser vs server)
- Add beforeLoad with auth state caching to prevent repeated auth checks
- Add comprehensive error component with graceful degradation
- Eliminates flash of loading states and improves performance

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Add Core Pattern description and data flow architecture
- Include detailed patterns for reading/writing data
- Add Critical Rules section with key constraints
- Document collection definition patterns with examples
- Add naming conventions and schema management guidance
- Provide component patterns for consistent development

Improves developer onboarding and prevents common mistakes.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Enable defaultPreload: 'viewport' in router configuration
- Preloads routes when they come into viewport
- Significantly improves perceived performance with minimal overhead
- Routes load instantly when user navigates to them

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Import ELECTRIC_PROTOCOL_QUERY_PARAMS from @electric-sql/client
- Replace hardcoded array with official constant
- Ensures compatibility with Electric SQL client updates
- More maintainable and follows best practices

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Use correct error code USER_ALREADY_EXISTS_USE_ANOTHER_EMAIL
- Better error message display using error.message
- Improved console logging for debugging
- More robust error handling in catch block

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@codecov
Copy link

codecov bot commented Sep 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.22%. Comparing base (c978ee6) to head (6d45c95).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3091      +/-   ##
==========================================
+ Coverage   75.85%   77.22%   +1.36%     
==========================================
  Files         163      170       +7     
  Lines        8355     9050     +695     
  Branches      106      280     +174     
==========================================
+ Hits         6338     6989     +651     
- Misses       2016     2059      +43     
- Partials        1        2       +1     
Flag Coverage Δ
elixir 76.00% <ø> (-0.06%) ⬇️
elixir-client 73.87% <ø> (-0.54%) ⬇️
packages/experimental 87.73% <ø> (ø)
packages/react-hooks ?
packages/typescript-client 93.49% <ø> (?)
packages/y-electric 55.12% <ø> (ø)
postgres-140000 76.12% <ø> (-0.06%) ⬇️
postgres-150000 75.96% <ø> (-0.02%) ⬇️
postgres-170000 76.10% <ø> (+0.20%) ⬆️
sync-service 76.22% <ø> (ø)
typescript 85.75% <ø> (+13.35%) ⬆️
unit-tests 77.22% <ø> (+1.36%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

KyleAMathews and others added 9 commits September 3, 2025 08:39
- Fix TipTap editor type issues in linearlite example with proper type assertions
- Fix react-beautiful-dnd JSX component type errors in both linearlite examples
- Fix react-window List JSX component type errors in both examples
- Fix y-electric BodyInit type compatibility with Uint8Array
- Fix crypto API BufferSource type compatibility in encryption example
- Convert SST config to use dynamic imports instead of top-level imports

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Sep 3, 2025

Examples

  • linearlite-read-only: deploy failed
  • burn: deploy failed

@thruflo
Copy link
Contributor

thruflo commented Sep 9, 2025

N.b.: I think AGENTS.md is now more standard than AGENT.md.

@thruflo
Copy link
Contributor

thruflo commented Sep 9, 2025

FYI: e6f1ab7 without it netlify complains when deployed via CLI (it's fine via Github deployment):

Sep 9, 04:19:34 PM: ff83f607 ERROR  [request error] [unhandled] [GET] https://hilarious-seahorse-f27f1f.netlify.app/
 Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/var/task/node_modules/zod/index.js' imported from /var/task/chunks/_/ssr.mjs
    at finalizeResolution (node:internal/modules/esm/resolve:281:11)
    ... 7 lines matching cause stack trace ...
    at ModuleJob._link (node:internal/modules/esm/module_job:137:49) {
  cause: Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/var/task/node_modules/zod/index.js' imported from /var/task/chunks/_/ssr.mjs
      at finalizeResolution (node:internal/modules/esm/resolve:281:11)
      at moduleResolve (node:internal/modules/esm/resolve:866:10)
      at moduleResolveWithNodePath (node:internal/modules/esm/resolve:990:14)
      at defaultResolve (node:internal/modules/esm/resolve:1033:79)
      at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:780:12)
      at #cachedDefaultResolve (node:internal/modules/esm/loader:704:25)
      at ModuleLoader.resolve (node:internal/modules/esm/loader:687:38)
      at ModuleLoader.getModuleJobForImport (node:internal/modules/esm/loader:305:38)
      at ModuleJob._link (node:internal/modules/esm/module_job:137:49) {
    code: 'ERR_MODULE_NOT_FOUND',
    url: 'file:///var/task/node_modules/zod/index.js'
  },
  statusCode: 500,
  fatal: false,
  unhandled: true,
  statusMessage: undefined,
  data: undefined
}

@thruflo
Copy link
Contributor

thruflo commented Sep 10, 2025

Also FYI, draft @electric-sql/quickstart package wrapping this: #3114

- Add comprehensive Quick Start section with clear 4-step timeline
- Add troubleshooting table for common pitfalls (Docker, Caddy, ports)
- Add detailed Caddy rationale explaining HTTP/2 benefits for Electric shapes
- Add production deployment checklist with auth security warnings
- Add complete 'Adding a New Table' walkthrough with 7-step process
- Extract generateTxId to shared trpc.ts file (DRY up duplicate code)
- Add migrate:generate script for cleaner migration workflow
- Add core architecture rules explaining Electric/tRPC/collection patterns
- Update .env.example with production environment variables and comments
- Remove unused @tanstack/react-router-with-query dependency

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@thruflo
Copy link
Contributor

thruflo commented Sep 10, 2025

@thruflo made your suggested changes + pushed some more improvements — this should be ready to merge now

Cool, I'll just have a go at using it.

@KyleAMathews
Copy link
Contributor Author

@thruflo can you approve?

@KyleAMathews KyleAMathews merged commit e2dc275 into main Sep 10, 2025
27 of 30 checks passed
@KyleAMathews KyleAMathews deleted the starter-improvements branch September 10, 2025 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants