Fix ALTER DATABASE statement handling

- Add UseCurrent field to AlterDatabaseRemoveFileGroupStatement
- Differentiate READONLY/READWRITE (old syntax) from READ_ONLY/READ_WRITE
  - READONLY -> ReadOnlyOld
  - READ_ONLY -> ReadOnly
  - READWRITE -> ReadWriteOld
  - READ_WRITE -> ReadWrite
- Always output UseCurrent in AlterDatabaseModifyFileGroupStatement
- Update PhaseOne tests to include new fields

Author: claude

ast/alter_database_set_statement.go

@@ -97,6 +97,7 @@ func (a *AlterDatabaseRemoveFileStatement) statement() {}
 type AlterDatabaseRemoveFileGroupStatement struct {
 	DatabaseName  *Identifier
 	FileGroupName *Identifier
+	UseCurrent    bool
 }
 
 func (a *AlterDatabaseRemoveFileGroupStatement) node()      {}

parser/marshal.go

@@ -6331,10 +6331,7 @@ func alterDatabaseModifyFileGroupStatementToJSON(s *ast.AlterDatabaseModifyFileG
 		node["FileGroup"] = identifierToJSON(s.FileGroupName)
 	}
 	node["MakeDefault"] = s.MakeDefault
-	// Only output UseCurrent when UpdatabilityOption is an autogrow option
-	if strings.HasPrefix(s.UpdatabilityOption, "Autogrow") {
-		node["UseCurrent"] = false
-	}
+	node["UseCurrent"] = false
 	if s.NewFileGroupName != nil {
 		node["NewFileGroupName"] = identifierToJSON(s.NewFileGroupName)
 	}
@@ -6380,6 +6377,7 @@ func alterDatabaseRemoveFileGroupStatementToJSON(s *ast.AlterDatabaseRemoveFileG
 	if s.FileGroupName != nil {
 		node["FileGroup"] = identifierToJSON(s.FileGroupName)
 	}
+	node["UseCurrent"] = s.UseCurrent
 	return node
 }
 

parser/parse_ddl.go

@@ -1580,10 +1580,16 @@ func (p *Parser) parseAlterDatabaseModifyStatement(dbName *ast.Identifier) (ast.
 			case "DEFAULT":
 				stmt.MakeDefault = true
 				p.nextToken()
-			case "READONLY", "READ_ONLY":
+			case "READONLY":
+				stmt.UpdatabilityOption = "ReadOnlyOld"
+				p.nextToken()
+			case "READ_ONLY":
 				stmt.UpdatabilityOption = "ReadOnly"
 				p.nextToken()
-			case "READWRITE", "READ_WRITE":
+			case "READWRITE":
+				stmt.UpdatabilityOption = "ReadWriteOld"
+				p.nextToken()
+			case "READ_WRITE":
 				stmt.UpdatabilityOption = "ReadWrite"
 				p.nextToken()
 			case "AUTOGROW_ALL_FILES":

parser/testdata/PhaseOne_AlterDatabaseModifyFilegroup1StatementTest/ast.json

@@ -17,6 +17,7 @@
             "Value": "fg1"
           },
           "MakeDefault": false,
+          "UseCurrent": false,
           "UpdatabilityOption": "ReadOnly"
         }
       ]

parser/testdata/PhaseOne_AlterDatabaseModifyFilegroup2StatementTest/ast.json

@@ -17,6 +17,7 @@
             "Value": "fg1"
           },
           "MakeDefault": false,
+          "UseCurrent": false,
           "NewFileGroupName": {
             "$type": "Identifier",
             "QuoteType": "NotQuoted",

parser/testdata/PhaseOne_AlterDatabaseModifyFilegroup3StatementTest/ast.json

@@ -16,7 +16,8 @@
             "QuoteType": "SquareBracket",
             "Value": "fg1"
           },
-          "MakeDefault": true
+          "MakeDefault": true,
+          "UseCurrent": false
         }
       ]
     }

parser/testdata/PhaseOne_AlterDatabaseRemoveFilegroupStatementTest/ast.json

@@ -15,7 +15,8 @@
             "$type": "Identifier",
             "QuoteType": "SquareBracket",
             "Value": "fg1"
-          }
+          },
+          "UseCurrent": false
         }
       ]
     }