wasp/waspc
2024-08-02 13:44:10 +02:00
..
cli Fix db reset failing after wasp clean (#2177) 2024-07-15 17:25:26 +02:00
data Replace oslo/password with directly using @node-rs/argon2 (#2210) 2024-08-02 13:44:10 +02:00
docs fixed typos in waspc/design-docs/db-seeding.org and waspc/design-docs/server-setup.md (#1499) 2023-10-17 17:36:29 +02:00
e2e-test Replace oslo/password with directly using @node-rs/argon2 (#2210) 2024-08-02 13:44:10 +02:00
examples Replace oslo/password with directly using @node-rs/argon2 (#2210) 2024-08-02 13:44:10 +02:00
headless-test Using Prisma file alongside Wasp (#2035) 2024-07-03 13:51:29 +02:00
images Replace Wasp with AppSpec and Parser with Analyzer. (#423) 2022-01-20 11:45:14 +01:00
packages Bump braces from 3.0.2 to 3.0.3 in /waspc/packages/studio (#2147) 2024-07-03 13:45:07 +02:00
src Replace oslo/password with directly using @node-rs/argon2 (#2210) 2024-08-02 13:44:10 +02:00
test Adds support for view and type blocks (#2179) 2024-07-15 13:57:53 +02:00
tools Made examples/todoApp more approachable (+ documented what it should be used for) (#2094) 2024-06-18 21:02:10 +02:00
waspls Make sure entity declaration is not used in the Wasp file (#2152) 2024-07-09 23:44:56 +02:00
.gitattributes Add test framework for language server autocompletion (#1223) 2023-06-01 11:03:01 -04:00
.gitignore Make CLI code testable (#510) 2022-03-23 08:04:09 -04:00
.hlint.yaml Fixed old hlint warnings/suggestions. 2024-01-08 15:54:10 +01:00
.nvmrc Relax node version (#1533) 2024-01-04 11:30:42 +01:00
.stylish-haskell.yaml wasp start now does setup and start while forwarding output. 2020-09-11 17:33:10 +02:00
cabal.project Implemented wasp start db (managed dev db) (#1044) 2023-03-21 16:37:20 +01:00
ChangeLog.md Update date in ChangeLog 2024-07-17 16:27:43 +02:00
dev-tool.project Implemented wasp start db (managed dev db) (#1044) 2023-03-21 16:37:20 +01:00
LICENSE Moved all code into waspc directory. 2020-02-20 12:30:58 +01:00
new-release fixed e2e tests, fixed release script. 2022-11-22 19:31:43 +01:00
Operation.js Added fixing of NodeJS operations files. 2023-06-28 22:04:57 +02:00
README.md Made examples/todoApp more approachable (+ documented what it should be used for) (#2094) 2024-06-18 21:02:10 +02:00
run Updated run script to also run headless tests. 2024-02-19 14:28:12 +01:00
Setup.hs Formatted whole codebase with ormolu. 2021-04-28 17:44:00 +02:00
testEnv.hs Formatted whole codebase with ormolu. 2021-04-28 17:44:00 +02:00
waspc.cabal Add css wildcard for template files (#2180) 2024-07-15 14:49:14 +02:00

Waspc

This directory contains source code of the wasp compiler (aka waspc), and this README is aimed at the contributors to the project.

If you are a Wasp user and not a contributor (yet :)), you might want to look into following resources instead (Project page, Docs).

First time contributor checklist

If you would like to make your first contribution, here is a handy checklist we made for you:

  • Read Quick overview.
  • Compile the project successfully and get todoApp example running (follow Basics).
  • Join Discord and say hi :)!
  • Pick an issue labeled with "good first issue" and let us know you would like to work on it - ideally immediatelly propose a plan of action and ask questions. If you can't find a suitable issue for you, reach out to us on Discord and we can try to find smth for you together.
  • Make a PR targeting main and have it accepted! Check Typical workflow and Branching and merging strategy for guidance, and consult Codebase overview for more details on how Wasp compiler works internally.

Quick overview

Wasp compiler is implemented in Haskell, but you will also see a lot of Javascript and other web technologies because Wasp compiles it's own code into them.

You don't have to be expert in Haskell to contribute or understand the code, since we don't use complicated Haskell features much -> most of the code is relatively simple and straight-forward, and we are happy to help with the part that is not.

Main result of building the project is wasp executable (also reffered to as CLI), which is both Wasp compiler, CLI and Wasp project runner in one - one tool for everything Wasp-related.

wasp executable takes .wasp files and ext/ dir as input and generates a web app from them.

Wasp compilation

It can then also run that web app for you, deploy it (not yet but that is coming), and manage it in other ways.

Basics

Setup

We use the cabal command line tool to build the project. The best way to install it is via ghcup. The ghcup tui terminal user interface is a convenient way of installing and selecting versions of cabal, hls and ghc.

In cabal.project we're explicitly saying what compiler version to build Wasp with.

with-compiler: ghc-x.y.z

Ensure that you have this ghc version installed.

Repo

Fork this repo and clone the fork to your machine (or clone this repo directly if you don't plan to contribute but just want to try it out).

Position yourself in this directory (waspc/) and make sure that you are on the main branch.

Build

cabal build

to build the library and wasp executable.

This might take a while (10 mins) if you are doing it for the very first time, since cabal will need to download the external dependencies. If that is the case, relax and feel free to get yourself a cup of coffee! When somebody asks what you are doing, you can finally rightfully say "compiling!" :D.

⚠️ You may need to run cabal update before attempting to build if it has been some time since your last update.

⚠️ If you are on Mac and get "Couldn't figure out LLVM version!" error message while building, make sure you have LLVM installed and that it is correctly exposed via env vars (PATH, LDFLAGS, CPPFLAGS). The easiest way to do it is by just running brew install llvm@13, this should install LLVM and also set up env vars in your ~/.zshrc.

Test

cabal test

to ensure all the unit and end-to-end tests are passing (this will also build the project if needed).

Executable

cabal run wasp-cli

to run the wasp-cli executable that you just built! It should print "Usage" information.

You can pass more arguments by just adding them to the command, e.g.: cabal run wasp-cli new MyProject.

Run example app

Position yourself in waspc/examples/todoApp/ and run

cabal run wasp-cli db migrate-dev

to update database schema (this is done only on schema changes).

Then,

cabal run wasp-cli start

to run web app in development mode.

If you are doing this for the very first time, it might take a minutes or so to download and install npm dependencies.

When done, new tab in your browser should open and you will see a Todo App! NOTE: Reload page if blank.

Typical development workflow

  1. Create a new feature branch from main.
  2. If you don't have a good/reliable working HLS (Haskell Language Server) in your IDE, you will want to instead run ./run ghcid from the root of the project instead: this will run a process that watches the Haskell project and reports any Haskell compiler errors. Leave it running.
    NOTE: You will need to install ghcid globally first. You can do it with cabal install ghcid.
  3. Do a change in the codebase (most often in src/ or cli/src/ or data/) (together with tests if that makes sense: see "Tests"). Fix any errors shown by HLS/ghcid. Rinse and repeat.
  4. Run cabal test to confirm that the project's unit and integration tests are passing (both new and old).
  5. For easily testing the changes you did on a Wasp app, you have examples/todoApp, which we always keep updated. Also, if you added a new feature, add it to this app. Check its README for more details (including how to run it).
  6. There is also headless-test/examples/todoApp, that comes equiped with Playwright tests. This one is not so much for experimenting as examples/todoApp is, but more for running Playwright tests in the CI. Make sure this one is also passing, and add a new feature to it (+ tests) if needed. We will be merging this one with examples/todoApp in the future so it is a single app.
  7. If you did a bug fix, added new feature or did a breaking change, add short info about it to Changelog.md. Also, bump version in waspc.cabal and ChangeLog.md if needed -> check the version of latest release when doing it. If you are not sure how to decide which version to go with, check later in this file instructions on it.
  8. Squash all the commits into a single commit (or a few in case it makes more sense) and create a PR. Keep an eye on CI tests -> they should all be passing, if not, look into it.
  9. If your PR changes how users(Waspers) use Wasp, make sure to also update the documentation, which is in this same repo, but under /web/docs.
  10. Work with reviewer(s) to get the PR approved. Keep adding "fix" commits until PR is approved, then again squash them all into one commit.
  11. Reviewer will merge the branch into main. Yay!

NOTE: Why don't we use a cabal freeze file to lock our dependencies? In order to better support a wider range of developer operating systems, we have decided against using a cabal freeze file and instead use cabal's index-state feature to get package version pinning from hackage. See this question for more: https://github.com/haskell/cabal/issues/8059

Design docs (aka RFCs)

If the feature you are implementing is complex, be it due to its design or technical implementation, we recommend creating a design doc (aka RFC). It is a great way to share the idea you have with others while also getting help and feedback.

To create one, make a PR that adds a markdown document under wasp/docs/design-docs, and in that markdown document explain the thinking behind and choice made when deciding how to implement a feature.

Others will comment on your design doc, and once it has gone through needed iterations and is approved, you can start with the implementation of the feature (in a separate PR).

Codebase overview

Wasp is implemented in Haskell.

Codebase is split into library (src/) and CLI (which itself has a library cli/src/ and thin executable wrapper cli/exe/). CLI is actually wasp executable, and it uses the library, where most of the logic is.

Wasp compiler takes .wasp files + everything in the ext/ dir (JS, HTML, ...) and generates a web app that consists of client, server and database.

Wasp compiler code is split into 2 basic layers: Analyzer (frontend) and Generator (backend).

Wasp file(s) are analyzed by Analyzer, where they are first parsed, then typechecked, and then evaluated into a central IR (Intermediate Representation), which is AppSpec (src/Wasp/AppSpec.hs). Check src/Wasp/Analyzer.hs for more details.

AppSpec is passed to the Generator, which based on it decides how to generate a web app. Output of Generator is a list of FileDrafts, where each FileDraft explains how to create a file on the disk. Therefore, Generator doesn't generate anything itself, instead it provides instructions (FileDrafts) on how to generate the web app. FileDrafts are using mustache templates a lot (they can be found in data/Generator/templates).

Generator is split into three generators, for the three main parts of the web app: WebAppGenerator, ServerGenerator and DbGenerator.

After Generator provides us with FileDrafts, web app is generated by writing the FileDrafts to the disk.

Waspc implementation diagram

Generated web app consists of client, server and database.

Client is written with React and react-query. Server is written in NodeJS and uses ExpressJs. Database is abstracted via Prisma.

We can run Wasp project with wasp start. This will first compile the app, generate JS code in the .wasp/out/ dir, and then run npm start for the client, npm start for the server, and also run the database. On any changes you do to the source code of Wasp, Wasp project gets recompiled, and then changes in the generated code are picked up by the npm start of the client/server, therefore updating the web app.

Important directories (in waspc/)

  • src/ -> main source code, library
  • cli/src/ -> rest of the source code, cli, uses library
  • cli/exe/ -> thin executable wrapper around cli library code
  • test/, e2e-test/, cli/test/ -> tests
  • data/Generator/templates/ -> mustache templates for the generated client/server.
  • examples/ -> example apps

Building / development (detailed)

Some useful cabal commands:

  • cabal update to update your package information.
  • cabal build to build the project, including wasp binary which is both CLI and compiler in one.
  • cabal run wasp-cli <arguments> to run the wasp binary that was previously built.
  • cabal test to build the whole project + tests and then also run tests.
  • cabal install -> builds the project and places the binary so it is in PATH (so you can call it directly, from anywhere, with just wasp).

For live compilation and error checking of your code we recommend using Haskell Language Server (hls) via your IDE, but if that is not working as it should, then safe fallback is always ghcid. You can install ghcid globally with cabal install ghcid and then just type ghcid when in the project -> it will watch for any file changes and report errors.

Run script

For more convenient running of common build/dev commands, we created run script. It mostly runs cabal commands described above, reducing the number of characters you have to type to run certain commands. It also allows you to easily run some of the helper tools, for example tools for static analysis of our code.

The idea is that you normally use this for development, and you use cabal directly when you need more control. It is up to you, using cabal directly is also perfectly fine and sometimes easier.

You can run ./run help to learn how to use it.

Examples:

  • ./run ghcid-test will run ghcid that watches tests, while passing correct arguments to ghcid.
  • ./run test:unit will run only unit tests (skipping e2e tests, which is useful since they are relatively slow).
  • ./run ormolu:format will format the Haskell code for you.

Tip: to make it easy to run the run script from any place in your wasp codebase, you can create a bash alias that points to it:

alias wrun="/home/martin/git/wasp-lang/wasp/waspc/run"

Typescript packages

Wasp bundles some TypeScript packages into the installation artifact (eg: deployment scripts), which end up in the installed version's waspc_datadir. To do so in CI, it runs ./tools/install_packages_to_data_dir.sh.

waspc, while implemented in Haskell, for some of its functionality on TypeScript code (e.g. for parsing TS code, or for deployment scripts). In these cases, the Haskell code runs these TS packages as separate processes and communicates through input/output streams. These packages are located in packages/ and are normal npm projects. See packages/README.md for how the projects are expected to be set up.

To make these packages available to waspc in development, run ./run wasp-packages:compile. When any changes are made to these packages, run the same command again.

Tests

For tests we are using Tasty testing framework. Tasty let's us combine different types of tests into a single test suite.

In Tasty, there is a main test file that is run when test suite is run. In that file we need to manually compose test tree out of tests that we wrote. We organize tests in test groups, which are then recursively grouped resulting in a test tree. Cool thing is that we can organize tests this way however we want and also mix different type of tests (hspec, quickcheck, and whatever else we want).

Tests are normally split in files of course, so we need to import those all the way up to the main test file, however we organize our test groups/trees.

In order to avoid need for manual organization and importing of test files described above, we are using tasty-discover which does this for us. It automatically detects files containing tests and organizes them for us into a test tree (and also takes care of importing). This means we only need to create a file, write tests in it and that is it. Test functions however do need to be prefixed with special prefix to indicate which type of test are they: spec* for Hspec, prop* for QuickCheck and similar. Check docs for more details. We can however still organize tests manually if we want in Tasty test trees, and then we just prefix them with test_ and tasty-discover will pick them up from there.

Additionally, currently we limited tasty-discover to auto-detect only files ending with Test.hs (*Test.hs glob). We might remove that requirement in the future if it proves to have no benefit.

To summarize: If you are writing new tests, just put them in a file that ends with Test.hs in test/ dir and that is it.

For unit testing, we use Hspec.

For property testing, we use Quickcheck.

We additionally use doctest for testing code examples in documentation.

All tests go into test/ directory. This is convention for Haskell, opposite to mixing them with source code as in Javascript for example. Not only that, but Haskell build tools don't have a good support for mixing them with source files, so even if we wanted to do that it is just not worth the hassle.

Tests are run with cabal test. They include both unit tests, and end-to-end tests of basic CLI commands.

To run unit tests only, you can do cabal test waspc-test (or ./run test:unit). To run individual unit test, you can do cabal test waspc-test --test-options "-p \"Some test description to match\"" (or just ./run test:unit "Some test description to match").

To run cli tests only, you can do cabal test cli-test (or ./run test:cli).

To run end-to-end tests only, you can do cabal test e2e-test (or /run test:e2e).

End-to-end (e2e) tests

Besides unit tests, we have e2e tests that run waspc on a couple of prepared projects, check that they successfully run, and also compare generated code with the expected generated code (golden output).

This means that when you make a change in your code that modifies the generated code, e2e tests will fail while showing a diff between the new generated code and the previous (golden) one. This gives you an opportunity to observe these differences and ensure that they are intentional and that you are satisfied with them. If you notice something unexpected or weird, you have an opportunity to fix it. Once you are indeed happy with the changes in the generated code, you will want to update the golden output to the new (current) output, so that tests pass. Basically, you want to say "I am ok with the changes and I accept them as the new state of things.". Easiest way to do this is to go to e2e-test/test-outputs/ dir, delete all the directories ending with -golden/, and then re-run e2e tests -> since there are no golden outputs, the new outputs will be used as new golden outputs and that is it. After that you commit that to git and you are done.

Instead of doing this manually, you can also use convenient command from the ./run script:

./run test:e2e:accept-all

Code analysis

To run the code analysis, run:

./run code-check

This will check if code is correctly formatted, if it satisfies linter, and if it passes static analysis.

These same checks are required to pass the CI, so make sure this is passing before making a PR. TODO: For now we check only the code formatting during the CI. In the future, once we make sure all the warnings are passing, we will also check linter and static analysis during the CI, but that is not happening yet.

Formatting

For formatting Haskell code we use Ormolu.

Normally we set it up in our editors to run on file save.

You can also run it manually with

./run ormolu:check

to see if there is any formatting that needs to be fixed, or with

./run ormolu:format

to have Ormolu actually format (in-place) all files that need formatting.

NOTE: When you run it for the first time it might take a while (~10 minutes) for all the dependencies to get installed. The subsequent runs will be much faster.

Linting

We use hlint for linting our Haskell code.

You can use

./run hlint

to run the hlint on Wasp codebase.

NOTE: When you run it for the first time it might take a while (~10 minutes) for all the dependencies to get installed. The subsequent runs will be much faster.

Static Analysis

We use stan to statically analyze our codebase.

The easiest way to run it is to use

./run stan

This will build the codebase, run stan on it (while installing it first, if needed, with the correct version of GHC) and then write results to the CLI and also generate report in the stan.html.

NOTE: When you run it for the first time it might take a while (~10 minutes) for all the dependencies to get installed. The subsequent runs will be much faster.

Commit message conventions

We use Conventional Commits convention when creating commits.

Branching and merging strategy

This repo contains both the source code that makes up a Wasp release (under waspc), as well as our website containing documentation and blog posts (under web), and also Mage web app (under mage). In order to facilitate the development of Wasp code while still allowing for website / Mage updates or hotfixes of the current release, we have decided on the following minimal branching strategy.

All Wasp development should be done on feature branches. They form the basis of PRs that will target one of the two following branches:

  • main: this branch contains all the actively developed new features and corresponding documentation updates. Some of these things may not yet be released, but anything merged into main should be in a release-ready state.
    • This is the default branch to target for any Wasp feature branches.
  • release: this branch contains the source code of current/latest Wasp release, as well as the documentation and blog posts currently published and therefore visible on the website, and also currently published version of Mage.
    • When doing a full release, which means making a new release based on what we have currently on main, we do the following:
      1. Update main branch by merging release into it. There might be conflicts but they shouldn't be too hard to fix. Once main is updated, you can create a new waspc release from it, as well as deploy the website from it.
      2. Update release branch to this new main by merging main into it. There will be no conflicts since we already resolved all of them in the previous step.

How do I know where I want to target my PR, to release or main?

  • If you have a change that you want to publish right now or very soon, certainly earlier than waiting till main is ready for publishing, then you want to target release. This could be website content update, new blog post, documentation (hot)fix, compiler hotfix that we need to release quickly via a new patch version, update for Mage that needs to go out now, ... .
  • If you have a change that is not urgent and can wait until the next "normal" Wasp release is published, then target main. These are new features, refactorings, docs accompanying new features, ... .
  • Stuff published on release (docs, Mage) uses/references version of wasp that was last released (so one that is also on release).
  • TLDR;
    • release represents the present, and is for changes to the already published stuff.
    • main represents near future, and is for changes to the to-be-published stuff.

Deployment / CI

We use Github Actions for CI.

CI runs for any commits on main branch, for pull requests, and for any commits tagged with tag that starts with v.

During CI, we build and test Wasp code on Linux, MacOS and Windows.

If commit is tagged with tag starting with v, github draft release is created from it containing binary packages.

If you put [skip ci] in commit message, that commit will be ignored by Github Actions.

We also wrote a new-release script which you can use to help you with creating new release: you need to provide it with new version (./new-release 0.3.0) and it will check that everything is all right, create appropriate tag and push it, therefore triggering CI to create new release on Github.

NOTE: If building of your commit is suddenly taking much longer time, it might be connected with cache on Github Actions. If it happens just once every so it is probably nothing to worry about. If it happens consistently, we should look into it.

Typical Release Process

Do the steps marked with 👉 for every release of waspc. Do the non-bold steps when necessary (decide for each step depending on the changes, e.g. some can be skipped if there were no breaking changes).

  • Update the starter templates if necessary (i.e., if there are breaking changes or new features they should make use of):
    • Context: they are used by used by wasp new, you can find reference to them in Wasp.Cli. ... .StarterTemplates.
    • In StarterTemplates.hs file, update git tag to new version of Wasp we are about to release (e.g. wasp-v0.13.1-template).
    • Ensure that all starter templates are working with this new version of Wasp. Update Wasp version in their main.wasp files. Finally, in their repos (for those templates that are on Github), create new git tag that is the same as the new one in StarterTemplates.hs (e.g. wasp-v0.13.1-template).
  • Make sure apps in examples are up to date and using a version compatible with the newest version of Wasp.
  • Make sure that Wasp AI (which is part of waspc and you can run it with e.g. wasp new:ai) is correctly producing apps that work with and use this newest version of Wasp. This usually means checking that templates and prompts (e.g. examples of Wasp code) are up to date. If there were no breaking changes, there is likely nothing to be done here.
  • Make sure mage is producing Wasp apps that support the newest version of Wasp (this is configured in its Dockerfile: which version of wasp CLI will it use to produce app via AI). Mage itself, as a Wasp app, doesn't have to be using the latest Wasp version to run, but the apps it is producing should.
  • 👉 ChangeLog.md and the version in waspc.cabal should already be up to date, but double check that they are correct and update them if needed. Also consider enriching and polishing ChangeLog.md a bit even if all the data is already there. Also make sure the ChangeLog.md specifies the correct version of Wasp.
    • If you modify ChangeLog.md or waspc.cabal, create a PR, wait for approval and all the checks (CI) to pass, then squash and merge mentioned PR into main.
  • 👉 Update your local repository state to have all remote changes (git fetch).**
  • 👉 Update main to contain changes from release by running git merge release while on the main branch. Resolve any conflicts.
  • Take a versioned "snapshot" of the current docs by running npm run docusaurus docs:version {version} in the web dir. Check the README in the web dir for more details. Commit this change to main.
  • 👉 Fast-forward release to this new, updated main by running git merge main while on the release branch.
  • 👉 Make sure you are on release and then run ./new-release 0.x.y.
    • This will do some checks, tag it with new release version, and push it.
  • 👉 Wait for CI to finish & succeed for the new tag.
    • This will automatically create a new draft release.
  • 👉 Find new draft release here: https://github.com/wasp-lang/wasp/releases and edit it with your release notes.
  • 👉 Publish the draft release when ready.
  • 👉 Merge release back into main (git merge release while on the main branch), if needed.
  • If there are changes to the docs, publish the new version from the release branch.
  • If you published new docs, rerun the Algolia Crawler to update the search index. If you published a new version of the docs, the search won't work until you do this.
  • If there are changes to Mage, publish the new version from the release branch.
  • If there are changes to the Wasp VSCode extension, publish the new version.
  • Announce new release in Discord if it makes sense.

Determining next version

waspc follows typical SemVer versioning scheme, so major.minor.patch. There is one slightly peculiar thing though: waspc, besides being a wasp compiler and CLI, also contains wasp language server (waspls) inside it, under the subcommand wasp waspls. So how do changes to waspls affect the version of waspc, since they are packaged together as one exe? We have decided, for practical reasons, to have them affect the patch number, possibly maybe minor, but not major.

Test releases (e.g. Release Candidate)

Making a test release, especially "Release Candidate" release is useful when you want to test the release without it being published to the normal users. If doing this, steps are the following:

  1. You can do it from whatever branch you want, probably you will be doing it from main.
  2. You will want to use a version name that indicates you are doing test, probably you will want to add -rc at the end. So for example: ./new-release 0.7.0-rc. Release script will throw some warnings which you should accept.
  3. Once draft release is created on Github, you should mark it in their UI as pre-release and publish it. This will automatically remove the checkmark from "latest release", which is exactly what we want. This is the crucial step that differentiates test release from the proper release.
  4. Since our wasp installer by default installs the latest release from Github, it will skip this release we made, because it is pre-release, which is great, it is what we wanted. Instead, you can install it by using the -v flag of wasp installer! That way user's don't get in touch with it, but we can install and use it normally.

Documentation

External documentation, for users of Wasp, is hosted at https://wasp-lang.dev/docs, and its source is available at web/docs, next to the website and blog.

Mage

Wasp's magic GPT web app generator aka Wasp AI aka Mage is hosted at https://usemage.ai and its source is available at mage.

Make sure to update it when changes modify how Wasp works.

Haskell

We are documenting best practices related to Haskell in our Haskell Handbook.

Code style guide

General

Comments

Grammar

When writing a comment, we prefer starting it with a capital letter.

If it starts with a capital letter, it must end with a punctuation.

If it doesn't start with a capital letter, it shouldn't end with a punctuation.

TODO / NOTE

When writing a TODO or NOTE, use all capital letters, like this:

-- TODO: Wash the car.

-- NOTE: This piece of code is slow.

If you wish, you can add your name to TODO / NOTE. This is useful if you think there is a fair chance that reader of that TODO / NOTE might want to consult with its author. You can do it like this:

-- TODO(martin): Doesn't work on my machine in some unusual use cases.

JavaScript

Functions

For detailed reasoning/discussion on all listed rules, check Issue #487.

Top-level named functions

When defining top-level named functions, we prefer to use statements:

// good
function foo(param) {
  // ...
}

// bad
const foo = (param) => {
  // ...
};

// bad
const foo = function (param) {
  // ...
};
Inline function expression

When defining inline function expressions, we prefer the arrow syntax:

// good
const squares = arr.map((x) => x * x);

// bad
const squares = arr.map(function (x) {
  return x * x;
});