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
| /* | |
| * AI-assisted refactor — SE 4900 Sprint 3, Arnold Joy | |
| * Based on XSSFWorkbook.cloneSheet at Apache POI REL_4_1_1 | |
| * | |
| * COMPLETE PROMPT CHAIN (Claude / ChatGPT session) | |
| * | |
| * PROMPT 1 | |
| * "Here is the cloneSheet(int, String) method from Apache POI's XSSFWorkbook | |
| * class at tag REL_4_1_1. Please refactor it according to the following | |
| * constraints: |
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
| /* | |
| * Manual refactor (no AI) — SE 4900 Sprint 3, Arnold Joy | |
| * Based on XSSFWorkbook.cloneSheet at Apache POI REL_4_1_1 | |
| * | |
| * Why I split it up: the original did way too much in one method, reused the same | |
| * error string everywhere, and mixed low-level byte streaming with high-level sheet calls. | |
| * I pulled out helpers for each step and dropped the extra createDrawingPatriarch() call | |
| * on the clone (POI only needs it once here). | |
| */ |