Skip to content

fix: Test: 自動加入看板功能#25

Closed
Pr0gCat wants to merge 2 commits intomainfrom
haunted/issue-10
Closed

fix: Test: 自動加入看板功能#25
Pr0gCat wants to merge 2 commits intomainfrom
haunted/issue-10

Conversation

@Pr0gCat
Copy link
Copy Markdown
Owner

@Pr0gCat Pr0gCat commented Dec 31, 2025

Summary

此 PR 實作了「自動加入看板功能」,當新 issue 被建立時,會自動加入到指定的 GitHub Project 看板中。這個功能是非阻塞式的,即使加入失敗也不會影響其他 issue 處理流程。

Changes

  • src/config/schema.ts: 在 ProjectConfigSchema 中新增三個設定欄位:
    • number: GitHub Project 編號
    • owner: GitHub Project 擁有者(用戶或組織名稱)
    • auto_add_issues: 是否自動將新 issue 加入 project(預設為 true)

Closes #10


🤖 Generated by Haunted AI

Test: 自動加入看板功能

Closes #10

🤖 Generated by Haunted AI
@Pr0gCat
Copy link
Copy Markdown
Owner Author

Pr0gCat commented Dec 31, 2025

[HouseMaster Review]

這是一個設計良好的 PR,實作了非阻塞式的自動加入看板功能。Schema 正確新增了 numberownerauto_add_issues 三個設定欄位,並提供了合理的預設值。測試覆蓋率完整,涵蓋了各種配置組合和錯誤處理情況。程式碼品質良好,遵循了專案的既有模式。雖然有一些小的改進建議(如測試穩定性和 GraphQL 安全性),但這些都不影響功能的正確性,可以在後續迭代中改進。批准合併。

Suggestions:

  • 考慮將測試中的固定 timeout (setTimeout(resolve, 10)) 改為使用 vi.waitFor() 來避免潛在的 flaky tests。
  • 未來可考慮在 issue 成功加入 project 後,自動將其移動到預設的 column(如 Backlog)以保持一致性。
  • 建議在 listProjects 函數中使用 GraphQL 變數 ($owner: String!) 而不是直接字串插值。

2 similar comments
@Pr0gCat
Copy link
Copy Markdown
Owner Author

Pr0gCat commented Dec 31, 2025

[HouseMaster Review]

這是一個設計良好的 PR,實作了非阻塞式的自動加入看板功能。Schema 正確新增了 numberownerauto_add_issues 三個設定欄位,並提供了合理的預設值。測試覆蓋率完整,涵蓋了各種配置組合和錯誤處理情況。程式碼品質良好,遵循了專案的既有模式。雖然有一些小的改進建議(如測試穩定性和 GraphQL 安全性),但這些都不影響功能的正確性,可以在後續迭代中改進。批准合併。

Suggestions:

  • 考慮將測試中的固定 timeout (setTimeout(resolve, 10)) 改為使用 vi.waitFor() 來避免潛在的 flaky tests。
  • 未來可考慮在 issue 成功加入 project 後,自動將其移動到預設的 column(如 Backlog)以保持一致性。
  • 建議在 listProjects 函數中使用 GraphQL 變數 ($owner: String!) 而不是直接字串插值。

@Pr0gCat
Copy link
Copy Markdown
Owner Author

Pr0gCat commented Dec 31, 2025

[HouseMaster Review]

這是一個設計良好的 PR,實作了非阻塞式的自動加入看板功能。Schema 正確新增了 numberownerauto_add_issues 三個設定欄位,並提供了合理的預設值。測試覆蓋率完整,涵蓋了各種配置組合和錯誤處理情況。程式碼品質良好,遵循了專案的既有模式。雖然有一些小的改進建議(如測試穩定性和 GraphQL 安全性),但這些都不影響功能的正確性,可以在後續迭代中改進。批准合併。

Suggestions:

  • 考慮將測試中的固定 timeout (setTimeout(resolve, 10)) 改為使用 vi.waitFor() 來避免潛在的 flaky tests。
  • 未來可考慮在 issue 成功加入 project 後,自動將其移動到預設的 column(如 Backlog)以保持一致性。
  • 建議在 listProjects 函數中使用 GraphQL 變數 ($owner: String!) 而不是直接字串插值。

**[HouseMaster Review]**  這是一個設計良好的 PR,實作了非阻塞式的自動加...

🤖 Generated by Haunted AI
@Pr0gCat
Copy link
Copy Markdown
Owner Author

Pr0gCat commented Dec 31, 2025

[Haunted]

Applied revisions based on your feedback.

Changes:

  • src/events/handlers/issue.test.ts
  • src/github/projects.ts

@Pr0gCat Pr0gCat closed this Dec 31, 2025
@Pr0gCat Pr0gCat deleted the haunted/issue-10 branch December 31, 2025 15:16
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.

Test: 自動加入看板功能

1 participant