Merge branch 'stdlib-js:develop' into gammasgnf

Author: vivekmaurya001

.github/labeler.yml

@@ -27,3 +27,7 @@ LAPACK:
 Math:
 - changed-files:
   - any-glob-to-all-files: '**/math/**/*'
+
+Statistics:
+- changed-files:
+  - any-glob-to-all-files: '**/stats/**/*'

.github/workflows/cleanup_coverage.yml

@@ -59,19 +59,19 @@ jobs:
           script: |
             const prNumber = context.payload.pull_request.number;
             const { data: comments } = await github.rest.issues.listComments({
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              issue_number: prNumber,
+              'owner': context.repo.owner,
+              'repo': context.repo.repo,
+              'issue_number': prNumber
             });
             const coverageComment = comments.find( comment => comment.body && comment.body.includes( '## Coverage Report' ) );
             if ( coverageComment ) {
               // Replace URLs with plain text in the coverage report comment body:
               const updatedBody = coverageComment.body.replace( /<a href="[^"]+">([^<]+)<\/a>/g, '$1' );
               await github.rest.issues.updateComment({
-                owner: context.repo.owner,
-                repo: context.repo.repo,
-                comment_id: coverageComment.id,
-                body: updatedBody,
+                'owner': context.repo.owner,
+                'repo': context.repo.repo,
+                'comment_id': coverageComment.id,
+                'body': updatedBody
               });
             } else {
               console.log( 'No Coverage Report comment found.' );

.github/workflows/labeler.yml

@@ -31,6 +31,7 @@ on:
       - review_request_removed
       - ready_for_review
       - converted_to_draft
+      - labeled
 
 # Workflow jobs:
 jobs:
@@ -75,12 +76,12 @@ jobs:
               'owner': context.repo.owner,
               'repo': context.repo.repo,
               'issue_number': context.payload.pull_request.number,
-              'labels': ['Needs Review'],
+              'labels': [ 'Needs Review' ]
             })
 
-      # Add "Needs Review" label when PR is marked ready for review:
-      - name: 'Add "Needs Review" label if PR is ready for review'
-        if: ${{ github.event.action == 'ready_for_review' }}
+      # Add "Needs Review" label when PR is marked ready for review or review is requested:
+      - name: 'Add "Needs Review" label if PR is ready for review or review is requested'
+        if: ${{ github.event.action == 'ready_for_review' || github.event.action == 'review_requested' }}
         # Pin action to full length commit SHA
         uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
         with:
@@ -90,7 +91,7 @@ jobs:
               'owner': context.repo.owner,
               'repo': context.repo.repo,
               'issue_number': context.payload.pull_request.number,
-              'labels': ['Needs Review'],
+              'labels': [ 'Needs Review' ]
             })
 
       # Remove "Needs Review" label when PR is converted to draft or closed:
@@ -106,8 +107,29 @@ jobs:
                 'owner': context.repo.owner,
                 'repo': context.repo.repo,
                 'issue_number': context.payload.pull_request.number,
-                'name': 'Needs Review',
+                'name': 'Needs Review'
               })
             } catch ( error ) {
               console.log( 'Error removing label: %s', error.message );
             }
+
+      # Remove "Needs Review" and "Needs Changes" labels when "Ready To Merge" is assigned:
+      - name: 'Remove "Needs Review" and "Needs Changes" labels when "Ready To Merge" is assigned'
+        if: ${{ github.event.action == 'labeled' && github.event.label.name == 'Ready To Merge' }}
+        uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
+        with:
+          github-token: ${{ secrets.CHATBOT_GITHUB_TOKEN }}
+          script: |
+            const labelsToRemove = [ 'Needs Review', 'Needs Changes' ];
+            for ( const label of labelsToRemove ) {
+              try {
+                await github.rest.issues.removeLabel({
+                  'owner': context.repo.owner,
+                  'repo': context.repo.repo,
+                  'issue_number': context.payload.pull_request.number,
+                  'name': label
+                })
+              } catch ( error ) {
+                console.log( 'Error removing label %s: %s', label, error.message );
+              }
+            }

