Skip to content

docs(calendar): document meeting minute creation - #2170

Open
mushanyoung wants to merge 1 commit into
larksuite:mainfrom
mushanyoung:docs/calendar-meeting-minutes
Open

docs(calendar): document meeting minute creation#2170
mushanyoung wants to merge 1 commit into
larksuite:mainfrom
mushanyoung:docs/calendar-meeting-minutes

Conversation

@mushanyoung

@mushanyoung mushanyoung commented Aug 3, 2026

Copy link
Copy Markdown

Summary

Document the supported workflow and capability boundary for creating native meeting-minute documents from Calendar events. This helps agents use the official raw Calendar OpenAPI when no typed command exists, while avoiding unsupported claims about binding arbitrary existing Doc/Wiki documents.

Changes

  • Route meeting-minute creation and stale meeting_note diagnosis to the existing Calendar meeting reference.
  • Document the official raw API endpoint, prerequisites, scopes, request shape, and data.doc_url response.
  • Explain the current API limitation and supported alternatives for requests to associate an existing document.
  • Keep unstable public calendar-share page scraping and tenant-specific troubleshooting out of scope.

Test Plan

  • make unit-test passes in an isolated CI-equivalent environment.
  • go vet ./... passes.
  • gofmt -l . produces no output.
  • go mod tidy leaves go.mod and go.sum unchanged.
  • go run github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.1.6 run --new-from-rev=origin/main reports 0 issues.
  • Manual lark-cli api ... --dry-run verification confirms the POST method and meeting-minute endpoint path.

Related Issues

  • None

Summary by CodeRabbit

  • Documentation
    • Added guidance for creating native meeting notes from calendar events.
    • Added troubleshooting steps for invalid meeting note references.
    • Documented required permissions, conditions, and limitations for creating or linking meeting minutes.
    • Added alternative handling for requests to associate existing documents.

@CLAassistant

CLAassistant commented Aug 3, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

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

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Changes

Calendar meeting notes

Layer / File(s) Summary
Meeting-note routing and validation
skills/lark-calendar/SKILL.md, skills/lark-calendar/references/lark-calendar-meeting.md
Meeting-note creation and invalid association requests route to the meeting reference. The reference validates meeting_note documents and deleted associations.
Meeting-minutes creation and document handling
skills/lark-calendar/references/lark-calendar-meeting.md
The reference documents native meeting-minutes creation through Calendar OpenAPI, required conditions, and handling for existing documents that cannot be bound directly.

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

Suggested reviewers: liangshuo-1

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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.
Title check ✅ Passed The title clearly and concisely describes the documentation change for meeting-minute creation.
Description check ✅ Passed The description covers the summary, changes, verification steps, and related issues with relevant implementation details.
✨ 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

domain/calendar PR touches the calendar 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.

2 participants