Audit log review

The audit-log branch is mostly safe. There is one risky change — the addition of a new event type to the persistence enum. Open the diff.

Findings

• Three commits are doc-only and can be merged immediately.
• One commit touches the database schema. It should be reviewed against the existing idx_audit_log_event index.
• The last commit only renames test fixtures.

Suggested order

1. Land the doc commits today.
2. Hold the schema commit for review.
3. Land the test fixture rename last.

Note

The schema change does not include a backfill migration. New rows will carry the new event type; older rows will not. That is probably fine, but worth confirming with the platform team.

Reproduction

The failing assertion, for reference:

test("sum adds", () => {
  expect(sum(2, 2)).toBe(4); // received 1 — the impl subtracts
});

Commits