feat/CU 86ewywc0n add staleness tracking to html reporter#4
Merged
tonytonyjan merged 2 commits intoMar 18, 2026
Merged
Conversation
Replace hidden divs with <template> elements, extract helper functions, use insertRow/insertCell for table building, and switch to hashchange event for correct hash-based routing. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
choznerol
approved these changes
Mar 17, 2026
Contributor
choznerol
left a comment
choznerol
left a comment
There was a problem hiding this comment.
LGTM. Thanks!
praise: the commits are organized and easy to review 🙏
| let staleThresholdDays = 30; | ||
|
|
||
| function getStaleThreshold(el) { | ||
| const input = el.querySelector("input[type=number]"); |
Contributor
There was a problem hiding this comment.
suggestion:
Stale threshold input has duplicate IDs across templates — #stale-threshold in tmpl-index and #stale-threshold-file in tmpl-file. The getStaleThreshold function uses el.querySelector("input[type=number]") which scopes correctly, but the IDs are inconsistent. Consider using the same ID in both templates (since only one is in the DOM at a time) or removing IDs entirely since they're not used for selection.
Suggested change
| const input = el.querySelector("input[type=number]"); | |
| const input = el.querySelector("input#stale-threshold"); |
Collaborator
Author
There was a problem hiding this comment.
Thanks! I've made the change as you suggested!
Add configurable stale threshold input, staleness percentage column, stale line count, and blue highlighting for stale lines. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
tonytonyjan
force-pushed
the
feat/CU-86ewywc0n_add-staleness-tracking-to-html-reporter
branch
from
410c5d2 to
ac87cf5
Compare
tonytonyjan
deleted the
feat/CU-86ewywc0n_add-staleness-tracking-to-html-reporter
branch
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
3 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.
Commits
Screenshots
Screen.Recording.2026-03-17.at.16.44.54.mov
Screen.Recording.2026-03-17.at.16.45.54.mov
Review Tips
Testing on your local environment
Feel free to make use of the file pstore.zip to verify the feature.