Customization of folder structure for a template

[osher]

Description

I'm not sure I'm in the right place, please bear with me...

1. I'm evaluating the tools for a nodejs team.
2. I need to customize the folder structure: add a generated tests folder, separate the web-controllers from the "services" (uh, we don't call them services in node...)
   and such.

For example,

eval-codegen-nodejs
│   .swagger-codegen-ignore
│   index.js
│   LICENSE
│   package.json
│   README.md
├───api
│       swagger.yaml
└───controllers
        <name>.js
        <name>Service.js

to:

eval-codegen-nodejs
│   .swagger-codegen-ignore
│   LICENSE
│   package.json
│   README.md
├───bin
│   └──server.js
├───scheme
│   └──scheme.yaml
├───lib
│   └──<name>.js
├───web
│   └──<name>.js
└─── test
    ├───lib.web.index.js
    └───lib.    

I don't find any documentations about it :(
Can anybody point me to the right direction?

Swagger-codegen version

cloned from master + a PR I applied (which was merged to master)
So call it 2.2.1-latest?

Swagger declaration file content or url

This is not a bug the spec file produces.

Command line used for generation

java -jar swagger-codegen\modules\swagger-codegen-cli\target\swagger-codegen-cli.jar generate -l nodejs-server -o eval-codegen-hello -i hello.openapi.yaml -t swagger-codegen-nodejs

(windows user)

Steps to reproduce

It works, but I want it to work differently.
I want to control the folder-structure.

Related issues

none

Suggest a Fix

Maybe it's a completely new feature request.

Ideally,
As a user - I want to provide a path, and a list of a per-controller files (call it option 1)
or - alternatively - a project dir path, a controller-router dir path (call it option 2)

Then, the expected behavior would be:

every file in the project dir recursively is bound with the model once and rendered to the target folder, preserving relative path it's found in the source dir

if we're doing 1st option - the files listed as per-controller-router files are excluded of the first round, but are bound once per controller-router with a model that features also a reference to the current operation (can be null when binding project files)

if we're doing 2nd option - then, recursively, per controller-router - every file in the controller dir path is bound with the model once and rendered to the target folder, again preserving relative path it's was found in the source dir
(2nd option probably allow more code-reuse with the folder-structure recursion and all...)

I deem this will grant enough control to use codeGen to provide any folder structure.

[osher]

I've done some more research.
It looks like the nodejs scene has progressed a lot since last the nodejs templates that are suggested here were created.

Currently, there are two main generators in the swagger family: node-swagger and this project.

Comparison in short:

This project is more advanced in the manner that it generates handlers capable of responding mock replies based on the responses declared in the spec, where node-swagger generates only a boilerplate from which the developer has to proceed by himself.
However, the node-server generated by codeGen renders a solution that is a very low-level project and in modern terms - even partial.
Node-swagger also generates tests, and acts as a minimalist CLI task-runner that lets you for instance in one command launch the server and the swagger-editor.

So codeGen is more advanced in that it creates a server capable of returning mock replies
but node-swagger is more advanced in the tools that it brings to the generated project, generation of tests, the folder-structure it lets you work with, and the CLI workflow helpers.

By supporting folder-structure customization, IMHO - this project could gain an additional edge aside to the mock responses.
If I could customize the folder structure like I described above - I could create a template for this engine that renders a project that brings modern tools AND is capable of rendering mock replies.
I suppose that with the right configuration - one could still use node-swagger's CLI helpers - and enjoy all benefits.

Thoughts?

[wing328]

@osher thanks for the feedback on the nodejs-server generator.

For nodejs server-side project, is there a recommended folder structure for the project?

Technically it's possible to introduce CLI options to customize the folder for different auto-generated JS files but I think it's important to understand if there's any best practices for nodejs folder structure.

For auto-generated test cases, we have that for other language generator (but not all), e.g. Ruby, Python, PHP, C#, Java, etc: #2276 and we're looking for contribution to add those for nodejs-server. Let me know if you want to contribute on that.

[osher]

For nodejs server-side project, is there a recommended folder structure for the project?

Such a typical question for Java folk ;) anyway - Good question, I'm happy you asked.
You deserve a good answer, however I'll need a little time to write it well.
Sorry to keep you waiting - I now just wanted to let you know I heard you

I may not be able to post it today, and this is the end of my week. If not to day - I'll post it on sunday.

[wing328]

@osher please take your time. We always welcome feedback to make Swagger Codegen better. Your suggested folder structure looks good to me.

What I would like to do it to make sure we're following the NodeJS industry standard/style guide, if any, to avoid changing it again in the future.

[osher]

First - disclaimer:

Any folder-structure would be _opinionated_ - and JS developers are often kinda ...gently put - sensitive to opinionsm.
Bottom line is that if you want to address the lion share of the JS community - you want to avoid opinionist structure, and just let them do what the heck the tools they use require.
It's a template after all - a template for a generator.
Whatever they decide on - will become the repeated form they work with - and it's OK.

