Skip to content

[codex] Polish watchlist skill guardrails#11

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

[codex] Polish watchlist skill guardrails#11
dd3ok merged 1 commit into
mainfrom
codex/watchlist-skill-polish

Conversation

@dd3ok
Copy link
Copy Markdown
Owner

@dd3ok dd3ok commented May 16, 2026

Summary

  • Narrow the watchlist skill trigger wording for pending results and scope pre-authorized recording to the current repo/workflow.
  • Clarify safe source links, due_at wording, and timezone precedence in runtime and lifecycle docs.
  • Compress the SKILL.md Add section while preserving the guardrails; Add section is now 249 words and the body is 745 words.
  • Document strict-safety conservatism and open-item field expectations in English and Korean READMEs.
  • Add regression checks for the new guardrail, concise Add section, and documentation markers.

Validation

  • python3 -m unittest discover -s evals -p 'test_*.py' — 72 tests OK
  • python3 evals/check_release_metadata.py — passed
  • python3 evals/check_policy_markers.py — passed
  • python3 evals/check_semantic_cases.py — 23 cases passed
  • python3 evals/check_watchlist.py examples/WATCHLIST.example.md --strict-format --strict-safety --require-archive-section — passed
  • python3 evals/check_watchlist.py .agents/skills/watchlist-md/assets/WATCHLIST.template.md --strict-format --strict-safety --require-archive-section — passed
  • python3 .agents/skills/watchlist-md/scripts/validate_watchlist.py .agents/skills/watchlist-md/assets/WATCHLIST.template.md --strict-format --strict-safety --require-archive-section — passed
  • git diff --check origin/main...HEAD — passed

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 and its documentation to clarify field requirements, timezone precedence for ID generation, and safety policies regarding sensitive links. Key changes include the addition of the --strict-safety flag documentation, refined validator rules for open items, and updated test cases to verify these guidelines. Feedback suggests improving consistency between the skill definition and the READMEs by explicitly listing all required fields and using precise field names like done_when instead of descriptive labels.

Comment thread .agents/skills/watchlist-md/SKILL.md Outdated
Comment thread .agents/skills/watchlist-md/SKILL.md Outdated
@dd3ok dd3ok force-pushed the codex/watchlist-skill-polish branch from de864b2 to 3ea8f30 Compare May 16, 2026 13:42
@dd3ok dd3ok force-pushed the codex/watchlist-skill-polish branch from 3ea8f30 to 63dd629 Compare May 16, 2026 14:35
@dd3ok dd3ok marked this pull request as ready for review May 16, 2026 14:50
@dd3ok dd3ok merged commit b234bd1 into main May 16, 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