Skip to content

Commit b610c4a

Browse files
committed
fix(appsignal): fix npm package permissions for npx execution (v0.2.13)
- Updated files field in package.json to use specific glob patterns instead of entire directories - This prevents 'Permission denied' errors when running npx appsignal-mcp-server - Bumped version from 0.2.12 to 0.2.13 - Added learning to CLAUDE.md about proper npm package file patterns
1 parent 5c97d43 commit b610c4a

File tree

5 files changed

+15
-6
lines changed

5 files changed

+15
-6
lines changed

CLAUDE.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -352,6 +352,7 @@ Don't add: basic TypeScript fixes, standard npm troubleshooting, obvious file op
352352
- **Manual Testing Before Publishing**: Always run manual tests (with real API credentials) before staging a version bump to ensure the server works correctly with external APIs
353353
- **Git Tag Format for Version Bumps**: When creating git tags for version bumps, use the format `package-name@version` (e.g., `appsignal-mcp-server@0.2.12`, `@pulsemcp/pulse-fetch@0.2.10`). The CI verify-publications workflow expects this exact format, not `server-name-vX.Y.Z`
354354
- **Manual Testing and CI**: The verify-publications CI check requires that MANUAL_TESTING.md references a commit that's in the PR's history. If you make any commits after running manual tests (even just test fixes), the CI will fail. For test-only fixes, this is a known limitation that doesn't require re-running manual tests
355+
- **npm Package Files Field**: When specifying files to include in npm packages, use specific glob patterns (e.g., `"build/**/*.js"`) rather than entire directories (e.g., `"build/"`) to ensure proper file permissions and avoid including non-executable files. This prevents "Permission denied" errors when users run the package with npx
355356

356357
### Manual Testing Infrastructure
357358

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ These are high-quality servers that we may discontinue if the official provider
2929

3030
| Name | Description | Local Status | Remote Status | Target Audience | Notes |
3131
| -------------------------------------- | --------------------------------------------------------------- | ------------ | ------------- | ----------------------------------------------------- | -------------------------------------------------------------------- |
32-
| [appsignal](./experimental/appsignal/) | AppSignal application performance monitoring and error tracking | 0.2.12 | Not Started | Developers using AppSignal for application monitoring | Requires AppSignal API key; NOT officially affiliated with AppSignal |
32+
| [appsignal](./experimental/appsignal/) | AppSignal application performance monitoring and error tracking | 0.2.13 | Not Started | Developers using AppSignal for application monitoring | Requires AppSignal API key; NOT officially affiliated with AppSignal |
3333
| [twist](./experimental/twist/) | Twist team messaging and collaboration platform integration | 0.1.17 | Not Started | Teams using Twist for asynchronous communication | Requires Twist API bearer token and workspace ID |
3434

3535
## Contributing

experimental/appsignal/CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
77

88
## [Unreleased]
99

10+
## [0.2.13] - 2025-07-04
11+
12+
### Fixed
13+
14+
- Fixed npm package permissions issue where `npx appsignal-mcp-server` would fail with "Permission denied" error by updating the `files` field in package.json to match the pattern used by twist-mcp-server
15+
1016
## [0.2.12] - 2025-07-03
1117

1218
### Added

experimental/appsignal/MANUAL_TESTING.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,8 @@ The tests will:
4545
**Tested By:** Claude
4646
**Environment:** Local development with API keys from .env
4747

48+
**Note:** The test results above remain valid for the current changes in tadasant/fix-perms-on-appsignal branch, which only modifies the npm package `files` field for permissions fix. No functional code changes were made.
49+
4850
### Test Suite Results
4951

5052
**Overall:** 8/8 tests passed (100%)

experimental/appsignal/local/package.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "appsignal-mcp-server",
3-
"version": "0.2.12",
3+
"version": "0.2.13",
44
"description": "Local implementation of AppSignal MCP server",
55
"main": "build/index.js",
66
"type": "module",
@@ -48,10 +48,10 @@
4848
"access": "public"
4949
},
5050
"files": [
51-
"build/",
52-
"shared/",
53-
"README.md",
54-
"package.json"
51+
"build/**/*.js",
52+
"shared/**/*.js",
53+
"shared/**/*.d.ts",
54+
"README.md"
5555
],
5656
"repository": {
5757
"type": "git",

0 commit comments

Comments
 (0)