Support inline script (#229)

* Support inline script

* Fix lints

Author: piercefreeman

mountaineer/__tests__/test_render.py

@@ -88,6 +88,34 @@
                 '<script src="/script4.js" test-attr="test-value"></script>',
             ],
         ),
+        # Inline script tags
+        (
+            Metadata(
+                scripts=[
+                    ScriptAttribute(
+                        inline="console.log('Hello, world!');",
+                    ),
+                    ScriptAttribute(
+                        inline="alert('Test');",
+                        asynchronous=True,
+                    ),
+                    ScriptAttribute(
+                        inline="window.onload = function() {};",
+                        defer=True,
+                    ),
+                    ScriptAttribute(
+                        inline="var x = 1;",
+                        optional_attributes={"type": "text/javascript"},
+                    ),
+                ],
+            ),
+            [
+                "<script>console.log('Hello, world!');</script>",
+                "<script async>alert('Test');</script>",
+                "<script defer>window.onload = function() {};</script>",
+                '<script type="text/javascript">var x = 1;</script>',
+            ],
+        ),
     ],
 )
 def test_build_header(metadata: Metadata, expected_tags: list[str]):
@@ -332,6 +360,18 @@ def test_merge_metadatas(metadatas: list[Metadata], expected_metadata: Metadata)
     assert metadata == expected_metadata
 
 
+def test_script_attribute_requires_src_or_inline():
+    """Test that ScriptAttribute requires either src or inline."""
+    with pytest.raises(ValueError, match="Either 'src' or 'inline' must be provided"):
+        ScriptAttribute()
+
+
+def test_script_attribute_cannot_have_both_src_and_inline():
+    """Test that ScriptAttribute cannot have both src and inline."""
+    with pytest.raises(ValueError, match="Cannot specify both 'src' and 'inline'"):
+        ScriptAttribute(src="/script.js", inline="console.log('test');")
+
+
 @pytest.mark.parametrize(
     "initial_url,new_sha,expected_url",
     [

mountaineer/render.py

@@ -213,13 +213,30 @@ class ScriptAttribute(HashableAttribute, BaseModel):
     )
     ```
 
+    Or with inline content:
+
+    ```python {{ sticky: True }}
+    ScriptAttribute(
+        inline="console.log('Hello, world!');",
+    )
+    ```
+
     """
 
-    src: str
+    src: str | None = None
+    inline: str | None = None
     asynchronous: bool = False
     defer: bool = False
     optional_attributes: dict[str, str] = {}
 
+    @model_validator(mode="after")
+    def validate_src_or_inline(self):
+        if self.src is None and self.inline is None:
+            raise ValueError("Either 'src' or 'inline' must be provided")
+        if self.src is not None and self.inline is not None:
+            raise ValueError("Cannot specify both 'src' and 'inline'")
+        return self
+
 
 class Metadata(BaseModel):
     """
@@ -332,13 +349,33 @@ def format_optional_keys(payload: Mapping[str, str | bool | None]) -> str:
             tags.append(f"<meta {format_optional_keys(meta_attributes)} />")
 
         for script_definition in self.scripts:
-            script_attributes: dict[str, str | bool] = {
-                "src": script_definition.src,
-                "async": script_definition.asynchronous,
-                "defer": script_definition.defer,
-                **script_definition.optional_attributes,
-            }
-            tags.append(f"<script {format_optional_keys(script_attributes)}></script>")
+            if script_definition.inline is not None:
+                # Inline scripts render the content directly inside the tag
+                script_attributes: dict[str, str | bool] = {
+                    "async": script_definition.asynchronous,
+                    "defer": script_definition.defer,
+                    **script_definition.optional_attributes,
+                }
+                formatted_attrs = format_optional_keys(script_attributes)
+                if formatted_attrs:
+                    tags.append(
+                        f"<script {formatted_attrs}>{script_definition.inline}</script>"
+                    )
+                else:
+                    tags.append(f"<script>{script_definition.inline}</script>")
+            else:
+                # External scripts use the src attribute
+                # src is guaranteed to be non-None here due to the model validator
+                assert script_definition.src is not None
+                script_attributes = {
+                    "src": script_definition.src,
+                    "async": script_definition.asynchronous,
+                    "defer": script_definition.defer,
+                    **script_definition.optional_attributes,
+                }
+                tags.append(
+                    f"<script {format_optional_keys(script_attributes)}></script>"
+                )
 
         for link_definition in self.links:
             if build_metadata and link_definition.add_static_sha:

src/bundle_common.rs

@@ -219,7 +219,8 @@ pub fn bundle_common(
     debug!("Resolve: {:?}", resolve);
     debug!("Bundle mode: {:?}", mode);
 
-    let inline_dynamic_imports = matches!(mode, BundleMode::SingleClient | BundleMode::SingleServer);
+    let inline_dynamic_imports =
+        matches!(mode, BundleMode::SingleClient | BundleMode::SingleServer);
 
     // https://github.com/rolldown/rolldown/blob/cb5e05c8d9683fd5c190daaad939e5364d7060b2/crates/rolldown_common/src/inner_bundler_options/mod.rs#L41
     let bundler_options = BundlerOptions {
@@ -618,8 +619,7 @@ mod tests {
             }
         "#;
 
-        create_test_js_file(temp_path, "lazy.js", lazy_js)
-            .expect("Failed to create lazy.js file");
+        create_test_js_file(temp_path, "lazy.js", lazy_js).expect("Failed to create lazy.js file");
 
         // Create a main entry file that imports both modules
         let entry_js = r#"
@@ -765,9 +765,7 @@ mod tests {
 
         let css_path = fake_package.join("styles.css");
         let mut css_file = File::create(&css_path).unwrap();
-        css_file
-            .write_all(b".fake-lib { display: flex; }")
-            .unwrap();
+        css_file.write_all(b".fake-lib { display: flex; }").unwrap();
 
         // Create JS that imports the node_modules CSS
         let server_js = r#"