Skip to content

fix: enforce zone policy after phase output - #689

Open
ApexWorm wants to merge 1 commit into
peteromallet:mainfrom
ApexWorm:fix/generated-zone-phase-filter
Open

fix: enforce zone policy after phase output#689
ApexWorm wants to merge 1 commit into
peteromallet:mainfrom
ApexWorm:fix/generated-zone-phase-filter

Conversation

@ApexWorm

@ApexWorm ApexWorm commented Aug 1, 2026

Copy link
Copy Markdown

Problem

Generated migrations are classified correctly, and generated-zone policy skips smell and structural detectors. However, some phase output was admitted after individual phase filtering, so generated migration findings could persist and enter the execution queue.

Fix

Apply one normalized issue-level zone-policy gate after phase execution and before issue context is stamped. Add a queue-level defense for historical persisted findings whose stored zone and detector are already policy-skipped.

Verification

  • 40 queue and plan tests passed
  • 23 zone-policy tests passed
  • Agent verification: 63 focused tests passed
  • Agent verification: full suite had 5,812 passed and 4 skipped; two unrelated existing duplicated review-prompt assertions failed
  • Ruff and diff whitespace checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant