From 26771d40fef9db268b4d3edab7224394797d73c0 Mon Sep 17 00:00:00 2001
From: Felix Angelov <felangelov@gmail.com>
Date: Sat, 1 Nov 2025 16:04:40 -0500
Subject: [PATCH] feat(dart_frog)!: serve `index.html` by default

---
 .../lib/src/create_static_file_handler.dart   |  2 +-
 .../src/create_static_file_handler_test.dart  | 67 +++++++++++++++++--
 2 files changed, 62 insertions(+), 7 deletions(-)

diff --git a/packages/dart_frog/lib/src/create_static_file_handler.dart b/packages/dart_frog/lib/src/create_static_file_handler.dart
index 8538d9074..349787468 100644
--- a/packages/dart_frog/lib/src/create_static_file_handler.dart
+++ b/packages/dart_frog/lib/src/create_static_file_handler.dart
@@ -5,7 +5,7 @@ import 'package:shelf_static/shelf_static.dart';
 /// Defaults to the `public` directory.
 Handler createStaticFileHandler({
   String path = 'public',
-  String? defaultDocument,
+  String? defaultDocument = 'index.html',
 }) {
   return fromShelfHandler(
     createStaticHandler(path, defaultDocument: defaultDocument),
diff --git a/packages/dart_frog/test/src/create_static_file_handler_test.dart b/packages/dart_frog/test/src/create_static_file_handler_test.dart
index ad6ea063c..6628ee6e1 100644
--- a/packages/dart_frog/test/src/create_static_file_handler_test.dart
+++ b/packages/dart_frog/test/src/create_static_file_handler_test.dart
@@ -56,14 +56,69 @@ void main() {
       tempDir.delete(recursive: true).ignore();
     });
 
-    test('serves default document', () async {
+    test('serves index.html by default', () async {
       const port = 3003;
       final tempDir = Directory.systemTemp.createTempSync();
 
-      const messageContents = 'hello world';
+      const contents = '<h1>Hello World</h1>';
+      File(
+        path.join(tempDir.path, 'index.html'),
+      ).writeAsStringSync(contents);
+
+      final server = await serve(
+        createStaticFileHandler(path: tempDir.path),
+        'localhost',
+        port,
+      );
+
+      final response = await http.get(Uri.parse('http://localhost:$port/'));
+      expect(response.statusCode, equals(HttpStatus.ok));
+      expect(response.body, equals(contents));
+
+      await server.close();
+      tempDir.delete(recursive: true).ignore();
+    });
+
+    test(
+        'does not serve index.html when default document '
+        'is explicitly set to null', () async {
+      const port = 3003;
+      final tempDir = Directory.systemTemp.createTempSync();
+
+      const contents = '<h1>Hello World</h1>';
+      File(
+        path.join(tempDir.path, 'index.html'),
+      ).writeAsStringSync(contents);
+
+      final server = await serve(
+        createStaticFileHandler(path: tempDir.path, defaultDocument: null),
+        'localhost',
+        port,
+      );
+
+      final indexHtmlResponse = await http.get(
+        Uri.parse('http://localhost:$port/index.html'),
+      );
+      expect(indexHtmlResponse.statusCode, equals(HttpStatus.ok));
+      expect(indexHtmlResponse.body, equals(contents));
+
+      final rootResponse = await http.get(
+        Uri.parse('http://localhost:$port/'),
+      );
+      expect(rootResponse.statusCode, equals(HttpStatus.notFound));
+
+      await server.close();
+      tempDir.delete(recursive: true).ignore();
+    });
+
+    test('serves custom default document', () async {
+      const port = 3003;
+      final tempDir = Directory.systemTemp.createTempSync();
+
+      const contents = 'hello world';
       File(
         path.join(tempDir.path, 'message.txt'),
-      ).writeAsStringSync(messageContents);
+      ).writeAsStringSync(contents);
 
       final server = await serve(
         createStaticFileHandler(
@@ -74,11 +129,11 @@ void main() {
         port,
       );
 
-      final messageResponse = await http.get(
+      final response = await http.get(
         Uri.parse('http://localhost:$port/'),
       );
-      expect(messageResponse.statusCode, equals(HttpStatus.ok));
-      expect(messageResponse.body, equals(messageContents));
+      expect(response.statusCode, equals(HttpStatus.ok));
+      expect(response.body, equals(contents));
 
       await server.close();
       tempDir.delete(recursive: true).ignore();