Add ChatRole enum for message history schema
メンテナーはふだん 5 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
まず redisvl/extensions/message_history/schema.py から始め、次に redisvl/extensions/message_history/base_history.py のロール検証と比較します。既存のメッセージ履歴テストを確認し、有効な強制変換と無効なロールのカバレッジを追加して、make format、make check-types、テストスイートで文字列入力の後方互換性を確認します。
索引モデルが issue の本文から書いたものです。
説明
Summary
Introduce a ChatRole enum for message history while preserving backward compatibility with string role inputs.
Level: Beginner
Current State
ChatMessage.roleis currently a plainstrinredisvl/extensions/message_history/schema.py.- Role handling is split between schema and base history validation logic.
Proposed Change
- Add a
ChatRoleenum (user,assistant,system,tool). - Update
ChatMessageto accept enum values and coerce valid strings. - Align checks in
redisvl/extensions/message_history/base_history.py. - Export the enum from the message history module.
Definition of Done
- Enum-based role validation is in place.
- Existing string-based behavior remains valid (non-breaking).
- Unit tests cover valid coercion and invalid role errors.
make format,make check-types, and tests pass.
Out of Scope
- Redesigning message history schemas beyond the role type.
- 主要言語
- Python
- スター
- 431
- フォーク
- 107
- 平均マージ
- 5日 17時間
- マージ済み PR(30日)
- 16
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
redis/redis-vl-python のほかの issue
-
Add Python 3.15 support対応中かも @vishal-bala が 1 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
redis/redis-vl-python#757 · コメント 1 件 · リアクション 1 件 ·
メンテナーはふだん 5 日以内に返信
-
SearchIndex.drop_keys does not validate cluster hash-tag co-location (inconsistent with drop_documents)再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープンbug good first issue index
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
redis/redis-vl-python#601 ·
メンテナーはふだん 5 日以内に返信
-
documentation
難易度 1/5 1時間未満 初心者へのやさしさ 62/100
redis/redis-vl-python#300 ·
メンテナーはふだん 5 日以内に返信
-
Migration executor cannot read an FT.AGGREGATE cursor over a RESP3 connection, raising KeyError対応中かも @Kuang-xianxin が 41 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
redis/redis-vl-python#714 ·
メンテナーはふだん 5 日以内に返信
-
Migration readiness checks read FT.INFO with string keys, so under RESP3 they always report a healthy index対応中かも @Kuang-xianxin が 41 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 74/100
redis/redis-vl-python#713 ·
メンテナーはふだん 5 日以内に返信
redis/redis-vl-python の issue をすべて見る
似ている issue
-
`lvm_compact` has no self-hosted twin, and `docs/158` §5 says every language-face component mustオープンbug sev-3 vm
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
sierra-research/tau2-bench#635 ·
メンテナーはふだん 1 日以内に返信
-
The add-ssm-model skill requires a Cython simulator; the contributing guide allows Python ones対応中かも @fmuia が今日担当しました。 オープンdocumentation skill
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
lnccbrown/ssm-simulators#406 ·
メンテナーはふだん 4 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 95/100
btclib-org/btclib-ecc#309 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
autowarefoundation/autoware_carla_scenario#281 ·
メンテナーはふだん 1 日以内に返信