Skip to content

Conversation

@pacificbelt30
Copy link
Collaborator

Overview

ユーザ定義のコマンドをシームレスに実行できるようにした。
現在の Alert-Menta の構成では、.alert-menta.user.yaml にカスタムコマンドを定義した場合、それに合わせて alert-menta.yaml のワークフローにも変更を加える必要があった。
また、カスタムコマンドにユーザによる入力(intent)を加えることは出来なかった。

Cause and remedy (in the case of bug fixes)

What i did

  • alert-menta.yaml のカスタムコマンドへの対応
    • 現在は、alert-menta.yaml 中に/ask等の事前定義コマンドがコメントされたときにトリガーするような設定が記載されている
    • カスタムコマンドを定義した際にはここにカスタムコマンドを追加する必要があった
  • .alert-menta.user.yaml に require-intent フィールドを追加
    • intent の有無を指定できる
  • コマンドの指定、intent の指定についてバリデーションを行い、不正な場合は Usage をコメントに出力する
    image
    image

Change Result

特になし

Content outside the scope of this PR

  • README の変更は行っていない

Precautions (What to let members know)

  • alert-menta.yaml と.alert-menta.user.yaml に変更を加えているため、Actions 上での動作を確認する必要がある
  • yaml に項目を追加しているためそれに合わせて main_test.go を修正する必要がある。(本 PR にてテストを提出)
  • 本 PR に合わせて README の変更が必要となる。

Usage and procedures after the change

  • .alert-menta.user.yaml に require_intent というフィールドを追加する

Especially where you want us to review

特になし

notes

特になし

@pacificbelt30 pacificbelt30 requested a review from kechigon as a code owner March 21, 2025 12:09
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.

2 participants