Fix CREATE POLICY quote handling to only quote when necessary

- Replace hardcoded quote wrapping with QuoteUtils.quote() in CreatePolicyStmt and AlterPolicyStmt methods
- Policy names like 'user_policy', 'admin_policy', 'simple_policy' are now correctly unquoted
- Policy names with spaces or special characters (e.g. 'Simple Policy') are still properly quoted
- Updated test snapshots to reflect correct quoting behavior
- All 269 test suites pass with 380 tests passing

Co-Authored-By: Dan Lynch <[email protected]>

Author: devin-ai-integration[bot]

packages/deparser/__tests__/pretty/__snapshots__/create-policy-pretty.test.ts.snap

@@ -1,21 +1,21 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`non-pretty: pretty/create_policy-1.sql 1`] = `"CREATE POLICY "user_policy" ON users AS PERMISSIVE FOR ALL TO authenticated_users USING (user_id = current_user_id())"`;
+exports[`non-pretty: pretty/create_policy-1.sql 1`] = `"CREATE POLICY user_policy ON users AS PERMISSIVE FOR ALL TO authenticated_users USING (user_id = current_user_id())"`;
 
-exports[`non-pretty: pretty/create_policy-2.sql 1`] = `"CREATE POLICY "admin_policy" ON sensitive_data AS RESTRICTIVE FOR SELECT TO admin_role USING (department = current_user_department()) WITH CHECK (approved = true)"`;
+exports[`non-pretty: pretty/create_policy-2.sql 1`] = `"CREATE POLICY admin_policy ON sensitive_data AS RESTRICTIVE FOR SELECT TO admin_role USING (department = current_user_department()) WITH CHECK (approved = true)"`;
 
-exports[`non-pretty: pretty/create_policy-3.sql 1`] = `"CREATE POLICY "complex_policy" ON documents AS PERMISSIVE FOR UPDATE TO document_editors USING (owner_id = current_user_id() OR (shared = true AND permissions @> '{"edit": true}')) WITH CHECK (status <> 'archived' AND last_modified > (now() - '1 day'::interval))"`;
+exports[`non-pretty: pretty/create_policy-3.sql 1`] = `"CREATE POLICY complex_policy ON documents AS PERMISSIVE FOR UPDATE TO document_editors USING (owner_id = current_user_id() OR (shared = true AND permissions @> '{"edit": true}')) WITH CHECK (status <> 'archived' AND last_modified > (now() - '1 day'::interval))"`;
 
-exports[`non-pretty: pretty/create_policy-4.sql 1`] = `"CREATE POLICY "simple_policy" ON posts AS PERMISSIVE FOR SELECT TO PUBLIC USING (published = true)"`;
+exports[`non-pretty: pretty/create_policy-4.sql 1`] = `"CREATE POLICY simple_policy ON posts AS PERMISSIVE FOR SELECT TO PUBLIC USING (published = true)"`;
 
-exports[`non-pretty: pretty/create_policy-5.sql 1`] = `"CREATE POLICY "simple_policy" ON posts AS PERMISSIVE FOR SELECT TO PUBLIC USING (published = true)"`;
+exports[`non-pretty: pretty/create_policy-5.sql 1`] = `"CREATE POLICY simple_policy ON posts AS PERMISSIVE FOR SELECT TO PUBLIC USING (published = true)"`;
 
 exports[`non-pretty: pretty/create_policy-6.sql 1`] = `"CREATE POLICY "Simple Policy" ON posts AS PERMISSIVE FOR SELECT TO PUBLIC USING (published = true)"`;
 
-exports[`non-pretty: pretty/create_policy-7.sql 1`] = `"CREATE POLICY "simplepolicy" ON posts AS PERMISSIVE FOR SELECT TO PUBLIC USING (published = true)"`;
+exports[`non-pretty: pretty/create_policy-7.sql 1`] = `"CREATE POLICY simplepolicy ON posts AS PERMISSIVE FOR SELECT TO PUBLIC USING (published = true)"`;
 
 exports[`pretty: pretty/create_policy-1.sql 1`] = `
-"CREATE POLICY "user_policy"
+"CREATE POLICY user_policy
   ON users
   AS PERMISSIVE
   FOR ALL
@@ -26,7 +26,7 @@ exports[`pretty: pretty/create_policy-1.sql 1`] = `
 `;
 
 exports[`pretty: pretty/create_policy-2.sql 1`] = `
-"CREATE POLICY "admin_policy"
+"CREATE POLICY admin_policy
   ON sensitive_data
   AS RESTRICTIVE
   FOR SELECT
@@ -40,7 +40,7 @@ exports[`pretty: pretty/create_policy-2.sql 1`] = `
 `;
 
 exports[`pretty: pretty/create_policy-3.sql 1`] = `
-"CREATE POLICY "complex_policy"
+"CREATE POLICY complex_policy
   ON documents
   AS PERMISSIVE
   FOR UPDATE
@@ -57,7 +57,7 @@ exports[`pretty: pretty/create_policy-3.sql 1`] = `
 `;
 
 exports[`pretty: pretty/create_policy-4.sql 1`] = `
-"CREATE POLICY "simple_policy"
+"CREATE POLICY simple_policy
   ON posts
   AS PERMISSIVE
   FOR SELECT
@@ -68,7 +68,7 @@ exports[`pretty: pretty/create_policy-4.sql 1`] = `
 `;
 
 exports[`pretty: pretty/create_policy-5.sql 1`] = `
-"CREATE POLICY "simple_policy"
+"CREATE POLICY simple_policy
   ON posts
   AS PERMISSIVE
   FOR SELECT
@@ -90,7 +90,7 @@ exports[`pretty: pretty/create_policy-6.sql 1`] = `
 `;
 
 exports[`pretty: pretty/create_policy-7.sql 1`] = `
-"CREATE POLICY "simplepolicy"
+"CREATE POLICY simplepolicy
   ON posts
   AS PERMISSIVE
   FOR SELECT

packages/deparser/src/deparser.ts

@@ -6679,7 +6679,7 @@ export class Deparser implements DeparserVisitor {
 
     const initialParts = ['CREATE', 'POLICY'];
     if (node.policy_name) {
-      initialParts.push(`"${node.policy_name}"`);
+      initialParts.push(QuoteUtils.quote(node.policy_name));
     }
 
     output.push(initialParts.join(' '));
@@ -6758,7 +6758,7 @@ export class Deparser implements DeparserVisitor {
     const output: string[] = ['ALTER', 'POLICY'];
 
     if (node.policy_name) {
-      output.push(`"${node.policy_name}"`);
+      output.push(QuoteUtils.quote(node.policy_name));
     }
 
     if (node.table) {