Skip to content

Conversation

@shakebbq-dot
Copy link

Pull Request | PR 提交

📋 选择专用模板 | Choose Specialized Template

我们现在提供了针对不同类型PR的专用模板,帮助你更快速地填写PR信息:
We now offer specialized templates for different types of PRs to help you fill out the information faster:

如何使用?| How to use?

  • 创建PR时,在URL中添加 ?template=backend.md 或其他模板名称
  • When creating a PR, add ?template=backend.md or other template name to the URL
  • 或者直接复制粘贴对应模板的内容
  • Or simply copy and paste the content from the corresponding template

💡 提示 Tip: 推荐 PR 标题格式 type(scope): description
例如: feat(trader): add new strategy | fix(api): resolve auth issue


📝 Description | 描述

English:中文:


🎯 Type of Change | 变更类型

  • 🐛 Bug fix | 修复 Bug
  • ✨ New feature | 新功能
  • 💥 Breaking change | 破坏性变更
  • 📝 Documentation update | 文档更新
  • 🎨 Code style update | 代码样式更新
  • ♻️ Refactoring | 重构
  • ⚡ Performance improvement | 性能优化
  • ✅ Test update | 测试更新
  • 🔧 Build/config change | 构建/配置变更
  • 🔒 Security fix | 安全修复

🔗 Related Issues | 相关 Issue

  • Closes # | 关闭 #
  • Related to # | 相关 #

📋 Changes Made | 具体变更

English:中文:


🧪 Testing | 测试

  • Tested locally | 本地测试通过
  • Tests pass | 测试通过
  • Verified no existing functionality broke | 确认没有破坏现有功能

✅ Checklist | 检查清单

Code Quality | 代码质量

  • Code follows project style | 代码遵循项目风格
  • Self-review completed | 已完成代码自查
  • Comments added for complex logic | 已添加必要注释

Documentation | 文档

  • Updated relevant documentation | 已更新相关文档

Git

  • Commits follow conventional format | 提交遵循 Conventional Commits 格式
  • Rebased on latest dev branch | 已 rebase 到最新 dev 分支
  • No merge conflicts | 无合并冲突

📚 Additional Notes | 补充说明

English:中文:


By submitting this PR, I confirm | 提交此 PR,我确认:

  • I have read the Contributing Guidelines | 已阅读贡献指南
  • I agree to the Code of Conduct | 同意行为准则
  • My contribution is licensed under AGPL-3.0 | 贡献遵循 AGPL-3.0 许可证

🌟 Thank you for your contribution! | 感谢你的贡献!

@the-dev-z
Copy link
Owner

❌ PR 無法接受 - 嚴重問題

感謝你的貢獻,但這個 PR 存在多個嚴重問題,無法合併:

🚨 安全問題

1. 洩露 JWT Secret

"jwt_secret": "71b73ixQ03ofao0iwl255QaJhwAp88MULkKeY3qyasseEZPlLFZzYm5AFB918GtS"
  • ❌ 你提交了 config.json 文件,包含真實的 JWT secret
  • ⚠️ 這是嚴重的安全漏洞 - 任何人都可以看到你的密鑰
  • ✅ 正確做法:只提交 config.json.example,真實配置應該在 .gitignore

📝 PR 規範問題

2. 缺少標題和描述

  • ❌ 標題只有 "1"
  • ❌ PR 描述完全空白
  • ✅ 正確做法:使用有意義的標題(如 fix: resolve line ending issues)並填寫 PR 模板

3. 錯誤的 Base Branch

  • z-dev-v2 -> z-dev-v2 (合併到自己)
  • ✅ 正確做法:Fork 後應該是 your-fork:feature-branch -> upstream:z-dev-v2

4. 無意義的變更

  • ❌ 9,550 additions / 9,178 deletions
  • ❌ 所有變更都只是換行符變更(CRLF ↔ LF)
  • ❌ 沒有實際功能改進
  • ✅ 正確做法:
    • 配置 .gitattributes 統一換行符
    • 不要提交純格式化變更
    • 專注於實際功能改進

🛡️ 安全建議

立即行動:

  1. ⚠️ 輪換你的 JWT Secret - 已洩露的密鑰不能再使用
  2. 🗑️ 刪除你本地的 config.json,使用 config.json.example 複製一份
  3. ✅ 確保 config.json.gitignore 中(已有)

📚 如何正確貢獻

請參考我們的貢獻指南:

如果你想修復實際問題:

  1. 創建新的分支(不是 z-dev-v2)
  2. 做有意義的變更(功能、bug 修復等)
  3. 填寫完整的 PR 描述
  4. 確保不包含敏感信息

此 PR 將被關閉。 如有疑問,歡迎在 Discussions 中討論。

Thanks for understanding! | 感謝理解!

🤖 Reviewed by Claude Code

@the-dev-z
Copy link
Owner

關閉原因:包含安全漏洞(JWT secret 洩露)+ 無意義的格式化變更 + 缺少 PR 描述。請參考上方評論了解詳情。

@the-dev-z the-dev-z closed this Nov 20, 2025
the-dev-z pushed a commit that referenced this pull request Nov 24, 2025
fix(decision): 修复 AI 开仓保证金计算问题
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants