code review fixup

.gitignore

@@ -1,4 +1,4 @@
 node_modules/
 .wrangler/
 dist/
-.dev.vars
+.dev.vars

src/constants/limits.ts

@@ -0,0 +1,4 @@
+/**
+ * Max amount of retries for S3 requests
+ */
+export const S3_RETRY_LIMIT = 3;

src/handlers/strategies/directoryListing.ts

@@ -12,6 +12,7 @@ import { getFile } from './serveFile';
 
 // Imports the Precompiled Handlebars Template
 import htmlTemplate from '../../templates/directoryListing.out.js';
+import { S3_RETRY_LIMIT } from '../../constants/limits';
 
 // Applies the Template into a Handlebars Template Function
 const handleBarsTemplate = Handlebars.template(htmlTemplate);
@@ -120,7 +121,7 @@ async function fetchR2Result(
 ): Promise<ListObjectsV2CommandOutput> {
   let result: ListObjectsV2CommandOutput | undefined = undefined;
 
-  let retriesRemaining = 3;
+  let retriesRemaining = S3_RETRY_LIMIT;
   while (retriesRemaining > 0) {
     try {
       // Send request to R2

tests/e2e/directory.test.ts

@@ -10,6 +10,11 @@ async function startS3Mock(): Promise<http.Server> {
     const url = new URL(req.url!, `http://${req.headers.host}`);
 
     let xmlFilePath = './tests/e2e/test-data/expected-s3/';
+
+    // Check if it's a path that's supposed to exist in
+    //  later tests. If so, return a S3 response indicating that
+    //  the path exists. Otherwise return a S3 response indicating
+    //  that the path doesn't exist
     if (
       ['nodejs/release/', 'nodejs/', 'metrics/'].includes(
         url.searchParams.get('prefix')!