refactor: message format and enable cache prompt

[jjleng]

No description provided.

[Copilot]

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

cp-agent/cp_agent/utils/message_formats.py:46

• Removing the timestamp field from the Message TypedDict may impact parts of the system (e.g., chat history) that rely on message time information. Consider adding timestamp handling elsewhere if it is expected in production.

-    timestamp: str = field(default_factory=lambda: datetime.now().isoformat())

[Copilot]

The caching implementation for assistant messages uses an ephemeral cache_control via create_text_block in one branch and appends a separate cachePoint block in the other branch. Consider standardizing the caching representation to avoid potential inconsistencies in message format.

Suggested change

	message_content: list[MessagePart] = [create_text_block(content)]
	if not IS_BEDROCK:
	message_content = [create_text_block(content, "ephemeral")]
	else:
	message_content.append({"cachePoint": {"type": "default"}})
	cache_type = "ephemeral" if not IS_BEDROCK else "default"
	message_content: list[MessagePart] = [
	create_text_block(content, cache_type),
	{"cachePoint": {"type": cache_type}},
	]

[jjleng]

bedrock and other vendors use different formats, sigh