Neatly align log columns

We're not working on it for now

.vscode/launch.json

@@ -9,8 +9,13 @@
 			"type": "go",
 			"request": "launch",
 			"mode": "auto",
-			"program": "${fileDirname}",
+			"program": "${workspaceFolder}",
-			"args": []
+			"cwd": "C:/Users/Administrator/Seafile/Games-Barotrauma",
+			"args": [
+				"LightComponent!anyrange=\"(!num)\"",
+				"*4",
+				"**/*.xml"
+			]
 		}
 	]
 }

README.md

@@ -0,0 +1,116 @@
+# Big Chef
+
+A Go-based tool for modifying XML, JSON, and text documents using XPath/JSONPath/Regex expressions and Lua transformations.
+
+## Features
+
+- **Multi-Format Processing**:
+  - XML (XPath)
+  - JSON (JSONPath) 
+  - Text (Regex)
+- **Node Value Modification**: Update text values in XML elements, JSON properties or text matches
+- **Attribute Manipulation**: Modify XML attributes, JSON object keys or regex capture groups
+- **Conditional Logic**: Apply transformations based on document content
+- **Complex Operations**: 
+  - Mathematical calculations
+  - String manipulations
+  - Date conversions
+  - Structural changes
+  - Whole ass Lua environment
+- **Error Handling**: Comprehensive error detection for:
+  - Invalid XML/JSON
+  - Malformed XPath/JSONPath
+  - Lua syntax errors
+
+## Usage Examples
+
+### 1. Basic field modification
+```xml
+<!-- Input -->
+<price>44.95</price>
+
+<!-- Command -->
+chef -xml "//price" "v=v*2" input.xml
+
+<!-- Output -->
+<price>89.9</price>
+```
+
+### 2. Supports glob patterns
+```xml
+chef -xml "//price" "v=v*2" data/**.xml
+```
+
+### 3. Attribute Update
+```xml
+<!-- Input -->
+<item price="10.50"/>
+
+<!-- Command -->
+chef -xml "//item/@price" "v=v*2" input.xml
+
+<!-- Output --> 
+<item price="21"/>
+```
+
+### 3. JSONPath Transformation
+```json
+// Input
+{
+  "products": [
+    {"name": "Widget", "price": 19.99},
+    {"name": "Gadget", "price": 29.99}
+  ]
+}
+
+// Command
+chef -json "$.products[*].price" "v=v*0.75" input.json
+
+// Output
+{
+  "products": [
+    {"name": "Widget", "price": 14.99},
+    {"name": "Gadget", "price": 22.49}
+  ]
+}
+```
+
+### 4. Regex Text Replacement
+Regex works slightly differently, up to 12 match groups are provided as v1..v12 and s1..s12 for numbers and strings respectively.
+A special shorthand "!num" is also provided that simply expands to `(\d*\.?\d+)`.
+```xml
+<!-- Input -->
+<description>Price: $15.00 Special Offer</description>
+
+<!-- Command -->
+chef "Price: $!num Special Offer" "v1 = v1 * 0.92" input.xml
+
+<!-- Output -->
+<description>Price: $13.80 Special Offer</description>
+```
+
+### 5. Conditional Transformation
+```xml
+<!-- Input -->
+<item stock="5" price="10.00"/>
+
+<!-- Command -->
+chef -xml "//item" "if tonumber(v.stock) > 0 then v.price = v.price * 0.8 end" input.xml
+
+<!-- Output -->
+<item stock="5" price="8.00"/>
+```
+
+## Installation
+
+```bash
+go build -o chef main.go
+```
+
+```bash
+# Process XML file
+./chef -xml "//price" "v=v*1.2" input.xml
+
+# Process JSON file 
+./chef -json "$.prices[*]" "v=v*0.9" input.json
+```

cmd/log_format_test/main.go

@@ -0,0 +1,27 @@
+package main
+
+import (
+	"modify/logger"
+	"time"
+)
+
+func main() {
+	// Initialize logger with DEBUG level
+	logger.Init(logger.LevelDebug)
+
+	// Test different log levels
+	logger.Info("This is an info message")
+	logger.Debug("This is a debug message")
+	logger.Warning("This is a warning message")
+	logger.Error("This is an error message")
+	logger.Trace("This is a trace message (not visible at DEBUG level)")
+
+	// Test with a goroutine
+	logger.SafeGo(func() {
+		time.Sleep(10 * time.Millisecond)
+		logger.Info("Message from goroutine")
+	})
+
+	// Wait for goroutine to complete
+	time.Sleep(20 * time.Millisecond)
+}

go.mod

@@ -3,18 +3,37 @@ module modify
 go 1.24.1
 
 require (
+	github.com/PaesslerAG/jsonpath v0.1.1
 	github.com/antchfx/xmlquery v1.4.4
 	github.com/bmatcuk/doublestar/v4 v4.8.1
 	github.com/yuin/gopher-lua v1.1.1
 )
 
 require (
-	github.com/PaesslerAG/gval v1.0.0 // indirect
+	dario.cat/mergo v1.0.0 // indirect
-	github.com/PaesslerAG/jsonpath v0.1.1 // indirect
+	github.com/Microsoft/go-winio v0.6.2 // indirect
-	github.com/antchfx/xpath v1.3.3 // indirect
+	github.com/ProtonMail/go-crypto v1.1.5 // indirect
-	github.com/golang/groupcache v0.0.0-20210331224755-41bb18bfe9da // indirect
+	github.com/cloudflare/circl v1.6.0 // indirect
-	github.com/sergi/go-diff v1.3.1 // indirect
+	github.com/cyphar/filepath-securejoin v0.4.1 // indirect
-	github.com/stretchr/testify v1.10.0 // indirect
+	github.com/emirpasic/gods v1.18.1 // indirect
-	golang.org/x/net v0.33.0 // indirect
+	github.com/go-git/gcfg v1.5.1-0.20230307220236-3a3c6141e376 // indirect
-	golang.org/x/text v0.21.0 // indirect
+	github.com/go-git/go-billy/v5 v5.6.2 // indirect
+	github.com/jbenet/go-context v0.0.0-20150711004518-d14ea06fba99 // indirect
+	github.com/kevinburke/ssh_config v1.2.0 // indirect
+	github.com/pjbgf/sha1cd v0.3.2 // indirect
+	github.com/sergi/go-diff v1.3.2-0.20230802210424-5b0b94c5c0d3 // indirect
+	github.com/skeema/knownhosts v1.3.1 // indirect
+	github.com/xanzy/ssh-agent v0.3.3 // indirect
+	golang.org/x/crypto v0.35.0 // indirect
+	golang.org/x/sys v0.30.0 // indirect
+	gopkg.in/warnings.v0 v0.1.2 // indirect
+)
+
+require (
+	github.com/PaesslerAG/gval v1.0.0 // indirect
+	github.com/antchfx/xpath v1.3.3 // indirect
+	github.com/go-git/go-git/v5 v5.14.0
+	github.com/golang/groupcache v0.0.0-20241129210726-2c02b8208cf8 // indirect
+	golang.org/x/net v0.35.0 // indirect
+	golang.org/x/text v0.22.0 // indirect
 )

go.sum

@@ -1,38 +1,99 @@
+dario.cat/mergo v1.0.0 h1:AGCNq9Evsj31mOgNPcLyXc+4PNABt905YmuqPYYpBWk=
+dario.cat/mergo v1.0.0/go.mod h1:uNxQE+84aUszobStD9th8a29P2fMDhsBdgRYvZOxGmk=
+github.com/Microsoft/go-winio v0.5.2/go.mod h1:WpS1mjBmmwHBEWmogvA2mj8546UReBk4v8QkMxJ6pZY=
+github.com/Microsoft/go-winio v0.6.2 h1:F2VQgta7ecxGYO8k3ZZz3RS8fVIXVxONVUPlNERoyfY=
+github.com/Microsoft/go-winio v0.6.2/go.mod h1:yd8OoFMLzJbo9gZq8j5qaps8bJ9aShtEA8Ipt1oGCvU=
 github.com/PaesslerAG/gval v1.0.0 h1:GEKnRwkWDdf9dOmKcNrar9EA1bz1z9DqPIO1+iLzhd8=
 github.com/PaesslerAG/gval v1.0.0/go.mod h1:y/nm5yEyTeX6av0OfKJNp9rBNj2XrGhAf5+v24IBN1I=
 github.com/PaesslerAG/jsonpath v0.1.0/go.mod h1:4BzmtoM/PI8fPO4aQGIusjGxGir2BzcV0grWtFzq1Y8=
 github.com/PaesslerAG/jsonpath v0.1.1 h1:c1/AToHQMVsduPAa4Vh6xp2U0evy4t8SWp8imEsylIk=
 github.com/PaesslerAG/jsonpath v0.1.1/go.mod h1:lVboNxFGal/VwW6d9JzIy56bUsYAP6tH/x80vjnCseY=
+github.com/ProtonMail/go-crypto v1.1.5 h1:eoAQfK2dwL+tFSFpr7TbOaPNUbPiJj4fLYwwGE1FQO4=
+github.com/ProtonMail/go-crypto v1.1.5/go.mod h1:rA3QumHc/FZ8pAHreoekgiAbzpNsfQAosU5td4SnOrE=
+github.com/anmitsu/go-shlex v0.0.0-20200514113438-38f4b401e2be h1:9AeTilPcZAjCFIImctFaOjnTIavg87rW78vTPkQqLI8=
+github.com/anmitsu/go-shlex v0.0.0-20200514113438-38f4b401e2be/go.mod h1:ySMOLuWl6zY27l47sB3qLNK6tF2fkHG55UZxx8oIVo4=
 github.com/antchfx/xmlquery v1.4.4 h1:mxMEkdYP3pjKSftxss4nUHfjBhnMk4imGoR96FRY2dg=
 github.com/antchfx/xmlquery v1.4.4/go.mod h1:AEPEEPYE9GnA2mj5Ur2L5Q5/2PycJ0N9Fusrx9b12fc=
 github.com/antchfx/xpath v1.3.3 h1:tmuPQa1Uye0Ym1Zn65vxPgfltWb/Lxu2jeqIGteJSRs=
 github.com/antchfx/xpath v1.3.3/go.mod h1:i54GszH55fYfBmoZXapTHN8T8tkcHfRgLyVwwqzXNcs=
+github.com/armon/go-socks5 v0.0.0-20160902184237-e75332964ef5 h1:0CwZNZbxp69SHPdPJAN/hZIm0C4OItdklCFmMRWYpio=
+github.com/armon/go-socks5 v0.0.0-20160902184237-e75332964ef5/go.mod h1:wHh0iHkYZB8zMSxRWpUBQtwG5a7fFgvEO+odwuTv2gs=
 github.com/bmatcuk/doublestar/v4 v4.8.1 h1:54Bopc5c2cAvhLRAzqOGCYHYyhcDHsFF4wWIR5wKP38=
 github.com/bmatcuk/doublestar/v4 v4.8.1/go.mod h1:xBQ8jztBU6kakFMg+8WGxn0c6z1fTSPVIjEY1Wr7jzc=
+github.com/cloudflare/circl v1.6.0 h1:cr5JKic4HI+LkINy2lg3W2jF8sHCVTBncJr5gIIq7qk=
+github.com/cloudflare/circl v1.6.0/go.mod h1:uddAzsPgqdMAYatqJ0lsjX1oECcQLIlRpzZh3pJrofs=
+github.com/cyphar/filepath-securejoin v0.4.1 h1:JyxxyPEaktOD+GAnqIqTf9A8tHyAG22rowi7HkoSU1s=
+github.com/cyphar/filepath-securejoin v0.4.1/go.mod h1:Sdj7gXlvMcPZsbhwhQ33GguGLDGQL7h7bg04C/+u9jI=
 github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
+github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
 github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
-github.com/golang/groupcache v0.0.0-20210331224755-41bb18bfe9da h1:oI5xCqsCo564l8iNU+DwB5epxmsaqB+rhGL0m5jtYqE=
+github.com/elazarl/goproxy v1.7.2 h1:Y2o6urb7Eule09PjlhQRGNsqRfPmYI3KKQLFpCAV3+o=
+github.com/elazarl/goproxy v1.7.2/go.mod h1:82vkLNir0ALaW14Rc399OTTjyNREgmdL2cVoIbS6XaE=
+github.com/emirpasic/gods v1.18.1 h1:FXtiHYKDGKCW2KzwZKx0iC0PQmdlorYgdFG9jPXJ1Bc=
+github.com/emirpasic/gods v1.18.1/go.mod h1:8tpGGwCnJ5H4r6BWwaV6OrWmMoPhUl5jm/FMNAnJvWQ=
+github.com/gliderlabs/ssh v0.3.8 h1:a4YXD1V7xMF9g5nTkdfnja3Sxy1PVDCj1Zg4Wb8vY6c=
+github.com/gliderlabs/ssh v0.3.8/go.mod h1:xYoytBv1sV0aL3CavoDuJIQNURXkkfPA/wxQ1pL1fAU=
+github.com/go-git/gcfg v1.5.1-0.20230307220236-3a3c6141e376 h1:+zs/tPmkDkHx3U66DAb0lQFJrpS6731Oaa12ikc+DiI=
+github.com/go-git/gcfg v1.5.1-0.20230307220236-3a3c6141e376/go.mod h1:an3vInlBmSxCcxctByoQdvwPiA7DTK7jaaFDBTtu0ic=
+github.com/go-git/go-billy/v5 v5.6.2 h1:6Q86EsPXMa7c3YZ3aLAQsMA0VlWmy43r6FHqa/UNbRM=
+github.com/go-git/go-billy/v5 v5.6.2/go.mod h1:rcFC2rAsp/erv7CMz9GczHcuD0D32fWzH+MJAU+jaUU=
+github.com/go-git/go-git-fixtures/v4 v4.3.2-0.20231010084843-55a94097c399 h1:eMje31YglSBqCdIqdhKBW8lokaMrL3uTkpGYlE2OOT4=
+github.com/go-git/go-git-fixtures/v4 v4.3.2-0.20231010084843-55a94097c399/go.mod h1:1OCfN199q1Jm3HZlxleg+Dw/mwps2Wbk9frAWm+4FII=
+github.com/go-git/go-git/v5 v5.14.0 h1:/MD3lCrGjCen5WfEAzKg00MJJffKhC8gzS80ycmCi60=
+github.com/go-git/go-git/v5 v5.14.0/go.mod h1:Z5Xhoia5PcWA3NF8vRLURn9E5FRhSl7dGj9ItW3Wk5k=
 github.com/golang/groupcache v0.0.0-20210331224755-41bb18bfe9da/go.mod h1:cIg4eruTrX1D+g88fzRXU5OdNfaM+9IcxsU14FzY7Hc=
+github.com/golang/groupcache v0.0.0-20241129210726-2c02b8208cf8 h1:f+oWsMOmNPc8JmEHVZIycC7hBoQxHH9pNKQORJNozsQ=
+github.com/golang/groupcache v0.0.0-20241129210726-2c02b8208cf8/go.mod h1:wcDNUvekVysuuOpQKo3191zZyTpiI6se1N1ULghS0sw=
 github.com/google/go-cmp v0.6.0/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY=
+github.com/google/go-cmp v0.7.0 h1:wk8382ETsv4JYUZwIsn6YpYiWiBsYLSJiTsyBybVuN8=
+github.com/google/go-cmp v0.7.0/go.mod h1:pXiqmnSA92OHEEa9HXL2W4E7lf9JzCmGVUdgjX3N/iU=
+github.com/jbenet/go-context v0.0.0-20150711004518-d14ea06fba99 h1:BQSFePA1RWJOlocH6Fxy8MmwDt+yVQYULKfN0RoTN8A=
+github.com/jbenet/go-context v0.0.0-20150711004518-d14ea06fba99/go.mod h1:1lJo3i6rXxKeerYnT8Nvf0QmHCRC1n8sfWVwXF2Frvo=
+github.com/kevinburke/ssh_config v1.2.0 h1:x584FjTGwHzMwvHx18PXxbBVzfnxogHaAReU4gf13a4=
+github.com/kevinburke/ssh_config v1.2.0/go.mod h1:CT57kijsi8u/K/BOFA39wgDQJ9CxiF4nAY/ojJ6r6mM=
 github.com/kr/pretty v0.1.0/go.mod h1:dAy3ld7l9f0ibDNOQOHHMYYIIbhfbHSm3C4ZsoJORNo=
+github.com/kr/pretty v0.3.1 h1:flRD4NNwYAUpkphVc1HcthR4KEIFJ65n8Mw5qdRn3LE=
+github.com/kr/pretty v0.3.1/go.mod h1:hoEshYVHaxMs3cyo3Yncou5ZscifuDolrwPKZanG3xk=
 github.com/kr/pty v1.1.1/go.mod h1:pFQYn66WHrOpPYNljwOMqo10TkYh1fy3cYio2l3bCsQ=
 github.com/kr/text v0.1.0/go.mod h1:4Jbv+DJW3UT/LiOwJeYQe1efqtUx/iVham/4vfdArNI=
+github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY=
+github.com/kr/text v0.2.0/go.mod h1:eLer722TekiGuMkidMxC/pM04lWEeraHUUmBw8l2grE=
+github.com/onsi/gomega v1.34.1 h1:EUMJIKUjM8sKjYbtxQI9A4z2o+rruxnzNvpknOXie6k=
+github.com/onsi/gomega v1.34.1/go.mod h1:kU1QgUvBDLXBJq618Xvm2LUX6rSAfRaFRTcdOeDLwwY=
+github.com/pjbgf/sha1cd v0.3.2 h1:a9wb0bp1oC2TGwStyn0Umc/IGKQnEgF0vVaZ8QF8eo4=
+github.com/pjbgf/sha1cd v0.3.2/go.mod h1:zQWigSxVmsHEZow5qaLtPYxpcKMMQpa09ixqBxuCS6A=
+github.com/pkg/errors v0.9.1 h1:FEBLx1zS214owpjy7qsBeixbURkuhQAwrK5UwLGTwt4=
+github.com/pkg/errors v0.9.1/go.mod h1:bwawxfHBFNV+L2hUp1rHADufV3IMtnDRdf1r5NINEl0=
+github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=
 github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
-github.com/sergi/go-diff v1.3.1 h1:xkr+Oxo4BOQKmkn/B9eMK0g5Kg/983T9DqqPHwYqD+8=
+github.com/rogpeppe/go-internal v1.14.1 h1:UQB4HGPB6osV0SQTLymcB4TgvyWu6ZyliaW0tI/otEQ=
-github.com/sergi/go-diff v1.3.1/go.mod h1:aMJSSKb2lpPvRNec0+w3fl7LP9IOFzdc9Pa4NFbPK1I=
+github.com/rogpeppe/go-internal v1.14.1/go.mod h1:MaRKkUm5W0goXpeCfT7UZI6fk/L7L7so1lCWt35ZSgc=
+github.com/sergi/go-diff v1.3.2-0.20230802210424-5b0b94c5c0d3 h1:n661drycOFuPLCN3Uc8sB6B/s6Z4t2xvBgU1htSHuq8=
+github.com/sergi/go-diff v1.3.2-0.20230802210424-5b0b94c5c0d3/go.mod h1:A0bzQcvG0E7Rwjx0REVgAGH58e96+X0MeOfepqsbeW4=
+github.com/sirupsen/logrus v1.7.0/go.mod h1:yWOB1SBYBC5VeMP7gHvWumXLIWorT60ONWic61uBYv0=
+github.com/skeema/knownhosts v1.3.1 h1:X2osQ+RAjK76shCbvhHHHVl3ZlgDm8apHEHFqRjnBY8=
+github.com/skeema/knownhosts v1.3.1/go.mod h1:r7KTdC8l4uxWRyK2TpQZ/1o5HaSzh06ePQNxPwTcfiY=
 github.com/stretchr/objx v0.1.0/go.mod h1:HFkY916IF+rwdDfMAkV7OtwuqBVzrE8GR6GFx+wExME=
+github.com/stretchr/testify v1.2.2/go.mod h1:a8OnRcib4nhh0OaRAV+Yts87kKdq0PP7pXfy6kDkUVs=
 github.com/stretchr/testify v1.4.0/go.mod h1:j7eGeouHqKxXV5pUuKE4zz7dFj8WfuZ+81PSLYec5m4=
 github.com/stretchr/testify v1.10.0 h1:Xv5erBjTwe/5IxqUQTdXv5kgmIvbHo3QQyRwhJsOfJA=
 github.com/stretchr/testify v1.10.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
+github.com/xanzy/ssh-agent v0.3.3 h1:+/15pJfg/RsTxqYcX6fHqOXZwwMP+2VyYWJeWM2qQFM=
+github.com/xanzy/ssh-agent v0.3.3/go.mod h1:6dzNDKs0J9rVPHPhaGCukekBHKqfl+L3KghI1Bc68Uw=
 github.com/yuin/goldmark v1.4.13/go.mod h1:6yULJ656Px+3vBD8DxQVa3kxgyrAnzto9xy5taEt/CY=
 github.com/yuin/gopher-lua v1.1.1 h1:kYKnWBjvbNP4XLT3+bPEwAXJx262OhaHDWDVOPjL46M=
 github.com/yuin/gopher-lua v1.1.1/go.mod h1:GBR0iDaNXjAgGg9zfCvksxSRnQx76gclCIb7kdAd1Pw=
 golang.org/x/crypto v0.0.0-20190308221718-c2843e01d9a2/go.mod h1:djNgcEr1/C05ACkg1iLfiJU5Ep61QUkGW8qpdssI0+w=
 golang.org/x/crypto v0.0.0-20210921155107-089bfa567519/go.mod h1:GvvjBRRGRdwPK5ydBHafDWAxML/pGHZbMvKqRZ5+Abc=
