Commit 871e751
Fix gulp-zip import and add consistent modified time to zip (#138)
* Fix yarn zip failing after gulp-zip 6.x update
gulp-zip 6.x switched to ESM exports, so CommonJS require now returns
an object with a default property instead of the function directly.
Also added explicit modifiedTime option for consistent behavior.
Fixes DES-1353
https://claude.ai/code/session_01YXpwhKnsb98MQHXC5c5HLW
* Declare Node >=22.12.0 engine requirement
gulp-zip 6.x is pure ESM, so require('gulp-zip').default only works
on Node 22.12.0+ where synchronous require of ESM is enabled by
default. Make the requirement explicit to surface a clear error on
older Node versions instead of ERR_REQUIRE_ESM.
https://claude.ai/code/session_01YXpwhKnsb98MQHXC5c5HLW
* Remove unnecessary modifiedTime option from zip call
The modifiedTime: new Date() option was speculative and unneeded. The
gulp-zip 6.x default (preserve file stat mtimes) works correctly and
matches the effective pre-6.x behavior.
https://claude.ai/code/session_01YXpwhKnsb98MQHXC5c5HLW
---------
Co-authored-by: Claude <noreply@anthropic.com>1 parent 65b90e0 commit 871e751
2 files changed
Lines changed: 3 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
11 | | - | |
| 11 | + | |
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
| 7 | + | |
| 8 | + | |
8 | 9 | | |
9 | 10 | | |
10 | 11 | | |
| |||
0 commit comments