Skip to content

feat(vc): add meeting screenshot shortcut - #2344

Draft
Ren1104 wants to merge 14 commits into
mainfrom
meeting-final-composite-screenshot-openapi-v3
Draft

feat(vc): add meeting screenshot shortcut#2344
Ren1104 wants to merge 14 commits into
mainfrom
meeting-final-composite-screenshot-openapi-v3

Conversation

@Ren1104

@Ren1104 Ren1104 commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds vc +meeting-screenshot so an authorized user or application can save the current final composite JPEG from an ongoing recorded meeting.

Changes

  • Register the screenshot shortcut with user and application identity support.
  • Validate output paths and JPEG responses; require --overwrite before replacing an existing file.
  • Add shortcut documentation, unit tests, and dry-run E2E coverage.

Test Plan

  • Remote unit tests were not started: the shared runner timed out before it reached the CLI test lane.
  • git diff --check passed.
  • Manual CLI verification has not been run in this worktree.

Related Issues

  • None

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: e2a601b8-73b2-44d3-83f5-41190f7fab90

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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/vc PR touches the vc domain size/L Large or sensitive change across domains or core paths labels Aug 13, 2026
@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown

PR Quality Summary

CI did not complete successfully. Use the failed check links below to decide whether this PR needs a code change or a rerun.

Failed checks

@Ren1104
Ren1104 force-pushed the meeting-final-composite-screenshot-openapi-v3 branch from e435b63 to 0e6c33b Compare August 17, 2026 08:18
@github-actions github-actions Bot added domain/calendar PR touches the calendar domain domain/ccm PR touches the ccm domain domain/task PR touches the task domain size/XL Architecture-level or global-impact change and removed size/L Large or sensitive change across domains or core paths labels Aug 17, 2026
@fangshuyu-768 fangshuyu-768 removed the domain/ccm PR touches the ccm domain label Aug 18, 2026
@Ren1104
Ren1104 requested a review from zhaoleibd August 18, 2026 06:25
zhaoleibd and others added 14 commits August 18, 2026 19:44
Source-Branch: meeting-final-composite-screenshot-openapi-v3
Source-Commit: 0dd844c
Source-Subject: feat: support docs fetch selection anchors (#1815)
Repo: lark-cli
Synced-By: renaocheng
Timestamp: 20260813_191425Z
Source-Branch: meeting-final-composite-screenshot-openapi-v3
Source-Commit: 0e6c33b
Source-Subject: feat(vc): add meeting screenshot shortcut
Repo: lark-cli
Synced-By: renaocheng
Timestamp: 20260817_103207Z
Source-Branch: meeting-final-composite-screenshot-openapi-v3
Source-Commit: 650661f
Source-Subject: fix(cli): route screenshot requests to target lane
Repo: lark-cli
Synced-By: renaocheng
Timestamp: 20260817_190006Z
@Ren1104
Ren1104 force-pushed the meeting-final-composite-screenshot-openapi-v3 branch from 1a5e6dd to 3e978df Compare August 19, 2026 04:26
@github-actions github-actions Bot added the domain/ccm PR touches the ccm domain label Aug 19, 2026
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.

❌ zhaoleibd
❌ Ren1104
You have signed the CLA already but the status is still pending? Let us recheck it.

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 domain/ccm PR touches the ccm domain domain/task PR touches the task domain domain/vc PR touches the vc domain size/XL Architecture-level or global-impact change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants