Merge branch 'main' into release-automation-notes

Author: Flyingliuhub

.browserslistrc

@@ -1,9 +1,14 @@
 [production]
-last 2 Firefox versions
-last 2 Chrome versions
-last 2 Safari versions
-> 0.25%
-not ie 11
+Firefox > 0 and last 2 years and > 0.01%
+Chrome > 0 and last 2 years and > 0.01%
+Safari > 0 and last 2 years and > 0.01%
+Edge > 0 and last 1 years and > 0.01%
+Opera > 0 and last 2 years and > 0.01%
+> 0.2%
+not op_mini all
+not and_uc < 100
+not android < 100
+not dead
 
 [dev]
 last 1 chrome versions

.eslintignore

@@ -3,6 +3,7 @@
 /.chromium
 /build
 /built_assets
+/bwc_tmp
 /config/apm.dev.js
 /data
 /html_docs
@@ -17,8 +18,7 @@ target
 # plugin overrides
 /src/core/lib/osd_internal_native_observable
 /src/legacy/plugin_discovery/plugin_pack/__tests__/fixtures/plugins/broken
-/src/plugins/data/common/opensearch_query/kuery/ast/_generated_/**
-/src/plugins/vis_type_timeline/public/_generated_/**
+/src/plugins/**/_generated_/**
 
 # package overrides
 /packages/opensearch-eslint-config-opensearch-dashboards

.eslintrc.js

@@ -272,26 +272,22 @@ module.exports = {
             basePath: __dirname,
             zones: [
               {
-                target: ['(src)/**/*', '!src/core/**/*'],
+                target: ['src/**/*', '!src/core/**/*'],
                 from: ['src/core/utils/**/*'],
                 errorMessage: `Plugins may only import from src/core/server and src/core/public.`,
               },
               {
-                target: ['(src)/plugins/*/server/**/*'],
-                from: ['(src)/plugins/*/public/**/*'],
+                target: ['src/plugins/*/server/**/*'],
+                from: ['src/plugins/*/public/**/*'],
                 errorMessage: `Server code can not import from public, use a common directory.`,
               },
               {
-                target: ['(src)/plugins/*/common/**/*'],
-                from: ['(src)/plugins/*/(server|public)/**/*'],
+                target: ['src/plugins/*/common/**/*'],
+                from: ['src/plugins/*/(server|public)/**/*'],
                 errorMessage: `Common code can not import from server or public, use a common directory.`,
               },
               {
-                target: [
-                  'src/legacy/**/*',
-                  '(src)/plugins/**/(public|server)/**/*',
-                  'examples/**/*',
-                ],
+                target: ['src/legacy/**/*', 'src/plugins/**/(public|server)/**/*', 'examples/**/*'],
                 from: [
                   'src/core/public/**/*',
                   '!src/core/public/index.ts', // relative import
@@ -320,30 +316,30 @@ module.exports = {
               {
                 target: [
                   'src/legacy/**/*',
-                  '(src)/plugins/**/(public|server)/**/*',
+                  'src/plugins/**/(public|server)/**/*',
                   'examples/**/*',
-                  '!(src)/**/*.test.*',
+                  '!src/**/*.test.*',
                 ],
                 from: [
-                  '(src)/plugins/**/(public|server)/**/*',
-                  '!(src)/plugins/**/(public|server)/mocks/index.{js,mjs,ts}',
-                  '!(src)/plugins/**/(public|server)/(index|mocks).{js,mjs,ts,tsx}',
+                  'src/plugins/**/(public|server)/**/*',
+                  '!src/plugins/**/(public|server)/mocks/index.{js,mjs,ts}',
+                  '!src/plugins/**/(public|server)/(index|mocks).{js,mjs,ts,tsx}',
                 ],
                 allowSameFolder: true,
                 errorMessage: 'Plugins may only import from top-level public and server modules.',
               },
               {
                 target: [
-                  '(src)/plugins/**/*',
-                  '!(src)/plugins/**/server/**/*',
+                  'src/plugins/**/*',
+                  '!src/plugins/**/server/**/*',
 
                   'examples/**/*',
                   '!examples/**/server/**/*',
                 ],
                 from: [
                   'src/core/server',
                   'src/core/server/**/*',
-                  '(src)/plugins/*/server/**/*',
+                  'src/plugins/*/server/**/*',
                   'examples/**/server/**/*',
                 ],
                 errorMessage:
@@ -355,7 +351,7 @@ module.exports = {
                 errorMessage: 'The core cannot depend on any plugins.',
               },
               {
-                target: ['(src)/plugins/*/public/**/*'],
+                target: ['src/plugins/*/public/**/*'],
                 from: ['ui/**/*'],
                 errorMessage: 'Plugins cannot import legacy UI code.',
               },
@@ -459,7 +455,7 @@ module.exports = {
      * Files that run BEFORE node version check
      */
     {
-      files: ['scripts/**/*.js', 'src/setup_node_env/**/*.js'],
+      files: ['scripts/**/*.js', 'src/setup_node_env/**/!(*.test).js'],
       rules: {
         'import/no-commonjs': 'off',
         'prefer-object-spread/prefer-object-spread': 'off',

.github/.codecov.yml

@@ -1,7 +1,11 @@
 # https://docs.codecov.com/docs/codecov-yaml
+codecov:
+  require_ci_to_pass: yes
+
 coverage:
   status:
     project:
       default:
-        # https://docs.codecov.com/docs/commit-status#target
-        target: auto  # coverage must be equal or above the previous commit
+        target: auto
+        threshold: 2% # the leniency in hitting the target
+

.github/CODEOWNERS

@@ -1,2 +1 @@
-# This should match the owning team set up in https://github.com/orgs/opensearch-project/teams
-*   @opensearch-project/opensearch-dashboards-core
+*   @ananzh @kavilla @AMoo-Miki @ashwin-pc @joshuarrrr @abbyhu2000 @zengyan-amazon @zhongnansu @manasvinibs @ZilongX @Flyingliuhub @curq @bandinib-amzn @SuZhou-Joe @ruanyl @BionIT @xinruiba @zhyuanqi @mengweieric @LDrago27

.github/pull_request_template.md

@@ -1,15 +1,42 @@
 ### Description
-[Describe what this change achieves]
- 
+
+<!-- Describe what this change achieves-->
+
 ### Issues Resolved
-[List any issues this PR will resolve]
- 
+
+<!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix -->
+<!-- Example: closes #1234 or fixes <Issue_URL> -->
+
+## Screenshot
+
+<!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description -->
+
+## Testing the changes
+
+<!--
+  Please provide detailed steps for validating your changes. This could involve specific commands to run,
+  pages to visit, scenarios to try or any other information that would help reviewers verify
+  the functionality of your change
+-->
+
+## Changelog
+<!--
+Add a short but concise sentence about the impact of this pull request. Prefix an entry with the type of change they correspond to: breaking, chore, deprecate, doc, feat, fix, infra, refactor, test.
+- fix: Update the graph
+- feat: Add a new feature
+
+If this change does not need to added to the changelog, just add a single `skip` line e.g.
+- skip
+
+Descriptions following the prefixes must be 100 characters long or less
+-->
+
 ### Check List
+
 - [ ] All tests pass
   - [ ] `yarn test:jest`
   - [ ] `yarn test:jest_integration`
-  - [ ] `yarn test:ftr`
 - [ ] New functionality includes testing.
 - [ ] New functionality has been documented.
 - [ ] Update [CHANGELOG.md](./../CHANGELOG.md)
-- [ ] Commits are signed per the DCO using --signoff 
+- [ ] Commits are signed per the DCO using --signoff

.github/workflows/add-untriaged.yml

@@ -0,0 +1,46 @@
+name: Apply 'untriaged' label during issue lifecycle
+
+on:
+  issues:
+    types: [opened, reopened, transferred]
+
+jobs:
+  apply-label-if-not-collaborator:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check if issue author is a collaborator
+        id: check-collaborator
+        uses: actions/github-script@v6
+        with:
+          script: |
+            const issueAuthor = context.payload.issue.user.login;
+            const repoOwner = context.repo.owner;
+            const repoName = context.repo.repo;
+            let isCollaborator = false;
+            try {
+              // Attempt to fetch user's permission level
+              await github.rest.repos.getCollaboratorPermissionLevel({
+                owner: repoOwner,
+                repo: repoName,
+                username: issueAuthor,
+              });
+
+              // If no error is thrown, the user is a collaborator
+              isCollaborator = true;
+            } catch (error) {
+              // Error thrown indicates the user is not a collaborator,
+              // or does not have explicit permission set.
+              console.log(`${issueAuthor} is not a collaborator.`);
+            }
+            core.setOutput('is_collaborator', isCollaborator.toString());
+      - name: Apply label if not a collaborator
+        if: steps.check-collaborator.outputs.is_collaborator == 'false'
+        uses: actions/github-script@v6
+        with:
+          script: |
+            github.rest.issues.addLabels({
+              issue_number: context.issue.number,
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              labels: ['untriaged']
+            })

.github/workflows/backport.yml

@@ -34,8 +34,10 @@ jobs:
           installation_id: 22958780
 
       - name: Backport
-        uses: VachaShah/backport@v2.1.0
+        uses: VachaShah/backport@v2.2.0
         with:
           github_token: ${{ steps.github_app_token.outputs.token }}
           head_template: backport/backport-<%= number %>-to-<%= base %>
           files_to_skip: "CHANGELOG.md"
+          labels_template: "<%= JSON.stringify([...labels, 'autocut']) %>"
+          failure_labels: "failed backport"