.github/workflows/lint_changed_files.yml

@@ -142,43 +142,7 @@ jobs:
       - name: 'Lint package.json files'
         if: success() || failure()
         run: |
-          # Determine root directory:
-          root=$(git rev-parse --show-toplevel)
-
-          # Define the path to a utility for linting package.json files:
-          lint_package_json="${root}/lib/node_modules/@stdlib/_tools/lint/pkg-json/bin/cli"
-
-          # Define paths to utilities for updating package.json metadata fields:
-          update_package_json_directories="${root}/lib/node_modules/@stdlib/_tools/package-json/scripts/update_directories"
-          update_package_json_gypfile="${root}/lib/node_modules/@stdlib/_tools/package-json/scripts/update_gypfile"
-
-          # Lint changed package.json files:
-          files=$(echo "${{ steps.changed-files.outputs.files }}" | tr ' ' '\n' | grep 'package\.json$' | grep -v 'datapackage\.json$' | tr '\n' ' ' | sed 's/ $//')
-          if [ -n "${files}" ]; then
-            echo "Linting package.json files that have changed..."
-            printf "${files}" | "${lint_package_json}" --split=" "
-          else
-            echo "No package.json files to lint."
-          fi
-
-          # Check if metadata fields need to be updated in package.json files of affected packages:
-          dirs=$(echo "${{ steps.changed-files.outputs.files }}" | tr ' ' '\n' | xargs dirname | sort -u)
-          needs_changes=0
-          for dir in ${dirs}; do
-            echo "Checking package.json in ${dir}..."
-            "${update_package_json_directories}" "${dir}"
-            "${update_package_json_gypfile}" "${dir}"
-            if [[ `git status --porcelain` ]]; then
-              echo "::error::Package.json in ${dir} needs updates to directories and/or gypfile fields"
-              git diff
-              needs_changes=1
-            fi
-          done
-
-          # Exit with failure if any needed changes were detected:
-          if [ $needs_changes -eq 1 ]; then
-            exit 1
-          fi
+          . "$GITHUB_WORKSPACE/.github/workflows/scripts/lint_package_json_files" "${{ steps.changed-files.outputs.files }}"
 
       # Lint REPL help files...
       - name: 'Lint REPL help files'

.github/workflows/lint_random_files.yml

@@ -393,15 +393,30 @@ jobs:
         env:
           GITHUB_TOKEN: ${{ secrets.STDLIB_BOT_GITHUB_TOKEN }}
         run: |
