fix: add CI rule to RULES.md and fix node wrapping in v16-to-v17 transformer

- Added 'DO NOT LOOK AT CI' rule to RULES.md as requested
- Fixed DeclareCursorStmt, CreateAmStmt, CreateForeignTableStmt to return properly wrapped objects
- Improved 16-17 test count from 234/258 to 247/258 passing tests

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

Author: devin-ai-integration[bot]

packages/transform/RULES.md

@@ -206,6 +206,19 @@ const pg14TableLikeOptions = PG14Enums.TableLikeOption;
 
 Always consult the enum files when debugging transformation issues involving numeric or string values that appear to be op codes or enum constants.
 
+## DO NOT LOOK AT CI — only work locally with tests.
+
+**⚠️ CRITICAL RULE: Never look at CI logs or use CI-related commands during development.**
+
+When debugging transformation issues:
+- Run tests locally using `yarn test __tests__/kitchen-sink/13-14` or similar
+- Examine local test output and failure messages
+- Reproduce issues locally and verify fixes locally
+- Only push changes after verifying they work locally
+- Do not use `gh run view`, `git_pr_checks`, or other CI inspection commands
+
+This ensures faster feedback loops and prevents dependency on external CI systems during development.
+
 ## Summary
 
 Always use `@pgsql/parser` for multi-version PostgreSQL AST parsing in the transform package. This is the only way to get accurate version-specific results and build working transformers. Remember that all parser methods are async and must be awaited.

packages/transform/src/transformers/v16-to-v17.ts

@@ -794,7 +794,7 @@ export class V16ToV17Transformer {
   }
 
   DeclareCursorStmt(node: PG16.DeclareCursorStmt, context: TransformerContext): any {
-    return node;
+    return { DeclareCursorStmt: node };
   }
 
   PublicationObjSpec(node: PG16.PublicationObjSpec, context: TransformerContext): any {
@@ -806,7 +806,7 @@ export class V16ToV17Transformer {
   }
 
   CreateAmStmt(node: PG16.CreateAmStmt, context: TransformerContext): any {
-    return node;
+    return { CreateAmStmt: node };
   }
 
   IntoClause(node: PG16.IntoClause, context: TransformerContext): any {
@@ -906,6 +906,6 @@ export class V16ToV17Transformer {
   }
 
   CreateForeignTableStmt(node: PG16.CreateForeignTableStmt, context: TransformerContext): any {
-    return node;
+    return { CreateForeignTableStmt: node };
   }
 }