fix (#26)

go.mod

@@ -5,7 +5,7 @@ go 1.22.5
 require (
 	github.com/hashicorp/go-retryablehttp v0.7.7
 	github.com/lmittmann/tint v1.0.5
-	github.com/promiseofcake/artifactsmmo-go-client v1.7.0
+	github.com/promiseofcake/artifactsmmo-go-client v1.7.1
 	github.com/spf13/cobra v1.8.1
 	github.com/spf13/pflag v1.0.5
 	github.com/spf13/viper v1.19.0

go.sum

@@ -52,8 +52,8 @@ github.com/pelletier/go-toml/v2 v2.2.2/go.mod h1:1t835xjRzz80PqgE6HHgN2JOsmgYu/h
 github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
 github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 h1:Jamvg5psRIccs7FGNTlIRMkT8wgtp5eCXdBlqhYGL6U=
 github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
-github.com/promiseofcake/artifactsmmo-go-client v1.7.0 h1:h7sceXRlPdf7AqdijxIEY7frcoRycvSBGum9SbIVDCI=
-github.com/promiseofcake/artifactsmmo-go-client v1.7.0/go.mod h1:UPeTbBmAMzjX0EtFMxF4KVPdHrejptIwHASP4NxkjxI=
+github.com/promiseofcake/artifactsmmo-go-client v1.7.1 h1:lQEJJqRrbK2ebj/OHLXqVUg9iY7UiQ5ydraNhO37stY=
+github.com/promiseofcake/artifactsmmo-go-client v1.7.1/go.mod h1:UPeTbBmAMzjX0EtFMxF4KVPdHrejptIwHASP4NxkjxI=
 github.com/rogpeppe/go-internal v1.12.0 h1:exVL4IDcn6na9z1rAb56Vxr+CgyK3nn3O+epU5NdKM8=
 github.com/rogpeppe/go-internal v1.12.0/go.mod h1:E+RYuTGaKKdloAfM02xzb0FW3Paa99yedzYV+kq4uf4=
 github.com/russross/blackfriday/v2 v2.1.0/go.mod h1:+Rmxgy9KzJVeS9/2gXHxylqXiyQDYRxCVz55jmeOWTM=

internal/engine/refine.go

@@ -104,7 +104,11 @@ func Refine(ctx context.Context, r *actions.Runner, character string) error {
 		}
 
 		// min level current level - 10
-		minLevel := int(math.Max(0, float64(refineLevel-10)))
+		// TODO make overridable
+		//minLevel := int(math.Max(0, float64(refineLevel-10)))
+
+		// allow all items to be refined
+		minLevel := 0
 
 		// get items that match
 		items, iErr := r.GetItems(ctx, minLevel, refineLevel, skillType, res.Code)
@@ -121,10 +125,9 @@ func Refine(ctx context.Context, r *actions.Runner, character string) error {
 		return NoItemsToRefine
 	}
 
-	// given the first item in the list (we should sort it)
-	// determine how much we want to withdraw.
+	// sort list with higher level items first
 	slices.SortFunc(refinable, func(a, b *models.Item) int {
-		return cmp.Compare(a.Level, b.Level)
+		return cmp.Compare(b.Level, a.Level)
 	})
 
 	// look over all the resources we have in the bank