To show you around the versatility in the JS world, I will provide 2 structures bellow, but please hear me here:

honest recommendation

I recommend to strive to decouple the generator logic from the folder structure

My heart tells me that if you can do that - you won't need so many generator classes as you now have in the project, because you will work out of the generic base common to all of them. But that's a side-effect of my proposition, it's not my goal.
But if we're already here: We want that all the differences between the projects will be described in the source template folder where now it's described partly in template files and partly in hardcoded generator classes. IMHO - for no good reason.
(oh, you'd still be able to inherit the base class and override it's method to customize it by need, but in majority of cases - why would you...).

According to this recommendation - the feature would work best if a source folder structure is cloned, and by recursive iteration - every file in that source structure is bound to it's model and rendered into the relative path in the target.

In this I can distinct between 2 types of source template files, according to the model they require and the times they are bound

• once per project - with model binding that reflect only the api-docs.
  this is the README, the package.json, the .gitignore, - i.e - generic files (like POM.xml in maven projects)
• once per entry in api-doc.paths - with model that has a currentPath entry.
  this is for the router and controller - or as you call it here - service.
  (one could argue that per-operation is also a type, but for now I see it as a subsection of per-path, so I don't count it. If some designs would like each operation in it's own file - let them name it's own controller-router).

mm. well. my bad. I missed in my original post a 3rd type.
As a JS developer I did not consider it, but I can see cases where it may be needed, especially when I try to think how I would use the same generator to produce typed C#/Java/C++/Erlang/etc... projects.
So the 3rd type is:

• once per entry in api-doc.definition - with a model that has currentDefinition entry

The user wants e2e tests? great. The template structure will include test files, bound per apiDoc.path.
The user wants model tests? great. The template structure will include test files, bound per apiDoc.definition.
The user wants serializers? encoders? utilities? great. same: let them add a template. We just need to support binding with the relevant part of the api-doc model, and we don't need a generator per language.
All we need to know is if the template should be bound only with swagger document, or bound once per project, once per-path, or once per definition.

Well. There could be a discussion about case in which same controller handles few paths.
I have a strong feeling you've solved this modeling problem already.
However you do it - I refer the same model and same iteration times as depicted by the current logic.
(assuming the x-controller-router & multiple tags bug is solved... 😄 )

sample folder structures

So now, like I promised - here are two options of folder structures.

what is provided by the swagger-node

The swagger-node supports 5 templates for 5 competing nodejs frameworks: connect, express, restify, hapi, sails.
The first 4 result with the exact same structure, where the difference is what libs are loaded in the main file, how they are initiated, and how they are accessed by the cotroller & service - so it's basically slightly different text in same files.
The sails is more bloated with sails instrumentation, I don't want to get into that. But if we support dynamic folder structure - we cover it too with ease.

The command is swagger project create --framework <framework>
The result is:

C:.<foo>
├───api
│   ├───controllers
│   │       hello_world.js
│   │       README.md    - sais: "Place your controller files in this directory"
│   ├───helpers
│   │       README.md     - sais: "Place your helper files in this directory"
│   ├───mocks
│   │       README.md     - sais: "Place your mock files in this directory"
│   └───swagger
│           swagger.yaml
├───config
│       default.yaml
│       README.md        - sais: "Place your config files in this directory"
└───test
    └───api
        ├───controllers
        │       hello_world.js
        │       README.md     - sais: Place your controller tests in this directory
        └───helpers
                README.md      - sais: Place your helper tests in this directory
    .gitignore
    app.js
    package.json
    README.md

However, the boiler-plate there is very simplistic. It does not read a spec-file, and does not use a templating engine to bind it. It's purpose is just to show you an opinionated way (which is already a pitfall in the JS community).
The project probably still is in early steps in it's evolution. It's very clean, but the prepared extensibility points are clear the keen eye - so there's lot's of work there.

Anyway - this structure, contradict the opinions in my school.
So here:

What I would do if I had the freedom to do it

First - we use a flat test directory. Any directories in the test folder are reserved to test tools, fixtures and mocks, and paths of test targets are encoded in file-names . instead of /.
Second - we put all the files that need coverage tests and JsDoc generation in ./lib, where ./lib/index is the main file in the pacackage.
If the package is a service - then a file in ./bin will require ./lib, and pass to it arguments extracted from shell context (like switches, configs read from disk and env-variables). This file in the ./bin will also handle SIGINT and SIGTERM signalls, initiate loggers, and other context-related works.
If the package is an ETL job - same. Or a CLI tool to setup some things. donno. all the same :)
All files in ./bin execute things and are tested in e2e/integration tests.
All files in ./lib declare modules that may be require()ed. are tested by unit tests, tracked by code-coverage, and JsDocs are generated for the modules they declare.

