Skip to content

docs(factories): document scorers/ in the definition syntax page - #584

Open
hongyi-chen wants to merge 2 commits into
mainfrom
hyc/factory-as-code-scorers
Open

docs(factories): document scorers/ in the definition syntax page#584
hongyi-chen wants to merge 2 commits into
mainfrom
hyc/factory-as-code-scorers

Conversation

@hongyi-chen

Copy link
Copy Markdown
Collaborator

scorers/<name>/scorer.md is real in the published v1alpha1 schema and the product, but the factory-as-code page omitted it — flagged during the factory-examples groundwork (wilson's proposal, appendix gap #1).

Adds scorers/ to the directory tree and a section with a worked example and field notes (agents, labels, passingScore, samplingRate, model, selfImprovement), linking to Measure and improve.

Co-Authored-By: Warp agent@warp.dev

scorers/<name>/scorer.md is supported by the v1alpha1 schema and the product
(file-defined scorers with LLM-judge classification), but the definition
syntax page omitted it. Add the directory to the tree and a section with a
worked example and field notes, linking to Measure and improve.

Co-Authored-By: Warp <agent@warp.dev>
@vercel

vercel Bot commented Aug 20, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Aug 20, 2026 11:47pm

Request Review

@cla-bot cla-bot Bot added the cla-signed label Aug 20, 2026
@warp-for-oss

warp-for-oss Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

@hongyi-chen

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@warp-for-oss warp-for-oss Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overview

This PR adds scorers/<name>/scorer.md to the factory definition syntax page, including the directory tree, a worked example, and field descriptions. The change is directionally aligned with the existing reference page, but the new scorer section omits supported schema keys even though the page is positioned as the lookup for every file and key.

Concerns

  • scorers/<name>/scorer.md documents most scorer frontmatter fields but leaves out enabled and output, which are supported schema keys and should be included in this reference.

Verdict

Found: 0 critical, 1 important, 0 suggestions

Request changes

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Comment thread src/content/docs/factories/factory-as-code.mdx Outdated
Co-authored-by: warp-for-oss[bot] <277970191+warp-for-oss[bot]@users.noreply.github.com>
@warp-agent-staging warp-agent-staging Bot added the warpy-factory Opened by the Warp factory agents label Aug 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed warpy-factory Opened by the Warp factory agents

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant