Skip to content

Commit 64ece58

Browse files
committed
fix: address PR review comments on data-model and tech-debt skills
- Add deleted_at to ORDER_ITEMS in ER diagram - Fix contradictory numbered filename for tech debt register
1 parent 04bea4f commit 64ece58

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

skills/create-data-model/SKILL.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,7 @@ erDiagram
151151
decimal unit_price
152152
timestamp created_at
153153
timestamp updated_at
154+
timestamp deleted_at
154155
}
155156
```
156157

skills/manage-tech-debt/SKILL.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,17 +62,17 @@ Track, categorize, and prioritize technical debt using a structured register. Th
6262
- **Coupling to upcoming work**: If you are already changing the affected area, fix the debt now (lowest marginal cost)
6363
- **Risk**: Debt that could cause incidents or data loss gets a priority boost regardless of effort
6464

65-
7. **Write or update the register** -- Save to `.chalk/docs/engineering/<n>_tech_debt_register.md`. If updating an existing register, preserve history (do not delete resolved items; mark them as resolved with the date).
65+
7. **Write or update the register** -- Save to `.chalk/docs/engineering/tech_debt_register.md`. If updating an existing register, preserve history (do not delete resolved items; mark them as resolved with the date).
6666

6767
8. **Summarize** -- Tell the user what was added/changed, the current top 5 priorities, and any items that align with upcoming work.
6868

6969
## Filename Convention
7070

7171
```
72-
<number>_tech_debt_register.md
72+
tech_debt_register.md
7373
```
7474

75-
A project should have exactly one tech debt register. If one already exists, update it instead of creating a new one.
75+
A project should have exactly one tech debt register at a fixed path (e.g., `.chalk/docs/engineering/tech_debt_register.md`). If one already exists, update it instead of creating a new one.
7676

7777
## Tech Debt Register Format
7878

0 commit comments

Comments
 (0)