Skip to content

demurThe reviewer that argues against you

Even granting every fact in the pull request, there is still no case for merging it.

demurdemur

The shape of a review

demur publishes one review event and one check run per completed run. The body leads with the verdict, then the ranked findings, then exactly what the run covered and what it cost.

markdown
## demur: changes requested

Granting every fact in this pull request, there is still no case for merging it.

The token issuer writes a live credential into a log that ships to a third
party, and the migration it adds has no rollback path.

### Findings (ranked)

1. **[blocker]** `src/auth/token.rs:41-44`: the issued token is logged at info
   Merging ships live credentials to the log sink, where anyone with read
   access to logs can replay them.

2. **[warning]** `migrations/0007_add_index.sql:1-9`: no down migration
   A failed deploy cannot be rolled back without manual database surgery.

### Coverage and spend

- Full coverage: every pass ran without degradation.
- triage spend: $0.0021
- deep dive security spend: $0.1087
- Total spend this run: $0.3418
- Cumulative spend for this pull request: $0.4106 (earlier runs: $0.0688)

Two minutes of setup is on the setup page. If you would rather try it on one pull request without installing anything, start with the local CLI.

MIT or Apache-2.0. No hosted service, no key custody, no metered billing.