Merge remote-tracking branch 'upstream/develop' into ndarray-base-binary-reduce-strided1d-dispatch

Author: stdlib-bot

.github/workflows/pr_commands_comment.yml

@@ -29,6 +29,11 @@ on:
         description: 'Pull request number'
         required: true
         type: number
+      debug:
+        description: 'Enable debug output'
+        required: false
+        default: false
+        type: boolean
 
     # Define the secrets accessible by the workflow:
     secrets:
@@ -43,6 +48,14 @@ on:
         description: 'Pull request number'
         required: true
         type: number
+      debug:
+        description: 'Enable debug output'
+        required: false
+        default: 'false'
+        type: choice
+        options:
+          - 'true'
+          - 'false'
 
 # Global permissions:
 permissions:
@@ -63,11 +76,23 @@ jobs:
 
     # Define the sequence of job steps...
     steps:
+      # Checkout the repository:
+      - name: 'Checkout repository'
+        # Pin action to full length commit SHA
+        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
+        with:
+          # Ensure we have access to the scripts directory:
+          sparse-checkout: |
+              .github/workflows/scripts
+          sparse-checkout-cone-mode: false
+        timeout-minutes: 10
 
       # Leave comment with package make command instructions:
       - name: 'Leave comment with package make command instructions'
         env:
           PR_NUMBER: ${{ inputs.pull_request_number }}
+          GITHUB_TOKEN: ${{ secrets.STDLIB_BOT_GITHUB_TOKEN || secrets.STDLIB_BOT_PAT_REPO_WRITE }}
+          DEBUG: ${{ inputs.debug || 'false' }}
         run: |
           . "$GITHUB_WORKSPACE/.github/workflows/scripts/package_commands_comment" "$PR_NUMBER"
         timeout-minutes: 30

.github/workflows/scripts/package_commands_comment

@@ -27,6 +27,7 @@
 # Environment variables:
 #
 #   GITHUB_TOKEN   GitHub token for authentication.
+#   DEBUG          Whether to enable verbose debug output. Default: `false`.
 
 # Ensure that the exit status of pipelines is non-zero in the event that at least one of the commands in a pipeline fails:
 set -o pipefail
@@ -44,9 +45,22 @@ github_api_url="https://api.github.com"
 repo_owner="stdlib-js"
 repo_name="stdlib"
 
+# Set debug mode:
+debug="${DEBUG:-false}"
+
 
 # FUNCTIONS #
 
+# Prints debug messages if DEBUG environment variable is set to "true".
+#
+# $1 - debug message
+debug_log() {
+	# Only print debug messages if DEBUG environment variable is set to "true":
+	if [ "$debug" = true ]; then
+		echo "[DEBUG] $1" >&2
+	fi
+}
+
 # Error handler.
 #
 # $1 - error status
