Skip to content

find which models are most likely to work for security reviews #963

Open

Description

@JacobStephens2
No description provided.

Activity

  1. JacobStephens2 commented on Oct 21, 2026

    @JacobStephens2
    OwnerAuthor

    This was generated by AI during triage.

    Brief (ready-for-human)

    Category: enhancement
    Summary: Choose the Harness and Model for Security reviews, from a record of which Harness actually ran each review.

    Correction: an earlier version of this brief read today's six failed Security reviews as Codex failures. They ran on Grok Build. security.harness = "grok" in the User config on thirdshift sent them there, and Grok Build isn't signed in on that host, so each session ended at once. #974 covers the hidden cause; #975 adds a preflight check and fallback.

    What the record shows:

    • Until Run the Security review on security.harness when it is set #910 shipped in 0.13.2 on 2026-10-21, a Security review always ran on its Command's Harness, so a PR's "Built with" line named the review's Harness too. Since 0.13.2, security.harness picks the review's Harness when it's set. The Run's Command log names it: security-review: on <Harness> · <Model> · <effort>, chosen by ….
    • Before 2026-10-21, 59 thirdshift PRs had a "Security review outcome" section:
    Harness · Model Completed Incomplete Refused
    Claude · claude-opus-5-5 · medium 13 0 0
    Codex · gpt-6.1-sol · high 38 3 (#605, #743, #810) 3 (#679, #764, #842)
    Muse · default model 2 0 0

    Steps:

    • Decide which Harness and Model security.harness should name on each host. stephens.page leaves it blank, so its reviews run on Codex, its default Harness.
    • If Grok Build stays on thirdshift, sign it in there (grok login --device-code) and check that a review completes.
    • File any follow-ups as their own issues, and close this one with the decision.

    Why this can't go to an agent: signing in a Harness needs the host, and choosing a Model means weighing cost and subscriptions, which is your call.

    Out of scope: the preflight check and fallback (#975) and the failure's reported cause (#974).

  2. JacobStephens2 commented on Oct 21, 2026

    @JacobStephens2
    OwnerAuthor

    This was generated by AI during triage.

    Two more data points for the Harness choice:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions