docs: update BUGBOT#39322
Merged
Merged
Conversation
Contributor
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
|
Looks like a good structure for the BugBot.md and the rules. and the PRs are well tested |
Contributor
Builds ready [b15b1ca]
UI Startup Metrics (1305 ± 113 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
jvbriones
previously approved these changes
Jan 16, 2026
seaona
reviewed
Jan 16, 2026
gauthierpetetin
previously approved these changes
Jan 16, 2026
75d5413
Contributor
Builds ready [75d5413]
UI Startup Metrics (1306 ± 105 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
seaona
approved these changes
Jan 19, 2026
Contributor
Builds ready [745a1e9]
UI Startup Metrics (1261 ± 101 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
gauthierpetetin
approved these changes
Jan 19, 2026
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Updated:
*.test.*)test/e2e/**/*.spec.{ts,js})BUGBOT tests
This was tested in https://github.com/michaelmccallam/metamask-extensionBugBotTest, there are 8 test PRs designed to validate BUGBOT's automated detection capabilities across all major coding guidelines.
#2: Unit Testing Guidelines Violations: Tests unit testing best practices and patterns
#4: E2E Testing Guidelines Violations: Tests end-to-end testing patterns and Page Object Model
#7: Controller Guidelines Violations: Tests controller architecture and state management patterns
#8: Coding Guidelines Violations: Tests general coding standards and best practices
#10: Hooks & Effects Performance Violations: Tests Section 5.1: Hooks & Effects optimization rules
#11: React Compiler & Anti-Patterns Violations: Tests Section 5.2: React Compiler considerations and anti-patterns
#12: Rendering Performance Violations: Tests Section 5.3: Rendering performance optimization
#13: State Management & Redux Violations: Tests Section 5.4: Redux and state management patterns
Note
Modernizes BUGBOT documentation and rule configuration.
BUGBOT.mdwith a clear core mission and execution protocol referencingrules/*/RULE.md, including auto-detection patterns for each domainglobssyntax by quoting patterns across all RULE filesWritten by Cursor Bugbot for commit 745a1e9. This will update automatically on new commits. Configure here.