| 1 | # PR Credits — Thank After V2 Goes Live |
| 2 | |
| 3 | When V2 is pushed to the public repo, comment on each PR to thank the contributor and let them know their work was integrated. |
| 4 | |
| 5 | ## Integrated (cherry-picked into V2) |
| 6 | |
| 7 | | PR | Author | What | Status | |
| 8 | |---|---|---|---| |
| 9 | | [#17](https://github.com/mvanhorn/last30days-skill/pull/17) | **@JosephOIbrahim** | Windows Unicode fix (cp1252 emoji crash) | Merge or close with thanks | |
| 10 | | [#16](https://github.com/mvanhorn/last30days-skill/pull/16) | **@levineam** | Handle 403 model access errors + gpt-4.1 fallback | Merge or close with thanks | |
| 11 | | [#18](https://github.com/mvanhorn/last30days-skill/pull/18) | **@jonthebeef** | `--days=N` configurable lookback flag | Merge or close with thanks | |
| 12 | | [#1](https://github.com/mvanhorn/last30days-skill/pull/1) | **@galligan** (Matt Galligan) | Marketplace plugin conversion — we took a lighter approach inspired by his PR | Close with thanks, explain lighter approach | |
| 13 | |
| 14 | ## Already Fixed in V2 (close with thanks) |
| 15 | |
| 16 | | PR | Author | What | |
| 17 | |---|---|---| |
| 18 | | [#15](https://github.com/mvanhorn/last30days-skill/pull/15) | **@rszrszrsz** | YAML argument-hint fix — already fixed in V2 | |
| 19 | | [#11](https://github.com/mvanhorn/last30days-skill/pull/11) | **@nerveband** | Same YAML fix (earlier) — already fixed in V2 | |
| 20 | |
| 21 | ## Not Integrated (close with explanation) |
| 22 | |
| 23 | | PR | Author | What | Why | |
| 24 | |---|---|---|---| |
| 25 | | [#5](https://github.com/mvanhorn/last30days-skill/pull/5) | **@jblwilliams** | Codex auth with OpenAI Responses API | Good idea, too complex for now (358 lines SSE/JWT). May revisit. | |
| 26 | | [#14](https://github.com/mvanhorn/last30days-skill/pull/14) | **@thangman1** | WebSearch-first, API keys optional | Philosophical shift — V2 already does WebSearch in parallel | |
| 27 | | [#10](https://github.com/mvanhorn/last30days-skill/pull/10) | **@thetechreviewer** | OpenRouter API integration | Too large (1029 lines), adds MCP server | |
| 28 | |
| 29 | ## Suggested Comment Template |
| 30 | |
| 31 | > Thanks for this PR! We integrated your [fix/feature] into V2 (commit XXXXX). Really appreciate the contribution. 🙏 |
| 32 | > |
| 33 | > Closing this PR since the changes are now in main via a different commit, but full credit to you for the idea and implementation. |
| 34 |