Skip to content

[codex] note external review schedulers#12

Merged
dd3ok merged 1 commit into
mainfrom
codex/watchlist-skill-polish
May 17, 2026
Merged

[codex] note external review schedulers#12
dd3ok merged 1 commit into
mainfrom
codex/watchlist-skill-polish

Conversation

@dd3ok
Copy link
Copy Markdown
Owner

@dd3ok dd3ok commented May 17, 2026

Summary

  • Add a short README note that external schedulers such as cron may prompt periodic explicit WATCHLIST.md reviews.
  • Keep the scheduler outside the skill boundary and clarify it must not mutate items, run checks, or promise autonomous wakeups.
  • Add the same note to README.ko.md.

Validation

  • git diff --check origin/main...HEAD
  • PYTHONDONTWRITEBYTECODE=1 python3 -m unittest discover -s evals -p 'test_*.py'
  • python3 evals/check_policy_markers.py

@dd3ok dd3ok changed the title [codex] note external review schedulers [codex] polish watchlist guardrails and review cadence May 17, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the watchlist-md skill by standardizing field requirements, refining ID generation logic with timezone precedence, and adding safety guidelines for sensitive links. It also documents external scheduler usage and the --strict-safety flag in the README files, with corresponding updates to policy markers and tests. Review feedback suggested including ID in the list of required fields for open items in both English and Korean documentation to ensure consistency with the skill's core instructions.

Comment thread README.md
Comment thread README.ko.md
@dd3ok dd3ok force-pushed the codex/watchlist-skill-polish branch from f3c2f93 to d2b63fa Compare May 17, 2026 06:46
@dd3ok dd3ok changed the title [codex] polish watchlist guardrails and review cadence [codex] note external review schedulers May 17, 2026
@dd3ok dd3ok marked this pull request as ready for review May 17, 2026 08:57
@dd3ok dd3ok merged commit febd874 into main May 17, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant