前回、Cloudflare Workers + Rust + Slack botを開発しました。
(https://reo0306.hatenablog.com/entry/2024/07/10/214315)
これはissueのみなので、改修して、Pull Requestもできるようにしようと思ったが、先にバッチを作ることにしました。
なので、今回は、表題の通り、PullRequestをSlackに通知するバッチを作ってみました。
仕様
github apiからの取得結果を元に、mrkdwn形式のSlackメッセージにフォーマットして、通知する
という感じです。
ざっくりこんな流れです。
- github apiから全リポジトリ取得
- リポジトリ一覧をループし、各リポジトリのOpenになってるPullRequestを取得
- PullRequestのreviewersとreviewを取得
- PullRequestをapprovalにしてないreviewersのmember取得
- Slack通知メッセージ形式にフォーマット
- Slackへ通知
実装
概要
reqwestを使って、github apiから情報を取得し、slackへpostする、だけなんだが、
github apiにどうリクエストして、どのレスポンスがあってなどなどを調べるのに時間がかかった。
github apiのコードだけ軽く説明。詳しくはgithubのリポジトリを見てみてください。
github api
headersで注意点はUSER_AGENTが必要なこと。これがないとリクエストが失敗する。(適当な値を設定してます。)
github apiの各サンプルのcurlコマンドにUSER_AGENTはないので、ハマってしまいました。
pub fn new() -> Self { let token = env::var("GITHUB_TOKEN").unwrap(); let mut headers = HeaderMap::new(); headers.insert(USER_AGENT, HeaderValue::from_static("request")); headers.insert(ACCEPT, HeaderValue::from_static("application/vnd.github.v3+json")); headers.insert(AUTHORIZATION, HeaderValue::from_str(&format!("token {}", token)).unwrap()); headers.insert("X-GitHub-Api-Version", HeaderValue::from_static("2022-11-28")); let client = reqwest::Client::new(); Self { client, headers } } pub async fn fetch<T: for<'de> Deserialize<'de>>(&self, url: &str) -> Result<Vec<T>> { let response = self.client .get(url) .headers(self.headers.clone()) .send() .await?; match response.error_for_status_ref() { Ok(_) => { let repos = response.json::<Vec<T>>().await?; Ok(repos) }, Err(e) => { Err(e.into()) } } }
苦労した点
fetch関数のジェネリクスの使い方で時間がかかりました。
github apiリクエスト処理のコードはどのapiでも同じ関数を使って取得できますが、jsonからデシリアライズするときに指定する型はapiによって、異なるようにしてました。
型パラメータでどの型でも任意の型として戻り値になるようにしたかったのですが、デシリアライズのトレイト境界が必要だったようで、これに時間がかかりました。
関数のコードを変更することや前段のコードから変更することで、どうにかなったかもしれませんが、これで1つの関数で複数のapiを処理できるようになったので、時間がかかったかいはありました。
通知結果
※メッセージフォーマット
Open Pull Request タイトル - PRのURL PRの状態 - Created by 作成者 on 作成日
上記フォーマットで通知されるようにして実際に通知されたのが、これです。
見やすいように修正が必要だなと思ってますが、未レビューのPullRequestをSlackへ通知することはできました。
まとめ
勉強がてら個人のリポジトリ内での通知で開発しましたが、業務用にするのであれば、Organizations指定でリポジトリ一覧を取得したりなど、少し変更が必要かなと思いました。
Rustでいえば、共通関数を用意するにあたって、ジェネリクスやトレイト境界は必要になってくるんだなと、というのが勉強になりました。とはいえ、DIとかDIコンテナするとか、ディスパッチ使うとか、今回実装した方法以外ですっきりしたやり方があるんだろうなーとも思いました。ここは経験不足、Rustの技術不足を感じました。
次はPullRequestしたとき、gihtubのwebhookで、通知されるようにissueで開発したコードを改修しようと思います。