Reliability and Dependency Hell (Create Solid Development Setup)

[mudgen]

I just created a new solid project with: npm init solid app-ts my-app.

It worked perfectly. I started up the development server and all is nice. It looks great and I love it already.

But I noticed that creating the project added 1,805 npm packages to the project. So, in the future if one of these packages has an issue then my project might not work right?

I think this is probably the fault of the NodeJS ecosystem in general that has the convention to depend on hundreds or thousands of packages for projects. I just bring this to attention in case it is something that this project might want to mitigate in the future.

Or can someone tell me how this makes sense?

I do understand that the project that Solid sets up includes a lot of useful things like a testing library, development server, hot reload, babel, Typescript. But 1,805 dependencies still seems a lot to me. I know that it is similar for create-react-app.

[ryansolid]

The price you pay for using all the common DX libraries. This initial project is a fork of Create React App(https://github.com/facebook/create-react-app). They picked those packages to deliver what they felt was good DX experience for newcomers. I never use this for my own projects which are considerably smaller. I considered making my own starter but I doubt I would make something this comprehensive, atleast on my own.

I added this because it is familiar to people who have used CLI tools like this. It saved me from having to work through all the things someone might be look for. This is extensive considering a lot of concerns. I honestly don't even know how every part of it works it is that deep. Luckily React has good documentation.

[mudgen]

@ryansolid Thanks Ryan. What do you use for your own projects?

[s0kil]

It would be useful if Solid had a simple template starter similar to https://github.com/sveltejs/template.

[ryansolid]

Rollup but that is hardly the nicest DX. No dev-server. Less plugin support than Webpack. I feel like it isn't something I could just give to someone. I basically use whatever I feel like, but I also consider my self well versed. This is an area that a lot people find tricky. There are a number of starters people have made using Webpack as well (and even Parcel). If you are looking for a good TS starter this one was decent but a bit out of date:
https://github.com/atfzl/solid-starter

I often scan through here. There are atleast half a dozen starters but I haven't had a chance to review them all:
https://github.com/ryansolid/solid/network/dependents?package_id=UGFja2FnZS02MjAwMDg1Mg%3D%3D

[mudgen]

I understand. Thanks Ryan.

[ryansolid]

@s0kil I think that is Svelte's way around not having dedicated CLI. I don't mind that approach. But same stuff applies. It needs to be maintained and offer what people are looking for. I was hoping Create Solid handled that for us, but I guess it misaligns with peoples expectation of lightness.

[mudgen]

@ryansolid I appreciate Create Solid and I am using it for now. Thank you for setting it up! But I do like to keep dependencies to a minimum and I like to have some understanding about each dependency that I am using. In the future I probably will change my development setup.

@ryansolid It's possible that SolidJS might attract a different kind of developer than developers that stay with react.

[ryansolid]

Yeah I think so too on both accounts. I need to figure out how to better highlight community projects so that maybe it can naturally evolve to fit needs. I could just make a template on the main repo and people can start contributing but it needs guidance. We don't need me to set this up. It's a good task for someone else to do.

[ryansolid]

I agree in general so what way do people think we should take this:

1. Continue to work on a batteries included Create Solid Experience, work out options like Ejection.
2. a) Opt for a simpler "no config" with Parcel so modifying is on the end user.
3. b) Use a system like Rollup or Webpack with a simple config but you get what you get and there is no upgrade path.

[high1]

I've converted most of my starter webpack react config to solid without much hassle. I had to use a local rule to make ESLint counts JSX as usage. One thing remains - tests. I was using testing-library - and I think that @testing-library/dom could be used. Is there a minimal sample which would create a valid dom output from a component somewhere in solid source, that could be used for testing components in isolation?

[ryansolid]

Yeah you bring up a good point. I've always just did simple testing winging it with Jest but a nicer higher level structured library would be great. Ok.. well simple DOM elements are simple.

Since well:

const el: HTMLDivElement = <div />

// so if:
const MyComp: Component = () => <div />

// then also:
const el2: HTMLDivElement = <MyComp />

It's a little bit trickier with fragments but still holds:

const frag: [HTMLDivElement, HTMLDivElement] = <><div /><div /></>

And this holds true for all Components that return those. But the problem is we can also return functions and fragments of functions for dynamic parts. The only thing that resolves them currently is the insert method, which is called inside render currently. Since dynamic insertion often requires the parent to insert into. So passing something to render and mounting in an arbitrary element (doesn't need to be connected to the DOM) is the only way to ensure DOM elements.

const container = document.createElement("div");
render(() => <MyComp />, container);
// our element/s will be found in container.childNodes

Although I'm not sure I understand your question in terms of minimal sample. The test folder under Solid JS should have some examples. In there it is a bit older where I do stuff like this as well but it's basically the same thing:

const container = createRoot(() =>
  <div>
    <MyComp /> 
  </div>
)

Does that help?

[high1]

Here is the entry point to the application - it's minimal, but with all the boilerplate set up:

import { render, MountableElement } from 'solid-js/dom';

import Hello from 'components/Hello';

render(
  () => <Hello title="Welcome to Solid!" />,
  document.getElementById('root') as MountableElement
);

This works.

Then I tried to update the test I had for React.

import { render } from 'solid-js/dom';
import { getByText } from '@testing-library/dom';

test('renders without crashing', () => {
  const title = new Date().toTimeString();
  const renderMock = jest.fn((component, element) =>
    render(() => ({ ...component, props: { title } }), element)
  );
  jest.mock('solid-js/dom', () => ({
    render: renderMock,
  }));
  const root = document.createElement('div');
  root.setAttribute('id', 'root');
  document.body.appendChild(root);
  require('index');
  expect(renderMock).toHaveBeenCalledTimes(1);
  expect(getByText(root, 'root')).toHaveTextContent(title);
});

but this fails with:

dom_1.render(() => <Hello_1.default title="Welcome to Solid!"/>, document.getElementById('root'));
^

SyntaxError: Unexpected token '<'