The id field is not always called id, I think we need to get a model's id name in the template.

Good point. AFAICT, lb4 model does allow users to create a primary key property with a different name than id.

Example prompts & answers to use pk instead of id:

Model Product will be created in src/models/product.model.ts

Let's add a property to Product
Enter an empty property name when done

? Enter the property name: pk
? Property type: string
? Is pk the ID property? Yes
? Is it required?: No
? Default value [leave blank for none]:

I agree with @jannyHou that we should honor custom PK name, there should be a new test to verify. If this is too much work then I can live with adding support for custom PK names in a follow-up pull request.

If this is too much work then I can live with adding support for custom PK names in a follow-up pull request.

Yeah forgot to say, feel free to address it in another PR :)

Will do in a separate PR 👍

Should we support include for todoList: Pick<TodoList, 'id'>?

I'm fine to do it in a separate PR.

I am not sure what would you use include for? A kind of a white-list? That would make sense 👍

Let's discuss include and Pick in a separate PR. I think it's best to wait until there is (user) demand for that feature.