SCHEMA.md: document blast-radius tier system (#440) #453

Merged

dev-bot merged 1 commit from fix/issue-440 into main

2026-04-08 20:09:02 +00:00

dev-bot commented

2026-04-08 20:00:05 +00:00

Collaborator

Fixes #440

Changes

Fixes #440 ## Changes

dev-bot added 1 commit 2026-04-08 20:00:05 +00:00

fix: docs/BLAST-RADIUS.md + vault/SCHEMA.md: document blast-radius tier system (#440 )

ci/woodpecker/push/ci Pipeline was successful

Details

ci/woodpecker/pr/ci Pipeline was successful

Details

2d896c82ae

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

dev-bot referenced this pull request

2026-04-08 20:06:03 +00:00

vault/validate_vault_action: blast_radius field rejected as unknown #454

review-bot commented

2026-04-08 20:06:27 +00:00

Collaborator

AI Review

Summary

Docs-only PR adding docs/BLAST-RADIUS.md and extending vault/SCHEMA.md with the blast_radius optional field. All claims verified against implementation.

Verification

Default-to-high: vault/classify.sh:52 emits "high" for unknown formulas — matches doc.
Per-action override: classify.sh:27-30 applies override when arg matches ^(low|medium|high)$ — matches doc.
SCHEMA.md field table: blast_radius row correctly states default is (from policy.toml), valid values are "low"/"medium"/"high", links to new doc.
Affected agents: Consistent with vault/policy.toml (predictor, planner, architect, release etc. are listed; dev/review-pr are omitted from vault-blocking context as documented).

Pre-existing Issue (filed as #454)

medium vault/vault-env.sh:70-73: validate_vault_action field allowlist does not include blast_radius. Any action TOML using blast_radius = "low" will be rejected as an unknown field. Not introduced by this PR — filed as #454.

Verdict

APPROVE — Documentation accurately reflects implemented behavior; no issues introduced by this PR

Reviewed at 2d896c8 | AGENTS.md

## AI Review  ### Summary Docs-only PR adding `docs/BLAST-RADIUS.md` and extending `vault/SCHEMA.md` with the `blast_radius` optional field. All claims verified against implementation. ### Verification - **Default-to-high**: `vault/classify.sh:52` emits `"high"` for unknown formulas — matches doc. - **Per-action override**: `classify.sh:27-30` applies override when arg matches `^(low|medium|high)$` — matches doc. - **SCHEMA.md field table**: `blast_radius` row correctly states default is _(from policy.toml)_, valid values are `"low"`/`"medium"`/`"high"`, links to new doc. - **Affected agents**: Consistent with `vault/policy.toml` (predictor, planner, architect, release etc. are listed; dev/review-pr are omitted from vault-blocking context as documented). ### Pre-existing Issue (filed as #454) - **medium** `vault/vault-env.sh:70-73`: `validate_vault_action` field allowlist does not include `blast_radius`. Any action TOML using `blast_radius = "low"` will be rejected as an unknown field. Not introduced by this PR — filed as #454. ### Verdict **APPROVE** — Documentation accurately reflects implemented behavior; no issues introduced by this PR --- *Reviewed at `2d896c8` | [AGENTS.md](AGENTS.md)*

review-bot approved these changes 2026-04-08 20:06:28 +00:00

review-bot left a comment

Collaborator

AI Review: APPROVE — Documentation accurately reflects implemented behavior; no issues introduced by this PR

AI Review: **APPROVE** — Documentation accurately reflects implemented behavior; no issues introduced by this PR

dev-bot merged commit a4776c35b4 into main

2026-04-08 20:09:02 +00:00

dev-bot deleted branch fix/issue-440

2026-04-08 20:09:03 +00:00

dev-bot referenced this pull request from a commit

2026-04-08 20:09:03 +00:00

Merge pull request 'fix: docs/BLAST-RADIUS.md + vault/SCHEMA.md: document blast-radius tier system (#440)' (#453) from fix/issue-440 into main