Merge branch 'main' into fix/docs-typos-and-doc-links

Author: Mayank Raj

.github/ISSUE_TEMPLATE/bug_report.md

@@ -2,7 +2,7 @@
 name: "\U0001F41B Bug report"
 about: Create a report to help us improve
 title: ""
-labels: bug
+type: "Bug"
 assignees: ""
 ---
 

.github/ISSUE_TEMPLATE/custom.md

@@ -1,8 +1,9 @@
 ---
 name: Custom
-about: Open an issue in the repo that is neither a bug or a feature, such a new idea
+about: Open an issue in the repo that is neither a bug or a feature.
 title: ""
 labels: ""
+type: ""
 assignees: ""
 ---
 

.github/ISSUE_TEMPLATE/feature_request.md

@@ -2,7 +2,8 @@
 name: "\U0001F680 Feature request"
 about: Suggest a new ECMAScript feature to be implemented, or a new capability of the engine.
 title: ""
-labels: enhancement
+type: "Feature"
+labels: ""
 assignees: ""
 ---
 

.github/labeler.yml

@@ -0,0 +1,85 @@
+C-Actions:
+- changed-files:
+  - any-glob-to-any-file:
+    - '.github/**'
+
+C-AST:
+- changed-files:
+  - any-glob-to-any-file: 'core/ast/**'
+
+C-Benchmark:
+- changed-files:
+  - any-glob-to-any-file:
+    - 'benches/**'
+    - 'core/engine/benches/**'
+
+C-Builtins:
+- changed-files:
+  - any-glob-to-any-file:
+    - 'core/engine/src/builtins/**'
+    - 'core/engine/src/object/builtins/**'
+  - all-globs-to-all-files:
+    - '!core/engine/src/object/builtins/intl/**'
+
+C-CLI:
+- changed-files:
+  - any-glob-to-any-file:
+    - 'cli/**'
+
+C-Dependencies:
+- changed-files:
+  - any-glob-to-any-file:
+    - '**/Cargo.lock'
+    - '**/Cargo.toml'
+
+C-Documentation:
+- changed-files:
+  - any-glob-to-any-file:
+    - '**/*.md'
+
+C-FFI:
+- changed-files:
+  - any-glob-to-any-file:
+    - 'ffi/**'
+
+C-GC:
+- changed-files:
+  - any-glob-to-any-file:
+    - 'core/gc/**'
+
+C-Intl:
+- changed-files:
+  - any-glob-to-any-file:
+    - 'core/engine/src/builtins/intl/**'
+
+C-Javascript:
+- changed-files:
+  - any-glob-to-any-file:
+    - '**/*.js'
+
+C-Parser:
+- changed-files:
+  - any-glob-to-any-file:
+    - 'core/parser/**'
+
+C-Runtime:
+- changed-files:
+  - any-glob-to-any-file:
+    - 'core/runtime/**'
+
+C-Tests:
+- changed-files:
+  - any-glob-to-any-file:
+    - '**/tests/**'
+    - '**/test*'
+
+C-VM:
+- changed-files:
+  - any-glob-to-any-file:
+    - 'core/engine/src/bytecompiler/**'
+    - 'core/engine/src/vm/**'
+
+C-WebAssembly:
+- changed-files:
+  - any-glob-to-any-file:
+    - 'ffi/wasm/**'

.github/release.yml

@@ -7,14 +7,18 @@ changelog:
   categories:
     - title: Feature Enhancements
       labels:
-        - enhancement
+        - A-Enhancement
     - title: Bug Fixes
       labels:
-        - bug
+        - A-Bug
+    - title: Performance Improvements
+      labels:
+        - A-Performance
+        - A-Memory
     - title: Internal Improvements
       labels:
-        - Internal
-        - memory
+        - A-Internal
+        - A-Technical Debt
     - title: Other Changes
       labels:
         - "*"

.github/workflows/labeler.yml

@@ -0,0 +1,12 @@
+name: "Pull Request Labeler"
+on:
+- pull_request_target
+
+jobs:
+  labeler:
+    permissions:
+      contents: read
+      pull-requests: write
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/labeler@v6

.github/workflows/pr_management.yml

@@ -0,0 +1,65 @@
+name: PR Management
+
+on:
+  pull_request_target:
+    types: [opened, reopened, synchronize, closed]
+
+jobs:
+  manage_pr:
+    runs-on: ubuntu-latest
+    permissions:
+      pull-requests: write
+      issues: write
+    steps:
+      - name: Auto Add Label
+        if: github.event.action == 'opened' || github.event.action == 'reopened' || github.event.action == 'synchronize'
+        uses: actions/github-script@v8
+        with:
+          script: |
+            github.rest.issues.addLabels({
+              issue_number: context.issue.number,
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              labels: ['Waiting On Review']
+            })
+
+      - name: Auto Remove Label
+        if: github.event.action == 'closed'
+        uses: actions/github-script@v8
+        continue-on-error: true
+        with:
+          script: |
+            try {
+              await github.rest.issues.removeLabel({
+                issue_number: context.issue.number,
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                name: 'Waiting On Review'
+              });
+            } catch (error) {
+              console.log('Label "Waiting On Review" not found or could not be removed.');
+            }
+
+      - name: Auto Assign Milestone
+        if: github.event.action == 'opened'
+        uses: actions/github-script@v8
+        with:
+          script: |
+            // Fetch open milestones and assign the closest one
+            const { data: milestones } = await github.rest.issues.listMilestones({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              state: 'open',
+              sort: 'due_on',
+              direction: 'asc'
+            });
+
+            if (milestones.length > 0) {
+              const latestMilestone = milestones[0];
+              await github.rest.issues.update({
+                issue_number: context.issue.number,
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                milestone: latestMilestone.number
+              });
+            }

