fix: resolve code review findings for PR #30

- [P1] Add publishConfig.access=public so the scoped @influxdata/*
  package publishes publicly on first npm publish.
- [P2] Skip cloud-serverless integration job for forked PRs — fork
  runs cannot access environment secrets and would otherwise fail
  on empty credentials.

Author: jstirnaman

.github/workflows/ci.yml

@@ -105,7 +105,12 @@ jobs:
   test-integration-cloud-serverless:
     name: Integration tests (InfluxDB Cloud Serverless)
     runs-on: ubuntu-latest
-    if: github.repository_owner == 'influxdata'
+    # Skip on forked PRs — environment secrets (INFLUXDB_CLOUD_URL/TOKEN) are
+    # not exposed to fork runs, which would fail on empty credentials.
+    if: |
+      github.repository_owner == 'influxdata' &&
+      (github.event_name == 'push' ||
+       github.event.pull_request.head.repo.full_name == github.repository)
     environment: cloud-serverless
     steps:
       - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd  # v6.0.2

package.json

@@ -4,6 +4,9 @@
   "description": "Official InfluxDB MCP server for Model Context Protocol integration",
   "license": "(Apache-2.0 OR MIT)",
   "private": false,
+  "publishConfig": {
+    "access": "public"
+  },
   "type": "module",
   "engines": {
     "node": ">=20.11.0"