Skip to content

Commit bd34193

Browse files
committed
refactor: simplify native module loading and remove unused scripts
- Replace custom install script with standard node-gyp-build usage - Remove redundant fallback logic in src/syslog.ts - Delete unused test scripts (test-basic.js, test-direct.js, test-simple.js) - Remove container-test.js wrapper script - Trust node-gyp-build to handle module resolution automatically - Align with Node.js native addon best practices
1 parent 276b0cd commit bd34193

9 files changed

Lines changed: 24 additions & 507 deletions

File tree

.opencode/command/openspec-proposal.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ The user has requested the following change proposal. Use the openspec instructi
1212
- Keep changes tightly scoped to the requested outcome.
1313
- Refer to `openspec/AGENTS.md` (located inside the `openspec/` directory—run `ls openspec` or `openspec update` if you don't see it) if you need additional OpenSpec conventions or clarifications.
1414
- Identify any vague or ambiguous details and ask the necessary follow-up questions before editing files.
15+
- Do not write any code during the proposal stage. Only create design documents (proposal.md, tasks.md, design.md, and spec deltas). Implementation happens in the apply stage after approval.
1516

1617
**Steps**
1718
1. Review `openspec/project.md`, run `openspec list` and `openspec list --specs`, and inspect related code or docs (e.g., via `rg`/`ls`) to ground the proposal in current behaviour; note any gaps that require clarification.

package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,8 @@
1919
"dev": "tsc --watch",
2020
"lint": "eslint src --ext .ts",
2121
"prepublishOnly": "pnpm run build && pnpm run build:native && pnpm run build:native:prebuild",
22-
"install": "node scripts/install.js",
23-
"install:native": "node scripts/install.js",
22+
"install": "node-gyp-build",
23+
"install:native": "node-gyp-build",
2424
"docs:build": "typedoc && node scripts/build-docs.js",
2525
"docs:serve": "bundle exec jekyll serve --livereload",
2626
"docs:deploy": "pnpm run docs:build && bundle exec jekyll build"

pnpm-lock.yaml

Lines changed: 21 additions & 21 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)