[CSCITech]

⚠️ 提交说明 / PR Notice

Important

• 请提供人工撰写的简洁摘要，避免直接粘贴未经整理的 AI 输出。

📝 变更描述 / Description

(简述：做了什么？为什么这样改能生效？请基于你对代码逻辑的理解来写，避免粘贴未经整理的内容)

🚀 变更类型 / Type of change

• 🐛 Bug 修复 (Bug fix) - 请关联对应 Issue，避免将设计取舍、理解偏差或预期不一致直接归类为 bug
• ✨ 新功能 (New feature) - 重大特性建议先通过 Issue 沟通
• ⚡ 性能优化 / 重构 (Refactor)
• 📝 文档更新 (Documentation)

🔗 关联任务 / Related Issue

• Closes # (如有)

✅ 提交前检查项 / Checklist

• 人工确认: 我已亲自整理并撰写此描述，没有直接粘贴未经处理的 AI 输出。
• 非重复提交: 我已搜索现有的 Issues 与 PRs，确认不是重复提交。
• Bug fix 说明: 若此 PR 标记为 Bug fix，我已提交或关联对应 Issue，且不会将设计取舍、预期不一致或理解偏差直接归类为 bug。
• 变更理解: 我已理解这些更改的工作原理及可能影响。
• 范围聚焦: 本 PR 未包含任何与当前任务无关的代码改动。
• 本地验证: 已在本地运行并通过测试或手动验证，维护者可以据此复核结果。
• 安全合规: 代码中无敏感凭据，且符合项目代码规范。

📸 运行证明 / Proof of Work

(请在此粘贴截图、关键日志或测试报告，以证明变更生效)

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 607085f9-60d8-4315-97ce-0793b39aeb3a

📥 Commits

Reviewing files that changed from the base of the PR and between 41314a8 and 75e1359.

📒 Files selected for processing (5)

• .github/workflows/release.yml
• .gitignore
• Dockerfile
• Dockerfile.dev
• electron/package.json

💤 Files with no reviewable changes (2)

• electron/package.json
• .github/workflows/release.yml

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.

📜 Recent review details

⏰ Context from checks skipped due to timeout. (2)

• GitHub Check: Backend checks
• GitHub Check: Frontend checks

🧰 Additional context used

🪛 Checkov (3.3.16)

Dockerfile.dev

[low] 1-35: Ensure that HEALTHCHECK instructions have been added to container images

(CKV_DOCKER_2)

---

[low] 1-35: Ensure that a user for the container has been created

(CKV_DOCKER_3)

Dockerfile

[low] 1-44: Ensure that HEALTHCHECK instructions have been added to container images

(CKV_DOCKER_2)

---

[low] 1-44: Ensure that a user for the container has been created

(CKV_DOCKER_3)

🔇 Additional comments (3)

.gitignore (1)

17-18: LGTM!

Dockerfile (1)

41-41: LGTM!

Dockerfile.dev (1)

32-32: LGTM!

---

📝 Summary

Summary by CodeRabbit

• Release Packaging
  • Release downloads and packaged applications now include LICENSE and THIRD-PARTY-LICENSES.md alongside build artifacts and checksums.
  • The NOTICE file is no longer included in GitHub releases, desktop application packages, or Docker images.
• Chores
  • Updated repository ignore settings for generated project directories.

Walkthrough

The change removes NOTICE from release assets, container images, and Electron packages. It retains LICENSE and THIRD-PARTY-LICENSES.md. It also adds two .gitignore patterns.

Changes

License distribution

Layer / File(s)	Summary
Release asset lists .github/workflows/release.yml	Linux, macOS, and Windows releases no longer include NOTICE.
Container and Electron license contents Dockerfile, Dockerfile.dev, electron/package.json	Container images and Electron packages no longer include NOTICE. The other license files remain included.

Ignore rules

Layer / File(s)	Summary
Local artifact ignore rules .gitignore	The repository now ignores sub2api and CLIProxyAPI.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title "update" is too vague to identify the main change, which removes NOTICE files from release assets, Docker images, and Electron packages.	Replace the title with a specific summary, such as "Remove NOTICE from release and package assets".
Description check	❓ Inconclusive	The description contains only the PR template and does not state the actual changes, reason, change type, or validation results.	Add a concise description of the NOTICE removal from release assets and packaged images, including the reason for the change and any validation performed.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{A rabbit checks the license trail
NOTICE hops off every detail
LICENSE stays in place
With third-party text apace
Two ignored paths hide from the veil}

_{Comment @coderabbitai help to get the list of available commands.}