watch: scan the marketplace daily (was weekly)#79
Merged
Conversation
askalf
enabled auto-merge (squash)
July 20, 2026 22:22
sprayberry-reviewer
approved these changes
Jul 20, 2026
sprayberry-reviewer
left a comment
Collaborator
There was a problem hiding this comment.
Automated review from the Sprayberry Labs fleet code reviewer.
Verdict: Clean — no blocking issues found. Approving.
What I checked
gh pr diff 79— the entire change is a 2-line workflow edit in.github/workflows/marketplace-watch.yml: the cron schedule flips from weekly (17 6 * * 1, Mondays) to daily (17 6 * * *), plus the matching comment/inline-comment text update ("weekly" → "daily").- Cron field arithmetic:
17 6 * * *is a valid 5-field cron expression meaning "06:17 UTC every day" — correct for the stated intent, and the minute offset (:17) still staggers it off the top-of-hour grid as the updated inline comment claims. - CI check-rollup (
statusCheckRollup): all 9 checks green —testmatrix (ubuntu/windows/macos × node 20/22), CodeQL analyze, and the repo's own "verify pinned skills" gate all reportSUCCESS. - No code paths, permissions, or trigger conditions besides the cron string change —
workflow_dispatch: {}andpermissions:blocks are untouched.
What's good
- Minimal, single-purpose diff exactly matching the PR description (weekly → daily latency reduction for a deterministic, low-cost scan).
- Comment text was kept in sync with the behavior change (no stale "weekly" reference left behind).
- All CI signals green before review.
No correctness, security, reuse, or test-coverage concerns in the changed lines — nothing to request changes on.
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
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.
Flips marketplace-watch from weekly (Mondays 06:17) to daily (06:17). Cuts worst-case poison-detection latency 7d→1d for the live board at https://truecopy.askalf.org; cheap (public repo, deterministic scan), matches the existing daily npm-drift-watch. Comment/header wording updated to match. Site copy flips weekly→daily once this merges.
🤖 Generated with Claude Code