fix: Fix not to include publication.json itself in its resources prop…

…erty fix #523
vivliostyle · Oct 6, 2024 · 0c55356 · 0c55356
1 parent ea882d1
commit 0c55356
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/output/webbook.ts b/src/output/webbook.ts
@@ -522,7 +522,11 @@ export async function copyWebPublicationAssets({
  manifest.resources = [
  ...[manifest.resources || []].flat(),
  ...resources.flatMap((file) => {
- if (preDefinedResources.includes(file)) {
+ if (
+ preDefinedResources.includes(file) ||
+ // Omit publication.json itself
+ pathEquals(file, upath.relative(outputDir, actualManifestPath))
+ ) {
  return [];
  }
  return file;

diff --git a/tests/__snapshots__/webbook.test.ts.snap b/tests/__snapshots__/webbook.test.ts.snap
@@ -248,9 +248,7 @@ Object {
  "url": "foo.html",
  },
  ],
- "resources": Array [
- "publication.json",
- ],
+ "resources": Array [],
  "type": "Book",
 }
 `;
@@ -349,7 +347,6 @@ Object {
  "rel": "cover",
  "url": "cover.png",
  },
- "publication.json",
  "style%20sheet.css",
  "themes/packages/mytheme/%25style%25.css",
  ],