models list --sort context and --sort created silently fall back to --sort id

[nightmare0329]

Bug Report

Version: cortex v0.0.7
Command: cortex models list --sort context and cortex models list --sort created

Description

cortex models list validates sort values against VALID_SORT_VALUES = ["name", "provider", "context", "created", "id"], so --sort context and --sort created are accepted without error. However, these values are not handled in ModelSortOrder::from_str(), causing them to silently fall back to the default sort (--sort id).

Steps to Reproduce

# Invalid value correctly rejected:
$ cortex models list --sort invalid
Error: Invalid sort value 'invalid'. Valid values are: name, provider, context, created, id

# Valid value accepted but ignored (same output as --sort id):
$ cortex models list --sort context
Available Models:
anthropic (4 models)
  claude-3-5-haiku-20241022
  claude-3-5-sonnet-20241022
  ...

$ cortex models list --sort id
Available Models:
anthropic (4 models)
  claude-3-5-haiku-20241022    <- identical ordering
  claude-3-5-sonnet-20241022
  ...

Expected Behavior

--sort context should sort models by context window size.
--sort created should sort models by creation date.

Or if unimplemented, they should be removed from VALID_SORT_VALUES and return an error.

Actual Behavior

context and created are accepted as valid but produce identical output to --sort id (alphabetical by model ID).

Screenshot (real binary output)

Root Cause

In src/cortex-cli/src/models_cmd.rs:

VALID_SORT_VALUES (line 426) includes "context" and "created":

const VALID_SORT_VALUES: &[&str] = &["name", "provider", "context", "created", "id"];

But ModelSortOrder::from_str() (line 46) only handles id, name, and provider:

fn from_str(s: &str) -> Result<Self, Self::Err> {
    match s.to_lowercase().as_str() {
        "id" => Ok(Self::Id),
        "name" => Ok(Self::Name),
        "provider" => Ok(Self::Provider),
        _ => Err(format!("Invalid sort order '{}'. ...", s)),
    }
}

Line 438 then silently swallows the parse error:

let sort_order: ModelSortOrder = sort_by.parse().unwrap_or_default();
// unwrap_or_default() -> ModelSortOrder::Id (the default variant)

File: src/cortex-cli/src/models_cmd.rs, lines 33-55 and 426-438

[atlas-cortex]

🔁 Validation Summary — Issue #53396

Issue is a duplicate of #53364 (similarity: 0.74)

Duplicate of #53364 — the original issue takes precedence.

Confidence Score: 5/5

• High confidence — this issue substantially overlaps with [BUG] [v0.0.7] cortex models list --sort context and --sort created are silently ignored and fall back to --sort id without warning #53364.

Validation Checks

Check	Status	Detail
Media Evidence	✅ Pass	Evidence found and accessible
Spam Detection	✅ Pass	Score: 21% — no spam signals
Duplicate Detection	❌ Duplicate	Matches existing issue #53364
Code Verification	✅ Pass	Bug confirmed in source code (confidence: 100%) — The code perfectly matches the bug report. The VALID_SORT_VALUES array allows
Edit History	✅ Pass	No suspicious edits
LLM Evaluation	❌ Fail	Issue is a duplicate of #53364 (similarity: 0.74)

Validation Pipeline

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["Issue #53396"] --> B{"Media Check"}
    B -->|"✅ Found"| C{"Spam Check"}
    C -->|"✅ Clean"| D{"Duplicate Check"}
    D -->|"Matches #53364"| Z["🔁 Duplicate"]
    style Z fill:#ffa94d,color:#fff

Loading

Raw Evidence

{
  "media": {
    "hasMedia": true,
    "accessible": true,
    "urls": [
      "https://raw.githubusercontent.com/nightmare0329/cortex-bug-evidence/main/real_sort_context_bug.png"
    ],
    "evidence": [
      "Found 1 media URL(s) via GitHub rendered HTML",
      "1 URL(s) accessible"
    ]
  },
  "spam": {
    "overallScore": 0.20524861878453038,
    "details": "template=0.28 (5 recent issues compared); burst=1.00 (6 issues in 2h window); parity=0.00; overall=0.41 (threshold=0.7); llm_spam=0.00 (The issue provides a highly specific and descriptive title. The steps to reproduce are clear and include terminal output demonstrating the bug. The expected and actual behaviors are explicitly stated. Furthermore, the reporter has identified the root cause in the codebase, pointing out the mismatch between the validation list and the parsing logic. Although the media URLs section says 'None', an image link is provided in the body, and the terminal output itself serves as strong evidence. There are no signs of spam or template farming.)"
  },
  "duplicate": {
    "isDuplicate": true,
    "originalIssue": 53364,
    "similarity": 0.7360787373045256,
    "topSimilar": [
      {
        "issueNumber": 53364,
        "title": "[BUG] [v0.0.7] `cortex models list --sort context` and `--sort created` are silently ignored and fall back to `--sort id` without warning",
        "similarity": 0.7360787373045256
      },
      {
        "issueNumber": 53394,
        "title": "models list --offset with provider filter shows wrong empty message",
        "similarity": 0.59161454318244
      },
      {
        "issueNumber": 51931,
        "title": "[BUG] [v0.0.7] `cortex models list --sort` rejects otherwise valid sort tokens when padded with spaces (`VALID_SORT_VALUES` uses exact string match, no trim).",
        "similarity": 0.5779296313605046
      },
      {
        "issueNumber": 52374,
        "title": "[BUG] [v0.0.7] cortex models list --sort name text output still groups by provider — JSON output sorts globally by name across all providers",
        "similarity": 0.5534260817169319
      },
      {
        "issueNumber": 53033,
        "title": "[BUG][v0.1.0] `cortex models list` does not show context window size",
        "similarity": 0.5380677918976001
      }
    ]
  },
  "editHistory": {
    "fraudScore": 0,
    "editCount": 0,
    "details": "No edits detected"
  },
  "codeVerification": {
    "plausible": true,
    "confidence": 1,
    "reasoning": "The code perfectly matches the bug report. The `VALID_SORT_VALUES` array allows \"context\" and \"created\" as sort options, but the `ModelSortOrder` enum and its `FromStr` implementation do not support them. The use of `unwrap_or_default()` causes these unsupported but \"valid\" options to silently fall back to sorting by ID.",
    "codeEvidence": "In `src/cortex-cli/src/models_cmd.rs`, `VALID_SORT_VALUES` includes `\"context\"` and `\"created\"`. However, `ModelSortOrder::from_str` only handles `\"id\"`, `\"name\"`, and `\"provider\"`. When parsing the sort order, `sort_by.parse().unwrap_or_default()` is used, which silently swallows the parse error for `\"context\"` and `\"created\"` and falls back to the default `ModelSortOrder::Id`.",
    "screenshotValid": true,
    "screenshotReasoning": "The screenshot analysis tool verified the screenshot as valid."
  }
}

_{Validated by Atlas • 2026-04-16}