From 4acd5cbce9945ef044dc997cb82cfad8273a0dbc Mon Sep 17 00:00:00 2001 From: Teddy Wing Date: Sat, 25 Jul 2020 18:38:10 +0200 Subject: Add git-sugapply bin An executable to apply a suggested change to the appropriate file in the repo. Add a new `SuggestionUrl` type that allows us to extract the necessary data to fetch a suggestion comment from the GitHub API using a GitHub pull request comment URL. --- Cargo.lock | 1 + Cargo.toml | 1 + src/bin/git-sugapply.rs | 25 +++++++++++++++++++++++++ src/lib.rs | 7 ++++++- src/url.rs | 45 +++++++++++++++++++++++++++++++++++++++++++++ 5 files changed, 78 insertions(+), 1 deletion(-) create mode 100644 src/bin/git-sugapply.rs create mode 100644 src/url.rs diff --git a/Cargo.lock b/Cargo.lock index e8b4f7f..90162ca 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -249,6 +249,7 @@ dependencies = [ "tempfile", "thiserror", "unidiff", + "url", ] [[package]] diff --git a/Cargo.toml b/Cargo.toml index 7f5a472..4fd28dd 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -12,3 +12,4 @@ serde_json = "1.0.56" tempfile = "3.1.0" thiserror = "1.0.20" unidiff = "0.3.3" +url = "2.1.1" diff --git a/src/bin/git-sugapply.rs b/src/bin/git-sugapply.rs new file mode 100644 index 0000000..d569147 --- /dev/null +++ b/src/bin/git-sugapply.rs @@ -0,0 +1,25 @@ +use std::env; +use std::process; + +use git_suggested_patch::{Client, SuggestionUrl}; + + +fn main() { + let args: Vec<_> = env::args().collect(); + + if args.len() < 2 { + process::exit(111); + } + + let url: SuggestionUrl = args[1].parse().unwrap(); + + let client = Client::new( + env!("GITHUB_TOKEN"), + &url.owner, + &url.repo, + ); + + let suggestion = client.fetch(&url.comment_id).unwrap(); + + suggestion.apply().unwrap(); +} diff --git a/src/lib.rs b/src/lib.rs index 3b94993..2febe7d 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -1,6 +1,11 @@ #![warn(rust_2018_idioms)] +mod url; + +pub use crate::url::SuggestionUrl; + + use std::fs; use std::fs::{File, OpenOptions}; use std::io::{BufRead, BufReader, Write}; @@ -109,7 +114,7 @@ impl Suggestion { s.replace("```", "") } - fn apply(&self) -> Result<(), Error> { + pub fn apply(&self) -> Result<(), Error> { let repo = Repository::open(".").unwrap(); let repo_root = repo.workdir().unwrap(); diff --git a/src/url.rs b/src/url.rs new file mode 100644 index 0000000..d635d6f --- /dev/null +++ b/src/url.rs @@ -0,0 +1,45 @@ +use std::str::FromStr; + +use thiserror::Error; + +use url; +use url::Url; + + +#[derive(Debug, Error)] +pub enum Error { + #[error("Unable to parse URL")] + Url(#[from] url::ParseError), + + #[error("URL has no path")] + NoPath, + + #[error("URL has no fragment")] + NoFragment, +} + +#[derive(Debug)] +pub struct SuggestionUrl { + pub owner: String, + pub repo: String, + pub comment_id: String, +} + +impl FromStr for SuggestionUrl { + type Err = Error; + + fn from_str(s: &str) -> Result { + let url = Url::parse(s)?; + let path = url.path_segments() + .ok_or(Error::NoPath)? + .collect::>(); + + Ok(SuggestionUrl { + owner: path[0].to_owned(), + repo: path[1].to_owned(), + comment_id: url.fragment() + .ok_or(Error::NoFragment)? + .replacen("discussion_r", "", 1), + }) + } +} -- cgit v1.2.3