Skip to content

fix: guard against undefined processedExtensions + fix Playwright CI - #1870

Merged
ia3andy merged 2 commits into
quarkusio:mainfrom
ia3andy:fix-extensions-values-undefined
Sep 29, 2026
Merged

ia3andy merged 2 commits into
quarkusio:mainfrom
ia3andy:fix-extensions-values-undefined

Conversation

@ia3andy

@ia3andy ia3andy commented Sep 28, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Guard the debounced search effect against processedExtensions being undefined, fixing a TypeError: Cannot read properties of undefined (reading 'extensionsValues') reported in Sentry
  • Fix Playwright CI flakiness caused by an augmentation order race condition: the Quarkus test framework ran separate augmentations per test profile, all writing to the same Web Bundler output directory (target/web-bundler/test/dist/). When the non-Playwright augmentation ran last, it cleared the directory and deleted community-app bundle files, causing NoSuchFileException or silent render failures. Fixed by enabling community-app bundle via test resources and removing the now-unnecessary PlaywrightTestProfile (which eliminates the second augmentation entirely)

processedExtensions starts as undefined and is set asynchronously via
useEffect. The debounced search effect could fire before it was ready,
causing "Cannot read properties of undefined (reading 'extensionsValues')".

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ia3andy ia3andy changed the title fix: guard against undefined processedExtensions in search fix: guard against undefined processedExtensions + fix Playwright CI Sep 28, 2026
…dition

The Quarkus test framework runs separate augmentations per test profile,
all writing to the same Web Bundler output directory. The second
augmentation clears the first's files. When the non-Playwright
augmentation ran last, community-app bundle files were deleted,
causing NoSuchFileException or silent render failures.

Fix by:
- Enabling community-app bundle via test resources (bypasses profile
  resolution issues with BUILD_TIME map config)
- Removing PlaywrightTestProfile (no longer needed, eliminates the
  second augmentation entirely)
@ia3andy
ia3andy force-pushed the fix-extensions-values-undefined branch from 3c7d29f to f194450 Compare September 29, 2026 08:41
@ia3andy
ia3andy merged commit 7799a16 into quarkusio:main Sep 29, 2026
2 of 3 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