Skip to content

docs(mail): fix HTML reference links - #2348

Open
chenfan414-hash wants to merge 1 commit into
larksuite:mainfrom
chenfan414-hash:audit-2026-08-14-fix-mail-links
Open

docs(mail): fix HTML reference links#2348
chenfan414-hash wants to merge 1 commit into
larksuite:mainfrom
chenfan414-hash:audit-2026-08-14-fix-mail-links

Conversation

@chenfan414-hash

@chenfan414-hash chenfan414-hash commented Aug 14, 2026

Copy link
Copy Markdown

Summary

Changes

  • Change 1
  • Change 2

Test Plan

  • Unit tests pass
  • Manual local verification confirms the lark-cli <domain> <command> flow works as expected

Related Issues

  • None

Summary by CodeRabbit

  • Documentation
    • Updated email drafting, editing, forwarding, replying, and sending instructions to use explicit relative links for the HTML formatting guidelines.
    • Improved the reliability of accessing the required email formatting documentation.

@github-actions github-actions Bot added domain/mail PR touches the mail domain size/M Single-domain feat or fix with limited business impact labels Aug 14, 2026
@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: c295403e-d913-4f91-921e-4f4cf38f1942

📥 Commits

Reviewing files that changed from the base of the PR and between 0a20372 and 4dc7d33.

📒 Files selected for processing (6)
  • skills/lark-mail/references/lark-mail-draft-create.md
  • skills/lark-mail/references/lark-mail-draft-edit.md
  • skills/lark-mail/references/lark-mail-forward.md
  • skills/lark-mail/references/lark-mail-reply-all.md
  • skills/lark-mail/references/lark-mail-reply.md
  • skills/lark-mail/references/lark-mail-send.md

📝 Walkthrough

Walkthrough

Six Lark Mail reference documents now use explicit ./ relative paths for the HTML guidance document.

Changes

Lark Mail HTML reference paths

Layer / File(s) Summary
Use explicit relative HTML guidance paths
skills/lark-mail/references/lark-mail-*.md
Updated the draft, edit, forward, reply, reply-all, and send instructions to reference the HTML guidance document with an explicit relative path.

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

Merge Risk: ⚪ Minimal · up to 4dc7d

This change only corrects HTML reference links in mail documentation, with no identified impact to runtime behavior or product functionality. No actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: liangshuo-1

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description uses the required headings but retains placeholder change items and provides no actual verification details. Replace the placeholders with the specific link updates and document the verification performed, including whether manual checks were completed.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: fixing HTML reference links in mail documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ 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.

Warning

⚠️ This pull request shows signs of AI-generated slop (description_diff_mismatch). It has been flagged by CodeRabbit slop detection and should be reviewed carefully.

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

Labels

domain/mail PR touches the mail domain size/M Single-domain feat or fix with limited business impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant