nikivdev/code · error

remote review URL not configured

Error message

remote review URL not configured

What it means

The remote review feature requires a review service URL, obtained from commit_with_check_review_url(). When that configuration is absent the review cannot be dispatched and the library bails immediately, before building the request.

Source

Thrown at src/commit.rs:5458

fn normalize_review_url(url: &str) -> String {
    let trimmed = url.trim().trim_end_matches('/');
    if trimmed.ends_with("/review") {
        trimmed.to_string()
    } else {
        format!("{}/review", trimmed)
    }
}

fn run_remote_claude_review(
    diff: &str,
    session_context: Option<&str>,
    review_instructions: Option<&str>,
    model: ClaudeModel,
) -> Result<ReviewResult> {
    let url = match commit_with_check_review_url() {
        Some(url) => url,
        None => bail!("remote review URL not configured"),
    };

    let review_url = normalize_review_url(&url);
    let payload = RemoteReviewRequest {
        diff: diff.to_string(),
        context: session_context.map(|value| value.to_string()),
        model: model.as_claude_arg().to_string(),
        review_instructions: review_instructions.map(|v| v.to_string()),
    };

    let client = crate::http_client::blocking_with_timeout(Duration::from_secs(
        commit_with_check_timeout_secs(),
    ))
    .context("failed to create HTTP client for remote review")?;

    let mut request = client.post(&review_url).json(&payload);
    if let Some(token) = commit_with_check_review_token() {
        request = request.bearer_auth(token);

View on GitHub (pinned to a747e741ae)

Solutions

  1. Configure the remote review URL in your myflow config (the key commit_with_check_review_url reads, e.g. review_url).
  2. Run `f auth` / myflow setup to populate defaults.
  3. Verify the config file exists and the key spelling matches the current version's schema.
  4. Check you're in the right environment (config may be per-user/per-project).

Example fix

// config before (missing key)
[review]
// after
[review]
url = "https://myflow.example.com/review"
Defensive patterns

Strategy: validation

Validate before calling

if commit_with_check_review_url().is_none() {
    anyhow::bail!("remote review URL not configured; set [review] url in myflow config");
}

Type guard

fn review_url_configured() -> bool {
    commit_with_check_review_url().is_some()
}

Prevention

When it happens

Trigger: Calling the remote review function (e.g. via `f commit` with check/review) when commit_with_check_review_url() returns None — the review URL config key is unset.

Common situations: Fresh install without myflow configuration; config file missing the review URL key; running with a stripped config in CI; typo in config key name; older config format after upgrade.

Understand the failure class

Background: "missing required config value" errors: why libraries refuse to start when a configuration key is empty, unset, or blank — this error's family across 48 libraries.

Related errors


AI-assisted analysis of nikivdev/code@a747e741ae (2026-09-01). Data as JSON: /api/errors/0d7b47f5f4ce0385. Report an issue: GitHub.