Allow contributes: formats:

and internally use `formats` rather than `format`

Author: dragonstyle

src/command/list/cmd.ts

@@ -77,20 +77,20 @@ async function outputExtensions(
         extension.contributes.filters &&
         extension.contributes.filters?.length > 0
       ) {
-        contribs.push("filter");
+        contribs.push("filters");
       }
 
       if (
         extension.contributes.shortcodes &&
         extension.contributes.shortcodes?.length > 0
       ) {
-        contribs.push("shortcode");
+        contribs.push("shortcodes");
       }
 
       if (
-        extension.contributes.format
+        extension.contributes.formats
       ) {
-        contribs.push("format");
+        contribs.push("formats");
       }
       return contribs.join(", ");
     };

src/command/render/render-contexts.ts

@@ -44,7 +44,6 @@ import {
   kIncludeBeforeBody,
   kIncludeInHeader,
   kIpynbFilters,
-  kLatexInputPaths,
   kMetadataFormat,
   kOutputExt,
   kOutputFile,
@@ -573,7 +572,7 @@ const readExtensionFormat = async (
     );
 
     // Read the yaml file and resolve / bucketize
-    const extensionFormat = extension?.contributes.format;
+    const extensionFormat = extension?.contributes.formats;
     if (extensionFormat) {
       const extensionMetadata =
         (extensionFormat[formatDesc.baseFormat] || {}) as Metadata;

src/extension/extension-shared.ts

@@ -25,7 +25,7 @@ export interface Extension extends Record<string, unknown> {
   contributes: {
     shortcodes?: string[];
     filters?: QuartoFilter[];
-    format?: Record<string, unknown>;
+    formats?: Record<string, unknown>;
   };
 }
 
@@ -42,7 +42,7 @@ export interface ExtensionContext {
   find(
     name: string,
     input: string,
-    contributes?: "shortcodes" | "filters" | "format",
+    contributes?: "shortcodes" | "filters" | "formats",
     project?: ProjectContext,
   ): Extension[];
 }

src/extension/extension.ts

@@ -63,7 +63,7 @@ export function createExtensionContext(): ExtensionContext {
   const find = (
     name: string,
     input: string,
-    contributes?: "shortcodes" | "filters" | "format",
+    contributes?: "shortcodes" | "filters" | "formats",
     project?: ProjectContext,
   ): Extension[] => {
     const extId = toExtensionId(name);
@@ -145,15 +145,15 @@ const loadExtension = (
 function findExtensions(
   extensions: Extension[],
   extensionId: ExtensionId,
-  contributes?: "shortcodes" | "filters" | "format",
+  contributes?: "shortcodes" | "filters" | "formats",
 ) {
   // Filter the extension based upon what they contribute
   const exts = extensions.filter((ext) => {
     if (contributes === "shortcodes" && ext.contributes.shortcodes) {
       return true;
     } else if (contributes === "filters" && ext.contributes.filters) {
       return true;
-    } else if (contributes === "format" && ext.contributes.format) {
+    } else if (contributes === "formats" && ext.contributes.formats) {
       return true;
     } else {
       return contributes === undefined;
@@ -352,7 +352,7 @@ function validateExtension(extension: Extension) {
   const contribs = [
     extension.contributes.filters,
     extension.contributes.shortcodes,
-    extension.contributes.format,
+    extension.contributes.formats,
   ];
   contribs.forEach((contrib) => {
     if (contrib) {
@@ -409,19 +409,20 @@ function readExtension(
       return resolveFilter(embeddedExtensions, extensionDir, filter);
     },
   );
-  const format = contributes?.format as Metadata || {};
+  const formats = contributes?.formats as Metadata ||
+    contributes?.format as Metadata || {};
 
   // Process the special 'common' key by merging it
   // into any key that isn't 'common' and then removing it
-  Object.keys(format).filter((key) => {
+  Object.keys(formats).filter((key) => {
     return key !== kCommon;
   }).forEach((key) => {
-    format[key] = mergeConfigs(
-      format[kCommon] || {},
-      format[key],
+    formats[key] = mergeConfigs(
+      formats[kCommon] || {},
+      formats[key],
     );
   });
-  delete format[kCommon];
+  delete formats[kCommon];
 
   // Create the extension data structure
   return {
@@ -433,7 +434,7 @@ function readExtension(
     contributes: {
       shortcodes: shortcodes.map((code) => join(extensionDir, code)),
       filters,
-      format,
+      formats,
     },
   };
 }

src/extension/install.ts

@@ -115,7 +115,7 @@ async function determineInstallDir(
     // use its path as the target for installation
     const extension = context.extension(extensionName, dir);
     if (extension) {
-      if (Object.keys(extension?.contributes.format || {}).length > 0) {
+      if (Object.keys(extension?.contributes.formats || {}).length > 0) {
         return extension?.path;
       } else {
         throw new Error(
@@ -324,10 +324,10 @@ async function confirmInstallation(
     const contributes = to.contributes;
     const extTypes: string[] = [];
     if (
-      contributes.format &&
-      Object.keys(contributes.format).length > 0
+      contributes.formats &&
+      Object.keys(contributes.formats).length > 0
     ) {
-      Object.keys(contributes.format).length === 1
+      Object.keys(contributes.formats).length === 1
         ? extTypes.push("format")
         : extTypes.push("formats");
     }

tests/docs/extensions/format/academic/_extensions/quarto-journals/elsevier/_extension.yml

@@ -1,6 +1,6 @@
 name: Elsevier Journal Format
 contributes:
-  format:
+  formats:
     common:
       date-format: full
     pdf: