fix(lambda-at-edge): don't include JSON static prop files in lambda a…

…rtefact

packages/lambda-at-edge/src/build.ts

@@ -124,10 +124,16 @@ class Builder {
  join(this.nextConfigDir, ".next/serverless/pages"),
  join(this.outputDir, DEFAULT_LAMBDA_CODE_DIR, "pages"),
  {
- // skip api pages from default lambda code
- filter: file => {
- const isHTMLPage = path.extname(file) === ".html";
- return pathToPosix(file).indexOf("pages/api") === -1 && !isHTMLPage;
+ filter: (file: string) => {
+ const isNotPrerenderedHTMLPage = path.extname(file) !== ".html";
+ const isNotStaticPropsJSONFile = path.extname(file) !== ".json";
+ const isNotApiPage = pathToPosix(file).indexOf("pages/api") === -1;
+
+ return (
+ isNotApiPage &&
+ isNotPrerenderedHTMLPage &&
+ isNotStaticPropsJSONFile
+ );
  }
  }
  )

packages/lambda-at-edge/tests/build.test.ts

@@ -152,7 +152,7 @@ describe("Builder Tests", () => {
 
  describe("Default Handler Artefact Files", () => {
  it("copies build files", async () => {
- expect.assertions(5);
+ expect.assertions(7);
 
  const files = await fse.readdir(
  join(outputDir, `${DEFAULT_LAMBDA_CODE_DIR}`)
@@ -185,7 +185,11 @@ describe("Builder Tests", () => {
  // api pages should not be included in the default lambda
  expect(apiDirExists).toEqual(false);
 
- // html pages should not be included in the default lambda
+ // HTML Prerendered pages or JSON static props files
+ // should not be included in the default lambda
+ expect(pages).not.toContain(["blog.json"]);
+ expect(pages).not.toContain(["about.html", "terms.html"]);
+
  expect(pages).toEqual(["_error.js", "blog.js", "customers"]);
  expect(customerPages).toEqual(["[...catchAll].js", "[post].js"]);
  });