fix: sanitize aceSchema identifier to prevent SQL breakage with non-simple names

mason-sharp · claude · mason-sharp · commit fcd68db5be75 · 2026-04-14T10:58:25.000-07:00
Use pgx.Identifier.Sanitize() to quote the configurable ACE schema name
in SQL templates, consistent with how all other identifiers are handled
in queries.go.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/db/queries/templates.go b/db/queries/templates.go
@@ -14,13 +14,14 @@ package queries
 import (
 	"text/template"
 
+	"github.com/jackc/pgx/v5"
 	"github.com/pgedge/ace/pkg/config"
 )
 
 // aceTemplateFuncs provides the {{aceSchema}} function to SQL templates.
 // The function is evaluated at render time (after config is loaded), not at parse time.
 var aceTemplateFuncs = template.FuncMap{
-	"aceSchema": func() string { return config.Get().MTree.Schema },
+	"aceSchema": func() string { return pgx.Identifier{config.Get().MTree.Schema}.Sanitize() },
 }
 
 type Templates struct {

Original file line number	Diff line number	Diff line change
`@@ -14,13 +14,14 @@ package queries`
`14`	`14`	`import (`
`15`	`15`	`"text/template"`
`16`	`16`
	`17`	`+ "github.com/jackc/pgx/v5"`
`17`	`18`	`"github.com/pgedge/ace/pkg/config"`
`18`	`19`	`)`
`19`	`20`
`20`	`21`	`// aceTemplateFuncs provides the {{aceSchema}} function to SQL templates.`
`21`	`22`	`// The function is evaluated at render time (after config is loaded), not at parse time.`
`22`	`23`	`var aceTemplateFuncs = template.FuncMap{`
`23`		`- "aceSchema": func() string { return config.Get().MTree.Schema },`
	`24`	`+ "aceSchema": func() string { return pgx.Identifier{config.Get().MTree.Schema}.Sanitize() },`
`24`	`25`	`}`
`25`	`26`
`26`	`27`	`type Templates struct {`