Skip to content

Commit 8ccf83b

Browse files
samuelho-devclaude
andauthored
ci: drop attw from package verification, keep publint (#43)
@arethetypeswrong/cli crashes ("Cannot read properties of undefined (reading 'filename')") decompressing this package's tarball — its core overwrites the fflate Gunzip output per chunk instead of concatenating, so any package large enough to stream in multiple chunks (ours: 341KB/92 files) loses all but the last empty chunk. That's a checker bug, not a problem with the generated package. Rather than carry a local patch + pinned dep for a third-party tool, drop attw entirely. publint remains the gating export validator. Removes the pinned @arethetypeswrong/cli dev dep, the bun patch, and the attw script/CI step added in #41/#42. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 9f2eb46 commit 8ccf83b

4 files changed

Lines changed: 14 additions & 145 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -47,17 +47,7 @@ jobs:
4747
run: bun run build
4848

4949
- name: Verify package
50-
run: |
51-
npx --yes publint
52-
53-
# Use the locally installed (pinned + patched) attw, NOT npx-latest.
54-
# Upstream @arethetypeswrong/core crashes ("Cannot read properties of
55-
# undefined (reading 'filename')") on tarballs that gunzip into multiple
56-
# chunks; fixed via patches/@arethetypeswrong%2Fcore@0.18.2.patch (upstream
57-
# PR: arethetypeswrong/arethetypeswrong.github.io#263). attw gates again —
58-
# drop the patch + pin and return to `npx --yes` once the fix is released.
59-
# ESM-only package: don't require CJS support, and ignore node10.
60-
bun run attw --pack . --profile node16 --ignore-rules cjs-resolves-to-esm
50+
run: npx --yes publint
6151

6252
coverage:
6353
name: Coverage

0 commit comments

Comments
 (0)