Changing npm publishing

Author: Fabiana Severin

.github/workflows/build-and-release.yml

@@ -98,7 +98,8 @@ jobs:
     - name: Setup NPM authentication
       run: |
         NPM_TOKEN=$(aws secretsmanager get-secret-value --secret-id aws-lambda-runtimes/github/nodejs/npm-token --query SecretString --output text)
-        echo "//registry.npmjs.org/:_authToken=${NPM_TOKEN}" > ~/.npmrc
+        echo "//registry.npmjs.org/:_authToken=${NPM_TOKEN}" > .npmrc
+        chmod 0600 .npmrc
     
     - name: Determine version and package name
       id: version
@@ -117,10 +118,11 @@ jobs:
     
     - name: Publish to npm
       run: |
+        PACKAGE_FILE=$(ls aws-lambda-ric-*.tgz)
         if [[ "${{ steps.version.outputs.is_rc }}" == "true" ]]; then
-          npm publish aws-lambda-ric-*.tgz --tag rc
+          npm publish $PACKAGE_FILE --tag rc --access=public
         else
-          npm publish aws-lambda-ric-*.tgz
+          npm publish $PACKAGE_FILE --access=public
         fi
     
     - name: Create GitHub Release

package.json

@@ -20,8 +20,8 @@
     "format": "npm run format:src && npm run format:test",
     "format:src": "prettier --check \"src/*.*js\" --write",
     "format:test": "prettier --check \"test/**/*.*js\" --write",
-    "check-headers": "node ./scripts/check-headers.js",
-    "add-headers": "node ./scripts/add-headers.js",
+    "check-headers": "./scripts/check-headers.sh",
+    "add-headers": "./scripts/add-headers.sh",
     "lint": "eslint --ext \".js\" src test && npm run check-headers",
 
     "fix": "eslint --fix --ext \".js\" src test",
@@ -68,7 +68,7 @@
     "eslint": "8.42.0",
     "eslint-config-prettier": "8.8.0",
     "eslint-plugin-prettier": "4.2.1",
-    "glob": "^10.3.10",
+
     "husky": "^8.0.3",
     "lambda-runtime": "file:./src/",
     "mocha": "^10.8.2",

scripts/add-headers.js

@@ -0,0 +1,42 @@
+#!/bin/bash
+# Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+# SPDX-License-Identifier: Apache-2.0
+
+set -e
+
+# Find files missing copyright headers
+files=$(git ls-files 'bin/**' 'scripts/**' 'src/**' 'test/**' | \
+  grep -E '\.(js|ts|mjs|mts|jsx|tsx|c|cpp|h)$' | \
+  xargs grep -L 'Copyright.*Amazon\.com' || true)
+
+if [ -z "$files" ]; then
+  echo "✓ All files already have copyright headers"
+  exit 0
+fi
+
+echo "Found $(echo "$files" | wc -l) files missing headers"
+
+# Add headers
+for file in $files; do
+  if [[ "$file" == *.sh ]]; then
+    header="#!/bin/bash
+# Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+# SPDX-License-Identifier: Apache-2.0
+
+"
+  else
+    header="/*
+Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+SPDX-License-Identifier: Apache-2.0
+*/
+
+"
+  fi
+  
+  echo "$header$(cat "$file")" > "$file"
+  echo "✓ Added header to $file"
+done
+
+echo
+echo "✓ All copyright headers added successfully"
+echo "Run 'git diff' to review changes before committing"

scripts/add-headers.sh

@@ -0,0 +1,21 @@
+#!/bin/bash
+# Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+# SPDX-License-Identifier: Apache-2.0
+
+set -e
+
+# Find files missing copyright headers
+files=$(git ls-files 'bin/**' 'scripts/**' 'src/**' 'test/**' | \
+  grep -E '\.(js|ts|mjs|mts|jsx|tsx|c|cpp|h|sh)$' | \
+  xargs grep -L 'Copyright.*Amazon\.com' || true)
+
+if [ -n "$files" ]; then
+  echo "❌ Copyright header check failed."
+  echo "Files missing required \"Copyright\" and \"Amazon.com\" keywords:"
+  echo "$files" | sed 's/^/  - /'
+  echo
+  echo "Run 'npm run add-headers' to fix these issues."
+  exit 1
+fi
+
+echo "✅ All files have proper copyright headers."

scripts/check-headers.js

@@ -12,7 +12,7 @@ const { execSync } = require('child_process');
 describe('check-headers script', function() {
   it('should detect files with proper headers', function() {
     // This test file itself has the proper header
-    const result = execSync('node ./scripts/check-headers.js', {
+    const result = execSync('./scripts/check-headers.sh', {
       env: { ...process.env, NODE_ENV: 'test' },
       stdio: 'pipe'
     }).toString();