Skip to content

fix(config): add claude-opus-4.8 to FALLBACK_MODELS - #242

Open
fengseekling-coder wants to merge 1 commit into
jwadow:mainfrom
fengseekling-coder:fix/add-claude-opus-4.8-and-venv-ignore
Open

fix(config): add claude-opus-4.8 to FALLBACK_MODELS#242
fengseekling-coder wants to merge 1 commit into
jwadow:mainfrom
fengseekling-coder:fix/add-claude-opus-4.8-and-venv-ignore

Conversation

@fengseekling-coder

Copy link
Copy Markdown

What

  • Adds claude-opus-4.8 to FALLBACK_MODELS so the fallback list stays
    in sync with the model catalog (4.5/4.6/4.7 are already present).
  • Adds .venv/ and .DS_Store to .gitignore — these are dev-machine
    artifacts that show up in git status and add noise.

Why

  • claude-opus-4.8 should match the same upstream flag pattern as the
    rest of the catalog.
  • .gitignore was missing the two most common Python-on-macOS
    artifacts.

Test

  • python -c "from kiro.config import FALLBACK_MODELS; print(any(m['modelId']=='claude-opus-4.8' for m in FALLBACK_MODELS))"
    True
  • git status no longer reports .venv/ or .DS_Store as untracked.

Co-authored-by: Cursor cursoragent@cursor.com

Made with Cursor

- kiro/config.py: register claude-opus-4.8 alongside 4.5/4.6/4.7
  so the fallback list stays in sync with the upstream model catalog
- .gitignore: ignore .venv/ and .DS_Store (macOS artifact) — these
  are accidentally visible to git on dev machines and noise the status

Co-authored-by: Cursor <cursoragent@cursor.com>
@cla-bot

cla-bot Bot commented Jul 4, 2026

Copy link
Copy Markdown

Thanks for the PR! 🎉

Before merge, we need a one-time CLA confirmation.
It confirms that you have the right to contribute this code and allow the project to use it.

Full CLA text:
https://github.com/jwadow/kiro-gateway/blob/main/CLA.md

Please reply once with:

I have read the CLA and I accept its terms

You need to write once, all further messages from me can be ignored.

@fengseekling-coder

fengseekling-coder commented Jul 7, 2026 via email

Copy link
Copy Markdown
Author

@ankitcharolia

Copy link
Copy Markdown

@fengseekling-coder could you give a try to this gateway: https://github.com/ankitcharolia/kiro-gateway

It works quite well with All AI harness and actively being developed. The most important thing is that it is ACP compliant

@fengseekling-coder

fengseekling-coder commented Aug 14, 2026 via email

Copy link
Copy Markdown
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants