fix(models): omit rejected GLM Flash effort on OpenCode Go - #102

ℹ️ No critical issues. Two small suggestions, one inline and one below.

Reviewed changes

Reviewed the whole diff. It stops the direct OpenCode Go GLM Flash route from sending an effort variant, and it keeps the OpenRouter ladder.

• Direct route sends no variant: effort: [] makes getModelEffortLevels return an empty ladder. resolveRung then returns undefined, so runPromptTurn leaves out variant for this route. openRouterEffort is now explicit, so the OpenRouter route and ossEffortFloor still get low, high, max.
• Regression test: the new test/models.test.ts case can fail. The old ladder resolved position 0.5 to high on the direct route, and the test now requires undefined.
• Catalog drift exception: the ladder test in models-catalog.main.test.ts now skips opencode-go/glm-flash and only checks that effort equals [].

ℹ️ Nitpicks

• test/models-catalog.main.test.ts:167 — the special case turns off the drift check for this route entirely. If models.dev stops listing the ladder, nothing will say the exception is no longer needed. Adding expect(rungs(publishedEffort(alias.resolve))).toBeDefined() before the return would make the test fail at that point, which prompts someone to remove the exception.

  ^{｜ Fix all ➔ ｜ Fix 👍s ➔ ｜ View workflow run ｜ Using Claude Opus (free via Pullfrog for OSS) ｜ 𝕏}

✅ No new issues found. Both points from the last review are fixed.

Reviewed changes

Reviewed cdaa458, which changes only a code comment and one test assertion since the last Pullfrog review.

• Rewrote the glm-flash code comment: the comment at models.ts:877-878 now quotes the provider's error text. The cross-repo #964 and run-ID reference is gone, so the comment no longer points at a pullfrog issue number that has nothing to do with this.
• Added a check that the drift exception is still needed: test/models-catalog.main.test.ts:168 now asserts rungs(publishedEffort(alias.resolve)) is defined. rungs returns undefined when the list is empty or missing, so this fails if models.dev stops listing any effort rung for opencode-go/glm-5.3-flash. That failure tells someone to remove the special case.

  ^{｜ View workflow run ｜ Using Claude Opus (free via Pullfrog for OSS) ｜ 𝕏}