+golang.org/x/crypto v0.0.0-20220622213112-05595931fe9d/go.mod h1:IxCIyHEi3zRg3s0A5j5BB6A9Jmi73HwBIUl50j+osU4=
 golang.org/x/crypto v0.13.0/go.mod h1:y6Z2r+Rw4iayiXXAIxJIDAJ1zMW4yaTpebo8fPOliYc=
 golang.org/x/crypto v0.19.0/go.mod h1:Iy9bg/ha4yyC70EfRS8jz+B6ybOBKMaSxLj6P6oBDfU=
 golang.org/x/crypto v0.23.0/go.mod h1:CKFgDieR+mRhux2Lsu27y0fO304Db0wZe70UKqHu0v8=
 golang.org/x/crypto v0.31.0/go.mod h1:kDsLvtWBEx7MV9tJOj9bnXsPbxwJQ6csT/x4KIN4Ssk=
+golang.org/x/crypto v0.35.0 h1:b15kiHdrGCHrP6LvwaQ3c03kgNhhiMgvlhxHQhmg2Xs=
+golang.org/x/crypto v0.35.0/go.mod h1:dy7dXNW32cAb/6/PRuTNsix8T+vJAqvuIy5Bli/x0YQ=
+golang.org/x/exp v0.0.0-20240719175910-8a7402abbf56 h1:2dVuKD2vS7b0QIHQbpyTISPd0LeHDbnYEryqj5Q1ug8=
+golang.org/x/exp v0.0.0-20240719175910-8a7402abbf56/go.mod h1:M4RDyNAINzryxdtnbRXRL/OHtkFuWGRjvuhBJpk2IlY=
 golang.org/x/mod v0.6.0-dev.0.20220419223038-86c51ed26bb4/go.mod h1:jJ57K6gSWd91VN4djpZkiMVwK6gcyfeH4XE8wZrZaV4=
 golang.org/x/mod v0.8.0/go.mod h1:iBbtSCu2XBx23ZKBPSOrRkjjQPZFPuis4dIYUhu/chs=
 golang.org/x/mod v0.12.0/go.mod h1:iBbtSCu2XBx23ZKBPSOrRkjjQPZFPuis4dIYUhu/chs=
@@ -40,14 +101,16 @@ golang.org/x/mod v0.15.0/go.mod h1:hTbmBsO62+eylJbnUtE2MGJUyE7QWk4xUqPFrRgJ+7c=
 golang.org/x/mod v0.17.0/go.mod h1:hTbmBsO62+eylJbnUtE2MGJUyE7QWk4xUqPFrRgJ+7c=
 golang.org/x/net v0.0.0-20190620200207-3b0461eec859/go.mod h1:z5CRVTTTmAJ677TzLLGU+0bjPO0LkuOLi4/5GtJWs/s=
 golang.org/x/net v0.0.0-20210226172049-e18ecbb05110/go.mod h1:m0MpNAwzfU5UDzcl9v0D8zg8gWTRqZa9RBIspLL5mdg=
+golang.org/x/net v0.0.0-20211112202133-69e39bad7dc2/go.mod h1:9nx3DQGgdP8bBQD5qxJ1jj9UTztislL4KSBs9R2vV5Y=
 golang.org/x/net v0.0.0-20220722155237-a158d28d115b/go.mod h1:XRhObCWvk6IyKnWLug+ECip1KBveYUHfp+8e9klMJ9c=
 golang.org/x/net v0.6.0/go.mod h1:2Tu9+aMcznHK/AK1HMvgo6xiTLG5rD5rZLDS+rp2Bjs=
 golang.org/x/net v0.10.0/go.mod h1:0qNGK6F8kojg2nk9dLZ2mShWaEBan6FAoqfSigmmuDg=
 golang.org/x/net v0.15.0/go.mod h1:idbUs1IY1+zTqbi8yxTbhexhEEk5ur9LInksu6HrEpk=
 golang.org/x/net v0.21.0/go.mod h1:bIjVDfnllIU7BJ2DNgfnXvpSvtn8VRwhlsaeUTyUS44=
 golang.org/x/net v0.25.0/go.mod h1:JkAGAh7GEvH74S6FOH42FLoXpXbE/aqXSrIQjXgsiwM=
-golang.org/x/net v0.33.0 h1:74SYHlV8BIgHIFC/LrYkOGIwL19eTYXQ5wc6TBuO36I=
 golang.org/x/net v0.33.0/go.mod h1:HXLR5J+9DxmrqMwG9qjGCxZ+zKXxBru04zlTvWlWuN4=
+golang.org/x/net v0.35.0 h1:T5GQRQb2y08kTAByq9L4/bz8cipCdA8FbRTXewonqY8=
+golang.org/x/net v0.35.0/go.mod h1:EglIi67kWsHKlRzzVMUD93VMSWGFOMSZgxFjparz1Qk=
 golang.org/x/sync v0.0.0-20190423024810-112230192c58/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
 golang.org/x/sync v0.0.0-20220722155255-886fb9371eb4/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
 golang.org/x/sync v0.1.0/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
@@ -56,9 +119,13 @@ golang.org/x/sync v0.6.0/go.mod h1:Czt+wKu1gCyEFDUtn0jG5QVvpJ6rzVqr5aXyt9drQfk=
 golang.org/x/sync v0.7.0/go.mod h1:Czt+wKu1gCyEFDUtn0jG5QVvpJ6rzVqr5aXyt9drQfk=
 golang.org/x/sync v0.10.0/go.mod h1:Czt+wKu1gCyEFDUtn0jG5QVvpJ6rzVqr5aXyt9drQfk=
 golang.org/x/sys v0.0.0-20190215142949-d0b11bdaac8a/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY=
+golang.org/x/sys v0.0.0-20191026070338-33540a1f6037/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/sys v0.0.0-20201119102817-f84b799fce68/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
+golang.org/x/sys v0.0.0-20210124154548-22da62e12c0c/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
+golang.org/x/sys v0.0.0-20210423082822-04245dca01da/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/sys v0.0.0-20210615035016-665e8c7367d1/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.0.0-20220520151302-bc2c85ada10a/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
+golang.org/x/sys v0.0.0-20220715151400-c0bba94af5f8/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.0.0-20220722155257-8c9f86f7a55f/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.5.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.8.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
@@ -66,6 +133,8 @@ golang.org/x/sys v0.12.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.17.0/go.mod h1:/VUhepiaJMQUp4+oa/7Zr1D23ma6VTLIYjOOTFZPUcA=
 golang.org/x/sys v0.20.0/go.mod h1:/VUhepiaJMQUp4+oa/7Zr1D23ma6VTLIYjOOTFZPUcA=
 golang.org/x/sys v0.28.0/go.mod h1:/VUhepiaJMQUp4+oa/7Zr1D23ma6VTLIYjOOTFZPUcA=
+golang.org/x/sys v0.30.0 h1:QjkSwP/36a20jFYWkSue1YwXzLmsV5Gfq7Eiy72C1uc=
+golang.org/x/sys v0.30.0/go.mod h1:/VUhepiaJMQUp4+oa/7Zr1D23ma6VTLIYjOOTFZPUcA=
 golang.org/x/telemetry v0.0.0-20240228155512-f48c80bd79b2/go.mod h1:TeRTkGYfJXctD9OcfyVLyj2J3IxLnKwHJR8f4D8a3YE=
 golang.org/x/term v0.0.0-20201126162022-7de9c90e9dd1/go.mod h1:bj7SfCRtBDWHUb9snDiAeCFNEtKQo2Wmx5Cou7ajbmo=
 golang.org/x/term v0.0.0-20210927222741-03fcf44c2211/go.mod h1:jbD1KX2456YbFQfuXm/mYQcufACuNUgVhRMnK/tPxf8=
@@ -75,16 +144,20 @@ golang.org/x/term v0.12.0/go.mod h1:owVbMEjm3cBLCHdkQu9b1opXd4ETQWc3BhuQGKgXgvU=
 golang.org/x/term v0.17.0/go.mod h1:lLRBjIVuehSbZlaOtGMbcMncT+aqLLLmKrsjNrUguwk=
 golang.org/x/term v0.20.0/go.mod h1:8UkIAJTvZgivsXaD6/pH6U9ecQzZ45awqEOzuCvwpFY=
 golang.org/x/term v0.27.0/go.mod h1:iMsnZpn0cago0GOrHO2+Y7u7JPn5AylBrcoWkElMTSM=
+golang.org/x/term v0.29.0 h1:L6pJp37ocefwRRtYPKSWOWzOtWSxVajvz2ldH/xi3iU=
+golang.org/x/term v0.29.0/go.mod h1:6bl4lRlvVuDgSf3179VpIxBF0o10JUpXWOnI7nErv7s=
 golang.org/x/text v0.3.0/go.mod h1:NqM8EUOU14njkJ3fqMW+pc6Ldnwhi/IjpwHt7yyuwOQ=
 golang.org/x/text v0.3.3/go.mod h1:5Zoc/QRtKVWzQhOtBMvqHzDpF6irO9z98xDceosuGiQ=
+golang.org/x/text v0.3.6/go.mod h1:5Zoc/QRtKVWzQhOtBMvqHzDpF6irO9z98xDceosuGiQ=
 golang.org/x/text v0.3.7/go.mod h1:u+2+/6zg+i71rQMx5EYifcz6MCKuco9NR6JIITiCfzQ=
 golang.org/x/text v0.7.0/go.mod h1:mrYo+phRRbMaCq/xk9113O4dZlRixOauAjOtrjsXDZ8=
 golang.org/x/text v0.9.0/go.mod h1:e1OnstbJyHTd6l/uOt8jFFHp6TRDWZR/bV3emEE/zU8=
 golang.org/x/text v0.13.0/go.mod h1:TvPlkZtksWOMsz7fbANvkp4WM8x/WCo/om8BMLbz+aE=
 golang.org/x/text v0.14.0/go.mod h1:18ZOQIKpY8NJVqYksKHtTdi31H5itFRjB5/qKTNYzSU=
 golang.org/x/text v0.15.0/go.mod h1:18ZOQIKpY8NJVqYksKHtTdi31H5itFRjB5/qKTNYzSU=
-golang.org/x/text v0.21.0 h1:zyQAAkrwaneQ066sspRyJaG9VNi/YJ1NfzcGB3hZ/qo=
 golang.org/x/text v0.21.0/go.mod h1:4IBbMaMmOPCJ8SecivzSH54+73PCFmPWxNTLm+vZkEQ=
+golang.org/x/text v0.22.0 h1:bofq7m3/HAFvbF51jz3Q9wLg3jkvSPuiZu/pD1XwgtM=
+golang.org/x/text v0.22.0/go.mod h1:YRoo4H8PVmsu+E3Ou7cqLVH8oXWIHVoX0jqUWALQhfY=
 golang.org/x/tools v0.0.0-20180917221912-90fa682c2a6e/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ=
 golang.org/x/tools v0.0.0-20191119224855-298f0cb1881e/go.mod h1:b+2E5dAYhXwXZwtnZ6UAqBI28+e2cm9otk0dWdXHAEo=
 golang.org/x/tools v0.1.12/go.mod h1:hNGJHUnrk76NpqgfD5Aqm5Crs+Hm0VOH/i9J2+nxYbc=
@@ -94,5 +167,11 @@ golang.org/x/tools v0.21.1-0.20240508182429-e35e4ccd0d2d/go.mod h1:aiJjzUbINMkxb
 golang.org/x/xerrors v0.0.0-20190717185122-a985d3407aa7/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0=
 gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
 gopkg.in/check.v1 v1.0.0-20190902080502-41f04d3bba15/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
+gopkg.in/check.v1 v1.0.0-20201130134442-10cb98267c6c h1:Hei/4ADfdWqJk1ZMxUNpqntNwaWcugrBjAiHlqqRiVk=
+gopkg.in/check.v1 v1.0.0-20201130134442-10cb98267c6c/go.mod h1:JHkPIbrfpd72SG/EVd6muEfDQjcINNoR0C8j2r3qZ4Q=
+gopkg.in/warnings.v0 v0.1.2 h1:wFXVbFY8DY5/xOe1ECiWdKCzZlxgshcYVNkBHstARME=
+gopkg.in/warnings.v0 v0.1.2/go.mod h1:jksf8JmL6Qr/oQM2OXTHunEvvTAsrWBLb6OOjuVWRNI=
 gopkg.in/yaml.v2 v2.2.2/go.mod h1:hI93XBmqTisBFMUTm0b8Fm+jr3Dg1NNxqwp+5A1VGuI=
 gopkg.in/yaml.v2 v2.4.0/go.mod h1:RDklbk79AGWmwhnvt/jBztapEOGDOx6ZbXqjP6csGnQ=
+gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA=
+gopkg.in/yaml.v3 v3.0.1/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=

logger/logger.go

@@ -0,0 +1,445 @@
+package logger
+
+import (
+	"bytes"
+	"fmt"
+	"io"
+	"log"
+	"os"
+	"path/filepath"
+	"runtime"
+	"strconv"
+	"strings"
+	"sync"
+	"time"
+)
+
+// LogLevel defines the severity of log messages
+type LogLevel int
+
+const (
+	// LevelError is for critical errors that should always be displayed
+	LevelError LogLevel = iota
+	// LevelWarning is for important warnings
+	LevelWarning
+	// LevelInfo is for informational messages
+	LevelInfo
+	// LevelDebug is for detailed debugging information
+	LevelDebug
+	// LevelTrace is for very detailed tracing information
+	LevelTrace
+)
+
+var levelNames = map[LogLevel]string{
+	LevelError:   "ERROR",
+	LevelWarning: "WARNING",
+	LevelInfo:    "INFO",
+	LevelDebug:   "DEBUG",
+	LevelTrace:   "TRACE",
+}
+
+var levelColors = map[LogLevel]string{
+	LevelError:   "\033[1;31m", // Bold Red
+	LevelWarning: "\033[1;33m", // Bold Yellow
+	LevelInfo:    "\033[1;32m", // Bold Green
+	LevelDebug:   "\033[1;36m", // Bold Cyan
+	LevelTrace:   "\033[1;35m", // Bold Magenta
+}
+
+// ResetColor is the ANSI code to reset text color
+const ResetColor = "\033[0m"
+
+// Logger is our custom logger with level support
+type Logger struct {
+	mu            sync.Mutex
+	out           io.Writer
+	currentLevel  LogLevel
+	prefix        string
+	flag          int
+	useColors     bool
+	callerOffset  int
+	defaultFields map[string]interface{}
+	showGoroutine bool
+}
+
+var (
+	// DefaultLogger is the global logger instance
+	DefaultLogger *Logger
+	// defaultLogLevel is the default log level if not specified
+	defaultLogLevel = LevelInfo
+	// Global mutex for DefaultLogger initialization
+	initMutex sync.Mutex
+)
+
+// ParseLevel converts a string log level to LogLevel
+func ParseLevel(levelStr string) LogLevel {
+	switch strings.ToUpper(levelStr) {
+	case "ERROR":
+		return LevelError
+	case "WARNING", "WARN":
+		return LevelWarning
+	case "INFO":
+		return LevelInfo
+	case "DEBUG":
+		return LevelDebug
+	case "TRACE":
+		return LevelTrace
+	default:
+		return defaultLogLevel
+	}
+}
+
+// String returns the string representation of the log level
+func (l LogLevel) String() string {
+	if name, ok := levelNames[l]; ok {
+		return name
+	}
+	return fmt.Sprintf("Level(%d)", l)
+}
+
+// New creates a new Logger instance
+func New(out io.Writer, prefix string, flag int) *Logger {
+	return &Logger{
+		out:           out,
+		currentLevel:  defaultLogLevel,
+		prefix:        prefix,
+		flag:          flag,
+		useColors:     true,
+		callerOffset:  0,
+		defaultFields: make(map[string]interface{}),
+		showGoroutine: true,
+	}
+}
+
+// Init initializes the DefaultLogger
+func Init(level LogLevel) {
+	initMutex.Lock()
+	defer initMutex.Unlock()
+
+	if DefaultLogger == nil {
+		DefaultLogger = New(os.Stdout, "", log.Lmicroseconds|log.Lshortfile)
+	}
+	DefaultLogger.SetLevel(level)
+}
+
+// SetLevel sets the current log level
+func (l *Logger) SetLevel(level LogLevel) {
+	l.mu.Lock()
+	defer l.mu.Unlock()
+	l.currentLevel = level
+}
+
+// GetLevel returns the current log level
+func (l *Logger) GetLevel() LogLevel {
+	l.mu.Lock()
+	defer l.mu.Unlock()
+	return l.currentLevel
+}
+
+// SetCallerOffset sets the caller offset for correct file and line reporting
+func (l *Logger) SetCallerOffset(offset int) {
+	l.mu.Lock()
+	defer l.mu.Unlock()
+	l.callerOffset = offset
+}
+
+// SetShowGoroutine sets whether to include goroutine ID in log messages
+func (l *Logger) SetShowGoroutine(show bool) {
+	l.mu.Lock()
+	defer l.mu.Unlock()
+	l.showGoroutine = show
+}
+
+// ShowGoroutine returns whether goroutine ID is included in log messages
+func (l *Logger) ShowGoroutine() bool {
+	l.mu.Lock()
+	defer l.mu.Unlock()
+	return l.showGoroutine
+}
+
+// WithField adds a field to the logger's context
+func (l *Logger) WithField(key string, value interface{}) *Logger {
+	newLogger := &Logger{
+		out:           l.out,
+		currentLevel:  l.currentLevel,
+		prefix:        l.prefix,
+		flag:          l.flag,
+		useColors:     l.useColors,
+		callerOffset:  l.callerOffset,
+		defaultFields: make(map[string]interface{}),
+		showGoroutine: l.showGoroutine,
+	}
+
+	// Copy existing fields
+	for k, v := range l.defaultFields {
+		newLogger.defaultFields[k] = v
+	}
+
+	// Add new field
+	newLogger.defaultFields[key] = value
+	return newLogger
+}
+
+// WithFields adds multiple fields to the logger's context
+func (l *Logger) WithFields(fields map[string]interface{}) *Logger {
+	newLogger := &Logger{
+		out:           l.out,
+		currentLevel:  l.currentLevel,
+		prefix:        l.prefix,
+		flag:          l.flag,
+		useColors:     l.useColors,
+		callerOffset:  l.callerOffset,
+		defaultFields: make(map[string]interface{}),
+		showGoroutine: l.showGoroutine,
+	}
+
+	// Copy existing fields
+	for k, v := range l.defaultFields {
+		newLogger.defaultFields[k] = v
+	}
+
+	// Add new fields
+	for k, v := range fields {
+		newLogger.defaultFields[k] = v
+	}
+	return newLogger
+}
+
+// GetGoroutineID extracts the goroutine ID from the runtime stack
+func GetGoroutineID() string {
+	buf := make([]byte, 64)
+	n := runtime.Stack(buf, false)
+	// Format of first line is "goroutine N [state]:"
+	// We only need the N part
+	buf = buf[:n]
+	idField := bytes.Fields(bytes.Split(buf, []byte{':'})[0])[1]
+	return string(idField)
+}
+
+// formatMessage formats a log message with level, time, file, and line information
+func (l *Logger) formatMessage(level LogLevel, format string, args ...interface{}) string {
+	var msg string
+	if len(args) > 0 {
+		msg = fmt.Sprintf(format, args...)
+	} else {
+		msg = format
+	}
+
+	// Format default fields if any
+	var fields string
+	if len(l.defaultFields) > 0 {
+		var pairs []string
+		for k, v := range l.defaultFields {
+			pairs = append(pairs, fmt.Sprintf("%s=%v", k, v))
+		}
+		fields = " " + strings.Join(pairs, " ")
+	}
+
+	var levelColor, resetColor string
+	if l.useColors {
+		levelColor = levelColors[level]
+		resetColor = ResetColor
+	}
+
+	var caller string
+	if l.flag&log.Lshortfile != 0 || l.flag&log.Llongfile != 0 {
+		// Find the actual caller by scanning up the stack
+		// until we find a function outside the logger package
+		var file string
+		var line int
+		var ok bool
+
+		// Start at a reasonable depth and scan up to 10 frames
+		for depth := 4; depth < 15; depth++ {
+			_, file, line, ok = runtime.Caller(depth)
+			if !ok {
+				break
+			}
+
+			// If the caller is not in the logger package, we found our caller
+			if !strings.Contains(file, "logger/logger.go") {
+				break
+			}
+		}
+
+		if !ok {
+			file = "???"
+			line = 0
+		}
+
+		if l.flag&log.Lshortfile != 0 {
+			file = filepath.Base(file)
+		}
+		caller = fmt.Sprintf("%-25s ", file+":"+strconv.Itoa(line))
+	}
+
+	// Format the timestamp with fixed width
+	var timeStr string
+	if l.flag&(log.Ldate|log.Ltime|log.Lmicroseconds) != 0 {
+		t := time.Now()
+		if l.flag&log.Ldate != 0 {
+			timeStr += fmt.Sprintf("%04d/%02d/%02d ", t.Year(), t.Month(), t.Day())
+		}
+		if l.flag&(log.Ltime|log.Lmicroseconds) != 0 {
+			timeStr += fmt.Sprintf("%02d:%02d:%02d", t.Hour(), t.Minute(), t.Second())
+			if l.flag&log.Lmicroseconds != 0 {
+				timeStr += fmt.Sprintf(".%06d", t.Nanosecond()/1000)
+			}
+		}
+		timeStr = fmt.Sprintf("%-15s ", timeStr)
+	}
+
+	// Add goroutine ID if enabled, with fixed width
+	var goroutineStr string
+	if l.showGoroutine {
+		goroutineID := GetGoroutineID()
+		goroutineStr = fmt.Sprintf("[g:%-4s] ", goroutineID)
+	}
+
+	// Create a colored level indicator with both brackets colored
+	levelStr := fmt.Sprintf("%s[%s]%s", levelColor, levelNames[level], levelColor)
+	// Add a space after the level and before the reset color
+	levelColumn := fmt.Sprintf("%s %s", levelStr, resetColor)
+
+	return fmt.Sprintf("%s%s%s%s%s%s%s\n",
+		l.prefix, timeStr, caller, goroutineStr, levelColumn, msg, fields)
+}
+
+// log logs a message at the specified level
+func (l *Logger) log(level LogLevel, format string, args ...interface{}) {
+	if level > l.currentLevel {
+		return
+	}
+
+	l.mu.Lock()
+	defer l.mu.Unlock()
+
+	msg := l.formatMessage(level, format, args...)
+	fmt.Fprint(l.out, msg)
+}
+
+// Error logs an error message
+func (l *Logger) Error(format string, args ...interface{}) {
+	l.log(LevelError, format, args...)
+}
+
+// Warning logs a warning message
+func (l *Logger) Warning(format string, args ...interface{}) {
+	l.log(LevelWarning, format, args...)
+}
+
+// Info logs an informational message
+func (l *Logger) Info(format string, args ...interface{}) {
+	l.log(LevelInfo, format, args...)
+}
+
+// Debug logs a debug message
+func (l *Logger) Debug(format string, args ...interface{}) {
+	l.log(LevelDebug, format, args...)
+}
+
+// Trace logs a trace message
+func (l *Logger) Trace(format string, args ...interface{}) {
+	l.log(LevelTrace, format, args...)
+}
+
+// Global log functions that use DefaultLogger
+
+// Error logs an error message using the default logger
+func Error(format string, args ...interface{}) {
+	if DefaultLogger == nil {
+		Init(defaultLogLevel)
+	}
+	DefaultLogger.Error(format, args...)
+}
+
+// Warning logs a warning message using the default logger
+func Warning(format string, args ...interface{}) {
+	if DefaultLogger == nil {
+		Init(defaultLogLevel)
+	}
+	DefaultLogger.Warning(format, args...)
+}
+
+// Info logs an informational message using the default logger
+func Info(format string, args ...interface{}) {
+	if DefaultLogger == nil {
+		Init(defaultLogLevel)
+	}
+	DefaultLogger.Info(format, args...)
+}
+
+// Debug logs a debug message using the default logger
+func Debug(format string, args ...interface{}) {
+	if DefaultLogger == nil {
+		Init(defaultLogLevel)
+	}
+	DefaultLogger.Debug(format, args...)
+}
+
+// Trace logs a trace message using the default logger
+func Trace(format string, args ...interface{}) {
+	if DefaultLogger == nil {
+		Init(defaultLogLevel)
+	}
+	DefaultLogger.Trace(format, args...)
+}
+
+// LogPanic logs a panic error and its stack trace
+func LogPanic(r interface{}) {
+	if DefaultLogger == nil {
+		Init(defaultLogLevel)
+	}
+	stack := make([]byte, 4096)
+	n := runtime.Stack(stack, false)
+	DefaultLogger.Error("PANIC: %v\n%s", r, stack[:n])
+}
+
+// SetLevel sets the log level for the default logger
+func SetLevel(level LogLevel) {
+	if DefaultLogger == nil {
+		Init(level)
+		return
+	}
+	DefaultLogger.SetLevel(level)
+}
+
+// GetLevel gets the log level for the default logger
+func GetLevel() LogLevel {
+	if DefaultLogger == nil {
+		Init(defaultLogLevel)
+	}
+	return DefaultLogger.GetLevel()
+}
+
+// WithField returns a new logger with the field added to the default logger's context
+func WithField(key string, value interface{}) *Logger {
+	if DefaultLogger == nil {
+		Init(defaultLogLevel)
+	}
+	return DefaultLogger.WithField(key, value)
+}
+
+// WithFields returns a new logger with the fields added to the default logger's context
+func WithFields(fields map[string]interface{}) *Logger {
+	if DefaultLogger == nil {
+		Init(defaultLogLevel)
+	}
+	return DefaultLogger.WithFields(fields)
+}
+
+// SetShowGoroutine enables or disables goroutine ID display in the default logger
+func SetShowGoroutine(show bool) {
+	if DefaultLogger == nil {
+		Init(defaultLogLevel)
+	}
+	DefaultLogger.SetShowGoroutine(show)
+}
+
+// ShowGoroutine returns whether goroutine ID is included in default logger's messages
+func ShowGoroutine() bool {
+	if DefaultLogger == nil {
+		Init(defaultLogLevel)
+	}
+	return DefaultLogger.ShowGoroutine()
+}

