Skip to content

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

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

feat(vc): add meeting screenshot shortcut#2344
Ren1104 wants to merge 5 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: 9ebe30bd-97c7-4a5f-b10f-cc1266864bcb

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

deterministic-gate

  • public_content_change_id_trailercommit:fe51c19bf783:3 — public contribution contains a Change-Id trailer — Action: remove the value from the public contribution and replace it with a non-sensitive placeholder

zhaoleibd and others added 3 commits August 17, 2026 12:11
Change-Id: I3e20cf41b5a2ecd87e40af0118821e065fcbf6b1
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
@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
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
@fangshuyu-768 fangshuyu-768 removed the domain/ccm PR touches the ccm domain label Aug 18, 2026
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/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.

3 participants