refactor: package cleanup (#4808)

* feat(cosmosgen): fetch fallback buf token

* package refactor

* fix `CaptureException called with nil error`

* cl

* updates

* updates

* refactor: package cleanup

* updates

* remove script

* cl

* cl

Author: julienrbrt

changelog.md

@@ -2,10 +2,13 @@
 
 ## Unreleased
 
+## [`v29.4.1`](https://github.com/ignite/cli/releases/tag/v29.4.1)
+
 ### Changes
 
-- [#4807](https://github.com/ignite/cli/pull/4807) Improve unconfigured path message when building a chain.
 - [#4805](https://github.com/ignite/cli/pull/4805) Fetch fallback buf token.
+- [#4807](https://github.com/ignite/cli/pull/4807) Improve unconfigured path message when building a chain.
+- [#4808](https://github.com/ignite/cli/pull/4808) Remove unused packages.
 
 ## [`v29.4.0`](https://github.com/ignite/cli/releases/tag/v29.4.0)
 

ignite/cmd/ignite/main.go

@@ -18,7 +18,6 @@ import (
 	"github.com/ignite/cli/v29/ignite/pkg/cliui/colors"
 	"github.com/ignite/cli/v29/ignite/pkg/cliui/icons"
 	"github.com/ignite/cli/v29/ignite/pkg/errors"
-	"github.com/ignite/cli/v29/ignite/pkg/validation"
 	"github.com/ignite/cli/v29/ignite/pkg/xstrings"
 	"github.com/ignite/cli/v29/ignite/version"
 )
@@ -64,7 +63,7 @@ func run() int {
 
 	if err != nil {
 		var (
-			validationErr validation.Error
+			validationErr errors.ValidationError
 			versionErr    chainconfig.VersionError
 			msg           string
 		)

ignite/cmd/scaffold_module.go

@@ -10,7 +10,6 @@ import (
 
 	"github.com/ignite/cli/v29/ignite/pkg/cliui"
 	"github.com/ignite/cli/v29/ignite/pkg/errors"
-	"github.com/ignite/cli/v29/ignite/pkg/validation"
 	"github.com/ignite/cli/v29/ignite/pkg/xgenny"
 	"github.com/ignite/cli/v29/ignite/services/scaffolder"
 	modulecreate "github.com/ignite/cli/v29/ignite/templates/module/create"
@@ -182,7 +181,7 @@ func scaffoldModuleHandler(cmd *cobra.Command, args []string) error {
 	}
 
 	if err := sc.CreateModule(name, options...); err != nil {
-		var validationErr validation.Error
+		var validationErr errors.ValidationError
 		if !requireRegistration && errors.As(err, &validationErr) {
 			fmt.Fprintf(&msg, "Can't register module '%s'.\n", name)
 			fmt.Fprintln(&msg, validationErr.ValidationInfo())

ignite/internal/analytics/analytics.go

@@ -15,7 +15,6 @@ import (
 	"github.com/ignite/cli/v29/ignite/internal/sentry"
 	"github.com/ignite/cli/v29/ignite/pkg/cliui"
 	"github.com/ignite/cli/v29/ignite/pkg/errors"
-	"github.com/ignite/cli/v29/ignite/pkg/matomo"
 	"github.com/ignite/cli/v29/ignite/pkg/randstr"
 	"github.com/ignite/cli/v29/ignite/version"
 )
@@ -28,7 +27,7 @@ const (
 	igniteAnonIdentity = "anon_identity.json"
 )
 
-var matomoClient matomo.Client
+var matomoClient MatomoClient
 
 // anonIdentity represents an analytics identity file.
 type anonIdentity struct {
@@ -39,10 +38,10 @@ type anonIdentity struct {
 }
 
 func init() {
-	matomoClient = matomo.New(
+	matomoClient = NewMatomoClient(
 		telemetryEndpoint,
-		matomo.WithIDSite(4),
-		matomo.WithSource("https://cli.ignite.com"),
+		WithIDSite(4),
+		WithSource("https://cli.ignite.com"),
 	)
 }
 
@@ -73,7 +72,7 @@ func SendMetric(wg *sync.WaitGroup, cmd *cobra.Command) {
 		}
 	}
 
-	met := matomo.Metric{
+	met := Metric{
 		Name:            cmd.Name(),
 		Cmd:             path,
 		ScaffoldType:    scaffoldType,

ignite/pkg/matomo/matomo.go ignite/internal/analytics/matomo.goignite/pkg/matomo/matomo.go renamed to ignite/internal/analytics/matomo.go

@@ -1,5 +1,4 @@
-// Package matomo is a client for Matomo to send data points for hint-type=event.
-package matomo
+package analytics
 
 import (
 	"crypto/rand"
@@ -17,17 +16,17 @@ import (
 )
 
 type (
-	// Client is an analytics client.
-	Client struct {
+	// MatomoClient is a matomo client.
+	MatomoClient struct {
 		endpoint   string
 		idSite     uint   // Matomo ID Site.
 		tokenAuth  string // Matomo Token Auth.
 		source     string
 		httpClient http.Client
 	}
 
-	// Params analytics metrics body.
-	Params struct {
+	// MatomoParams analytics metrics body.
+	MatomoParams struct {
 		IDSite      uint   `url:"idsite"`
 		Rec         uint   `url:"rec"`
 		ActionName  string `url:"action_name"`
@@ -121,32 +120,32 @@ type (
 )
 
 // Option configures code generation.
-type Option func(*Client)
+type Option func(*MatomoClient)
 
 // WithIDSite adds an id site.
 func WithIDSite(idSite uint) Option {
-	return func(c *Client) {
+	return func(c *MatomoClient) {
 		c.idSite = idSite
 	}
 }
 
 // WithTokenAuth adds a matomo token authentication.
 func WithTokenAuth(tokenAuth string) Option {
-	return func(c *Client) {
+	return func(c *MatomoClient) {
 		c.tokenAuth = tokenAuth
 	}
 }
 
 // WithSource adds a matomo URL source.
 func WithSource(source string) Option {
-	return func(c *Client) {
+	return func(c *MatomoClient) {
 		c.source = source
 	}
 }
 
-// New creates a new Matomo client.
-func New(endpoint string, opts ...Option) Client {
-	c := Client{
+// NewMatomoClient creates a new Matomo client.
+func NewMatomoClient(endpoint string, opts ...Option) MatomoClient {
+	c := MatomoClient{
 		endpoint: endpoint,
 		source:   endpoint,
 		httpClient: http.Client{
@@ -161,7 +160,7 @@ func New(endpoint string, opts ...Option) Client {
 }
 
 // Send sends metric event to analytics.
-func (c Client) Send(params Params) error {
+func (c MatomoClient) Send(params MatomoParams) error {
 	requestURL, err := url.Parse(c.endpoint)
 	if err != nil {
 		return err
@@ -189,7 +188,7 @@ func (c Client) Send(params Params) error {
 }
 
 // SendMetric build the metrics and send to analytics.
-func (c Client) SendMetric(sessionID string, metric Metric) error {
+func (c MatomoClient) SendMetric(sessionID string, metric Metric) error {
 	var (
 		now       = time.Now()
 		r, _      = rand.Int(rand.Reader, big.NewInt(math.MaxInt64))
@@ -201,7 +200,7 @@ func (c Client) SendMetric(sessionID string, metric Metric) error {
 
 	cmd := splitCommand(metric.Cmd)
 
-	return c.Send(Params{
+	return c.Send(MatomoParams{
 		IDSite:      c.idSite,
 		Rec:         1,
 		APIVersion:  1,
@@ -261,6 +260,6 @@ func splitCommand(cmd string) []string {
 }
 
 // metricURL build the metric URL.
-func (c Client) metricURL(cmd string) string {
+func (c MatomoClient) metricURL(cmd string) string {
 	return fmt.Sprintf("%s/%s", c.source, strings.ReplaceAll(cmd, " ", "_"))
 }