mcr update and readme (#9)

Reviewed-on: #9
Co-authored-by: jake <jake.young.dev@gmail.com>
Co-committed-by: jake <jake.young.dev@gmail.com>

.gitea/workflows/security.yaml

@@ -1,25 +1,17 @@
 name: "code scans"
-on: [push, pull_request] #runs on pushes to any branch
+on: 
+  push:
+    branches:
+      - main
+  tags:
+    - v*
+  pull_request:
 
 jobs:
   scans:
-    runs-on: smoke-test
+    runs-on: fire
     steps:
-      - name: "clone code"
-        uses: actions/checkout@v4
-      
-      - name: "install go"
-        uses: https://code.jakeyoungdev.com/actions/install-go@v0.1.3
-        with:
-          commands: |
-              golang.org/x/vuln/cmd/govulncheck@latest
+      - uses: actions/checkout@v4
 
-      - name: "dependency and stdlib scan"
-        uses: https://code.jakeyoungdev.com/actions/report-vulns@master
-        with:
-          manager: go
-
-      - name: "static code analysis"
-        uses: securego/gosec@master
-        with:
-          args: ./...
+      - name: "dependency scan and static code analysis"
+        uses: https://code.jakeyoungdev.com/actions/donotpassgo@v1.1.0

README.md

@@ -1,14 +1,24 @@
 # mctl
 mctl is a terminal-friendly remote console client
 
-## Installation
+# Index
+1. [Installation](#installation)
+2. [Setup](#setup)
+3. [Documentation](#documentation)
+4. [Security](#security)
+5. [Development](#development)
+
+<br />
+
+# Installation
 Install mctl using golang
 ```bash
 go install code.jakeyoungdev.com/jake/mctl@main #it is recommended to use a tagged version
 ```
+
 <br />
 
-## Setup
+# Setup
 ### Configuring mctl
 mctl requires a one-time setup via the 'config' command before interacting with any servers, password is entered securely from the terminal and encrypted
 ```bash
@@ -75,7 +85,16 @@ Commands can be deleted with:
 mctl delete <name>
 ```
 
-## Documentation
+### Clear configuration file
+To clear all fields from the configuration file use:
+```bash
+#CAUTION: If the config file is cleared all data previously saved will be lost forever
+mctl clear
+```
+
+<br />
+
+# Documentation
 ### Commands
 |Command|Description|
 |---|---|
@@ -85,21 +104,19 @@ mctl delete <name>
 |view \<name>|displays saved command|
 |delete \<name>|deletes saved command|
 |run \<name> args...|runs saved command filling placeholders with supplied args|
-
-### Flags
-#### config
-|Flag|Shorthand|Required|Description|
-|---|---|---|---|
-|port|p|yes|RCon port|
-|server|s|yes|RCon address|
+|clear|clears config file|
 
 ### Configuration file
 All configuration data will be kept in the home directory and any sensitive data is encrypted for added security
 
-## Security
+<br />
+
+# Security
 RCon is an inherently insecure protocol, passwords are sent in plaintext and, if possible, the port should not be exposed to the internet. It is best to keep these connections local or over a VPN.
 
 mctl utilizes [govulncheck](https://pkg.go.dev/golang.org/x/vuln/cmd/govulncheck) and [gosec](https://github.com/securego/gosec) in workflows to ensure quality, secure code is being pushed. These workflow steps must pass before a PR will be accepted
 
-## Development
-this repo is currently in heavy development and may encounter breaking changes, use a tag to prevent any surprises
+<br />
+
+# Development
+this repo is currently in development and may encounter breaking changes, use a tag to prevent any surprises

client/mcr.go

@@ -14,7 +14,7 @@ import (
 */
 
 type Client struct {
-	cli *mcr.Client
+	cli mcr.Client
 }
 
 type IClient interface {

cmd/clear.go

@@ -0,0 +1,43 @@
+/*
+Copyright © 2025 Jake jake.young.dev@gmail.com
+*/
+package cmd
+
+import (
+	"fmt"
+	"os"
+
+	"code.jakeyoungdev.com/jake/mctl/models"
+	"github.com/spf13/cobra"
+	"github.com/spf13/viper"
+)
+
+// clearCmd represents the clear command
+var clearCmd = &cobra.Command{
+	Use:   "clear",
+	Short: "Clear config file",
+	Long:  `Clears all configuration values for mctl, all server configuration will be lost`,
+	Run: func(cmd *cobra.Command, args []string) {
+		home, err := os.UserHomeDir()
+		cobra.CheckErr(err)
+
+		viper.AddConfigPath(home)
+		viper.SetConfigType("yaml")
+		viper.SetConfigName(".mctl")
+
+		err = viper.ReadInConfig()
+		if err == nil {
+			//clear values if file exists
+			for _, v := range models.ConfigFields {
+				viper.Set(v, "")
+			}
+			err := viper.WriteConfig()
+			cobra.CheckErr(err)
+			fmt.Println("Config file cleared, use 'config' command to re-populate it")
+		}
+	},
+}
+
+func init() {
+	rootCmd.AddCommand(clearCmd)
+}

cmd/config.go

@@ -76,9 +76,8 @@ func initConfig() {
 	viper.SetConfigName(".mctl")
 	viper.AutomaticEnv()
 	err = viper.ReadInConfig()
-	cobra.CheckErr(err)
 
-	if err := viper.ReadInConfig(); err != nil {
+	if err != nil {
 		//file does not exist, create it
 		viper.Set("server", cfgserver)
 		viper.Set("password", "")

cmd/root.go

@@ -28,5 +28,7 @@ func Execute() {
 }
 
 func init() {
-
+	rootCmd.CompletionOptions = cobra.CompletionOptions{
+		DisableDefaultCmd: true,
+	}
 }

go.mod

@@ -3,7 +3,7 @@ module code.jakeyoungdev.com/jake/mctl
 go 1.24.2
 
 require (
-	github.com/jake-young-dev/mcr v1.3.1
+	github.com/jake-young-dev/mcr v1.4.0
 	github.com/spf13/cobra v1.9.1
 	github.com/spf13/viper v1.20.1
 	golang.org/x/term v0.31.0

go.sum

@@ -14,6 +14,8 @@ github.com/inconshreveable/mousetrap v1.1.0 h1:wN+x4NVGpMsO7ErUn/mUI3vEoE6Jt13X2
 github.com/inconshreveable/mousetrap v1.1.0/go.mod h1:vpF70FUmC8bwa3OWnCshd2FqLfsEA9PFc4w1p2J65bw=
 github.com/jake-young-dev/mcr v1.3.1 h1:ELJsrJHwQsMiM09o+q8auUaiGXXX3DWIgh/TfZQc0B0=
 github.com/jake-young-dev/mcr v1.3.1/go.mod h1:74yZHGf9h3tLUDUpInA17grKLrNp9lVesWvisCFCXKY=
+github.com/jake-young-dev/mcr v1.4.0 h1:cXZImkfI8aNIiVPrONE6qP+nfblTGsD2iXpPKTcA25U=
+github.com/jake-young-dev/mcr v1.4.0/go.mod h1:74yZHGf9h3tLUDUpInA17grKLrNp9lVesWvisCFCXKY=
 github.com/kr/pretty v0.3.1 h1:flRD4NNwYAUpkphVc1HcthR4KEIFJ65n8Mw5qdRn3LE=
 github.com/kr/pretty v0.3.1/go.mod h1:hoEshYVHaxMs3cyo3Yncou5ZscifuDolrwPKZanG3xk=
 github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY=

models/data.go

@@ -0,0 +1,4 @@
+package models
+
+//list of all fields kept in config file
+var ConfigFields = [6]string{"customcmd", "device", "nonce", "port", "server", "password"}