feat: add docs eval suite with docs team ownership - #282
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
|
Dispatching workflow to verify the generated PR and results file:
Run: https://github.com/supabase/evals/actions/runs/34625680189 Resulting PR: #283 |
Rodriguespn
left a comment
There was a problem hiding this comment.
LGTM.
One thing for @supabase/docs to be aware of (not a blocker, not caused by this PR): build-docs-002-rls-guide shows high run-to-run variance.
Run 3 in the auto-generated #283 results comes back with RLS effectively disabled: anon still holds write grants, and the todo/list ownership checks show cross-tenant writes actually landing, while runs 1–2 pass almost everything.
Result snippet
evals/apps/web/src/data/docs-eval-results.json
Lines 729 to 992 in 85a0ad9
Confirmed via main's regression-eval-results.json that this is due to flakiness and predate this PR, not related with the folder restructuring itself.
Worth a look once docs evals settle into their own cadence.
|
https://github.com/supabase/evals/settings/rules/18775100 Ruleset has been updated to now require 1 approval from Code Owner instead of specifying just AI team. |

This change reorganizes all evals under per-suite folders e.g.
evals/<suite>/<id>/which replaces thesuite:frontmatter, so CODEOWNERS can scope by path.It then gives the docs team their own
docseval suite so they can add and refresh evals on their own cadence, per the meeting notes and Slack. Docs evals run on the Luna no-skills experiment only.The new
CODEOWNERSlists AI team as default codeowner, with docs team as owner for theevals/docs/and its results.jsonfile.How to review
Dry run to see the list of experiment x evals that run under the docs suite:
Which produces the full set of docs runs:
For more concrete evidence of the new docs results structure, see this sample PR #283 which was generated by the refresh workflow on the current PR.
See also a new Docs evals section of the contributing guide to explain common evals workflows to docs contributors.
After merge
Flip main: review required to require code owner review with 1 approval and drop the
aiteam required-reviewers entry, so CODEOWNERS decides who approves docs PRs.Closes AI-1203