-          # Create a temporary file for storing lint error messages:
-          ERROR_LOG=$(mktemp)
-          grep -B 1 -A 2 "style:\|warning:\|error:" <<< "${{ steps.lint-c.outputs.stderr }}" > $ERROR_LOG
-
-          . "$GITHUB_WORKSPACE/.github/workflows/scripts/create_lint_sub_issue" \
-            "${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}" \
-            "C" \
-            "3235" \ # Number of C lint error tracking issue
-            "$ERROR_LOG"
+          BODY_FILE="$GITHUB_WORKSPACE/lint_issue_body.md"
+
+          cat << EOF > "$BODY_FILE"
+            ## C Linting Failures
+
+            Linting failures were detected in the automated lint workflow run.
+
+            ### Workflow Details
+            -   Run: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
+            -   Type: C Linting
+            -   Date: $(date -u +"%Y-%m-%d %H:%M:%S UTC")
+
+            ### Error Details
+            \`\`\`
+            $(grep -B 1 -A 2 "style:\|warning:\|error:" <<< "${{ steps.lint-c.outputs.stderr }}")
+            \`\`\`
+            EOF
+
+          . "$GITHUB_WORKSPACE/.github/workflows/scripts/create_sub_issue" \
+            'Fix C lint errors' \
+            "$BODY_FILE" \
+            "3235"  # Number of C lint error tracking issue
+
+          rm "$BODY_FILE"
 
       # Lint TypeScript declarations files:
       - name: 'Lint TypeScript declarations files'

.github/workflows/run_tests_coverage.yml

@@ -226,25 +226,25 @@ jobs:
           github-token: ${{ secrets.CHATBOT_GITHUB_TOKEN }}
           script: |
             const { data: comments } = await github.rest.issues.listComments({
-              issue_number: context.issue.number,
-              owner: context.repo.owner,
-              repo: context.repo.repo,
+              'issue_number': context.issue.number,
+              'owner': context.repo.owner,
+              'repo': context.repo.repo,
             });
 
             const botComment = comments.find( comment => comment.user.login === 'stdlib-bot' && comment.body.includes( '## Coverage Report' ) );
             if ( botComment ) {
               await github.rest.issues.updateComment({
-                owner: context.repo.owner,
-                repo: context.repo.repo,
-                comment_id: botComment.id,
-                body: `${{ steps.create-report.outputs.report }}`,
+                'owner': context.repo.owner,
+                'repo': context.repo.repo,
+                'comment_id': botComment.id,
+                'body': `${{ steps.create-report.outputs.report }}`
               });
             } else {
               await github.rest.issues.createComment({
-                issue_number: context.issue.number,
-                owner: context.repo.owner,
-                repo: context.repo.repo,
-                body: `${{ steps.create-report.outputs.report }}`,
+                'issue_number': context.issue.number,
+                'owner': context.repo.owner,
+                'repo': context.repo.repo,
+                'body': `${{ steps.create-report.outputs.report }}`
               });
             }
 
@@ -257,10 +257,10 @@ jobs:
           github-token: ${{ secrets.CHATBOT_GITHUB_TOKEN }}
           script: |
             github.rest.repos.createCommitComment({
-              commit_sha: context.sha,
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              body: '${{ steps.create-report.outputs.report }}'
+              'commit_sha': context.sha,
+              'owner': context.repo.owner,
+              'repo': context.repo.repo,
+              'body': '${{ steps.create-report.outputs.report }}'
             })
 
       # Checkout coverage repository:

.github/workflows/scripts/create_lint_sub_issue renamed to .github/workflows/scripts/create_sub_issue

@@ -16,17 +16,16 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# Script to create a sub-issue for linting failures
+# Script to create a sub-issue.
 #
-# Usage: ./create_lint_sub_issue.sh <workflow-url> <lint-type> <parent-issue-number>
+# Usage: ./create_sub_issue.sh <issue-title> <body-file> <parent-issue-number> [<labels>]
 #
 # Arguments:
 #
-#   workflow-url         URL of the workflow run.
-#   lint-type            Type of linting failure.
+#   issue-title          Title for the new sub-issue.
+#   body-file            Path to the file containing the issue body.
 #   parent-issue-number  Number of the parent issue.
-#   error-log-file       Path to the error log file.
-#
+#   labels               Optional comma-separated list of labels to apply to the new sub-issue.
 #
 # Environment variables:
 #
@@ -41,10 +40,10 @@ set -o pipefail
 # VARIABLES #
 
 # Assign command line arguments to variables:
-workflow_url="$1"
-lint_type="$2"
+issue_title="$1"
+body_file="$2"
 parent_issue_number="$3"
-error_log_file="$4"
+labels="$4"
 
 # Repository information:
 owner="stdlib-js"
@@ -57,28 +56,20 @@ if [ -z "$github_token" ]; then
 		exit 1
 fi
 
-# Read and format the error log
-if [ ! -f "$error_log_file" ]; then
-	echo -e "Error log file not found: ${error_log_file}"
+# Read and validate the body file:
+if [ ! -f "$body_file" ]; then
+	echo -e "ERROR: Body file not found: ${body_file}"
 	exit 1
 fi
-error_log_content=$(cat "$error_log_file")
-
-# Create issue body with formatted error log
-issue_body="## ${lint_type} Linting Failures
-
-Linting failures were detected in the automated lint workflow run.
-
-### Workflow Details
--   Run: ${workflow_url}
--   Type: ${lint_type} Linting
--   Date: $(date -u +"%Y-%m-%d %H:%M:%S UTC")
-
-### Error Details
-\`\`\`
-${error_log_content}
-\`\`\`
-"
+issue_body=$(cat "$body_file")
+
+# Process labels into an array if provided:
+if [ -n "$labels" ]; then
+    # Convert comma-separated string to JSON array...
+    label_array="[$(echo "$labels" | sed 's/[[:space:]]*,[[:space:]]*/","/g' | sed 's/.*/"&"/')]"
+else
+    label_array="[]"
+fi
 
 # FUNCTIONS #
 
@@ -114,7 +105,7 @@ EOF
 	echo "$response" | jq -r '.data.repository.id'
 }
 
-# Creates a child issue.
+# Creates a child issue with labels.
 #
 # $1 - repository node ID
 # $2 - issue body
@@ -127,11 +118,12 @@ create_child_issue() {
 		-H "Content-Type: application/json" \
 		--data @- << EOF
 {
-	"query": "mutation CreateIssue(\$repositoryId: ID!, \$title: String!, \$body: String!) { createIssue(input: {repositoryId: \$repositoryId, title: \$title, body: \$body}) { issue { id number } } }",
+	"query": "mutation CreateIssue(\$repositoryId: ID!, \$title: String!, \$body: String!, \$labelIds: [ID!]) { createIssue(input: {repositoryId: \$repositoryId, title: \$title, body: \$body, labelIds: \$labelIds}) { issue { id number } } }",
 	"variables": {
 		"repositoryId": "${repo_id}",
-		"title": "Fix ${lint_type} lint errors",
-		"body": $(echo "$issue_body" | jq -R -s '.')
+		"title": "${issue_title}",
+		"body": $(echo "$issue_body" | jq -R -s '.'),
+		"labelIds": ${label_array}
 	}
 }
 EOF
@@ -159,6 +151,37 @@ EOF
 	echo "$response" | jq -r '.data.repository.issue.id'
 }
 
+# Fetches label IDs for given label names.
+fetch_label_ids() {
+	if [ -z "$labels" ]; then
+		echo "[]"
+		return
+	fi
+
+	local label_names="${labels//,/\",\"}"
+	local response
+	response=$(curl -s -X POST 'https://api.github.com/graphql' \
+		-H "Authorization: bearer ${github_token}" \
+		-H "Content-Type: application/json" \
+		--data @- << EOF
+{
+	"query": "query(\$owner: String!, \$repo: String!) { repository(owner: \$owner, name: \$repo) { labels(first: 100) { nodes { id name } } } }",
+	"variables": {
+		"owner": "${owner}",
+		"repo": "${repo}"
+	}
+}
+EOF
+)
+
+	# Extract and filter label IDs that match our requested labels...
+	echo "$response" | jq --arg names "${label_names}" '
+		.data.repository.labels.nodes |
+		map(select(.name as $n | [$names] | contains([$n]))) |
+		map(.id)
+	'
+}
+
 # Creates a sub-issue relationship.
 #
 # $1 - parent issue ID
@@ -194,7 +217,15 @@ main() {
 		exit 1
 	fi
 
-	echo "Creating child issue for ${lint_type} lint failures..."
+	if [ -n "$labels" ]; then
+		echo "Fetching label IDs..."
+		label_array=$(fetch_label_ids)
+		if [ "$label_array" = "[]" ]; then
+			echo -e "Warning: No valid labels found for the provided label names."
+		fi
+	fi
+
+	echo "Creating child issue..."
 	child_issue_response=$(create_child_issue "$repo_id" "$issue_body")
 
 	child_issue_id=$(echo "$child_issue_response" | jq -r '.data.createIssue.issue.id')