fix: 附件 URL 認不得 release asset、非 ASCII 檔名靜默相撞 - #308
Merged
Conversation
#284 idd-issue Step 4 上傳到 attachments release 並把 releases/download URL 寫進 issue body,但 URL 偵測器沒有這個 pattern,所以 idd-diagnose Step 1.5 回報「no attachments」並寫空 manifest —— 同一個 plugin 的寫入端與讀取端對 URL 形狀的認知分岔, 而且分岔是靜默的(空 manifest 與「真的沒附件」長得一模一樣)。順手收緊字元類, 裸 URL 不再吃掉整行。 #286 GitHub 會剝除 release asset 檔名的非 ASCII 字元。實測兩個中文檔名讀回只剩 ASCII 前綴;那次沒出事只因前綴剛好不同。DESC 若整段是中文(對中文使用者很自然), 兩個不同附件會 sanitize 成同名,配上 --clobber 就是第二個靜默覆蓋第一個。改為 上傳前 ASCII-fold、全被摺掉時退回序號、同名已存在則改名並印 note。
This was referenced Aug 13, 2026
Closed
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
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.
Refs #284
Refs #286
(#285 與 #287 是 #284 的重複票,已於 2026-08-14 合併結案。)
兩個缺陷都是同一個 plugin 內部的認知分岔,而且都靜默:
idd-issue寫入releases/download/…形式的 URL,process-attachments.sh的偵測器不認得它 →idd-diagnose回報「no attachments」並寫空 manifest。空 manifest 與「真的沒附件」在下游看起來完全一樣。--clobber就是靜默覆蓋。43/43 suites 綠。