Fix: Kiro Gateway Anthropic system message compatibility

[ytchen175]

Branch: main
Title: Fix: Kiro Gateway Anthropic system message compatibility

Description

• Fixes Claude Code requests that place a system message inside the Anthropic messages array.
• Promotes inline system instructions into the Kiro system prompt instead of rejecting them or treating them as user input.
• Preserves the existing user/assistant conversation history and adds regression coverage.

Root Cause

Claude Code sent a request containing messages[1].role = "system". The gateway schema only accepted user and assistant roles, so Pydantic rejected the request with HTTP 422 before conversion.

Old Logic

• AnthropicMessage.role was limited to user and assistant.
• The converter attempted to process every message as conversation history.
• Inline system messages could not pass validation and never reached the Kiro adapter.

New Logic

• Allow system as a compatibility role in AnthropicMessage.
• Extract all inline system messages before conversation conversion.
• Combine extracted system text in source order and merge it with the top-level Anthropic system field.
• Convert only user and assistant messages into Kiro conversation history.

Why it works

• Claude Code's actual request shape is accepted by the API model.
• System instructions retain system semantics and are not silently converted into user messages.
• Existing conversation ordering remains unchanged.
• The new tests cover extraction, ordering, content blocks, and the complete Anthropic-to-Kiro payload.

DoD

• Accept Anthropic inline system messages.
• Preserve system semantics during Kiro conversion.
• Add regression tests for the reported HTTP 422 shape.
• Run the targeted gateway test suite.
• Verify a direct gateway request and a Claude Code smoke request.

Code Changes

• kiro/models_anthropic.py
  • Allows the compatibility system role in AnthropicMessage.
• kiro/converters_anthropic.py
  • Adds inline system-message extraction and combines it with the top-level system prompt.
  • Excludes system messages from user/assistant conversation history.
• tests/unit/test_converters_anthropic.py
  • Adds regression coverage for extraction, ordering, content blocks, and final payload conversion.
• .gitignore
  • Ignores the local Python virtual environment and normalizes the file ending.

Validate Results

./.venv/bin/pytest -q \
  tests/unit/test_models_anthropic.py \
  tests/unit/test_converters_anthropic.py \
  tests/unit/test_routes_anthropic.py

Result: 249 passed, 1 warning.

Additional smoke checks:

• Direct Anthropic-compatible gateway request: HTTP 200.
• Claude Code request through the gateway: successful response.
• Claude Code model usage: claude-opus-5.
• git diff --check: passed.

Commit Messages

• Fix: Kiro Gateway Anthropic system message compatibility
  • Accepts Claude Code's inline system-message format.
  • Preserves system prompt semantics and adds regression tests.

[cla-bot]

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.

[Copilot]

Pull request overview

This PR updates the Anthropic adapter to accept Claude Code’s non-standard inline system messages (i.e., messages[].role == "system") and promotes them into the effective system prompt so they keep system semantics rather than being rejected at validation time.

Changes:

• Expanded AnthropicMessage.role validation to allow a compatibility "system" role.
• Added split_anthropic_system_messages() and updated the Anthropic→Kiro conversion path to extract/merge inline system instructions into the system prompt while excluding them from conversation history.
• Added regression tests covering inline system-message acceptance and payload promotion; updated .gitignore to ignore .venv/.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
kiro/models_anthropic.py	Allows "system" as a compatibility role in AnthropicMessage.
kiro/converters_anthropic.py	Extracts inline system messages into the system prompt and skips them during message-history conversion.
tests/unit/test_converters_anthropic.py	Adds tests for inline system-message skipping/promotion and related behavior.
fix_anthropic-system-message-compatibility_en.md	English write-up of the change rationale/validation steps.
fix_anthropic-system-message-compatibility_zh.md	Chinese write-up of the change rationale/validation steps.
.gitignore	Ignores local virtualenv directory and normalizes the htmlcov/ entry.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[ankitcharolia]

@ytchen175 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