Skip to content

Commit 823e006

Browse files
committed
update coderabbit.yaml
1 parent 2323119 commit 823e006

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

.coderabbit.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,13 @@ reviews:
1313
instructions: |
1414
You are reviewing PHP domain-layer code. Enforce domain purity, with a relaxed policy for DynamicListAttr:
1515
16-
- ❌ Do not allow persistence or transaction side effects here for *normal* domain models.
17-
- Flag ANY usage of Doctrine persistence APIs on regular domain entities, especially:
16+
- ❌ Do not allow, flag ANY DB write / finalization:
1817
- `$entityManager->flush(...)`, `$this->entityManager->flush(...)`
19-
- `$em->persist(...)`, `$em->remove(...)`
20-
- `$em->beginTransaction()`, `$em->commit()`, `$em->rollback()`
18+
- `$em->beginTransaction()`, `$em->commit()`, `$em->rollback()`, `$em->transactional(...)`
19+
- `$em->getConnection()->executeStatement(...)` for DML/DDL (INSERT/UPDATE/DELETE/ALTER/...)
2120
- ✅ Accessing Doctrine *metadata*, *schema manager*, or *read-only schema info* is acceptable
22-
as long as it does not modify state or perform writes.
21+
as long as it does not modify state or perform writes. Accessing Doctrine *persistence APIs*
22+
persist, remove, etc.) is acceptable, allow scheduling changes in the UnitOfWork (no DB writes)
2323
2424
- ✅ **Relaxed rule for DynamicListAttr-related code**:
2525
- DynamicListAttr is a special case dealing with dynamic tables/attrs.

0 commit comments

Comments
 (0)