@windyroad/architect 0.18.4 → 0.18.5-preview.896
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.
|
@@ -61,7 +61,7 @@ check_architect_gate() {
|
|
|
61
61
|
fi
|
|
62
62
|
fi
|
|
63
63
|
|
|
64
|
-
ARCHITECT_GATE_REASON="No architect review marker found for this session. Delegate to wr-architect:agent via the Agent tool (subagent_type: 'wr-architect:agent') so the architect can review and create the marker."
|
|
64
|
+
ARCHITECT_GATE_REASON="No architect review marker found for this session. Delegate to wr-architect:agent via the Agent tool (subagent_type: 'wr-architect:agent') so the architect can review and create the marker. NOTE (P400): a verdict upgrade after an ISSUES FOUND review must be a FRESH Agent spawn of wr-architect:agent whose output leads with '**Architecture Review: PASS**' — a SendMessage resume of the same architect agent does NOT fire the marker hook, so a clean PASS from a resumed agent leaves the marker unwritten and this edit stays blocked. If you already hold a genuine PASS, assert the marker manually: touch /tmp/architect-reviewed-\$SID && rm -f /tmp/architect-reviewed-\$SID.hash (SID = newest architect-plan-reviewed-* / architect-announced-* basename)."
|
|
65
65
|
return 1 # No marker, deny
|
|
66
66
|
}
|
|
67
67
|
|
|
@@ -44,6 +44,20 @@ teardown() {
|
|
|
44
44
|
[[ "$ARCHITECT_GATE_REASON" == *"Agent tool"* ]]
|
|
45
45
|
}
|
|
46
46
|
|
|
47
|
+
# P400 — the no-marker reason must also document the SendMessage-resume
|
|
48
|
+
# recovery path: a verdict upgrade after ISSUES FOUND is a FRESH Agent spawn
|
|
49
|
+
# (a SendMessage resume does NOT fire the marker hook), or the manual marker
|
|
50
|
+
# assertion. Without this directive the deny message is a dead-end for the
|
|
51
|
+
# most common recovery flow.
|
|
52
|
+
@test "ARCHITECT_GATE_REASON documents SendMessage-resume recovery when no marker" {
|
|
53
|
+
ARCHITECT_GATE_REASON=""
|
|
54
|
+
check_architect_gate "$TEST_SESSION" || true
|
|
55
|
+
[[ "$ARCHITECT_GATE_REASON" == *"SendMessage"* ]]
|
|
56
|
+
[[ "$ARCHITECT_GATE_REASON" == *"FRESH Agent spawn"* ]]
|
|
57
|
+
[[ "$ARCHITECT_GATE_REASON" == *"touch /tmp/architect-reviewed-"* ]]
|
|
58
|
+
[[ "$ARCHITECT_GATE_REASON" == *"rm -f /tmp/architect-reviewed-"* ]]
|
|
59
|
+
}
|
|
60
|
+
|
|
47
61
|
@test "ARCHITECT_GATE_REASON names re-delegate directive when TTL expired" {
|
|
48
62
|
touch "/tmp/architect-reviewed-${TEST_SESSION}"
|
|
49
63
|
ARCHITECT_GATE_REASON=""
|