为 deepseek-harness 声明截图 - #35
Conversation
|
ⓘ Qodo reviews are paused because your trial has ended. Ask your workspace admin to add credits to resume reviews. Manage billing |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthrough新增 Changes截图资源
Estimated code review effort: 1(简单)| ~2 分钟 ✨ Finishing Touches🧪 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 |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f6f91a8327
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -0,0 +1,4 @@ | |||
| [ | |||
There was a problem hiding this comment.
Include the screenshot manifest in published packages
When screenshot metadata is read from the published plugin, this new manifest is unavailable: integrations/deepseek-harness/package.json has a restrictive files list that omits screenshots.json, and the release pipeline in scripts/pack-dsh-plugin.mjs separately copies only PLUGIN_FILES, which also omits it. Consequently, both direct npm pack and npm run plugin:publish exclude the manifest, so the screenshot convention this commit implements cannot work for package consumers; add it to both publication allowlists.
Useful? React with 👍 / 👎.
变更说明
验证
这是对 awesome-dsh-plugin PR #1834 中新截图约定的跟进。
Summary by CodeRabbit