Having that said - assuming the package is named foo - we end with:

├─── bin
│       foo-server
│       foo-etl                         //we don't care about this one - it won't be generated
│       foo                             //we don't care about this one - it won't be generated
├─── config
│   ├─── default
│   │      config.yaml  
│   └─── {env}                  //this will also never be generated. it's domain specific
│          config.yaml  
├─── lib
│   ├─── dal
│   │   {persistence/service/whatever}.js  //this will also never be generated...
│   ├─── model
│   │   {definition}.js
│   └─── web
│       {controller}.js
│       index.js
├─── scheme
│    api-docs.yaml / api-docs.json    //howevre we got it in the -i switch
├─── test
│       lib.model.{definition}.test.js
│       lib.web.{controller}.test.js
│       mocha.opts
└─── test-e2e
        foo.{path}.test.js
        foo-etl.test.js                           //not generated
        foo.test.js                               //not generated
    .gitignore
    .npmignore
    CHANGELOG.md
    package.json
    README.md

I saw somewhere that you already support a flag to suppress Model generations.
That's good. Most of loosly-type languages (like JS) won't need models.
However, models may be facilitators that aggregate the raw JSON.parse()ed object and facilitate traversing in it, and their generation is usually domain-specific - and that's where the power custom-mustache should come in :D

[randavidovitz]

I also agree that we need to have different folder structure that is more up to date, as a default I would say above will be good starting point

I am not sure its important on phaze 1 to enable client to decide on different folder structure other than above

Must share with you that this project remind me of Yoman

[osher]

I thought of another way to tell what model should be bound with what file.

The proposition relays on encoding in the file name all the information about how it should be bound.
This leaves us with just one folder

• if it ends with $scheme.mustache - it's bound with the scheme.
• if it ends with $controller.mustache - it's bound with the controller
• if it ends with $model.mustache - it's bound with the model
• if it ends with .partial - it's not rendered by itself, but is used as a partial in other templates.
  if a folder contains only partials -its excluded from the cloning of folder-structure to target directory.
• Any other ways - it's coppied as is from the source directory to the target, preserving it's relative path.

file-names should also be bound with placeholders, for {{controller}} and {{definition}}

Examples:
assume 1 api doc with 8 operations in 4 paths handled by 2 controllers, and 3 definitions:

• ./README.md$scheme.mustache
  • renders ./README.md once, bound with the apidoc.
• ./lib/web/{{controller}}.js$controller.mustache
  • renders ./lib/web/ctrl-1.js and ,./lib/web/ctrl-2.js
  • bound with their respective controller binding model (The 8 operations are covered between these 2 controllers).
• ./test/model.{{definition}}.test.js$definition.mustache
  • renders ./test/model.view-1.test.js, ./test/model.view-2.test.js and ./test/model.view-3.test.js
  • bound with their respective definition binding model.

Please excuse me - I did not take the time to find the entry names you currently use for {{controller}} and {{definition}} - I really have to call it a day (and a week actually)
Buy as a semantic person - I'd love to see them adopted and supported side by side to their current names...

I'm not quite happy with the part in the proposition that handles partials.
maybe the template directory needs to be split to ./main and ./partials or something in that spirit,
but I really have to go.

I'd be happy to hear your thoughts.

Last - I presented my research here in the organization, and showed them the fast responses I got from you. Although the last time I really did Java it was with Java1.2, this organization have a strong Java team, and they are currently considering allocating some dev-time to contribute a PR with this functionality.
How does that sound to you?

[fehguy]

There are definitely many opinions for this. I suggest avoiding making this too smart right now, and target a 3.0 release. As it stands, the technique to change folder structures is embedded in java logic and until we do a major redo on the generation step, let's put this on hold.

[osher]

@fehguy: I think I understand.
However, we might want to fork and have a start anyway the effort, from which a PR could be applied. It's pending leadership decision here.

In that storyline of developing a template for ourselves - as a first step, my POC is to create a template that uses modern node-js tools for the existing NodeJS server generator.
That will be done right after selecting between express, restify, and hapi. (connect is too low level, and sails is overbloated for us).
Would you like this as a separate PR?

[JeffML]

I, for one, would just like to see the mocks separated from the controllers, and have the controllers call the mocks based on an environment variable. Something like (pseudocode):

var mock = process.env(‘mock’);

if (mock) {
	exports = require(‘./mocks/<thisfilename>’)
} else {
... /* real service here */
}

I've poked around a bit at the the languages source code, to figure out how I would generated a mock folder; looks like the override apiFileFolder() determines the (single) output folder for the controller api, used by AbstractJavaJAXRSServerCodegen.java. This is just the start of the rabbit hole, I'm pretty sure.