Skip to content

Commit c8818ce

Browse files
committed
Capture deferred cleanup backlog as explicit next steps
The main cleanup sequence is complete, but a few deferred items still need a tracked follow-up path. This note records the remaining schema, render, lifecycle, and prompt risks as concrete next actions so the work can resume intentionally instead of being rediscovered later. Constraint: Deferred items should stay visible without forcing more risky refactors into the completed cleanup lane Rejected: Leave deferred work only in conversation history | too easy to lose and harder to prioritize later Confidence: high Scope-risk: narrow Reversibility: clean Directive: Keep NEXT_STEPS.md focused on real deferred work and update it only when scope meaningfully changes Tested: Document reviewed against current repo state and analysis notes Not-tested: Issue tracker sync or backlog import
1 parent 14ffc9b commit c8818ce

1 file changed

Lines changed: 53 additions & 0 deletions

File tree

NEXT_STEPS.md

Lines changed: 53 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,53 @@
1+
# Next Steps
2+
3+
## Deferred architecture / cleanup backlog
4+
5+
### 1. Tool schema vs runtime schema boundary
6+
- **Status:** deferred by design
7+
- **Priority:** high
8+
- **Why:** similar payload shapes still exist in `src/tools.ts` and `src/runtime/schema.ts`
9+
- **Risk:** high — crosses the OpenCode `tool.schema` vs runtime Zod boundary
10+
- **Recommended approach:** treat as a design task, not a cleanup pass
11+
- **Concrete next action:** choose a source-of-truth strategy, likely adapters or shared primitive fragments only
12+
- **Reference:** `analysis/tool-runtime-schema-boundary-design.md`
13+
14+
### 2. Schema drift on high-churn payloads
15+
- **Status:** partially mitigated, not solved
16+
- **Priority:** medium-high
17+
- **Why:** worker/reviewer/plan payloads still have parallel definitions
18+
- **Risk:** medium
19+
- **Concrete next action:** prototype a single adapter path for `flow_run_complete_feature`
20+
21+
### 3. Render layer simplification
22+
- **Status:** improved, not fully simplified
23+
- **Priority:** medium
24+
- **Why:** `src/runtime/render.ts` still relies on large markdown assembly functions
25+
- **Risk:** medium
26+
- **Concrete next action:** only if needed, split `renderIndexDoc()` into smaller section builders under existing test coverage
27+
28+
### 4. Session lifecycle separation
29+
- **Status:** boundary clarified, not redesigned
30+
- **Priority:** medium-low
31+
- **Why:** `saveSession()` still synchronously triggers doc generation
32+
- **Risk:** medium
33+
- **Concrete next action:** revisit only if performance or lifecycle complexity grows
34+
35+
### 5. Prompt-layer semantic duplication
36+
- **Status:** reduced, not eliminated
37+
- **Priority:** low-medium
38+
- **Why:** planner/worker/auto prompts still overlap in some instructions
39+
- **Risk:** medium if wording changes break prompt behavior
40+
- **Concrete next action:** continue only when prompt maintenance becomes painful; preserve tested wording
41+
42+
### 6. Convert deferred items into tracked issues
43+
- **Status:** not done
44+
- **Priority:** medium
45+
- **Concrete next action:** create issues or milestone entries for items 1-5
46+
47+
## Definition of done for future follow-up
48+
- design approved before any risky schema-boundary refactor
49+
- behavior preserved under:
50+
- `bun test tests/config.test.ts`
51+
- `bun test tests/runtime.test.ts`
52+
- `bun run check`
53+
- avoid mixing architecture changes with small cleanup passes

0 commit comments

Comments
 (0)