Merge branch 'main' into feat-console-table

Author: ParthMozarkar

.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

@@ -20,7 +20,7 @@ jobs:
               issue_number: context.issue.number,
               owner: context.repo.owner,
               repo: context.repo.repo,
-              labels: ['waiting-for-review']
+              labels: ['Waiting On Review']
             })
 
       - name: Auto Remove Label
@@ -34,10 +34,10 @@ jobs:
                 issue_number: context.issue.number,
                 owner: context.repo.owner,
                 repo: context.repo.repo,
-                name: 'waiting-for-review'
+                name: 'Waiting On Review'
               });
             } catch (error) {
-              console.log('Label "waiting-for-review" not found or could not be removed.');
+              console.log('Label "Waiting On Review" not found or could not be removed.');
             }
 
       - name: Auto Assign Milestone

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;

core/engine/src/builtins/date/mod.rs

@@ -50,7 +50,7 @@ impl Date {
 
     /// Creates a new `Date` from the current UTC time of the host.
     pub(crate) fn utc_now(context: &mut Context) -> Self {
-        Self(context.clock().now().millis_since_epoch() as f64)
+        Self(context.clock().system_time_millis() as f64)
     }
 }
 
@@ -210,7 +210,7 @@ impl BuiltInConstructor for Date {
         // 1. If NewTarget is undefined, then
         if new_target.is_undefined() {
             // a. Let now be the time value (UTC) identifying the current time.
-            let now = context.clock().now().millis_since_epoch();
+            let now = context.clock().system_time_millis();
 
             // b. Return ToDateString(now).
             return Ok(JsValue::from(to_date_string_t(
@@ -328,7 +328,7 @@ impl Date {
     /// [mdn]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date/now
     #[allow(clippy::unnecessary_wraps)]
     pub(crate) fn now(_: &JsValue, _: &[JsValue], context: &mut Context) -> JsResult<JsValue> {
-        Ok(JsValue::new(context.clock().now().millis_since_epoch()))
+        Ok(JsValue::new(context.clock().system_time_millis()))
     }
 
     /// `Date.parse()`