Reviewed 11 Sept, at 6ac523c — open the Check Run
Nothing was posted.
Below the posting bar, not confirmed by the second look, or already commented on by an earlier review.
Lowsrc/auth.ts:26debugging
`console.log` is left in.
Every call writes to stdout in production, adding noise to the logs and possibly printing values that should not be logged.
Suggested fix: Remove it, or use the application logger at an appropriate level.
Not checked
Lowsrc/auth.ts:32debugging
`console.log` is left in.
Every call writes to stdout in production, adding noise to the logs and possibly printing values that should not be logged.
Suggested fix: Remove it, or use the application logger at an appropriate level.
Not checked
Lowsrc/helpers.ts:10maintainability
A new TODO is added.
The TODO marks work this change leaves undone; once merged it is easy to forget, and the behaviour it describes stays missing.
Suggested fix: Do the work in this PR, or track it in an issue and link it here.
Not checked
Mediumsrc/new-feature.ts:2correctness
A `debugger` statement is left in.
Whenever this code runs with an inspector attached — a browser with devtools open, `node --inspect` — execution stops here until someone resumes it.
Suggested fix: Remove the `debugger` statement.
Not checked