Most active commenters

    ←back to thread

    298 points sangeeth96 | 21 comments | | HN request time: 1.104s | source | bottom
    Show context
    simonw ◴[] No.46237795[source]
    React Server Components always felt uncomfortable to me because they make it hard to look at a piece of JavaScript code and derive which parts of it are going to run on the client and which parts will run on the server.

    It turns out this introduces another problem too: in order to get that to work you need to implement some kind of DEEP serialization RPC mechanism - which is kind of opaque to the developer and, as we've recently seen, is a risky spot in terms of potential security vulnerabilities.

    replies(10): >>46237967 #>>46238102 #>>46238147 #>>46239075 #>>46240339 #>>46240602 #>>46240620 #>>46240996 #>>46241208 #>>46242116 #
    tom1337 ◴[] No.46237967[source]
    I was a fan of NextJS in the pages router era. You knew exactly where the line was between server and client code and it was pretty easy to keep track of that. Then I've began a new project and wanted to try out app router and I hated it. So many (to me common things) where just not possible because the code can run in the client and on the server so Headers might not always be available and it was just pure confusion whats running where.
    replies(2): >>46238010 #>>46238602 #
    1. Uehreka ◴[] No.46238602[source]
    I think we (the Next.js user community) need to organize and either convince Vercel to announce official support of the Pages router forever (or at least indefinitely, and stop posturing it as a deprecated-ish thing), or else fork Next.js and maintain the stable version of it that so many of us enjoyed. Every time Next comes up I see a ton of comments like this, everyone I talk to says this, and I almost never hear anyone say they like the App Router (and this is a pretty contrarian site, so if they existed I’d expect to see them here).
    replies(8): >>46238811 #>>46239616 #>>46240521 #>>46240593 #>>46240938 #>>46241244 #>>46241298 #>>46242753 #
    2. hmcdona1 ◴[] No.46238811[source]
    I would highly recommend just checking out TanStack Router/Start instead. It fills a different niche, with a slightly different approach, that the Next.js app router just hasn't prioritized enabling anymore.

    What app router has become has its ideal uses, but if you explicitly preferred the DX of the pages router, you might enjoy TanStack Router/Start even more.

    replies(1): >>46241304 #
    3. berekuk ◴[] No.46239616[source]
    I've been using React since its initial release; I think both RSC and App Router are great, and things are better than ever.

    It's the first stack that allows me to avoid REST or GraphQL endpoints by default, which was the main source of frontend overhead before RSC. Previously I had to make choices on how to organize API, which GraphQL client to choose (and none of them are perfect), how to optimize routes and waterfalls, etc. Now I just write exactly what I mean, with the very minimal set of external helper libs (nuqs and next-safe-action), and the framework matches my mental model of where I want to get very well.

    Anti-React and anti-Next.js bias on HN is something that confuses me a lot; for many other topics here I feel pretty aligned with the crowd opinion on things, but not on this.

    replies(2): >>46239755 #>>46240113 #
    4. c-hendricks ◴[] No.46239755[source]
    Some of the anti-next might be from things like solid-start and tanstack-start existing, which can do similar things but without the whole "you've used state without marking as a client component thus I will stop everything" factor of nextjs.

    Not to mention the whole middleware and being able to access the incoming request wherever you like.

    replies(1): >>46240834 #
    5. codemonkey-zeta ◴[] No.46240113[source]
    Can you describe how rsc allows you to avoid rest endpoints? Are you just putting your rsc server directly on top of your database?
    replies(1): >>46240719 #
    6. reissbaker ◴[] No.46240521[source]
    Personally, I love App Router: it reminds me of the Meta monorepos, where everything related to a certain domain is kept in the same directory. For example, anything related to user login/creation/deletion might be kept in the /app/users directory, etc.

    But I really, really do not like React Server Components as they work today. I think it's probably better to strip them out in favor of just a route.ts file in the directory, rather than the actions files with "use server" and all the associated complexity.

    Technically, you can build apps like that using App Router by just not having "use server" anywhere! But it's an annoying, sometimes quite dangerous footgun to have all the associated baggage there waiting for an exploit... The underlying code is there even if you aren't using it.

    I think my ideal setup would be:

    1. route.ts for RESTful routes

    2. actions/SOME_FORM_NAME.ts for built-in form parsing + handling. Those files can only expose a POST, and are basically a named route file that has form data parsing. There's no auto-RPC, it's just an HTTP handler that accepts form data at the named path.

    3. no other built-in magic.

    replies(1): >>46240997 #
    7. stack_framer ◴[] No.46240593[source]
    I find myself just wanting to go all the way back to SPAs—no more server-side rendering at all. The arguments about performance, time to first paint, and whatever else we're supposed to care about just don't seem to matter on any projects I've worked on.

    Vercel has become a merchant of complexity, as DHH likes to say.

    replies(2): >>46240770 #>>46241002 #
    8. berekuk ◴[] No.46240719{3}[source]
    If I control both the backend and the frontend, yes. Server-only async components on top of layout/page component hierarchy, components -> DTO layer -> Prisma. Similar to this: https://nextjs.org/blog/security-nextjs-server-components-ac...

    You still need API routes for stuff like data-heavy async dropdowns, or anything else that's hard to express as a pure URL -> HTML, but it cuts down the number of routes you need by 90% or more.

    replies(1): >>46242761 #
    9. farley13 ◴[] No.46240770[source]
    I think the context matters here - for SEO heavy marketing pages I still see google only executing a full browser based crawl for a subset of pages. So SSR matters for the remainder.
    10. kyleee ◴[] No.46240834{3}[source]
    And vercel
    11. bryanrasmussen ◴[] No.46240938[source]
    OK I am personally surprised that anyone likes the Pages router? Pages routing has all the benefits (simple to get started the first time) and all the downsides (maintainability of larger projects goes to hell) of having your routing being determined by where in the file system things are.

    I don't care about having things simple to get started the first time, because soon I will have to start things a second or third time. If I have a little bit more complexity to get things started because routing is handled by code and not filesystem placement then I will pretty quickly develop templates to handle this, and in the end it will be easier to get things started the nth time than it is with the simple version.

    Do I like the app router? No, Vercel does a crap job on at least two things - routing and building (server codes etc. can be considered as a subset of the routing problem), but saying I dislike app router is praising page router with too faint a damnation.

    12. robertoandred ◴[] No.46240997[source]
    Route files are still RSCs. Actions/“use server” are unrelated.
    replies(1): >>46241299 #
    13. robertoandred ◴[] No.46241002[source]
    SPAs can still be server rendered.
    14. morsmodr ◴[] No.46241244[source]
    Remix 2 is beautiful in its abstractions. The thing with NextJS Roadmap is that it is tightly coupled with Vercel's financial incentives. A more complex & more server code runs ensure more $$$ for them. I don't see community being able to do much change just like how useContextSelector was deprioritized by the React Core team.

    Align early on wrt values of a framework and take a closer look at the funder's incentives.

    15. awestroke ◴[] No.46241298[source]
    We're migrating away from both Next and Vercel post-haste
    replies(1): >>46241360 #
    16. reissbaker ◴[] No.46241299{3}[source]
    Route files are no different than the pages router that preceded them, except they sit in a different filepath. They're not React components, and definitely not React Server Components. They're not even tsx/jsx files, which should hint at the fact that they're not components! They just declare ordinary HTTP endpoints.

    RSCs are React components that call server side code. https://react.dev/reference/rsc/server-components

    Actions/"use server" functions are part of RSC: https://react.dev/reference/rsc/server-functions They're the RPC system used by client components to call server functions.

    And they're what everyone here is talking about: the vulnerabilities were all in the action/use server codepaths. I suppose the clearest thing I could have said is that I like App Router + route files, but I dislike the magic RPC system: IMO React should simplify to JSON+HTTP and forms+HTTP, rather than a novel RPC system that doesn't interoperate with anything else and is much more difficult to secure.

    17. cjonas ◴[] No.46241304[source]
    Last time I tried tanstack router, I spent half a day trying to get breadcrumbs to work. Nit: I also can't stand their docs site.
    18. Seattle3503 ◴[] No.46241360[source]
    What are you migrating to? Vanilla React?
    replies(1): >>46242990 #
    19. spoiler ◴[] No.46242753[source]
    Probably an unpopular take, but I really think Vercel has lost the plot. I don't know what happened to the company internally. But, it feels like the first few, early, iterations of Next were great, and then it all started progressively turning into slop from a design perspective.

    An example of this is filesystem routing. Started off great, but now most Next projects look like the blast radius of a shell script gone terribly wrong.

    There's also a(n in)famous GitHub response from one of the maintainers backwards-rationalising tech debt and accidental complexity as necessary. They're clearly smart, but the feeling I got from reading that comment was that they developed Stockholm syndrome towards their own codebase.

    20. skydhash ◴[] No.46242761{4}[source]
    You’re just shifting the problem from HTTP to an adhoc protocol on top of it.
    21. awestroke ◴[] No.46242990{3}[source]
    Vanilla react, ts-rest