Skip to content

Commit ee0764d

Browse files
author
codex-bot
committed
fix(sdk/native): keep prepublish from dirtying package.json
1 parent f3eb507 commit ee0764d

2 files changed

Lines changed: 17 additions & 1 deletion

File tree

‎sdk/native/package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@
4444
"verify:build": "node -e \"const fs=require('fs');const hasNode=fs.readdirSync('.').some(f=>f.endsWith('.node'));const hasDist=fs.existsSync('dist/index.mjs')&&fs.existsSync('dist/index.d.ts');if(!hasNode||!hasDist){console.error('Build incomplete: missing .node or dist files');process.exit(1)}console.log('✓ Build verification passed')\"",
4545
"create-npm-dirs": "napi create-npm-dirs",
4646
"artifacts": "napi artifacts",
47-
"prepublishOnly": "napi prepublish -t npm --skip-optional-publish",
47+
"prepublishOnly": "napi prepublish -t npm --skip-optional-publish && node scripts/ensure-trailing-newline.mjs package.json",
4848
"pretest": "node -e \"const fs=require('fs');const hasNode=fs.readdirSync('.').some(f=>f.endsWith('.node'));if(!hasNode){process.exitCode=1;console.error('missing .node binary; run npm run build:ci');} else {console.log('✓ native binary present; skipping rebuild');}\"",
4949
"test": "node ./scripts/run-jest.mjs --runInBand",
5050
"test:watch": "node ./scripts/run-jest.mjs --watch",
Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
import fs from "node:fs";
2+
3+
const files = process.argv.slice(2);
4+
if (files.length === 0) {
5+
throw new Error("Usage: node scripts/ensure-trailing-newline.mjs <file...>");
6+
}
7+
8+
for (const file of files) {
9+
const contents = fs.readFileSync(file, "utf8");
10+
if (contents.length === 0) {
11+
continue;
12+
}
13+
if (!contents.endsWith("\n")) {
14+
fs.writeFileSync(file, `${contents}\n`, "utf8");
15+
}
16+
}

0 commit comments

Comments
 (0)