[quality] the OWNERS filters accepted-but-ignored arm (owners.ts parseOwners) is never driven through dist/index.js #364

Finding

src/utils/owners.ts parseOwners (line 46) accepts an OWNERS filters section and ignores it with a debug line, exactly as the docs promise (docs/prow-for-your-org.md:41 — "Accepted but ignored (debug-logged)"; docs/commands.md:321). That promise has unit coverage only:

• Unit: npx vitest run --coverage on main @ 3fc21f2 — line 46 covered by __tests__/utils/owners.test.ts ("notes that filters are ignored"), which calls parseOwners directly.
• E2E: npm run test:coverage:e2e (the __tests__/bundle suite against a source-mapped bundle, per [quality] the bundle e2e suite's coverage of src/ is never captured — runBundle.ts drops NODE_V8_COVERAGE and dist/ has no source map #235/test(coverage): add test:coverage:e2e to report the bundle suite's coverage of src/ #274) on main @ 3fc21f2 plus all 44 open held PR heads merged (chore(coverage): emit lcovonly and json-summary reporters from vitest.config.mjs #230..test(bundle): drive syncApprovalReview's not-open and nothing-to-dismiss short-circuits and withdrawalReason's no-files and lgtm-skip arms through dist/index.js #363, every merge clean; 47 files / 428 tests pass): owners.ts:46 has 0 hits. No open PR serves an OWNERS file carrying filters or asserts the debug line.

So nothing exercises, through the shipped dist/index.js, that a real OWNERS blob with filters (a) still authorizes its top-level approvers and (b) grants nothing to a login listed only inside a filter — the behaviour a Prow user migrating OWNERS files would be surprised by.

Recommendation

• Add __tests__/bundle/ownersFilters.test.ts: route one OWNERS blob (approvers: [alice], filters: { '.*': { approvers: [bob] } }) via routeOwners and drive /approve through dist/index.js twice: alice gets approved and stdout carries OWNERS at OWNERS: filters are not supported; ignoring; bob is refused with bob is not an approver for any changed file and no label write.

Priority

• Impact: medium — unit-covered, not e2e-covered (priority 2 coverage-gap)
• Effort: low

Filed by quality agent (hold-gated mode)

🐝 Hive Agent: quality | Instance: hosted-available-lke648397-260827-5q9t | SHA: 3fc21f2

— hive: agent=quality backend=copilot model=claude-fable-5.1 copilot=1.0.88