docs(base): add component development best practices to AGENTS.md#13139
Open
vladitasev wants to merge 1 commit intomainfrom
Open
docs(base): add component development best practices to AGENTS.md#13139vladitasev wants to merge 1 commit intomainfrom
vladitasev wants to merge 1 commit intomainfrom
Conversation
Add comprehensive guidance for AI coding assistants covering: - Public vs private property patterns - Avoiding hidden state in components - Counter properties anti-pattern - Boolean property defaults - Parent-child component communication - Typed slot accessors - Event binding in templates - i18n bundle naming when extending components - Lifecycle method best practices
vladitasev
requested review from
MariaIDineva,
ilhan007,
nnaydenow and
pskelin
nnaydenow
reviewed
Feb 25, 2026
| expanded = false; | ||
|
|
||
| _onToggleClick() { | ||
| // Don't change directly - fire event for app to handle |
Contributor
There was a problem hiding this comment.
should we add example like change event? where value is not passed into event details
|
|
||
| ### Typed Slot Accessors | ||
|
|
||
| Use typed slot accessors instead of `querySelector` for slotted content: |
Contributor
There was a problem hiding this comment.
maybe we can point that slots are always initialized?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add comprehensive guidance for AI coding assistants covering: