Skip to content

Commit 2634722

Browse files
committed
todo
1 parent b13e98b commit 2634722

File tree

1 file changed

+2
-29
lines changed

1 file changed

+2
-29
lines changed

packages/transform/TODO.md

Lines changed: 2 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -1,30 +1,3 @@
1-
# TODO: Transform Package Improvements
1+
# TODO
22

3-
## Type Safety Improvements
4-
5-
### Add Return Type Annotations to Transformer Methods
6-
- **Priority**: High
7-
- **Description**: Add proper TypeScript return type annotations to all transformer methods in v15-to-v16.ts (and other transformers)
8-
- **Benefit**: Would catch structural issues like double-wrapping at compile time instead of runtime
9-
- **Example**:
10-
```typescript
11-
TypeName(node: PG15.TypeName, context: TransformerContext): { TypeName: PG16.TypeName } {
12-
// implementation
13-
}
14-
```
15-
- **Impact**: Prevents bugs like the double-wrapping issue we encountered where `{"TypeName": {"TypeName": {...}}}` was produced instead of `{"TypeName": {...}}`
16-
17-
### Improve Type Definitions
18-
- Add stricter typing for node transformation methods
19-
- Consider using mapped types for consistent return type patterns
20-
- Add compile-time validation for node wrapping consistency
21-
22-
## Testing Improvements
23-
- Add unit tests for individual transformer methods
24-
- Create focused test cases for edge cases like empty Integer nodes
25-
- Improve error messages in transformation mismatches
26-
27-
## Documentation
28-
- Document transformation patterns and conventions
29-
- Add examples of proper node wrapping
30-
- Document debugging strategies for transformation issues
3+
- [ ] add skip file that annotates which files for full transform to skip, including reason

0 commit comments

Comments
 (0)