logger/panic_handler.go

@@ -0,0 +1,49 @@
+package logger
+
+import (
+	"fmt"
+	"runtime/debug"
+)
+
+// PanicHandler handles a panic and logs it
+func PanicHandler() {
+	if r := recover(); r != nil {
+		goroutineID := GetGoroutineID()
+		stackTrace := debug.Stack()
+		Error("PANIC in goroutine %s: %v\n%s", goroutineID, r, stackTrace)
+	}
+}
+
+// SafeGo launches a goroutine with panic recovery
+// Usage: logger.SafeGo(func() { ... your code ... })
+func SafeGo(f func()) {
+	go func() {
+		defer PanicHandler()
+		f()
+	}()
+}
+
+// SafeGoWithArgs launches a goroutine with panic recovery and passes arguments
+// Usage: logger.SafeGoWithArgs(func(arg1, arg2 interface{}) { ... }, "value1", 42)
+func SafeGoWithArgs(f func(...interface{}), args ...interface{}) {
+	go func() {
+		defer PanicHandler()
+		f(args...)
+	}()
+}
+
+// SafeExec executes a function with panic recovery
+// Useful for code that should not panic
+func SafeExec(f func()) (err error) {
+	defer func() {
+		if r := recover(); r != nil {
+			goroutineID := GetGoroutineID()
+			stackTrace := debug.Stack()
+			Error("PANIC in goroutine %s: %v\n%s", goroutineID, r, stackTrace)
+			err = fmt.Errorf("panic recovered: %v", r)
+		}
+	}()
+
+	f()
+	return nil
+}

main.go

@@ -5,10 +5,15 @@ import (
 	"fmt"
 	"log"
 	"os"
+	"path/filepath"
 	"sync"
+	"time"
 
 	"github.com/bmatcuk/doublestar/v4"
+	"github.com/go-git/go-git/v5"
+	"github.com/go-git/go-git/v5/plumbing/object"
 
+	"modify/logger"
 	"modify/processor"
 )
 
@@ -19,62 +24,86 @@ type GlobalStats struct {
 	FailedFiles        int
 }
 
-type FileMode string
-
-const (
-	ModeRegex FileMode = "regex"
-	ModeXML   FileMode = "xml"
-	ModeJSON  FileMode = "json"
-)
-
 var stats GlobalStats
