feat: adds bot user tracking and per-repo monitor-all mode#38
Merged
wgordon17 merged 10 commits intogordon-code:mainfrom Mar 31, 2026
Merged
feat: adds bot user tracking and per-repo monitor-all mode#38wgordon17 merged 10 commits intogordon-code:mainfrom
wgordon17 merged 10 commits intogordon-code:mainfrom
Conversation
- removes trackedSearchRepos fallback that sent involves: queries to monitored repos when all repos were monitored - extracts SEARCH_RESULT_CAP module-level constant (replaces 4 local declarations) - extracts LIGHT_ISSUE_FRAGMENT GraphQL fragment (eliminates 3rd copy of inline issue field selection) - simplifies SettingsPage onMonitorToggle lambda - types monitoredRepos prop as RepoRef[] in tab components - documents poll.ts mount-guard permanence across resetPollState() - adds 6 tests: unfiltered search error handling, all-monitored + tracked users, onLightData suppression, upstream repo toggle guard
- types fetchIssuesAndPullRequests monitoredRepos param as RepoRef[] - uses SEARCH_RESULT_CAP in pushNotification strings (removes hardcoded 1,000) - removes stale section comment header - tightens test assertion for retryable error - updates test callsites to use full RepoRef objects
- adds poll notification reset reactive effect tests (qa-5): verifies _resetNotificationState fires on monitoredRepos and trackedUsers config changes, skips on initial mount, detects key swaps - adds .max(10) constraint on monitoredRepos schema (perf-2): limits unfiltered search volume, adds guard in setMonitoredRepo - adds schema constraint tests for monitoredRepos max
wgordon17
force-pushed
the
feat/bot-tracking
branch
2 times, most recently
from
1086d16 to
abff219
Compare
wgordon17
force-pushed
the
feat/bot-tracking
branch
from
abff219 to
6e3547c
Compare
- invalidates notifications gate when monitoredRepos or trackedUsers change so next poll fetches fresh data instead of returning stale 304 - tightens hot poll PR qualifier to require enriched === true, excluding unenriched and null-checkStatus PRs from unnecessary polling - switches monitor-all eyeball icon from opacity toggle to text-info color for better enabled/disabled contrast - adds "Monitoring all" indicator with repo names on Settings page - adds regex constraints to RepoRefSchema for defense-in-depth validation - sets makePullRequest helper default to enriched: true (steady state) - adds gate bypass integration tests and Settings indicator render tests
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
1 participant
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.
Summary