fix: refactor gnoblog-cli (#63)

* start refac

* add cmds

* wip setup post command

* wip save, move things around

* finished singlepost

* added comments

* extract errors

* support multicall

* update gno version

* update titles & remove title frontmatter

* add title & disclaimer to poc2 for consistency

* update cfg caller

* remove validate-chain func

* fix post title parsing

* fix

* chore: better error output

Signed-off-by: moul <[email protected]>

* chore: fmt + tidy

Signed-off-by: moul <[email protected]>

* chore: fix frontmatter parsing

Signed-off-by: moul <[email protected]>

* chore: add missing errcheck

Signed-off-by: moul <[email protected]>

* chore: fixup

Signed-off-by: moul <[email protected]>

* move folders

* add gitignore, fix package names

* unify post function, minor modifications

* fix md text in posts

* move err check

* move IO init to main

* add comments, fix removeWhitespace

* clarify error

* add key check

* move parsePost, change err check

* change post func signature

* add initial test & mocks

* integration test setup

* remove ./build

* fix leftover titles

* add output confirmation

* wip integ tests

* remove integration test setup

* use tm2/pkg/commands instead of ffcli

* update err message

* updat err msg again

* add condition

* make if more readable, add exit condition

* go mod tidy

* add readme

* update readme

* update makefile, chagne error

* update main readme

---------

Signed-off-by: moul <[email protected]>
Co-authored-by: moul <[email protected]>

Author: leohhhn

.gitignore

@@ -2,3 +2,5 @@ reporting/reporting
 reporting/output
 .idea
 .DS_Store
+
+./build

Makefile

@@ -1,5 +1,5 @@
 install:
-	go install ./gnoblog-cli
+	go install ./cmd/gnoblog-cli
 
 test:
 	go test -v ./...

README.md

@@ -15,7 +15,7 @@ The blog posts are written in markdown format, and include a frontmatter prefix
 
 See [CONTRIBUTING.md](./CONTRIBUTING.md).
 
-## CLI ([`./gnoblog-cli`](./gnoblog-cli))
+## CLI ([`./gnoblog-cli`](./cmd/gnoblog-cli))
 
 _See [#1](https://github.com/gnolang/blog/issues/1)_
 

cmd/gnoblog-cli/README.md

@@ -0,0 +1,27 @@
+# `gnoblog-cli`
+
+`gnoblog-cli` is a configurable tool that allows for easy deployment of blog posts to the
+`r/gnoland/blog` realm.
+
+All blog posts posted with `gnoblog-cli` need to be in the correct format:
+- Filename: `README.md`,
+- Frontmatter must be of a specific format,
+- There must be an H1 in below the frontmatter, which will be treated as the post title.
+
+For reference, see existing posts, like [Peace!](../../posts/2022-05-02_peace)
+
+`gnoblog-cli` utilizes keys from the local Gno keybase to deploy to
+the blog realm, whose path is configurable via the `--pkgpath` flag.
+
+`gnoblog-cli` can has the ability to:
+- Post a single blog post `README.md` file to the chain, in which case
+it takes the direct path to the file as an argument,
+- Batch post multiple `README.md` files by recursive looking through directories,
+finding `README.md`s, and pack all of them to a single transaction to be 
+broadcast to the chain.
+
+
+  
+
+
+

cmd/gnoblog-cli/clients.go

@@ -0,0 +1,26 @@
+package main
+
+import (
+	"github.com/gnolang/gno/gno.land/pkg/gnoclient"
+	rpcclient "github.com/gnolang/gno/tm2/pkg/bft/rpc/client"
+	"github.com/gnolang/gno/tm2/pkg/crypto/keys"
+)
+
+func initSigner(cfg *cliCfg, password string) (gnoclient.SignerFromKeybase, error) {
+	kb, err := keys.NewKeyBaseFromDir(cfg.GnoHome)
+	if err != nil {
+		return gnoclient.SignerFromKeybase{}, err
+	}
+
+	return gnoclient.SignerFromKeybase{
+		Keybase:  kb,
+		Account:  cfg.KeyName,
+		Password: password,
+		ChainID:  cfg.ChainId,
+	}, nil
+}
+
+func initRPCClient(cfg *cliCfg) rpcclient.Client {
+	remote := cfg.Remote // todo: validate if chainID & remote match?
+	return rpcclient.NewHTTP(remote, "/websocket")
+}

cmd/gnoblog-cli/errors.go

@@ -0,0 +1,9 @@
+package main
+
+import "errors"
+
+var (
+	ErrInvalidNumberOfArgs = errors.New("invalid number of args; please input exactly 1 argument")
+	ErrEmptyKeyName        = errors.New("key name cannot be empty")
+	ErrNoNewOrChangedPosts = errors.New("no new posts have been added, nor have any existing posts been changed")
+)

cmd/gnoblog-cli/main.go

@@ -0,0 +1,31 @@
+package main
+
+import (
+	"context"
+	"github.com/gnolang/gno/tm2/pkg/commands"
+	"os"
+)
+
+func main() {
+	io := commands.NewDefaultIO()
+	cmd := newRootCmd(io)
+
+	cmd.Execute(context.Background(), os.Args[1:])
+}
+
+func newRootCmd(io commands.IO) *commands.Command {
+	cmd := commands.NewCommand(
+		commands.Metadata{
+			ShortUsage: "<subcommand> [flags] [<arg>...]",
+			LongHelp:   "The CLI for easy use of the r/blog realm",
+		},
+		commands.NewEmptyConfig(),
+		commands.HelpExec,
+	)
+
+	cmd.AddSubCommands(
+		newPostCommand(io),
+	)
+
+	return cmd
+}

cmd/gnoblog-cli/mock_test.go

@@ -0,0 +1,90 @@
+package main
+
+import (
+	"io"
+)
+
+type (
+	mockGetPassword func() (string, error)
+)
+
+type mockIO struct {
+	getPassword mockGetPassword
+}
+
+func (m mockIO) In() io.Reader {
+	//TODO implement me
+	panic("implement me")
+}
+
+func (m mockIO) Out() io.WriteCloser {
+	//TODO implement me
+	panic("implement me")
+}
+
+func (m mockIO) Err() io.WriteCloser {
+	//TODO implement me
+	panic("implement me")
+}
+
+func (m mockIO) SetIn(in io.Reader) {
+	//TODO implement me
+	panic("implement me")
+}
+
+func (m mockIO) SetOut(out io.WriteCloser) {
+	//TODO implement me
+	panic("implement me")
+}
+
+func (m mockIO) SetErr(err io.WriteCloser) {
+	//TODO implement me
+	panic("implement me")
+}
+
+func (m mockIO) Println(args ...interface{}) {
+	//TODO implement me
+	panic("implement me")
+}
+
+func (m mockIO) Printf(format string, args ...interface{}) {
+	//TODO implement me
+	panic("implement me")
+}
+
+func (m mockIO) Printfln(format string, args ...interface{}) {
+	//TODO implement me
+	panic("implement me")
+}
+
+func (m mockIO) ErrPrintln(args ...interface{}) {
+	//TODO implement me
+	panic("implement me")
+}
+
+func (m mockIO) ErrPrintfln(format string, args ...interface{}) {
+	//TODO implement me
+	panic("implement me")
+}
+
+func (m mockIO) GetCheckPassword(prompts [2]string, insecure bool) (string, error) {
+	//TODO implement me
+	panic("implement me")
+}
+
+func (m mockIO) GetConfirmation(prompt string) (bool, error) {
+	//TODO implement me
+	panic("implement me")
+}
+
+func (m mockIO) GetPassword(prompt string, insecure bool) (string, error) {
+	if m.getPassword != nil {
+		return m.getPassword()
+	}
+	return "", nil
+}
+
+func (m mockIO) GetString(prompt string) (string, error) {
+	//TODO implement me
+	panic("implement me")
+}