Skip to content

learning to change opencode is going to be hard - #10

Merged
Zenwoh merged 1 commit into
devfrom
fix-api-showing-model-provider-id-instead-of-our-naming-n1920
Feb 13, 2026
Merged

Zenwoh merged 1 commit into
devfrom
fix-api-showing-model-provider-id-instead-of-our-naming-n1920

Conversation

@Zenwoh

@Zenwoh Zenwoh commented Feb 13, 2026

Copy link
Copy Markdown
Collaborator

What does this PR do?

buh

How did you verify your code works?

buh

@Zenwoh Zenwoh self-assigned this Feb 13, 2026
@github-actions

Copy link
Copy Markdown

Hey! Your PR title learning to change opencode is going to be hard doesn't follow conventional commit format.

Please update it to start with one of:

  • feat: or feat(scope): new feature
  • fix: or fix(scope): bug fix
  • docs: or docs(scope): documentation changes
  • chore: or chore(scope): maintenance tasks
  • refactor: or refactor(scope): code refactoring
  • test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, opencode).

See CONTRIBUTING.md for details.

@Zenwoh
Zenwoh merged commit 857e22b into dev Feb 13, 2026
2 checks passed
@coderabbitai

coderabbitai Bot commented Feb 13, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

Model identifier display normalization logic is added to two files. When the provider ID is "mammouth-ai", the displayed model identifier is derived by extracting the last segment after a forward slash from the full model ID. For other providers, the original model ID is displayed unchanged. A helper function implements this normalization in run.ts for the start-banner display, and a memoized selector implements it in the session component. The changes affect only the text shown in user-facing messages and UI elements; no internal logic or model ID handling is modified.

🚥 Pre-merge checks | ❌ 4
❌ Failed checks (3 warnings, 1 inconclusive)
Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is largely incomplete, with required sections filled only with placeholder text ('buh') instead of substantive information about the changes and verification. Complete both sections: explain what the PR does (model display normalization) and how verification was performed (testing the changes).
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Merge Conflict Detection ⚠️ Warning ⚠️ Unable to check for merge conflicts: Failed to fetch base branch: From https://github.com/mammouth-ai/code
! [rejected] dev -> dev (non-fast-forward)
+ 0eb2199...857e22b dev -> origin/dev (forced update)
Title check ❓ Inconclusive The pull request title is vague and generic, using non-descriptive language that fails to convey meaningful information about the actual changes (model display normalization in CLI and UI). Replace with a descriptive title like 'Normalize model display for mammouth-ai provider' that clearly indicates the main change.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-api-showing-model-provider-id-instead-of-our-naming-n1920
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch fix-api-showing-model-provider-id-instead-of-our-naming-n1920
  • Create stacked PR with resolved conflicts
  • Post resolved changes as copyable diffs in a comment

No actionable comments were generated in the recent review. 🎉


Comment @coderabbitai help to get the list of available commands and usage tips.

@Zenwoh
Zenwoh deleted the fix-api-showing-model-provider-id-instead-of-our-naming-n1920 branch February 17, 2026 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant