Skip to content

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

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

feat(vc): add meeting screenshot shortcut#2344
Ren1104 wants to merge 2 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

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: 3afbfbc
Source-Subject: feat(vc): add meeting screenshot shortcut
Repo: lark-cli
Synced-By: renaocheng
Timestamp: 20260813_191648Z
@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: 3d155b50-bbd5-41e0-8d98-4d3e98bbffe4

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

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

Labels

domain/vc PR touches the vc domain size/L Large or sensitive change across domains or core paths

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant