toga-ai 1.0.956 → 1.0.957
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.
|
@@ -6,7 +6,7 @@ project: Worker
|
|
|
6
6
|
client: shared
|
|
7
7
|
type: feature
|
|
8
8
|
status: active
|
|
9
|
-
updated: 2026-10-
|
|
9
|
+
updated: 2026-10-06
|
|
10
10
|
owners: ["jcardinal", "ajean"]
|
|
11
11
|
files:
|
|
12
12
|
- worker2/Worker/Clickup/QaReview.php
|
|
@@ -110,6 +110,9 @@ Batch) QC Review → Stage or Client Review → To Deploy → **manual** deploy.
|
|
|
110
110
|
**Needs Review** ([general automation](./clickup-general-automation.md) rule 7); `QaReview` /
|
|
111
111
|
`FeatureReview` ignore Needs Review, so there is no loop.
|
|
112
112
|
- **Deny** sends the task to **Rework** and **reverts** the task's changes from the QA/QC branch.
|
|
113
|
+
- **BE `Conflict`** (BE Review option orderindex 4) = Denied: Rework + clear Environment + revert
|
|
114
|
+
(`REVIEW_CONFLICT`, reason `be-review-conflict`). Entry reset already clears it; `replayEarlyReview`
|
|
115
|
+
does not replay it.
|
|
113
116
|
- **Unit-test gate is a STUB today.** `runUnitTests()` currently returns `true`. Real gating (Cypress /
|
|
114
117
|
Postman / Codeception) is pending — do not treat the gate as enforcing anything yet.
|
|
115
118
|
|
|
@@ -172,9 +175,17 @@ Batch) QC Review → Stage or Client Review → To Deploy → **manual** deploy.
|
|
|
172
175
|
so `FeatureReview` owns them; `StageBatch` ships a Feature as one feature branch and finalizes its
|
|
173
176
|
subtasks (and skips subtasks in the batch scan).
|
|
174
177
|
- The feature's own **BE Review is a MANUAL field for now** (jcardinal + CTO decision).
|
|
178
|
+
- `rollUpBe`: feature's own BE **Conflict** → `reworkFeature` (`feature-be-conflict`); any subtask BE
|
|
179
|
+
**Denied or Conflict** → those subtasks + feature to Rework.
|
|
175
180
|
|
|
176
181
|
## Gotchas
|
|
177
182
|
|
|
183
|
+
- **A task already sitting in BE `Conflict` before the 2026-10-06 fix must be moved to Rework by hand** —
|
|
184
|
+
code reacts only to a field change.
|
|
185
|
+
- **ClickUP for Dev MCP is allowlist-scoped.** Its read scope must include Dev Sprints `90116585845`;
|
|
186
|
+
otherwise `get_task` on a custom id fails with "outside the authorized scope" or a misleading
|
|
187
|
+
"Invalid ClickUp API token". Workaround: prod `Team.Tasks` `customClickupIdentifier` →
|
|
188
|
+
`clickupIdentifier`, then `get_task` with the internal id.
|
|
178
189
|
- **Entry is on ENTERING the status, not leaving it.** The legacy trigger fired on leaving; the QA/QC
|
|
179
190
|
entry point had to move to the entering event or the whole chain never starts.
|
|
180
191
|
- **The unit-test gate is a stub (`runUnitTests()` returns true).** It does not block anything yet.
|
|
@@ -220,5 +231,6 @@ Batch) QC Review → Stage or Client Review → To Deploy → **manual** deploy.
|
|
|
220
231
|
- [ClickUp custom-field conventions](./clickup-custom-field-conventions.md)
|
|
221
232
|
|
|
222
233
|
## Change history
|
|
234
|
+
- 2026-10-06 — BE Review `Conflict` now routes to Rework + revert like Denied (QaReview + FeatureReview) (jcardinal)
|
|
223
235
|
- 2026-10-01 — DevFinished CatchUp (cron 86); QC Batch turned on (id 75); review fields reset to Needs Review on entry; Stage deploys confirmed manual (ajean; worker2 PR #178, dbchanges2 PR #504)
|
|
224
236
|
- 2026-09-25 — Scope narrowed to Dev Sprints `90116585845` (jcardinal)
|
package/package.json
CHANGED