fix: rename package to query_to_header to match plugin import path

Traefik's Yaegi-based plugin loader derives the import alias from the
last segment of the module path (query-to-header -> query_to_header)
and fails with "undefined: query_to_header" if the declared package
name doesn't match exactly. The package was previously named
traefik_query_to_header.

Author: cmuench

AGENTS.md

@@ -26,8 +26,11 @@ is only a compile-correctness check.
 
 ## Architecture
 
-Everything lives in a single package, `traefik_query_to_header` (package name uses underscores
-because the module path `query-to-header` is not a valid Go identifier):
+Everything lives in a single package, `query_to_header`. The package name must exactly match
+the import path's last segment with hyphens converted to underscores
+(`query-to-header` → `query_to_header`) — Traefik's Yaegi-based plugin loader derives the
+import alias from the path and fails with `undefined: query_to_header` if the declared package
+name doesn't match.
 
 - **`query_to_header.go`** — the entire plugin:
   - `Config` / `Mapping` — JSON-tagged structs populated by Traefik from the dynamic

query_to_header.go

@@ -1,7 +1,7 @@
-// Package traefik_query_to_header is a Traefik middleware plugin that
+// Package query_to_header is a Traefik middleware plugin that
 // copies values from incoming request query parameters into HTTP request
 // headers before the request reaches the next handler in the chain.
-package traefik_query_to_header
+package query_to_header
 
 import (
 	"context"

query_to_header_test.go

@@ -1,36 +1,36 @@
-package traefik_query_to_header_test
+package query_to_header_test
 
 import (
 	"context"
 	"net/http"
 	"net/http/httptest"
 	"testing"
 
-	traefik_query_to_header "github.com/muench-dev/query-to-header"
+	"github.com/muench-dev/query-to-header"
 )
 
 func TestNew_RejectsNilConfig(t *testing.T) {
 	next := http.HandlerFunc(func(rw http.ResponseWriter, req *http.Request) {})
 
-	if _, err := traefik_query_to_header.New(context.Background(), next, nil, "test"); err == nil {
+	if _, err := query_to_header.New(context.Background(), next, nil, "test"); err == nil {
 		t.Fatal("expected an error for a nil config, got none")
 	}
 }
 
 func TestNew_RejectsIncompleteMapping(t *testing.T) {
 	next := http.HandlerFunc(func(rw http.ResponseWriter, req *http.Request) {})
-	cfg := &traefik_query_to_header.Config{
-		Mappings: []traefik_query_to_header.Mapping{{Query: "token"}},
+	cfg := &query_to_header.Config{
+		Mappings: []query_to_header.Mapping{{Query: "token"}},
 	}
 
-	if _, err := traefik_query_to_header.New(context.Background(), next, cfg, "test"); err == nil {
+	if _, err := query_to_header.New(context.Background(), next, cfg, "test"); err == nil {
 		t.Fatal("expected an error for a mapping missing a header name, got none")
 	}
 }
 
 func TestServeHTTP_SetsHeaderFromQuery(t *testing.T) {
-	cfg := &traefik_query_to_header.Config{
-		Mappings: []traefik_query_to_header.Mapping{
+	cfg := &query_to_header.Config{
+		Mappings: []query_to_header.Mapping{
 			{Query: "token", Header: "X-Token"},
 		},
 	}
@@ -40,7 +40,7 @@ func TestServeHTTP_SetsHeaderFromQuery(t *testing.T) {
 		gotHeader = req.Header.Get("X-Token")
 	})
 
-	handler, err := traefik_query_to_header.New(context.Background(), next, cfg, "test")
+	handler, err := query_to_header.New(context.Background(), next, cfg, "test")
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -56,8 +56,8 @@ func TestServeHTTP_SetsHeaderFromQuery(t *testing.T) {
 }
 
 func TestServeHTTP_PrefixesValueWithBearer(t *testing.T) {
-	cfg := &traefik_query_to_header.Config{
-		Mappings: []traefik_query_to_header.Mapping{
+	cfg := &query_to_header.Config{
+		Mappings: []query_to_header.Mapping{
 			{Query: "token", Header: "Authorization", Bearer: true},
 		},
 	}
@@ -67,7 +67,7 @@ func TestServeHTTP_PrefixesValueWithBearer(t *testing.T) {
 		gotHeader = req.Header.Get("Authorization")
 	})
 
-	handler, err := traefik_query_to_header.New(context.Background(), next, cfg, "test")
+	handler, err := query_to_header.New(context.Background(), next, cfg, "test")
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -83,8 +83,8 @@ func TestServeHTTP_PrefixesValueWithBearer(t *testing.T) {
 }
 
 func TestServeHTTP_DoesNotOverwriteExistingHeaderByDefault(t *testing.T) {
-	cfg := &traefik_query_to_header.Config{
-		Mappings: []traefik_query_to_header.Mapping{
+	cfg := &query_to_header.Config{
+		Mappings: []query_to_header.Mapping{
 			{Query: "token", Header: "X-Token"},
 		},
 	}
@@ -94,7 +94,7 @@ func TestServeHTTP_DoesNotOverwriteExistingHeaderByDefault(t *testing.T) {
 		gotHeader = req.Header.Get("X-Token")
 	})
 
-	handler, err := traefik_query_to_header.New(context.Background(), next, cfg, "test")
+	handler, err := query_to_header.New(context.Background(), next, cfg, "test")
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
@@ -111,8 +111,8 @@ func TestServeHTTP_DoesNotOverwriteExistingHeaderByDefault(t *testing.T) {
 }
 
 func TestServeHTTP_RemovesQueryParamWhenConfigured(t *testing.T) {
-	cfg := &traefik_query_to_header.Config{
-		Mappings: []traefik_query_to_header.Mapping{
+	cfg := &query_to_header.Config{
+		Mappings: []query_to_header.Mapping{
 			{Query: "token", Header: "X-Token", Remove: true},
 		},
 	}
@@ -122,7 +122,7 @@ func TestServeHTTP_RemovesQueryParamWhenConfigured(t *testing.T) {
 		gotQuery = req.URL.RawQuery
 	})
 
-	handler, err := traefik_query_to_header.New(context.Background(), next, cfg, "test")
+	handler, err := query_to_header.New(context.Background(), next, cfg, "test")
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}