@@ -70,6 +84,8 @@ github_api() {
 	local endpoint="$2"
 	local data="$3"
 
+	debug_log "Making API request: ${method} ${endpoint}"
+
 	# Initialize an array to hold curl headers:
 	local headers=()
 
@@ -117,12 +133,20 @@ main() {
 		on_error 1
 	fi
 
+	debug_log "Processing PR #${pr_number}"
+
 	# Fetch changed files in pull request:
 	response=$(github_api "GET" "/repos/${repo_owner}/${repo_name}/pulls/${pr_number}/files?per_page=100")
 	files=$(echo "${response}" | jq -r '.[] | .filename')
+	debug_log "Found $(echo "${files}" | wc -l) changed files"
 
 	# Extract files associated with native add-ons:
-	c_files=$(echo "${files}" | grep -e '/benchmark/c' -e '/examples/c' -e '/binding.gyp' -e '/include.gypi' -e '/src/')
+	c_files=$(echo "${files}" | grep -e '/benchmark/c' -e '/examples/c' -e '/binding.gyp' -e '/include.gypi' -e '/src/' || true)
+	if [[ -z "${c_files}" ]]; then
+		debug_log "No native add-on files found"
+	else
+		debug_log "Found native add-on files: $(echo "${c_files}" | wc -l) files"
+	fi
 
 	# Find unique package directories:
 	directories=$(echo "${files}" | tr ' ' '\n' | \
@@ -133,6 +157,7 @@ main() {
 
 	# Extract package names from changed package directories (e.g., @stdlib/math/base/special/sin) by removing the leading 'lib/node_modules/':
 	packages=$(echo "${directories}" | sed -E 's/^lib\/node_modules\///')
+	debug_log "Found packages: $(echo "${packages}" | tr '\n' ' ')"
 
 	# Documentation links:
 	docs_links="
@@ -148,9 +173,11 @@ main() {
 
 	# Count the number of packages:
 	package_count=$(echo "${packages}" | wc -l)
+	debug_log "Package count: ${package_count}"
 
 	if [[ $package_count -gt 1 ]]; then
 		# Multiple packages case:
+		debug_log "Multiple packages detected, generating multi-package comment"
 		comment="Hello! 👋
 
 Pro-tip: This PR changes multiple packages. You can use various \`make\` rules with \`*_FILTER\` environment variables to run tests, benchmarks, and examples for specific packages.
@@ -174,7 +201,9 @@ ${docs_links}"
 
 	else
 		# Single package case:
-		if [[ "${#c_files[@]}" -eq 0 ]]; then
+		debug_log "Single package detected: ${packages}"
+		if [[ -z "${c_files}" ]]; then
+			debug_log "No C files found, generating JS-only comment"
 			comment="Hello! 👋
 
 Pro-tip: Use the \`make\` commands below during local development to ensure that all tests, examples, and benchmark files in your PR run successfully.
@@ -200,6 +229,7 @@ make examples EXAMPLES_FILTER=\".*/${packages}/.*\"
 \`\`\`
 ${docs_links}"
 		else
+			debug_log "C files found, generating native addon comment"
 			comment="Hello! 👋
 
 Pro-tip: Use the \`make\` below commands during local development to ensure that all tests, examples, and benchmark files in your PR run successfully.
@@ -241,10 +271,12 @@ ${docs_links}"
 		fi
 	fi
 
+	debug_log "Posting comment on PR #${pr_number}"
 	if ! github_api "POST" "/repos/${repo_owner}/${repo_name}/issues/${pr_number}/comments" "{\"body\":$(echo "${comment}" | jq -R -s -c .)}"; then
 		echo "Failed to post comment on PR."
 		on_error 1
 	fi
+	debug_log "Successfully posted comment on PR #${pr_number}"
 
 	print_success
 	exit 0

CONTRIBUTORS

@@ -189,6 +189,7 @@ Sivam Das <[email protected]>
 Snehil Shah <[email protected]>
 Soumajit Chatterjee <[email protected]>
 Spandan Barve <[email protected]>
+Srinivas Batthula <[email protected]>
 Stephannie Jiménez Gacha <[email protected]>
 Suhaib Ilahi <[email protected]>
 Suraj Kumar <[email protected]>

docs/git-notes/154ff94132f2435ab70bfdf2ebf661c6441a040c.txt

@@ -0,0 +1,4 @@
+---
+type: amend-message
+---
+feat: add constants to `float64` namespace

docs/git-notes/5e3f0ad64fdac2a68ee1b1fcf88081bdc274d1e7.txt

@@ -0,0 +1,4 @@
+---
+type: amend-message
+---
+feat: add `planck` distribution namespace

lib/node_modules/@stdlib/_tools/eslint/rules/doctest/lib/main.js

@@ -26,6 +26,7 @@ var join = require( 'path' ).join;
 var format = require( 'util' ).format;
 var dirname = require( 'path' ).dirname;
 var logger = require( 'debug' );
+var rootDir = require( '@stdlib/_tools/utils/root-dir' );
 var Buffer = require( '@stdlib/buffer/ctor' );
 var isNull = require( '@stdlib/assert/is-null' );
 var isString = require( '@stdlib/assert/is-string' ).isPrimitive;
@@ -43,6 +44,7 @@ var windowShim = require( './window.js' );
 // VARIABLES //
 
 var debug = logger( 'doctest' );
+var ROOT_DIR = rootDir();
 var RE_JSDOC = /\/\*\*[\s\S]+?\*\//g;
 var RE_ESLINT_INLINE = / ?\/\/ eslint-disable-(?:next-)?line[^\n]*\n/g;
 var RE_NEWLINE = /\r?\n/g;
@@ -137,9 +139,16 @@ function main( context ) {
 	* @returns {*} required module
 	*/
 	function customRequire( path ) {
+		var stdlibPath;
+
 		if ( startsWith( path, './' ) ) {
 			return require( join( dir, path ) ); // eslint-disable-line stdlib/no-dynamic-require
 		}
+		// Force `@stdlib` packages to be loaded from `lib/node_modules/@stdlib`:
+		if ( startsWith( path, '@stdlib/' ) ) {
+			stdlibPath = join( ROOT_DIR, 'lib', 'node_modules', path );
+			return require( stdlibPath ); // eslint-disable-line stdlib/no-dynamic-require
+		}
 		return require( path ); // eslint-disable-line stdlib/no-dynamic-require
 	}
 	scope = {

lib/node_modules/@stdlib/_tools/eslint/rules/jsdoc-doctest/lib/main.js

@@ -27,6 +27,7 @@ var format = require( 'util' ).format;
 var dirname = require( 'path' ).dirname;
 var logger = require( 'debug' );
 var parseJSDoc = require( 'doctrine' ).parse;
+var rootDir = require( '@stdlib/_tools/utils/root-dir' );
 var Buffer = require( '@stdlib/buffer/ctor' );
 var isNull = require( '@stdlib/assert/is-null' );
 var isNumber = require( '@stdlib/assert/is-number' );
@@ -46,6 +47,7 @@ var windowShim = require( './window.js' );
 // VARIABLES //
 
 var debug = logger( 'jsdoc-doctest' );
+var ROOT_DIR = rootDir();
 var RE_ANNOTATION = /(?:\n|^)(?:var|let|const)? ?([a-zA-Z0-9._]+) ?=?[^;]*;\n\/\/ ?(returns|([A-Za-z][A-Za-z_0-9]*)? ?=>|throws) {0,1}([\s\S]*?)(\n|$)/g;
 var RE_CONSOLE = /console\.(?:dir|error|log)/;
 var RE_MODULE_TAG = /\* @module[^\n]*\n/g;
@@ -147,9 +149,16 @@ function main( context ) {
 	* @returns {*} required module
 	*/
 	function customRequire( path ) {
+		var stdlibPath;
+
 		if ( startsWith( path, './' ) ) {
 			return require( join( dir, path ) ); // eslint-disable-line stdlib/no-dynamic-require
 		}
+		// Force `@stdlib` packages to be loaded from `lib/node_modules/@stdlib`:
+		if ( startsWith( path, '@stdlib/' ) ) {
+			stdlibPath = join( ROOT_DIR, 'lib', 'node_modules', path );
+			return require( stdlibPath ); // eslint-disable-line stdlib/no-dynamic-require
+		}
 		return require( path ); // eslint-disable-line stdlib/no-dynamic-require
 	}
 	scope = {

lib/node_modules/@stdlib/_tools/eslint/rules/namespace-export-all/lib/main.js

@@ -23,16 +23,15 @@
 var basename = require( 'path' ).basename;
 var dirname = require( 'path' ).dirname;
 var join = require( 'path' ).join;
-var relative = require( 'path' ).relative;
 var statSync = require( 'fs' ).statSync; // eslint-disable-line node/no-sync
+var exists = require( '@stdlib/fs/exists' ).sync;
 var endsWith = require( '@stdlib/string/ends-with' );
-var replace = require( '@stdlib/string/replace' );
 var isString = require( '@stdlib/assert/is-string' ).isPrimitive;
 var trim = require( '@stdlib/string/trim' );
 var camelcase = require( '@stdlib/string/base/camelcase' );
 var readDir = require( '@stdlib/fs/read-dir' ).sync;
 var contains = require( '@stdlib/assert/contains' );
-var rootDir = require( '@stdlib/_tools/utils/root-dir' );
+var resolvePackage = require( './resolve_package.js' );
 
 
 // VARIABLES //
@@ -41,7 +40,6 @@ var rule;
 var ORDER_COMMENT = 'When adding modules to the namespace, ensure that they are added in alphabetical order according to module name.';
 var RE_CUSTOM_EXCLUDES = /The following modules are intentionally not exported: ?([^\n]+)\n/;
 var RE_NS_VAR = /@namespace ([a-zA-Z0-9_]+)/;
-var LIB_DIR = join( rootDir(), 'lib', 'node_modules' );
 var EXCLUDE_LIST = [
 	'node_modules',
 	'benchmark',
@@ -184,15 +182,15 @@ function main( context ) {
 		* @returns {Object} fix
 		*/
 		function fix( fixer ) {
-			var nsPath;
+			var packagePath;
 			var entry;
 			var match;
 			var pos;
 			var ns;
 			var re;
 			var i;
 
-			nsPath = replace( relative( LIB_DIR, dir ), '\\', '/' );
+			packagePath = resolvePackage( dir, pkg );
 			match = RE_NS_VAR.exec( source.text );
 			if ( !match ) {
 				return null;
@@ -208,7 +206,7 @@ function main( context ) {
 				'*/',
 				'setReadOnly( <<ns>>, \'<<alias>>\', require( \'<<pkg>>\' ) );'
 			].join( '\n' );
-			entry = entry.replace( /<<pkg>>/g, nsPath + '/' + pkg );
+			entry = entry.replace( /<<pkg>>/g, packagePath );
 			entry = entry.replace( /<<alias>>/g, camelcase( pkg ) );
 			entry = entry.replace( /<<ns>>/g, ns );
 
@@ -218,7 +216,7 @@ function main( context ) {
 				if ( pkg.localeCompare( packages[ i ], 'en', OPTS_COMPARE ) < 0 ) {
 					break;
 				}
-				re = new RegExp( 'require\\( \'' + nsPath + '\\/' + packages[ i ] + '\' \\) \\);' );
+				re = new RegExp( 'require\\( \'' + resolvePackage( dir, packages[ i ] ) + '\' \\) \\);' );
 				if ( re.test( source.text ) ) {
 					pos = source.text.search( re );
 				}
@@ -252,16 +250,21 @@ function main( context ) {
 	* @returns {Array} namespace package directories
 	*/
 	function packageDirectories( contents ) {
+		var pkgJSONPath;
+		var contentPath;
 		var content;
 		var out;
 		var i;
 
 		out = [];
 		for ( i = 0; i < contents.length; i++ ) {
 			content = contents[ i ];
+			contentPath = join( dir, content );
+			pkgJSONPath = join( contentPath, 'package.json' );
 			if (
-				isDirectory( join( dir, content ) ) &&
-				!contains( excludes, content )
+				isDirectory( contentPath ) &&
+				!contains( excludes, content ) &&
+				exists( pkgJSONPath )
 			) {
 				out.push( content );
 			}

lib/node_modules/@stdlib/_tools/eslint/rules/namespace-export-all/lib/resolve_package.js

@@ -0,0 +1,62 @@
+/**
+* @license Apache-2.0
+*
+* Copyright (c) 2025 The Stdlib Authors.
+*
+* Licensed under the Apache License, Version 2.0 (the "License");
+* you may not use this file except in compliance with the License.
+* You may obtain a copy of the License at
+*
+*    http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing, software
+* distributed under the License is distributed on an "AS IS" BASIS,
+* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+* See the License for the specific language governing permissions and
+* limitations under the License.
+*/
+
+'use strict';
+
+// MODULES //
+
+var join = require( 'path' ).join;
+var relative = require( 'path' ).relative;
+var replace = require( '@stdlib/string/replace' );
+var rootDir = require( '@stdlib/_tools/utils/root-dir' );
+
+
+// VARIABLES //
+
+var LIB_DIR = join( rootDir(), 'lib', 'node_modules' );
+
+
+// MAIN //
+
+/**
+* Resolves a stdlib package identifier from directory path and package name.
+*
+* @private
+* @param {string} dir - directory path containing the package
+* @param {string} pkg - package name
+* @returns {string} stdlib package identifier relative to the lib directory
+*
+* @example
+* var join = require( 'path' ).join;
+* var rootDir = require( '@stdlib/_tools/utils/root-dir' );
+*
+* var LIB_DIR = join( rootDir(), 'lib', 'node_modules' );
+* var dir = join( LIB_DIR, '@stdlib', 'math', 'base' );
+*
+* var pkgId = resolvePackage( dir, 'abs' );
+* // returns '@stdlib/math/base/abs'
+*/
+function resolvePackage( dir, pkg ) {
+	var nsPath = replace( relative( LIB_DIR, dir ), '\\', '/' );
+	return nsPath + '/' + pkg;
+}
+
+
+// EXPORTS //
+
+module.exports = resolvePackage;