Skip to content

Commit 3c433e6

Browse files
authored
Merge pull request #14 from Leftyshields/improve/workflow-postmortem-h3m8
Workflow: local dev parity gates from Wallet Watcher postmortem
2 parents fe2cb4b + e2b9745 commit 3c433e6

7 files changed

Lines changed: 47 additions & 7 deletions

File tree

.cursor/commands/capture_issue.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,9 @@ Write the issue to `.ai/context/last_capture.md` using **exactly** this structur
9797
# Priority
9898
[low / medium / high]
9999

100+
# Environment
101+
[optional: local-dev | staging | production | vendor-sandbox]
102+
100103
# Open Questions
101104
[questions here]
102105

.cursor/commands/design_decisions.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,14 @@ For auth, payments, email, or third-party APIs, document:
7171

7272
Copy [DEV_RUNBOOK_TEMPLATE.md](docs/DEV_RUNBOOK_TEMPLATE.md)`docs/DEV_RUNBOOK.md` and add rows as you learn.
7373

74+
### 9. Data provenance (when multiple sources)
75+
76+
If data comes from external APIs **and** local seed/fixtures:
77+
78+
- List sources (e.g. vendor API, seed script, manual entry)
79+
- UI/API field for source or inference rule
80+
- Cleanup behavior on reconnect/relink/re-auth
81+
7482
### 10. External API identifiers (verify at implement time)
7583

7684
For LLM, payment, or vendor APIs, document:

.cursor/commands/pre_implementation_checklist.md

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,18 @@ If adding new API endpoints:
8181

8282
---
8383

84-
### 5. Content Rendering Strategy
84+
### 5. Monorepo & local runtime
85+
86+
- [ ] Documented whether API/services import workspace packages from **`dist/`** or **`src/`**
87+
- [ ] Dev workflow notes shared rebuild if applicable
88+
- [ ] Emulator/database: single-instance rule and port recovery in `DEV_RUNBOOK`
89+
- [ ] If feature needs demo data: seed script planned and wired to `package.json`
90+
91+
**Next Step:** Continue to step 6.
92+
93+
---
94+
95+
### 6. Content Rendering Strategy
8596

8697
If adding formatted content (markdown, HTML, rich text):
8798

@@ -93,11 +104,11 @@ If adding formatted content (markdown, HTML, rich text):
93104

94105
**Action:** Add to design decisions document
95106

96-
**Next Step:** Continue to step 6.
107+
**Next Step:** Continue to step 7.
97108

98109
---
99110

100-
### 6. React State Management Patterns
111+
### 7. React State Management Patterns
101112

102113
If using React state updates:
103114

.cursor/commands/workflow.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,7 @@ See also: [Product vs genome mission](docs/PRODUCT_VS_GENOME_MISSION.md).
6363
- Apply backend changes to **all documented entry points** (see `docs/ARCHITECTURE.md`)
6464
- Use functional state updates when updating state based on props/state
6565
- Implement format conversions as documented
66+
- **Ship local dev tooling** (seed scripts, npm scripts, runbook rows) when UI needs data to demo
6667

6768
**Optional:** Run `/tdd` for complex logic (test-driven development)
6869

@@ -86,8 +87,9 @@ See also: [Product vs genome mission](docs/PRODUCT_VS_GENOME_MISSION.md).
8687
**Next:** Fix critical/high findings, then run `/qa_checklist`
8788

8889
9. **QA Checklist** (`/qa_checklist`) — *standard*
89-
- Manual testing checklist: happy path, edge cases, error handling, visual/UX
90-
- Human validation only (no automated tests)
90+
- **Automated:** agent runs `npm test` and builds for affected workspaces
91+
- **Manual:** checklist for happy path, edge cases, error handling, visual/UX
92+
- Save issue-scoped checklist to `.ai/context/qa_checklist_<feature>.md` when helpful
9193

9294
**Next:** Fix any issues found, then run `/peer_review` (optional)
9395

