Skip to content

Commit d6c92af

Browse files
suryaiyer95claude
andcommitted
fix: copy node_python_bridge.py in publish.ts so npm tarball includes it
`copyAssets` copied `dbt-tools/dist/index.js` but not the `.py` file the patched `__dirname` resolves to. Without this, the fix would still break in published packages. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent da04c8e commit d6c92af

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

packages/opencode/script/publish.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,9 @@ async function copyAssets(targetDir: string) {
6565
await $`cp ../dbt-tools/bin/altimate-dbt ${targetDir}/dbt-tools/bin/altimate-dbt`
6666
await $`mkdir -p ${targetDir}/dbt-tools/dist`
6767
await $`cp ../dbt-tools/dist/index.js ${targetDir}/dbt-tools/dist/`
68+
// node_python_bridge.py must live next to index.js — the patched __dirname
69+
// resolves to this directory at runtime (see copy-python.ts)
70+
await $`cp ../dbt-tools/dist/node_python_bridge.py ${targetDir}/dbt-tools/dist/`
6871
// A package.json with "type": "module" must be present so Node loads
6972
// dist/index.js as ESM instead of CJS. We synthesize a minimal one rather
7073
// than copying the full source package.json (which contains devDependencies

0 commit comments

Comments
 (0)