@windyroad/voice-tone 0.6.0-preview.764 → 0.6.0-preview.765
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.
|
@@ -98,16 +98,15 @@ run_hook() {
|
|
|
98
98
|
[[ "$output" == *"wr-voice-tone:external-comms"* ]]
|
|
99
99
|
}
|
|
100
100
|
|
|
101
|
-
#
|
|
102
|
-
#
|
|
103
|
-
|
|
104
|
-
# delegation to the external-comms subagent (already named in the deny).
|
|
105
|
-
@test "P173 marker-absent deny clarifies the env override is pre-session" {
|
|
101
|
+
# P377/RFC-029: the BYPASS_RISK_GATE env override was removed. The only
|
|
102
|
+
# clearance path named in the deny is delegation to the external-comms subagent.
|
|
103
|
+
@test "marker-absent deny names the reviewer and offers no env override (P377/RFC-029)" {
|
|
106
104
|
INPUT=$(build_bash_input "gh issue create --title T --body 'we observed a build failure on Node 20'")
|
|
107
105
|
run_hook "$INPUT"
|
|
108
106
|
[ "$status" -eq 0 ]
|
|
109
107
|
[[ "$output" == *"deny"* ]]
|
|
110
|
-
[[ "$output" == *"
|
|
108
|
+
[[ "$output" == *"wr-voice-tone:external-comms"* ]]
|
|
109
|
+
[[ "$output" != *"BYPASS_RISK_GATE=1"* ]]
|
|
111
110
|
}
|
|
112
111
|
|
|
113
112
|
@test "voice-tone evaluator skips leak pre-filter (EXTERNAL_COMMS_LEAK_PREFILTER=no)" {
|
|
@@ -125,11 +124,14 @@ run_hook() {
|
|
|
125
124
|
[[ "$output" != *"financial"* ]]
|
|
126
125
|
}
|
|
127
126
|
|
|
128
|
-
@test "BYPASS_RISK_GATE=1
|
|
127
|
+
@test "BYPASS_RISK_GATE=1 does NOT bypass the gate (removed P377/RFC-029)" {
|
|
129
128
|
INPUT=$(build_bash_input "gh issue create --title T --body 'we observed a build failure'")
|
|
130
129
|
run bash -c "cd '$TEST_PROJECT_DIR' && BYPASS_RISK_GATE=1 printf '%s' \"\$1\" | BYPASS_RISK_GATE=1 '$HOOK'" _ "$INPUT"
|
|
131
130
|
[ "$status" -eq 0 ]
|
|
132
|
-
|
|
131
|
+
# The env override no longer short-circuits — the gate still denies the
|
|
132
|
+
# unreviewed external-comms draft.
|
|
133
|
+
[[ "$output" == *"deny"* ]]
|
|
134
|
+
[[ "$output" == *"wr-voice-tone:external-comms"* ]]
|
|
133
135
|
}
|
|
134
136
|
|
|
135
137
|
@test "per-evaluator marker (external-comms-voice-tone-reviewed-<KEY>) allows the call" {
|
|
@@ -337,11 +339,15 @@ run_hook() {
|
|
|
337
339
|
[ -z "$output" ]
|
|
338
340
|
}
|
|
339
341
|
|
|
340
|
-
@test "P082: BYPASS_RISK_GATE=1 short-
|
|
341
|
-
|
|
342
|
+
@test "P082→P377/RFC-029: BYPASS_RISK_GATE=1 is inert — does not short-circuit the gate" {
|
|
343
|
+
# gh-issue is an unconditionally-gated external surface (no visibility
|
|
344
|
+
# precondition). Setting BYPASS_RISK_GATE=1 has no effect: the unreviewed
|
|
345
|
+
# draft still denies, identically to the no-env-var case.
|
|
346
|
+
INPUT=$(build_bash_input "gh issue create --title T --body 'we observed a build failure on Node 20'")
|
|
342
347
|
run bash -c "cd '$TEST_PROJECT_DIR' && BYPASS_RISK_GATE=1 printf '%s' \"\$1\" | BYPASS_RISK_GATE=1 '$HOOK'" _ "$INPUT"
|
|
343
348
|
[ "$status" -eq 0 ]
|
|
344
|
-
[
|
|
349
|
+
[[ "$output" == *"deny"* ]]
|
|
350
|
+
[[ "$output" == *"wr-voice-tone:external-comms"* ]]
|
|
345
351
|
}
|
|
346
352
|
|
|
347
353
|
@test "P360: the skip is surface-scoped — gh-issue is NOT skipped (still denies+delegates)" {
|