Skip to content

fix: prevent PM wizard request storm and auth 401s#1167

Merged
zbigniewsobiecki merged 1 commit intodevfrom
fix/pm-wizard-storm-and-401
Apr 23, 2026
Merged

fix: prevent PM wizard request storm and auth 401s#1167
zbigniewsobiecki merged 1 commit intodevfrom
fix/pm-wizard-storm-and-401

Conversation

@zbigniewsobiecki
Copy link
Copy Markdown
Member

@zbigniewsobiecki zbigniewsobiecki commented Apr 23, 2026

Summary

  • map PM discovery provider auth failures to tRPC UNAUTHORIZED
  • call manifest provider hooks once per provider instead of once per wizard step
  • add regression coverage for auth mapping and hook storm guard

Verification

  • npm run lint
  • npm run typecheck
  • npm test
  • npm run build:web

@zbigniewsobiecki zbigniewsobiecki merged commit 2b1c8a6 into dev Apr 23, 2026
7 checks passed
@zbigniewsobiecki zbigniewsobiecki deleted the fix/pm-wizard-storm-and-401 branch April 23, 2026 10:58
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 23, 2026

Codecov Report

❌ Patch coverage is 92.30769% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/api/routers/pm-discovery.ts 92.30% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@zbigniewsobiecki zbigniewsobiecki mentioned this pull request Apr 23, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@zbigniewsobiecki