Skip to content

chore: filter out attributes for non-public members in CEM config#11174

Merged
web-padawan merged 1 commit intomainfrom
chore/cem-remove-protected
Mar 4, 2026
Merged

chore: filter out attributes for non-public members in CEM config#11174
web-padawan merged 1 commit intomainfrom
chore/cem-remove-protected

Conversation

@web-padawan
Copy link
Copy Markdown
Member

Summary

  • Attributes whose corresponding field member is protected or private (e.g. type from InputMixin) were not being excluded from the custom elements manifest
  • Adds a filter in the CEM config that removes attributes when their fieldName does not match any remaining public member

Test plan

  • Run yarn release:cem and verify that packages/checkbox/custom-elements.json no longer contains the type attribute
  • Verify other packages (e.g. text-field) are not affected by the change

🤖 Generated with Claude Code

Attributes whose corresponding field member is protected or private
(e.g. `type` from InputMixin) were not being excluded from the manifest.
This adds a filter that removes attributes when their `fieldName` does
not match any remaining public member.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@web-padawan web-padawan requested a review from vursen February 20, 2026 07:52
@web-padawan web-padawan changed the title fix: filter out attributes for non-public members in CEM config chore: filter out attributes for non-public members in CEM config Feb 20, 2026
@sonarqubecloud
Copy link
Copy Markdown

@web-padawan web-padawan removed the request for review from vursen March 4, 2026 14:20
@web-padawan web-padawan merged commit ec8ef63 into main Mar 4, 2026
10 checks passed
@web-padawan web-padawan deleted the chore/cem-remove-protected branch March 4, 2026 14:20
@vaadin-bot
Copy link
Copy Markdown
Collaborator

This ticket/PR has been released with Vaadin 25.1.0-beta2.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants