Skip to content

feat(calendar): allow bot auth for search-event - #2186

Merged
calendar-assistant merged 1 commit into
mainfrom
feat/calendar-search-event-bot-auth
Aug 4, 2026
Merged

feat(calendar): allow bot auth for search-event#2186
calendar-assistant merged 1 commit into
mainfrom
feat/calendar-search-event-bot-auth

Conversation

@calendar-assistant

@calendar-assistant calendar-assistant commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

Enable bot identity for the calendar +search-event shortcut so app/bot credentials can search calendar events, not just user auth.

Changes

  • Add "bot" to the AuthTypes of the CalendarSearchEvent shortcut alongside the existing "user" type.

Test Plan

  • Unit tests pass
  • Manual local verification confirms the lark-cli calendar +search-event flow works as expected with bot auth

Related Issues

  • None

Summary by CodeRabbit

  • New Features
    • Calendar event search now supports bot authentication in addition to user authentication.

Enable bot identity for the calendar +search-event shortcut so app/bot
credentials can search calendar events, not just user auth.
@coderabbitai

coderabbitai Bot commented Aug 4, 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: 09b38423-7b84-4c60-aa1a-d4535248e1c9

📥 Commits

Reviewing files that changed from the base of the PR and between 811d37e and b558e80.

📒 Files selected for processing (1)
  • shortcuts/calendar/calendar_search_event.go

📝 Walkthrough

Walkthrough

The calendar search event shortcut now permits both user and bot authentication instead of user-only authentication.

Changes

Calendar authentication

Layer / File(s) Summary
Allow bot authentication
shortcuts/calendar/calendar_search_event.go
The shortcut configuration now includes both "user" and "bot" authentication types.

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

Suggested reviewers: liangshuo-1

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main change: adding bot authentication support to the calendar search-event shortcut.
Description check ✅ Passed The description includes all required sections with clear information about motivation, specific changes, test status, and related issues.
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 docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/calendar-search-event-bot-auth

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.

@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 4, 2026
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

npm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@b558e801421b981f8788e16ccdafea03c32ba4e8

🧩 Skill update

npx skills add larksuite/cli#feat/calendar-search-event-bot-auth -y -g

@codecov

codecov Bot commented Aug 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.72%. Comparing base (8b50cae) to head (b558e80).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2186   +/-   ##
=======================================
  Coverage   75.72%   75.72%           
=======================================
  Files         944      944           
  Lines      100355   100355           
=======================================
  Hits        75994    75994           
  Misses      18565    18565           
  Partials     5796     5796           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@calendar-assistant
calendar-assistant merged commit 3516952 into main Aug 4, 2026
28 of 41 checks passed
@calendar-assistant
calendar-assistant deleted the feat/calendar-search-event-bot-auth branch August 4, 2026 13:28
@liangshuo-1 liangshuo-1 mentioned this pull request Aug 5, 2026
3 tasks
zkh-bytedance pushed a commit that referenced this pull request Aug 6, 2026
Enable bot identity for the calendar +search-event shortcut so app/bot
credentials can search calendar events, not just user auth.

Co-authored-by: TRAE CLI <traecli@bytedance.com>
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