RiskoZS

Style Guide

1

Luvit 2.x is fairly inconsistent when it comes to casing, as it's mostly a mix of camel-case (`fs.readFile()`, `pathjoin.pathJoin()`, `timer.setInterval()`) and all-lowercase (`fs.sendfile()`, `querystring.urldecode()`, `path.basename()`). It may be worthwhile to...

Provide general-purpose utilities

1

Lua's default libraries are relatively sparse and I oftentimes find myself repeatedly implementing certain common functions, such number clamping/rounding, table joining/merging/filtering/mapping, string padding/splitting and so on. I think it would...

Luvit 2.x provides the `ustring` library, which is a pain to use as it uses wrapper objects resulting in a lot of unnecessary overhead. Luvit 3.x is a good time...

`querystring` and `json` function names

4

Luvit 2.x's `json` module provides the functions `stringify()` and `parse()` as aliases to the `encode()` and `decode()` functions. Node.js's `querystring` module does the same (technically, the latter are aliases for...

Just spitballing some ideas. 1. Provide a shortcut reference to `uv.new_async` (`thread.new_async`? Would have to decide on naming style), since `async` really only has use in the context of multiple...

Utility functions for `fs`

1

Currently, besides `readFile`/`writeFile`, the `fs` module only provides wrappers for functions which are directly provided by luv. There are, however, some miscellaneous functions that one may find useful. The purpose...

Fix querystring module

3

Currently, the `querystring` module has several issues. Besides neither mirroring Node.js nor adhering to any specification when encoding/decoding strings*, the module's behaviour is definitely not correct as it does not...

Client option to log full reason for HTTP errors

11

This pull request adds a new `boolean` client option `logFullErrors` (`true` by default) which, when `true`, makes the `API` class log the full error reason (found inside the response body)...

Fix Emitter memory leak

11

Listeners created by the `Emitter:once()` method get put into an internal table, but are never removed from said table, causing them to never be GC'd. This PR fixes this issue...

This pull request adds a new `boolean` client option `prettyNewlines` (`true` by default) and a new `Logger` option + mutators which, when `true`, makes the logger pad newlines so that...