field notes · two reviews on every diff

Field notes from reviewing every diff.

Short, opinionated reads on repository memory, architecture drift, review noise, historical context, GitLab-native workflows, and audit evidence — from production merge requests, not slide decks.

Security scans

// vulnerabilities · audit · per-repo how the security pipeline reasons across files and writes audit evidence as it goes

Code review

// quality · architecture · rulebook how the second pipeline talks back to PRs without behaving like a noisy bot
// review etiquette 2026-04-22

Code review without the noise: one inline thread per remark, zero walls of text

Most teams that try a PR-review bot mute it by sprint two. The reason is almost always the same: the bot drops a top-level essay on every PR. Here's the design choices that make a code-review bot survive the third week — verdicts that mean something, threads anchored to lines, suppressions with a memory, and threads that close themselves.

Read note
// architecture 2026-04-15

Architecture as code: turning your bounded contexts into a CI gate, not a Confluence page

The architecture rules a team agreed on in some kickoff meeting — "controllers don't talk to Eloquent", "Services never inject other Services" — almost always live in a Confluence page nobody opens. The bypass merges because nobody remembered the rule. Here's how the per-repo rulebook (.codeguards/review.yml) turns those agreements into something the bot enforces on every PR, with a one-line citation of the rule.

Read note
// suppression policy 2026-04-08

Per-repo suppression memory works on review remarks, too

The same per-repo memory that handles security false positives applies on the Code review side: click "Suppress" once with a one-line reason, and the same fingerprint stops nagging this repo. Category-path fallback means a rephrased AI output still hits the same rule — you don't suppress the same nit twelve times.

Read note

Platform & commercial

// bundling · billing · audit shape how two products on one workspace add up — operationally and on the invoice
// commercial logic 2026-04-29

One workspace, two products: the operational case for bundling Security scans with Code review

Each product is €189/mo + €0.50/run. The bundle is €289/mo + €0.75 per "combined run" — meaning when both pipelines fire on the same diff you pay €0.75 once, not 2 × €0.50. The €89 base discount and the €0.25 saved on every double-run aren't a marketing trick — they reflect the real overhead saving of running both pipelines on shared infrastructure. Here's how it adds up, what counts as a combined run, and where the bundle breaks even.

Read note

On the roadmap

// field notes

Topics we're writing next — same engineering tone, same repository-context lens:

  • Repository memory beats generic suppressions
  • Architecture rules as review infrastructure
  • Why code review bots get muted
  • Two reviewers on one diff: security + architecture
  • How per-repo rulebooks reduce review noise