@@ -159,7 +161,7 @@ For smaller changes or bug fixes:
159161
| `/tdd` | Optional | Red–green–refactor for complex logic during implementation |
160162
| `/code_review` | Standard | Automated review: security, correctness, architecture, quality, performance |
161163
| `/security_scan` | Optional | Dedicated security pass: secrets, deps, auth, data at rest, encryption, proprietary/IP |
162-
| `/qa_checklist` | Standard | Human manual-test checklist (happy path, edge cases, UX) |
164+
| `/qa_checklist` | Standard | Automated tests + human manual-test checklist (happy path, edge cases, UX) |
163165
| `/peer_review` | Optional | Human review of changes; accept/reject feedback with rationale |
164166
| `/postmortem` | Standard | Reflect on friction and rework; improve process and docs |
165167
| `/project_wrap_up` | Optional | Handover: security audit, onboarding docs, next-op briefing |

.genome/expression_profiles/organ-Build.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,4 @@
44
- Decomposition: organs→tissues and below
55
- Role library: Build (organs), Implementation (tissues), Worker (cells), read_file (molecules)
66
- Contracts: Build → Implementation
7+
- **Handoff gate:** Before declaring implementation complete, verify the smallest user-visible happy path in the target environment (or document exact commands for the human to do so).

.genome/expression_profiles/tissue-Implementation.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,4 @@
44
- Decomposition: tissues→cells, cells→molecules
55
- Role library: Implementation (tissues), Worker (cells), read_file (molecules)
66
- Contracts: Build → Implementation
7+
- **Handoff gate:** Ship seed/demo tooling with features that need persisted data; state exact happy-path commands before `/code_review`.

docs/INSTANTIATED_APP_FEEDBACK.md

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,18 @@ Changelog of workflow improvements derived from real app development (postmortem
2424

2525
Local dev parity was documented aspirationally but not treated as part of “done.” Demoability should be a deliverable, not a QA surprise.
2626

27+
### Changes incorporated into Genesis (PR `improve/workflow-postmortem-h3m8`#14)
28+
29+
- `/execute_plan` — Local Dev Verification gate
30+
- `/pre_implementation_checklist` — Monorepo & local runtime section; plan ID must match capture
31+
- `/workflow` — Flexible design/plan order; closure docs; common mistakes expanded
32+
- `/qa_checklist` — Automated tests + manual checklist split
33+
- `/postmortem` — Required closure artifact outputs
34+
- `/capture_issue` — Optional `# Environment` field
35+
- `/design_decisions` — Data provenance subsection
36+
- `DEV_RUNBOOK_TEMPLATE` — Script integrity rule + terminology table
37+
- Expression profiles — Happy-path handoff gates
38+
2739
---
2840

2941
## 2026-06-07 — AI Tastemakers (Daily Digest, EPH-20260606-DIG1)
@@ -82,6 +94,8 @@ Local dev parity was documented aspirationally but not treated as part of “don
8294
- `DEV_RUNBOOK_TEMPLATE` — same-day re-run + post-bot deploy rows
8395
- `/execute_plan` — same-day pipeline warning; post-digest Pages verification
8496

97+
---
98+
8599
## 2026-07-02 — AI Tastemakers (Landing Layout v2, EPH-20260701-LAND)
86100

87101
**Source:** [ai-tastemakers](https://github.com/Leftyshields/ai-tastemakers) — full workflow through QA close; flag-gated static layout experiment.
@@ -100,7 +114,7 @@ Local dev parity was documented aspirationally but not treated as part of “don
100114

101115
Static HTML generators treated Tailwind class strings like JSX utilities, but Tailwind v4 content scanning does not reliably emit rules from dynamic TS template literals.
102116

103-
### Changes incorporated into Genesis (PR `improve/tastemakers-postmortem-land`)
117+
### Changes incorporated into Genesis (PR `improve/tastemakers-postmortem-land`#17)
104118

105119
- `/explore` — partial/WIP conflicts table in snapshot
106120
- `/design_decisions` — static HTML generator styling section

0 commit comments

Comments
 (0)