forge-orkes 0.3.13 → 0.3.14
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/package.json
CHANGED
|
@@ -128,25 +128,42 @@ Attempt 1:
|
|
|
128
128
|
1. Read error output
|
|
129
129
|
2. Caused by current task?
|
|
130
130
|
- YES → fix code, stage fixes, amend commit
|
|
131
|
-
- NO → mark advisory for this session
|
|
131
|
+
- NO → mark advisory for this session; append to .forge/deferred-issues.md; continue
|
|
132
132
|
3. Re-run command
|
|
133
133
|
4. Pass → next command
|
|
134
134
|
5. Fail → next attempt (up to max_retries)
|
|
135
135
|
|
|
136
136
|
After max_retries exhausted:
|
|
137
|
+
→ Append to .forge/deferred-issues.md
|
|
137
138
|
→ Log failure in execution summary
|
|
138
139
|
→ Continue to next task
|
|
139
|
-
→ Verifying skill catches persistent failures later
|
|
140
140
|
```
|
|
141
141
|
|
|
142
142
|
### 3-Strike Integration
|
|
143
143
|
Verification retries count toward the task's 3-strike limit. 2 strikes used = 1 verification retry max.
|
|
144
144
|
|
|
145
145
|
### Do NOT Fix
|
|
146
|
-
- **Pre-existing failures** not from current task → mark advisory
|
|
146
|
+
- **Pre-existing failures** not from current task → mark advisory; append to `.forge/deferred-issues.md`
|
|
147
147
|
- **Flaky tests** passing on re-run without changes → note in summary, no strike
|
|
148
148
|
- **Unrelated warnings** (deprecation, non-blocking lint) → ignore
|
|
149
149
|
|
|
150
|
+
### Deferred Issues Format
|
|
151
|
+
|
|
152
|
+
Append one entry per failure to `.forge/deferred-issues.md` (create if missing):
|
|
153
|
+
|
|
154
|
+
```yaml
|
|
155
|
+
issues:
|
|
156
|
+
- id: DI-001
|
|
157
|
+
type: test_failure | lint_failure | build_failure
|
|
158
|
+
command: "npm test"
|
|
159
|
+
summary: "HealthEndpointTest — Redis unavailable in test env"
|
|
160
|
+
first_seen: "2026-04-14"
|
|
161
|
+
milestone: "m3"
|
|
162
|
+
status: pending
|
|
163
|
+
```
|
|
164
|
+
|
|
165
|
+
ID = `DI-{N}` where N = next integer after existing max (or 001 if new file).
|
|
166
|
+
|
|
150
167
|
### Quick Tier
|
|
151
168
|
Run all non-advisory verification commands once after commit. 1 retry max.
|
|
152
169
|
|
|
@@ -29,6 +29,7 @@ Read: .forge/project.yml → tech stack, framework, database, dependencies
|
|
|
29
29
|
Read: .forge/state/milestone-{id}.yml → milestone ID and name
|
|
30
30
|
Read: .forge/constitution.md → active architectural gates (if exists)
|
|
31
31
|
Read: .forge/refactor-backlog.yml → existing backlog items (if any)
|
|
32
|
+
Read: .forge/deferred-issues.md → pre-existing failures logged during execution (if exists)
|
|
32
33
|
```
|
|
33
34
|
|
|
34
35
|
Skip by stack: no DB->SQL/NoSQL N/A, no frontend->XSS N/A, no CI/CD->Pipeline N/A.
|
|
@@ -94,7 +95,7 @@ security_audit:
|
|
|
94
95
|
|-----------|--------|
|
|
95
96
|
| **Scalability** | Synchronous blocking, missing pagination, unbounded queries, N+1, missing caching, SPOFs, hardcoded limits |
|
|
96
97
|
| **Maintainability** | Files >300 lines, nesting >4, god components/classes, circular deps, dup logic |
|
|
97
|
-
| **Code Health** | Dead code/unused exports, TODO/FIXME inventory, untested critical paths, stale deps |
|
|
98
|
+
| **Code Health** | Dead code/unused exports, TODO/FIXME inventory, untested critical paths, stale deps, deferred issues in `.forge/deferred-issues.md` |
|
|
98
99
|
| **Structural Quality** | Biz logic in UI, inconsistent patterns, missing error boundaries, API contract drift |
|
|
99
100
|
|
|
100
101
|
**Rules:** Actual code, not theory. Specific files + evidence. `critical`=prod issues/blocking, `warning`=quality, `info`=improvement. Respect ADRs + constitution.
|
|
@@ -254,6 +255,13 @@ Refactoring triage (max 10): *"{N} opportunities:"*
|
|
|
254
255
|
|
|
255
256
|
**Accept**->backlog | **Dismiss**->skip | **Accept all** | **Dismiss all**
|
|
256
257
|
|
|
258
|
+
Deferred issues triage: If `.forge/deferred-issues.md` has `status: pending` items, surface them:
|
|
259
|
+
*"**Test debt** ({N} pre-existing failures): 1. `{summary}` -- first seen {date}. [Accept/Dismiss/Fix-now]*"*
|
|
260
|
+
|
|
261
|
+
- **Accept** → add to refactor-backlog.yml as `category: test-debt`, mark `status: triaged` in deferred-issues.md
|
|
262
|
+
- **Fix-now** → route to `planning` fix mode before completing milestone
|
|
263
|
+
- **Dismiss** → mark `status: dismissed` in deferred-issues.md with reason
|
|
264
|
+
|
|
257
265
|
## Step 7: Backlog + Route
|
|
258
266
|
|
|
259
267
|
### Backlog
|