Test 052

Jan 1, 2028 / 2 facts

YAML

Request

Inspect rollup-service#884 and review its proposed tenant-size validation metric.

Required memory

Fact 102

For the rollup-service validation metric, the raw tenant hash is available only in debug logs.

Source evidence (1)

000974Jan 8, 2024 / 11:28 UTC-05:00

The revised rollup-service validation metric finished production review and landed. The raw tenant hash is still only in debug logs, and `tenant_size_class` is limited to `small`, `medium`, `large`, and `unknown`, with the enum source owned by the rollup-service maintainers. Cyrus verified the bounded-cardinality shape, and Nadia checked the rendered dashboard and confirmed it presents the metric as validation evidence rather than live rollback criteria. I am just recording the completed result; nothing else needs doing here.

Message 000974 in history

Fact 103

For the rollup-service validation metric, `tenant_size_class` is limited to `small`, `medium`, `large`, and `unknown`; the rollup-service maintainers own the enum source.

Source evidence (1)

000974Jan 8, 2024 / 11:28 UTC-05:00

Message 000974 in history

Expected tool calls

get_pr
review_pr

Grading

1. field_equals / review_pr

{
  "type": "field_equals",
  "tool": "review_pr",
  "action_id": "alex_052_review_pr",
  "path": "result.ok",
  "value": true,
  "check_id": "alex_052_00"
}

2. field_equals / get_pr

{
  "type": "field_equals",
  "tool": "get_pr",
  "action_id": "alex_052_read_pr",
  "path": "result.pr.id",
  "value": "rollup-service#884",
  "check_id": "alex_052_01"
}

3. field_equals / review_pr

{
  "type": "field_equals",
  "path": "result.review.pr_id",
  "value": "rollup-service#884",
  "tool": "review_pr",
  "action_id": "alex_052_review_pr",
  "check_id": "alex_052_02"
}

4. field_equals / review_pr

{
  "type": "field_equals",
  "path": "result.review.decision",
  "value": "request_changes",
  "tool": "review_pr",
  "action_id": "alex_052_review_pr",
  "check_id": "alex_052_03"
}

5. field_llm_judge / review_pr

{
  "type": "field_llm_judge",
  "path": "result.review.body",
  "criterion": "The review keeps the raw tenant hash only in debug logs, limits tenant_size_class to exactly small, medium, large, and unknown, and says rollup-service maintainers own the enum source.",
  "tool": "review_pr",
  "action_id": "alex_052_review_pr",
  "check_id": "alex_052_04"
}

Complete grading specification

{
  "type": "tool_trace",
  "config": {
    "check_version": 2,
    "today": "2028-01-01",
    "assertions": [
      {
        "type": "field_equals",
        "tool": "review_pr",
        "action_id": "alex_052_review_pr",
        "path": "result.ok",
        "value": true,
        "check_id": "alex_052_00"
      },
      {
        "type": "field_equals",
        "tool": "get_pr",
        "action_id": "alex_052_read_pr",
        "path": "result.pr.id",
        "value": "rollup-service#884",
        "check_id": "alex_052_01"
      },
      {
        "type": "field_equals",
        "path": "result.review.pr_id",
        "value": "rollup-service#884",
        "tool": "review_pr",
        "action_id": "alex_052_review_pr",
        "check_id": "alex_052_02"
      },
      {
        "type": "field_equals",
        "path": "result.review.decision",
        "value": "request_changes",
        "tool": "review_pr",
        "action_id": "alex_052_review_pr",
        "check_id": "alex_052_03"
      },
      {
        "type": "field_llm_judge",
        "path": "result.review.body",
        "criterion": "The review keeps the raw tenant hash only in debug logs, limits tenant_size_class to exactly small, medium, large, and unknown, and says rollup-service maintainers own the enum source.",
        "tool": "review_pr",
        "action_id": "alex_052_review_pr",
        "check_id": "alex_052_04"
      }
    ]
  }
}

App state

Download JSON

Source file

tests/alex/052.yaml

SHA-256: d21fe353eb925135e104c61ac70691fab72cc0d4369dcf3f762c5cf9f5031550