-var logger *log.Logger
+var stdLogger *log.Logger // Legacy logger for compatibility
 
 var (
-	fileModeFlag = flag.String("mode", "regex", "Processing mode: regex, xml, json")
+	jsonFlag  = flag.Bool("json", false, "Process JSON files")
-	verboseFlag  = flag.Bool("verbose", false, "Enable verbose output")
+	xmlFlag   = flag.Bool("xml", false, "Process XML files")
+	gitFlag   = flag.Bool("git", false, "Use git to manage files")
+	resetFlag = flag.Bool("reset", false, "Reset files to their original state")
+	logLevel  = flag.String("loglevel", "INFO", "Set log level: ERROR, WARNING, INFO, DEBUG, TRACE")
+	repo      *git.Repository
+	worktree  *git.Worktree
 )
 
 func init() {
+	// Keep standard logger setup for compatibility with legacy code
 	log.SetFlags(log.Lmicroseconds | log.Lshortfile)
-	logger = log.New(os.Stdout, "", log.Lmicroseconds|log.Lshortfile)
+	stdLogger = log.New(os.Stdout, "", log.Lmicroseconds|log.Lshortfile)
 
 	stats = GlobalStats{}
 }
 
 func main() {
+	// TODO: Implement some sort of git integration
+	// Maybe use go-git
+	// Specify a -git flag
+	// If we are operating with git then:
+	// Inmitialize a repo if one doesn't exist (try to open right?)
+	// For each file matched by glob first figure out if it's already tracked
+	// If not tracked then track it and commit (either it alone or maybe multiple together somehow)
+	// Then reset the file (to undo previous modifications)
+	// THEN change the file
+	// In addition add a -undo flag that will ONLY reset the files without changing them
+	// Only for the ones matched by glob
+	// ^ important because binary files would fuck us up
 	flag.Usage = func() {
 		fmt.Fprintf(os.Stderr, "Usage: %s [options] <pattern> <lua_expression> <...files_or_globs>\n", os.Args[0])
 		fmt.Fprintf(os.Stderr, "\nOptions:\n")
+		fmt.Fprintf(os.Stderr, "  -json\n")
+		fmt.Fprintf(os.Stderr, "        Process JSON files\n")
+		fmt.Fprintf(os.Stderr, "  -xml\n")
+		fmt.Fprintf(os.Stderr, "        Process XML files\n")
+		fmt.Fprintf(os.Stderr, "  -git\n")
+		fmt.Fprintf(os.Stderr, "        Use git to manage files\n")
+		fmt.Fprintf(os.Stderr, "  -reset\n")
+		fmt.Fprintf(os.Stderr, "        Reset files to their original state\n")
+		fmt.Fprintf(os.Stderr, "  -loglevel string\n")
+		fmt.Fprintf(os.Stderr, "        Set logging level: ERROR, WARNING, INFO, DEBUG, TRACE (default \"INFO\")\n")
 		fmt.Fprintf(os.Stderr, "  -mode string\n")
 		fmt.Fprintf(os.Stderr, "        Processing mode: regex, xml, json (default \"regex\")\n")
-		fmt.Fprintf(os.Stderr, "  -xpath string\n")
-		fmt.Fprintf(os.Stderr, "        XPath expression (for XML mode)\n")
-		fmt.Fprintf(os.Stderr, "  -jsonpath string\n")
-		fmt.Fprintf(os.Stderr, "        JSONPath expression (for JSON mode)\n")
-		fmt.Fprintf(os.Stderr, "  -verbose\n")
-		fmt.Fprintf(os.Stderr, "        Enable verbose output\n")
 		fmt.Fprintf(os.Stderr, "\nExamples:\n")
 		fmt.Fprintf(os.Stderr, "  Regex mode (default):\n")
 		fmt.Fprintf(os.Stderr, "    %s \"<value>(\\d+)</value>\" \"*1.5\" data.xml\n", os.Args[0])
 		fmt.Fprintf(os.Stderr, "  XML mode:\n")
-		fmt.Fprintf(os.Stderr, "    %s -mode=xml -xpath=\"//value\" \"*1.5\" data.xml\n", os.Args[0])
+		fmt.Fprintf(os.Stderr, "    %s -xml \"//value\" \"*1.5\" data.xml\n", os.Args[0])
 		fmt.Fprintf(os.Stderr, "  JSON mode:\n")
-		fmt.Fprintf(os.Stderr, "    %s -mode=json -jsonpath=\"$.items[*].value\" \"*1.5\" data.json\n", os.Args[0])
+		fmt.Fprintf(os.Stderr, "    %s -json \"$.items[*].value\" \"*1.5\" data.json\n", os.Args[0])
 		fmt.Fprintf(os.Stderr, "\nNote: v1, v2, etc. are used to refer to capture groups as numbers.\n")
 		fmt.Fprintf(os.Stderr, "      s1, s2, etc. are used to refer to capture groups as strings.\n")
 		fmt.Fprintf(os.Stderr, "      Helper functions: num(str) converts string to number, str(num) converts number to string\n")
 		fmt.Fprintf(os.Stderr, "      is_number(str) checks if a string is numeric\n")
+		fmt.Fprintf(os.Stderr, "      For XML and JSON, the captured values are exposed as 'v', which can be of any type we capture (string, number, table).\n")
 		fmt.Fprintf(os.Stderr, "      If expression starts with an operator like *, /, +, -, =, etc., v1 is automatically prepended\n")
 		fmt.Fprintf(os.Stderr, "      You can use any valid Lua code, including if statements, loops, etc.\n")
 		fmt.Fprintf(os.Stderr, "      Glob patterns are supported for file selection (*.xml, data/**.xml, etc.)\n")
 	}
 
 	flag.Parse()
+
+	// Initialize logger with the specified log level
+	level := logger.ParseLevel(*logLevel)
+	logger.Init(level)
+	logger.Info("Initializing with log level: %s", level.String())
+
 	args := flag.Args()
+	if *resetFlag {
+		*gitFlag = true
+	}
 
 	if len(args) < 3 {
-		fmt.Fprintf(os.Stderr, "%s mode requires %d arguments minimum\n", *fileModeFlag, 3)
+		logger.Error("At least %d arguments are required", 3)
 		flag.Usage()
 		return
 	}
@@ -83,100 +112,228 @@ func main() {
 	var pattern, luaExpr string
 	var filePatterns []string
 
-	if *fileModeFlag == "regex" {
+	pattern = args[0]
-		pattern = args[0]
+	luaExpr = args[1]
-		luaExpr = args[1]
+	filePatterns = args[2:]
-		filePatterns = args[2:]
-	} else {
-		// For XML/JSON modes, pattern comes from flags
-		luaExpr = args[0]
-		filePatterns = args[1:]
-	}
 
 	// Prepare the Lua expression
 	originalLuaExpr := luaExpr
 	luaExpr = processor.BuildLuaScript(luaExpr)
 	if originalLuaExpr != luaExpr {
-		logger.Printf("Transformed Lua expression from %q to %q", originalLuaExpr, luaExpr)
+		logger.Debug("Transformed Lua expression from %q to %q", originalLuaExpr, luaExpr)
+	}
+
+	if *gitFlag {
+		logger.Info("Git integration enabled, setting up git repository")
+		err := setupGit()
+		if err != nil {
+			logger.Error("Failed to setup git: %v", err)
+			fmt.Fprintf(os.Stderr, "Error setting up git: %v\n", err)
+			return
+		}
 	}
 
 	// Expand file patterns with glob support
+	logger.Debug("Expanding file patterns: %v", filePatterns)
 	files, err := expandFilePatterns(filePatterns)
 	if err != nil {
+		logger.Error("Failed to expand file patterns: %v", err)
 		fmt.Fprintf(os.Stderr, "Error expanding file patterns: %v\n", err)
 		return
 	}
 
 	if len(files) == 0 {
+		logger.Warning("No files found matching the specified patterns")
 		fmt.Fprintf(os.Stderr, "No files found matching the specified patterns\n")
 		return
 	}
 
+	if *gitFlag {
+		logger.Info("Cleaning up git files before processing")
+		err := cleanupGitFiles(files)
+		if err != nil {
+			logger.Error("Failed to cleanup git files: %v", err)
+			fmt.Fprintf(os.Stderr, "Error cleaning up git files: %v\n", err)
+			return
+		}
+	}
+	if *resetFlag {
+		logger.Info("Files reset to their original state, nothing more to do")
+		log.Printf("Files reset to their original state, nothing more to do")
+		return
+	}
+
 	// Create the processor based on mode
 	var proc processor.Processor
-	switch *fileModeFlag {
+	switch {
-	case "regex":
+	case *xmlFlag:
-		proc = &processor.RegexProcessor{}
+		proc = &processor.XMLProcessor{}
-		logger.Printf("Starting regex modifier with pattern %q, expression %q on %d files",
+		logger.Info("Starting XML modifier with XPath %q, expression %q on %d files",
+			pattern, luaExpr, len(files))
+	case *jsonFlag:
+		proc = &processor.JSONProcessor{}
+		logger.Info("Starting JSON modifier with JSONPath %q, expression %q on %d files",
+			pattern, luaExpr, len(files))
+	default:
+		proc = &processor.RegexProcessor{}
+		logger.Info("Starting regex modifier with pattern %q, expression %q on %d files",
 			pattern, luaExpr, len(files))
-		// case "xml":
-		// 	proc = &processor.XMLProcessor{}
-		// 	pattern = *xpathFlag
-		// 	logger.Printf("Starting XML modifier with XPath %q, expression %q on %d files",
-		// 		pattern, luaExpr, len(files))
-		// case "json":
-		// 	proc = &processor.JSONProcessor{}
-		// 	pattern = *jsonpathFlag
-		// 	logger.Printf("Starting JSON modifier with JSONPath %q, expression %q on %d files",
-		// 		pattern, luaExpr, len(files))
 	}
 
 	var wg sync.WaitGroup
 	// Process each file
 	for _, file := range files {
 		wg.Add(1)
-		go func(file string) {
+		logger.SafeGoWithArgs(func(args ...interface{}) {
 			defer wg.Done()
-			logger.Printf("Processing file: %s", file)
+			fileToProcess := args[0].(string)
+			logger.Debug("Processing file: %s", fileToProcess)
 
-			modCount, matchCount, err := proc.Process(file, pattern, luaExpr)
+			// It's a bit fucked, maybe I could do better to call it from proc... But it'll do for now
+			modCount, matchCount, err := processor.Process(proc, fileToProcess, pattern, luaExpr)
 			if err != nil {
-				fmt.Fprintf(os.Stderr, "Failed to process file %s: %v\n", file, err)
+				logger.Error("Failed to process file %s: %v", fileToProcess, err)
+				fmt.Fprintf(os.Stderr, "Failed to process file %s: %v\n", fileToProcess, err)
 				stats.FailedFiles++
 			} else {
-				logger.Printf("Successfully processed file: %s", file)
+				if modCount > 0 {
+					logger.Info("Successfully processed file %s: %d modifications from %d matches",
+						fileToProcess, modCount, matchCount)
+				} else if matchCount > 0 {
+					logger.Info("Found %d matches in file %s but made no modifications",
+						matchCount, fileToProcess)
+				} else {
+					logger.Debug("No matches found in file: %s", fileToProcess)
+				}
 				stats.ProcessedFiles++
 				stats.TotalMatches += matchCount
 				stats.TotalModifications += modCount
 			}
-		}(file)
+		}, file)
 	}
 	wg.Wait()
 
 	// Print summary
 	if stats.TotalModifications == 0 {
+		logger.Warning("No modifications were made in any files")
 		fmt.Fprintf(os.Stderr, "No modifications were made in any files\n")
 	} else {
+		logger.Info("Operation complete! Modified %d values in %d/%d files",
+			stats.TotalModifications, stats.ProcessedFiles, stats.ProcessedFiles+stats.FailedFiles)
 		fmt.Printf("Operation complete! Modified %d values in %d/%d files\n",
 			stats.TotalModifications, stats.ProcessedFiles, stats.ProcessedFiles+stats.FailedFiles)
 	}
 }
 
+func setupGit() error {
+	cwd, err := os.Getwd()
+	if err != nil {
+		return fmt.Errorf("failed to get current working directory: %w", err)
+	}
+	logger.Debug("Current working directory obtained: %s", cwd)
+
+	logger.Debug("Attempting to open git repository at %s", cwd)
+	repo, err = git.PlainOpen(cwd)
+	if err != nil {
+		logger.Debug("No existing git repository found at %s, attempting to initialize a new git repository.", cwd)
+		repo, err = git.PlainInit(cwd, false)
+		if err != nil {
+			return fmt.Errorf("failed to initialize a new git repository at %s: %w", cwd, err)
+		}
+		logger.Info("Successfully initialized a new git repository at %s", cwd)
+	} else {
+		logger.Info("Successfully opened existing git repository at %s", cwd)
+	}
+
+	logger.Debug("Attempting to obtain worktree for repository at %s", cwd)
+	worktree, err = repo.Worktree()
+	if err != nil {
+		return fmt.Errorf("failed to obtain worktree for repository at %s: %w", cwd, err)
+	}
+	logger.Debug("Successfully obtained worktree for repository at %s", cwd)
+	return nil
+}
+
 func expandFilePatterns(patterns []string) ([]string, error) {
 	var files []string
 	filesMap := make(map[string]bool)
 
+	cwd, err := os.Getwd()
+	if err != nil {
+		return nil, fmt.Errorf("failed to get current working directory: %w", err)
+	}
+
+	logger.Debug("Expanding patterns from directory: %s", cwd)
 	for _, pattern := range patterns {
-		matches, _ := doublestar.Glob(os.DirFS("."), pattern)
+		logger.Trace("Processing pattern: %s", pattern)
+		matches, _ := doublestar.Glob(os.DirFS(cwd), pattern)
+		logger.Debug("Found %d matches for pattern %s", len(matches), pattern)
 		for _, m := range matches {
-			if info, err := os.Stat(m); err == nil && !info.IsDir() && !filesMap[m] {
+			info, err := os.Stat(m)
+			if err != nil {
+				logger.Warning("Error getting file info for %s: %v", m, err)
+				continue
+			}
+			if !info.IsDir() && !filesMap[m] {
+				logger.Trace("Adding file to process list: %s", m)
 				filesMap[m], files = true, append(files, m)
 			}
 		}
 	}
 
 	if len(files) > 0 {
-		logger.Printf("Found %d files to process", len(files))
+		logger.Debug("Found %d files to process: %v", len(files), files)
 	}
 	return files, nil
 }
+
+func cleanupGitFiles(files []string) error {
+	for _, file := range files {
+		logger.Debug("Checking git status for file: %s", file)
+		status, err := worktree.Status()
+		if err != nil {
+			logger.Error("Error getting worktree status: %v", err)
+			fmt.Fprintf(os.Stderr, "Error getting worktree status: %v\n", err)
+			return fmt.Errorf("error getting worktree status: %w", err)
+		}
+		if status.IsUntracked(file) {
+			logger.Info("Detected untracked file: %s. Adding to git index.", file)
+			_, err = worktree.Add(file)
+			if err != nil {
+				logger.Error("Error adding file to git: %v", err)
+				fmt.Fprintf(os.Stderr, "Error adding file to git: %v\n", err)
+				return fmt.Errorf("error adding file to git: %w", err)
+			}
+
+			filename := filepath.Base(file)
+			logger.Info("File %s added successfully. Committing with message: 'Track %s'", filename, filename)
+			_, err = worktree.Commit("Track "+filename, &git.CommitOptions{
+				Author: &object.Signature{
+					Name:  "Big Chef",
+					Email: "bigchef@bigchef.com",
+					When:  time.Now(),
+				},
+			})
+			if err != nil {
+				logger.Error("Error committing file: %v", err)
+				fmt.Fprintf(os.Stderr, "Error committing file: %v\n", err)
+				return fmt.Errorf("error committing file: %w", err)
+			}
+			logger.Info("Successfully committed file: %s", filename)
+		} else {
+			logger.Info("File %s is already tracked. Restoring it to the working tree.", file)
+			err := worktree.Restore(&git.RestoreOptions{
+				Files:    []string{file},
+				Staged:   true,
+				Worktree: true,
+			})
+			if err != nil {
+				logger.Error("Error restoring file: %v", err)
+				fmt.Fprintf(os.Stderr, "Error restoring file: %v\n", err)
+				return fmt.Errorf("error restoring file: %w", err)
+			}
+			logger.Info("File %s restored successfully", file)
+		}
+	}
+	return nil
+}

processor/json.go

@@ -3,10 +3,8 @@ package processor
 import (
 	"encoding/json"
 	"fmt"
+	"modify/logger"
 	"modify/processor/jsonpath"
-	"os"
-	"path/filepath"
-	"strings"
 
 	lua "github.com/yuin/gopher-lua"
 )
@@ -14,152 +12,174 @@ import (
 // JSONProcessor implements the Processor interface for JSON documents
 type JSONProcessor struct{}
 
-// Process implements the Processor interface for JSONProcessor
-func (p *JSONProcessor) Process(filename string, pattern string, luaExpr string) (int, int, error) {
-	// Read file content
-	cwd, err := os.Getwd()
-	if err != nil {
-		return 0, 0, fmt.Errorf("error getting current working directory: %v", err)
-	}
-
-	fullPath := filepath.Join(cwd, filename)
-	content, err := os.ReadFile(fullPath)
-	if err != nil {
-		return 0, 0, fmt.Errorf("error reading file: %v", err)
-	}
-
-	fileContent := string(content)
-
-	// Process the content
-	modifiedContent, modCount, matchCount, err := p.ProcessContent(fileContent, pattern, luaExpr)
-	if err != nil {
-		return 0, 0, err
-	}
-
-	// If we made modifications, save the file
-	if modCount > 0 {
-		err = os.WriteFile(fullPath, []byte(modifiedContent), 0644)
-		if err != nil {
-			return 0, 0, fmt.Errorf("error writing file: %v", err)
-		}
-	}
-
-	return modCount, matchCount, nil
-}
-
 // ProcessContent implements the Processor interface for JSONProcessor
 func (p *JSONProcessor) ProcessContent(content string, pattern string, luaExpr string) (string, int, int, error) {
+	logger.Debug("Processing JSON content with JSONPath: %s", pattern)
+
 	// Parse JSON document
+	logger.Trace("Parsing JSON document")
 	var jsonData interface{}
 	err := json.Unmarshal([]byte(content), &jsonData)
 	if err != nil {
+		logger.Error("Failed to parse JSON: %v", err)
 		return content, 0, 0, fmt.Errorf("error parsing JSON: %v", err)
 	}
 
 	// Find nodes matching the JSONPath pattern
+	logger.Debug("Executing JSONPath query: %s", pattern)
 	nodes, err := jsonpath.Get(jsonData, pattern)
 	if err != nil {
+		logger.Error("Failed to execute JSONPath: %v", err)
 		return content, 0, 0, fmt.Errorf("error getting nodes: %v", err)
 	}
 
 	matchCount := len(nodes)
+	logger.Debug("Found %d nodes matching JSONPath", matchCount)
 	if matchCount == 0 {
+		logger.Warning("No nodes matched the JSONPath pattern: %s", pattern)
 		return content, 0, 0, nil
 	}
 
-	// Initialize Lua
+	modCount := 0
-	L, err := NewLuaState()
+	for i, node := range nodes {
-	if err != nil {
+		logger.Trace("Processing node #%d at path: %s with value: %v", i+1, node.Path, node.Value)
-		return content, len(nodes), 0, fmt.Errorf("error creating Lua state: %v", err)
-	}
-	defer L.Close()
 
-	err = p.ToLua(L, nodes)
+		// Initialize Lua
-	if err != nil {
+		L, err := NewLuaState()
-		return content, len(nodes), 0, fmt.Errorf("error converting to Lua: %v", err)
+		if err != nil {
-	}
+			logger.Error("Failed to create Lua state: %v", err)
+			return content, len(nodes), 0, fmt.Errorf("error creating Lua state: %v", err)
+		}
+		defer L.Close()
+		logger.Trace("Lua state initialized successfully")
 
-	// Execute Lua script
+		err = p.ToLua(L, node.Value)
-	if err := L.DoString(luaExpr); err != nil {
+		if err != nil {
-		return content, len(nodes), 0, fmt.Errorf("error executing Lua %s: %v", luaExpr, err)
+			logger.Error("Failed to convert value to Lua: %v", err)
-	}
+			return content, len(nodes), 0, fmt.Errorf("error converting to Lua: %v", err)
+		}
+		logger.Trace("Converted node value to Lua: %v", node.Value)
 
-	// Get modified value
+		originalScript := luaExpr
-	result, err := p.FromLua(L)
+		fullScript := BuildLuaScript(luaExpr)
-	if err != nil {
+		logger.Debug("Original script: %q, Full script: %q", originalScript, fullScript)
-		return content, len(nodes), 0, fmt.Errorf("error getting result from Lua: %v", err)
-	}
 
-	// Apply the modification to the JSON data
+		// Execute Lua script
-	err = p.updateJSONValue(jsonData, pattern, result)
+		logger.Trace("Executing Lua script: %q", fullScript)
-	if err != nil {
+		if err := L.DoString(fullScript); err != nil {
-		return content, len(nodes), 0, fmt.Errorf("error updating JSON: %v", err)
+			logger.Error("Failed to execute Lua script: %v", err)
-	}
+			return content, len(nodes), 0, fmt.Errorf("error executing Lua %q: %v", fullScript, err)
+		}
+		logger.Trace("Lua script executed successfully")
 
-	// Convert the modified JSON back to a string with same formatting
+		// Get modified value
-	var jsonBytes []byte
+		result, err := p.FromLua(L)
-	if indent, err := detectJsonIndentation(content); err == nil && indent != "" {
+		if err != nil {
-		// Use detected indentation for output formatting
+			logger.Error("Failed to get result from Lua: %v", err)
-		jsonBytes, err = json.MarshalIndent(jsonData, "", indent)
+			return content, len(nodes), 0, fmt.Errorf("error getting result from Lua: %v", err)
-	} else {
+		}
-		// Fall back to standard 2-space indent
+		logger.Trace("Retrieved modified value from Lua: %v", result)
-		jsonBytes, err = json.MarshalIndent(jsonData, "", "  ")
-	}
 
-	// We changed all the nodes trust me bro
+		modified := false
-	return string(jsonBytes), len(nodes), len(nodes), nil
+		modified = L.GetGlobal("modified").String() == "true"
-}
+		if !modified {
-
+			logger.Debug("No changes made to node at path: %s", node.Path)
-// detectJsonIndentation tries to determine the indentation used in the original JSON
-func detectJsonIndentation(content string) (string, error) {
-	lines := strings.Split(content, "\n")
-	if len(lines) < 2 {
-		return "", fmt.Errorf("not enough lines to detect indentation")
-	}
-
-	// Look for the first indented line
-	for i := 1; i < len(lines); i++ {
-		line := lines[i]
-		trimmed := strings.TrimSpace(line)
-		if trimmed == "" {
 			continue
 		}
 
-		// Calculate leading whitespace
+		// Apply the modification to the JSON data
-		indent := line[:len(line)-len(trimmed)]
+		logger.Debug("Updating JSON at path: %s with new value: %v", node.Path, result)
-		if len(indent) > 0 {
+		err = p.updateJSONValue(jsonData, node.Path, result)
-			return indent, nil
+		if err != nil {
+			logger.Error("Failed to update JSON at path %s: %v", node.Path, err)
+			return content, len(nodes), 0, fmt.Errorf("error updating JSON: %v", err)
 		}
+		logger.Debug("Updated JSON at path: %s successfully", node.Path)
+		modCount++
 	}
 
-	return "", fmt.Errorf("no indentation detected")
+	logger.Info("JSON processing complete: %d modifications from %d matches", modCount, matchCount)
+
+	// Convert the modified JSON back to a string with same formatting
+	logger.Trace("Marshalling JSON data back to string")
+	var jsonBytes []byte
+	jsonBytes, err = json.MarshalIndent(jsonData, "", "  ")
+	if err != nil {
+		logger.Error("Failed to marshal JSON: %v", err)
+		return content, modCount, matchCount, fmt.Errorf("error marshalling JSON: %v", err)
+	}
+	return string(jsonBytes), modCount, matchCount, nil
 }
 
-// / 	Selects from the root node
-// // 	Selects nodes in the document from the current node that match the selection no matter where they are
-// . 	Selects the current node
-// @ 	Selects attributes
-
-// /bookstore/* 	Selects all the child element nodes of the bookstore element
-// //* 	Selects all elements in the document
-
-// /bookstore/book[1] 	Selects the first book element that is the child of the bookstore element.
-// /bookstore/book[last()] 	Selects the last book element that is the child of the bookstore element
-// /bookstore/book[last()-1] 	Selects the last but one book element that is the child of the bookstore element
-// /bookstore/book[position()<3] 	Selects the first two book elements that are children of the bookstore element
-// //title[@lang] 	Selects all the title elements that have an attribute named lang
-// //title[@lang='en'] 	Selects all the title elements that have a "lang" attribute with a value of "en"
-// /bookstore/book[price>35.00] 	Selects all the book elements of the bookstore element that have a price element with a value greater than 35.00
-// /bookstore/book[price>35.00]/title 	Selects all the title elements of the book elements of the bookstore element that have a price element with a value greater than 35.00
-
 // updateJSONValue updates a value in the JSON structure based on its JSONPath
 func (p *JSONProcessor) updateJSONValue(jsonData interface{}, path string, newValue interface{}) error {
+	logger.Trace("Updating JSON value at path: %s", path)
+
+	// Special handling for root node
+	if path == "$" {
+		logger.Debug("Handling special case for root node update")
+		// For the root node, we'll copy the value to the jsonData reference
+		// This is a special case since we can't directly replace the interface{} variable
+
+		// We need to handle different types of root elements
+		switch rootValue := newValue.(type) {
+		case map[string]interface{}:
+			// For objects, we need to copy over all keys
+			rootMap, ok := jsonData.(map[string]interface{})
+			if !ok {
+				// If the original wasn't a map, completely replace it with the new map
+				// This is handled by the jsonpath.Set function
+				logger.Debug("Root was not a map, replacing entire root")
+				return jsonpath.Set(jsonData, path, newValue)
+			}
+
+			// Clear the original map
+			logger.Trace("Clearing original root map")
+			for k := range rootMap {
+				delete(rootMap, k)
+			}
+
+			// Copy all keys from the new map
+			logger.Trace("Copying keys to root map")
+			for k, v := range rootValue {
+				rootMap[k] = v
+			}
+			return nil
+
+		case []interface{}:
+			// For arrays, we need to handle similarly
+			rootArray, ok := jsonData.([]interface{})
+			if !ok {
+				// If the original wasn't an array, use jsonpath.Set
+				logger.Debug("Root was not an array, replacing entire root")
+				return jsonpath.Set(jsonData, path, newValue)
+			}
+
+			// Clear and recreate the array
+			logger.Trace("Replacing root array")
+			*&rootArray = rootValue
+			return nil
+
+		default:
+			// For other types, use jsonpath.Set
+			logger.Debug("Replacing root with primitive value")
+			return jsonpath.Set(jsonData, path, newValue)
+		}
+	}
+
+	// For non-root paths, use the regular Set method
+	logger.Trace("Using regular Set method for non-root path")
+	err := jsonpath.Set(jsonData, path, newValue)
+	if err != nil {
+		logger.Error("Failed to set JSON value at path %s: %v", path, err)
+		return fmt.Errorf("failed to update JSON value at path '%s': %w", path, err)
+	}
 	return nil
 }
 
 // ToLua converts JSON values to Lua variables
 func (p *JSONProcessor) ToLua(L *lua.LState, data interface{}) error {
-	table, err := ToLuaTable(L, data)
+	table, err := ToLua(L, data)
 	if err != nil {
 		return err
 	}
@@ -170,5 +190,5 @@ func (p *JSONProcessor) ToLua(L *lua.LState, data interface{}) error {
 // FromLua retrieves values from Lua
 func (p *JSONProcessor) FromLua(L *lua.LState) (interface{}, error) {
 	luaValue := L.GetGlobal("v")
-	return FromLuaTable(L, luaValue.(*lua.LTable))
+	return FromLua(L, luaValue)
 }

processor/jsonpath/jsonpath.go

@@ -138,10 +138,6 @@ func Set(data interface{}, path string, value interface{}) error {
 		return fmt.Errorf("failed to parse JSONPath %q: %w", path, err)
 	}
 
-	if len(steps) <= 1 {
-		return fmt.Errorf("cannot set root node; the provided path %q is invalid", path)
-	}
-
 	success := false
 	err = setWithPath(data, steps, &success, value, "$", ModifyFirstMode)
 	if err != nil {
@@ -157,10 +153,6 @@ func SetAll(data interface{}, path string, value interface{}) error {
 		return fmt.Errorf("failed to parse JSONPath %q: %w", path, err)
 	}
 
-	if len(steps) <= 1 {
-		return fmt.Errorf("cannot set root node; the provided path %q is invalid", path)
-	}
-
 	success := false
 	err = setWithPath(data, steps, &success, value, "$", ModifyAllMode)
 	if err != nil {
@@ -178,17 +170,20 @@ func setWithPath(node interface{}, steps []JSONStep, success *bool, value interf
 	// Skip root step
 	actualSteps := steps
 	if len(steps) > 0 && steps[0].Type == RootStep {
-		if len(steps) == 1 {
-			return fmt.Errorf("cannot set root node; the provided path %q is invalid", currentPath)
-		}
 		actualSteps = steps[1:]
 	}
 
-	// Process the first step
+	// If we have no steps left, we're setting the root value
 	if len(actualSteps) == 0 {
-		return fmt.Errorf("cannot set root node; no steps provided for path %q", currentPath)
+		// For the root node, we need to handle it differently depending on what's passed in
+		// since we can't directly replace the interface{} variable
+
+		// We'll signal success and let the JSONProcessor handle updating the root
+		*success = true
+		return nil
 	}
 
+	// Process the first step
 	step := actualSteps[0]
 	remainingSteps := actualSteps[1:]
 	isLastStep := len(remainingSteps) == 0

processor/jsonpath/jsonpath_get_set_test.go

@@ -355,14 +355,14 @@ func TestSet(t *testing.T) {
 		}
 	})
 
-	t.Run("setting on root should fail", func(t *testing.T) {
+	t.Run("setting on root should not fail (anymore)", func(t *testing.T) {
 		data := map[string]interface{}{
 			"name": "John",
 		}
 
 		err := Set(data, "$", "Jane")
-		if err == nil {
+		if err != nil {
-			t.Errorf("Set() returned no error, expected error for setting on root")
+			t.Errorf("Set() returned error: %v", err)
 			return
 		}
 

processor/processor.go

@@ -1,17 +1,24 @@
 package processor
 
 import (
+	"crypto/md5"
 	"fmt"
-	"reflect"
+	"os"
+	"path/filepath"
 	"strings"
+	"time"
 
+	"github.com/antchfx/xmlquery"
 	lua "github.com/yuin/gopher-lua"
+
+	"modify/logger"
 )
 
 // Processor defines the interface for all file processors
 type Processor interface {
 	// Process handles processing a file with the given pattern and Lua expression
-	Process(filename string, pattern string, luaExpr string) (int, int, error)
+	// Now implemented as a base function in processor.go
+	// Process(filename string, pattern string, luaExpr string) (int, int, error)
 
 	// ProcessContent handles processing a string content directly with the given pattern and Lua expression
 	// Returns the modified content, modification count, match count, and any error
@@ -38,92 +45,274 @@ func NewLuaState() (*lua.LState, error) {
 	// defer L.Close()
 
 	// Load math library
+	logger.Debug("Loading Lua math library")
 	L.Push(L.GetGlobal("require"))
 	L.Push(lua.LString("math"))
 	if err := L.PCall(1, 1, nil); err != nil {
+		logger.Error("Failed to load Lua math library: %v", err)
 		return nil, fmt.Errorf("error loading Lua math library: %v", err)
 	}
 
 	// Initialize helper functions
+	logger.Debug("Initializing Lua helper functions")
 	if err := InitLuaHelpers(L); err != nil {
+		logger.Error("Failed to initialize Lua helper functions: %v", err)
 		return nil, err
 	}
 
 	return L, nil
 }
 
-// ToLuaTable converts a struct or map to a Lua table recursively
+func Process(p Processor, filename string, pattern string, luaExpr string) (int, int, error) {
-func ToLuaTable(L *lua.LState, data interface{}) (*lua.LTable, error) {
+	logger.Debug("Processing file %q with pattern %q", filename, pattern)
-	luaTable := L.NewTable()
 
+	// Read file content
+	cwd, err := os.Getwd()
+	if err != nil {
+		logger.Error("Failed to get current working directory: %v", err)
+		return 0, 0, fmt.Errorf("error getting current working directory: %v", err)
+	}
+
+	fullPath := filepath.Join(cwd, filename)
+	logger.Trace("Reading file from: %s", fullPath)
+
+	stat, err := os.Stat(fullPath)
+	if err != nil {
+		logger.Error("Failed to stat file %s: %v", fullPath, err)
+		return 0, 0, fmt.Errorf("error getting file info: %v", err)
+	}
+	logger.Debug("File size: %d bytes, modified: %s", stat.Size(), stat.ModTime().Format(time.RFC3339))
+
+	content, err := os.ReadFile(fullPath)
+	if err != nil {
+		logger.Error("Failed to read file %s: %v", fullPath, err)
+		return 0, 0, fmt.Errorf("error reading file: %v", err)
+	}
+
+	fileContent := string(content)
+	logger.Trace("File read successfully: %d bytes, hash: %x", len(content), md5sum(content))
+
+	// Detect and log file type
+	fileType := detectFileType(filename, fileContent)
+	if fileType != "" {
+		logger.Debug("Detected file type: %s", fileType)
+	}
+
+	// Process the content
+	logger.Debug("Starting content processing with %s processor", getProcessorType(p))
+	modifiedContent, modCount, matchCount, err := p.ProcessContent(fileContent, pattern, luaExpr)
+	if err != nil {
+		logger.Error("Processing error: %v", err)
+		return 0, 0, err
+	}
+
+	logger.Debug("Processing results: %d matches, %d modifications", matchCount, modCount)
+
+	// If we made modifications, save the file
+	if modCount > 0 {
+		// Calculate changes summary
+		changePercent := float64(len(modifiedContent)) / float64(len(fileContent)) * 100
+		logger.Info("File size change: %d → %d bytes (%.1f%%)",
+			len(fileContent), len(modifiedContent), changePercent)
+
+		logger.Debug("Writing modified content to %s", fullPath)
+		err = os.WriteFile(fullPath, []byte(modifiedContent), 0644)
+		if err != nil {
+			logger.Error("Failed to write to file %s: %v", fullPath, err)
+			return 0, 0, fmt.Errorf("error writing file: %v", err)
+		}
+		logger.Debug("File written successfully, new hash: %x", md5sum([]byte(modifiedContent)))
+	} else if matchCount > 0 {
+		logger.Debug("No content modifications needed for %d matches", matchCount)
+	} else {
+		logger.Debug("No matches found in file")
+	}
+
+	return modCount, matchCount, nil
+}
+
+// Helper function to get a short MD5 hash of content for logging
+func md5sum(data []byte) []byte {
+	h := md5.New()
+	h.Write(data)
+	return h.Sum(nil)[:4] // Just use first 4 bytes for brevity
+}
+
+// Helper function to detect basic file type from extension and content
+func detectFileType(filename string, content string) string {
+	ext := strings.ToLower(filepath.Ext(filename))
+
+	switch ext {
+	case ".xml":
+		return "XML"
+	case ".json":
+		return "JSON"
+	case ".html", ".htm":
+		return "HTML"
+	case ".txt":
+		return "Text"
+	case ".go":
+		return "Go"
+	case ".js":
+		return "JavaScript"
+	case ".py":
+		return "Python"
+	case ".java":
+		return "Java"
+	case ".c", ".cpp", ".h":
+		return "C/C++"
+	default:
+		// Try content-based detection for common formats
+		if strings.HasPrefix(strings.TrimSpace(content), "<?xml") {
+			return "XML"
+		}
+		if strings.HasPrefix(strings.TrimSpace(content), "{") ||
+			strings.HasPrefix(strings.TrimSpace(content), "[") {
+			return "JSON"
+		}
+		if strings.HasPrefix(strings.TrimSpace(content), "<!DOCTYPE html") ||
+			strings.HasPrefix(strings.TrimSpace(content), "<html") {
+			return "HTML"
+		}
+		return ""
+	}
+}
+
+// Helper function to get processor type name
+func getProcessorType(p Processor) string {
+	switch p.(type) {
+	case *RegexProcessor:
+		return "Regex"
+	case *XMLProcessor:
+		return "XML"
+	case *JSONProcessor:
+		return "JSON"
+	default:
+		return "Unknown"
+	}
+}
+
+// ToLua converts a struct or map to a Lua table recursively
+func ToLua(L *lua.LState, data interface{}) (lua.LValue, error) {
 	switch v := data.(type) {
+	case *xmlquery.Node:
+		luaTable := L.NewTable()
+		luaTable.RawSetString("text", lua.LString(v.Data))
+		// Should be a map, simple key value pairs
+		attr, err := ToLua(L, v.Attr)
+		if err != nil {
+			return nil, err
+		}
+		luaTable.RawSetString("attr", attr)
+		return luaTable, nil
 	case map[string]interface{}:
+		luaTable := L.NewTable()
 		for key, value := range v {
-			luaValue, err := ToLuaTable(L, value)
+			luaValue, err := ToLua(L, value)
 			if err != nil {
 				return nil, err
 			}
 			luaTable.RawSetString(key, luaValue)
 		}
-	case struct{}:
+		return luaTable, nil
-		val := reflect.ValueOf(v)
+	case []interface{}:
-		for i := 0; i < val.NumField(); i++ {
+		luaTable := L.NewTable()
-			field := val.Type().Field(i)
+		for i, value := range v {
-			luaValue, err := ToLuaTable(L, val.Field(i).Interface())
+			luaValue, err := ToLua(L, value)
 			if err != nil {
 				return nil, err
 			}
-			luaTable.RawSetString(field.Name, luaValue)
+			luaTable.RawSetInt(i+1, luaValue) // Lua arrays are 1-indexed
 		}
+		return luaTable, nil
 	case string:
-		luaTable.RawSetString("v", lua.LString(v))
+		return lua.LString(v), nil
 	case bool:
-		luaTable.RawSetString("v", lua.LBool(v))
+		return lua.LBool(v), nil
 	case float64:
-		luaTable.RawSetString("v", lua.LNumber(v))
+		return lua.LNumber(v), nil
+	case nil:
+		return lua.LNil, nil
 	default:
 		return nil, fmt.Errorf("unsupported data type: %T", data)
 	}
-	return luaTable, nil
 }
 
-// FromLuaTable converts a Lua table to a struct or map recursively
+// FromLua converts a Lua table to a struct or map recursively
-func FromLuaTable(L *lua.LState, luaTable *lua.LTable) (map[string]interface{}, error) {
+func FromLua(L *lua.LState, luaValue lua.LValue) (interface{}, error) {
-	result := make(map[string]interface{})
+	switch v := luaValue.(type) {
-
+	// Well shit...
-	luaTable.ForEach(func(key lua.LValue, value lua.LValue) {
+	// Tables in lua are both maps and arrays
-		switch v := value.(type) {
+	// As arrays they are ordered and as maps, obviously, not
-		case *lua.LTable:
+	// So when we parse them to a go map we fuck up the order for arrays
-			nestedMap, err := FromLuaTable(L, v)
+	// We have to find a better way....
-			if err != nil {
+	case *lua.LTable:
-				return
+		isArray, err := IsLuaTableArray(L, v)
-			}
+		if err != nil {
-			result[key.String()] = nestedMap
+			return nil, err
-		case lua.LString:
-			result[key.String()] = string(v)
-		case lua.LBool:
-			result[key.String()] = bool(v)
-		case lua.LNumber:
-			result[key.String()] = float64(v)
-		default:
-			result[key.String()] = nil
 		}
-	})
+		if isArray {
+			result := make([]interface{}, 0)
+			v.ForEach(func(key lua.LValue, value lua.LValue) {
+				converted, _ := FromLua(L, value)
+				result = append(result, converted)
+			})
+			return result, nil
+		} else {
+			result := make(map[string]interface{})
+			v.ForEach(func(key lua.LValue, value lua.LValue) {
+				converted, _ := FromLua(L, value)
+				result[key.String()] = converted
+			})
+			return result, nil
+		}
+	case lua.LString:
+		return string(v), nil
+	case lua.LBool:
+		return bool(v), nil
+	case lua.LNumber:
+		return float64(v), nil
+	default:
+		return nil, nil
+	}
+}
 
+func IsLuaTableArray(L *lua.LState, v *lua.LTable) (bool, error) {
+	logger.Trace("Checking if Lua table is an array")
+	L.SetGlobal("table_to_check", v)
+
+	// Use our predefined helper function from InitLuaHelpers
+	err := L.DoString(`is_array = isArray(table_to_check)`)
+	if err != nil {
+		logger.Error("Error determining if table is an array: %v", err)
+		return false, fmt.Errorf("error determining if table is array: %w", err)
+	}
+
+	// Check the result of our Lua function
+	isArray := L.GetGlobal("is_array")
+	// LVIsFalse returns true if a given LValue is a nil or false otherwise false.
+	result := !lua.LVIsFalse(isArray)
+	logger.Trace("Lua table is array: %v", result)
 	return result, nil
 }
 
 // InitLuaHelpers initializes common Lua helper functions
 func InitLuaHelpers(L *lua.LState) error {
+	logger.Debug("Loading Lua helper functions")
+
 	helperScript := `
 -- Custom Lua helpers for math operations
 function min(a, b) return math.min(a, b) end
 function max(a, b) return math.max(a, b) end
-function round(x) return math.floor(x + 0.5) end
+function round(x, n)
+	if n == nil then n = 0 end
+	return math.floor(x * 10^n + 0.5) / 10^n
+end
 function floor(x) return math.floor(x) end
 function ceil(x) return math.ceil(x) end
 function upper(s) return string.upper(s) end
 function lower(s) return string.lower(s) end
+function format(s, ...) return string.format(s, ...) end
 
 -- String to number conversion helper
 function num(str)
@@ -139,10 +328,30 @@ end
 function is_number(str)
     return tonumber(str) ~= nil
 end
+
+function isArray(t)
+    if type(t) ~= "table" then return false end
+    local max = 0
+    local count = 0
+    for k, _ in pairs(t) do
+        if type(k) ~= "number" or k < 1 or math.floor(k) ~= k then
+            return false
+        end
+        max = math.max(max, k)
+        count = count + 1
+    end
+    return max == count
+end
+
+modified = false
 `
 	if err := L.DoString(helperScript); err != nil {
+		logger.Error("Failed to load Lua helper functions: %v", err)
 		return fmt.Errorf("error loading helper functions: %v", err)
 	}
+
+	logger.Debug("Setting up Lua print function to Go")
+	L.SetGlobal("print", L.NewFunction(printToGo))
 	return nil
 }
 
@@ -157,8 +366,7 @@ func LimitString(s string, maxLen int) string {
 	return s[:maxLen-3] + "..."
 }
 
-// BuildLuaScript prepares a Lua expression from shorthand notation
+func PrependLuaAssignment(luaExpr string) string {
-func BuildLuaScript(luaExpr string) string {
 	// Auto-prepend v1 for expressions starting with operators
 	if strings.HasPrefix(luaExpr, "*") ||
 		strings.HasPrefix(luaExpr, "/") ||
@@ -176,10 +384,43 @@ func BuildLuaScript(luaExpr string) string {
 	if !strings.Contains(luaExpr, "=") {
 		luaExpr = "v1 = " + luaExpr
 	}
-
 	return luaExpr
 }
 
+// BuildLuaScript prepares a Lua expression from shorthand notation
+func BuildLuaScript(luaExpr string) string {
+	logger.Debug("Building Lua script from expression: %s", luaExpr)
+
+	luaExpr = PrependLuaAssignment(luaExpr)
+
+	// This allows the user to specify whether or not they modified a value
+	// If they do nothing we assume they did modify (no return at all)
+	// If they return before our return then they themselves specify what they did
+	// If nothing is returned lua assumes nil
+	// So we can say our value was modified if the return value is either nil or true
+	// If the return value is false then the user wants to keep the original
+	fullScript := fmt.Sprintf(`
+		function run()
+			%s
+		end
+		local res = run()
+		modified = res == nil or res
+	`, luaExpr)
+
+	return fullScript
+}
+
+func printToGo(L *lua.LState) int {
+	top := L.GetTop()
+	args := make([]interface{}, top)
+	for i := 1; i <= top; i++ {
+		args[i-1] = L.Get(i)
+	}
+	message := fmt.Sprint(args...)
+	logger.Info("[Lua] %s", message)
+	return 0
+}
+
 // Max returns the maximum of two integers
 func Max(a, b int) int {
 	if a > b {

processor/regex.go

@@ -2,120 +2,114 @@ package processor
 
 import (
 	"fmt"
-	"os"
-	"path/filepath"
 	"regexp"
+	"sort"
 	"strconv"
 	"strings"
 
 	lua "github.com/yuin/gopher-lua"
+
+	"modify/logger"
 )
 
 // RegexProcessor implements the Processor interface using regex patterns
 type RegexProcessor struct{}
 
-// Process implements the Processor interface for RegexProcessor
-func (p *RegexProcessor) Process(filename string, pattern string, luaExpr string) (int, int, error) {
-	// Read file content
-	fullPath := filepath.Join(".", filename)
-	content, err := os.ReadFile(fullPath)
-	if err != nil {
-		return 0, 0, fmt.Errorf("error reading file: %v", err)
-	}
-
-	fileContent := string(content)
-
-	// Process the content
-	modifiedContent, modCount, matchCount, err := p.ProcessContent(fileContent, pattern, luaExpr)
-	if err != nil {
-		return 0, 0, err
-	}
-
-	// If we made modifications, save the file
-	if modCount > 0 {
-		err = os.WriteFile(fullPath, []byte(modifiedContent), 0644)
-		if err != nil {
-			return 0, 0, fmt.Errorf("error writing file: %v", err)
-		}
-	}
-
-	return modCount, matchCount, nil
-}
-
 // ToLua sets capture groups as Lua variables (v1, v2, etc. for numeric values and s1, s2, etc. for strings)
 func (p *RegexProcessor) ToLua(L *lua.LState, data interface{}) error {
-	captures, ok := data.([]string)
+	captureGroups, ok := data.([]*CaptureGroup)
 	if !ok {
-		return fmt.Errorf("expected []string for captures, got %T", data)
+		return fmt.Errorf("expected []*CaptureGroup for captures, got %T", data)
 	}
 
-	// Set variables for each capture group, starting from v1/s1 for the first capture
+	groupindex := 0
-	for i := 0; i < len(captures); i++ {
+	for _, capture := range captureGroups {
-		// Set string version (always available as s1, s2, etc.)
+		if capture.Name == "" {
-		L.SetGlobal(fmt.Sprintf("s%d", i+1), lua.LString(captures[i]))
+			// We don't want to change the name of the capture group
+			// Even if it's empty
+			tempName := fmt.Sprintf("%d", groupindex+1)
+			groupindex++
 
-		// Try to convert to number and set v1, v2, etc.
+			L.SetGlobal("s"+tempName, lua.LString(capture.Value))
-		if val, err := strconv.ParseFloat(captures[i], 64); err == nil {
+
-			L.SetGlobal(fmt.Sprintf("v%d", i+1), lua.LNumber(val))
+			val, err := strconv.ParseFloat(capture.Value, 64)
+			if err == nil {
+				L.SetGlobal("v"+tempName, lua.LNumber(val))
+			}
+		} else {
+			val, err := strconv.ParseFloat(capture.Value, 64)
+			if err == nil {
+				L.SetGlobal(capture.Name, lua.LNumber(val))
+			} else {
+				L.SetGlobal(capture.Name, lua.LString(capture.Value))
+			}
 		}
 	}
 
 	return nil
 }
 
-// FromLua implements the Processor interface for RegexProcessor
 func (p *RegexProcessor) FromLua(L *lua.LState) (interface{}, error) {
-	// Get the modified values after Lua execution
+	// Stub to satisfy interface
-	modifications := make(map[int]string)
+	return nil, nil
+}
 
-	// Check for modifications to v1-v12 and s1-s12
+// FromLua implements the Processor interface for RegexProcessor
-	for i := 0; i < 12; i++ {
+func (p *RegexProcessor) FromLuaCustom(L *lua.LState, captureGroups []*CaptureGroup) ([]*CaptureGroup, error) {
-		// Check both v and s variables to see if any were modified
+	captureIndex := 0
-		vVarName := fmt.Sprintf("v%d", i+1)
+	for _, capture := range captureGroups {
-		sVarName := fmt.Sprintf("s%d", i+1)
+		if capture.Name == "" {
+			capture.Name = fmt.Sprintf("%d", captureIndex+1)
 
-		vLuaVal := L.GetGlobal(vVarName)
+			vVarName := fmt.Sprintf("v%s", capture.Name)
-		sLuaVal := L.GetGlobal(sVarName)
+			sVarName := fmt.Sprintf("s%s", capture.Name)
+			captureIndex++
 
-		// If our value is a number then it's very likely we want it to be a number
+			vLuaVal := L.GetGlobal(vVarName)
-		// And not a string
+			sLuaVal := L.GetGlobal(sVarName)
-		// If we do want it to be a string we will cast it into a string in lua
+
-		// wait that wouldn't work... Casting v to a string would not load it here
+			if sLuaVal.Type() == lua.LTString {
-		if vLuaVal.Type() == lua.LTNumber {
+				capture.Updated = sLuaVal.String()
-			modifications[i] = vLuaVal.String()
+			}
-			continue
+			// Numbers have priority
+			if vLuaVal.Type() == lua.LTNumber {
+				capture.Updated = vLuaVal.String()
+			}
+		} else {
+			// Easy shit
+			capture.Updated = L.GetGlobal(capture.Name).String()
 		}
-		if sLuaVal.Type() == lua.LTString {
-			modifications[i] = sLuaVal.String()
-			continue
-		}
-
 	}
 
-	return modifications, nil
+	return captureGroups, nil
+}
+
+type CaptureGroup struct {
+	Name    string
+	Value   string
+	Updated string
+	Range   [2]int
+}
+type ReplaceCommand struct {
+	From int
+	To   int
+	With string
 }
 
 // ProcessContent applies regex replacement with Lua processing
 func (p *RegexProcessor) ProcessContent(content string, pattern string, luaExpr string) (string, int, int, error) {
-	// Handle special pattern modifications
+	pattern = ResolveRegexPlaceholders(pattern)
-	if !strings.HasPrefix(pattern, "(?s)") {
+	logger.Debug("Compiling regex pattern: %s", pattern)
-		pattern = "(?s)" + pattern
-	}
-
 	compiledPattern, err := regexp.Compile(pattern)
 	if err != nil {
+		logger.Error("Error compiling pattern: %v", err)
 		return "", 0, 0, fmt.Errorf("error compiling pattern: %v", err)
 	}
+	logger.Debug("Compiled pattern successfully: %s", pattern)
 
 	previous := luaExpr
 	luaExpr = BuildLuaScript(luaExpr)
-	fmt.Printf("Changing Lua expression from: %s to: %s\n", previous, luaExpr)
+	logger.Debug("Transformed Lua expression: %q → %q", previous, luaExpr)
-
-	L, err := NewLuaState()
-	if err != nil {
-		return "", 0, 0, fmt.Errorf("error creating Lua state: %v", err)
-	}
-	defer L.Close()
 
 	// Initialize Lua environment
 	modificationCount := 0
@@ -123,12 +117,29 @@ func (p *RegexProcessor) ProcessContent(content string, pattern string, luaExpr
 	// Process all regex matches
 	result := content
 	indices := compiledPattern.FindAllStringSubmatchIndex(content, -1)
+	logger.Debug("Found %d matches in content of length %d", len(indices), len(content))
+
 	// We walk backwards because we're replacing something with something else that might be longer
 	// And in the case it is longer than the original all indicces past that change will be fucked up
 	// By going backwards we fuck up all the indices to the end of the file that we don't care about
 	// Because there either aren't any (last match) or they're already modified (subsequent matches)
 	for i := len(indices) - 1; i >= 0; i-- {
+		logger.Debug("Processing match %d of %d", i+1, len(indices))
+
+		L, err := NewLuaState()
+		if err != nil {
+			logger.Error("Error creating Lua state: %v", err)
+			return "", 0, 0, fmt.Errorf("error creating Lua state: %v", err)
+		}
+		// Hmm... Maybe we don't want to defer this..
+		// Maybe we want to close them every iteration
+		// We'll leave it as is for now
+		defer L.Close()
+		logger.Trace("Lua state created successfully for match %d", i+1)
+
 		matchIndices := indices[i]
+		logger.Trace("Match indices: %v (match position %d-%d)", matchIndices, matchIndices[0], matchIndices[1])
+
 		// Why we're doing this whole song and dance of indices is to properly handle empty matches
 		// Plus it's a little cleaner to surgically replace our matches
 		// If we were to use string.replace and encountered an empty match there'd be nothing to replace
@@ -137,60 +148,199 @@ func (p *RegexProcessor) ProcessContent(content string, pattern string, luaExpr
 		// As if concatenating in the middle of the array
 		// Plus it supports lookarounds
 		match := content[matchIndices[0]:matchIndices[1]]
+		matchPreview := match
+		if len(match) > 50 {
+			matchPreview = match[:47] + "..."
+		}
+		logger.Trace("Matched content: %q (length: %d)", matchPreview, len(match))
 
 		groups := matchIndices[2:]
 		if len(groups) <= 0 {
-			fmt.Println("No capture groups for lua to chew on")
+			logger.Warning("No capture groups found for match %q and regex %q", matchPreview, pattern)
 			continue
 		}
 		if len(groups)%2 == 1 {
-			fmt.Println("Odd number of indices of groups, what the fuck?")
+			logger.Warning("Invalid number of group indices (%d), should be even: %v", len(groups), groups)
 			continue
 		}
 
-		captures := make([]string, 0, len(groups)/2)
+		// Count how many valid groups we have
+		validGroups := 0
 		for j := 0; j < len(groups); j += 2 {
-			captures = append(captures, content[groups[j]:groups[j+1]])
+			if groups[j] != -1 && groups[j+1] != -1 {
+				validGroups++
+			}
+		}
+		logger.Debug("Found %d valid capture groups in match", validGroups)
+
+		for _, index := range groups {
+			if index == -1 {
+				logger.Warning("Negative index encountered in match indices %v. This may indicate an issue with the regex pattern or an empty/optional capture group.", matchIndices)
+				continue
+			}
 		}
 
-		if err := p.ToLua(L, captures); err != nil {
+		// We have to use array to preserve order
-			fmt.Println("Error setting Lua variables:", err)
+		// Very important for the reconstruction step
+		// Because we must overwrite the values in reverse order
+		// See comments a few dozen lines above for more details
+		captureGroups := make([]*CaptureGroup, 0, len(groups)/2)
+		groupNames := compiledPattern.SubexpNames()[1:]
+		for i, name := range groupNames {
+			start := groups[i*2]
+			end := groups[i*2+1]
+			if start == -1 || end == -1 {
+				continue
+			}
+
+			value := content[start:end]
+			captureGroups = append(captureGroups, &CaptureGroup{
+				Name:  name,
+				Value: value,
+				Range: [2]int{start, end},
+			})
+
+			// Include name info in log if available
+			if name != "" {
+				logger.Trace("Capture group '%s': %q (pos %d-%d)", name, value, start, end)
+			} else {
+				logger.Trace("Capture group #%d: %q (pos %d-%d)", i+1, value, start, end)
+			}
+		}
+
+		if err := p.ToLua(L, captureGroups); err != nil {
+			logger.Error("Failed to set Lua variables: %v", err)
 			continue
 		}
+		logger.Trace("Set %d capture groups as Lua variables", len(captureGroups))
 
 		if err := L.DoString(luaExpr); err != nil {
-			fmt.Printf("Error executing Lua code %s for group %s: %v", luaExpr, captures, err)
+			logger.Error("Lua script execution failed: %v\nScript: %s\nCapture Groups: %+v",
+				err, luaExpr, captureGroups)
 			continue
 		}
+		logger.Trace("Lua script executed successfully")
 
 		// Get modifications from Lua
-		modResult, err := p.FromLua(L)
+		captureGroups, err = p.FromLuaCustom(L, captureGroups)
 		if err != nil {
-			fmt.Println("Error getting modifications:", err)
+			logger.Error("Failed to retrieve modifications from Lua: %v", err)
+			continue
+		}
+		logger.Trace("Retrieved updated values from Lua")
+
+		replacement := ""
+		replacementVar := L.GetGlobal("replacement")
+		if replacementVar.Type() != lua.LTNil {
+			replacement = replacementVar.String()
+			logger.Debug("Using global replacement: %q", replacement)
+		}
+
+		// Check if modification flag is set
+		modifiedVal := L.GetGlobal("modified")
+		if modifiedVal.Type() != lua.LTBool || !lua.LVAsBool(modifiedVal) {
+			logger.Debug("Skipping match - no modifications made by Lua script")
 			continue
 		}
 
-		// Apply modifications to the matched text
+		if replacement == "" {
-		modsMap, ok := modResult.(map[int]string)
+			commands := make([]ReplaceCommand, 0, len(captureGroups))
-		if !ok || len(modsMap) == 0 {
+			// Apply the modifications to the original match
-			fmt.Println("No modifications to apply")
+			replacement = match
-			continue
+
+			// Count groups that were actually modified
+			modifiedGroups := 0
+			for _, capture := range captureGroups {
+				if capture.Value != capture.Updated {
+					modifiedGroups++
+				}
+			}
+			logger.Debug("%d of %d capture groups were modified", modifiedGroups, len(captureGroups))
+
+			for _, capture := range captureGroups {
+				if capture.Value == capture.Updated {
+					logger.Trace("Capture group unchanged: %s", capture.Value)
+					continue
+				}
+
+				// Log what changed with context
+				logger.Debug("Modifying group %s: %q → %q",
+					capture.Name, capture.Value, capture.Updated)
+
+				// Indices of the group are relative to content
+				// To relate them to match we have to subtract the match start index
+				// replacement = replacement[:groupStart] + newVal + replacement[groupEnd:]
+				commands = append(commands, ReplaceCommand{
+					From: capture.Range[0] - matchIndices[0],
+					To:   capture.Range[1] - matchIndices[0],
+					With: capture.Updated,
+				})
+			}
+
+			// Sort commands in reverse order for safe replacements
+			sort.Slice(commands, func(i, j int) bool {
+				return commands[i].From > commands[j].From
+			})
+			logger.Trace("Applying %d replacement commands in reverse order", len(commands))
+
+			for _, command := range commands {
+				logger.Trace("Replace pos %d-%d with %q", command.From, command.To, command.With)
+				replacement = replacement[:command.From] + command.With + replacement[command.To:]
+			}
 		}
 
-		// Apply the modifications to the original match
+		// Preview the replacement for logging
-		replacement := match
+		replacementPreview := replacement
-		for i := len(modsMap) - 1; i >= 0; i-- {
+		if len(replacement) > 50 {
-			newVal := modsMap[i]
+			replacementPreview = replacement[:47] + "..."
-			// Indices of the group are relative to content
-			// To relate them to match we have to subtract the match start index
-			groupStart := groups[i*2] - matchIndices[0]
-			groupEnd := groups[i*2+1] - matchIndices[0]
-			replacement = replacement[:groupStart] + newVal + replacement[groupEnd:]
 		}
+		logger.Debug("Replacing match %q with %q", matchPreview, replacementPreview)
 
 		modificationCount++
 		result = result[:matchIndices[0]] + replacement + result[matchIndices[1]:]
+		logger.Debug("Match #%d processed, running modification count: %d", i+1, modificationCount)
 	}
 
+	logger.Info("Regex processing complete: %d modifications from %d matches", modificationCount, len(indices))
 	return result, modificationCount, len(indices), nil
 }
+
+// The order of these replaces is important
+// This one handles !num-s inside of named capture groups
+// If it were not here our !num in a named capture group would
+// Expand to another capture group in the capture group
+// We really only want one (our named) capture group
+func ResolveRegexPlaceholders(pattern string) string {
+	// Handle special pattern modifications
+	if !strings.HasPrefix(pattern, "(?s)") {
+		pattern = "(?s)" + pattern
+		// Use fmt.Printf for test compatibility
+		fmt.Printf("Pattern modified to include (?s): %s\n", pattern)
+	}
+
+	namedGroupNum := regexp.MustCompile(`(?:(\?<[^>]+>)(!num))`)
+	pattern = namedGroupNum.ReplaceAllStringFunc(pattern, func(match string) string {
+		parts := namedGroupNum.FindStringSubmatch(match)
+		if len(parts) != 3 {
+			return match
+		}
+		replacement := `-?\d*\.?\d+`
+		return parts[1] + replacement
+	})
+	pattern = strings.ReplaceAll(pattern, "!num", `"?(-?\d*\.?\d+)"?`)
+	pattern = strings.ReplaceAll(pattern, "!any", `.*?`)
+	repPattern := regexp.MustCompile(`!rep\(([^,]+),\s*(\d+)\)`)
+	// !rep(pattern, count) repeats the pattern n times
+	// Inserting !any between each repetition
+	pattern = repPattern.ReplaceAllStringFunc(pattern, func(match string) string {
+		parts := repPattern.FindStringSubmatch(match)
+		if len(parts) != 3 {
+			return match
+		}
+		repeatedPattern := parts[1]
+		count := parts[2]
+		repetitions, _ := strconv.Atoi(count)
+		return strings.Repeat(repeatedPattern+".*?", repetitions-1) + repeatedPattern
+	})
+	return pattern
+}

processor/regex_test.go

@@ -35,10 +35,11 @@ func TestBuildLuaScript(t *testing.T) {
 		{"v1 * 2", "v1 = v1 * 2"},
 		{"v1 * v2", "v1 = v1 * v2"},
 		{"v1 / v2", "v1 = v1 / v2"},
+		{"12", "v1 = 12"},
 	}
 
 	for _, c := range cases {
-		result := BuildLuaScript(c.input)
+		result := PrependLuaAssignment(c.input)
 		if result != c.expected {
 			t.Errorf("BuildLuaScript(%q): expected %q, got %q", c.input, c.expected, result)
 		}
@@ -546,3 +547,793 @@ func TestEdgeCases(t *testing.T) {
 		})
 	}
 }
+
+func TestNamedCaptureGroups(t *testing.T) {
+	content := `<config>
+    <item>
+        <value>100</value>
+    </item>
+</config>`
+
+	expected := `<config>
+    <item>
+        <value>200</value>
+    </item>
+</config>`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(content, `(?s)<value>(?<amount>\d+)</value>`, "amount = amount * 2")
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestNamedCaptureGroupsNum(t *testing.T) {
+	content := `<config>
+    <item>
+        <value>100</value>
+    </item>
+</config>`
+
+	expected := `<config>
+    <item>
+        <value>200</value>
+    </item>
+</config>`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(content, `(?s)<value>(?<amount>!num)</value>`, "amount = amount * 2")
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestMultipleNamedCaptureGroups(t *testing.T) {
+	content := `<product>
+    <name>Widget</name>
+    <price>15.99</price>
+    <quantity>10</quantity>
+</product>`
+
+	expected := `<product>
+    <name>WIDGET</name>
+    <price>23.99</price>
+    <quantity>15</quantity>
+</product>`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`(?s)<name>(?<prodName>[^<]+)</name>.*?<price>(?<prodPrice>\d+\.\d+)</price>.*?<quantity>(?<prodQty>\d+)</quantity>`,
+		`prodName = string.upper(prodName)
+		 prodPrice = round(prodPrice + 8, 2)
+		 prodQty = prodQty + 5`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestMixedIndexedAndNamedCaptures(t *testing.T) {
+	content := `<entry>
+    <id>12345</id>
+    <data>value</data>
+</entry>`
+
+	expected := `<entry>
+    <id>24690</id>
+    <data>VALUE</data>
+</entry>`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`(?s)<id>(\d+)</id>.*?<data>(?<dataField>[^<]+)</data>`,
+		`v1 = v1 * 2
+		 dataField = string.upper(dataField)`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestComplexNestedNamedCaptures(t *testing.T) {
+	content := `<person>
+		<details>
+			<name>John Smith</name>
+			<age>32</age>
+		</details>
+		<contact>
+			<email>john@example.com</email>
+		</contact>
+	</person>`
+
+	expected := `<person>
+		<details>
+			<name>JOHN SMITH (32)</name>
+			<age>32</age>
+		</details>
+		<contact>
+			<email>john@example.com</email>
+		</contact>
+	</person>`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`(?s)<details>.*?<name>(?<fullName>[^<]+)</name>.*?<age>(?<age>\d+)</age>`,
+		`fullName = string.upper(fullName) .. " (" .. age .. ")"`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestNamedCaptureWithVariableReadback(t *testing.T) {
+	content := `<stats>
+    <health>100</health>
+    <mana>200</mana>
+</stats>`
+
+	expected := `<stats>
+    <health>150</health>
+    <mana>300</mana>
+</stats>`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`(?s)<health>(?<hp>\d+)</health>.*?<mana>(?<mp>\d+)</mana>`,
+		`hp = hp * 1.5
+		 mp = mp * 1.5`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestNamedCaptureWithSpecialCharsInName(t *testing.T) {
+	content := `<data value="42" min="10" max="100" />`
+
+	expected := `<data value="84" min="10" max="100" />`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`<data value="(?<val_1>\d+)"`,
+		`val_1 = val_1 * 2`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestEmptyNamedCapture(t *testing.T) {
+	content := `<tag attr="" />`
+
+	expected := `<tag attr="default" />`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`attr="(?<value>.*?)"`,
+		`value = value == "" and "default" or value`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestMultipleNamedCapturesInSameLine(t *testing.T) {
+	content := `<rect x="10" y="20" width="100" height="50" />`
+
+	expected := `<rect x="20" y="40" width="200" height="100" />`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`x="(?<x>\d+)" y="(?<y>\d+)" width="(?<w>\d+)" height="(?<h>\d+)"`,
+		`x = x * 2
+		 y = y * 2
+		 w = w * 2
+		 h = h * 2`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestConditionalNamedCapture(t *testing.T) {
+	content := `
+	<item status="active" count="5" />
+	<item status="inactive" count="10" />
+	`
+
+	expected := `
+	<item status="active" count="10" />
+	<item status="inactive" count="10" />
+	`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`<item status="(?<status>[^"]+)" count="(?<count>\d+)"`,
+		`count = status == "active" and count * 2 or count`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 2 {
+		t.Errorf("Expected 2 matches, got %d", matches)
+	}
+
+	if mods != 2 {
+		t.Errorf("Expected 2 modifications, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestLuaFunctionsOnNamedCaptures(t *testing.T) {
+	content := `
+	<user name="john doe" role="user" />
+	<user name="jane smith" role="admin" />
+	`
+
+	expected := `
+	<user name="John Doe" role="user" />
+	<user name="JANE SMITH" role="admin" />
+	`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`<user name="(?<name>[^"]+)" role="(?<role>[^"]+)"`,
+		`-- Capitalize first letters for regular users
+		 if role == "user" then
+		   name = name:gsub("(%w)(%w*)", function(first, rest) return first:upper()..rest end):gsub(" (%w)(%w*)", " %1%2")
+		 else
+		   -- Uppercase for admins
+		   name = string.upper(name)
+		 end`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 2 {
+		t.Errorf("Expected 2 matches, got %d", matches)
+	}
+
+	if mods != 2 {
+		t.Errorf("Expected 2 modifications, got %d", mods)
+	}
+
+	// For simpler tests, we can use this. More complex string modifications
+	// might need additional transformations before comparison
+	normalizedResult := normalizeWhitespace(result)
+	normalizedExpected := normalizeWhitespace(expected)
+	if normalizedResult != normalizedExpected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestNamedCaptureWithMath(t *testing.T) {
+	content := `
+	<item price="19.99" quantity="3" />
+	`
+
+	expected := `
+	<item price="19.99" quantity="3" total="59.97" />
+	`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`<item price="(?<price>\d+\.\d+)" quantity="(?<qty>\d+)"!any$`,
+		`-- Calculate and add total
+		 replacement = string.format('<item price="%s" quantity="%s" total="%.2f" />',
+			price, qty, price * qty)`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	result = normalizeWhitespace(result)
+	expected = normalizeWhitespace(expected)
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestNamedCaptureWithGlobals(t *testing.T) {
+	content := `<temp unit="C">25</temp>`
+
+	expected := `<temp unit="F">77</temp>`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`<temp unit="(?<unit>[CF]?)">(?<value>\d+)</temp>`,
+		`if unit == "C" then
+		  value = value * 9/5 + 32
+		  unit = "F"
+		elseif unit == "F" then
+		  value = (value - 32) * 5/9
+		  unit = "C"
+		end`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestMixedDynamicAndNamedCaptures(t *testing.T) {
+	content := `
+	<color rgb="255,0,0" name="red" />
+	<color rgb="0,255,0" name="green" />
+	`
+
+	expected := `
+	<color rgb="255,0,0" name="RED" hex="#FF0000" />
+	<color rgb="0,255,0" name="GREEN" hex="#00FF00" />
+	`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`<color rgb="(?<r>\d+),(?<g>\d+),(?<b>\d+)" name="(?<colorName>[^"]+)" />`,
+		`-- Uppercase the name
+		 colorName = string.upper(colorName)
+
+		 -- Create hex color
+		 local hex = string.format("#%02X%02X%02X", tonumber(r), tonumber(g), tonumber(b))
+
+		 -- Replace the entire match
+		 replacement = string.format('<color rgb="%s,%s,%s" name="%s" hex="%s" />',
+		                            r, g, b, colorName, hex)`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 2 {
+		t.Errorf("Expected 2 matches, got %d", matches)
+	}
+
+	if mods != 2 {
+		t.Errorf("Expected 2 modifications, got %d", mods)
+	}
+
+	result = normalizeWhitespace(result)
+	expected = normalizeWhitespace(expected)
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestNamedCapturesWithMultipleReferences(t *testing.T) {
+	content := `<text>Hello world</text>`
+
+	expected := `<text format="uppercase" length="11">HELLO WORLD</text>`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`<text>(?<content>[^<]+)</text>`,
+		`local uppercaseContent = string.upper(content)
+		 local contentLength = string.len(content)
+		 replacement = string.format('<text format="uppercase" length="%d">%s</text>',
+		                            contentLength, uppercaseContent)`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestNamedCaptureWithJsonData(t *testing.T) {
+	content := `<data>{"name":"John","age":30}</data>`
+
+	expected := `<data>{"name":"JOHN","age":30}</data>`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`<data>(?<json>\{.*?\})</data>`,
+		`-- Parse JSON (simplified, assumes valid JSON)
+		 local name = json:match('"name":"([^"]+)"')
+		 local upperName = string.upper(name)
+		 json = json:gsub('"name":"([^"]+)"', '"name":"' .. upperName .. '"')`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestNamedCaptureInXML(t *testing.T) {
+	content := `
+	<product>
+		<sku>ABC-123</sku>
+		<price currency="USD">19.99</price>
+		<stock>25</stock>
+	</product>
+	`
+
+	expected := `
+	<product>
+		<sku>ABC-123</sku>
+		<price currency="USD">23.99</price>
+		<stock>20</stock>
+	</product>
+	`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`(?s)<price currency="(?<currency>[^"]+)">(?<price>\d+\.\d+)</price>.*?<stock>(?<stock>\d+)</stock>`,
+		`-- Add 20% to price if USD
+		 if currency == "USD" then
+		   price = round(price * 1.20, 2)
+		 end
+
+		 -- Reduce stock by 5
+		 stock = stock - 5`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestComprehensiveNamedCaptures(t *testing.T) {
+	content := `
+	<products>
+		<product sku="AB-123" status="in-stock">
+			<name>Widget A</name>
+			<price currency="USD">19.99</price>
+			<quantity>15</quantity>
+		</product>
+		<product sku="CD-456" status="out-of-stock">
+			<name>Widget B</name>
+			<price currency="EUR">29.99</price>
+			<quantity>0</quantity>
+		</product>
+		<product sku="EF-789" status="in-stock">
+			<name>Widget C</name>
+			<price currency="GBP">39.99</price>
+			<quantity>5</quantity>
+		</product>
+	</products>
+	`
+
+	expected := `
+	<products>
+		<product sku="AB-123" status="in-stock" discounted="true">
+			<name>WIDGET A</name>
+			<price currency="USD">15.99</price>
+			<quantity>15</quantity>
+		</product>
+		<product sku="CD-456" status="out-of-stock" discounted="false">
+			<name>Widget B</name>
+			<price currency="EUR">29.99</price>
+			<quantity>0</quantity>
+		</product>
+		<product sku="EF-789" status="in-stock" discounted="true">
+			<name>WIDGET C</name>
+			<price currency="GBP">39.99</price>
+			<quantity>5</quantity>
+		</product>
+	</products>
+	`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`(?s)<product sku="(?<sku>[^"]+)" status="(?<status>[^"]+)"[^>]*>\s*<name>(?<product_name>[^<]+)</name>\s*<price currency="(?<currency>[^"]+)">(?<price>\d+\.\d+)</price>\s*<quantity>(?<qty>\d+)</quantity>`,
+		`-- Only process in-stock items
+		if status == "in-stock" then
+			-- Transform name to uppercase
+			product_name = string.upper(product_name)
+
+			-- Apply discount based on currency
+			local discounted = true
+			if currency == "USD" then
+				price = round(price * 0.8, 2)  -- 20% discount for USD
+			elseif currency == "GBP" then
+				price = round(price * 0.8, 2)  -- 20% discount for GBP
+				price = price + 8  -- Add shipping cost for GBP
+			else
+				discounted = false
+			end
+
+			-- Add discounted attribute
+			replacement = string.format('<product sku="%s" status="%s" discounted="%s">\n\t\t\t<name>%s</name>\n\t\t\t<price currency="%s">%.2f</price>\n\t\t\t<quantity>%s</quantity>',
+				sku, status, tostring(discounted), product_name, currency, price, qty)
+		else
+			-- Add discounted attribute for out-of-stock items (always false)
+			replacement = string.format('<product sku="%s" status="%s" discounted="false">\n\t\t\t<name>%s</name>\n\t\t\t<price currency="%s">%s</price>\n\t\t\t<quantity>%s</quantity>',
+				sku, status, product_name, currency, price, qty)
+		end`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 3 {
+		t.Errorf("Expected 3 matches, got %d", matches)
+	}
+
+	if mods != 3 {
+		t.Errorf("Expected 3 modifications, got %d", mods)
+	}
+
+	// Normalize whitespace for comparison
+	normalizedResult := normalizeWhitespace(result)
+	normalizedExpected := normalizeWhitespace(expected)
+
+	if normalizedResult != normalizedExpected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestVariousNamedCaptureFormats(t *testing.T) {
+	content := `
+	<data>
+		<entry id="1" value="100" />
+		<entry id="2" value="200" status="active" />
+		<entry id="3" value="300" status="inactive" />
+	</data>
+	`
+
+	expected := `
+	<data>
+		<entry id="ID-1" value="200" />
+		<entry id="ID-2" value="400" status="ACTIVE" />
+		<entry id="ID-3" value="300" status="inactive" />
+	</data>
+	`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`<entry id="(?<id_num>\d+)" value="(?<val>\d+)"(?: status="(?<status>[^"]*)")? />`,
+		`-- Prefix the ID with "ID-"
+		id_num = "ID-" .. id_num
+		print(id_num)
+		print(val)
+		print(status)
+
+		-- Double the value except for inactive status
+		if not status or status ~= "inactive" then
+			val = val * 2
+		end
+
+		-- Convert status to uppercase if present and active
+		if status and status == "active" then
+			status = string.upper(status)
+		end
+
+		-- Build the replacement based on whether status exists
+		if status then
+			replacement = string.format('<entry id="%s" value="%s" status="%s" />', id_num, val, status)
+		else
+			replacement = string.format('<entry id="%s" value="%s" />', id_num, val)
+		end`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 3 {
+		t.Errorf("Expected 3 matches, got %d", matches)
+	}
+
+	if mods != 3 {
+		t.Errorf("Expected 3 modifications, got %d", mods)
+	}
+
+	normalizedResult := normalizeWhitespace(result)
+	normalizedExpected := normalizeWhitespace(expected)
+
+	if normalizedResult != normalizedExpected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}
+
+func TestSimpleNamedCapture(t *testing.T) {
+	content := `<product name="Widget" price="19.99"/>`
+
+	expected := `<product name="WIDGET" price="19.99"/>`
+
+	p := &RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(
+		content,
+		`name="(?<product_name>[^"]+)"`,
+		`product_name = string.upper(product_name)`)
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != expected {
+		t.Errorf("Expected content to be:\n%s\n\nGot:\n%s", expected, result)
+	}
+}

processor/test_helper.go

@@ -0,0 +1,26 @@
+package processor
+
+import (
+	"io"
+	"modify/logger"
+	"os"
+)
+
+func init() {
+	// Only modify logger in test mode
+	// This checks if we're running under 'go test'
+	if os.Getenv("GO_TESTING") == "1" || os.Getenv("TESTING") == "1" {
+		// Initialize logger with ERROR level for tests
+		// to minimize noise in test output
+		logger.Init(logger.LevelError)
+
+		// Optionally redirect logger output to discard
+		// This prevents logger output from interfering with test output
+		disableTestLogs := os.Getenv("ENABLE_TEST_LOGS") != "1"
+		if disableTestLogs {
+			// Create a new logger that writes to nowhere
+			silentLogger := logger.New(io.Discard, "", 0)
+			logger.DefaultLogger = silentLogger
+		}
+	}
+}

processor/xml.go

@@ -2,8 +2,8 @@ package processor
 
 import (
 	"fmt"
-	"os"
+	"modify/logger"
-	"path/filepath"
+	"modify/processor/xpath"
 	"strings"
 
 	"github.com/antchfx/xmlquery"
@@ -13,205 +13,422 @@ import (
 // XMLProcessor implements the Processor interface for XML documents
 type XMLProcessor struct{}
 
-// Process implements the Processor interface for XMLProcessor
-func (p *XMLProcessor) Process(filename string, pattern string, luaExpr string) (int, int, error) {
-	// Read file content
-	fullPath := filepath.Join(".", filename)
-	content, err := os.ReadFile(fullPath)
-	if err != nil {
-		return 0, 0, fmt.Errorf("error reading file: %v", err)
-	}
-
-	fileContent := string(content)
-
-	// Process the content
-	modifiedContent, modCount, matchCount, err := p.ProcessContent(fileContent, pattern, luaExpr)
-	if err != nil {
-		return 0, 0, err
-	}
-
-	// If we made modifications, save the file
-	if modCount > 0 {
-		err = os.WriteFile(fullPath, []byte(modifiedContent), 0644)
-		if err != nil {
-			return 0, 0, fmt.Errorf("error writing file: %v", err)
-		}
-	}
-
-	return modCount, matchCount, nil
-}
-
 // ProcessContent implements the Processor interface for XMLProcessor
-func (p *XMLProcessor) ProcessContent(content string, pattern string, luaExpr string) (string, int, int, error) {
+func (p *XMLProcessor) ProcessContent(content string, path string, luaExpr string) (string, int, int, error) {
+	logger.Debug("Processing XML content with XPath: %s", path)
+
 	// Parse XML document
+	// We can't really use encoding/xml here because it requires a pre defined struct
+	// And we HAVE TO parse dynamic unknown XML
+	logger.Trace("Parsing XML document")
 	doc, err := xmlquery.Parse(strings.NewReader(content))
 	if err != nil {
+		logger.Error("Failed to parse XML: %v", err)
 		return content, 0, 0, fmt.Errorf("error parsing XML: %v", err)
 	}
 
 	// Find nodes matching the XPath pattern
-	nodes, err := xmlquery.QueryAll(doc, pattern)
+	logger.Debug("Executing XPath query: %s", path)
+	nodes, err := xpath.Get(doc, path)
 	if err != nil {
+		logger.Error("Failed to execute XPath: %v", err)
 		return content, 0, 0, fmt.Errorf("error executing XPath: %v", err)
 	}
 
 	matchCount := len(nodes)
+	logger.Debug("Found %d nodes matching XPath", matchCount)
 	if matchCount == 0 {
+		logger.Warning("No nodes matched the XPath pattern: %s", path)
 		return content, 0, 0, nil
 	}
 
-	// Initialize Lua
-	L := lua.NewState()
-	defer L.Close()
-
-	// Load math library
-	L.Push(L.GetGlobal("require"))
-	L.Push(lua.LString("math"))
-	if err := L.PCall(1, 1, nil); err != nil {
-		return content, 0, 0, fmt.Errorf("error loading Lua math library: %v", err)
-	}
-
-	// Load helper functions
-	if err := InitLuaHelpers(L); err != nil {
-		return content, 0, 0, err
-	}
-
 	// Apply modifications to each node
 	modCount := 0
-	for _, node := range nodes {
+	for i, node := range nodes {
-		// Reset Lua state for each node
+		logger.Trace("Processing node #%d: %s", i+1, node.Data)
-		L.SetGlobal("v1", lua.LNil)
-		L.SetGlobal("s1", lua.LNil)
 
-		// Get the node value
+		L, err := NewLuaState()
-		var originalValue string
-		if node.Type == xmlquery.AttributeNode {
-			originalValue = node.InnerText()
-		} else if node.Type == xmlquery.TextNode {
-			originalValue = node.Data
-		} else {
-			originalValue = node.InnerText()
-		}
-
-		// Convert to Lua variables
-		err = p.ToLua(L, originalValue)
 		if err != nil {
+			logger.Error("Failed to create Lua state: %v", err)
+			return content, 0, 0, fmt.Errorf("error creating Lua state: %v", err)
+		}
+		defer L.Close()
+
+		logger.Trace("Converting XML node to Lua")
+		err = p.ToLua(L, node)
+		if err != nil {
+			logger.Error("Failed to convert XML node to Lua: %v", err)
 			return content, modCount, matchCount, fmt.Errorf("error converting to Lua: %v", err)
 		}
 
-		// Execute Lua script
+		luaScript := BuildLuaScript(luaExpr)
-		if err := L.DoString(luaExpr); err != nil {
+		logger.Trace("Executing Lua script: %s", luaScript)
+		err = L.DoString(luaScript)
+		if err != nil {
+			logger.Error("Failed to execute Lua script: %v", err)
 			return content, modCount, matchCount, fmt.Errorf("error executing Lua: %v", err)
 		}
 
-		// Get modified value
 		result, err := p.FromLua(L)
 		if err != nil {
+			logger.Error("Failed to get result from Lua: %v", err)
 			return content, modCount, matchCount, fmt.Errorf("error getting result from Lua: %v", err)
 		}
+		logger.Trace("Lua returned result: %#v", result)
 
-		newValue, ok := result.(string)
+		modified := false
-		if !ok {
+		modified = L.GetGlobal("modified").String() == "true"
-			return content, modCount, matchCount, fmt.Errorf("expected string result from Lua, got %T", result)
+		if !modified {
-		}
+			logger.Debug("No changes made to node at path: %s", node.Data)
-
-		// Skip if no change
-		if newValue == originalValue {
 			continue
 		}
 
-		// Apply modification
+		// Apply modification based on the result
-		if node.Type == xmlquery.AttributeNode {
+		if updatedValue, ok := result.(string); ok {
-			// For attribute nodes, update the attribute value
+			// If the result is a simple string, update the node value directly
-			node.Parent.Attr = append([]xmlquery.Attr{}, node.Parent.Attr...)
+			logger.Debug("Updating node with string value: %s", updatedValue)
-			for i, attr := range node.Parent.Attr {
+			xpath.Set(doc, path, updatedValue)
-				if attr.Name.Local == node.Data {
+		} else if nodeData, ok := result.(map[string]interface{}); ok {
-					node.Parent.Attr[i].Value = newValue
+			// If the result is a map, apply more complex updates
-					break
+			logger.Debug("Updating node with complex data structure")
-				}
+			updateNodeFromMap(node, nodeData)
-			}
-		} else if node.Type == xmlquery.TextNode {
-			// For text nodes, update the text content
-			node.Data = newValue
-		} else {
-			// For element nodes, replace inner text
-			// Simple approach: set the InnerText directly if there are no child elements
-			if node.FirstChild == nil || (node.FirstChild != nil && node.FirstChild.Type == xmlquery.TextNode && node.FirstChild.NextSibling == nil) {
-				if node.FirstChild != nil {
-					node.FirstChild.Data = newValue
-				} else {
-					// Create a new text node and add it as the first child
-					textNode := &xmlquery.Node{
-						Type: xmlquery.TextNode,
-						Data: newValue,
-					}
-					node.FirstChild = textNode
-				}
-			} else {
-				// Complex case: node has mixed content or child elements
-				// Replace just the text content while preserving child elements
-				// This is a simplified approach - more complex XML may need more robust handling
-				for child := node.FirstChild; child != nil; child = child.NextSibling {
-					if child.Type == xmlquery.TextNode {
-						child.Data = newValue
-						break // Update only the first text node
-					}
-				}
-			}
 		}
 
 		modCount++
+		logger.Debug("Successfully modified node #%d", i+1)
 	}
 
+	logger.Info("XML processing complete: %d modifications from %d matches", modCount, matchCount)
+
 	// Serialize the modified XML document to string
 	if doc.FirstChild != nil && doc.FirstChild.Type == xmlquery.DeclarationNode {
 		// If we have an XML declaration, start with it
 		declaration := doc.FirstChild.OutputXML(true)
 		// Remove the firstChild (declaration) before serializing the rest of the document
 		doc.FirstChild = doc.FirstChild.NextSibling
-		return declaration + doc.OutputXML(true), modCount, matchCount, nil
+		return ConvertToNamedEntities(declaration + doc.OutputXML(true)), modCount, matchCount, nil
 	}
 
-	return doc.OutputXML(true), modCount, matchCount, nil
+	// Convert numeric entities to named entities for better readability
+	return ConvertToNamedEntities(doc.OutputXML(true)), modCount, matchCount, nil
+}
+
+func (p *XMLProcessor) ToLua(L *lua.LState, data interface{}) error {
+	table, err := p.ToLuaTable(L, data)
+	if err != nil {
+		return err
+	}
+	L.SetGlobal("v", table)
+	return nil
 }
 
 // ToLua converts XML node values to Lua variables
-func (p *XMLProcessor) ToLua(L *lua.LState, data interface{}) error {
+func (p *XMLProcessor) ToLuaTable(L *lua.LState, data interface{}) (lua.LValue, error) {
-	value, ok := data.(string)
+	// Check if data is an xmlquery.Node
+	node, ok := data.(*xmlquery.Node)
 	if !ok {
-		return fmt.Errorf("expected string value, got %T", data)
+		return nil, fmt.Errorf("expected xmlquery.Node, got %T", data)
 	}
 
-	// Set as string variable
+	// Create a simple table with essential data
-	L.SetGlobal("s1", lua.LString(value))
+	table := L.NewTable()
 
-	// Try to convert to number if possible
+	// For element nodes, just provide basic info
-	L.SetGlobal("v1", lua.LNumber(0)) // Default to 0
+	L.SetField(table, "type", lua.LString(nodeTypeToString(node.Type)))
-	if err := L.DoString(fmt.Sprintf("v1 = tonumber(%q) or 0", value)); err != nil {
+	L.SetField(table, "name", lua.LString(node.Data))
-		return fmt.Errorf("error converting value to number: %v", err)
+	L.SetField(table, "value", lua.LString(node.InnerText()))
+
+	// Add children if any
+	children := L.NewTable()
+	for child := node.FirstChild; child != nil; child = child.NextSibling {
+		childTable, err := p.ToLuaTable(L, child)
+		if err == nil {
+			children.Append(childTable)
+		}
 	}
+	L.SetField(table, "children", children)
 
-	return nil
+	attrs := L.NewTable()
+	if len(node.Attr) > 0 {
+		for _, attr := range node.Attr {
+			L.SetField(attrs, attr.Name.Local, lua.LString(attr.Value))
+		}
+	}
+	L.SetField(table, "attr", attrs)
+
+	return table, nil
 }
 
 // FromLua gets modified values from Lua
 func (p *XMLProcessor) FromLua(L *lua.LState) (interface{}, error) {
-	// Check if string variable was modified
+	luaValue := L.GetGlobal("v")
-	s1 := L.GetGlobal("s1")
+
-	if s1 != lua.LNil {
+	// Handle string values directly
-		if s1Str, ok := s1.(lua.LString); ok {
+	if luaValue.Type() == lua.LTString {
-			return string(s1Str), nil
+		return luaValue.String(), nil
-		}
 	}
 
-	// Check if numeric variable was modified
+	// Handle tables (for attributes and more complex updates)
-	v1 := L.GetGlobal("v1")
+	if luaValue.Type() == lua.LTTable {
-	if v1 != lua.LNil {
+		return luaTableToMap(L, luaValue.(*lua.LTable)), nil
-		if v1Num, ok := v1.(lua.LNumber); ok {
-			return fmt.Sprintf("%v", v1Num), nil
-		}
 	}
 
-	// Default return empty string
+	return luaValue.String(), nil
-	return "", nil
+}
+
+// Simple helper to convert a Lua table to a Go map
+func luaTableToMap(L *lua.LState, table *lua.LTable) map[string]interface{} {
+	result := make(map[string]interface{})
+
+	table.ForEach(func(k, v lua.LValue) {
+		if k.Type() == lua.LTString {
+			key := k.String()
+
+			if v.Type() == lua.LTTable {
+				result[key] = luaTableToMap(L, v.(*lua.LTable))
+			} else {
+				result[key] = v.String()
+			}
+		}
+	})
+
+	return result
+}
+
+// Simple helper to convert node type to string
+func nodeTypeToString(nodeType xmlquery.NodeType) string {
+	switch nodeType {
+	case xmlquery.ElementNode:
+		return "element"
+	case xmlquery.TextNode:
+		return "text"
+	case xmlquery.AttributeNode:
+		return "attribute"
+	default:
+		return "other"
+	}
+}
+
+// Helper function to update an XML node from a map
+func updateNodeFromMap(node *xmlquery.Node, data map[string]interface{}) {
+	// Update node value if present
+	if value, ok := data["value"]; ok {
+		if strValue, ok := value.(string); ok {
+			// For element nodes, replace text content
+			if node.Type == xmlquery.ElementNode {
+				// Find the first text child if it exists
+				var textNode *xmlquery.Node
+				for child := node.FirstChild; child != nil; child = child.NextSibling {
+					if child.Type == xmlquery.TextNode {
+						textNode = child
+						break
+					}
+				}
+
+				if textNode != nil {
+					// Update existing text node
+					textNode.Data = strValue
+				} else {
+					// Create new text node
+					newText := &xmlquery.Node{
+						Type:   xmlquery.TextNode,
+						Data:   strValue,
+						Parent: node,
+					}
+
+					// Insert at beginning of children
+					if node.FirstChild != nil {
+						newText.NextSibling = node.FirstChild
+						node.FirstChild.PrevSibling = newText
+						node.FirstChild = newText
+					} else {
+						node.FirstChild = newText
+						node.LastChild = newText
+					}
+				}
+			} else if node.Type == xmlquery.TextNode {
+				// Directly update text node
+				node.Data = strValue
+			} else if node.Type == xmlquery.AttributeNode {
+				// Update attribute value
+				if node.Parent != nil {
+					for i, attr := range node.Parent.Attr {
+						if attr.Name.Local == node.Data {
+							node.Parent.Attr[i].Value = strValue
+							break
+						}
+					}
+				}
+			}
+		}
+	}
+
+	// Update attributes if present
+	if attrs, ok := data["attr"].(map[string]interface{}); ok && node.Type == xmlquery.ElementNode {
+		for name, value := range attrs {
+			if strValue, ok := value.(string); ok {
+				// Look for existing attribute
+				found := false
+				for i, attr := range node.Attr {
+					if attr.Name.Local == name {
+						node.Attr[i].Value = strValue
+						found = true
+						break
+					}
+				}
+
+				// Add new attribute if not found
+				if !found {
+					node.Attr = append(node.Attr, xmlquery.Attr{
+						Name: struct {
+							Space, Local string
+						}{Local: name},
+						Value: strValue,
+					})
+				}
+			}
+		}
+	}
+}
+
+// Helper function to get a string representation of node type
+func nodeTypeName(nodeType xmlquery.NodeType) string {
+	switch nodeType {
+	case xmlquery.ElementNode:
+		return "element"
+	case xmlquery.TextNode:
+		return "text"
+	case xmlquery.AttributeNode:
+		return "attribute"
+	case xmlquery.CommentNode:
+		return "comment"
+	case xmlquery.DeclarationNode:
+		return "declaration"
+	default:
+		return "unknown"
+	}
+}
+
+// ConvertToNamedEntities replaces numeric XML entities with their named counterparts
+func ConvertToNamedEntities(xml string) string {
+	// Basic XML entities
+	replacements := map[string]string{
+		// Basic XML entities
+		""": """, // double quote
+		"'": "'", // single quote
+		"<": "<",   // less than
+		">": ">",   // greater than
+		"&": "&",  // ampersand
+
+		// Common symbols
+		" ":  " ",   // non-breaking space
+		"©":  "©",   // copyright
+		"®":  "®",    // registered trademark
+		"€": "€",   // euro
+		"£":  "£",  // pound
+		"¥":  "¥",    // yen
+		"¢":  "¢",   // cent
+		"§":  "§",   // section
+		"™": "™",  // trademark
+		"♠": "♠", // spade
+		"♣": "♣",  // club
+		"♥": "♥", // heart
+		"♦": "♦",  // diamond
+
+		// Special characters
+		"¡":  "¡",  // inverted exclamation
+		"¿":  "¿", // inverted question
+		"«":  "«",  // left angle quotes
+		"»":  "»",  // right angle quotes
+		"·":  "·", // middle dot
+		"•": "•",   // bullet
+		"…": "…", // horizontal ellipsis
+		"′": "′",  // prime
+		"″": "″",  // double prime
+		"‾": "‾",  // overline
+		"⁄": "⁄",  // fraction slash
+
+		// Math symbols
+		"±":  "±", // plus-minus
+		"×":  "×",  // multiplication
+		"÷":  "÷", // division
+		"∞": "∞",  // infinity
+		"≈": "≈",  // almost equal
+		"≠": "≠",     // not equal
+		"≤": "≤",     // less than or equal
+		"≥": "≥",     // greater than or equal
+		"∑": "∑",    // summation
+		"√": "√",  // square root
+		"∫": "∫",    // integral
+
+		// Accented characters
+		"À": "À", // A grave
+		"Á": "Á", // A acute
+		"Â": "Â",  // A circumflex
+		"Ã": "Ã", // A tilde
+		"Ä": "Ä",   // A umlaut
+		"Å": "Å",  // A ring
+		"Æ": "Æ",  // AE ligature
+		"Ç": "Ç", // C cedilla
+		"È": "È", // E grave
+		"É": "É", // E acute
+		"Ê": "Ê",  // E circumflex
+		"Ë": "Ë",   // E umlaut
+		"Ì": "Ì", // I grave
+		"Í": "Í", // I acute
+		"Î": "Î",  // I circumflex
+		"Ï": "Ï",   // I umlaut
+		"Ð": "Ð",    // Eth
+		"Ñ": "Ñ", // N tilde
+		"Ò": "Ò", // O grave
+		"Ó": "Ó", // O acute
+		"Ô": "Ô",  // O circumflex
+		"Õ": "Õ", // O tilde
+		"Ö": "Ö",   // O umlaut
+		"Ø": "Ø", // O slash
+		"Ù": "Ù", // U grave
+		"Ú": "Ú", // U acute
+		"Û": "Û",  // U circumflex
+		"Ü": "Ü",   // U umlaut
+		"Ý": "Ý", // Y acute
+		"Þ": "Þ",  // Thorn
+		"ß": "ß",  // Sharp s
+		"à": "à", // a grave
+		"á": "á", // a acute
+		"â": "â",  // a circumflex
+		"ã": "ã", // a tilde
+		"ä": "ä",   // a umlaut
+		"å": "å",  // a ring
+		"æ": "æ",  // ae ligature
+		"ç": "ç", // c cedilla
+		"è": "è", // e grave
+		"é": "é", // e acute
+		"ê": "ê",  // e circumflex
+		"ë": "ë",   // e umlaut
+		"ì": "ì", // i grave
+		"í": "í", // i acute
+		"î": "î",  // i circumflex
+		"ï": "ï",   // i umlaut
+		"ð": "ð",    // eth
+		"ñ": "ñ", // n tilde
+		"ò": "ò", // o grave
+		"ó": "ó", // o acute
+		"ô": "ô",  // o circumflex
+		"õ": "õ", // o tilde
+		"ö": "ö",   // o umlaut
+		"ø": "ø", // o slash
+		"ù": "ù", // u grave
+		"ú": "ú", // u acute
+		"û": "û",  // u circumflex
+		"ü": "ü",   // u umlaut
+		"ý": "ý", // y acute
+		"þ": "þ",  // thorn
+		"ÿ": "ÿ",   // y umlaut
+	}
+
+	result := xml
+	for numeric, named := range replacements {
+		result = strings.ReplaceAll(result, numeric, named)
+	}
+	return result
 }

processor/xpath/parser_manual_test.go

@@ -0,0 +1,4 @@
+// The package is now using github.com/antchfx/xmlquery for XPath parsing.
+// The parsing functionality tests have been removed since we're now
+// delegating XPath parsing to the xmlquery library.
+package xpath

processor/xpath/parser_test.go

@@ -0,0 +1,4 @@
+// The package is now using github.com/antchfx/xmlquery for XPath parsing.
+// The parsing functionality tests have been removed since we're now
+// delegating XPath parsing to the xmlquery library.
+package xpath

processor/xpath/xpath.go

@@ -0,0 +1,133 @@
+package xpath
+
+import (
+	"errors"
+	"fmt"
+
+	"github.com/antchfx/xmlquery"
+)
+
+// Get retrieves nodes from XML data using an XPath expression
+func Get(node *xmlquery.Node, path string) ([]*xmlquery.Node, error) {
+	if node == nil {
+		return nil, errors.New("nil node provided")
+	}
+
+	// Execute xpath query directly
+	nodes, err := xmlquery.QueryAll(node, path)
+	if err != nil {
+		return nil, fmt.Errorf("failed to execute XPath query: %v", err)
+	}
+
+	return nodes, nil
+}
+
+// Set updates a single node in the XML data using an XPath expression
+func Set(node *xmlquery.Node, path string, value interface{}) error {
+	if node == nil {
+		return errors.New("nil node provided")
+	}
+
+	// Find the node to update
+	nodes, err := xmlquery.QueryAll(node, path)
+	if err != nil {
+		return fmt.Errorf("failed to execute XPath query: %v", err)
+	}
+
+	if len(nodes) == 0 {
+		return fmt.Errorf("no nodes found for path: %s", path)
+	}
+
+	// Update the first matching node
+	updateNodeValue(nodes[0], value)
+
+	return nil
+}
+
+// SetAll updates all nodes that match the XPath expression
+func SetAll(node *xmlquery.Node, path string, value interface{}) error {
+	if node == nil {
+		return errors.New("nil node provided")
+	}
+
+	// Find all nodes to update
+	nodes, err := xmlquery.QueryAll(node, path)
+	if err != nil {
+		return fmt.Errorf("failed to execute XPath query: %v", err)
+	}
+
+	if len(nodes) == 0 {
+		return fmt.Errorf("no nodes found for path: %s", path)
+	}
+
+	// Update all matching nodes
+	for _, matchNode := range nodes {
+		updateNodeValue(matchNode, value)
+	}
+
+	return nil
+}
+
+// Helper function to update a node's value
+func updateNodeValue(node *xmlquery.Node, value interface{}) {
+	strValue := fmt.Sprintf("%v", value)
+
+	// Handle different node types
+	switch node.Type {
+	case xmlquery.AttributeNode:
+		// For attribute nodes, update the attribute value
+		parent := node.Parent
+		if parent != nil {
+			for i, attr := range parent.Attr {
+				if attr.Name.Local == node.Data {
+					parent.Attr[i].Value = strValue
+					break
+				}
+			}
+		}
+	case xmlquery.TextNode:
+		// For text nodes, update the text content
+		node.Data = strValue
+	case xmlquery.ElementNode:
+		// For element nodes, clear existing text children and add a new text node
+		// First, remove all existing text children
+		var nonTextChildren []*xmlquery.Node
+		for child := node.FirstChild; child != nil; child = child.NextSibling {
+			if child.Type != xmlquery.TextNode {
+				nonTextChildren = append(nonTextChildren, child)
+			}
+		}
+
+		// Clear all children
+		node.FirstChild = nil
+		node.LastChild = nil
+
+		// Add a new text node
+		textNode := &xmlquery.Node{
+			Type:   xmlquery.TextNode,
+			Data:   strValue,
+			Parent: node,
+		}
+
+		// Set the text node as the first child
+		node.FirstChild = textNode
+		node.LastChild = textNode
+
+		// Add back non-text children
+		for _, child := range nonTextChildren {
+			child.Parent = node
+
+			// If this is the first child being added back
+			if node.FirstChild == textNode && node.LastChild == textNode {
+				node.FirstChild.NextSibling = child
+				child.PrevSibling = node.FirstChild
+				node.LastChild = child
+			} else {
+				// Add to the end of the chain
+				node.LastChild.NextSibling = child
+				child.PrevSibling = node.LastChild
+				node.LastChild = child
+			}
+		}
+	}
+}

processor/xpath/xpath_test.go

@@ -0,0 +1,474 @@
+package xpath
+
+import (
+	"strings"
+	"testing"
+
+	"github.com/antchfx/xmlquery"
+)
+
+// Parse test XML data once at the beginning for use in multiple tests
+func parseTestXML(t *testing.T, xmlData string) *xmlquery.Node {
+	doc, err := xmlquery.Parse(strings.NewReader(xmlData))
+	if err != nil {
+		t.Fatalf("Failed to parse test XML: %v", err)
+	}
+	return doc
+}
+
+// XML test data as a string for our tests
+var testXML = `
+<store>
+  <book category="fiction">
+    <title lang="en">The Fellowship of the Ring</title>
+    <author>J.R.R. Tolkien</author>
+    <year>1954</year>
+    <price>22.99</price>
+  </book>
+  <book category="fiction">
+    <title lang="en">The Two Towers</title>
+    <author>J.R.R. Tolkien</author>
+    <year>1954</year>
+    <price>23.45</price>
+  </book>
+  <book category="technical">
+    <title lang="en">Learning XML</title>
+    <author>Erik T. Ray</author>
+    <year>2003</year>
+    <price>39.95</price>
+  </book>
+  <bicycle>
+    <color>red</color>
+    <price>199.95</price>
+  </bicycle>
+</store>
+`
+
+func TestEvaluator(t *testing.T) {
+	// Parse the test XML data once for all test cases
+	doc := parseTestXML(t, testXML)
+
+	tests := []struct {
+		name  string
+		path  string
+		error bool
+	}{
+		{
+			name: "simple_element_access",
+			path: "/store/bicycle/color",
+		},
+		{
+			name: "recursive_element_access",
+			path: "//price",
+		},
+		{
+			name: "wildcard_element_access",
+			path: "/store/book/*",
+		},
+		{
+			name: "attribute_exists_predicate",
+			path: "//title[@lang]",
+		},
+		{
+			name: "attribute_equals_predicate",
+			path: "//title[@lang='en']",
+		},
+		{
+			name:  "value_comparison_predicate",
+			path:  "/store/book[price>35.00]/title",
+			error: true,
+		},
+		{
+			name:  "last_predicate",
+			path:  "/store/book[last()]/title",
+			error: true,
+		},
+		{
+			name:  "last_minus_predicate",
+			path:  "/store/book[last()-1]/title",
+			error: true,
+		},
+		{
+			name:  "position_predicate",
+			path:  "/store/book[position()<3]/title",
+			error: true,
+		},
+		{
+			name:  "invalid_index",
+			path:  "/store/book[10]/title",
+			error: true,
+		},
+		{
+			name: "nonexistent_element",
+			path: "/store/nonexistent",
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			result, err := Get(doc, tt.path)
+
+			// Handle expected errors
+			if tt.error {
+				if err == nil && len(result) == 0 {
+					// If we expected an error but got empty results instead, that's okay
+					return
+				}
+				if err != nil {
+					// If we got an error as expected, that's okay
+					return
+				}
+			} else if err != nil {
+				// If we didn't expect an error but got one, that's a test failure
+				t.Errorf("Get(%q) returned unexpected error: %v", tt.path, err)
+				return
+			}
+
+			// Special cases where we don't care about exact matches
+			switch tt.name {
+			case "wildcard_element_access":
+				// Just check that we got some elements
+				if len(result) == 0 {
+					t.Errorf("Expected multiple elements for wildcard, got none")
+				}
+				return
+			case "attribute_exists_predicate", "attribute_equals_predicate":
+				// Just check that we got some titles
+				if len(result) == 0 {
+					t.Errorf("Expected titles with lang attribute, got none")
+				}
+				// Ensure all are title elements
+				for _, node := range result {
+					if node.Data != "title" {
+						t.Errorf("Expected title elements, got: %s", node.Data)
+					}
+				}
+				return
+			case "nonexistent_element":
+				// Just check that we got empty results
+				if len(result) != 0 {
+					t.Errorf("Expected empty results for nonexistent element, got %d items", len(result))
+				}
+				return
+			}
+
+			// For other cases, just verify we got results
+			if len(result) == 0 {
+				t.Errorf("Expected results for path %s, got none", tt.path)
+			}
+		})
+	}
+}
+
+func TestEdgeCases(t *testing.T) {
+	t.Run("nil_node", func(t *testing.T) {
+		result, err := Get(nil, "/store/book")
+		if err == nil {
+			t.Errorf("Expected error for nil node")
+			return
+		}
+		if len(result) > 0 {
+			t.Errorf("Expected empty result, got %v", result)
+		}
+	})
+
+	t.Run("invalid_xml", func(t *testing.T) {
+		invalidXML, err := xmlquery.Parse(strings.NewReader("<invalid>xml"))
+		if err != nil {
+			// If parsing fails, that's expected
+			return
+		}
+
+		_, err = Get(invalidXML, "/store")
+		if err == nil {
+			t.Error("Expected error for invalid XML structure")
+		}
+	})
+
+	// For these tests with the simple XML, we expect just one result
+	simpleXML := `<root><book><title lang="en">Test</title></book></root>`
+	doc := parseTestXML(t, simpleXML)
+
+	t.Run("current_node", func(t *testing.T) {
+		result, err := Get(doc, "/root/book/.")
+		if err != nil {
+			t.Errorf("Get() returned error: %v", err)
+			return
+		}
+		if len(result) > 1 {
+			t.Errorf("Expected at most 1 result, got %d", len(result))
+		}
+		if len(result) > 0 {
+			// Verify it's the book node
+			if result[0].Data != "book" {
+				t.Errorf("Expected book node, got %v", result[0].Data)
+			}
+		}
+	})
+
+	t.Run("attributes", func(t *testing.T) {
+		result, err := Get(doc, "/root/book/title/@lang")
+		if err != nil {
+			t.Errorf("Get() returned error: %v", err)
+			return
+		}
+		if len(result) != 1 || result[0].InnerText() != "en" {
+			t.Errorf("Expected 'en', got %v", result[0].InnerText())
+		}
+	})
+}
+
+func TestGetWithPaths(t *testing.T) {
+	// Use a simplified, well-formed XML document
+	simpleXML := `<store>
+  <book category="fiction">
+    <title lang="en">The Book Title</title>
+    <author>Author Name</author>
+    <price>19.99</price>
+  </book>
+  <bicycle>
+    <color>red</color>
+    <price>199.95</price>
+  </bicycle>
+</store>`
+
+	// Parse the XML for testing
+	doc := parseTestXML(t, simpleXML)
+
+	// Debug: Print the test XML
+	t.Logf("Test XML:\n%s", simpleXML)
+
+	tests := []struct {
+		name          string
+		path          string
+		expectedValue string
+	}{
+		{
+			name:          "simple_element_access",
+			path:          "/store/bicycle/color",
+			expectedValue: "red",
+		},
+		{
+			name:          "attribute_access",
+			path:          "/store/book/title/@lang",
+			expectedValue: "en",
+		},
+		{
+			name:          "recursive_with_attribute",
+			path:          "//title[@lang='en']",
+			expectedValue: "The Book Title",
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			// Debug: Print the path we're looking for
+			t.Logf("Looking for path: %s", tt.path)
+
+			result, err := Get(doc, tt.path)
+			if err != nil {
+				t.Errorf("Get(%q) returned error: %v", tt.path, err)
+				return
+			}
+
+			// Debug: Print the results
+			t.Logf("Got %d results", len(result))
+			for i, r := range result {
+				t.Logf("Result %d: Node=%s, Value=%v", i, r.Data, r.InnerText())
+			}
+
+			// Check that we got results
+			if len(result) == 0 {
+				t.Errorf("Get(%q) returned no results", tt.path)
+				return
+			}
+
+			// For attribute access test, do more specific checks
+			if tt.name == "attribute_access" {
+				// Check the first result's value matches expected
+				if result[0].InnerText() != tt.expectedValue {
+					t.Errorf("Attribute value: got %v, expected %s", result[0].InnerText(), tt.expectedValue)
+				}
+			}
+
+			// For simple element access, check the text content
+			if tt.name == "simple_element_access" {
+				if text := result[0].InnerText(); text != tt.expectedValue {
+					t.Errorf("Element text: got %s, expected %s", text, tt.expectedValue)
+				}
+			}
+
+			// For recursive with attribute test, check title elements with lang="en"
+			if tt.name == "recursive_with_attribute" {
+				for _, node := range result {
+					// Check the node is a title
+					if node.Data != "title" {
+						t.Errorf("Expected title element, got %s", node.Data)
+					}
+
+					// Check text content
+					if text := node.InnerText(); text != tt.expectedValue {
+						t.Errorf("Text content: got %s, expected %s", text, tt.expectedValue)
+					}
+
+					// Check attributes - find the lang attribute
+					hasLang := false
+					for _, attr := range node.Attr {
+						if attr.Name.Local == "lang" && attr.Value == "en" {
+							hasLang = true
+							break
+						}
+					}
+					if !hasLang {
+						t.Errorf("Expected lang=\"en\" attribute, but it was not found")
+					}
+				}
+			}
+		})
+	}
+}
+
+func TestSet(t *testing.T) {
+	t.Run("simple element", func(t *testing.T) {
+		xmlData := `<root><name>John</name></root>`
+		doc := parseTestXML(t, xmlData)
+
+		err := Set(doc, "/root/name", "Jane")
+		if err != nil {
+			t.Errorf("Set() returned error: %v", err)
+			return
+		}
+
+		// Verify the change
+		result, err := Get(doc, "/root/name")
+		if err != nil {
+			t.Errorf("Get() returned error: %v", err)
+			return
+		}
+		if len(result) != 1 {
+			t.Errorf("Expected 1 result, got %d", len(result))
+			return
+		}
+
+		// Check text content
+		if text := result[0].InnerText(); text != "Jane" {
+			t.Errorf("Expected text 'Jane', got '%s'", text)
+		}
+	})
+
+	t.Run("attribute", func(t *testing.T) {
+		xmlData := `<root><element id="123"></element></root>`
+		doc := parseTestXML(t, xmlData)
+
+		err := Set(doc, "/root/element/@id", "456")
+		if err != nil {
+			t.Errorf("Set() returned error: %v", err)
+			return
+		}
+
+		// Verify the change
+		result, err := Get(doc, "/root/element/@id")
+		if err != nil {
+			t.Errorf("Get() returned error: %v", err)
+			return
+		}
+		if len(result) != 1 {
+			t.Errorf("Expected 1 result, got %d", len(result))
+			return
+		}
+
+		// For attributes, check the inner text
+		if text := result[0].InnerText(); text != "456" {
+			t.Errorf("Expected attribute value '456', got '%s'", text)
+		}
+	})
+
+	t.Run("indexed element", func(t *testing.T) {
+		xmlData := `<root><items><item>first</item><item>second</item></items></root>`
+		doc := parseTestXML(t, xmlData)
+
+		err := Set(doc, "/root/items/item[1]", "changed")
+		if err != nil {
+			t.Errorf("Set() returned error: %v", err)
+			return
+		}
+
+		// Verify the change using XPath that specifically targets the first item
+		result, err := Get(doc, "/root/items/item[1]")
+		if err != nil {
+			t.Errorf("Get() returned error: %v", err)
+			return
+		}
+
+		// Check if we have results
+		if len(result) == 0 {
+			t.Errorf("Expected at least one result for /root/items/item[1]")
+			return
+		}
+
+		// Check text content
+		if text := result[0].InnerText(); text != "changed" {
+			t.Errorf("Expected text 'changed', got '%s'", text)
+		}
+	})
+}
+
+func TestSetAll(t *testing.T) {
+	t.Run("multiple elements", func(t *testing.T) {
+		xmlData := `<root><items><item>first</item><item>second</item></items></root>`
+		doc := parseTestXML(t, xmlData)
+
+		err := SetAll(doc, "//item", "changed")
+		if err != nil {
+			t.Errorf("SetAll() returned error: %v", err)
+			return
+		}
+
+		// Verify all items are changed
+		result, err := Get(doc, "//item")
+		if err != nil {
+			t.Errorf("Get() returned error: %v", err)
+			return
+		}
+		if len(result) != 2 {
+			t.Errorf("Expected 2 results, got %d", len(result))
+			return
+		}
+
+		// Check each node
+		for i, node := range result {
+			if text := node.InnerText(); text != "changed" {
+				t.Errorf("Item %d: expected text 'changed', got '%s'", i, text)
+			}
+		}
+	})
+
+	t.Run("attributes", func(t *testing.T) {
+		xmlData := `<root><item id="1"/><item id="2"/></root>`
+		doc := parseTestXML(t, xmlData)
+
+		err := SetAll(doc, "//item/@id", "new")
+		if err != nil {
+			t.Errorf("SetAll() returned error: %v", err)
+			return
+		}
+
+		// Verify all attributes are changed
+		result, err := Get(doc, "//item/@id")
+		if err != nil {
+			t.Errorf("Get() returned error: %v", err)
+			return
+		}
+		if len(result) != 2 {
+			t.Errorf("Expected 2 results, got %d", len(result))
+			return
+		}
+
+		// For attributes, check inner text
+		for i, node := range result {
+			if text := node.InnerText(); text != "new" {
+				t.Errorf("Attribute %d: expected value 'new', got '%s'", i, text)
+			}
+		}
+	})
+}

regression/regression_test.go

@@ -0,0 +1,160 @@
+package regression
+
+import (
+	"modify/processor"
+	"testing"
+)
+
+func TestTalentsMechanicOutOfRange(t *testing.T) {
+	given := `<Talent identifier="quickfixer">
+    <Icon texture="Content/UI/TalentsIcons2.png" sheetindex="5,2" sheetelementsize="128,128"/>
+    <Description tag="talentdescription.quickfixer">
+      <Replace tag="[amount]" value="20" color="gui.green"/>
+      <Replace tag="[duration]" value="10" color="gui.green"/>
+    </Description>
+    <Description tag="talentdescription.repairmechanicaldevicestwiceasfast"/>
+    <AbilityGroupEffect abilityeffecttype="None">
+      <Abilities>
+        <CharacterAbilityGiveStat stattype="MechanicalRepairSpeed" value="1"/>
+      </Abilities>
+    </AbilityGroupEffect>
+    <AbilityGroupEffect abilityeffecttype="OnRepairComplete">
+      <Conditions>
+        <AbilityConditionItem tags="fabricator,door,engine,oxygengenerator,pump,turretammosource,deconstructor,medicalfabricator,ductblock"/>
+      </Conditions>
+      <Abilities>
+        <CharacterAbilityApplyStatusEffects>
+          <StatusEffects>
+            <StatusEffect type="OnAbility" target="Character" disabledeltatime="true">
+              <Affliction identifier="quickfixer" amount="10.0"/>
+            </StatusEffect>
+          </StatusEffects>
+        </CharacterAbilityApplyStatusEffects>
+      </Abilities>
+    </AbilityGroupEffect>
+  </Talent>`
+
+	actual := `<Talent identifier="quickfixer">
+    <Icon texture="Content/UI/TalentsIcons2.png" sheetindex="5,2" sheetelementsize="128,128"/>
+    <Description tag="talentdescription.quickfixer">
+      <Replace tag="[amount]" value="30" color="gui.green"/>
+      <Replace tag="[duration]" value="20" color="gui.green"/>
+    </Description>
+    <Description tag="talentdescription.repairmechanicaldevicestwiceasfast"/>
+    <AbilityGroupEffect abilityeffecttype="None">
+      <Abilities>
+        <CharacterAbilityGiveStat stattype="MechanicalRepairSpeed" value="2"/>
+      </Abilities>
+    </AbilityGroupEffect>
+    <AbilityGroupEffect abilityeffecttype="OnRepairComplete">
+      <Conditions>
+        <AbilityConditionItem tags="fabricator,door,engine,oxygengenerator,pump,turretammosource,deconstructor,medicalfabricator,ductblock"/>
+      </Conditions>
+      <Abilities>
+        <CharacterAbilityApplyStatusEffects>
+          <StatusEffects>
+            <StatusEffect type="OnAbility" target="Character" disabledeltatime="true">
+              <Affliction identifier="quickfixer" amount="20"/>
+            </StatusEffect>
+          </StatusEffects>
+        </CharacterAbilityApplyStatusEffects>
+      </Abilities>
+    </AbilityGroupEffect>
+  </Talent>`
+
+	p := &processor.RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(given, `<Talent identifier="quickfixer">!anyvalue="(?<movementspeed>!num)"!anyvalue="(?<duration>!num)"!anyvalue="(?<repairspeed>!num)"!anyamount="(?<durationv>!num)"`, "movementspeed=round(movementspeed*1.5, 2) duration=round(duration*2, 2) repairspeed=round(repairspeed*2, 2) durationv=duration")
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != actual {
+		t.Errorf("expected %s, got %s", actual, result)
+	}
+}
+
+func TestIndexExplosions(t *testing.T) {
+	given := `<Talent identifier="quickfixer">
+    <Icon texture="Content/UI/TalentsIcons2.png" sheetindex="5,2" sheetelementsize="128,128"/>
+    <Description tag="talentdescription.quickfixer">
+      <Replace tag="[amount]" value="20" color="gui.green"/>
+      <Replace tag="[duration]" value="10" color="gui.green"/>
+    </Description>
+    <Description tag="talentdescription.repairmechanicaldevicestwiceasfast"/>
+    <AbilityGroupEffect abilityeffecttype="None">
+      <Abilities>
+        <CharacterAbilityGiveStat stattype="MechanicalRepairSpeed" value="1"/>
+      </Abilities>
+    </AbilityGroupEffect>
+    <AbilityGroupEffect abilityeffecttype="OnRepairComplete">
+      <Conditions>
+        <AbilityConditionItem tags="fabricator,door,engine,oxygengenerator,pump,turretammosource,deconstructor,medicalfabricator,ductblock"/>
+      </Conditions>
+      <Abilities>
+        <CharacterAbilityApplyStatusEffects>
+          <StatusEffects>
+            <StatusEffect type="OnAbility" target="Character" disabledeltatime="true">
+              <Affliction identifier="quickfixer" amount="10.0"/>
+            </StatusEffect>
+          </StatusEffects>
+        </CharacterAbilityApplyStatusEffects>
+      </Abilities>
+    </AbilityGroupEffect>
+  </Talent>`
+
+	actual := `<Talent identifier="quickfixer">
+    <Icon texture="Content/UI/TalentsIcons2.png" sheetindex="5,2" sheetelementsize="128,128"/>
+    <Description tag="talentdescription.quickfixer">
+      <Replace tag="[amount]" value="30" color="gui.green"/>
+      <Replace tag="[duration]" value="20" color="gui.green"/>
+    </Description>
+    <Description tag="talentdescription.repairmechanicaldevicestwiceasfast"/>
+    <AbilityGroupEffect abilityeffecttype="None">
+      <Abilities>
+        <CharacterAbilityGiveStat stattype="MechanicalRepairSpeed" value="2"/>
+      </Abilities>
+    </AbilityGroupEffect>
+    <AbilityGroupEffect abilityeffecttype="OnRepairComplete">
+      <Conditions>
+        <AbilityConditionItem tags="fabricator,door,engine,oxygengenerator,pump,turretammosource,deconstructor,medicalfabricator,ductblock"/>
+      </Conditions>
+      <Abilities>
+        <CharacterAbilityApplyStatusEffects>
+          <StatusEffects>
+            <StatusEffect type="OnAbility" target="Character" disabledeltatime="true">
+              <Affliction identifier="quickfixer" amount="20"/>
+            </StatusEffect>
+          </StatusEffects>
+        </CharacterAbilityApplyStatusEffects>
+      </Abilities>
+    </AbilityGroupEffect>
+  </Talent>`
+
+	p := &processor.RegexProcessor{}
+	result, mods, matches, err := p.ProcessContent(given, `<Talent identifier="quickfixer">!anyvalue="(?<movementspeed>!num)"!anyvalue="(?<duration>!num)"!anyvalue="(?<repairspeed>!num)"!anyamount="(?<durationv>!num)"`, "movementspeed=round(movementspeed*1.5, 2) duration=round(duration*2, 2) repairspeed=round(repairspeed*2, 2) durationv=duration")
+
+	if err != nil {
+		t.Fatalf("Error processing content: %v", err)
+	}
+
+	if matches != 1 {
+		t.Errorf("Expected 1 match, got %d", matches)
+	}
+
+	if mods != 1 {
+		t.Errorf("Expected 1 modification, got %d", mods)
+	}
+
+	if result != actual {
+		t.Errorf("expected %s, got %s", actual, result)
+	}
+}

release.sh

@@ -0,0 +1,49 @@
+#!/bin/bash
+
+echo "Figuring out the tag..."
+TAG=$(git describe --tags --exact-match 2>/dev/null || echo "")
+if [ -z "$TAG" ]; then
+  # Get the latest tag
+  LATEST_TAG=$(git describe --tags $(git rev-list --tags --max-count=1))
+  # Increment the patch version
+  IFS='.' read -r -a VERSION_PARTS <<< "$LATEST_TAG"
+  VERSION_PARTS[2]=$((VERSION_PARTS[2]+1))
+  TAG="${VERSION_PARTS[0]}.${VERSION_PARTS[1]}.${VERSION_PARTS[2]}"
+  # Create a new tag
+  git tag $TAG
+  git push origin $TAG
+fi
+echo "Tag: $TAG"
+
+echo "Building the thing..."
+go build -o chef.exe .
+go install .
+
+echo "Creating a release..."
+TOKEN="$GITEA_API_KEY"
+GITEA="https://git.site.quack-lab.dev"
+REPO="dave/BigChef"
+# Create a release
+RELEASE_RESPONSE=$(curl -s -X POST \
+  -H "Authorization: token $TOKEN" \
+  -H "Accept: application/json" \
+  -H "Content-Type: application/json" \
+  -d '{
+    "tag_name": "'"$TAG"'",
+    "name": "'"$TAG"'",
+    "draft": false,
+    "prerelease": false
+  }' \
+  $GITEA/api/v1/repos/$REPO/releases)
+
+# Extract the release ID
+echo $RELEASE_RESPONSE
+RELEASE_ID=$(echo $RELEASE_RESPONSE | awk -F'"id":' '{print $2+0; exit}')
+echo "Release ID: $RELEASE_ID"
+
+echo "Uploading the things..."
+curl -X POST \
+  -H "Authorization: token $TOKEN" \
+  -F "attachment=@chef.exe" \
+  "$GITEA/api/v1/repos/$REPO/releases/${RELEASE_ID}/assets?name=chef.exe"
+rm chef.exe