fix gen-rpc-doc

Author: chenyukang

Makefile

@@ -104,9 +104,8 @@ doc-deps: ## Build the documentation for the local package and all dependencies.
 .PHONY: gen-rpc-doc
 gen-rpc-doc: submodule-init ## Generate rpc documentation
 	cd devtools/doc/rpc-gen && cargo build
-	cd docs/ckb_rpc_openrpc/ && git reset --hard && git clean -fd
-	./target/debug/ckb-rpc-gen rpc/README.md
 	./target/debug/ckb-rpc-gen --json
+	./target/debug/ckb-rpc-gen rpc/README.md
 
 .PHONY: gen-hashes
 gen-hashes: ## Generate docs/hashes.toml

devtools/doc/rpc-gen/src/main.rs

@@ -22,18 +22,22 @@ fn dump_openrpc_json() -> Result<(), Box<dyn std::error::Error>> {
         "finished dump openrpc json for version: {:?} at dir: {:?}",
         version, json_dir
     );
-    // run git commit all changes before generate rpc readme
-    run_command("git", &["add", "."], Some(OPENRPC_DIR));
-    run_command(
-        "git",
-        &[
-            "commit",
-            "-m",
-            &format!("update openrpc json for version: {:?}", version),
-        ],
-        Some(OPENRPC_DIR),
-    );
-    run_command("git", &["push"], Some(OPENRPC_DIR));
+
+    if is_git_repo_dirty() {
+        // run git commit all changes before generate rpc readme
+        eprintln!("commit OpenRPC changes to repo: {}", get_git_remote_url());
+        run_command("git", &["add", "."], Some(OPENRPC_DIR));
+        run_command(
+            "git",
+            &[
+                "commit",
+                "-m",
+                &format!("update openrpc json for version: {:?}", version),
+            ],
+            Some(OPENRPC_DIR),
+        );
+        run_command("git", &["push"], Some(OPENRPC_DIR));
+    }
     Ok(())
 }
 

devtools/doc/rpc-gen/src/utils.rs

@@ -67,3 +67,17 @@ pub(crate) fn checkout_tag_branch(version: &str) {
         run_command("git", &["checkout", "-b", version], dir);
     }
 }
+
+pub(crate) fn is_git_repo_dirty() -> bool {
+    let res = run_command("git", &["status", "--porcelain"], Some(OPENRPC_DIR));
+    res.map(|s| !s.is_empty()).unwrap_or(false)
+}
+
+pub(crate) fn get_git_remote_url() -> String {
+    run_command(
+        "git",
+        &["config", "--get", "remote.origin.url"],
+        Some(OPENRPC_DIR),
+    )
+    .map_or("".to_string(), |s| s.trim().to_string())
+}