added some small code improvements

[turnerran]

Checklist

• run npm run test and npm run benchmark
• tests and/or benchmarks are included
• documentation is changed or added
• commit message and code follows the Developer's Certification of Origin
  and the Code of conduct

[Fdawgs]

I think the optional chaining changes are good, but the rest are just stylistic and don't really improve on anything.

[mcollina]

CI seems to be failing.

[mcollina]

Please rebase on top of master.

[turnerran]

Please rebase on top of master.

Hey @mcollina , I'm up to date with master

[mcollina]

I've updated it for you.

[Uzlopak]

LGTM

[Uzlopak]

@Fdawgs PTAL

[gurgunday]

lgtm

[Uzlopak]

Btw. I am old school and never benchmarked the benefit of || vs. ??. Keep in mind, that || can be used in branchless programming. Personally I never checked how ?? is implemented in nodejs. So should be in cases like calling another function, ?? func() definetly be slower, but in cases where we do || false vs. ?? false that || false be faster . But I am fine with this implementation.

[gurgunday]

Btw. I am old school and never benchmarked the benefit of || vs. ??. Keep in mind, that || can be used in branchless programming. Personally I never checked how ?? is implemented in nodejs. So should be in cases like calling another function, ?? func() definetly be slower, but in cases where we do || false vs. ?? false that || false be faster . But I am fine with this implementation.

I see, I thought it'd be a clear case since ?? checks for less things (2 falsy values instead of 6) and short circuits more often – I'll keep this in mind