HARI KAPADIA comments

Results 12 comments of


                                            HARI KAPADIA

perf(turbo-task-fs): use batch fs op to reduce fs latency

Fascinating performance optimization! This is a sophisticated approach to reducing filesystem latency. Let me provide an in-depth analysis: ## 🎯 **Core Innovation** **Problem:** Individual blocking FS operations create latency overhead,...

docs: Add custom labels configuration example

Excellent documentation addition! 🎯 This is a super practical tip that many developers will appreciate. Having multiple `page.tsx` or `layout.tsx` tabs open can be confusing, and this VS Code/Cursor configuration...

fix: handle notFound in Suspense with cacheComponents enabled

Excellent fix for a tricky edge case! 🎯 **Problem Identified:** When `cacheComponents: true` is enabled and `notFound()` is thrown inside a Suspense boundary, the RSC stream was being sent directly...

fix: patch fetch() signature instead of extending RequestInit

Great fix! 👍 This approach is much cleaner than extending `RequestInit` globally. By patching the `fetch()` signature directly with `NextFetchRequestInit`, you're: 1. ✅ Preventing the `next` option from leaking into...

fix: otel error spans from streamed responses

Excellent fix for a subtle OpenTelemetry issue! This addresses a fundamental challenge with streaming and observability. Let me provide a deep analysis: ## 🎯 **Problem Statement** **The Core Issue:** Errors...

fix: otel error spans from streamed responses

## 💡 **SOLUTION: Complete Span Lifecycle Management** Based on my ultra-deep analysis, here's a **production-ready solution** with proper span cleanup and error handling: --- ## 🔧 **COMPLETE IMPLEMENTATION** ```typescript //...

examples: Update example commands in README for with-emotion

Perfect fix! ✅ The README was incorrectly referencing `with-emotion-swc` instead of `with-emotion`. This correction ensures users can actually run the example commands successfully. Simple, straightforward, and necessary. LGTM! 🚀

avoid double encoding path params

Interesting approach to the double-encoding issue! 🤔 **Problem:** When a URL parameter is already decoded (like `%` from `%25`), calling `decodeURIComponent()` again throws an error. **Your Solution:** Re-encode the param...

Deprecate custom `use-local-storage` composable in favor of `vueuse`

Hi! I'd like to work on this issue. I'll migrate the custom `useLocalStorage` composable to use `@vueuse/core`'s `useLocalStorage` instead. The migration plan: 1. Replace all imports of the custom composable...

Better error log for router state header in parseAndValidateFlightRouterState

Great improvement to error diagnostics! 🎯 **What This Fixes:** Makes debugging router state header issues much easier by including actual data in error messages instead of generic errors. **Strengths:** ✅...