Matches using map keys containing a "-" do not work

[mschexnaydre]

A match statement comparing a map[string]string key with an embedded dash/hyphen character "-" such as the example below never matches, even though the entry is present in the map. Other characters such as "_" work fine so there seems to be some sort of bug with the hyphen processing.

MyMap.some-key = foobar

[mschexnaydre]

Modified unit test shows the issue, seems that the parser is tripping over the "-"

func TestMapFields(t *testing.T) {
// Map field access should now be working

people := []Person{
	{
		Name: "Alice",
		Tags: map[string]string{
			"team-name": "backend",
			"location":  "remote",
		},
	},
	{
		Name: "Bob",
		Tags: map[string]string{
			"team-name": "frontend",
			"location":  "office",
		},
	},
}

tests := []struct {
	name     string
	query    string
	expected int
}{
	{"Map field equality", "Tags.team-name = 'backend'", 1},
	{"Map field contains", "Tags.location CONTAINS 'remote'", 1},
	{"Case insensitive map key", "Tags.TEAM-NAME = 'frontend'", 1},
}

for _, tt := range tests {
	t.Run(tt.name, func(t *testing.T) {
		results, err := Parse(tt.query, people)
		if err != nil {
			t.Fatalf("Error parsing query '%s': %v", tt.query, err)
		}

		if len(results) != tt.expected {
			t.Errorf("Query '%s' returned %d results, expected %d", tt.query, len(results), tt.expected)
		}
	})
}

}

parser git:(main) ✗ go test -v --run TestMapFields
=== RUN TestMapFields
=== RUN TestMapFields/Map_field_equality
parser_test.go:406: Error parsing query 'Tags.team-name = 'backend'': failed to parse query: -
=== RUN TestMapFields/Map_field_contains
=== RUN TestMapFields/Case_insensitive_map_key
parser_test.go:406: Error parsing query 'Tags.TEAM-NAME = 'frontend'': failed to parse query: -
--- FAIL: TestMapFields (0.00s)
--- FAIL: TestMapFields/Map_field_equality (0.00s)
--- PASS: TestMapFields/Map_field_contains (0.00s)
--- FAIL: TestMapFields/Case_insensitive_map_key (0.00s)
FAIL
exit status 1
FAIL github.com/zveinn/parser 0.007s

[mschexnaydre]

I found the issue and the fix is easy -- it is here:

https://github.com/zveinn/parser/blob/main/enhanced_lexer.go#L143

Changing this line to this fixes the issue and the modified unit test above will pass.

for isLetter(l.ch) || isDigit(l.ch) || l.ch == '.' || l.ch == '_' || l.ch == '-' {

I would be happy to create a PR but I am not a member of this project. Thanks for this package, it is very useful.