Skip to content

Commit 304613e

Browse files
rom1504claude
andcommitted
Use sync gzip, remove bash retry workaround
All zlib calls in the dependency chain now use sync versions, eliminating the uncaught async errors. No need for process-level retries. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent c72ea65 commit 304613e

2 files changed

Lines changed: 3 additions & 9 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -52,9 +52,4 @@ jobs:
5252
- name: Install Dependencies
5353
run: npm install
5454
- name: Start Tests
55-
run: |
56-
for attempt in 1 2 3; do
57-
npm run mocha_test -- -g ${{ matrix.mcVersion }}v && exit 0
58-
echo "Attempt $attempt failed, retrying..."
59-
done
60-
exit 1
55+
run: npm run mocha_test -- -g ${{ matrix.mcVersion }}v

src/lib/playerDat.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,8 @@ const zlib = require('zlib')
88
const { promisify } = require('util')
99
const convertInventorySlotId = require('./convertInventorySlotId')
1010

11-
const _gzip = promisify(zlib.gzip)
12-
async function gzip (data) {
13-
try { return await _gzip(data) } catch (err) { throw new Error('gzip failed: ' + err.message) }
11+
function gzip (data) {
12+
return zlib.gzipSync(data)
1413
}
1514

1615
const nbtParse = promisify(nbt.parse)

0 commit comments

Comments
 (0)