feat(docs): upload local image and source resources#1864
Open
SunPeiYang996 wants to merge 1 commit into
Open
Conversation
📝 WalkthroughWalkthroughChangesThe PR adds local image and attachment handling for Docs V2 create/update flows, including parsing, validation, upload, token binding, cleanup, dry-run support, pacing, scope enforcement, tests, and documentation. Local document resources
Estimated code review effort: 5 (Critical) | ~120 minutes Sequence Diagram(s)sequenceDiagram
participant DocsCLI
participant DocsOpenAPI
participant MediaAPI
DocsCLI->>DocsCLI: parse and rewrite local resources
DocsCLI->>DocsOpenAPI: create or update document with placeholders
DocsOpenAPI-->>DocsCLI: document response with placeholder blocks
DocsCLI->>MediaAPI: upload local files and images
MediaAPI-->>DocsCLI: return media tokens
DocsCLI->>DocsOpenAPI: bind tokens and clean placeholders
Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
github-actions
Bot
added
domain/ccm
🚀 PR Preview Install Guide🧰 CLI updatenpm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@d960f15de3f7c2ab2c5db8a273afd73f8560e93e🧩 Skill updatenpx skills add larksuite/cli#sun/lark-cli-local-resources -y -g |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1864 +/- ##
==========================================
- Coverage 74.58% 74.38% -0.21%
==========================================
Files 861 872 +11
Lines 89971 92358 +2387
==========================================
+ Hits 67106 68699 +1593
- Misses 17669 18271 +602
- Partials 5196 5388 +192 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
shortcuts/doc/doc_media_insert.go (1)
524-561: 📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick winMove recovery guidance into
.WithHint(...)for the"doc"case.Line 534 bakes recovery guidance ("use a docx token/URL or a wiki URL that resolves to docx") into the main error message instead of
.WithHint(...), while the sibling wiki/default branches keep the message fact-only. As per coding guidelines: "recovery guidance goes in.WithHint(...)".♻️ Proposed fix
case "doc": - return "", errs.NewValidationError(errs.SubtypeInvalidArgument, "this document operation only supports docx documents; use a docx token/URL or a wiki URL that resolves to docx").WithParam("--doc") + return "", errs.NewValidationError(errs.SubtypeInvalidArgument, "this document operation only supports docx documents"). + WithParam("--doc"). + WithHint("use a docx token/URL or a wiki URL that resolves to docx")🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@shortcuts/doc/doc_media_insert.go` around lines 524 - 561, Update the "doc" branch of resolveDocxDocumentID so the main validation error remains fact-focused, and move the existing recovery guidance into a chained WithHint call. Keep the existing error subtype and --doc parameter unchanged.Source: Coding guidelines
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@shortcuts/common/drive_media_upload.go`:
- Around line 8-12: Add tests around waitDriveMediaMultipartRequest covering
both MinRequestInterval pauses: prepare→part and part→finish. Verify each pause
enforces the configured interval, and add a cancellation case confirming the
wait exits promptly when the context is canceled.
In `@skills/lark-doc/references/lark-doc-update.md`:
- Around line 59-60: Keep the warning text beginning with “本地图片和附件” as one
continuous blockquote by removing the blank line inside it or adding the
blockquote marker to that line, preventing the warning from being split into
separate blocks.
In `@tests/cli_e2e/docs/coverage.md`:
- Around line 26-34: Update the command descriptions in the coverage table cells
for docs +fetch and docs +update to escape or replace the alternative separators
in “markdown|xml” and “overwrite|append”. Preserve the documented command
options while ensuring each row remains exactly six Markdown table columns and
satisfies MD056.
---
Outside diff comments:
In `@shortcuts/doc/doc_media_insert.go`:
- Around line 524-561: Update the "doc" branch of resolveDocxDocumentID so the
main validation error remains fact-focused, and move the existing recovery
guidance into a chained WithHint call. Keep the existing error subtype and --doc
parameter unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: ea3862b6-8026-45fd-872d-1e380e5b465f
📒 Files selected for processing (19)
cmd/root_test.goshortcuts/common/drive_media_upload.goshortcuts/doc/doc_media_insert.goshortcuts/doc/doc_media_upload.goshortcuts/doc/docs_create.goshortcuts/doc/docs_create_v2.goshortcuts/doc/docs_update.goshortcuts/doc/docs_update_v2.goshortcuts/doc/helpers.goshortcuts/doc/html5_block_resources.goshortcuts/doc/local_doc_resources.goshortcuts/doc/local_doc_resources_test.goskills/lark-doc/references/lark-doc-create.mdskills/lark-doc/references/lark-doc-md.mdskills/lark-doc/references/lark-doc-update.mdskills/lark-doc/references/lark-doc-xml.mdtests/cli_e2e/docs/coverage.mdtests/cli_e2e/docs/docs_local_resources_dryrun_test.gotests/cli_e2e/docs/docs_local_resources_workflow_test.go
Comment on lines
+8
to
+12
| "context" | ||
| "fmt" | ||
| "io" | ||
| "net/http" | ||
| "time" |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Search for tests covering the multipart pacing / MinRequestInterval behavior
rg -n 'MinRequestInterval\|waitDriveMediaMultipartRequest\|MinRequestInterval' --type go -g '*_test.go'Repository: larksuite/cli
Length of output: 151
🏁 Script executed:
#!/bin/bash
set -euo pipefail
# Map the target file and surrounding tests.
git ls-files 'shortcuts/common/drive_media_upload.go' 'shortcuts/common/*_test.go' 'shortcuts/**/*_test.go' | sed -n '1,200p'
echo '--- outline ---'
ast-grep outline shortcuts/common/drive_media_upload.go --view expanded
echo '--- search for pacing-related tests and symbols ---'
rg -n 'MinRequestInterval|waitDriveMediaMultipartRequest|multipart.*request|drive media upload|drive_media_upload' shortcuts -g '*_test.go' -g '*.go'Repository: larksuite/cli
Length of output: 11172
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo '--- drive_media_upload_test.go outline ---'
ast-grep outline shortcuts/common/drive_media_upload_test.go --view expanded
echo '--- drive_media_upload_typed_test.go outline ---'
ast-grep outline shortcuts/common/drive_media_upload_typed_test.go --view expanded
echo '--- relevant test snippets ---'
sed -n '1,260p' shortcuts/common/drive_media_upload_test.go
echo '--- typed tests ---'
sed -n '1,260p' shortcuts/common/drive_media_upload_typed_test.goRepository: larksuite/cli
Length of output: 17589
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo '--- all references to MinRequestInterval / waitDriveMediaMultipartRequest ---'
rg -n 'MinRequestInterval|waitDriveMediaMultipartRequest' shortcuts -g '*.go'
echo '--- continue drive_media_upload_typed_test.go ---'
sed -n '260,360p' shortcuts/common/drive_media_upload_typed_test.goRepository: larksuite/cli
Length of output: 2716
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo '--- tests that exercise doc media upload flow ---'
rg -n 'upload_prepare|upload_part|upload_finish|MinRequestInterval|localDocResourceUploadInterval|wait.*Request' shortcuts/doc -g '*_test.go'
echo '--- test files in shortcuts/doc ---'
git ls-files 'shortcuts/doc/*_test.go' | sed -n '1,200p'Repository: larksuite/cli
Length of output: 4190
Add tests for multipart pacing. Existing multipart tests cover the request bodies, but not the new MinRequestInterval wait path in waitDriveMediaMultipartRequest or its context-cancellation behavior. Add coverage for the prepare→part and part→finish pauses.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@shortcuts/common/drive_media_upload.go` around lines 8 - 12, Add tests around
waitDriveMediaMultipartRequest covering both MinRequestInterval pauses:
prepare→part and part→finish. Verify each pause enforces the configured
interval, and add a cancellation case confirming the wait exits promptly when
the context is canceled.
Source: Coding guidelines
Comment on lines
+59
to
+60
| > 本地图片和附件只允许用于 `append` 或 `block_insert_after`。`str_replace` 不会创建资源 block，而 `block_replace` / `overwrite` 一旦在后续上传绑定失败会先破坏旧内容，因此 CLI 会在写文档前拒绝这些组合。 | ||
|
|
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Keep the warning in one blockquote.
Line [60] is a blank line inside the blockquote, triggering MD028 and breaking the warning into separate blockquotes. Remove the blank line or prefix it with >.
Proposed fix
> 本地图片和附件只允许用于 `append` 或 `block_insert_after`。`str_replace` 不会创建资源 block，而 `block_replace` / `overwrite` 一旦在后续上传绑定失败会先破坏旧内容，因此 CLI 会在写文档前拒绝这些组合。
-
+>
> **匹配范围：**🧰 Tools
🪛 markdownlint-cli2 (0.22.1)
[warning] 60-60: Blank line inside blockquote
(MD028, no-blanks-blockquote)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@skills/lark-doc/references/lark-doc-update.md` around lines 59 - 60, Keep the
warning text beginning with “本地图片和附件” as one continuous blockquote by removing
the blank line inside it or adding the blockquote marker to that line,
preventing the warning from being split into separate blocks.
Source: Linters/SAST tools
Comment on lines
+26
to
+34
| | ✓ | docs +fetch | shortcut | docs_fetch_dryrun_test.go::TestDocsFetchDryRunIgnoresAPIVersionCompatFlag; docs_create_fetch_test.go::TestDocs_CreateAndFetchWorkflow/fetch as bot; docs_update_test.go::TestDocs_UpdateWorkflow/verify as bot; docs_create_fetch_test.go::TestDocs_CreateAndFetchWorkflowAsUser/fetch as user; docs_local_resources_workflow_test.go::testDocsLocalResourcesWorkflow/fetch verifies persisted resources; docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/fetch | `--doc <docToken>`; `--doc-format markdown|xml`; `--detail full`; default `extra_param.enable_user_cite_reference_map=true`; `--api-version v1` compatibility flag still dry-runs the v2 fetch endpoint | local-resource fetch asserts captions/file names persist and internal markers/paths do not leak | | ||
| | ✓ | docs +history-list | shortcut | docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/history list; docs_history_workflow_test.go::TestDocs_HistoryWorkflow | `--doc`; `--page-size`; `--page-token` | live workflow gated by `LARK_DOC_HISTORY_E2E=1` | | ||
| | ✓ | docs +history-revert | shortcut | docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/history revert; docs_history_workflow_test.go::TestDocs_HistoryWorkflow | `--doc`; `--history-version-id`; `--wait-timeout-ms` | live workflow gated by `LARK_DOC_HISTORY_E2E=1` | | ||
| | ✓ | docs +history-revert-status | shortcut | docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/history revert status; docs_history_workflow_test.go::TestDocs_HistoryWorkflow | `--doc`; `--task-id` | live workflow polls only when revert returns `running` | | ||
| | ✕ | docs +media-download | shortcut | | none | no media fixture workflow yet | | ||
| | ✕ | docs +media-insert | shortcut | | none | requires deterministic upload fixture and rollback assertions | | ||
| | ✕ | docs +media-preview | shortcut | | none | requires deterministic media fixture | | ||
| | ✕ | docs +search | shortcut | | none | search results are ambient and not yet stabilized for E2E | | ||
| | ✓ | docs +update | shortcut | docs_update_test.go::TestDocs_UpdateWorkflow/update-title-and-content as bot; docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/update | `--doc`; `--command overwrite`; `--doc-format markdown`; `--content`; optional `--reference-map` -> body `reference_map` | | | ||
| | ✓ | docs +update | shortcut | docs_update_test.go::TestDocs_UpdateWorkflow/update-title-and-content as bot; docs_local_resources_workflow_test.go::testDocsLocalResourcesWorkflow/append image and source; docs_local_resources_dryrun_test.go::TestDocs_LocalResourcesDryRun/update append; docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/update | `--doc`; `--command overwrite|append`; `--doc-format markdown|xml`; `--content`; local `<img path>` / `<source path>`; optional `--reference-map` -> body `reference_map` | local resources are covered under both bot and user identities | |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Escape or replace alternative separators inside table cells.
The literal | characters in markdown|xml and overwrite|append are interpreted as Markdown table delimiters, producing extra columns and triggering MD056. Use /, or, or escaped pipes so the command table remains six columns.
Proposed fix
-| ✓ | docs +fetch | ... | `--doc-format markdown|xml` | ...
+| ✓ | docs +fetch | ... | `--doc-format markdown or xml` | ...
-| ✓ | docs +update | ... | `--command overwrite|append`; `--doc-format markdown|xml`; ... | ...
+| ✓ | docs +update | ... | `--command overwrite or append`; `--doc-format markdown or xml`; ... | ...📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| | ✓ | docs +fetch | shortcut | docs_fetch_dryrun_test.go::TestDocsFetchDryRunIgnoresAPIVersionCompatFlag; docs_create_fetch_test.go::TestDocs_CreateAndFetchWorkflow/fetch as bot; docs_update_test.go::TestDocs_UpdateWorkflow/verify as bot; docs_create_fetch_test.go::TestDocs_CreateAndFetchWorkflowAsUser/fetch as user; docs_local_resources_workflow_test.go::testDocsLocalResourcesWorkflow/fetch verifies persisted resources; docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/fetch | `--doc <docToken>`; `--doc-format markdown|xml`; `--detail full`; default `extra_param.enable_user_cite_reference_map=true`; `--api-version v1` compatibility flag still dry-runs the v2 fetch endpoint | local-resource fetch asserts captions/file names persist and internal markers/paths do not leak | | |
| | ✓ | docs +history-list | shortcut | docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/history list; docs_history_workflow_test.go::TestDocs_HistoryWorkflow | `--doc`; `--page-size`; `--page-token` | live workflow gated by `LARK_DOC_HISTORY_E2E=1` | | |
| | ✓ | docs +history-revert | shortcut | docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/history revert; docs_history_workflow_test.go::TestDocs_HistoryWorkflow | `--doc`; `--history-version-id`; `--wait-timeout-ms` | live workflow gated by `LARK_DOC_HISTORY_E2E=1` | | |
| | ✓ | docs +history-revert-status | shortcut | docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/history revert status; docs_history_workflow_test.go::TestDocs_HistoryWorkflow | `--doc`; `--task-id` | live workflow polls only when revert returns `running` | | |
| | ✕ | docs +media-download | shortcut | | none | no media fixture workflow yet | | |
| | ✕ | docs +media-insert | shortcut | | none | requires deterministic upload fixture and rollback assertions | | |
| | ✕ | docs +media-preview | shortcut | | none | requires deterministic media fixture | | |
| | ✕ | docs +search | shortcut | | none | search results are ambient and not yet stabilized for E2E | | |
| | ✓ | docs +update | shortcut | docs_update_test.go::TestDocs_UpdateWorkflow/update-title-and-content as bot; docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/update | `--doc`; `--command overwrite`; `--doc-format markdown`; `--content`; optional `--reference-map` -> body `reference_map` | | | |
| | ✓ | docs +update | shortcut | docs_update_test.go::TestDocs_UpdateWorkflow/update-title-and-content as bot; docs_local_resources_workflow_test.go::testDocsLocalResourcesWorkflow/append image and source; docs_local_resources_dryrun_test.go::TestDocs_LocalResourcesDryRun/update append; docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/update | `--doc`; `--command overwrite|append`; `--doc-format markdown|xml`; `--content`; local `<img path>` / `<source path>`; optional `--reference-map` -> body `reference_map` | local resources are covered under both bot and user identities | | |
| | ✓ | docs +fetch | shortcut | docs_fetch_dryrun_test.go::TestDocsFetchDryRunIgnoresAPIVersionCompatFlag; docs_create_fetch_test.go::TestDocs_CreateAndFetchWorkflow/fetch as bot; docs_update_test.go::TestDocs_UpdateWorkflow/verify as bot; docs_create_fetch_test.go::TestDocs_CreateAndFetchWorkflowAsUser/fetch as user; docs_local_resources_workflow_test.go::testDocsLocalResourcesWorkflow/fetch verifies persisted resources; docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/fetch | `--doc <docToken>`; `--doc-format markdown or xml`; `--detail full`; default `extra_param.enable_user_cite_reference_map=true`; `--api-version v1` compatibility flag still dry-runs the v2 fetch endpoint | local-resource fetch asserts captions/file names persist and internal markers/paths do not leak | | |
| | ✓ | docs +history-list | shortcut | docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/history list; docs_history_workflow_test.go::TestDocs_HistoryWorkflow | `--doc`; `--page-size`; `--page-token` | live workflow gated by `LARK_DOC_HISTORY_E2E=1` | | |
| | ✓ | docs +history-revert | shortcut | docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/history revert; docs_history_workflow_test.go::TestDocs_HistoryWorkflow | `--doc`; `--history-version-id`; `--wait-timeout-ms` | live workflow gated by `LARK_DOC_HISTORY_E2E=1` | | |
| | ✓ | docs +history-revert-status | shortcut | docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/history revert status; docs_history_workflow_test.go::TestDocs_HistoryWorkflow | `--doc`; `--task-id` | live workflow polls only when revert returns `running` | | |
| | ✕ | docs +media-download | shortcut | | none | no media fixture workflow yet | | |
| | ✕ | docs +media-insert | shortcut | | none | requires deterministic upload fixture and rollback assertions | | |
| | ✕ | docs +media-preview | shortcut | | none | requires deterministic media fixture | | |
| | ✕ | docs +search | shortcut | | none | search results are ambient and not yet stabilized for E2E | | |
| | ✓ | docs +update | shortcut | docs_update_test.go::TestDocs_UpdateWorkflow/update-title-and-content as bot; docs_local_resources_workflow_test.go::testDocsLocalResourcesWorkflow/append image and source; docs_local_resources_dryrun_test.go::TestDocs_LocalResourcesDryRun/update append; docs_update_dryrun_test.go::TestDocs_DryRunDefaultsToV2OpenAPI/update | `--doc`; `--command overwrite or append`; `--doc-format markdown or xml`; `--content`; local `<img path>` / `<source path>`; optional `--reference-map` -> body `reference_map` | local resources are covered under both bot and user identities | |
🧰 Tools
🪛 markdownlint-cli2 (0.22.1)
[warning] 26-26: Spaces inside code span elements
(MD038, no-space-in-code)
[warning] 26-26: Spaces inside code span elements
(MD038, no-space-in-code)
[warning] 26-26: Spaces inside code span elements
(MD038, no-space-in-code)
[warning] 26-26: Table column count
Expected: 6; Actual: 7; Too many cells, extra data will be missing
(MD056, table-column-count)
[warning] 34-34: Spaces inside code span elements
(MD038, no-space-in-code)
[warning] 34-34: Spaces inside code span elements
(MD038, no-space-in-code)
[warning] 34-34: Spaces inside code span elements
(MD038, no-space-in-code)
[warning] 34-34: Spaces inside code span elements
(MD038, no-space-in-code)
[warning] 34-34: Images should have alternate text (alt text)
(MD045, no-alt-text)
[warning] 34-34: Table column count
Expected: 6; Actual: 8; Too many cells, extra data will be missing
(MD056, table-column-count)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@tests/cli_e2e/docs/coverage.md` around lines 26 - 34, Update the command
descriptions in the coverage table cells for docs +fetch and docs +update to
escape or replace the alternative separators in “markdown|xml” and
“overwrite|append”. Preserve the documented command options while ensuring each
row remains exactly six Markdown table columns and satisfies MD056.
Source: Linters/SAST tools
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add first-class local resource support to
lark-cli docsv2:<img path="@relative/path.png">and<source path="@relative/file.pdf">plus<source path="@relative/file.pdf">docs +createanddocs +update, including append andblock_insert_afterThe CLI validates local resources, sends only opaque placeholder markers to Docs AI, receives the new
block_id/block_type, uploads the real media, and binds the uploaded token to the placeholder block. Real local paths are never persisted or returned by the document service.What changed
lark-docskill documentation and CLI docs.Safety and compatibility
@lcli_img_<32hex>and@lcli_file_<32hex>.Cross-repository dependency chain
Recommended merge/deploy order:
<source name>during XML export: https://code.byted.org/larkdoc/docx_engine/merge_requests/2928v0.0.62-0.20260713111932-3aa1d6116278: https://code.byted.org/ccm-platform/ai_edit/merge_requests/176PPE versions used for acceptance:
creation/docx/engine 1.0.0.9466creation/platform/ai_edit 1.0.0.5743aa1d6116278ppe_sun_ai_testVerification
Pre-merge validation:
make vet build unit-test— passmake fmt-check script-test— pass, including 103 script testsgo mod tidy— no diffgolangci-lint— 0 findings)True PPE E2E:
3380003Full report: https://bytedance.larkoffice.com/docx/LFg3db8YZo5jh3xqSVhcnanHnYb
Rollback
Revert this PR to remove CLI-side local resource detection/upload/bind behavior. The feature has no persistent client migration. If the server chain is also rolled back, revert in reverse order: ai_edit, SDK, then engine. Documents already created contain normal image/file blocks and remain readable.
Summary by CodeRabbit
New Features
Bug Fixes
Documentation