benches/Cargo.toml

@@ -5,7 +5,7 @@ edition = "2024"
 publish = false
 
 [dependencies]
-boa_engine.workspace = true
+boa_engine = { workspace = true, features = ["intl_bundled"] }
 boa_runtime.workspace = true
 
 [dev-dependencies]

core/ast/src/expression/call.rs

@@ -182,6 +182,32 @@ impl VisitWith for SuperCall {
     }
 }
 
+/// The phase of a dynamic import call.
+///
+/// Determines how the imported module is handled:
+/// - `Evaluation` (default): `import(specifier)` — loads, links, and evaluates the module.
+/// - `Defer`: `import.defer(specifier)` — deferred evaluation of the module.
+/// - `Source`: `import.source(specifier)` — source phase import.
+///
+/// More information:
+///  - [import-defer proposal][defer]
+///  - [source-phase-imports proposal][source]
+///
+/// [defer]: https://github.com/tc39/proposal-defer-import-eval
+/// [source]: https://github.com/tc39/proposal-source-phase-imports
+#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
+#[cfg_attr(feature = "arbitrary", derive(arbitrary::Arbitrary))]
+#[derive(Clone, Copy, Debug, PartialEq, Eq, Default)]
+pub enum ImportPhase {
+    /// `import(specifier)` — standard dynamic import.
+    #[default]
+    Evaluation,
+    /// `import.defer(specifier)` — deferred import evaluation.
+    Defer,
+    /// `import.source(specifier)` — source phase import.
+    Source,
+}
+
 /// The `import()` syntax, commonly called dynamic import, is a function-like expression that allows
 /// loading an ECMAScript module asynchronously and dynamically into a potentially non-module
 /// environment.
@@ -198,20 +224,22 @@ impl VisitWith for SuperCall {
 pub struct ImportCall {
     specifier: Box<Expression>,
     options: Option<Box<Expression>>,
+    phase: ImportPhase,
     span: Span,
 }
 
 impl ImportCall {
     /// Creates a new `ImportCall` AST node.
     #[inline]
     #[must_use]
-    pub fn new<S>(specifier: S, options: Option<Expression>, span: Span) -> Self
+    pub fn new<S>(specifier: S, options: Option<Expression>, phase: ImportPhase, span: Span) -> Self
     where
         S: Into<Expression>,
     {
         Self {
             specifier: Box::new(specifier.into()),
             options: options.map(Box::new),
+            phase,
             span,
         }
     }
@@ -235,6 +263,13 @@ impl ImportCall {
         self.options.as_deref()
     }
 
+    /// Returns the phase of this import call.
+    #[inline]
+    #[must_use]
+    pub const fn phase(&self) -> ImportPhase {
+        self.phase
+    }
+
     /// Gets the module specifier of the import call.
     ///
     /// This is an alias for [`Self::specifier`] for backwards compatibility.
@@ -256,14 +291,24 @@ impl Spanned for ImportCall {
 impl ToInternedString for ImportCall {
     #[inline]
     fn to_interned_string(&self, interner: &Interner) -> String {
+        let phase_str = match self.phase {
+            ImportPhase::Evaluation => "",
+            ImportPhase::Defer => ".defer",
+            ImportPhase::Source => ".source",
+        };
         if let Some(options) = &self.options {
             format!(
-                "import({}, {})",
+                "import{}({}, {})",
+                phase_str,
                 self.specifier.to_interned_string(interner),
                 options.to_interned_string(interner)
             )
         } else {
-            format!("import({})", self.specifier.to_interned_string(interner))
+            format!(
+                "import{}({})",
+                phase_str,
+                self.specifier.to_interned_string(interner)
+            )
         }
     }
 }

core/ast/src/expression/mod.rs

@@ -45,7 +45,7 @@ use crate::{
     visitor::{VisitWith, Visitor, VisitorMut},
 };
 pub use r#await::Await;
-pub use call::{Call, ImportCall, SuperCall};
+pub use call::{Call, ImportCall, ImportPhase, SuperCall};
 pub use identifier::{Identifier, RESERVED_IDENTIFIERS_STRICT};
 pub use import_meta::ImportMeta;
 pub use new::New;