Skip to content

feat: add native read-only mode for Grok #37

Description

@oritwoen

What's missing?

Add native read-only invocation for Grok. The adapter currently rejects readOnly: true, although Grok 1.0.13 exposes a native sandbox profile. This prevents read-only review jobs from using Grok through the shared API.

Minimal reproduction, with no model run:

import { getHarness } from "@agntn/harnesses";

const grok = getHarness("grok");
console.log(grok.invocationError({ readOnly: true }));
console.log(grok.buildInvocation("Reply OK", { readOnly: true }));

Current output is Harness grok has no read-only full agent invocation and null. The MCP metadata also reports readOnly: false. Checked with @agntn/harnesses 0.0.1, Grok 1.0.13 (5e9a58528b76), and the adapter on main at cde8141.

Add a version-checked recipe after verifying the native sandbox and inherited tool configuration. Cover argument construction and refusal on unsupported versions; keep the current rejection wherever enforcement is unverified. CLI help alone is not enough to enable the mode.

#10 and #18 introduced the shared mode with Codex support. This request covers the missing Grok adapter recipe.

Activity

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

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions