Skip to content

docs(skills): verify current date for relative dates#1754

Open
godququ5-code wants to merge 1 commit into
larksuite:mainfrom
godququ5-code:fix-1752-current-date
Open

docs(skills): verify current date for relative dates#1754
godququ5-code wants to merge 1 commit into
larksuite:mainfrom
godququ5-code:fix-1752-current-date

Conversation

@godququ5-code

@godququ5-code godququ5-code commented Jul 5, 2026

Copy link
Copy Markdown

Summary

Add shared skill guidance so agents verify the current date/year before handling relative date requests. This prevents stale defaults such as assuming 2025 from examples, history, or model memory.

Changes

  • Add a 时间与年份 section to lark-shared with explicit current-date verification guidance.
  • Tell agents to use the runtime/system date, or run date when no reliable current date is available.

Test Plan

  • Unit tests pass (not run; docs/skill-only change)
  • Manual local verification confirms the lark-cli <domain> <command> flow works as expected (not applicable; no CLI behavior changed)

Validation run:

  • node scripts/skill-format-check/index.js
  • git diff --check

Related Issues

Summary by CodeRabbit

  • Documentation
    • 新增“时间与年份”说明,明确在处理“今天、明天、本周、今年、当前年份”等相对日期时,应以运行环境或系统消息中的当前日期为准。
    • 当缺少可靠当前日期时,需先获取本机日期,避免从示例、历史对话或旧资料推断年份。
    • 明确不再默认使用 2025 作为当前年份。

@CLAassistant

CLAassistant commented Jul 5, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions Bot added the size/M Single-domain feat or fix with limited business impact label Jul 5, 2026
@coderabbitai

coderabbitai Bot commented Jul 5, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Adds a new "Time and Year" documentation section to skills/lark-shared/SKILL.md instructing that relative date references must use the current date from the runtime environment or system message, falling back to the date command, and prohibiting year inference from examples, history, training data, or defaulting to 2025.

Changes

Documentation update

Layer / File(s) Summary
Time and year rule
skills/lark-shared/SKILL.md
New section instructs using runtime/system current date for relative dates, falling back to date command, and forbidding year inference from stale sources or defaulting to 2025.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • larksuite/cli#480: Also adjusts date-related guidance in SKILL.md, addressing similar documentation of date handling conventions.

Suggested reviewers: zhicong666-bytedance

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The change matches issue #1752 by adding guidance to verify the current date/year at runtime instead of assuming 2025.
Out of Scope Changes check ✅ Passed The PR appears scoped to the requested skill guidance update and does not add unrelated changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title clearly matches the main change: adding guidance to verify the current date for relative-date handling.
Description check ✅ Passed The description follows the template and includes a clear summary, change list, test plan, and related issue.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Single-domain feat or fix with limited business impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The agent defaults to taking 2025 as the current year

2 participants