Skip to content

fix(project-install): Handle prototype provider slugs#117006

Draft
scttcper wants to merge 1 commit into
masterfrom
scttcper/provider-slug-prototype-index
Draft

fix(project-install): Handle prototype provider slugs#117006
scttcper wants to merge 1 commit into
masterfrom
scttcper/provider-slug-prototype-index

Conversation

@scttcper

@scttcper scttcper commented Jun 5, 2026

Copy link
Copy Markdown
Member

Follow-up from #116996 while checking for other plain-object dynamic key indexes.\n\nThis updates messaging integration grouping to use own-property checks before reading provider-slug buckets. That keeps provider slugs like constructor from resolving to Object.prototype.constructor before the bucket exists.\n\nTested with:\n\npnpm test-ci static/app/views/projectInstall/issueAlertNotificationOptions.spec.tsx

Use own-property checks when grouping messaging integrations so provider slugs like constructor are treated as data keys.

Co-authored-by: Codex GPT-5 <noreply@openai.com>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jun 5, 2026
@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

Metric Before After Delta
Coverage 93.64% 93.64% ±0%
Typed 132,336 132,336 ±0
Untyped 8,986 8,987 🔴 +1
🔍 1 new type safety issue introduced

Non-null assertions (!) (1 new)

File Line Detail
static/app/views/projectInstall/issueAlertNotificationOptions.tsx 113 map[providerSlug]!

This is informational only and does not block the PR.

@getsantry

getsantry Bot commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

This pull request has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you add the label WIP, I will leave it alone unless WIP is removed ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry getsantry Bot added the Stale label Jun 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant